Round up for the batch size to improve par_iter performance

Objective
The default division for a usize rounds down which means the batch sizes were too small when the max_size isn't exactly divisible by the batch count.
Solution
Changing the division to round up fixes this which can dramatically improve performance when using par_iter.
I created a small example to proof this out and measured some results. I don't know if it's worth committing this permanently so I left it out of the PR for now.
use std::{thread, time::Duration};

use bevy::{
    prelude::*,
    window::{PresentMode, WindowPlugin},
};

fn main() {
    App::new()
        .add_plugins((DefaultPlugins.set(WindowPlugin {
            primary_window: Some(Window {
                present_mode: PresentMode::AutoNoVsync,
                ..default()
            }),
            ..default()
        }),))
        .add_systems(Startup, spawn)
        .add_systems(Update, update_counts)
        .run();
}

#[derive(Component, Default, Debug, Clone, Reflect)]
pub struct Count(u32);

fn spawn(mut commands: Commands) {
    // Worst case
    let tasks = bevy::tasks::available_parallelism() * 5 - 1;
    // Best case
    // let tasks = bevy::tasks::available_parallelism() * 5 + 1;
    for _ in 0..tasks {
        commands.spawn(Count(0));
    }
}

// changing the bounds of the text will cause a recomputation
fn update_counts(mut count_query: Query<&mut Count>) {
    count_query.par_iter_mut().for_each(|mut count| {
        count.0 += 1;
        thread::sleep(Duration::from_millis(10))
    });
}

Results
I ran this four times, with and without the change, with best case (should favour the old maths) and worst case (should favour the new maths) task numbers.
Worst case
Before the change the batches were 9 on each thread, plus the 5 remainder ran on one of the threads in addition. With the change its 10 on each thread apart from one which has 9. The results show a decrease from ~140ms to ~100ms which matches what you would expect from the maths (10 * 10ms vs (9 + 4) * 10ms).

Best case
Before the change the batches were 10 on each thread, plus the 1 remainder ran on one of the threads in addition. With the change its 11 on each thread apart from one which has 5. The results slightly favour the new change but are basically identical as the total time is determined by the worse case which is 11 * 10ms for both tests.

Thanks! I added a small clarification to the docs to be explicit about the rounding. Is there anywhere else the rough maths is mentioned? That is the docs the bevy 0.10 release notes point to and the only place I could find it mentioned.

I've been trying to construct a degenerate case where this might cause a perf regression but cannot think of one at this moment.

The worst case I can come up with is if I change the test to have multiple different par_iter systems in the same set and each has a different time. The batch calculation above assumes the executor will evenly spread out the tasks across threads and while this is true for the single case, it isn't true for multiple. It seems that we can hit a case where two of the longer tasks are assigned the same thread.
Here is an example update which shows a sub-optimal allocation. I struggled to check tracy to show this nicely, I don't know why the long ones are shown under the short task.

Anyway, in this case, this change may make things worse (two 10ms vs two 9ms going back to the original example) but the scheduling noise is quite high so I wasn't able to show this well. It also seems the root of the issue here is with the executor, although I don't know if that's something that could even be fixed (I guess it assumes all tasks are equal?). The only thing I can think of would be somehow tell it all tasks from a single par_iter loop are likely to be a similar duration, a hairy assumption.


Override laravel configuration

I would like to override laravel configuration from the settings.
Something like below in config/app.php
'name' => env('APP_NAME', setting('app.name', 'Laravel')),

When I try I get blank page.
Where is best place to do this?
Thanks

The issue here is setting is being called prior to it being setup.

I understand.. where is best place to put the override of config? in service provider?

You can put it in a service provider yes.


Detect hash in URL and open corresponding tab

I'd like to suggest a feature: when loading a page that has a hash in the URL corresponding the ID of an existing tabpanel, it would make sense to automatically show this tab.
What do you think?

Hello!
Thanks for the suggestion.
We try to keep this lib as tiny as possible and think this is not the aim of it.
The lib exposes all you need to achieve that on your own.


[TS] Typescript compatibility

While there are several versions that are Javascript-compatible, the "pure Typescript" version fails to type check.

It is possible to use specific versions, e.g. @apache-arrow/es2015-umd.

It is also possible to use the "umbrella" import apache-arrow.

However, the @apache-arrow/ts import fails to type check with tsc.

A sample of errors from one file:
../../node_modules/@apache-arrow/ts/schema.ts:46:49 - error TS2345: Argument of type '(Field<T[keyof T]> | undefined)[]' is not assignable to parameter of type 'Field<any>[]'.
  Type 'Field<T[keyof T]> | undefined' is not assignable to type 'Field<any>'.
    Type 'undefined' is not assignable to type 'Field<any>'.

46         return new Schema<{ [key: string]: K }>(columnIndices.map((i) => this.fields[i]).filter(Boolean), this.metadata);
                                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

../../node_modules/@apache-arrow/ts/schema.ts:64:58 - error TS2532: Object is possibly 'undefined'.

64                 metadata: mergeMaps(mergeMaps(new Map(), curFields[i].metadata), f2.metadata)
                                                            ~~~~~~~~~~~~

../../node_modules/@apache-arrow/ts/schema.ts:129:22 - error TS2532: Object is possibly 'undefined'.

129         const type = field.type;
                         ~~~~~

(Is there any plan to have the typescript code eventually type-check with tsc ?)

I can't even import from @apache-arrow/ts in version 7.0.0 because of the .js paths. I made a comment in the commit that added them. Not sure if I'm doing something wrong.
Notably<EMAIL_ADDRESS>gives me all the TS integration I expect so not sure if I even should have tried to import the pure @apache-arrow/ts package anyway.

The errors you're seeing are due to the fact that your tsconfig is different than the one used by this project.
It looks like your tsconfig has noUncheckedIndexedAccess turned on, which adds a | undefined to every index access.
One way to fix this is to have a tsconfig that is strictly looser than the one used in this repo.
However, almost always that isn't the right solution - you don't in fact want your tsc to compile the TypeScript of your dependencies. Rather your dependencies should publish pre-compiled *.js and *.d.ts files and your tsc should leave them untouched (other than checking that your code is compatible with the *.d.ts declarations).
That's exactly what<EMAIL_ADDRESS><EMAIL_ADDRESS>etc. already do, so you should use those! @apache-arrow/ts should probably not be published.


Binding Values to Prepared Statements Doesn't Work as Expected

Explain your user case and expected results
The doc's:
indicate you can use syntax like this:
var users []User
db.Raw("UPDATE users SET name = ? WHERE age = ? RETURNING id, name", "jinzhu", 20).Scan(&users)

But, the values don't seem to bind properly in my tests.

I was looking at the wrong doc's - this is where i should've been looking:
https://gorm.io/docs/sql_builder.html#ToSQL
You need to use a callback to get the raw SQL bound to the values.
Side note - if you're using a struct as a query (instead of binding raw text to values) - "default values" (exp: false for boolean, zero for number, etc) will not get injected into your query.
exp:
recordQuery := &models.FooRecord{Score:0, Complete:false}
db.Model(&models.FooRecord{}).Where(recordQuery)

The resulting SQL will have no where clause - so you need to use the raw text version if you want to query with default values - exp:
db.Model(&models.FooRecord{}).Where("Score = ?", 0).Where("Complete = ?", false)


Fix the local development environment instructions

Omit the docker-machine/ path prefix from the last cd command. Having followed the instructions the user should already be in the docker-machine directory.

Agreed. LGTM


Missing many entities like temperature & uv, but have about 30

It looks like this integration should provide a large number of sensors (I checked the box) but I'm seeing lots from the front page that are missing. I'm wondering if this is a common problem? Specifically I don't see temperature or UV index, but I do see things like battery, beaufort, distance last lightning strike, etc.

Hi,
It is hard to help without more info. Do you see anything in the Log files? Have you checked the WeatherFlow app, to see if the missing data is present there and the station is online?
If you get some data, then it seems you have a valid Station ID and Token, but the missing data is not supplied by the Weather Station.
I have found that occasionally that I needed to restart the Weather Station itself, especially if your Hub has a green light, then it might be a defective Tempest device or it simply needs a restart.
Please give me more info and I will do my best to help.

No new information, so this issue will be closed.


Can't open remote on bitbucket

GitLens Version: 9.5.1

VSCode Version:  1,32,3
OS Version: Windows 10

Steps to Reproduce:

Connect to a Bitbucket repo, e.g.

git remote -v
origin  https://bitbucket.tds.td.com/scm/tbsm/tbsm-vision-db.git (fetch)
origin  https://bitbucket.tds.td.com/scm/tbsm/tbsm-vision-db.git (push)

Click on :Open Branch On Remote

404 since the URL is incorrect
(tries to go to https://bitbucket.tds.td.com/projects/scm/repos/tbsm/tbsm-vision-db/commits?until=develop)

Correct url is: https://bitbucket.tds.td.com/projects/TBSM/repos/tbsm-vision-db/commits
The problem is the "scm/" inserted in the bad url.

Also having this issue with Bitbucket Server v5.16.0
scm appears in the git URL, however it is not used at all in formulating the url for browsers to visit. Even with the gitlens.remotes configuration, it is not possible to construct a dynamic URL that works for each of my repos, and I have to hardcode it per-project.
Example:
${repo}: scm/myproject/myrepo
${repoPath}: myproject/myrepo
${repoBase}: scm

An example URL I need is:
https://bitbucket.customurl.com/projects/myproject/repos/myrepo/commits/a1b2c3
h1. Solution
Ignore /scm when translating the remote names into browser-friendly URLs.

PR #767 submitted

When is the next release planned? I'm so tempted to hack a local copy version of the plugin just for this fix...

Most likely tonight or tomorrow night.

Most likely tonight or tomorrow night.

Excellent!
BTW, thank you for your work! It's appreciated!


Is the following method can solve the limit of memory?

Hi, I want to use large images set to retrain face landmark model , but I have not enough ram.  I think if I can use the method of training CNN to train face landmark model.
In particular, I choice a batch of images from large dataset randomly, and then call the function dlib.train_shape_predictor in while loop. Stop the loop until epoch reach a special number.  I doubt if the method workable. I want to listen to  opinion from you. Thank you.

I'm dealing with a similar issue, so let me share what I've learnt from reading the code:

The shape predictor uses a forest of regression trees (paper here), so if you specifically want to use a CNN you'll have to figure it out yourself.
train_shape_predictor trains and returns a new shape_predictor object from scratch, so you can't repeatedly call it for each batch/iteration.
If you can fit all the objects (landmarks and bounding boxes) in memory, then it looks like the image data is only used here: https://github.com/davisking/dlib/blob/master/dlib/image_processing/shape_predictor_trainer.h#L330 - while one could read images as needed in this loop, I need to read more of the code to figure out how memory-bound other parts are when you have a large dataset.

I think that because this is a forest of regression trees we should be able to have watch tree created from different partitions of the data?

During each iteration the trees are built on the entire dataset.  So that
sort of thing isn't going to work with this algorithm.
The deeper issue is that you shouldn't be running out of ram. A computer
with 32GB of ram ought to be plenty for normal sized datasets.    If they are
super huge downsample them first.  There isn't any reason to use crazily
high resolution images as training data with this algorithm.

Thanks for the clarification Davis.
I'm working with a small filtered subset of Open Images (full dataset is 9 million images). While images with human faces are only part of the overall dataset, this is still 300k images downsampled to have a max dimension of 1024px.
My aim is to create a massive and open set of face annotations that isn't unduly restricted like ibug, using a bootstrapping process from a smaller set of human labelled annotations.

Oh, sweet.  That would be pretty awesome.
Still though, just get more RAM.  For instance, if each image were 700x700
pixels and grayscale (the algorithm ignores color, so load images as
grayscale) then you could load over half a million images onto a machine
with 256GB of RAM.  You can just rent a VM on google cloud or AWS with that
to do the training if need be.

Thinks for your pretty answer. As ferrouswheel said, we can not solve the problem with loop method. I solve the problem using the computer with 32G ram.


[Keyring] Update to gridplus-sdk v2.1.0

Depends on gridplus-sdk/#418

[ ] test & review sign/decode SDK function (@douglance)
[ ] publish to NPM (@douglance)


Mavlink command retransmission

This adds a class to allow for retransmission of outgoing commands.
The sent commands are kept in a timestamped list to check if they are acked as required by the mavlink protocol.
If they are not acked within a timeout, they can be retransmitted.
Only commands to other component IDs are actually sent out and broadcast commands are also ignored.
Before merging, this will need a cleanup of all the debug printfs.

Awesome!


Unbanning users is broken

Trying to unban fails with Cannot leave user who was banned
Relevant Riot-Web bug is vector-im/riot-web#3091
More information in matrix-org/synapse#1860

Validated


KAFKA-6205: initialize topology after state stores restoration completed

Initialize topology after state store restoration.
Although IMHO updating some of the existing tests demonstrates the correct order of operations, I'll probably add an integration test, but I wanted to get this PR in for feedback on the approach.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage, and CI build status
[ ] Verify documentation (including upgrade notes)

Cherry-picked to 0.11.0 branch.


Create block_breaker.py

Added classic block breaker game

Not working...
Traceback (most recent call last):
  File "/Users/sparsh/Code/python/py-games/src/block_breaker/block_breaker.py", line 19, in <module>
    sound=pygame.mixer.Sound("mario.wav")
FileNotFoundError: No file 'mario.wav' found in working directory '/Users/sparsh/Code/python/py-games/src/block_breaker'.

Ok I will add all files and create a new PR

You should update this PR instead of creating new one.

Updated  PR

Thanks for the contribution, here are some issues that needs to be solved before I can merge it though

The block sometimes doesn't collide with blue ones as you can see below
It just crashes when game is over
Blue blocks overlap a lot and everything seems crowded
Player's motion should be continuous

Leave it bro! Dont have so much time


How to compile x86

hi, i want build x86 of vs2015,i have Configurated opencv (vc14, x86) and boost boost_1_62_0-msvc-32.0-.exe，but the compilation failed，what else do I need to do next?
and some others .dll and  .lib in ./external，does its affect compilation？

I think you may need to debug the compilation yourself. My biggest hint is to make sure that you are not mixing any x64 and x86 libraries, including all dependencies.


Fix description of touch input position

InputEventScreen{Touch,Drag}.position is in viewport coordinates.
Related engine codes are all using get_global_transform_with_canvas() to transform position to local coordinates.

Thanks!

Cherry-picked for 4.2.2.

Cherry-picked for 4.1.4.


[Suggestion] Variable theme by time of day (aka daymode / nightmode)

Purely esthetic, but it would be nice to i.e has the yellow/black (aka cab) theme during daytime, and a darker one (ie blackboard) during nighttime.

I was thinking about that. I have two ideas on how to implement it, either by creating user-defined scheduler or by leveraging isDay property from the weather api. User could then choose which theme to apply for day and night time.

Correct me if I'm wrong, but right now the page does not auto refresh the date/time field either.

It doesn't. You can open new issue and it should be fixed with next release.

Another option could be to use the CSS media feature "prefers-color-scheme" whitch would change the color based on the browser setting.
I just implemented it (after @pawelmalak showed me how to change the theme colors via custom CSS) and as I configured my OS to change the window style based on daytime/nighttime the browser and the website theme change their style too.
My custom CSS looks like this:
body {
  --color-primary: #eceae8 !important;
  --color-accent: #f65275 !important;
  --color-background: #041e42 !important;
}

@media (prefers-color-scheme: dark) {
  body {
    --color-primary: #eceae8 !important;
    --color-accent: #f65275 !important;
    --color-background: #041e42 !important;
  }
}

@media (prefers-color-scheme: light) {
  body {
    --color-primary: #041e42 !important;
    --color-accent: #f65275 !important;
    --color-background: #eceae8 !important;
  }
}


pickle error when running train.py

Using python 3.5.2 and tensorflow 1.3.0.
I'm getting this when running train.py:
Traceback (most recent call last):
File "C:\Data\Apps\eclipse-jee-oxygen-R-win32-x86_64\plugins\org.python.pydev_<IP_ADDRESS>708151115\pysrc\pydevd.py", line 1621, in
main()
File "C:\Data\Apps\eclipse-jee-oxygen-R-win32-x86_64\plugins\org.python.pydev_<IP_ADDRESS>708151115\pysrc\pydevd.py", line 1615, in main
globals = debugger.run(setup['file'], None, None, is_module)
File "C:\Data\Apps\eclipse-jee-oxygen-R-win32-x86_64\plugins\org.python.pydev_<IP_ADDRESS>708151115\pysrc\pydevd.py", line 1022, in run
pydev_imports.execfile(file, globals, locals)  # execute the script
File "C:\Data\Apps\eclipse-jee-oxygen-R-win32-x86_64\plugins\org.python.pydev_<IP_ADDRESS>708151115\pysrc_pydev_imps_pydev_execfile.py", line 25, in execfile
exec(compile(contents+"\n", file, 'exec'), glob, loc)
File "C:\Users\cpetroaca\workspace\tensorflow\Siamese_Networks\train.py", line 111, in
vocab_processor.save(os.path.join(checkpoint_dir, "vocab"))
File "C:\Users\cpetroaca\AppData\Local\Programs\Python\Python35\lib\site-packages\tensorflow\contrib\learn\python\learn\preprocessing\text.py", line 213, in save
f.write(pickle.dumps(self))
_pickle.PicklingError: args[0] from newobj args has the wrong class

i meet the same problem, do you have solve it?

No. I don't have much knowledge of Python and I don't want to start digging through the code now. I just needed something that works as is.

have you find another effective codes?   i have solve the problem. you can modify input_helpers.py
and modify the code as this.
    vocab_processor = learn.preprocessing.VocabularyProcessor(max_document_length)
    vocab_processor.fit_transform(np.concatenate((x2_text,x1_text),axis=0))
    # vocab_processor = MyVocabularyProcessor(max_document_length,min_frequency=0)
    # vocab_processor.fit_transform(np.concatenate((x2_text,x1_text),axis=0))

Cool Thanks. I'll try that.
I also tried this guy's implementation : https://medium.com/@eliorcohen/implementing-malstm-on-kaggles-quora-question-pairs-competition-8b31b0b16a07. There's a link to a github repo in the beginning.
It does sentence similarity which I am interested in. No phrase similarity.
I got it training for around 10 epochs out of 50, which took around 2hrs and it got to 80% accuracy on it's validation set.

Thank you for you reply,  what you recommend is exactly what I want. I want to find a method that does sentence similarity. I'll try it.

Did you find any other implementation that you can share?

you can see https://github.com/nelson-liu/paraphrase-id-tensorflow. It may help you.  But I haven't tried it. I'm just beginning to study the sentence similarity.
I trid the project you shared, but it lacks of the code which save the model, or reload the model ,  or input another new sentence pair  and output the similarity.

That's true but those can be easily added once the core model is functional.

hi
have you load the model which has been trained to predict the question pairs. I did that, but the same code, The results of each run are different. Did you meet the same problem, and did you solve it?
Looking forward to your reply
Thank you


accidental smart quotes in UltrasonicPrintRange.py

otherwise
SyntaxError: invalid syntax

in line 7.

Sigh, computers...
Thanks for spotting it.


Revert "lasers"

Reverts Woap/croc-hunter-jenkinsx-serverless#1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

:star: PR built and available in a preview environment woap-croc-hunter-jenkinsx-serverless-pr-2 here

/approve Ok

/hold

/approve


simpler implementation / promises support

I wrote the initial code quite some time ago and a lot of it is frankly unnecessary. I've just maintained it to fix bugs but might as well use a few hours to do it a bit better.

lets only run the actions when ready.. not yet


Accessibility testing

Footer link colour (and general link colour I think?) isn’t contrast compliant.

Improved on this.


fix: Update copy for InstitutionProfile page

Closes #194
Changes
Language updates

Financial institution => Email domain(s)
Affiliate information => Section intro

Screenshots

@meissadia
There are some additional content updates to this page. Would you rather handle those as a separate PR?

@natalia-fitzgerald Updated with content as of today (1/25/24).  This is not deployed to a DEV environment.


feat: waitUntilFirstFrameRendered; perf: Player.dispose event loop clean-up; refactor: API improvement etc.

package:media_kit

fix: Player.dispose event loop clean-up
refactor: Player implementation clean-up
feat: Initializer.dispose

feat: InitializerIsolate.dispose
feat: InitializerNativeEventLoop.dispose

feat: PlatformPlayer.waitForVideoControllerInitializationIfAttached

package:media_kit_libs_android_video

build: bump dependencies

package:media_kit_libs_android_audio

build: bump dependencies

package:media_kit_native_event_loop

feat: MediaKitEventLoopHandler::Dispose
perf: switch to std::condition_variable(s) for synchronization

package:media_kit_video

feat: VideoController.waitUntilFirstFrameRendered
refactor: clean-up package structure
refactor: remove VideoController.dispose
refactor: VideoController.create -> VideoController constructor
fix(android): add av1 to hwdec-codecs
fix(android): use --vo=gpu + --hwdec=mediacodec-copy /w enableHardwareAcceleration

I don't have any issues on Windows, GNU/Linux & Android.

I also merged #128 progress internally; since it makes some notable changes to package structure. However, it's not "plugged".


Need Help: Compilation problem with Shogun

Hi,
I am trying to compile Shogun and while configuring it I enables python and
octave variants as well. Now it compiles uptil 84% and then an error appears
while compilation uses octave files. Specifically it is unable to find
"config.h". Here is the specific error:
[ 84%] Building CXX object
src/interfaces/octave_modular/CMakeFiles/octave_modular.dir/modshogunOCTAVE_w
rap.cxx.o
In file included from /home/mani/Installations/shogun-
3.2.0/build/src/interfaces/octave_modular/modshogunOCTAVE_wrap.cxx:153:0:
/opt/octave3.8/include/octave-3.8.0/octave/oct.h:31:20: fatal error:
config.h: No such file or directory
compilation terminated.
make[2]: ***
[src/interfaces/octave_modular/CMakeFiles/octave_modular.dir/modshogunOCTAVE_
wrap.cxx.o] Error 1
make[1]: ***
[src/interfaces/octave_modular/CMakeFiles/octave_modular.dir/all] Error 2
make: *** [all] Error 2
I compiled octave myself (just a week ago) and its working fine. Earlier I was having a version of octave from my ubuntu distribution. I am using Ubuntu 12.10 and it contains octave3.6 while shogun needed octave3.8. So I compiled it my self.
Please help me resolve this issue.

It is a bit late but it appears that this still has not been resolved. I tried to build shogun 4.0 octave modular against octave 3.8.2 on Ubuntu 14.04 and got the same error
".. fatal error: config.h: No such file or directory"
Turned out it was because of this bug http://savannah.gnu.org/bugs/?41027, or rather, the attempt to fix it. Shogun octave built successfully after rebuilding octave with the patch from the thread above.

I guess I can close as this patch is very old.


Merge pull request #1 from OpenRefine/master

update from master

sorry I meant to do the opposite (update my personal from the master): https://github.com/magdmartin/OpenRefine/pull/2

I don't understand why one would use a pull request for this (in either direction).  If it's a repo that you control, you can merge things directly.  PRs on repos that you have commit privileges to are only normally used to allow for review of the proposed change (in which case you wouldn't merge your own PR).


Add built-in functions "not contains"

Is it possible add a  built-in function "not contains"?

You can emulate this combining the contains() function with a not-expression:
search( [? !contains(@, `2`)][], [[1, 3, 5], [2, 4, 6], [1, 2, 3]]) -> [1, 3, 5]

Perfect, thank you @springcomp!


React-router-dom 4.2.0 Error, BrowserRouter.js and HashRouter Error, Module not found.

Version
4.2.0
I am trying to build a web application using React-router-dom 4.2.0 on Jenkins. After running the website I am getting this error.
ERROR in ./~/react-router-dom/BrowserRouter.js
| Module not found: Error: Can't resolve 'warning' in '/usr/src/app/node_modules/react-router-dom'
  | @ ./~/react-router-dom/BrowserRouter.js 5:15-33
  | @ ./src/js/App.js
  | @ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 babel-polyfill ./src/js/App.js
ERROR in ./~/react-router-dom/es/HashRouter.js
| Module not found: Error: Can't resolve 'warning' in '/usr/src/app/node_modules/react-router-dom/es'
  | @ ./~/react-router-dom/es/HashRouter.js 7:0-30
  | @ ./~/react-router-dom/es/index.js
  | @ ./src/js/components/preferences/Preferences.js
  | @ ./src/js/App.js
  | @ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 babel-polyfill ./src/js/App.js
I am importing that package with this line:
import BrowserRouter from 'react-router-dom/BrowserRouter';

This isn't a bug in React Router. Your CI isn't installing the warning npm package for some reason. That doesn't have anything to do with this library.

@timdorr It looks like warning is being imported by files in the react-router-dom package, but it's not a dependency in the react-router-dom package.json (it's only in the react-router package.json)

Ah, you are correct. Adding react-router or warning as a dep on your project will fix this for now. I'll have to push out a fix tonight.

Awesome thank you!!

react-router-dom 4.2.1 should fix this for you.

As one "typescript + webpack" project, I fix this error by modifying the config of resolve field  in webpack.config.js, by changing its extensions from extensions: ['.ts', '.tsx' ] to ['.ts', '.tsx', '.js', '.jsx' ]

hello sir , i had try react-router-dom 4.2.1,but not work
ERROR in ./node_modules/react-router-dom/es/index.js
Module not found: Error: Can't resolve './BrowserRouter'
ERROR in ./node_modules/react-router-dom/es/index.js
Module not found: Error: Can't resolve './HashRouter'
ERROR in ./node_modules/react-router-dom/es/index.js
Module not found: Error: Can't resolve './Link'
"dependencies": {
"axios": "^0.18.0",
"express": "^4.16.3",
"react": "16.2.0",
"react-dom": "16.2.0",
"react-hot-loader": "next",
"react-router": "^4.2.0",
"react-router-dom": "4.2.1",
"serve-favicon": "^2.5.0"
}


Correct references to Ruby 2.2 in README

The latest dry-initializer has a gemspec requirement on Ruby 2.3+ and the only 2.3+ is tested in Travis.

Thank you!


Document how to port from oldname

Hi,
Could you document how to port from old name ?
Thanks

It should be just renaming to acorn-import-attributes, right?


memory leak when CRL file hot-reload with envoy v1.27.0

Title: memory leak when CRL file hot-reload with envoy v1.27.0
Description:
We have been using the Enovy as loading balancer in Kubernetes for multiple years. The listener filter type is HttpConnectionManager. With Envoy v1.27.0 release, we are happy to see the CRL rotation function is added. But, we found the memory leak whenever the CRL file rotation triggered.
The image we used is : The docker image is envoyproxy/envoy:v1.27.0.
Repro steps:
When the new CRL file is available, we copy it to the location /etc/ssl/crl/crl-1.pem, where the folder(etc/ssl/crl) is watched .
We will see the memory leak immediately after running below command which triggers the CRL hot reload.

ln -sf /etc/ssl/crl/crl-1.pem /etc/ssl/crl/crl.pem

The memory size leaked align with the crl.pem file size.
If the crl.pem is 17Mi, the memory leak will be about 130Mi
If the crl.pem is 145Mi, the memory leak will be about 1Gi.
Our regular CRL file size is larger than 100Mi, the memory leak is obvious.
If the DONOT trigger CRL file rotation, with v1.27.0, the envoy is working fine.
Besides the memory leak, we found the new file is taking effective. So the function is working fine.

Without traffic, do not see this issue. Have to run traffic to see this issue.

One thing to keep in mind is that an old version of the CRL will stay in memory until all TLS connections that were established when it was active are closed.
You could force this in your configuration by setting max_connection_duration to ensure that the old versions are not kept in memory forever.

One thing to keep in mind is that an old version of the CRL will stay in memory until all TLS connections that were established when it was active are closed.

@ggreenway It matches our observation. If envoy handles short live connections, the memory will back to normal quickly with CRL hot-reload. But all our traffics are GRPC long live sessions. Technically, those sessions can be active forever for steaming.
Configuring max_connection_duration is not recommended for my applications, since it will terminate the steaming when data transmission.
The CA, and CRL are used during SSL handshake. The existing active sessions already been validated. Why Envoy keep the old version CRL in memory?

The CA, and CRL are used during SSL handshake. The existing active sessions already been SSL validated. Why Envoy keep the old version CRL in memory?

That data is held by the ssl context in boringssl. That context is needed for the lifetime of the connection, and the contexts are generally not modified once they're created; that's why the memory isn't free'd earlier. It may be possible to free it sooner though; that would need some investigation.

Hi @ggreenway ,  I had lots google search and discussion, could not find a workaround for this use case. Could you or someone investigate if Envoy can release the memory sooner?
Thanks

Sorry, I will not have time to investigate this.


Just a documentation request

@mijaros ...
Hi Miroslav,
First of all thanks for an awesome library.  Please take pity on us n00bs and change the README to mention the requirement to run the code as root -- my code was failing quite spectacularly, and I was blaming everything under the sun (or more accurately under ubuntu/debian, thinking I had an old soft float image, conflicting library, etc) until I found your comment to that effect.
Thanks,
Nick

Hi,
here it is: 2ae0bf76065b2870a092538c67f62d9811559d27


Ports with new streams won't open

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug

Right after initial installation I added several streams for port forwarding to services in my setup.  I had no trouble til about a month ago.  Now when I add a new stream (and the accompanying posting of the ports to the docker-compose.yml file) it will record it and it shows good, but when I try to access the port it is closed.  I have checked firewalls, ufw, iptables, hosting provider and everything else I can think of and no joy.  I can ping the port internally but it won't connect to the ip of the service nor will it allow it to go from outside wan to inside ip4 address.  Netstat shows it is listening on the ports not open.  Not sure where to go from here.  Any advice or suggestions would be appreciated.
Nginx Proxy Manager Version

v2.9.20 © 2022 jc21.com. Theme by Tabler
To Reproduce
Steps to reproduce the behavior:

Go to 'streams'
Click on 'new stream, add a new stream'
Scroll down to '....'
See error No joy on opening port through to service.  Following same steps since installation.  Worked til about a month ago.

Expected behavior

Screenshots

Operating System

Ubuntu 20.04 server on proxmox container
Additional context

I have installed a fresh copy of proxy manager, created some streams, my very first created stream worked fine, anything after, didn't work at all.
I have then removed all my streams, however, the very first stream was still working, even after deleting all of them.
It seems that there is a bug, and at some point something hangs and no changes to streams give any effect.


Update README.rst to update python build badge and add number of downloads badge

Codecov Report
Patch and project coverage have no change.

Comparison is base (16286b5) 68.27% compared to head (e7a16ac) 68.27%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #367   +/-   ##
=======================================
  Coverage   68.27%   68.27%
=======================================
  Files          27       27
  Lines        3603     3603
=======================================
  Hits         2460     2460
  Misses       1143     1143

Flag
Coverage Δ

unittests
68.27% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


test(scanv4): Add operator-based install scenarios

Description
This PR adds a multi-namespace, operator-based installation with Scanner V4 enabled.
Checklist

[x] Investigated and inspected CI test results
[x] Unit test and regression tests added
[x] Evaluated and added CHANGELOG entry if required
[x] Determined and documented upgrade steps
[x] Documented user facing changes

/retest

/retest

/retest


Move git-push to the background after the git-commit

Move git-push to the background after the git-commit

Add a success state after commit
Add mechanism to inform the client about the status of the push - web hooks maybe?

Stale, Closing


Ignore click event following a drag

This prevents dragging on the canvas from changing the selection
Fixes #94

Coverage decreased (-0.09%) to 39.41% when pulling 2e9d5a13e2cbb9de8d5733d760e64d8d016bc6bd on jlfwong/pan-deselection into 970f1b2107f5e5a8b8e3e8352a4d8ee4abded66a on master.


Update requirements and image for UATS python 3.11 scipy 1.8

Closes: https://github.com/canonical/charmed-kubeflow-uats/issues/71
Closes: https://github.com/canonical/charmed-kubeflow-uats/issues/82
Closes: https://github.com/canonical/charmed-kubeflow-uats/issues/77
This PR partialy resolves https://github.com/canonical/charmed-kubeflow-uats/issues/73. Here we also need requirements for python 3.11. The only difference is the image for running the notebook.
Update requirements for python 3.11.6
Update UATs image to scipy 1.8
Uses pip install -r requirements in the notebook cells
Tested here and second run here.
Note:

I had to pin the kubeflow-training to version 1.6.0 because later version are breaking the imports in traing UATs. I have opened an issue.

Hey @orfeas-k thanks for review.
Here are two runs on EKS first  and second . Both cases 1.8 was working and this PR is about 1.8 so we can merge. the latest/edge is having the same katib error reported here.
To your python comment. We should not change that because this is changing python in the image for notebooks. The guide is talking about the python for tox environment which stays the same so 3.8 :)


Corrected Heading for the Manual on the Readme

Corrected heading tag for the manual on the Readme. Link is already there, so you could close #226.

Coverage remained the same at 84.833% when pulling b9130725d29660798171563bac38e64c37fb3597 on babetoduarte:master into 4071a708c3d3a449dbe4faf26394cf647305de6f on Mindwerks:master.

Codecov Report

Merging #254 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #254   +/-   ##
=======================================
  Coverage   79.49%   79.49%
=======================================
  Files          28       28
  Lines        3468     3468
  Branches      651      651
=======================================
  Hits         2757     2757
  Misses        526      526
  Partials      185      185

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 4071a70...b913072. Read the comment docs.

Thanks!


Focus Terminal Instances when switching between them

When you have multiple instances of terminals running and you switch between them, the focus should switch automatically to the selected terminal instance so you could start typing.

That's what happens AFAIK, pls include a gif with screen cast mode on if you're seeing otherwise

hope this works

I just noticed you need to double click the Instance, then it focuses. Makes sense to me.
I'm closing this issue. Thanks.

You can change that with terminal.integrated.tabs.focusMode if you want to

OK, that's even more awesome, thanks!


refactor the t36 test case

Changes to the T36 test case:

Created the istio-system2 namespace
Deployed the Multiple SMCP on that new namespace
Deleted Istio Operator pod, it will recreate the Webhook back again.

@luksa: Updated the code with your comments.
Thanks for checking the code.


fix: yaml issue causing issues with deployment of other PRs

Description
This PR resolves a lint issue which is blocking deployment of other code changes.

merged.

Closing this because the changes were represented in a different PR that was used to resolve migration to the new environment v2-2024.


Make stripe-ruby-mock an optional dependency

The currently released version of stripe-ruby-mock has a very specific constraint on the stripe gem (stripe <= 1.58.0, >= 1.31.0) that is causing bundler to drop from 3.x to 1.58.0. While the new testing functionality looks interesting, we shouldn't be forced to use an old version of Stripe just to test our webhooks. By having it be a required dependency we are also forced to load the mocking library into our production application where it won't be used and could have other side effects that we're not aware of.
The easiest way forward would be to remove stripe-ruby-mock from the gemspec. Users of the gem will need to require "stripe/testing" which will then require stripe-ruby-mock and fail if the gem is not installed. We then have the ability to add stripe-ruby-mock to our bundler test group to get that functionality when needed.
On a side note, I would expect it to be require "stripe/rails/testing" since the stripe gem should own the require "stripe/*" namespace and stripe-rails owns the require "stripe/rails/*" namespace.

Thanks @gaffneyc those are very good points! Getting stripe-ruby-mock onto production would be a very bad idea. Let me think about this a bit more before getting a fix out.

Just a note that I might actually have to fix namespacing because Stripe might merge this https://github.com/stripe/stripe-ruby/pull/473


⚠ Bofrost.fr has degraded performance

In 62d3e58, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 944 ms

Resolved: Bofrost.fr performance has improved in 1beeea7 after 6 minutes.


test/testdata: regenerate JSON files

It looks like the keys in nix show-derivation output were not sorted (but stable), and some Nix version update changed that.
Regenerate the JSON files to prevent CI from failing.

Yes, it looks like the golang serializer is not spitting out ordered keys, even though the golang docs suggest it should?!

I sent a workaround.
    encoding/json doesn't serialize struct keys in alphabetical, but struct key definition order.

    This means we can't compare the raw bytes produced from Nix in the
    fixtures directly, but instead need to parse both in and compare again.

Maybe there is a way to properly get encoding/json to sort, which would be preferrable?

I added a comment to the cmd entrypoint, and will merge this for now. Ideally, we'd probably replace encoding/json with something that does sort /all/ keys, but the CLI output being bit-by-bit compatible with what Nix does was never the intention of this anyways - and the test now checks for the JSON being equal.


feat: defineRequestMidleware, defineResponseMiddleware and rename object synctax hooks

🔗 Linked issue
Context: Find most of DX issues while working on a example for https://github.com/unjs/h3/issues/501
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This PR introduces two new define utils defineRequestMidleware and defineResponseMiddleware (with RequestMiddleware and ResponseMiddleware) types.
These utils are then used for event handlers with object syntax but can also used independently to define such hooks outside of the event handler in a composable way (imagine compressionMiddleware as util)
Also in order to keep consistency with global hook options and readability, the hooks are renamed from before/after to onRequest/beforeResponse.
Also we support non Array format for most consistency with top level
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

These middleware are same as current before/after. Do you mind to push any required placeholder generic?

I think prefixing with underscore would be enough to mark them as internal types for now.


PHP Fatal error:  Can't use function return value in write context in ...

Hi
When using the PHP lib from CLI with the crontab, I have the following error :
PHP Fatal error:  Can't use function return value in write context in /home/xxxxxxxx/app/vendor/sparkpost/php-sparkpost/lib/SparkPost/SparkPost.php on line 118
Line 118 of the SparkPost.php file is :
if (!isset($settingsConfig['key']) || empty(trim($settingsConfig['key']))){
I can't explain this error.
Can you help me with this ?
PS : the cron is executed with PHP7 CLI
Thanx

My bad.
I use php70 in the cron command, but logging the PHP version in my executed script gives me PHP 5.4, which is not compatible.
Now I have to find out why the 5.4 version is used instead of the 7.


Layout and sidebar improvements

Removed weird padding and margins, fixed sidebar behavior, and fixed issue where sidebar disappeared on small display widths.
Closes #139

Spun it up locally, and LGTM :rocket:


No bottom border when an accordion has a single card

There was once this issue that happened when there were nested cards inside an accordion not having a bottom border, https://github.com/twbs/bootstrap/pull/27881.
Even though the nested cards' problem was solved, there's still a problem when the accordion itself only has one card. Please try the following to check the problem:

              Accordian Title

Lorem Ipsum 1

Lorem Ipsum 2

Lorem Ipsum 3

Bug reports must include a live demo of the issue. Per our contributing guidelines, please create a reduced test case via CodePen or JS Bin and report back with your link, Bootstrap version, and specific browser and Operating System details.
This is an automated reply

Just to say that I mistakenly used bootsrap 4.1.3 in the codepen code I provided. I've corrected it and it now has bootstrap's current version (4.3.1).

You can work around this with card-body border-bottom.
Let's wait for @twbs/css-review to chime in


Remove deprecated extensions API group in document

The extensions API group is deprecated.
Changes in ABAC page

Use networking.k8s.io instead of extensions in apiGroups field.
Add more examples.

Changes in ABAC page

Remove extensions API group for Jobs

Ref: kubernetes/kubernetes#36355, kubernetes/kubernetes#38614

Change deployments resource to events

The extensions API group for deployments is deprecated.
Ref: kubernetes/kubernetes#70672

/lgtm

@kubernetes/sig-auth-pr-reviews does this look right?

Hi @liggitt,
I've force-pushed the changes according to your suggestions, could you take a look? :bowing_man:

/lgtm

/approve


Update documentation on commands

Currently there is no documentation regarding the new scamguard slash commands for reporting/checking bans. This should be properly documented on the website!

I would like to help out! Please give me more information!

Would comments in code documenting function code flow be wanted in this as well, or a markdown file / repo wiki page?

I think it's going to make more sense to have it as a markdown / wiki page. idk if I want to also have a wiki involved as well.

Closing, page created here https://github.com/SocksTheWolf/AntiScamBot/wiki/Command-Documentation


[7/x] Sort form on adding a new tier

What does this PR do and why?
Adds a sorting function so that new tiers can be inserted and removed and ensure they will be in the right staking range
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #1586

PR #1587

PR #1588

PR #1594

PR #1595

PR #1596

PR #1597   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: 5peKCVtRG7hIeC6BSrBI

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: 0agyc4HkFZeecGjH4TTU

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: I3QazDznUYZOKiV2bxLz


Added generic functionality for adding extensions to X509

Tested this with basic script (where server was verifying the below extensions) and this works.

getCertUnsigned :: IO X509
getCertUnsigned = do
deviceCSR <- readFile "new_csr.pem"
x509Object <- readX509Req deviceCSR
issuerCert <- getIssuerCert
x509Cert <- makeX509FromReq x509Object issuerCert
setSerialNumber x509Cert 112312
now <- getCurrentTime
setNotBefore x509Cert $ addUTCTime (-100) now
setNotAfter x509Cert $ addUTCTime (365 * 24 * 60 * 60) now
addExtensionToX509 x509Cert 87 "CA:FALSE"
addExtensionToX509 x509Cert 83 "digitalSignature, keyEncipherment"
addExtensionToX509 x509Cert 85 "serverAuth, clientAuth"
pure x509Cert

Thank you. It's now available in HsOpenSSL-<IP_ADDRESS>


Contrast issue with WISE blue

Ticket https://taskman.eionet.europa.eu/issues/257841

@eea-jenkins build all


cpuinfo: Add maintainer

cpuinfo: Add maintainer

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (cpuinfo) have been updated in this PR. Please review the changes.

@mmorel-35 Can I add you also as a maintainer for this module?


nucleo-h743zi:elf config got broken (nsh: nsh_session: readline failed: 13)

Steps to reproduce:
$ mkdir TEST_ROOT
$ git clone https://github.com/apache/incubator-nuttx.git TEST_ROOT/nuttx
$ git clone https://github.com/apache/incubator-nuttx-apps TEST_ROOT/apps
$ cd TEST_ROOT/nuttx
$ ./tools/configure.sh -l nucleo-h743zi:elf
$ make CROSSDEV=arm-none-eabi-
$ sudo openocd -f interface/stlink.cfg \
               -f board/st_nucleo_h743zi.cfg \
               -c "init" \
               -c "reset init" \
               -c "flash probe 0" \
               -c "flash info 0" \
               -c "flash write_image erase nuttx.bin 0x08000000" \
               -c "reset run" -c "shutdown"

nsh> ls
nsh> nsh: nsh_session: readline failed: 13

The last known working pair was:
nuttx: 705c64e5ff50df4f7ea202ad870e7106dddd243c (13-Dec-2020)
apps: f2100104663704be498179f06bd78adcbe62a809 (17-Dec-2020)
Thus the issue has appeared somewhere between 13-Dec-2020 and now.

Hi @patacongo,
Thank you for your investigation.
I tried your patch (loadinfo->filfd instead of loadinfo->fildfd). Yes, it resolves the issue.
The corrected patch:
diff --git a/binfmt/libelf/libelf_init.c b/binfmt/libelf/libelf_init.c
index 6658aa0b62..f2da035e41 100644
--- a/binfmt/libelf/libelf_init.c
+++ b/binfmt/libelf/libelf_init.c
@@ -134,6 +134,7 @@ int elf_init(FAR const char *filename, FAR struct elf_loadinfo_s *loadinfo)
   /* Clear the load info structure */

   memset(loadinfo, 0, sizeof(struct elf_loadinfo_s));
+  loadinfo->filfd = -1;

   /* Get the length of the file. */

I tried your patch (loadinfo->filfd instead of loadinfo->fildfd). Yes, it resolves the issue.

Can you submit a PR?


x/tools/gopls: provide a way to suppress “failed to compile” notifications

This warning pops up constantly for me, often several times a minute. There is no way I see to dismiss it permanently. I don't know what's wrong, and usually I don't care; I'm in the midst of typing or doing other work and it is a distraction. It always seems to recover by itself.
I'd love some way to suppress the notification, maybe a setting or a button in the notification itself. If set, maybe the "failed to compile" message, with useful detail, could go to the console instead, which is where I go looking if things aren't working as expected.

This error only pops up if gopls is in a bad enough state that we haven't been able to load any information for the packages in your workspace. It's almost meant to be annoying because we want the user to pay attention to it, but we are considering changing the frequency with which it pops up (https://golang.org/cl/248623).
Do you mind sharing your logs when you see this error? I really would not expect it to appear frequently if things are working  correctly.

I just started a different project, so this might not have the same root cause as before, but I'll just dump all instances here as they occur.
The code in the workspace failed to compile (see the error message below). If you believe this is a mistake, please file an issue: https://github.com/golang/go/issues/new. github.com/josharian/native [github.com/josharian/native.test] has no metadata

No output at all for gopls. gopls server has a bunch of entries like this:
[Trace - 10:06:44.652 AM] Sending request 'textDocument/codeLens - (409)'.
Params: {"textDocument":{"uri":"file:///Users/josh/src/github.com/josharian/native/endian.go"}}

[Trace - 10:06:44.652 AM] Received response 'textDocument/codeLens - (409)' in 0ms.
Result: null

[Trace - 10:06:44.653 AM] Sending request 'textDocument/codeAction - (410)'.
Params: {"textDocument":{"uri":"file:///Users/josh/src/github.com/josharian/native/endian.go"},"range":{"start":{"line":4,"character":19},"end":{"line":4,"character":19}},"context":{"diagnostics":[]}}

[Trace - 10:06:44.653 AM] Received response 'textDocument/codeAction - (410)' in 0ms.
Result: null

Under problems, I see:
{
	"resource": "/Users/josh/src/github.com/josharian/native/endian_test.go",
	"owner": "_generated_diagnostic_collection_name_#0",
	"severity": 8,
	"message": "package native_test; expected native",
	"source": "compiler",
	"startLineNumber": 1,
	"startColumn": 1,
	"endLineNumber": 1,
	"endColumn": 8
}

and
{
	"resource": "/Users/josh/src/github.com/josharian/native/endian_generic.go",
	"owner": "_generated_diagnostic_collection_name_#0",
	"severity": 8,
	"message": "undeclared name: binary",
	"source": "compiler",
	"startLineNumber": 3,
	"startColumn": 14,
	"endLineNumber": 3,
	"endColumn": 30
}

The first "problem" is bogus--native_test is a legitimate package name for an external test in a package called native.
The second "problem" is bogus--it is complaining about a line that reads import "encoding/binary".

@josharian We need the full log, I'm afraid.
This might be a dupe of #40578 if you have a folder with no Go code in it and GO111MODULE=on.

Where should I look for the full log?
GO111MODULE is unset; there is Go code present.

https://go.googlesource.com/tools/+/refs/heads/master/gopls/doc/troubleshooting.md#capturing-logs describes how to get them.

I created a new directory, then ran
$ touch x.go
$ touch x_test.go
$ code .

I opened x.go and typed package p and hit save. I opened x_test.go and typed package p_test and hit save. Even before I managed to hit save on x_test.go, the compilation failed message started popping up.
Logs:
https://gist.github.com/josharian/e997023e436db031dba79a90249a9c9f

I can't reproduce. Before I dig any further, it might be worth trying 1.15, which fixed a number of bugs related to partial/missing package declarations. Can you update and make sure it's first on your PATH when you start VS Code?

Still reproduces with 1.15, exactly the same set of steps.
https://gist.github.com/josharian/6021b95602f3ae70820bf20ec80147ee
I noticed that this reproducer happened to be located as a GOPATH subdir. Sorry for not mentioning that earlier. I have also hit the same set of symptoms outside of GOPATH, so it didn't occur to me as relevant, but maybe there are multiple different  things happening.

I am able to reproduce this consistently in vscode.

Create a new workspace,
add the go project (Add folder to workspace)
add another non go project (Add folder to workspace)
Go to go project and try to edit and save
You will see the warning.

when it's only go project in the workspace, there will not be any warning when editing go files.
once another project is added, the warning will show up in the go project.

@zhaohanweng That is #40578. Please see https://github.com/golang/go/issues/40578#issuecomment-676623170.

Thanks for investigating this, @heschik. This is similar to a number of bugs we've seen with empty files--all related to https://github.com/golang/go/issues/39986. The go/packages fix here is likely not feasible, so I think the full resolution to this issue will have to come with https://github.com/golang/go/issues/39986, which will hopefully be released as part of Go 1.16. In the meantime, I will mail a CL to disable the pop-up.

@stamblerre I am not getting these pop-ups in another project. I just re-installed gopls at HEAD, or at least, I tried to. Should I...open a new issue? Do something else?

I just re-installed gopls at HEAD, or at least, I tried to.

What's the output of gopls version? To install at HEAD, run GO111MODULE=on go get golang.org/x/tools/gopls@master golang.org/x/tools@master. If that doesn't fix the issue, feel free to open a new issue and please attach the contents of the error message and your logs.


IlMusu's enchantments 1.5.0 crash upon startup

Mod Name
IlMusu's Enchantments
Mod Homepage
https://modrinth.com/mod/ilmusuenchantments
Minecraft version
1.20.1
Describe the bug
Crash on startup with Forgified Fabric API, Connector & IlMusu
Steps to reproduce

Download all mods, crash on startup.
Logs
https://gist.github.com/Liamgrayskull/dc1deb27c9dbda31e3b75f837bf0d577
Additional context
No response

Fixed in Connector beta 40


Refactors elasticsearch service and span name query to be like c*

Service and span names are written to the same daily indexes as spans
and dependency links as the document type "servicespan".
The document is a simple structure, like:
{
  "serviceName": "MyServiceName",
  "spanName": "MySpanName",
}

When the "servicespan" type is missing from an index, or there's no
results returned, a fallback nested query is invoked.
Fixes #1526

ok code now sends service/span indexes in the same batch command. This should help with AWS transfer charges, especially since we gzip requests. cc @devinsba @semyonslepov
still need some code cleanup and other notes in the TODO

backport logic in place. going to give a stab at a backport script as particularly serviceNames being current helps.

I think the best way to backport is to sneak hidden feature into zipkin-dependencies (as it is already a spark job and can do everything)

If someone wants to migrate old data, they'd need to do the following.
For each span in an index (ex zipkin-2017-02-14/span)
If span.name isn't empty, create a document like this for each service name in span.annotations[].endpoint.serviceName, span.binaryAnnotations[].endpoint.serviceName,
id = myServiceName|mySpanName
{ "serviceName":"myServiceName",  "spanName":"mySpanName",
save the unique set of documents to the "servicespan" type  in the same index (ex zipkin-2017-02-14/servicespan)

depends on #1562

fixed a very silly test setup bug which consumed a lot of my last night.. will be ready to go on green
PS ignoring circleci because it thinks the date is different than it is (a routine problem unrelated to this code)


Thanks.


[Urgent] FGP_TV & TotalVariation call method

In the following lines, I load a tomophantom and create a random imagedata and compare the __call__ of the TotalVariation and FGP_TV. The results are very strange
    # Detectors
    N = 128
    detectors =  N

    # Angles
    angles = np.linspace(0,180,180, dtype='float32')

    # Setup acquisition geometry
    ag = AcquisitionGeometry.create_Parallel2D()\
                            .set_angles(angles)\
                            .set_panel(detectors, 0.1)
    # Get image geometry
    ig = ag.get_ImageGeometry()

    phantom = TomoPhantom.get_ImageData(12, ig)

    alpha = 2.

    F1 = alpha * TotalVariation(isotropic=True)
    F2 = alpha *  FGP_TV(isotropic=True)

    res1 = F1(phantom)
    res2 = F2(phantom)
    print(res1, res2)

    tmp_ig = ImageGeometry(4,5)
    x1 = tmp_ig.allocate('random')

    res1 = F1(x1)
    res2 = F2(x1)

    Grad = GradientOperator(tmp_ig, backend='c')
    tmp = alpha * MixedL21Norm()
    res3 = tmp(Grad.direct(x1))

    print(res1, res2, res3)

Initialised GradientOperator with C backend running with  20  threads
**31870.3046875 3187.02294921875**
/opt/anaconda3/envs/cil_devel_epaps/lib/python3.7/site-packages/cil/optimisation/operators/GradientOperator.py:365: RuntimeWarning: overflow encountered in true_divide
  ndout[i]/=el
/opt/anaconda3/envs/cil_devel_epaps/lib/python3.7/site-packages/cil/framework/framework.py:2361: RuntimeWarning: invalid value encountered in multiply
  out = pwop(self.as_array() , x2 , *args, **kwargs )
/opt/anaconda3/envs/cil_devel_epaps/lib/python3.7/site-packages/cil/framework/framework.py:2363: RuntimeWarning: overflow encountered in multiply
  out = pwop(self.as_array() , x2.as_array() , *args, **kwargs )
Initialised GradientOperator with C backend running with  20  threads
**nan 21.421253204345703 21.421253204345703**

What is this about? What is "very strange"?
You have nan coming from a division, apparently
https://github.com/TomographicImaging/CIL/blob/26f863aea98b339cc4f8f53f0d33f61a44742d5c/Wrappers/Python/cil/optimisation/operators/GradientOperator.py#L363-L365
I suppose you need to figure out what is in self.voxel_size_order

I think the bug is coming from the GradientOperator with c backend.
Grad = GradientOperator(ig, backend='c')

ig1 = ImageGeometry(3,4)
x1 = ig1.allocate('random',seed=5)

res1 = Grad.direct(x1)
res2 = Grad.direct(phantom)

print(np.any(np.isnan(res1[0].array)))
print(np.any(np.isnan(res1[1].array)))

print(np.any(np.isnan(res2[0].array)))
print(np.any(np.isnan(res2[1].array)))

Initialised GradientOperator with C backend running with  20  threads
True
True
False
False

whereas in the numpy backend
Grad = GradientOperator(ig, backend='numpy')

ig1 = ImageGeometry(3,4)
x1 = ig1.allocate('random',seed=5)

res1 = Grad.direct(x1)
res2 = Grad.direct(phantom)

print(np.any(np.isnan(res1[0].array)))
print(np.any(np.isnan(res1[1].array)))

print(np.any(np.isnan(res2[0].array)))
print(np.any(np.isnan(res2[1].array)))

operands could not be broadcast together with shapes (2,3) (2,3) (126,128)

In the numpy backend, the error is reasonable meaning that x1 does not belong in the ig geometry.

# If I comment the two lines below, get nan values
F1 = alpha*TotalVariation(isotropic=True)
F2 = (alpha/tmp_ig.voxel_size_x)*FGP_TV(isotropic=True)

If the wanted behaviour is to be able to use a TotalVariation class on data with different geometries, because Functions do not have domain, we need to not store _gradient and remove this:
https://github.com/TomographicImaging/CIL/blob/0e0f74da992781f4b1a90bb2c00333cbd137829b/Wrappers/Python/cil/optimisation/functions/TotalVariation.py#L303
This way the property gradient will always be in sync with the data it is applied to.


There are no Txs for ETH account

Describe the bug
Currently there are no Txs for ETH account
To Reproduce
Steps to reproduce the behavior:

Go to ETH account
Click on Tx list
See that there are no Txs in Tx list

Expected behavior
Txs list should be displayed.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: MacOS
Browser:  chrome
Version: 130.0.6723.119

Additional context
reproducible on prod branch.
https://github.com/user-attachments/assets/db6170c0-7150-408a-8350-90a596bde73a

fixed by #3409


encoder: Add metadata output

Add the option to output metadata to either a file or stdout, similar to in libcamera-still.
Defaults to a json output for useability, but can be switched to a pure text output like libcamera-still using --metadata-plain.
Example usage:
libcamera-raw -t 10000 --metadata metadata.json
libcamera-vid -t 10000 --metadata metadata.json
libcamera-vid -t 10000 --metadata metadata.txt --metadata-plain
Signed-off-by: William Vinnicombe<EMAIL_ADDRESS>

Thanks for the quick turnaround!
One thing I'm wondering is whether we like the metadata being output when we start encoding a frame, but obviously the encoded frame can only be output when then encoder has finished, and the two things are normally asynchronous. I guess the only difference is that you might get an extra frame or two of metadata compared to the stream?
I suppose the alternative might be to push the metadata into the encode queue and output it when we give the encoded buffer to the output object. We could also make it the output object's job rather than the encoder's... maybe that feels nicer?
More trivially, I was also wondering about auto-selecting the metadata format based on the file extension? I know we don't tend to do that elsewhere, so it might seem a bit inconsistent, on the other hand it avoids any proliferation of --metadata-plain type options. Or maybe --metadata-format plain would be better as it's more extensible. Or perhaps --metadata-format txt because it matches the associated file type. We could auto-select based on the file type but also allow it to be overridden with --metadata-format. I think I'm tying myself in knots. What do folks think?
Finally, if we have fancy alternate metadata formats for videos, I'm thinking stills should have them too... what would that involve?

One thing I'm wondering is whether we like the metadata being output when we start encoding a frame, but obviously the encoded frame can only be output when then encoder has finished, and the two things are normally asynchronous. I guess the only difference is that you might get an extra frame or two of metadata compared to the stream?

I haven't seen any occurrences of this in my testing, but yes, I suspect it could happen if the encoder drops a frame for some reason.

I suppose the alternative might be to push the metadata into the encode queue and output it when we give the encoded buffer to the output object. We could also make it the output object's job rather than the encoder's... maybe that feels nicer?

I'll have a look at this, I wasn't sure how to move it to the output object as I couldn't see the best way to pass the metadata pointer around, but I could look at moving the writing of the metadata to the encodeBufferDone callback which occurs after each frame is encoded.

More trivially, I was also wondering about auto-selecting the metadata format based on the file extension? I know we don't tend to do that elsewhere, so it might seem a bit inconsistent, on the other hand it avoids any proliferation of --metadata-plain type options. Or maybe --metadata-format plain would be better as it's more extensible. Or perhaps --metadata-format txt because it matches the associated file type. We could auto-select based on the file type but also allow it to be overridden with --metadata-format. I think I'm tying myself in knots. What do folks think?

I think --metadata-format would be the better option, with a default of json and the ability to set it to txt.

Finally, if we have fancy alternate metadata formats for videos, I'm thinking stills should have them too... what would that involve?

Probably just making the function to save the metadata accessible to libcamera-still as well - I'll have a look.

I've implemented these changes with these commits (I can squash them once we're ready to merge), but I'll also take another look at moving the metadata output to the output class, as that would be the proper place for it.

Thanks, this is looking good to me now! Just a couple of little memory related questions to have a quick think about.

I'm going to close this one because I'm going to tidy up one or two things and then I'll re-push.


onError is not called if onValue is called first

This logs the error fine.
var onceerr = Bacon.once(new Bacon.Error({}));
//onceerr.onValue(function(v) { console.log(v); });
onceerr.onError(function(v) { console.log(v); });

If you remove the comment nothing is printed.

This is known "issue". When you attach onValue -handler, the events are drained from onceerr. So at the (time) point, when you attach onError, the stream is already empty. You can circumvent the issue by using subscribe.

I am using this helper function to listen to all three events now:
subscribeStream = function(stream, handlers) {
  stream.subscribe(function(e) {
    if (e.isNext() && handlers.onValue) {
      handlers.onValue(e.value());
    }
    else if (e.isEnd() && handlers.onEnd) {
      handlers.onEnd(e);
    }
    else if (handlers.onError) {
      handlers.onError(e.error);
    }
  });
};

@eguneys You should use e.hasValue() instead of e.isNext(), or else initial events such as from properties will be treated as errors.

In the 0.8 branch, both once and fromArray are asynchronous, so this problem won't happen again.

This is a duplicate to #367 -> closing


Consolidate labeler

Summary

PR labeling automation is consolidated into 1 workflow that runs whenever a PR or its labels are updated. First, workflow checks if a semver label has been assigned. ONLY if there is no adequate semver label does the autolabeler step run. This should address the infinite loop that moved me to roll back changes in #441.
Use Cases

Checklist

[ ] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[ ] I have reviewed the styleguide for guidance on my code quality.
[ ] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

This looks good to me... although I was the one to merge it last time it wasn't quite right.. Second opinion requested from @ryanmoran or @ForestEckhardt please 🤓

@sophiewigmore Here's where I learned about that failure check: https://docs.github.com/en/actions/learn-github-actions/expressions#failure


Fix typo

Github -> GitHub

@changeworld : Thanks for your contribution! The author, @thomas1206, has been notified to review your proposed change.

Merging typo fix


I have added bottom to top button with all html,css and js codes. Kindly check it.

Oh okay. I will try it again.

@ashutoshkrris I have updated the button now. Kindly review.

The button doesn't work as expected.

Yes, it should be.
Please check in your Hacktoberfest profile.

it's showing me this

@ashutoshkrris

Please wait for some time to get the updated status.


Space dc-tag further from the message author

added 0.125rem margin to the left.
Before:

After:

Normally I edited the proper file, just can't test the modification.
Not sure if the difference is noticeable for others but it just felt to close compared to discord's app.
Feel free to close it if you don't think it's a valuable PR.
Great job on the rest of the Repo !

First of all thank you for your PR!
The gap between author text and tag is actually created by a gap inside _message.scss. I decided to do so to keep the tag clean from any margins in case it is needed somewhere else where the margin would be disturbing. Furthermore, I use the web app (not the desktop app) as reference as you can easily look up all values with the dev tools and those state a gap of 0.25rem which is also applied in discord.css.
However, the tag class requires a look up anyway as Discord changed some styles (like the font-weight).

Ahhhh okay, thanks for the explanation ;)


CFE is mis-lowering spreaded Maps to Iterable

For example, a statement like print(<int, int>{...map}); results in kernel output like:
    core::print( block {
      final core::Map<core::int, core::int> #t1 = <core::int, core::int>{};
      for (final core::MapEntry<core::int, core::int> #t2 in map)
        #t1.{core::Map::[]=}(#t2.{core::MapEntry::key}, #t2.{core::MapEntry::value});
    } =>#t1);

This explodes since map (ForIn.iterable) isn't iterable. Should this be map.entries instead?

fyi - @kmillikin.  Not sure if this is a known issue.

There's a missing call to the getter Map.entries.  Proposed fix here.


fix: add default maxToken value of watsonxai backend

Closes #
📑 Description

Add default maxToken value of watsonxai backend.
✅ Checks

[x] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information

@liyanwei93 LGTM


ruff: 0.0.189 -> 0.0.190

Diff: https://github.com/charliermarsh/ruff/compare/v0.0.189...v0.0.190
Changelog: https://github.com/charliermarsh/ruff/releases/tag/v0.0.190
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 207147 run on x86_64-linux 1

2 packages built:

darwin.builder
ruff


update R tutorial and include testing

I updated the name and some parts of the Austrian R tutorials and updated the documentation with new links. I also added some code that run the R tutorial while testing if the flag option --test-r is used.
How to review

Build the documentation and check the installation part and the tutorial page related to R
run the tests including the R tutorials, as described in the documentation py.test --test-r

Thanks to @guofei2016 I also found that running the tutorials' code on Rstudio fails at solving. It seems a path issue and after some debugging the path seems indeed correct.
Strangely, running on Jupyter or from command line with Rscript works
I attached an example, the austrian tutorial written in R.
@guofei2016 or @volker-krey could you please try, after making sure the austrian tutorial works for you, try to run file I send you in Rstudio and then form command line? Rscript austrian.R
To check whether these conclusions are correct. Then it would be something with Rstudio that I can not figure out. Thanks!

@adrivinca related to issues with running message_ix in RStudio, I noticed different setting of the PATH environment variable in Jupyter notebooks (where code works fine) compared to RStudio. The difference that I noticed is that in the Jupyter environment the user-specific part of the PATH environment variable which typically contains the Anaconda related paths, is added before the system part of the PATH variable. If I modify the PATH variable in RStudio manually in this way, then the code also works in RStudio. I have not figured out how to access the user- and system-specific parts of the PATH variable, so just did a quick and dirty manual addition which solves the issue, but is unfortunately not transferable.

@adrivinca @volker-krey something I'm not clear on: have you installed Anaconda and RStudio in parallel, or installed the latter via the former (e.g. as outlined here and here)?
There are many ways for a user to set up their system. Our install docs could recommend one, and/or qualify installation advice ("ensure your RStudio installation has the correct environment set up…") so it's specific to the case in which it applies, and/or mention that some combinations are not tested/supported.
Also found this while searching "RStudio anaconda paths": https://community.rstudio.com/t/how-to-add-conda-to-path-in-rstudio-with-new-conda-initialization/12182

@khaeru I think we both have anaconda and Rstudio installed separately. It is quite unlikely that any R user will have it installed through anaconda, that's why so far the parallel systems is the only case we look at.
@volker-krey I tried the path change you suggested but I still have the same solving issues

Postponing to after v3.0.

To rebase, resolve the conflicts, carry over R CI improvements from ixmp, and address #251, I opened #377 which supersedes this PR. Thanks @adrivinca for getting this started!


Branch protection entries: Keep entries, when disabling them

Gitea version (or commit ref): 1.12.3
Git version: 2.20.1
Operating system: Debian 10 (buster)
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (URL not necessary)
[ ] No
[ ] Not relevant

Log gist: Not needed

Description
It would be nice to keep branch protection entries, when set to disabled. Currently, disable is equal to delete.
Below is a suggestion, how it could look like.
Screenshots

Bad stale bot.

Good stale bot. yes yes yes.


Integration with Remix

Question
Hi @samchon! thanks for creating this amazing library! I am trying to add typia to Remix but I can't get it to work. I tried transformer mode and I got this error.
Error: Error on typia.misc.validatePrune(): no transform has been configured.

I then tried to follow the integration for Vite and got this error
app/routes/_auth/route.tsx:14:25
    14 export default function Auth() {
                               ~~~~
    An outer value of 'this' is shadowed by this container.
app/routes/_auth/route.tsx:34:6 - error TS2683: 'this' implicitly has type 'any' because it does not have a type annotation.

34   }, this);

this is my current vite.config.ts
import { vitePlugin as remix } from "@remix-run/dev";
import { installGlobals } from "@remix-run/node";
import { defineConfig } from "vite";
import tsconfigPaths from "vite-tsconfig-paths";
import react from '@vitejs/plugin-react'
import typescript from "rollup-plugin-typescript2";

installGlobals();

export default defineConfig({
  esbuild: false,
  plugins: [react(), typescript(), remix(), tsconfigPaths()],
});

any help would be greatly appreciated! thank you!

Now we have vite plugin, so you can use it


cmake: sparse.template: add COMMAND_ERROR_IS_FATAL

There are some situations like #67035 or #63003 where sparse aborts and returns an error code before the compiler has generated the .obj file; without any clear indication that the .obj is missing (in normal situations sparse prints warnings and does creates the .obj file)
Also, builds are parallel by default and sparse runs tend to be massive walls of text which all conspires to make it totally impossible to find the relevant error message. Instead, we get an link-time error.
The only clear indication is the exit code. So catch it and abort the build ASAP thanks to COMMAND_ERROR_IS_FATAL.
More generally speaking, the default behavior of execute_process() to ignore errors is crazy. How frequently does a build system run commands that do NOT matter?

Thanks for the review and additional context.

Hence a reason why most execute_process() calls are not using COMMAND_ERROR_IS_FATAL.

Time for a big search/replace?

Just a one-word long, CMake option fix... anyone besides Torsten?


Define a release pipeline for the FCL dev wallet in the Emulator

Context
The dev-wallet was integrated into the cli / emulator in https://github.com/onflow/flow-emulator/pull/90
Including the dev-wallet as part of the emulator removes the need for users to start the wallet as a separate service (Docker container or un-containerized web-app) and is an essential step towards creating  a more integrated local development experience on Flow.
Problem
The dev-wallet code that is currently integrated is a static build of the dev-wallet web-app, and no process exists to keep the integrated code up tp date with the latest bug-fixes and changes from the dev-wallet repository.
Impact
Users using the integrated dev-wallet will be testing their code against an outdated wallet protocol, wont have access to the latest features, and bugfixes.
We have a stated goal of removing Docker from Flow demo projects, specifically Kitty Items, with the general aim of reducing the overall complexity of showcase projects to make them easier for users to use and understand.
Requirement
Define a release pipeline the ensures users of the cli will always have the most up-to-date version of the integrated dev-wallet available.

@sideninja Can we sync on this soon?


Allow ability to support Amazon's hostname (aa.bb.cc.dd.ee) for TLS

RancherOS Version: 0.4.4
Where are you running RancherOS? AWS
I tried to configure TLS following this instructions but it didn't work. I receive this error when I try to connect from my local machine.

An error occurred trying to connect: Get https://<IP_ADDRESS>:2376/v1.23/version: x509: certificate is valid for <IP_ADDRESS>, not <IP_ADDRESS>

even when I made sudo ros tls gen -H <IP_ADDRESS> to generate the client certs.

same for RancherOS v0.4.5, seems the ros tls generate --server|-s produce an TLS /etc/docker/tls/server-cert.pem with wrong X509v3 Subject Alternative Name, no matter what ros config options are selected.
e.g.  see http://docs.rancher.com/os/rancheros-tools/ros/tls/
[rancher@rancher ~]$  sudo ros tls generate -s --hostname <IP_ADDRESS> --hostname localhost --hostname foo.example.com
[rancher@rancher ~]$ sudo openssl x509 -noout -text -in /etc/docker/tls/server-cert.pem
Certificate:
    Data:
        Version: 3 (0x2)

...(truncated)...

        X509v3 extensions:
            X509v3 Key Usage: critical
                Digital Signature, Key Encipherment, Key Agreement
            X509v3 Extended Key Usage:
                TLS Web Client Authentication, TLS Web Server Authentication
            X509v3 Basic Constraints: critical
                CA:FALSE
            X509v3 Subject Alternative Name:
                DNS:*, DNS:*.*, DNS:*.*.*, DNS:*.*.*.*, IP Address:<IP_ADDRESS>

Verified with v0.5.0-rc1 using an internal AMI.


Fix missing documentation

Fix a lot of documentation nits.  This is part of the fixes for #342.

PTAL.


Missing License

https://help.github.com/articles/licensing-a-repository/

I noticed that the NPM package shows MIT, but this is not indicated via a LICENSE file, nor mentioned in the README.

@ACReeser thank you very much for the report. It's my first public repository so i'm a little bit inexperienced about licenses.


[FLINK-16572] Clean up PubSub connector e2e test

What is the purpose of the change
The PubSub e2e test is failing quite regularly on CI. This change is about improving the debuggability of the test.
This change will not fix the e2e test failures. It might give us some indications why it is failing.
This change also contains some cleanups. It does not touch the main connector code.
Brief change log

fix licensing issue from copied code
change test to allow "Run until failure" in IntelliJ
log docker container output before stopping the container
add log output

Verifying this change
Covered by CI (I executed the test a few times here: https://dev.azure.com/rmetzger/Flink/_build/results?buildId=8123&view=logs&j=1f3ed471-1849-5d3c-a34c-19792af4ad16&t=2f5b54d0-1d28-5b01-d344-aa50ffe0cdf8)
I could not reproduce the error in 200 runs locally.
Does this pull request potentially affect one of the following parts:
test only change
Documentation
test only change

Thank for your the review.


Set file storage to be in a specific directory

part of https://github.com/CS2103JUN2017-T2/main/issues/120

@CS2103JUN2017-T2/developers Does this issue mean that the command to change the directory should be implemented?

@mattheuslee I suppose so. @CS2103JUN2017-T2/developers I was thinking we add a set command to the user guide to set the file path. Something like - set /Users/usernamehere/Desktop . If the directory exists then it starts storing the files there, and if it doesn't exist then it asks the user to input a proper file path. What dy'all think?


Feedback: Jinja support for notifications

What?
This PR adds Jinja support for message and data in notifications.
TODO

[X] Examples

[X] Docs

[ ] Changelog

Looks like tasks in an ActionChain do not get an special handling therefore may not get access to task name etc - right?

Looks like tasks in an ActionChain do not get an special handling therefore may not get access to task name etc - right?

Correct. We don't have that visibility in the liveaction or execution model.

@manasdk Can we talk about this PR in your morning so I can make progress on this?

@lakshmi-kannan I think you should go ahead and merge the changes.
+1


Web3: Implement eth_getTransactionReceipt

Fixes: runtimeverification/firefly#218

For blockhash field, let's put a dummy value there for now.
Transaction index probably refers to the order of transactions in the block itself (not a global list of all transactions).


ENOENT error when compiling from path containing a space

We've got a C# project where dotless isn't working for one of our developers (set as a build action on the .less file and working for other developers using the same project).
From what I can tell the problem is that his Windows username contains a space, so his checkedout less file is at C:\Users\First Last\Documents\GitHub\Project\Project\Website\Content\less\project.less
The output he gets when building includes:
lessc: ENOENT, open 'C:\Users\First Last\Documents\GitHub\Project\Project\Website\Content\less\Last\Documents\GitHub\Project\Project\Website\Content\css\project.css.map'
I believe the problem is because the parameters in monobuild/tasks/build.sh are not quoted, but I'm not sure how to test this out myself or create a patch.

Your error message kind of suggests that you're actually using less.js, not dotless -- the dotless executable would be dotless.Compiler.exe, not lessc, and ENOENT very much smells like something that comes from node.js.

That's quite possible, I set it up ages ago after trying a few alternatives and it just works for most people now. Thanks for the quick response.

Yes, I'm using ".Less Msbuild Tasks", which makes a "DotLess" build action available, hence my confusion. For anyone finding this in search results, I've refiled at https://github.com/Jehran/LessMsbuildTasks/issues/6

Thanks for the link, I'm sure it will help someone in the future. :-)


.travis.yml: Update osx_image versions

Ref:
In order to give users who explicitly use some of our less common Xcode versions some time to transition, we'll be maintaining all the current images through Monday, October 31st. On Oct. 31 we will be simplifying our image catalog to the following three available options:
osx_image: xcode6.4 (with 10.10)
osx_image: xcode7.3 (with 10.11 and default)
osx_image: xcode8 (currently 10.11, but tentatively planned to be 10.12 once we've done some vSphere compatibility testing)

Also, do we need to change the travis scripts in any other way? I see a few errors in the travis log, but nothing that stops our builds.

Well, I missed something. HOMEBREW_RUBY is used in before_script.sh.

https://github.com/caskroom/homebrew-cask/pull/26030 seemed just fine, and that was after this merged.


Update virtual machine extensions to include limitation

Tags on virtual machine extensions can only be updated when the VM is running.

@DerekRenew - thank you for your contribution
#sign-off


rapids_cpm_gtest adds support for BUILD_STATIC

Description
Extends the BUILD_STATIC option from GBench and NVBench to GTest. This will allow RAPIDS projects to package tests without any testing dependency libraries.
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The cmake-format.json is up to date with these changes.
[ ] I have added new files under rapids-cmake/

[ ] I have added include guards (include_guard(GLOBAL))
[ ] I have added the associated docs/ rst file and update the api.rst

Looks good to me. Thanks! Please open an issue (rapidsai/build-planning or this repo) where we can track migration of all RAPIDS packages to use static benchmarks/tests, if one doesn't exist yet.

https://github.com/rapidsai/build-planning/issues/32

/merge


Reduce GC during fuzz tests

Reduce GC by reducing the size of EC chain generated by the fuzz tests. The size of EC chain makes no difference to the coverage.
In one case reduce the honest count from 5 to 4 in order to avoid intermittent failures due to fuzz tests taking too long.

OK Fuzzed the changes here on CI for 2 minutes per test and it is looking good.


Bump Dependencies #9698976297

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/9698976297

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Redundant with: https://github.com/wso2/product-is/pull/20619


Change run_at property of privly.js?

Currently manifest.json doesn't specify a run_at property for privly.js, which means it will run at document_idle.
In poor network conditions, for example, while loading a script blocks(defers) the document_end and document_idle, privly.js injection will not be executed at the moment while the content has already been shown.
https://www.dropbox.com/s/cv5if1v2pnujch3/privly_2.mp4?dl=0
Change run_at to document_start may prevent this issue.

I experimented with this a while back and it caused more issues than it solved for me. Can you turn it on for yourself and see if this is still the case? I'll re-evaluate if you continue to support the change after a week.

I think there are more than just change the manifest.json. I will check the code.

Downgraded to minor.


AttributeError: 'PosixPath' object has no attribute 'startswith'

When I try to run this cell: Plot average NDVI I get the error: AttributeError: 'PosixPath' object has no attribute 'startswith'.
All library are installed.
This is the entire error:
AttributeError                            Traceback (most recent call last)
 in
----> 1 eopatch = EOPatch.load(patch_path/f'eopatch_11', lazy_loading=True)
2
3 plt.figure(figsize=(20,8))
4
5 ax = plt.subplot(1, 3, 1)
~/anaconda3/lib/python3.7/site-packages/eolearn/core/eodata.py in load(path, features, lazy_loading, filesystem)
523         """
524         if filesystem is None:
--> 525             filesystem = get_filesystem(path, create=False)
526             path = '/'
527
~/anaconda3/lib/python3.7/site-packages/eolearn/core/fs_utils.py in get_filesystem(path, create, **kwargs)
26     :rtype: fs.FS
27     """
---> 28     if path.startswith('s3://'):
29         return load_s3_filesystem(path, *kwargs)
30
AttributeError: 'PosixPath' object has no attribute 'startswith'

The problem exist and above trick doesn't seem to be working.
The other libraries are using fs_utils.py in eolearn.core which even I tried to put str(path).startswith.... in there but it doesn't help.
the following lines are generated in report.html file after running the execution cell.
 Traceback (most recent call last):   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/eotask.py", line 72, in _execute_handling     return_value = self.execute(*eopatches, **kwargs)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/core_tasks.py", line 119, in execute     eopatch.save(path, filesystem=self.filesystem, **self.kwargs)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/core_tasks.py", line 74, in filesystem     self._filesystem = get_filesystem(self.path, create=self._create)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/fs_utils.py", line 28, in get_filesystem     if path.startswith('s3://'): AttributeError: 'PosixPath' object has no attribute 'startswith'  During handling of the above exception, another exception occurred:  Traceback (most recent call last):   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/eoexecution.py", line 189, in _execute_workflow     results = workflow.execute(input_args, monitor=True)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/eoworkflow.py", line 172, in execute     results = WorkflowResults(self._execute_tasks(input_args=input_args, out_degs=out_degs, monitor=monitor))   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/eoworkflow.py", line 210, in _execute_tasks     monitor=monitor)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/eoworkflow.py", line 243, in _execute_task     return task(*inputs, **kw_inputs, monitor=monitor)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/eotask.py", line 59, in __call__     return self._execute_handling(*eopatches, **kwargs)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/eotask.py", line 85, in _execute_handling     raise extended_exception.with_traceback(traceback)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/eotask.py", line 72, in _execute_handling     return_value = self.execute(*eopatches, **kwargs)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/core_tasks.py", line 119, in execute     eopatch.save(path, filesystem=self.filesystem, **self.kwargs)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/core_tasks.py", line 74, in filesystem     self._filesystem = get_filesystem(self.path, create=self._create)   File "/home/azizi/anaconda3/envs/farm/lib/python3.7/site-packages/eolearn/core/fs_utils.py", line 28, in get_filesystem     if path.startswith('s3://'): AttributeError: During execution of task SaveToDisk: 'PosixPath' object has no attribute 'startswith'

I use this way and work ok.
eopatch = EOPatch.load(str(INPUT_EOPATCH), lazy_loading=True)

Traceback (most recent call last):
File "/app/manage.py", line 22, in
main()
File "/app/manage.py", line 18, in main
execute_from_command_line(sys.argv)
File "/usr/local/lib/python3.9/site-packages/django/core/management/init.py", line 419, in execute_from_command_line
utility.execute()
File "/usr/local/lib/python3.9/site-packages/django/core/management/init.py", line 413, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/usr/local/lib/python3.9/site-packages/django/core/management/init.py", line 257, in fetch_command
klass = load_command_class(app_name, subcommand)
File "/usr/local/lib/python3.9/site-packages/django/core/management/init.py", line 39, in load_command_class
module = import_module('%s.management.commands.%s' % (app_name, name))
File "/usr/local/lib/python3.9/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1030, in _gcd_import
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "", line 855, in exec_module
File "", line 228, in _call_with_frames_removed
File "/app/core/management/commands/populate_products.py", line 2, in
from faker import Faker
File "/usr/local/lib/python3.9/site-packages/faker/init.py", line 1, in
from faker.factory import Factory
File "/usr/local/lib/python3.9/site-packages/faker/factory.py", line 7, in
from faker.config import AVAILABLE_LOCALES, DEFAULT_LOCALE, PROVIDERS
File "/usr/local/lib/python3.9/site-packages/faker/config.py", line 11, in
PROVIDERS = find_available_providers(
File "/usr/local/lib/python3.9/site-packages/faker/utils/loading.py", line 57, in find_available_providers
for mod in list_module(providers_mod) if mod != 'pycache'
File "/usr/local/lib/python3.9/site-packages/faker/utils/loading.py", line 35, in list_module
return [name for _, name, is_pkg in pkgutil.iter_modules([path]) if is_pkg]
File "/usr/local/lib/python3.9/site-packages/faker/utils/loading.py", line 35, in
return [name for _, name, is_pkg in pkgutil.iter_modules([path]) if is_pkg]
File "/usr/local/lib/python3.9/pkgutil.py", line 130, in iter_modules
for i in importers:
File "/usr/local/lib/python3.9/pkgutil.py", line 420, in get_importer
importer = path_hook(path_item)
File "", line 1601, in path_hook_for_FileFinder
File "", line 1476, in init
File "", line 177, in _path_isabs
AttributeError: 'PosixPath' object has no attribute 'startswith'
any solution