text:
Migrate to Github Actions

Resolves #484, #486
Migrates CI to GitHub actions.
TODO:

[ ] Update .scripts/update-gh-pages to work with github actions

Codecov Report

Merging #486 (b02f681) into master (95b81b7) will decrease coverage by 98.87%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #486       +/-   ##
==========================================
- Coverage   98.87%   0.00%   -98.88%
==========================================
  Files           6       6
  Lines         177     175        -2
==========================================
- Hits          175       0      -175
- Misses          2     175      +173

Impacted Files
Coverage Δ

ReSwift/CoreTypes/Store.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

ReSwift/Utils/TypeHelper.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

ReSwift/Utils/Synchronized.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

ReSwift/CoreTypes/Subscription.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

ReSwift/CoreTypes/StoreSubscriber.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

ReSwift/Utils/Assertions.swift
0.00% <0.00%> (-71.43%)
:arrow_down:

Continue to review full report at Codecov.

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

@DivineDominion The update docs script appears to be heavily dependent on travis-ci environment variables. Haven't looked closely but I'm assuming it will need modifications. Will take a look today.

Codecov Report

Merging #486 (87cf01d) into master (95b81b7) will increase coverage by 0.09%.
The diff coverage is n/a.

:exclamation: Current head 87cf01d differs from pull request most recent head 5a17f3c. Consider uploading reports for the commit 5a17f3c to get more accurate results

@@            Coverage Diff             @@
##           master     #486      +/-   ##
==========================================
+ Coverage   98.87%   98.96%   +0.09%
==========================================
  Files           6        6
  Lines         177      193      +16
==========================================
+ Hits          175      191      +16
  Misses          2        2

Impacted Files
Coverage Δ

ReSwift/CoreTypes/Store.swift
100.00% <0.00%> (ø)

ReSwift/Utils/TypeHelper.swift
100.00% <0.00%> (ø)

ReSwift/CoreTypes/Subscription.swift
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

authors: codecov-commenter mjarvis


text:
version is not displayed

Screenshots of microG Settings' UI:

That's not an issue caused by ReVanced. Looks like your device

My device doesn't do anything specially to block microG to display its version. There must be something wrong in the code. 🤷♂

If you think so report the issue to upstream.

Forgot to say, that is screenshots of microG setting UI, not screenshots of my device's App manager.

I understand the issue now. This may be caused by the change in the version for the fork

I also want to suggest that the About page, or somewhere, should indicate that this is the Revanced fork of microG.

Resolve my request

no log

source: gharchive/issue

authors: Amyziya Myint12thu12 oSumAtrIX trmdi

repo: ReVanced/GmsCore

license_type: permissive

license_source: github-api


text:
URL pattern with undefined optional parameter in

"version": "4.0.0-alpha.5"

Documentation says that for 'pattern' property in  we can use 'any valid URL pattern that path-to-regexp understands'. So in particular (according to path-to-regexp docs) we can define optional params, something like:
<Match pattern='/foo/:bar?' component={MyComponent} />
and then in MyComponent we can access this.props.params.bar
In the case when this optional component in URI is not supplied at all (i.e. when path=/foo) I would expect params.bar to be undefined. But instead I get string 'undefined' - like in the case when path=/foo/undefined
It seems that it's because in this case:

path-to-regexp exec method returns array with undefined as element
in matchPattern.js (line 37) we use decodeURIComponent() function that returns string even if it's argument is undefined. So maybe in this line it would be better to replace
params[keys[index].name] = decodeURIComponent(value);
with
params[keys[index].name] = value && decodeURIComponent(value);

Or it's meant to work this way - and there's no reason to make changes?

Fixed in #4064, just not released yet.

authors: iamciroja pshrmn

repo: ReactTraining/react-router

license: mit

license_source: bigquery


text:
react-router-redux throws warnings with react 15.5

Version
react-router 3.0.3
react-router-redux 4.0.8
Issue
The changes in react 15.5 regarding PropTypes and createClass make react-router-redux throw two warnings.
Warning: Accessing PropTypes via the main React package is deprecated. Use the prop-types package from npm instead.

Warning: RouterContext: React.createClass is deprecated and will be removed in version 16. Use plain JavaScript classes instead. If you're not yet ready to migrate, create-react-class is available on npm as a drop-in replacement.

Will this issue be solved or is it something that will be addressed when react-router-redux 5 is released with support for react-router 4?

Sorry to nit, but these warnings do not stem from react-router-redux (current version living here), they are all from the v3 code base of react-router.

Should be fixed on master. I'll see about getting a release out soon.

@timdorr this issue is regarding RR3 (which also uses the now deprecated createClass call).
If you accept PR's for the v3 branch I'd be happy to contribute.

Yep, we do. I will take care of it now, though. It's not that hard to go through, luckily.

@simonkberg I noticed that the error was due to the react-router version that's why I mentioned it in the report 😉
@timdorr thanks for the speedy response and fix 👍

@timdorr I met the same warning with react-router@2.8.1, so will you update 2.x version?

No, we're not maintaining the 2.x versions. Upgrade to 3.x, it's just 2.x without the deprecation warnings.

modules.js?hash=1a6be65…:10730Warning: ReactTestUtils has been moved to react-dom/test-utils. Update references to remove this warning.printWarning @ modules.js?hash=1a6be65…:10730warning @ modules.js?hash=1a6be65…:10754meteorInstall.node_modules.react-addons-test-utils.index.js @ modules.js?hash=1a6be65…:31950fileEvaluate @ modules-runtime.js?hash=2b888cb…:207require @ modules-runtime.js?hash=2b888cb…:130meteorInstall.node_modules.meteor.react-runtime.react-runtime.js @ react-runtime.js:38fileEvaluate @ modules-runtime.js?hash=2b888cb…:207require @ modules-runtime.js?hash=2b888cb…:130(anonymous function) @ react-runtime.js?hash=be87be9…:115(anonymous function) @ react-runtime.js?hash=be87be9…:127
modules.js?hash=1a6be65…:10730Warning: HomeLayout: React.createClass is deprecated and will be removed in version 16. Use plain JavaScript classes instead. If you're not yet ready to migrate, create-react-class is available on npm as a drop-in replacement.printWarning @ modules.js?hash=1a6be65…:10730warning @ modules.js?hash=1a6be65…:10754ReactClass.createClass @ modules.js?hash=1a6be65…:8480meteorInstall.client.components.layouts.home.jsx @ home.jsx:1fileEvaluate @ modules-runtime.js?hash=2b888cb…:207require @ modules-runtime.js?hash=2b888cb…:130(anonymous function) @ app.js?hash=22e6ff3…:124
kadira_react-layout.js?hash=c418396…:76 Uncaught TypeError: React.render is not a function(anonymous function) @ kadira_react-layout.js?hash=c418396…:76(anonymous function) @ kadira_react-layout.js?hash=c418396…:90
```Just start using React today and getting this error.

Is this change still waiting to get into the v3 branch?

@FoxxMD the fixes has been released https://github.com/ReactTraining/react-router/releases/tag/v3.0.4

@simonkberg  thanks! I didn't see it on the release page but I didn't scroll down far enough :)

@simonkberg I upgraded to 3.0.4 and am still seeing these warnings.

@kevinguard are you sure they're coming from RR? could you post the full stack trace?

I am also still seeing these warnings

I just upgraded to 3.0.5, seeing the warning but they are from a different package. I guess lots of modules out there still not brought into compliance

authors: Fabiantjoeaon FoxxMD dmk23 ken4ward kevinguard patresi pinggod simonkberg timdorr


text:
Request: withRouter for v6

Reading through the v6 migration guide, it seems that the new API is intended to be hooks-only. However, you cannot use hooks inside class components. This would mean that in order to upgrade to v6, I would need to convert all my class components that use routing into functional components even though react has no intention of deprecating class components. I believe a new version of the withRouter HOC should be supported in order to enable use of class components.
Note: I don't need support older versions of React, so withRouter can use hooks under the hood. I believe this makes the implementation pretty trivial:
function withRouter (Child) {
     return function withRouter (props) {
          const location = useLocation();
          const history = useHistory();
          // other relevant props
          ...
          return <Child {...props} history={history} location={location} />;
    }
}

I could of course write this myself, but I think it makes sense for this to be in react-router core.

Because there are several different hooks and a non-trivial cost to using them all, having a big single mega-HOC wouldn't be the right approach. But given that writing a small wrapper component that uses the right hook and passes it as props, it's best that this is left to the user to implement.

authors: nicoburns timdorr


text:
[V6] useSearchParams returns an empty object

Version
Version 6.0.0-alpha.4
Test Case
https://codesandbox.io/s/react-router-1qjhd?file=/index.js
Steps to reproduce
just use the useSearchParams hook
Expected Behavior
return an object of searchParams at the first element of the returned list
Actual Behavior
the searchParams object is always an empty object

It is a URLSearchParams object. You need to use its API to access the data inside: https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams

authors: john-pc timdorr


text:
‘Cannot create __weak reference in file using manual reference counting’

Pod load ReactiveObjC compile errors：
Cannot create __weak reference in file using manual reference counting
To solve：
Set Weak References in Manual Retain Release:YES

yes,I have also encountered this problem, I am introducing class (MRC).

repo: ReactiveCocoa/ReactiveObjC

license: MIT


text:
Tck parser

This is just the parser utility classes for the tck driver

should we pull gflags as submodule? It seems that travis has issues compiling the sources because it is using old version of gflags: https://github.com/BVLC/caffe/issues/2597

source: gharchive/pull-request

repo: ReactiveSocket/reactivesocket-cpp


text:
chore(micro-perf): fix scheduling of retry perf test

Fix Observable.of() usage in micro perf test with immediate scheduler
Adds micro perf test using currentThread scheduler

change looks good to me.

@blesh done.

Merged with f275182, thanks @luisgabriel .

authors: kwonoj luisgabriel

repo: ReactiveX/RxJS


text:
tap operator incorrectly infers element type sometimes

Bug Report
When inferring element type as the result of the pipe, last tap operator in pipe chain incorrectly infers element type from the expected type.
Reproduction
Since EffectMap expects each property to return Subscribable with any arbitrary element type, the last tap seems to pick up this any type instead of the element type from the input observable.
type EffectMap = { [key: string]: Subscribable<any> };
type EffectBuilder<T> = {
    [key: string]: (stream: Observable<T>) => EffectMap
}
function factory<T>(spec: EffectBuilder<T>) { return false }

const thing = factory<number>({
   foo: stream$ => ({ // stream$ is Observable<number>
       BAR: stream$.pipe(
           tap(value => console.log(value)), // value is any instead of number
           // tap() // Uncommenting this will fix the issue for the first tap, but not the last
       )
   })
})

Expected behavior
tap operator propagates strong typings for values
Environment

Runtime: Node v10.0.0
RxJS version: 6.3.2
Typescript version: 3.0.3

This appears to be a TypeScript issue/bug.
Firstly, the return type of tap is inferred correctly in this scenario:
declare const s: Observable<number>;
const r = s.pipe(tap(value => console.log(value))); // Observable<number>

Secondly, if map is used - in the issue snippet - the return type is inferred correctly.
The difference is that the return type of map's project function - R - is used in the operator's return type of OperatorFunction<T, R>. If the signatures of tap are messed with so that its next is replaced with a project function, the type is inferred correctly.
IMO, this is a TypeScript bug that's exposed by the more complicated inference scenario in the issue's snippet.

The problem is the use of Subscribable<any> in EffectMap. That sees the value within tap inferred as any.
The same problem can be seen here:
declare const s: Observable<number>;
const r: Observable<any> = s.pipe(
  tap(value => console.log(value)) // value is inferred to be any
);

The problem does not occur with map because its return type is inferred from the project function's return type. That is, there's no need for value to be inferred from what's on the left-hand side of the assignment.
And it does not happen if an additional tap is added, because it's only the value in the last operator that's inferred from what's on the left-hand side of the assignment.
I don't think there's anything that can be done in RxJS to address this.

Closing this, as I don't see that there is anything that can be done within RxJS to address the issue.

authors: Nachasic cartant

repo: ReactiveX/rxjs


text:
Order of invocation of Multiple Finalize callbacks

Bug Report
Current Behavior
Callbacks registered with finalizing operator are called in flipped order.
Reproduction
Link to codepen
const problem1 = pipe(
  from([1,2,3]),
  finalize(() => log("1")),
  map(x => x * 2),
  finalize(() => log("2"))
)

problem1.subscribe()

Current output
"2"
"1"

Expected output
This is what I expect. I would like to finalize previous things first before the things further down my chain.
"1"
"2"

Environment

Runtime: Node v9.4.0, Chrome v68.0.3440.106
RxJS version: 5+, 6+

The order in which the finalize operators are called depends upon several factors.
In your snippet, they are called in 'reverse' order because the source and the composed operators complete synchronously.
In this example, they are not called in reverse order because the source observable completes before the observable within the mergeMap:
import { asyncScheduler, from, of } from "rxjs";
import { finalize, mergeMap } from "rxjs/operators";

const problem1 = from([1,2,3]).pipe(
  finalize(() => console.log("1")),
  mergeMap(x => of(x * 2, asyncScheduler)),
  finalize(() => console.log("2"))
);
problem1.subscribe();

Essentially, this behaviour is be design.


text:
Honour the NewtonSoft JsonObject NamingStrategy

Fixes #227

Oooh, thanks @caitchison , let me have a look and I'll try to get this out asap!

Looks good, thank you for this!

It will be in the 2.0.0.8 release which is on it's way to Nuget as I type..

authors: caitchison cskardon

license: MS-PL


text:
Account OOO users in Update Script

Presently, if a user designates themselves as "Out of Office" (OOO) starting from today, they are unable to transition back to an "Idle" or "Active" status once the OOO period concludes. In the past, users had the ability to manually set themselves as "Active" or "Idle" directly on the site. However, due to the transition towards a task-based status update we now run a script periodically we need to check if OOO period is over then we need to change their status accordingly.

ETA :
2 days for code and test
3 days for review
1 day for dev to main sync
14 Aug 2023

repo: Real-Dev-Squad/website-backend


text:
feat: Include userData in Progresses API responses

Date: 21 Dec 2024

Developer Name: Anuj Chhikara

Issue Ticket Number

Closes #2294

Description

This PR introduces a change to include the full userData (e.g., username, profile picture) alongside the progress data when fetching progress updates. Currently, the backend only returns the userId, requiring the frontend to make an additional API call to fetch user details. By including userData directly with the progress object, we aim to reduce the number of API calls and improve frontend performance and simplicity.
Documentation Updated?

[ ] Yes
[x] No

Under Feature Flag

[x] Yes
[ ] No

Database Changes

[ ] Yes
[x] No

Breaking Changes

[ ] Yes
[x] No

Development Tested?

[x] Yes
[ ] No

Screenshots

Screenshot 1

Without dev flag

With dev flag

Test Coverage

Screenshot 1

Additional Notes

Design Doc

Please fix the title as commit message is not the title


text:
Dark mode

Closes #239

After changes
https://user-images.githubusercontent.com/46373689/140518101-f0ca01f9-195f-471b-8f64-ebd91dbf5b96.mp4

Made changes to use the cookie as a feature flag
https://user-images.githubusercontent.com/46373689/163666642-a4e1b843-4dd4-46b6-bd09-f2b687fef041.mp4

Hey @shubham-y Can you make suggested changes?

As we are changing the UI of the crypto site. We don't need these changes at the moment. So I am marking this as stale.

authors: MehulKChaudhari shubham-y

repo: Real-Dev-Squad/website-crypto


text:
🛑 Main webpage is down

In 2062e31, Main webpage (https://nattawatt.com) was down:

HTTP code: 509
Response time: 148 ms

Resolved: Main webpage is back up in 536bd23 after 17 hours, 14 minutes.


text:
Error: DatePickerTheme' isn't a function.

Today I am getting an error: 'DatePickerTheme' isn't a function.
Try correcting the name to match an existing function, or define a method or function named 'DatePickerTheme'.

Any solutions? We should wait for fix?

If my answer helped you can close the issue thanks @AKovynev

There is a conflict between the custom DatePickerTheme with flutter material package.
The solution is
import 'package:flutter/material.dart' hide DatePickerTheme;

It may seem irrelevant, but when I referenced the package from github, not pubdev, this problem was solved.

This didn't work out in my case, I still get the error.
I'm referencing the package as such :
  flutter_datetime_picker:
    git:
      url: https://github.com/Realank/flutter_datetime_picker.git
      ref: master

It may seem irrelevant, but when I referenced the package from github, not pubdev, this problem was solved.

This didn't work out in my case, I still get the error. I'm referencing the package as such :
  flutter_datetime_picker:
    git:
      url: https://github.com/Realank/flutter_datetime_picker.git
      ref: master

I fixed this earlier internally. All I did was to rename the DatePickerTheme being imported from 'package:flutter_datetime_picker/src/datetime_picker_theme.dart' as it's conflicting with Flutter's native DatePickerTheme function. Or you can refer to the fix on this PR that fixes the issue using aliases.

You can try this
Change to this and use dtpicker or any name you want
import 'package:flutter_datetime_picker/flutter_datetime_picker.dart' as dtpicker;

authors: AKovynev Anyitechs SerdarYildiz06 TimestampWombat ajayprakashshukla rs-huypq sarp86


text:
Web scraping using OCR software .

Problem - Data scrapped from the website would be insufficient and inaccurate with only usage of selenium and beautiful soap for scrapping.
solution- I propose the usage of OCR software called Tesseract , which would extract text from images , selenium has a feature where it can take couple of screenshots of website  ,this will be given to tesseract for extraction.
Please assign me  this issue

I am eager to make a meaningful contribution to this project. Kindly assign me this tasks as it aligns with my skills and expertise.

assigned to you @litesh1123
Issue will be assigned to Only 1 person ...FCFS basis.
Others get a chance if the issue gets stale & the assignee is inactive for 5+ days
Also if you would like to work on more issue create a new issue and I will create label and assign you.

@sanjay-kv  thank you sir

Please assign me this issue

repo: Recode-Hive/Scrape-ML


text:
Add Device Status column: Running, Booting, Unspecified

Right now show only three type of statuses: running, booting and unspecified.
The unspecified will be shown if rpm_ostree_deployment is missing.

Codecov Report

Merging #106 (024cde4) into master (3415306) will decrease coverage by 0.74%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
- Coverage   28.49%   27.75%   -0.75%
==========================================
  Files          69       70       +1
  Lines        1193     1207      +14
  Branches      315      317       +2
==========================================
- Hits          340      335       -5
- Misses        787      803      +16
- Partials       66       69       +3

Impacted Files
Coverage Δ

src/Routes/Devices/DeviceStatus.js
0.00% <0.00%> (ø)

src/Routes/Devices/Devices.js
0.00% <0.00%> (ø)

src/api/index.js
48.57% <0.00%> (-4.77%)
:arrow_down:

Continue to review full report at Codecov.

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

repo: RedHatInsights/edge-frontend


text:
CCXDEV-12772: S3UploadEngine compatible with KafkaConsumer

Description
Modifying the way of interaction between consumers and S3Uploadengine in order to make it compatible with any Consumer that wants to use it.
In addition, adding compatibility for both Internal Data Pipeline Consumer and KafkaConsumer.
Fixes #CCXDEV-12772
Type of change

Bug fix (non-breaking change which fixes an issue)
Refactor (refactoring code, removing useless files)

Testing steps
Tested only with unit tests.
Checklist

[x] pre-commit run -a passes
[ ] updated documentation wherever necessary
[x] added or modified tests if necessary
[x] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Attention: Patch coverage is 61.76471% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 84.32%. Comparing base (3c46b28) to head (78a4d88).

Files
Patch %
Lines

ccx_messaging/consumers/idp_kafka_consumer.py
0.00%
13 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #205      +/-   ##
==========================================
- Coverage   85.03%   84.32%   -0.72%
==========================================
  Files          34       34
  Lines        1116     1129      +13
==========================================
+ Hits          949      952       +3
- Misses        167      177      +10

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

authors: codecov-commenter joselsegura

repo: RedHatInsights/insights-ccx-messaging


text:
feat: use unknown for the string "type" when we have anonymous types

This prevent the user from comparing with type === "Response200" instead of status === 200 which is
better

Codecov Report

Merging #48 (939d731) into master (bb73fea) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #48   +/-   ##
=======================================
  Coverage   88.26%   88.26%
=======================================
  Files          59       59
  Lines         886      886
  Branches      165      165
=======================================
  Hits          782      782
  Misses        104      104

Continue to review full report at Codecov.

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

authors: codecov-io josejulio


text:
Releasing 1.0.0

Once #28 gets merged and we figure out #27, I think we're ready for first release.
Thoughts?

I agree with this

authors: Kinlaw ryelo


text:
Accomodate changes to cdappconfig DB cert info

Looks like cdappconfig loadedConfig.database.rdsCa now contains the actual certificate, not a path to a cert file...

/retest

repo: RedHatInsights/insights-remediations


text:
OAuth Proxy

Since changelog will be accessible publicly on devshift.net, it needs to have some authentication. Managing users is probably outside our scope, so using openshift auth proxy is the option we are going with.
All requests would go through this new container, rejecting unauthorized ones and proxying them to the api at localhost:3000.
Modeled after GABI proxy config
Jira: RHCLOUD-24496

I haven't had to interact with this before, how does this affect non-prod environments like stage/ephemeral?

I haven't had to interact with this before, how does this affect non-prod environments like stage/ephemeral?

For ephemeral, I'm thinking you could port forward to the api and skip the oauth-proxy entirely. For stage/prod (whatever those will be on devshift), those URL would go through the proxy.
Derek and I got it coming up with a openshift login page in ephemeral, but it didn't load the app, giving:
{"error":"server_error","error_description":"The authorization server encountered an unexpected condition that prevented it from fulfilling the request.","state":"f507fa1fcf7c42bf9683bb3233084a60:/"}
Baby steps :sweat_smile:

All changes have been moved to the UI (closer to where it is in the flow; hit oauth proxy, then nginx.

authors: justinorringer skarekrow

repo: RedHatInsights/platform-changelog-go


text:
POL-497 Override the Maven version used by Quarkus

@aloubyansky Could you please confirm this is correct?

Quarkus was bumped to 1.13.6.Final which includes a Maven upgrade (see 1.13.5.Final blog post) so we don't need to override the Maven version anymore.

repo: RedHatInsights/policies-engine


text:
Stop dumping the heap on OutOfMemoryError

We no longer need that.

08:45:00 ERROR: Couldn't find any revision to build. Verify the repository and branch configuration for this job.
Looks like this isn't our issue.

The build initially failed because of an ephemeral timeout which was caused by a global voluntary slow-down of the ephemeral deployments (for stability purposes).

I'm merging anyway, we don't need the PR check for this change. I'll update the ephemeral timeout later.

authors: bennetelli gwenneg

repo: RedHatInsights/policies-ui-backend


text:
Ref-RHIROS-1215 added link to group name column value

Add group link to column :boom:
https://issues.redhat.com/browse/RHIROS-1215
Why do we need this change? :thought_balloon:
It is a bug fix we need as part of card https://issues.redhat.com/browse/RHIROS-1215
Documentation requires update? :memo:

[ ] Yes
[ ] No

Security Checklist :lock:
Upon raising this PR please go through RedHatInsights/secure-coding-checklist
:guardsman: Checklist :dart:

[ ] Bugfix
[ ] New Feature
[ ] Refactor
[ ] Unittests Added
[ ] DRY code
[ ] Dependency Added

Additional :mega:
Feel free to add any other relevant details such as links, notes, screenshots, here.

Thank you for the review @saltgen 🙌🏻

repo: RedHatInsights/ros-frontend


text:
[RHCLOUD-16316] Deploy Availability Status Listener

https://issues.redhat.com/browse/RHCLOUD-16316
Adding the pod for the availability status listener to the deployment. Also did some tiny refactors on the listener for handling case-insensitive resource names.
I ended up setting the availability status listener min replicas to 0 so we can merge this - we can scale it up and turn off the ruby-side at the same time at a later date.
WIP as I need to test it yet and due to my internet being out I can't.

/retest

/retest

/retest

I would suggest rebasing https://github.com/RedHatInsights/sources-api-go/pull/84, as it seems that we've introduced another TestMain function which clashes with the previous one that already existed.

Can I merge this now ?

@lpichler yes - I have it scaled to zero by default here: https://github.com/RedHatInsights/sources-api-go/pull/63/files#diff-50b41779d45e37968dc1bef161feebad3447f6c3cf72cfee62db774ecd5dfa4dR168

authors: MikelAlejoBR lindgrenj6 lpichler

repo: RedHatInsights/sources-api-go


text:
Gems and API updates, Ingress API send fix

Updating deprecated gems
Changing Topological API to v3.0
adding TopologicalInventory::Providers::Common::SaveInventory::Saver - current code is not compatible

@slemrmartin could we move this over into the topological_inventory-sync repo?  We've been talking about doing it for a while but I think now would be a good time

@agrare yes, it's good idea, this service is doing too few.

@agrare please merge this bug fix first. It'll take a day or two until I move it and adapt to 2nd repo including specs and e2e-deploy

authors: agrare slemrmartin


text:
Change enable_virtual_services to install via Hyper-V instead of SCVMM

The scvmm powershell method had issues with RHEL images that the Hyper-V method does not.  Most google posts said this is the most reliable way to install guest services and Amogh and I tested it on a variety of OS types and didn't fail.  SCVMM 2016 fixes this issue but doesn't help us now.

Lint report for commit d494a021abc948e9946800246240ecbdf2fb6cdb:
Everything seems all right :smile: .
CFME QE Bot

Coverage decreased (-0.01%) to 27.429% when pulling d494a021abc948e9946800246240ecbdf2fb6cdb on jteehan:scvmm330 into 69a8fc3bf8a7280f7cf3a5e264ef89ccd746845f on RedHatQE:master.

authors: coveralls dajoRH jteehan

repo: RedHatQE/mgmtsystem


text:
add pyhelper-utils to renovate - 627

[Auto generated]
Number: [#627]

openshift-cli-installer(2gktn)[PR 627]: Closing issue for PR: add pyhelper-utils to renovate.
PR was closed.

repo: RedHatQE/openshift-cli-installer


text:
Parts of the game become invisible

Tested in
Minecraft: 1.15.2 and 20w14infinite
SleepBackground: 3.6
SpeedrunIGT: 11.1 and 10.17
Bug
Entities, particles, block breaking animations and the players hand become invisible
How to reproduce the issue
Open a Minecraft Instance with a Minecraft version before 1.16 with SleepBackground 3.6 and SpeedrunIGT, create a new world and unfocus the window during world creation

https://youtu.be/mJQvTbWdXqk
Minecraft: 1.15.2
SleepBackground: 3.6
SpeedrunIGT: 11.1
Fabric: 0.14.6
sleepbg config
speedrunigt config
minecraft options.txt
a log of when i performed the bug (not the session in the video)

repo: RedLime/SleepBackground


text:
fix(Diagnostic): use Format.pp_infinity in string_of_text for OCaml 5.2

See https://github.com/ocaml/ocaml/discussions/12986
Otherwise the code fails with:
#=== ERROR while compiling asai.0.3.0 =========================================#
# context              2.2.0~beta2~dev | linux/x86_64 | ocaml-variants.5.2.0+trunk | file:///home/opam/opam-repository
# path                 ~/.opam/5.2/.opam-switch/build/asai.0.3.0
# command              ~/.opam/5.2/bin/dune build -p asai -j 1 @install @runtest
# exit-code            1
# env-file             ~/.opam/log/asai-62-9d2d7d.env
# output-file          ~/.opam/log/asai-62-9d2d7d.out
### output ###
# (cd _build/default/test && ./TestUserContent.exe)
# Testing `UserContent'.
# This run has ID `9FIAG1G5'.
#
#   [OK]          replace_control          0   tab.
#   [OK]          replace_control          1   C1 control.
#   [OK]          replace_control          2   C2 control.
#   [OK]          replace_control          3   invalid UTF-8.
#   [OK]          replace_control          4   invalid UTF-8.
#
# Full test results in `~/.opam/5.2/.opam-switch/build/asai.0.3.0/_build/default/test/_build/_tests/UserContent'.
# Test Successful in 0.000s. 5 tests run.
# File "test/dune", line 7, characters 24-38:
# 7 |  (names TestUserContent TestDiagnostic)
#                             ^^^^^^^^^^^^^^
# (cd _build/default/test && ./TestDiagnostic.exe)
# Testing `UserContent'.
# This run has ID `P63ACD2O'.
#
# > [FAIL]        string_of_text          0   \n.
#
# ┌──────────────────────────────────────────────────────────────────────────────┐
# │ [FAIL]        string_of_text          0   \n.                                │
# └──────────────────────────────────────────────────────────────────────────────┘
# [invalid] Format.pp_set_geometry: margin >= pp_infinity
#           Raised at Stdlib__Format.pp_set_geometry in file "format.ml", line 844, characters 4-63
#           Called from Asai__Diagnostic.string_of_text in file "src/Diagnostic.ml", line 45, characters 11-71
#           Called from Dune__exe__TestDiagnostic.of_test.(fun) in file "test/TestDiagnostic.ml", line 4, characters 2-38
#           Called from Alcotest_engine__Core.Make.protect_test.(fun) in file "src/alcotest-engine/core.ml", line 181, characters 17-23
#           Called from Alcotest_engine__Monad.Identity.catch in file "src/alcotest-engine/monad.ml", line 24, characters 31-35
#
# Logs saved to `~/.opam/5.2/.opam-switch/build/asai.0.3.0/_build/default/test/_build/_tests/UserContent/string_of_text.000.output'.
#  ──────────────────────────────────────────────────────────────────────────────
#
# Full test results in `~/.opam/5.2/.opam-switch/build/asai.0.3.0/_build/default/test/_build/_tests/UserContent'.
# 1 failure! in 0.000s. 1 test run.
# (cd _build/default/test && ./TestExplicator.exe)
# Testing `Explicator'.
# This run has ID `VS14DHRF'.
#
#   [OK]          single-line          0   traditional, empty.
#   [OK]          single-line          1   traditional, CR.
#   [OK]          single-line          2   traditional, LF.
#   [OK]          single-line          3   traditional, CRLF.
#   [OK]          single-line          4   unicode, empty.
#   [OK]          single-line          5   unicode, CR.
#   [OK]          single-line          6   unicode, LF.
#   [OK]          single-line          7   unicode, CRLF.
#   [OK]          single-line          8   unicode, VT.
#   [OK]          single-line          9   unicode, FF.
#   [OK]          single-line         10   unicode, NEL.
#   [OK]          single-line         11   unicode, LS.
#   [OK]          single-line         12   unicode, PS.
#   [OK]          multi-line           0   multi-line range with LS.
#   [OK]          multi-line           1   multi-line range with CRLF.
#
# Full test results in `~/.opam/5.2/.opam-switch/build/asai.0.3.0/_build/default/test/_build/_tests/Explicator'.
# Test Successful in 0.001s. 15 tests run.
# (cd _build/default/test && ./TestFlattener.exe)
# Testing `Flattener'.
# This run has ID `ZU125I9G'.
#
#   [OK]          flattening          0   single-line ranges.
#   [OK]          flattening          1   multi-line ranges.
#
# Full test results in `~/.opam/5.2/.opam-switch/build/asai.0.3.0/_build/default/test/_build/_tests/Flattener'.
# Test Successful in 0.000s. 2 tests run.

Due to the experimental nature of asai, I am happy to quickly release a new version for OCaml 5.2.0 that uses Format.pp_infinity. For obvious reasons I really don't want to hard-code the constant 1_000_000_000. When will OCaml 5.2.0 be released? Will a new version with ocaml >= 5.2 solve the problem?

Will a new version now with ocaml >= 5.2 solve the problem?

yes. The current code should be good

@kit-ty-kate BTW I realized pp_infinity would not work. It has to be pp_infinity-1.

authors: favonia kit-ty-kate

repo: RedPRL/asai


text:
Add "top" level as a 'façon de parler'

This would simplify the syntax quite a bit; judgments would be allowed to mention the 'top' level, but U ltop would not be a type. This would replace the optional-ness of level annotations in some judgments.

Yes we can do this. I did not do this purely because of trying to be excessively pedantic for the first experiment. One difficulty of super-clean-API is that the pretty printer needs to know whether it's omega or not. Or, we can just print "at omega" whenever the level is actually lacking.

Should we forbid (lmax lomega lomega)?

Same question for (+ lomega 0) and whether an exception should be thrown for (++ lomega).

@favonia Would it be weird to have (lmax lomega lomega) = lomega and (++ lomega) = lomega?

(lmax lomega lomega) = lomega and (++ lomega) = lomega

This is known as a 'parler trick'.

@jonsterling I am fine with (lmax lomega lomega) = lomega but not the other, because I wish (U l) is always a member of (U (++ l)).

@favonia Aha, I see! I think that is fair

authors: cangiuli favonia jonsterling

repo: RedPRL/sml-redprl


text:
Update Readme for Android permission handling

According to the code and compiler warnings, using OnRequestPermissionsResult() from the Android Permission Handler is deprecated. The Readme still suggests to use this though. Please clarify this.

Yup -documentation is well out of date for v3 of this plugin.
As long as you have Essentials 1.5.x - there is no need for anything extra in the OnRequestPermissionsResult() on android from our findings...

The Readme still suggests to use this though.

Can you open a PullRequest please?

Can you open a PullRequest please?

Sure, see #950.

authors: Lydecker knocte radioactiveman


text:
单行、多行文本溢出隐藏

单行文本溢出
overflow: hidden;            // 溢出隐藏
text-overflow: ellipsis;      // 溢出用省略号显示
white-space: nowrap;         // 规定段落中的文本不进行换行

多行文本溢出
overflow: hidden;            // 溢出隐藏
text-overflow: ellipsis;     // 溢出用省略号显示
display:-webkit-box;         // 作为弹性伸缩盒子模型显示。
-webkit-box-orient:vertical; // 设置伸缩盒子的子元素排列方式：从上到下垂直排列
-webkit-line-clamp:3;        // 显示的行数

repo: ReinerLau/Blog


text:
Add in symbol definition capability

This is not strictly an Emacs VHDL Mode capability, but it is a Sublime capability and it seems like it'd be useful to get the symbol definitions working now that I have good scoping.

Next revision is going to include this.  Seems to be working pretty well so far.

Initial support added.  Will continue to refine this feature over time.

repo: Remillard/VHDL-Mode


text:
Prizes Target

Ethereum Attestation Service
Nouns DAO
Quick Node
XMTP
Worldcoin
The Graph
Airstack
Gnosis Chain

Airstack: we will use Airstack to find out if an address is a nouns holder or not, and display the Nouns owned by the user using Airstack to retrieve the NFT image

Gnosis: we will deploy on Gnosis testnet

The Graph: we will index our own contract with The Graph and calculate reputation score

XMTP: we will allow users to send msg to each other via XMTP

Worldcoin: we will use Worldcoin to verify address's eligibility to become an attestor

Nouns DAO: we will allow Nouns to filter for proposals listed by other Nouns

Quick Node: we will switch users to use QN for RPC
Need to check if they have gnosis testnet rpc tho
Otherwise, use QN RPC to query some mainnet info

Ethereum Attestation Service: when a proposal is marked executed, an attestation record is issued from EAS
Need to check how we do that if deployed on Gnosis
Probably we can do a private off-chain attesation

repo: ReputeETHGlobalWaterloo/Repute


text:
Use 204 on empty delete response

See: http://jsonapi.org/format/#crud-deleting

No idea why the test is failing

@medokin thanks for the PR! I'll dig into it this afternoon.

the test you updated was intended to verify that the server returns a 404 if a DELETE request is received for an entity that doesn't exist. I've updated the appropriate tests.
Currently, the organization of the tests is less than desirable and I hope to change that in the near future

Perfect, thanks. It works fine now. Would be easier to contribute if #47 was finished.

#51 is open to resolve #47. However, it looks like AppVeyor has yet to upgrade their version of the SDK. Once that is fixed, I'll merge into master. Until then, if you'd like to contribute, you can work off the develop branch

authors: jaredcnance medokin

repo: Research-Institute/json-api-dotnet-core


text:
Is this still being developed?

Need to know if I should make pull requests or just fork.

Fork it, I don't think its managed.

Yes, fork if needed. I am not managing this anymore.
so 8. 9. 2018 v 15:44 odesílatel Najam notifications@github.com napsal:

Fork it, I don't think its managed.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ReshNesh/pixlserv/issues/14#issuecomment-419647176,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJkMfEk-aBzRvlCwJslcxFvQ3d7qqZ3ks5uY9fbgaJpZM4EaoPF
.

authors: ReshNesh najamkhn robclancy

repo: ReshNesh/pixlserv


text:
现在是不是失效了

发现了bug？(bug)

系统版本：[WIN7]
重现步骤：

[点击下载后一直是等待中]

还有希望如果日后更新了。文件的排列顺序按照日期排列而不是现在的文件名排列。

现在的版本能用？应该登陆不了吧

我的登录时提示连接服务器出现错误，你们有这种情况吗？

我也登录不上了

听说是百度封杀了

登陆不上去。。。

哎 ~

我也是登不上去了，一直服务器错误

登录不上了

repo: ResourceHunter/BaiduPanDownloadWpf


text:
Support managed object context detection with arrays

This detects if the object is an array and looks at the object inside it to determine if there is a managed object context available. Otherwise it defaults to the main thread leading to deadlocks in some situations.

This was never cleanly applied nor tested, please ping me if that's done.

authors: Neonkoala segiddins

repo: RestKit/RestKit


text:
关于精度没有达到论文中的水平的问题

关于您的模型，已按照您的配置执行，采用的是esc50数据集，并加载了swin_transformer的预训练模型，但最后的精度如图，只能达到0.88，测试也是如此，不知哪里出了问题，您可以给一些建议吗？十分感谢！

您好，训练esc-50的时候需要将esc-50里的数据重采样为32000，因为我们训练audioset的时候是32000的采样率；同时预训练模型是先训练在audioset的，不仅仅是swin-transformer在imagenet上的预训练模型。
谢谢！

您好，我不太理解您说的先在audioset上预训练的意思，是先把swin_transformer的参数迁移过来，然后再在audioset上训练，最后在esc-50上测试可以到0.97的精度吗？还是其他意思呢？
刚接触深度学习，如有打扰还请见谅。
谢谢！

您好，感谢您的指点，我已经成功复现了您的结果，祝您生活愉快！

是的，在audioset上预训练，不客气！

repo: RetroCirce/HTS-Audio-Transformer


text:
Expand & Split Out Specs

📃 Preview
Changelog

Remove pomo_db directory (they have their own repos now)
Add stack diagrams
Expand a couple sections
Switch to consolodated Markdown links

Alright, I've finished a first pass through this :)

Two nitpicks, but this looks great to me!

authors: QuinnWilton expede

repo: RhizomeDB/spec


text:
Is it possible to add a new contact to the phonebook?

I am working with groups and eventually I need to include people in the group, the number is being blocked. Is there anything that can be done?
Would including the number in the phonebook solve the problem?

No, you have to manage contacts for your contacts book from google your iPhone

authors: adarosci beshoo

repo: Rhymen/go-whatsapp


text:
Feature request: Public invite link

It would be great to obtain a public invite link for a group.

It should be pretty straigt forward I think. If I've got time I will implement it tomorrow.
Quick notes:
Query:
621.--137,,["query","inviteCode","491xxxxxx-1550606659@g.us"]
Answer:
621.--137,{status: 200, code: "KyvTtKp6ixxxxx"}
Resulting invite link:
https://chat.whatsapp.com/KyvTtKp6ixxxxx

Sorry for beeing late. But better late than never :)

authors: KrzysztofMadejski SchulteMK


text:
What does it do?

Does it auto update gzdoom, mods or what?

It checks if your version of gzdoom is out of date, and if it is, it downloads the newest version, and updates it.


text:
feat: 🎸 add support for 🤗 Hugging Face

On the Hugging Face Hub we are building the largest collection of models, datasets and metrics in order to democratize and advance AI for everyone 🚀. The Hugging Face Hub works as a central place where anyone can share and explore models and datasets.
_https://huggingface.co/docs/hub/main_

The 🤗 Hugging Face Hub hosts ~30k Git-based repositories which are storage spaces that can contain files for: models, datasets and spaces.

@severo I added hugging face support to tiged (fork of degit). Rich isn't really maintaining degit anymore. Tiged is already used by big projects like nuxt and has lots of bug fixes. https://github.com/tiged/tiged
(Full disclosure I only used merged your updated Readme and tests, because merging the index.js was a bit of a headache because tiged supports both ssh and https and all domains in git mode.).

repo: Rich-Harris/degit


text:
IE10/11 flicker

Sigh... I thought that IE11 at least might not be terrible. FFS. There's a noticeable flicker at the end of each transition - somehow it's getting the timing all messed up, and the callback is executing at the wrong moment. It's almost as if the people who make IE are rubbish at their jobs or something.

Relevant SO question: http://stackoverflow.com/questions/25086985/this-animation-flickers-at-the-end-in-ie-11
Ugh. Probably going to have to fall back to timer-based animations. By the sounds of it it's an IE11-only bug (don't have an IE10 VM to hand), so that'll involve some browser sniffing.
Seriously, IE team, just go home. We would all be so much better off if you went and did something else that didn't involve software.

repo: Rich-Harris/ramjet


text:
(piping) Koa + webpack-hot-middleware loses WebSocket connection when server restarts

Started using webpack-hot-middleware since a recent version, but just noticed that when piping restarts the Koa server, it also disconnects the webpack HMR client's WebSocket connection. This breaks client updates.
The only way to solve this currently is to disable piping for now if you want client updates. Sorry for this inconvenience.

Fixed by reverting to webpack-dev-server.

But then syntax-errors cannot be react-redboxed if you don't use webpack-dev-middleware and webpack-hot-middleware I believe

It's impossible to combine serverside webpack with webpack-dev-middleware because the latter cannot be bundled.
Why wouldn't react-redboxed work with webpack-dev-server?

The server-side doesn't need to be, it can stay as it is.
When I was trying to set react-redbox up with webpack-dev-server, I couldn't get it working. I had to use webpack-dev-middleware.
gaearon (Dan Abramov) is also using webpack-dev-middleware here: https://github.com/gaearon/react-transform-boilerplate

Syntax errors are displayed in an overlay using @glenjamin’s webpack-hot-middleware, which replaces Webpack Dev Server. This project does not use React Hot Loader.

All that you've mentioned, I already know.
Again, the server-side uses Webpack for server-side HMR and webpack loaders. webpack-dev-middleware can not be bundled by Webpack because of technicalities. That's why I had to revert that change.
If you've got an issue setting up React redbox please make a separate issue about it.

So wait, you already support syntax-error catching with react-redbox too? In that case I'll try actually running this project. I actually have my own project not based on this one, but I'm taking a lot of ideas from here :P

Okay I tried your project by cloning master, npm installing, then doing npm run watch, and no, there is no react-redbox if I write a blatant syntax error in Main.js
It just refreshes the page
Which points to that webpack-dev-middleware is needed if you want hmr with react-redbox with syntax errors

Apparently https://github.com/gaearon/react-transform-catch-errors is needed to catch errors. I will try to add it.

@amcsi Did you use the development build with npm run watch? It works for me. I get a nice red screen with the error.
If you use the production command npm run start then babel-preset-react-hmre is not configured and therefore redbox-react will not work.

Argh this is confusing
Okay so first I want to put it out there that gaearon himself said that for react-redbox to work with not just errors happening in render() methods (like it can inherently with react-transform-catch-errors), but also with syntax errors or any other error happening outside of render() methods, webpack-dev-server is not good enough:
https://github.com/KeywordBrain/redbox-react/issues/36#issuecomment-167248365
Secondly, I wanted to test again with your project so I could confirm that react-redbox does work, just not with syntax errors. However, I actually didn't get a redbox, but just a page refresh even with a throw within a render() method.
I might be doing something wrong though; what I did was:

npm run watch
go to http://localhost:8000/
write a syntax error outside the Main.js component definition (as expected, resulted in page refresh)
attempted to write a throw 'lol' in the render() method (unexpectedly the page refreshed when I was actually expecting this to work even with webpack-dev-server)

Could you try one thing for me? Please try to type this before class Main extends React.Component {:
asdfasdf dsafadsf
^ a blatant syntax error like that outside of the React component definition. Are you still getting the react-redbox like it would be cool to?

Previously I tried just regular errors, not syntax errors. Now I can confirm that syntax errors do not show in redbox-react.
Dan Abramov is right. And there seems no way to prevent the Webpack page reload. I'll look into this when I have time. Perhaps there is some undocumented setting.

@amcsi  You can set config.devServer.inline to false in webpack.client-watch.js to prevent the page reload! Webpack will update the page again after you fix the syntax error.
Is that good enough?

@RickWong I tried and yes it seems to not do the refresh, which is good in the sense that I get to keep my state, although it does have a slight flaw that now there's no visual clue on the page that some sort of error happened. By the way I'm still not able to reproduce a redbox at all, even when I throw an exception within render()
It's still not as good as displaying syntax errors in a big red box.
I recommend you to see Dan Abramov's video about Hot Reloading with Time Travel to see visually how useful it is that you get a red box when any kind of error happens while developing: https://www.youtube.com/watch?v=xsSnOQynTHs
I think this feature would be really important

@amcsi Try this in the render() function of Main.js:  throw Error("Hello world");

With that the redbox does appear. I know it was complaining before when I did throw "lol" in the render() method that the thrown value was expected to be an instance of Error when it was actually a string.

I'll leave the issue where it's at right now. Not fully reloading the page makes it possible to keep state. That's good enough at the moment. Thanks for debugging it with me!

No problem, good luck with your project which seems really promising!

authors: RickWong amcsi

repo: RickWong/react-isomorphic-starterkit


text:
https://github.com/RiemaLabs/nubit-cli.git repository 404

I want to build a nubit-cli，but I can't find the nubit-cli repository. When will the repository open ?

Hello luoyechao, Thank you for reaching out. As of now, nubit-cli has not been released to the public. We are diligently working on it to ensure the best possible experience to our users. Please keep an eye on our Github site for the latest information.

repo: RiemaLabs/modular-indexer-committee


text:
Added a performance hint.

When using naive_and aggregation, putting a tiny bitmap first can be beneficial. Let us communication this insight to the users.
Related to https://github.com/RoaringBitmap/RoaringBitmap/issues/608

Merging.

repo: RoaringBitmap/RoaringBitmap


text:
More off-by-one errors in bitmap contains_range

The CRoaring code uses an exclusive u32 range for the implementaiton of
a bitset container's contains_range function.
Had to think a bit about the best way to make it work, if you can think of a prettier way to write any of it, let me know.
Fixes #238

I think I thought of a better way, by being inclusive at the bit-level only

bors merge

bors merge

Same here, no obvious reasons why the CI is stopped...
bors merge

authors: Dr-Emann Kerollmops

repo: RoaringBitmap/roaring-rs


text:
will it work for multivariate time series prediction   both regression and classification

great code thanks
may you clarify :
will it work for multivariate time series prediction   both regression and classification
1
where all values are continues values
weight    height     age   target
1   56        160    34    1.2
2   77        170    54    3.5
3   87        167     43   0.7
4   55        198    72    0.5
5   88        176     32   2.3
2
or  even will it work for multivariate time series where values are mixture of continues  and categorical values
for example 2 dimensions have continues values and 3 dimensions are categorical values
color        weight     gender  height  age  target

1    black       56             m   160    34    yes
2    white       77             f   170    54    no
3    yellow      87             m   167    43    yes
4    white       55             m   198    72    no
5   white       88              f   176    32    yes

Thanks for reaching out, Sandy.
Yes, the LSTM will work for both ordinal, categorical, and real-valued
targets. In the case of a categorical, you might choose a Multinomial
likelihood function.
On Mon, 11 Jan 2021 at 00:28, Sandy4321 notifications@github.com wrote:

great code thanks
may you clarify :
will it work for multivariate time series prediction both regression and
classification
1
where all values are continues values
weight height age target
1 56 160 34 1.2
2 77 170 54 3.5
3 87 167 43 0.7
4 55 198 72 0.5
5 88 176 32 2.3
2
or even will it work for multivariate time series where values are mixture
of continues and categorical values
for example 2 dimensions have continues values and 3 dimensions are
categorical values
color        weight     gender  height  age  target
1 black 56 m 160 34 yes
2 white 77 f 170 54 no
3 yellow 87 m 167 43 yes
4 white 55 m 198 72 no
5 white 88 f 176 32 yes
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RobRomijnders/LSTM_tsc/issues/16, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3MXROPRBX7DDGHOEA52ZLSZIZ2BANCNFSM4V4Z3X5A
.

Thanks for reaching out, Sandy.
Yes, the LSTM will work for both ordinal, categorical, and real-valued
targets. In the case of a categorical, you might choose a Multinomial
likelihood function.
On Mon, 11 Jan 2021 at 00:28, Sandy4321 notifications@github.com wrote:

great code thanks
may you clarify :
will it work for multivariate time series prediction both regression and
classification
1
where all values are continues values
weight height age target
1 56 160 34 1.2
2 77 170 54 3.5
3 87 167 43 0.7
4 55 198 72 0.5
5 88 176 32 2.3
2
or even will it work for multivariate time series where values are mixture
of continues and categorical values
for example 2 dimensions have continues values and 3 dimensions are
categorical values
color        weight     gender  height  age  target
1 black 56 m 160 34 yes
2 white 77 f 170 54 no
3 yellow 87 m 167 43 yes
4 white 55 m 198 72 no
5 white 88 f 176 32 yes
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RobRomijnders/LSTM_tsc/issues/16, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3MXROPRBX7DDGHOEA52ZLSZIZ2BANCNFSM4V4Z3X5A
.

super, do you have code example?

super, do you have code example?


text:
improve interface of the class

[ ] rename? as library can be used for other symmetrical matrix storage.
[x] dimension()  => side
[x] memoryUsed() == what now is size()  ==> remove size().
[x] min() how to handle negative / 0 value - use a flag? What does it return?
[x] max() idem as min()  ==> maybe create an example with a search...

Won't rename for now;  others solved in 0.2.0

Won't rename for now;  others solved in 0.2.0

repo: RobTillaart/DistanceTable


text:
Add contextual information to be used in mocha reporters

I was unsatisfied with Mocha's default output for failures when used with frisby--
When a test fails it spits out a stack trace, which is less than useful when you're debugging an HTTP request.
This change attaches the request and response to the test so that they may be referenced from a custom mocha reporter. I may release the reporter as well, if this is merged. It basically just displays the exact request made (in the form of a curl command) and the raw response that was returned. This helps to quickly fix issues when they occur.

Coverage increased (+0.02%) to 89.86% when pulling 4c4a2398f16b2747a28253bec8b42aa441497670 on kbanman:reporter-context into b7ebe396042e6377e1449c29d35468f1d57262a0 on RobertHerhold:master.

Coverage increased (+0.02%) to 89.86% when pulling 4c4a2398f16b2747a28253bec8b42aa441497670 on kbanman:reporter-context into b7ebe396042e6377e1449c29d35468f1d57262a0 on RobertHerhold:master.

Coverage increased (+0.02%) to 89.86% when pulling 4c4a2398f16b2747a28253bec8b42aa441497670 on kbanman:reporter-context into b7ebe396042e6377e1449c29d35468f1d57262a0 on RobertHerhold:master.

Thanks!

Published to version 0.0.8 with your changes.

authors: RobertHerhold coveralls kbanman

repo: RobertHerhold/IcedFrisby


text:
onUnMask is not being called

Hello. We've been working with this plugin for some time, works really lovely but I recently came across a little problem I think. An input field uses 'percentage' alias and that doesn't have any value entered. I've passed an onUnMask function to the mask options but when I try to retrieve the unmaskedvalue of the input I get null and that method is not even called. Why is this so? I want it, because I want to make the input field's unmaskedvalue to return 0 when no value has been entered.

Set the nullable option to false

`PERCENTAGE_OPTIONS =
alias: 'percentage'
rightAlign: false
suffix: '%'
onUnMask: (maskedValue, unmaskedValue) ->
console.log 'onUnMask', maskedValue, unmaskedValue
nullable: false
$inputs.inputmask PERCENTAGE_OPTIONS`
This is how I add the mask after your answer. When I try to get: $el.inputmask 'unmaskedvalue' I still get null and the onUnMask function is not being called.

This will need a fix.  I will submit asap.

@Tomcatus
Grab the latest from github an try
Inputmask({
			alias: 'percentage',
			rightAlign: false,
			suffix: '%',
			autoUnmask: true,
			nullable: false
		}).mask($inputs);

Works like a charm, thanks! :) but I have a question, bower can't find the current version even if I pass him "jquery.inputmask": "https://github.com/RobinHerbots/jquery.inputmask.git#3.3.2-9" Any ideas why? It says the version is not available.

Because version 3.3.2 is not released.   You should take the latest from
github
Op ma 6 jun. 2016 om 09:32 schreef Tomasz Pawlak notifications@github.com:

Works like a charm, thanks! :) but I have a question, bower can't find the
current version even if I pass him "jquery.inputmask": "
https://github.com/RobinHerbots/jquery.inputmask.git#3.3.2-9" Any ideas
why? It says the version is not available.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/RobinHerbots/jquery.inputmask/issues/1291#issuecomment-223887240,
or mute the thread
https://github.com/notifications/unsubscribe/AATb7x3w8oeoFHTux2QR_In-I_Nhopw4ks5qI80FgaJpZM4ItSeP
.

Ok, thanks again for the quick fix :)

Hi,
Thank you for your work.  I have been trying to update my email and URL entry and when I saw your latest code, i thought i had reached my finished source; however, I can't seem to get the masks to work.  would you please recommend an approach to finish the URL and email masking from this page:
http://johnnysandaire.com/contact/index.php?modal=2

authors: RobinHerbots Tomcatus webzest


text:
Are the task resource limits respected?

Are the task resource limits set in https://github.com/Roblox/nomad-driver-iis/blob/28e9cadf42954f4a0b2a54e507d21c09ae7b111f/examples/iis-test.nomad#L28-L34 respected?
If so, can you point me to the code that sets those in the IIS application pool?

They are not limited by these settings, I set those to work with the nomad job spec requirements. What could be done now is set the application pool xml settings via
https://docs.microsoft.com/en-us/iis/configuration/system.applicationhost/applicationpools/add/cpu
https://docs.microsoft.com/en-us/windows-server/administration/performance-tuning/role/web-server/tuning-iis-10#worker-process-and-recycling-options
.
I could have the plugin do this automatically if what IIS does with these settings is to be expected.

authors: Vulfox rgl

repo: Roblox/nomad-driver-iis


text:
FileNotFoundException every 5 minutes

Hey,
the plugin is displaying the following Exception:
[08:35:54 WARN]: java.io.FileNotFoundException: ASkyGrid/pipes.dat (No such file or directory) [08:35:54 WARN]:        at java.io.FileOutputStream.open0(Native Method) [08:35:54 WARN]:        at java.io.FileOutputStream.open(FileOutputStream.java:270) [08:35:54 WARN]:        at java.io.FileOutputStream.<init>(FileOutputStream.java:213) [08:35:54 WARN]:        at java.io.FileOutputStream.<init>(FileOutputStream.java:162) [08:35:54 WARN]:        at de.robotricker.transportpipes.a.a.a.a(Unknown Source) [08:35:54 WARN]:        at de.robotricker.transportpipes.a.a.a.a(Unknown Source) [08:35:54 WARN]:        at sun.reflect.GeneratedMethodAccessor40.invoke(Unknown Source) [08:35:54 WARN]:        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) [08:35:54 WARN]:        at java.lang.reflect.Method.invoke(Method.java:497) [08:35:54 WARN]:        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:302) [08:35:54 WARN]:        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62) [08:35:54 WARN]:        at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:502) [08:35:54 WARN]:        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:487) [08:35:54 WARN]:        at net.minecraft.server.v1_11_R1.WorldServer.save(WorldServer.java:995) [08:35:54 WARN]:        at org.bukkit.craftbukkit.v1_11_R1.CraftWorld.save(CraftWorld.java:748) [08:35:54 WARN]:        at net.minecraft.server.v1_11_R1.MinecraftServer.C(MinecraftServer.java:703) [08:35:54 WARN]:        at net.minecraft.server.v1_11_R1.MinecraftServer.run(MinecraftServer.java:576) [08:35:54 WARN]:        at java.lang.Thread.run(Thread.java:745) [08:35:54 INFO]: [TransportPipes] saved 0 pipes in 4 worlds

You can access the build with this bug fixed here: http://frontfight.net:8080/job/Transport-Pipes/lastSuccessfulBuild/artifact/target/transportpipes-3.5.1-SNAPSHOT.jar

repo: RoboTricker/Transport-Pipes


text:
browser actionのフォントサイズを指定する

Describe the bug
browser actionでフォントサイズが設定されていないため、ChromeとFirefoxで外観が統一されていない。
Expected behavior
フォントサイズを指定し、できるだけChromeとFirefoxで同じ外観になるようにする。

関連：#173

repo: Robot-Inventor/spam-tweets-compressor


text:
Tree Viewer bug fixes

Fixes #478
Fixes #477

@patmarion I'll bring in your changes for #479 on this branch too

repo: RobotLocomotion/director


text:
Allow authorship comments

This is a proposal to allow Drake programmers who would like to do so to sign their significant contributions, so that their names appear in the source code and Doxygen. Note that there is already a Drake Credits page and of course git remembers everything, so this is entirely optional and not usually needed.
Comments on this proposal are welcome, and sign-on from other platform reviewers is required.

This change is

This seems broadly reasonable to me.  I can't say I really approve of the spirit of it, but if it is customary in the community of our major potential contributors then it will gain us contributors and the material benefits will firmly outweigh my stodgy qualms.

I do hope it ends up being used sparingly, and that platform reviewers are not obliged to become arbiters of Significant Authorship -- I've been down that road as a Wikipedia editor and it is not pleasant.  But if that happens I trust we will reconsider the policy.

Reviewed 1 of 1 files at r1.
Review status: all discussions resolved, LGTM missing from assignee sammy-tri, LGTM missing from assignee jwnimmer-tri, LGTM missing from assignee soonho-tri, LGTM missing from assignee seancurtis-tri, LGTM missing from assignee ericcousineau-tri, LGTM missing from assignee russtedrake, platform LGTM from [ggould-tri]

cppguide.html, line 5286 at r1 (raw file):

and you should definitely add yourself to that list. In
addition, git maintains a history of all code modifications,
and git blame can be used to find out who did

FYI, git annotate is an equivalent, more polite phrasing of that command.

cppguide.html, line 5293 at r1 (raw file):

a major co-author, you can sign your beautiful creation
with a Doxygen-formatted comment. Then your name
will appear in the

FYI trailing whitespace (here and on about a dozen random lines in the next two paragraphs).

Comments from Reviewable

What happens to authorship if others edit the component (possibly in significant ways) in the future?

Generally the original author would remain and additional authors would add themselves, possibly adding a per-author clarification, like "provided the original implementation" to the first author's name, or deleting the first author after a complete rewrite.

I would rather receive their contributions and hold my nose at the author tag, than not have their contributions at all.

Although that might be an issue, I don't actually know of anyone refusing to contribute unless they can sign. My motivation is less tangible -- to let people express their pride by signing their work. Not everyone wants to do that but some people do (according my statistically insignificant surveying) so I want to see if we can allow them that avenue of expression.

I personally would also appreciate the ease of looking to see whom I should
query about a piece of functionality without having to use git blame.

Evan Drumwright
Senior Research Scientist
http://positronicslab.github.io
Toyota Research Institute
Palo Alto, CA
On Tue, Jul 10, 2018 at 8:54 AM Michael Sherman notifications@github.com
wrote:

I would rather receive their contributions and hold my nose at the author
tag, than not have their contributions at all.
Although that might be an issue, I don't actually know of anyone
refusing to contribute unless they can sign. My motivation is less tangible
-- to let people express their pride by signing their work. Not everyone
wants to do that but some people do (according my statistically
insignificant surveying) so I want to see if we can allow them that avenue
of expression.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RobotLocomotion/styleguide/pull/11#issuecomment-403874090,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACHwz6N2jweBEGN3B4ajT7ENmVBgsXNQks5uFM5OgaJpZM4VIwvr
.

--
Confidential or protected information may be contained in this email
and/or attachment. Unless otherwise marked, all TRI email communications
are considered "PROTECTED" and should not be shared or distributed. Thank
you.

I personally view "@author" as about ownership- who has taken
responsibility for this piece of code?

Evan Drumwright
Senior Research Scientist
http://positronicslab.github.io
Toyota Research Institute
Palo Alto, CA
On Tue, Jul 10, 2018 at 9:19 AM Jeremy Nimmer notifications@github.com
wrote:

@jwnimmer-tri commented on this pull request.
Absolutely true. We should be clear that this proposal (and the @author
tag) is about pride / credit, not about where to direct questions. The
qualified / best people to respond to questions about some code is much
wider than @author, and sometimes will not even include the named author.
(It might not even be in the git annotate output -- in some cases, the
feature reviewer who proposed a change would be the best one to ask.)
Reviewable
https://reviewable.io/reviews/robotlocomotion/styleguide/11#-:-LH3yC8lC5usC-OFjXES:b-t0nm2g
status: 1 unresolved discussion, LGTM missing from assignee seancurtis-tri,
LGTM missing from assignee ericcousineau-tri, LGTM missing from assignee
russtedrake, platform LGTM from [ggould-tri, jwnimmer-tri, sammy-tri,
soonho-tri]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RobotLocomotion/styleguide/pull/11#pullrequestreview-135903481,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACHwzwy8yf2bD2dZ_MHSYKb7lyrg6ESpks5uFNP0gaJpZM4VIwvr
.

--
Confidential or protected information may be contained in this email
and/or attachment. Unless otherwise marked, all TRI email communications
are considered "PROTECTED" and should not be shared or distributed. Thank
you.

If we want to add an OWNERS file to our tree (and, e.g., tie it into feature review) that's fine by me too, but that's not what @author is.

(I see now that GitHub has their own CODEOWNERS file, that would be another option to publish ownership.)

@author kills two birds (ownership and taking pride in one's work) with one
stone, imo.

Evan Drumwright
Senior Research Scientist
http://positronicslab.github.io
Toyota Research Institute
Palo Alto, CA
On Tue, Jul 10, 2018 at 9:46 AM Jeremy Nimmer notifications@github.com
wrote:

(I see now that GitHub has their own CODEOWNERS
https://help.github.com/articles/about-codeowners/ file, that would be
another option to publish ownership.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RobotLocomotion/styleguide/pull/11#issuecomment-403890528,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACHwz2ISOMMCs4rmasvlfvPhYcJ40uxdks5uFNp6gaJpZM4VIwvr
.

--
Confidential or protected information may be contained in this email
and/or attachment. Unless otherwise marked, all TRI email communications
are considered "PROTECTED" and should not be shared or distributed. Thank
you.

I don't think @author kills any birds. It sends stones flying, but the birds are sitting in the tree, mocking us.  :)
If we want @author to have SWE utility, it becomes something that must be reviewed and asserted correct. And if it becomes owner, it supplants the role of providing credit. After all, if I leave TRI, do I lose the credit I once received for the work I did just because someone else assumes my responsibilities?
Like many voices, I wouldn't apply my name as an author in files. I'm content with being in the contributor list. If I saw an @author in drake, I would ignore it. It would be a soft warm fuzzy for someone but I would not consider it to be an actionable piece of information. And I think it should be clearly documented as such.
I do like the idea of ownership and documenting that so other developers have an additional resource in determining who they should contact. I like it happening in one centralized place so that one only has to look once -- even though that is, by no means, a silver bullet.
Conclusion: I don't think that @author can provide any meaningful development information and should not be hijacked to attempt to do so. But I won't fight against allowing people who value seeing their name in the code do so. But the meaning of an @author tag needs to be clearly delineated.

I'll have to side with the reservations; can I ask who concretely expressed interest in adding @author to the files?
Is there someone in particular, for a particular reason?
If it's a good one that can't be satisfied easily by some other mechanism, then I'm all for it; however, if there's another way, such as more explicit acknowledgements or accolades or whatever, then I'd prefer that rather than adding a new convention and review item.
Perhaps we can also expand on the existing Sphinx documentation so that people can more explicitly state their contributions, explicit ownership, etc.

As an aside, there would be a fair case to be made that -- while the platform review team should review this PR with an eye toward us being able to enforce the rule -- the review of the policy proposal itself should be open to all drake developers, not platform reviewers.

can I ask who concretely expressed interest in adding @author to the files?

No one asked or complained -- I had this idea thinking about how to support career growth for early-career Drake programmer/researchers. I expect that some academic and early-career contributors will want to sign their work as a way to build reputation, and I'm proposing that we allow that.  Asking around, there are current (paid) Drake contributors who do want to add their names, but they weren't boycotting before. Alternatives would be to disallow that altogether, or defer a decision until it comes up.

For something that is rare and/or not immediately needed, I'd also prefer if this PR could be as short as possible.  Maybe one paragraph of rationale, and then 1-2 codeblock examples that show the proper syntax.  (GSG tends to give rules by example, rather than by prose, and I do like that as a philosophy.)

My half cent on the subject. I see that pretty much that many people added to this discussions from a variety of angles so I won't add much to this. I value in all of the points here raised and I'll :lgtm: this proposal since I believe we should at least give people the option to sign their work if they wanted to.

Looks like this when formatted:

Here is what the authorship section looks like now:

The Authorship section looks like this now:

authors: EricCousineau-TRI SeanCurtis-TRI amcastro-tri edrumwri ggould-tri jwnimmer-tri sherm1

repo: RobotLocomotion/styleguide


text:
Refactor study day/duration derivations

Currently we have several functions to create common study day variables such as derive_var_ady() and derive_var_astdy(). These functions make use of derive_vars_duration() internally. I feel like we should directly expose a dedicated, parametrized function for study day creation to users. While currently this is already sort of existent in the form of derive_vars_duration() I feel like this function is way to complex for a simple study day derivation (a lot of parameters are not really needed for a study day derivation). Thus, I would propose to separate study day and duration derivation. If we do so we could drop the add_one parameter from derive_vars_duration(), thereby making it easier to understand. As for the studay day derivation I have two different approaches in mind.
First Proposal
# Function signature
derive_var_dy <- function(dataset, new_var, reference_date, date) {
  ...
}
# Example call creating `DTHDY`
derive_var_dy(adsl, new_var = DTHDY, reference_date = TRTSDT, date = DTHDT)

The downside here is that is adds a bit of boilerplate since most (all?) times new_var can be automatically determined from date by replacing the trailing T with Y.
Second Proposal
Alternatively, we could offer a function that does the derivation for multiple --DT variables in one go, similar to the existing derive_vars_dtm_to_dt() function.
# Function signature
derive_vars_dy <- function(datasets, reference_date, source_vars) {
  ...
}
# Example call creating `AENDY` and `ASTDY`
derive_vars_dy(adae, reference_date = TRTSDT, source_vars = vars(AENDT, ASTDT))

This assumes that the reference_date is the same for all (or at least most) of the --DY variables which I think is reasonable.
Any thoughts @Roche-GSK/admiral-devs?

@thomas-neitmann - i like the 2nd proposal.

I would prefer the second option, too. For source_vars I would accept both *DT and *DTM variables.
I would not change derive_vars_duration() because

the add_one parameter is required for durations (see the "Duration Algorithm" tab in the Roche standard specs) and
you could use derive_vars_duration() for study days if the new variable name is different from the source variable name where T or TM is replaced by Y.

@bundfussr Regarding your second point of explicitly renaming variables, I thought we could enable this also in the function I proposed by allowing named entries as part of source_vars, e.g.
derive_vars_dy(adae, reference_date = TRTSDT, source_vars = vars(AENDT, ASTDT, DEATHDY = DTHDY))

The logic would be to automatically infer the --DY variable name if no name is passed for an element and to used the provided name otherwise.
Should we do something similar for derive_vars_dtm_to_dt() and derive_vars_dtm_to_tm()?
Also to be in line with these two existing function I guess we should name this one derive_vars_dt_to_dy()?

@thomas-neitmann , I assume you meant
derive_vars_dy(adae, reference_date = TRTSDT, source_vars = vars(AENDT, ASTDT, DEATHDY = DTHDT))

I like the idea of allowing different names than the automatic ones and we should definitely make it consistent with the derive_vars_dtm_to_*() functions. However, using named parameters of the source_vars parameter does not look intuitive to me. What about replacing source_vars with new_vars? E.g.,
derive_vars_dy(adae, reference_date = TRTSDT, new_vars = vars(AENDY, ASTDY, DEATHDY = DTHDT))

I am not sure about the name. If we use derive_vars_dt_to_dy(), we restrict the source to *DT variables, which I would like to avoid.

What if we do not name the parameter altogether and use ... instead?
derive_vars_dy(adae, reference_date = TRTSDT, AENDY, ASTDY, DEATHDY = DTHDT)

I do not like it because it is not clear what ... expects. The source variables or the new variables?

@teckla-gsk Is this something you'd like to work on? Fits well with the other functins you've already developed 🙂

Sure, I got this will pick this up this week

Function should be named derive_vars_dy(). Parameter name source_vars vs. new_vars stays open questions.

OK , so to summarize the above;

name dervie_vars_dy()
allows both --DTM and --DT

My comment on the below - is it worth it to split them if we are keeping the functionality in derive_vars_duration?

I would not change derive_vars_duration() because

the add_one parameter is required for durations (see the "Duration Algorithm" tab in the Roche standard specs) and
you could use derive_vars_duration() for study days if the new variable name is different from the source variable name where T or TM is replaced by Y.

On the question of source_vars v. new_vars, my opinion is to use source vars, one; we can check if they exist in the dataset easier, two intuitively when scripting one would start with what one has i.e. input from existent dataset flows easier and lastly not sure how many functions use either of the two but maybe an opportunity to start establishing some consistency?

@teckla-gsk Yes, exactly. The name should be derive_vars_dy() and it should be able to handle both --DTM and --DT variables as input.
I also think having source_vars is more intuitive than using new_vars. It also is consistent with the existing derive_vars_dtm_to_dt() and derive_vars_dtm_to_tm().

authors: bundfussr kabis-ops teckla-gsk thomas-neitmann

repo: Roche-GSK/admiral


text:
UnicodeDecodeError when a png file is included in the template

Our template repository requires a png file in it, but it still raises an error when the png file is defined in excluded_files section.
Traceback (most recent call last):
  File "/Users/daiy24/.pyenv/versions/3.11.4/lib/python3.11/site-packages/foxops/engine/__main__.py", line 120, in cmd_initialize
    asyncio.run(
  File "/Users/daiy24/.pyenv/versions/3.11.4/lib/python3.11/asyncio/runners.py", line 190, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "/Users/daiy24/.pyenv/versions/3.11.4/lib/python3.11/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/daiy24/.pyenv/versions/3.11.4/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/Users/daiy24/.pyenv/versions/3.11.4/lib/python3.11/site-packages/foxops/engine/initialization.py", line 38, in initialize_incarnation
    return await _initialize_incarnation(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/daiy24/.pyenv/versions/3.11.4/lib/python3.11/site-packages/foxops/engine/initialization.py", line 88, in _initialize_incarnation
    await render_template(
  File "/Users/daiy24/.pyenv/versions/3.11.4/lib/python3.11/site-packages/foxops/engine/rendering.py", line 106, in render_template
    await _render_template_file(
  File "/Users/daiy24/.pyenv/versions/3.11.4/lib/python3.11/site-packages/foxops/engine/rendering.py", line 85, in _render_template_file
    return await render_template_file(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/daiy24/.pyenv/versions/3.11.4/lib/python3.11/site-packages/foxops/engine/rendering.py", line 132, in render_template_file
    rendered_content = template_file_path.read_text()
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/daiy24/.pyenv/versions/3.11.4/lib/python3.11/pathlib.py", line 1059, in read_text
    return f.read()
           ^^^^^^^^
  File "<frozen codecs>", line 322, in decode
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x89 in position 0: invalid start byte

Thanks for merging the code! @defreng
Currently I'm using fengine as a library, can you publish a new package so that the latest fix is included?

Hi @goldginkgo
I just released 2.0.0-alpha5 ... Make sure that you use the pip install --pre option when installing

authors: defreng goldginkgo

repo: Roche/foxops


text:
Process one bag at a time

Fixes #80

updates dependencies

Right now tests for everything except bag preparer are only running over one thing--which is why they're succeeding.
Since they're all using the same base routine and the logic around list vs. processing one is there, I think testing that in the Bag Preparer is sufficient? If not, I would like to avoid creating more files (like TIFFs) for fixtures, which is what I think we need for the other routines.

A more transparent way might be to test BaseRoutine directly:

Mock process_bag in the BaseRoutine so it doesn't throw an exception
set success_message, start_process_status and end_process_status
Assert on the output.

authors: bonniegee helrond

repo: RockefellerArchiveCenter/pictor


text:
Slider

Is the team working on any slider component?
I was expecting a slider in the Forms part in the storybook but couldn't find. Surprisingly, project hasn't needed it till now.
I require it for this feature:
Pausing notifications for a few minutes.
I want the user to be able to choose using a slider, although I can use a number input as well for the time being.
Expected behavior

Able to pass min and max value.
If possible, the component should handle checkpoints/marks/step-size.

Image

@umakantv We didn't have a case where we needed a more "advanced" range component. For our needs, we use the approach below, maybe it can help you.
<Box display='flex'>
    <Box is='input' type='range' value={100} min='0' max='100'/>
  <Tooltip placement='right-top' mis='x8'>{100}</Tooltip>
</Box>;

Thanks for your suggestion!

authors: dougfabris umakantv


text:
fix: SelectFiltered: Wrong keyboard navigation

Changed the options object that was sent to the _Options in the Select.js file. Please let me know if the issues need to be resolved in any other places as well and I will do so accordingly.
fixes issue #162

Thanks for reviewing.


text:
Failed giving a item & vehicles

cant get any items and vehicles
Failed giving a item to Barra (394,4)

same here

repo: RocketMod/Rocket


text:
add github workflows

it seems a good idea in various articles.
My ToDo list:

[x] learn how it works
[x] create a temporary repository to test first
[x] select the correct workflow(s)
[x] implement the selected workflow(s)

gitub workflow added in v0.3.3


text:
SocketGuild.CreateVoiceChannelAsync (string name) returns server error 0.

When I attempt to use the mentioned function, I get the following exception seen in the attached file:

The line which this occurs on is line 237 found here: https://github.com/Lomztein/Adminthulhu/blob/master/AdminthulhuCore/AutomatedVoiceChannels.cs

Strange... I did a quick test and couldn't repro. Which build are you using?

Do you have a channel with a position of Int32.MaxValue? It appears Discord refuses to create more channels in this scenario.

That.. was actually the problem. It worked prior to 1.0, so I assumed it was a bug in the API. Sorry for the inconvenience, and thanks for the help!

authors: Lomztein RogueException


text:
Achtung! Cthulhu 2d20 - New Sheet

Initial development of character sheet for the Achtung! Cthulhu game by Modiphius using the 2d20 system.

Submission Checklist
Required

[ X] The pull request title clearly contains the name of the sheet I am editing.
[ X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ X] The pull request makes changes to files in only one sub-folder.
[ X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: < Achtung! Cthulhu   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is:  (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <2d20   > (i.e. Dungeons & Dragons, FATE)

[X ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

New sheet for Achtung! Cthulhu using the 2d20 system.

Character Sheet Info Roll20 Internal Use only.

This is my first sheet submission, so not sure if I need to request a review from anybody in particular or how the process runs from here.  I noticed it shows the request failed "Character Sheet Validation" , but I see no errors listed.

Hey there @RandomUsername, the sheet itself looks good. Before merging it I would like you to clarify me how your sheet is different for the existing one for Achtung! Cthulhu that already exists in the repo. I am asking that because while in the past we allowed multiple versions of the sheet to exist side-by-side, this is a practice we are trying to avoid moving forward.

@mperes Hi, no problem. The current sheet is for the FATE gaming system. The sheet I want to add is for the 2d20 gaming system published by Modiphius. The gaming systems are not compatible.

authors: RandomUsername mperes roll20deploy


text:
Add Duration etc... argument resolvers

Helpful informations:
Duration parser: https://github.com/Rollczi/LiteCommands/blob/master/litecommands-core/src/dev/rollczi/litecommands/time/DurationParser.java
Package with basic arguments.

Register argument:

What about using https://github.com/BlackBaroness/duration-serializer-java/? I am ready to add more formats if necessary. I just would more flexible
I am going to implement this

Okay then. Assign this to me if possible please, I will do this in a couple of days

Okay then. Assign this to me if possible please, I will do this in a couple of days

One PR is already created https://github.com/Rollczi/LiteCommands/pull/227 (only for Duration)
But still no for Period or other time units like Instant, LocalTime etc...

Assuming how Period creation works, I don't see real difference between Duration and Period parsers

Duration - short units as seconds, minutes, hours, days etc.
Period - long units as days, weeks, months, years etc.
1 month in Period depending on the current time context it may represent after conversion to Duration e.g 28 days 30 days or 31 days

Also, https://docs.rollczi.dev/documentation/litecommands/features/argument-time-types.html documentation part says that LiteCommands already has mentioned types support

Documentation is not updated yet :/

Hello! I will finish this with Period Argument support.
Sorry for taking issue from You BlackBaroness

TODO Period

authors: BlackBaroness CitralFlo Rollczi

repo: Rollczi/LiteCommands


text:
'TimeSeries' object has no attribute 'get_quote_endpoint'

from alpha_vantage.timeseries import TimeSeries
ts = TimeSeries(key='xxxxxxxxxxxxxxxxxxx')
data, meta_data = ts.get_quote_endpoint('ACB')
Traceback (most recent call last):
File "<pyshell#2>", line 1, in
data, meta_data = ts.get_quote_endpoint('ACB')
AttributeError: 'TimeSeries' object has no attribute 'get_quote_endpoint'

It is not implemented yet :( I am kind of busy at the moment, I might be free over the weekend to implement it, but I am not sure! If anyone wants to do it before me, be my guest. It is actually quite simple.
:D

Yes, please. I need this endpoint https://www.alphavantage.co/documentation/#latestprice

I am getting "AttributeError: 'TimeSeries' object has no attribute 'get_intraday_extended'" error on python 3.7.4 for this line "ts.get_intraday_extended(symbol="FB", interval='15min', slice=f'year1month5')" same line works for Python 3.8.X

authors: RomelTorres aspenforest sanderbrauwers umutsoysal


text:
Init project, keyboard-avoiding-view and pressable components

@ipakhomov I've set up this folder architecture for now. If we need a different one (for example, a custom plugin), I'll redo it.

@ipakhomov check again, please :)

@VitalyMih few more comments

@ipakhomov fixed :) check again, please

authors: VitalyMih ipakhomov

repo: RonasIT/react-native-common-modules


text:
file for rings/swaps also considering policy

when opening several rings in a short period of time it'd be nice to handle then separately in files instead of editing the script
format per line: key: value
I for now added lines for following keys

members (comma separated list first_peer,…,own_peer … number of peers can be determined e.g. by count of commas+1)
capacity (optional, can be read from listchannel)
url (informational where a channel might be documented, e.g. <lightningnetwork.plus> )
policy_base (policy for base_fee_msat)
policy_min_htlc (policy for min_htlc_msat)
policy_fee_rate (policy for fee_rate)

any thoughts what else would be useful, also to automate the whole ring building and initial balancing as well as keeping additional information?

That was my original thought, but it has been decided to keep it all in one file, as file parsing could just by itself be consisted of more code than the whole current script.

If you can manage to do something reusable, simple and stable, feel free to suggest a pull request.

The aim here is to keep Igniter small and simple.
Nothing prevents from copying the script with different configurations as it is.
Since this issue hasn't got any traction in more than two months, it will be closed.
Thank you @vv01f for the suggestion!

repo: RooSoft/igniter


text:
Add clan family apple legends

Now I have given two but it is not random you can verify us.sorry for giving invalid last time.plzz create it as fast as you can.ok.thx in advance.
name: APPLE FAMILY
key: aple
color: red
emblem: Flame_03
info:
sort: trophy
  logo: https://www.instagram.com/p/CAscQNsHPzy/?utm_source=ig_web_copy_link .png
  description: >
pro players are our crush.  join us to be pro.we likes esports so we host tournament regularly.we likes iOS. and also to eat apple.  we are not partial to any one . so all are can join.so get it out now. play Jurassic.play better.
  social:
    - discord: https://discord.gg/2qrBwMk
    - reddit: https://www.reddit.com/user/Sgi7792ndid/
    - youtube: https://www.youtube.com/channel/UCzuvhbKbj1qU7Bb8OyVvCYg
    - instagram: https://www.instagram.com/applefamclash/
    - youtube: https://www.youtube.com/channel/UCzuvhbKbj1qU7Bb8OyVvCYg
    - email: mailto: minigamersgi11@gmail.com
clans:
  - name: 『JURASSIC™』
    tag: LRVL9CJR
  - name: Apple LEGENDS.
    tag: Y8LCQR9Y

Smlbiobot you can verify these two clans..

Plzz sml do it faster

Ok thx

Plz add it

You have not created yet. Weekend is overed

Plz say me exact date of configuration

Ok

repo: RoyaleAPI/family-config


text:
🛑 Website (rubenfixit.com) is down

In 3664c61, Website (rubenfixit.com) (https://rubenfixit.com) was down:

HTTP code: 503
Response time: 608 ms

Resolved: Website (rubenfixit.com) is back up in fa0f33a after 10 minutes.

repo: RubenFixit/upptime


text:
Add command to toggle chatbox

I love the feature where the Minimap closes when the Chatbox opens. What would be even more neat would be to have a way to open and close the chatbox without using your mouse, for example with "ALT+Z" Also allow users to change the command to any key command they prefer to use.

This can be achieved with the Toggle Chat plugin.

repo: Ruckwad/collapsible-modern-chat


text:
Generate samples at a multiple of channels

I ran into an issue where pausing and unpausing would swap the channels of a stereo source.  It was because the number of samples the pause source generated while paused wasn't a multiple of 2. This has two fixes for that issue.
One Pausable now always generates Zero's in a multiple of the number of channels.
Two Periodic only changes things at a multiple of the number of channels. This might help other similar issues.
I also fixed Delay since it has the same potential issue.
I suspect channel ordering issues can also occur if current_frame_len isn't a multiple of the channels, when called before iterating. Also if the iterator returns a None that's not aligned with current_frame_len. Those could cause the issue with Repeat, Stop and various ways of chaining Sources.

I think I fixed those issues. Long term using something like the sample crates Frame trait would avoid these issues at the type level, but that's a big change to the API.

repo: RustAudio/rodio


text:
Pip installation issue on MacOS 12.0.2

The error message is quite long, so here's the last bit:
SyntaxError: Got error in f-string: Invalid expression: invalid syntax. Got unexpected token ',' at line 408 column 27
        completed_str = f"{completed_ratio:,.{precision}f}"

This happens whether I build a version with ssl or run get-pip.py normally.

Installing pip used to be part of RustPython's CI suite, but it had to be disabled because pip vendors dependencies that RustPython currently doesn't support. The specific problem that you're running into is solved by #3532, but that by itself won't get pip up and running. (The next step along that journey, by the way, is adding mmap to RustPython, tracked in #2059.)

authors: chunkybanana fanninpm

repo: RustPython/RustPython


text:
Add get_int_max_str_digits and set_int_max_str_digits in sys.

Feature
These were newly added in 3.11 and should be relatively straightforward to add, reference: https://docs.python.org/3/library/stdtypes.html#int-max-str-digits
Additions to int_info already exist so just the get/set function should be added. (See PyIntInfo in sys.rs).

It seems like this issue can be closed. Is there more to add with this or is it finished?

Yeah, thanks for noticing! There's some more things to be done for it since it can be set via a command line arg but that should be a separate issue.

authors: DimitrisJim bobby-palmer


text:
Add a license to the repository

I recently came across your project Calculation-of-H2-energy-by-VQE and I noticed that the repository github.com/RyosukeNORO/Calculation-of-H2-energy-by-VQE currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Thank you.
I added a license.

authors: RyosukeNORO cbjuan


text:
[READY] v2.0: Support DSA and ECDSA signing keys

Fixes https://github.com/SAML-Toolkits/ruby-saml/issues/661
Replaces https://github.com/SAML-Toolkits/ruby-saml/pull/683
Depends on https://github.com/SAML-Toolkits/ruby-saml/pull/711, please merge it first.
Currently RubySaml supports only RSA keys. The SAML standard can also support ECDSA and DSA keys. This PR adds support for both:

Validating IdP EC/DSA sigs AND
Using SP EC/DSA signing keys.

JRuby supports DSA but cannot support ECDSA due to this issue.
You may not use EC/DSA keys for encryption. An ArgumentError is raised if you attempt to do so.
It includes the following changes, which are all done in a backward compatible manner:

When generating SP metadata/requests, settings.security[:signature_method] now ignores the "rsa" component of its user-set value and automatically uses whatever type of SP signing public key you actually set (e.g. a DSA key) plus the "sha" component of the value.

(Previously, only RSA was supported, so this doesn't break anything.)

settings.security[:signature_method] supports shortcut values :sha1, :sha256, etc.

Shortcuts :rsa_sha256, :dsa_sha256 etc. also work, but as per above the "rsa"/"dsa" are ignored in favor of the SP public key type.

Similar to above settings.security[:digest_method] supports shortcut values sha1, sha256, etc.
New module XMLSecurity::Crypto is extracted from XMLSecurity::Document
Cleaned-up code, including related to canonicalization

DONE:

[x] Ensure existing tests pass
[x] request_test.rb -- cover all 3 key types
[x] logoutrequest_test.rb -- cover all 3 key types
[x] slo_logoutresponse_test.rb -- cover all 3 key types
[x] metadata_test.rb -- cover all 3 key types
[x] Fix JRuby failures
[x] basic tests (settings, etc.) -- cover all 3 key types
[x] test EC/DSA keys when used by IdP
[x] disable encryption with EC/DSA keys

@pitbulk this is ready for your initial review. I'll give it another pass tomorrow but the core changes and tests are all done (and working 🎉 )

@pitbulk this is ready for your review and merge. (Please review https://github.com/SAML-Toolkits/ruby-saml/pull/711 first)

@pitbulk FYI I am now running this branch in prod without issue.

@johnnyshields, merged this one but rubocop needs to be fixed

authors: johnnyshields pitbulk

repo: SAML-Toolkits/ruby-saml