Added contextual image to AltImage

Adds a feature that allows users to see the image they are adding alt text to as described in #658
By default the image in the Alt Text modal will be shown fully and then it's height will be collapsed so the user can partially see the image, text box, and keyboard, along with the CTAs.

I really appreciate the thought you put into this, but I think there's a simpler option!
Rather than collapsing the image, what if we kept it full-sized but then put everything in a scrollview and autofocused the input so that the UI takes us down to it straight away? Then you can easily scroll up to reference your image

For the second, it might look better if the background behind the submit was semi-opaque. Right now it looks horrible being over the image

For the second, it might look better if the background behind the submit was semi-opaque. Right now it looks horrible being over the image

Absolutely agree. As I mentioned I was lazy and it was more of showing off a quick idea I had and not bothering to properly style it.


Support showing in-tool notifications about user surveys or other noteworthy things

We currently show a notification on the main page of the tool if a new version is available. It would be useful if we could use a similar mechanism to advertise other things as well, such as when we run a user survey or when we organize an event of some sort.
Proposed solution
In the JSON file that is queried by OpenRefine to retrieve the latest version, also add another field for an arbitrary notification, like this:
{
     "versions": …
     "notification": {
         "text": "Take part in our user survey",
         "link": "https://limesurvey.org…"
     }
}

The notification would be displayed in priority over new versions (so that we also reach people who haven't upgraded for a while):

Alternatives considered
The notification could also be displayed elsewhere. Maybe @OpenRefine/designers have suggestions?
Additional context
I would first migrate to stop querying the GitHub API and use a JSON file hosted on openrefine.org, as suggested by https://github.com/OpenRefine/OpenRefine/issues/6017 and #4463.

I generally agree.  However, push notifications (actually not a push, but a query out, but users will kinda feel its a push), might be frowned upon beyond basic tool updates.

Any way to have a checkbox on the notifications that says "never notify in the future about these" and a close X with hovertext labeled as "dismiss for now"?
Also, on the home screen of OpenRefine where these notifications would display... I imagine there is no timeout for them to hide, correct?  I think not having a timeout, and just having the checkbox for the user to opt-out of future notifications in that category would take care of any timeout concerns (show the notification until the user opts-out)?
What will it look like if there are 3 notifications?  Survey, New Version, Event Soon?  Just stacked up vertically, I assume? Crowded, but that's maybe why the X close on them would be useful, I think.

We can consider adding a news tab on our homepage that will display the RSS feed from our blog. This will provide a non-intrusive way for our users to stay informed about important project updates. However, it may be easier to miss updates with this approach since they won't be accompanied by a yellow notification.


Shooter injection will fail

Calling the shoot command causes the should_inject flag to be set for only a single control loop, this will be too fast for the injection to complete. We can either use a sensor to check that the note is gone or just use a longer timer.

how come the flag goes false despite the button being pressed? Doesnt the if statement get called regularly which sets the flag to true?

If the shoot command is called by code e.g. autonomous it will only be set for a single loop. Even in teleop, we don't want the drivers to have to think about whether they have held the button down for long enough.

This should be handled in the state machine, if you ask me.


Chore/ddw 201 introduce snapshot testing

Adds snapshot testing for components.
Adds event simulation testing for NumericInput.js

Reviewing this now …

@DominikGuzei
I added a story for ThemeProvider today as well.

Great thanks @MarcusHurney, reviewing this again now!


Add more server configuration in a separate recipe

Content of this PR:

Move server configuration in a new sql_server::configure recipe.
Add attributes to control network listeners via registry keys.
Fix service name when using named instance.
Set service restart command to allow restart when SQL Agent is present.
Update README accordingly.

To avoid conflicts it replaces #59 and #61, and fixes #39
Tests will not pass until #60 is merged
cc: @aboten

:+1:

I will take a look at this and will get it merged.

@Annih Can you rebase this against master and then I'll get it merged in

ok @tas50 rebase and adaptation done :)


improve s3 url parsing

support

s3-ENDOINT.amazonaws.com/BUCKET/PREFIX

s3.ENDOINT.amazonaws.com/BUCKET/PREFIX

localhost:8088/BUCKET/PREFIX

localhost/BUCKET/PREFIX

<IP_ADDRESS>:8088/BUCKET/PREFIX

<IP_ADDRESS>/BUCKET/PREFIX

Codecov Report

Merging #1234 (ccfa85e) into main (7bf5fea) will increase coverage by 0.04%.
The diff coverage is 37.50%.

@@            Coverage Diff             @@
##             main    #1234      +/-   ##
==========================================
+ Coverage   57.40%   57.44%   +0.04%
==========================================
  Files         127      127
  Lines       21348    21357       +9
==========================================
+ Hits        12255    12269      +14
+ Misses       7598     7595       -3
+ Partials     1495     1493       -2

Impacted Files
Coverage Δ

cmd/sync.go
60.91% <37.50%> (-1.20%)
:arrow_down:

pkg/vfs/handle.go
85.71% <0.00%> (-0.85%)
:arrow_down:

pkg/object/s3.go
43.89% <0.00%> (-0.34%)
:arrow_down:

pkg/meta/redis.go
71.50% <0.00%> (-0.19%)
:arrow_down:

pkg/meta/sql.go
68.11% <0.00%> (+0.86%)
:arrow_up:

Continue to review full report at Codecov.

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


[Ehancement] Refactor code editor container

Checklist

[x] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Client: I updated multiple integration tests (Jest) related to the features

Motivation and Context

This PR aims to improve the structure and readability of the code-editor-container component and remove duplicate code.
Description

Refactoring of the Online Code-Editor

Refactor the abstract 'code-editor-mode-container' to a standalone 'code-editor-container' component.

Included submodules: Actions, File-Browser, Ace-Editor, Build-Output
The Instructions module is injected via Content Projection
Additional Navbar Components are injected via Content Projection

Change 'student-container', 'exam-student-container' and 'instructor-container' to use new component.

Steps for Testing

There should not be any visual changes to the editors.
Please thoroughly test that all code-editor related functionality still works, in all three places.

Normal Programming Exercise Code Editor
Exam Code Editor
Instructor Code Editor

Don't worry about the amount of changed lines of code, more than half of them are in tests.

Don't worry about the amount of changed lines of code, more than half of them are in tests.

Can we increase the test coverage here? "More than half" is not enough.

Many client side tests are deleted. What is the issue here?

Can we increase the test coverage here? "More than half" is not enough.

This relates to the changed lines of code, not the test coverage, which is at 85% for the code-editor-container component

Many client side tests are deleted. What is the issue here?

I took the test logic from the three separate integration test files for each of the three existing containers and moved it to a new integration test for the new component. This just removed duplicate code / tests of duplicate code. I did not remove any test cases.

Smaller issues that are maybe unrelated to this PR:

Some overflow occuring

Some warnings in Firefox only

Both issues are preexistent and not introduced by this pr. Leaving this for now.

Have you tested that the exam still works correctly with multiple programming exercises when it comes to interactive tasks, instructions and UML diagrams? I don't see this covered in the testing instructions.
It took us a long time to get this working correctly, we should make 100% sure that this is not broken again!

I had no issues in testing exams with multiple exercises each with different instructions, tasks or uml diagrams. When switching between exercises, all correct information is loaded and displayed.
I still could add the providers to the programming-exam-submission component if this is wanted.

I also checked instructions and UML diagrams while testing. Had no issues with it.


Enabling Realsense Camera

Hello,
I am trying to use PointCloud2 data for 3D mapping purposes. What I have done so far:

I saw that it is being published under the topic /uav1/os1_cloud_node/points. However no node seems to publish it. I tried to enable Realsense camera by adding the following parameter (--enable-realsense-front-pitched)to the session.yaml. If the drone is supporting it, simulation launches without errors. But Realsense came does not appear to be working.

Here is the status window:
Image of Status Window
I think I am missing something. Do you have any recommendations?

Hello, the problem is not that the realsense is not working, but the status window is checking a wrong topic. I fixed the status node, so now it should display the realsense correctly. I also fixed the duplication of the Garmin topic.
Pull uav_core and recompile and the status should be displayed correctly. But the realsense was working all along.

Thank you for your help. I pulled the uav_core and did a catkin build. But now there is this error:

Hi, I dont have any problems with the simulation, so probably just going to ~/mrs_workspace and calling
catkin clean
catkin build
should fix this.

Thank you very much. I cleaned and built again, now Realsense and the status are working perfectly. Unfortunately I can't obtain any PointCloud2 data. I type rostopic find sensor_msgs/PointCloud2 and it returns:

/uav1/bumper/lidar3d_processed
/octomap_point_cloud_centers
/uav1/os1_cloud_node/points

When I type rostopic echo /uav1/bumper/lidar3d_processed it publishes nothing. Others return empty too. Do I have to do something extra for those topics to start publishing?
Thank you again for your time and interest.

Hello, our simulated realsense does not produce any point cloud messages. It produces a depth image on a topic
/$UAV_NAME/rs_d435/aligned_depth_to_color/image_raw
You can either modify the simulation plugin to publish point clouds, or use this package - http://wiki.ros.org/depth_image_proc
to convert the depth images into point clouds.

Hi all,
By my opinion Dan has provided a nice explanation of the PCL, I am closing the issue.
Thanks to all. :)


gulp.src should be a passthrough

https://github.com/wearefractal/vinyl-fs/issues/25

I've been struggling with this for the past few days. I had the following gulp task, where I was using gulp.src to add files to an existing stream, based on the release notes for 3.8.0. At first it appeared to be working, but then I noticed that only two of the files I was adding were in the final output, rather than all five.
var gulp = require('gulp');
var concat = require('gulp-concat');
var source = require('vinyl-source-stream');
var buffer = require('vinyl-buffer');
var browserify = require('browserify');
var hbsfy = require('hbsfy');
var uglifyify = require('uglifyify');

var browserifyEntries = ['/path/to/commonJS-index.js'];
var legacyScripts = [
  '/path/to/script1.js',
  '/path/to/script2.js',
  '/path/to/script3.js',
  '/path/to/script4.js',
  '/path/to/script5.js',
];

gulp.task('compile-js', function(){
  var browserified = browserify({entries: browserifyEntries})
  .transform(hbsfy)
  .transform(uglifyify)
  .bundle();

  browserified
    .pipe(source('browserified.js'))
    .pipe(gulp.src(legacyScripts))
    .pipe(buffer())
    .pipe(concat('scripts.js'))
    .pipe(gulp.dest('./assets/compiled/')); //Output only contains browserified.js, script1.js, and script2.js
});

My current solution is to just use gulp-add-src and replace the gulp.src() call with that. Would be nice to use gulp.src though.

@josiahsprague I would love a PR from the gulp-add-src people to fix this, I just haven't had time to look at it

Nobody wants to take a swing at this so we can land it in gulp 4?

@contra what's the spec for this functionality? AFAICS gulp.src (or, more specifically, glob-stream) filters out vinyl files whose paths have already went through the stream.
So what would be the expected behavior here in case of a vinyl fine being piped into a gulp.src stream which would produce a vinyl file with the same path? Should it only emit one? If so, which?

@UltCombo Same behavior as vinyl-add-src

Could someone tell me the steps to reproduce this properly and you'll have an PR for it later today.

@kevva I have example here: https://github.com/heikki/gulp-src-debug/blob/master/gulpfile.js

I think this problem looks like the one described here https://github.com/sindresorhus/gulp-filter/issues/29
gulp.src should be a PassThrough stream only if you ask it to be so since the above bug description need to wait for the passthrough stream to be ended before ending himself.
I suggest this API:
gulp.src('**/*', { passtrough: true })

As a temporary fix, you could look at this attempt:
https://github.com/sindresorhus/gulp-filter/pull/35

+1 for { passtrough: true }, that way the implementation doesn't have to rely on hacks such as listening for pipe events.

@UltCombo I'm +1 or passthrough: true to be more explicit

@contra not sure if I understand. Do you want passthrough to be more explicit? As in { duplex: true }?

@UltCombo I am +1 on a passthrough option because it makes this feature more explicit.

Oh ok, I was unsure whether you were ok with this approach or if you wanted to be more explicit than the suggested. =]
I believe it shouldn't be too hard to implement the passthrough option then.

wearefractal/vinyl-fs#55

@UltCombo Nice - looking forward to merging that one tonight. Closing this, will be in gulp 4.0 (via vinyl-fs 4.0)


Mast: Watch damiestechnologies.com

Mast requests the watch of the watch_keyword damiestechnologies\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

damiestechnologies\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


auto arrange projects function added

Related Issue
auto arrange projects function added
Closes: #667
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
Checklist:

[x ] My code follows the style guidelines of this project.
[ x] I have performed a self-review of my own code.
[ x] I have commented my code, particularly in hard-to-understand areas.
[ x] I have made corresponding changes to the documentation.
[ x] New and existing unit tests pass locally with my changes.

Screenshots
Put any screenshot(s) of the project here.
https://user-images.githubusercontent.com/67833077/116029258-82752880-a676-11eb-9ad7-bd93c71a069e.mp4

@harshgupta20 Please review my PR . Sir the previous PR which has been liked to this issue has level2 . So will you label it with level 2 ?

@harshgupta20 Please review my PR . Sir the previous PR which has been liked to this issue has level2 . So will you label it with level 2 ?

@Ankita297 i am giving this level 1 as there isn't a lot of changes involved :)


.Net: Add support to upload a file (image, pdf, audio, video...) to Google Gemini

The content of this request is similar to this one (https://github.com/microsoft/semantic-kernel/issues/6443), I guess, but related to Google Gemini. Using Google AI Studio, it is possible to upload files and ask questions about their content. It would be interesting if this feature could be triggered in Semantic Kernel.

Closed based on previous comment


Updating of a BEL Nanopub is not working in BEL Manager

We changed the review status of the BEL Nanopub https://pmi-belmgr.sbvimprover.com/#/edit/5873b587e4b07ff8f7a5bdff manually. After clicking the button and the success message "Updated Nanopub" appears. But reloading the page gives you the old result and shows that review status (and other fields) were not updated.

Fixed in OpenBEL API


HopSlotMap segfaults when items are inserted and removed out of order

HopSlotMap, not SlotMap or DenseSlotMap, segfaults on clear() or drain() when values have been inserted and removed out of order. Running the example below a few times give different kinds of segfault errors, most of them to do with double free. More info as comments in the MVC: https://github.com/ErikNatanael/slotmap_segfault

Well, that's embarrassing. I will investigate this in a bit and if valid fix and yank the affected versions.

I found the issue.
My freelist consists of contiguous blocks of slots. The metadata for such a contiguous block is stored entirely in the first element of the contiguous block, except that there is one member variable, other_end, that is maintained for both the first and last block, such that slots[first].other_end == last and slots[last].other_end == first. Note that if the contiguous block has size one that slots[i].other_end == i
This makes it possible with a bit of clever logic keep all this information this up to date and the blocks contiguous. The problem arises because the drain iterator assumes that after removing slot[i] that slot[i+1].other_end still points to the correct location, if slot[i+1] is vacant. However, in the event that slot[i+1] is a singleton vacant slot, after removing slot[i], in fact slot[i+1].other_end == i. Oops!
Unfortunately this affects all versions 0.3+, so I'll be yanking them. Looking at my usage charts it seems people pretty much exclusively use 0.4 and 1.0+, so I'll fix this in 0.4.1 and 1.0.4.

Done. Fixed in 1.0.4 and 0.4.1.

Would it be possible to backport this to a new release of 0.3? We use the yanked 0.3 release in older versions of glow https://github.com/grovesNL/glow/issues/179 We could workaround it in glow by updating all older releases of glow, but it would be nice if we could fix it here

Would it be possible to backport this to a new release of 0.3? We use the yanked 0.3 release in older versions of glow grovesNL/glow#179 We could workaround it in glow by updating all older releases of glow, but it would be nice if we could fix it here

I didn't realize slotmap 0.3 had active users, I can backport it yes, after 1.0.5 since a vulnerability to .remain() was found (with the same cause) by fuzzing. I'll unyank 0.3 for these couple of hours to ease the transition.

@orlp awesome, thank you! No problem :)

@grovesNL 1.0.5 is released and the fixes in 1.0.4 and 1.0.5 are backported to 0.3.1 and 0.4.2. Vulnerable old versions are now again yanked.

Great, thanks again!


testapp --peak-luminance option seems to have no effect

I'm testing with the following testapp cmds:
./testapp colorspace --width 192 --height 108 --fullrange-in --fullrange-out --peak-luminance 10000<EMAIL_ADDRESS><EMAIL_ADDRESS>rgb:st_2084:dcip3_d65 ictcp:st_2084:2020

./testapp colorspace --width 192 --height 108 --fullrange-in --fullrange-out --peak-luminance 1<EMAIL_ADDRESS><EMAIL_ADDRESS>rgb:st_2084:dcip3_d65 ictcp:st_2084:2020

The output videos SCD_192x108.rgb48.plr.pl10000.yuv and SCD_192x108.rgb48.plr.pl1.yuv are almost identical (with PSNR inf, 126.944 and 121.172 for the three channels).
The --peak-luminance seems to have no effect. Have I misinterpreted the meaning of the option?
This also seems to be the case in ffmpeg zscale filter (I assume the corresponding flag is npl).
The input video can be found here:
https://drive.google.com/file/d/1Jy-mOtucAZUXrtIAGmBFKIJ_337t4YnE/view?usp=sharing

--peak-luminance is only for SDR<-->HDR conversions. It maps the relative 1.0 luminance of SDR to a specific code value in PQ.

--peak-luminance is only for SDR<-->HDR conversions. It maps the relative 1.0 luminance of SDR to a specific code value in PQ.

Make sense. Thanks!

Make sense. Thanks!


Gantt scrolls to end if scale is changed

When my date range is a year changing the scaling (i.e. from year to week) results in scrolling to the end date of the gantt.
I would like to keep the view at the currentDateValue instead.
Is there an API available to react on attribute changes accordingly. Or how can I scroll to the currentDateValue ?

@Schweigi I am changing  view scale + headers at the same time and the scroll position isn't saved,
I must change the headers, Do you know how can I fix it?
Thanks!

Remove both marked watchers in https://github.com/angular-gantt/angular-gantt/blob/master/src/core/logic/column/columnsManager.factory.js#L30-40 and add them to the existing one above it.
So the one above will be: ['headers', 'headersFormat', 'viewScale', 'columnWidth', 'timeFramesWorkingMode', ... instead of ['viewScale', 'columnWidth', 'timeFramesWorkingMode', ...
As the watchCollection is changed into a normal watcher you will need to assign a completely new array for headers or headersFormat every time you want to change them.

I'll check it, Thanks
`        ColumnsManager.prototype.setScrollAnchor = function() {
if (this.gantt.scroll.$element && this.columns.length > 0) {
var el = this.gantt.scroll.$element[0];
var center = el.scrollLeft + el.offsetWidth / 2;
            this.scrollAnchor = this.gantt.getDateByPosition(center);
        }
    }; '

I am using the mouse wheel to change the gantt scale, and I want to set the center relative to the mouse position, I have tried:
' ( e.pageX - $(el).offset().left) + el.scrollLeft)'
e - is the mouse wheel event
I am getting the right X position but 'this.gantt.getDateByPosition' doesn't return the correct date.


Fix unique glyph codes check in getSubsetText

Don't use getUniqText() for glyphs subset, check for codes uniqueness in string2unicodes() instead.
getUniqueText() process passed string as a regular one. In some cases it doesn't handle unicode properly and will remove code points parts when several code points share the same numeric parts.
Ultimately, it will result in wrong code points being pushed to output.
In some cases, the glyphs count will match, not the codes themselves (tested and confirmed with FontAwesome's Duotone font variation).
Fix UTF-16 glyphs subset (and probably others), tested and confirmed with FontAwesome's Duotone font variation where glyphs code points look like this: \10f6ae (https://fontawesome.com/v5.15/icons/acorn?style=duotone).
Keeping getUniqText() method for backward compatibility / third party packages relying on it.

@Moustachos thank you 😄
bump<EMAIL_ADDRESS>

Nice :D


Unable to see breakpoint when code coverage is active

Even though there is a breakpoint set and it gets hit while debugging, it is not displayed if watching a file.

After removing the watch:

@Franklin89 yes, based on my knowledge the api hasnt improved for this but I can take a look with the re-architecture work to see if there is a way to play nicer with the breakpoint. Not sure if you saw but in the meantime you could use this approach https://github.com/ryanluker/vscode-coverage-gutters/issues/87#issuecomment-316275141.

Thank you for that @ryanluker

@Franklin89 looked into this and didnt find anything new from microsoft but they did mention extending the decoration system (what I use to make the coverage lines, vs the gutter coverage which is simply image icons) to the gutters so that should help with this in the future!


PATRIC_VF landing page

missing on alpha. Need to add it.
https://www.patricbrc.org/portal/portal/patric/SpecialtyGeneSource?source=PATRIC_VF&kw=

This is fixed.


default extractMeta implementation is truncating additional metadata

Use case: invoicing app asks API for list of invoices. Metadata includes pagination data as well as dialy, monthly and yearly turnarounds that can be displayed somewhere near list. However default extractMeta implementation used by Ember-Django adapter assumes that metadata will be used only for pagination, and thus discards other meta, breaking contract promised by Ember's docs.

I didn't consider that people use the metadata for things other than pagination but you seem to have a good use case. Can you send me a link to the Ember documentation you mention? Thanks.

You can solve this by overriding extractMeta for just the invoice record type. You can generate a DRFSerializer for invoice with this ember-cli command:
ember generate drf-serializer invoice

Then add your own metadata where I've indicated and also delete the extra properties (again as indicated). This is just off the top of my head so I haven't tested it.
// app/serializer/invoice.js
import DRFSerializer from './drf';

export default DRFSerializer.extend({
  extractMeta: function(store, type, payload) {
    if (payload && payload.results) {
      // Sets the metadata for the type.
      store.setMetadataFor(type, {
        count: payload.count,
        next: this.extractPageNumber(payload.next),
        previous:  this.extractPageNumber(payload.previous),

        // Add your custom metadata here.
      });

      // Keep ember data from trying to parse the metadata as a records
      delete payload.count;
      delete payload.next;
      delete payload.previous;

      // Delete your custom metadata properties here.
    }
  }
});

I'm still interested to see the documentation to know if we should be setting the unknown metadata so that you can avoid doing this. Thanks.

Hey there!
http://guides.emberjs.com/v1.12.0/models/handling-metadata/
The way I understand this if backend provides meta key in json response, that key's values are treated as metadata and are automatically put into meta for type.
Taking from there I've assumed that because django adapter is middleman translating DRF's json for Ember-Data json, and Rest Framework is not using meta key on its responses, adapter could treat all other keys not belonging to results as metadata. But I admit this interpretation may be debatable. :]
It's indeed easy to get fixed by onself. In fact it took me 5 minutes to write custom meta extractor in my app serializer (I'm doing heavy use on metadata in my api responses).

I agree with your analysis; extra metadata should be included with the metadata. I'll try to fix this for the next release. Thanks.


Feature/598 read unread state of conversations

resolves #598
changes:

readConversations endpoint in http lib
readConversations action + reducer
added check to api config because the readConversations endpoint returns a 202 with an empty response

Sadly there is a merge conflict 😢 . Sorry that you need to change something again.

Sadly there is a merge conflict 😢 . Sorry that you need to change something again.


upstream-url should not default to itself

That can cause a infinite redirect loop back to itself.
Maybe there is a reason why it defaults back to itself?
/cc @chrisns

default upstream has been removed in v1.1.0


Python coding standards

Python coding standards sample repository
Coding standards page: https://provizio.atlassian.net/wiki/spaces/PROVIZIO/pages/839155743/Proposal+Python+Coding+Standards

Copying it from a Teams discussion here, so it's available for reviewing by others and discussion:

[12:04] Iurii Ivanov
Is there anything imported from that repo to a client project that would help reuse the standard?
[12:04] Dane Mitrev
Nothing like a build system or CI yet
[12:04] Iurii Ivanov
F.e. in C++ standard it supports importing to CMake and then your whole project gets automatically configured to match these standards (with plenty of checks added, code styles applied and so on)>
[12:05] Dane Mitrev
The repo is basically :- showing folder structure- providing sample files (like setup.py, requirements.txt)
[12:05] Iurii Ivanov
Ah, I see. Are there any tools to achieve behavior like I described with CMake for Python?
[12:05] Dane Mitrev
I will think about adding such automated thing, but I doubt for Python it will work
[12:05] Dane Mitrev
since it's extremely flexible and people do things differently
[12:06] Iurii Ivanov
Well, there are some static code analysis tools for Python as far as I know (though I don't have experience with any)
[12:06] Iurii Ivanov
C++ is also very flexible, hence enforcing the standards
[12:06] Dane Mitrev
I will check
like 1
[12:06] Dane Mitrev
Thanks!
[12:07] Iurii Ivanov
https://blog.codacy.com/which-python-static-analysis-tools-should-i-use/
Which Python static analysis tools should I use?
As you might have read in our introductory article about static code analysis, there are a lot of reasons why you should implement it in your development
[12:07] Iurii Ivanov
F.e.
[12:08] Dane Mitrev
Yep, that's all described in the Confluence. I already added a tox file that if you include in the project it will:run teststry to create environments with different python versionperform flake8 checks (more advanced than pylint) for linting and PEP8 checks
[12:09] Dane Mitrev
The only thing is that user would need to actually use "tox" to perform checks
[12:10] Iurii Ivanov
Cool! You may discuss with Letizia and Clement regarding ways to somehow automate applying of latest version of this Python standard. So when someone updates the standards in the future, it doesn't require manually changing multiple repos to check if they're now complient... The simplest solution is to have a separate python script copied from the standard repo, that on manual execution pulls the latest standard and runs whatever checks it can...
like 1
[12:15] Iurii Ivanov
F.e. with .gitignore alone it might read the current version in client's repo, then check the standards one, and if some of the entries is missing it might automatically add it to .gitignore. Otherwise if you ever add a line to .gitignore, none of existing repos will ever notice it.
[12:15] Iurii Ivanov
Only future ones.
[12:15] Iurii Ivanov
Same applies to anything that can be automated. You get the idea.
[12:17] Dane Mitrev
Got it! Then I will actually add a python script that does does checks, users will only need to run it in order to check compliance
heart 1
[12:17] Dane Mitrev
Thanks!
[12:17] Iurii Ivanov
I also suggest to have a separate one, very simplistic: it would download the latest version of the main checking script from master branch of standards repo and then execute it.
like 1
[12:18] Iurii Ivanov
It's pretty much what happens in our standard CMake
like 1
[12:18] Dane Mitrev
Yep, makes sense!!
heart 1
[12:18] Iurii Ivanov
This simple one shouldn't really change. And the main one can be gitignored as it's pulled automatically anyway
like 1
[12:20] Dane Mitrev
Yes, got the idea, so that we can maintain the main script

@mdane-x , also please update the root README.md to include a link to the python section, like done with C++.

@mdane-x Thank you, now looks good to my basic-Python view. I rely on experts @letiziaprovizio and @ClementBotty to review.

Approved, but at least one of @letiziaprovizio and @ClementBotty needs to review and approve first for it to be good enough for merging, please.


changelogs ui error

I never saw the 'close' option till I saw this screenshot! I completely missed it and came to this issue to ask for a back button to dismiss the changelog.

Yeah I will fix it in the next stable release


Build scheduled Lambda function to search NIH/DOE for related works

Need to build a fairly generic lambda that can be used to schedule runs for our different API searches (aka ApiScheduler).
Consider having this Lambda fetch DMP IDs that are scan candidates

Ones with no grant id (no grant_id and funding_status: 'planned'
Ones approaching their project end date (or n-months past the start date?)

Then have this Lambda use EventBridge messaging to run scans async to a 2nd Lambda function (aka ApiScanner). This one will perform the following:

Call the specified API for the designated DMP IDs
If the item has an opportunity number that was clearly awarded to another project, mark it as funding_status: 'rejected'
Add grant_id and change funding_status: 'granted' for plans
Add dmproadmap_related_identifiers
Add PIs if not already present

All of the above changes should be placed into the dmphub_modifications section of the record!

Update the dmphub_modifications array so that it includes a confidence score as well as a note about how the match was determined (e.g. 'matched on grant_id and PI names')
@mariapraetzellis let me know what you think about the following
Here is preliminary logic for generating a confidence score:
Confidence levels:

Auto - We are so confident that no human intervention is required (humans can reject still if needed)
High - We are 75%+ certain that they items are related but it needs human confirmation
Med  - We are 50% - 75% certain that they items are related defintiely needs human confirmation
Low  - We are less than 50% certain which means human confirmation is required

Related Works (e.g. detected by DataCite EventData, OpenAlex, etc.):
Confidence         Match type
--------------------------------------------------------------------------------------------------
Auto               Grant ID match (in a scenario where the external system had record of the grant ID)
Auto               DMP ID match (in a scenario where the external system had record of the DMP ID)

High               1+ PI ORCID matches, funders match and title/abstract keywords match
High               1+ PI ROR matches, funders match and title/abstract keywords match
High               Funders match, repository match and title/abstract keywords match
High               1+ PI names match, repository match, output type match and title/abstract keywords match

Med                1+ PI names match and title/abstract keywords match
Med                Funders match and title/abstract keywords match
Med                one or more PI names match and title/abstract keywords match

Low                1+ PI names and title/abstract keywords match
Low                Funders match, repository match and title/abstract keywords match
Low                Funders match and title/abstract keywords match

Grant Information (e.g. detected by NIH Awards API, etc.):
Confidence         Match type
--------------------------------------------------------------------------------------------------
High                Funder match, opportunity number match and 1+ PI names match
High                Funder match, title exact match, 1+ PI names match
High                Funder match, 1+ PI names match, project start/end match and title/abstract keywords match

Med                Funder match, 1+ PI names match and title/abstract keywords match
Med                Funder match, 1+ PI names match and project start/end match
Med                Funder match, opportunity number match and title/abstract keywords match

Low                Funder match, 1+ PI names match and project start/end match
Low                Funder match, project start/end match and title/abstract keywords match

We could add a new harvester Lambda that searches for award information using the logic we built for the pilot project's funder project search page.
Any discovered information could be added to the HARVESTER_MODS item in the Dynamo table for a DMP and then tied into future curation workflows in the new system


Embedded newlines in .resx files are treated differently by .NET Core and full framework

The resources embedded in the output assembly differ between .NET Core and full framework msbuild when a .resx file contains a string with newlines in it.
On full framework, the newlines appear to be embedded as they are in the file (on my machine, that's CRLF), but when building using dotnet build the resources are embedded with plain LF.
(I noticed this while piloting SDK conversion in this repo, when the test HelpMessagesAreValid started failing because it splits on Environment.NewLine which didn't appear in the .NET Core-embedded resources.)

Duplicate of #2206


CPU Temperature alert giving 'No data' on Raspberry Pi CM3+

I'm testing this out on a Turing Pi cluster, with 7 Pi Compute Module 3+ boards.
On my Grafana dashboard, I'm seeing no data for CPU temperature:

I'm going to dig in and see where the monitor is supposed to be running. I modified the vars.jsonnet file like so, for my cluster:
{
  _config+:: {
    namespace: 'monitoring',
  },
  // Enable or disable additional modules
  modules: [
    {
      // After deployment, run the create_gmail_auth.sh script from scripts dir.
      name: 'smtpRelay',
      enabled: false,
      file: import 'smtp_relay.jsonnet',
    },
    {
      name: 'armExporter',
      enabled: true,
      file: import 'arm_exporter.jsonnet',
    },
    {
      name: 'upsExporter',
      enabled: false,
      file: import 'ups_exporter.jsonnet',
    },
    {
      name: 'metallbExporter',
      enabled: false,
      file: import 'metallb.jsonnet',
    },
    {
      name: 'traefikExporter',
      enabled: false,
      file: import 'traefik.jsonnet',
    },
    {
      name: 'elasticExporter',
      enabled: false,
      file: import 'elasticsearch_exporter.jsonnet',
    },
  ],

  k3s: {
    enabled: true,
    master_ip: ['<IP_ADDRESS>'],
  },

  // Domain suffix for the ingresses
  suffixDomain: '<IP_ADDRESS>.nip.io',
  // If TLSingress is true, a self-signed HTTPS ingress with redirect will be created
  TLSingress: true,
  // If UseProvidedCerts is true, provided files will be used on created HTTPS ingresses.
  // Use a wildcard certificate for the domain like ex. "*.<IP_ADDRESS>.nip.io"
  UseProvidedCerts: false,
  TLSCertificate: importstr 'server.crt',
  TLSKey: importstr 'server.key',

  // Setting these to false, defaults to emptyDirs
  enablePersistence: {
    prometheus: false,
    grafana: false,
  },

  // Grafana "from" email
  grafana: {
    from_address<EMAIL_ADDRESS>  },
}

Checking on the DaemonSet:
# kubectl get ds -n monitoring
NAME            DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR                   AGE
node-exporter   7         7         7       7            7           kubernetes.io/os=linux          140m
arm-exporter    0         0         0       0            0           beta.kubernetes.io/arch=arm64   11m

Also after editing the vars.jsonnet file and re-makeing everything I'm getting unrelated error:
failed: [<IP_ADDRESS>] (item=/home/pirate/cluster-monitoring/manifests/node-exporter-daemonset.yaml) => {"ansible_loop_var": "item", "changed": false, "error": 422, "item": "/home/pirate/cluster-monitoring/manifests/node-exporter-daemonset.yaml", "msg": "Failed to patch object: b'{\"kind\":\"Status\",\"apiVersion\":\"v1\",\"metadata\":{},\"status\":\"Failure\",\"message\":\"DaemonSet.apps \\\\\"node-exporter\\\\\" is invalid: spec.selector: Invalid value: v1.LabelSelector{MatchLabels:map[string]string{\\\\\"app\\\\\":\\\\\"node-exporter\\\\\", \\\\\"app.kubernetes.io/name\\\\\":\\\\\"node-exporter\\\\\"}, MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable\",\"reason\":\"Invalid\",\"details\":{\"name\":\"node-exporter\",\"group\":\"apps\",\"kind\":\"DaemonSet\",\"causes\":[{\"reason\":\"FieldValueInvalid\",\"message\":\"Invalid value: v1.LabelSelector{MatchLabels:map[string]string{\\\\\"app\\\\\":\\\\\"node-exporter\\\\\", \\\\\"app.kubernetes.io/name\\\\\":\\\\\"node-exporter\\\\\"}, MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable\",\"field\":\"spec.selector\"}]},\"code\":422}\\n'", "reason": "Unprocessable Entity", "status": 422}

(That's from Ansible—basically it's trying to change the selector but that field is immutable. Seems unrelated to the armexporter though... I'm not sure why the DS has 0 desired instances.

I wonder if this issue might be related to something like https://github.com/kubernetes/kubernetes/issues/51785. I'm going to manually delete the daemonset and re-create it and see if that works.

Ah... I think this is the issue:
      nodeSelector:
        beta.kubernetes.io/arch: arm64

And I'm getting CPU temperature values, yay!

Is there a way you can set it to a set of arm64|arm?

Although now I'm not getting the other values (CPU, Memory, etc.)—only the temperature data and up/down status :/ — possibly something silly I did though. I might just wipe the cluster and reinstall again, now that there are a number of small tweaks.

I replaced the nodeSelector with affinity, and it's now scheduling things correctly:
    spec:
      affinity:
        nodeAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            nodeSelectorTerms:
            - matchExpressions:
              - key: kubernetes.io/arch
                operator: In
                values:
                - arm
                - arm64

I may submit a PR in a few minutes; I'm also testing the fix for #40 first.

PR #42 filed for this—it is in draft, though, because jsonnet fails with a missing Field.

@geerlingguy funny story, I am watching your pi cluster video and I have the same problem -- funny to see you here 😂  Great video, thanks <3


fix after #172, Interpretermode tests also need to run all other tests

Sorry, I did not think that through. #172 is bad because it makes many tests from parent test class run without proper configuration.

Test failed. Did you get a automatic notification for the failures?

No, I get no mail notifications. Working on fixing...

two fixed, two to go...

Thanks! :)


📦 NEW: corona-cli

This PR adds corona-cli.

🦠 Track the Coronavirus disease (COVID-19) or the Novel Coronavirus Strain in the command line. Both worldwide for every country or for one country. Fast response time (< 100ms).

Looking forward, peace!

@ahmadawais Read the contribution guidelines please.

My bad! Fixing it soon

Here's the new PR #42


There is no "Invite" button when adding a guest to an application

Section: Add guest users to an application. Step 8: "If the guest user does not already exist in the directory, select Invite."
There is no Invite button. So the image published with this tutorial in step 8, is inaccurate.
Here is what I get.

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

ID: 751889da-f001-117f-d525-3b27d467edf2
Version Independent ID: ce46adec-de86-7c34-45ba-d735cba76d2d
Content: Add B2B collaboration users in the Azure portal - Azure Active Directory
Content Source: articles/active-directory/b2b/add-users-administrator.md
Service: active-directory
GitHub Login: @msmimart
Microsoft Alias: mimart

Hmm...
You must enter the UPN of the user, and if the user is not in an existing user, then the "Invite" button actually shows up.

@tbebekis Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.
@msmimart  Can you please check and update if the screenshot under Step 8 is old.

@tbebekis Thank you for the feedback and the follow-up, glad it worked. But I do think this warrants a note to avoid confusion. We'll update the article. Thanks again!

#please-close


Create workflow for BitBucket

Use Bitbucket API to automatically create a new repo on bitbucket using the user's credentials.

I have added the bitbucket implementation using their API.


Add Support for Vapor Leaf (HTML Templating)

Description
Add Support for Vapor Leaf (HTML Templating).
Vapor Leaf Documentation: https://docs.vapor.codes/3.0/leaf/overview/
Leaf is HTML plus a relatively small set of template patterns.

#()
#(variable)
#embed("template")
#set("title") { Welcome }
#count(friends)
#for(friend in friends) {
#(friend.name)
 }
#if (…) { … } else {…}

Simply formatting the HTML without particular formatting of the template parts could be helpful.
Input Before Beautification
Any .leaf file.
Here is a link to several example .leaf files in the raywenderlich/vapor-til
github example project
Expected Output
The beautified code should look like this:
<!DOCTYPE html>

  <meta charset="utf-8" />
  <title>#(title) | Acronyms</title>

Acronyms

  #if(acronyms) {

            Short

            Long

        #for(acronym in acronyms) {

#(acronym.short)	#(acronym.long)

        }

  } else {

There aren't any acronyms yet!

  }

Actual Output
Attempted runs of :

Atom Beautify could not find a supported beautifier for this file
Atom Beautify could not determine a supported beautifier to handle this file with grammar "HTML (Leaf)" and extension "leaf". If you would like to request support for this file and its language, please create an issue for Atom Beautify at https://github.com/Glavin001/atom-beautify/issues

Steps to Reproduce

Open a .leaf file.
Run Atom Beautify.
The error message pops up.

Debug
Here is a link to several example .leaf files in the raywenderlich/vapor-til
github example project
Checklist
I have:

[x] Uninstalling and reinstalling Atom Beautify. Fresh install. N/A
[x] Reloaded (or restarted) Atom. Fresh install & subsequent restart.
[x] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue

[x] Leaf based templating support has not been previously requested.

[x] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[x] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

Support for Vapor Leaf (HTML Templating) has yet to be added to atom-beautify.

Leaf 3.0.1 have been released. Support for Vapor Leaf (HTML Templating) has yet to be added to atom-beautify.

Leaf 3.0.2 has been released.
Support for Vapor Leaf (HTML Templating) has not yet to be added to atom-beautify.
Leaf Syntax Overview is provivded at https://docs.vapor.codes/3.0/leaf/overview/.

Support for Vapor Leaf (HTML Templating) has yet to be added to atom-beautify.

I just added Vapor Leaf to Pretty Diff.

The latest version of Pretty Diff is about to be merged into Unibeautify (Atom Beautify's successor), so this is just around the corner.  In the mean time you can test support at http://prettydiff.com?mode=beautify&language=vapor and let me know if you find any defects.  You can report any issues to me directly at https://github.com/prettydiff/prettydiff/issues


Fix typo in Perl 6 grammar name

Sorry. I was new to atom packages and did not test it.

All good, I just had to assume that was the one. :smile:

$ apm publish patch
Preparing and tagging a new version ✓
Pushing v2.25.1 tag ✓
Publishing<EMAIL_ADDRESS>✓

We have a light dev guide to help set up the package in dev mode if you come back for more. :wink:


Fixed Map menu content positioning for small screens.

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (e.g. Closes #XXXX): Closes  #8524

Description

Issue #8524 reports that padding: 5px needed adding the the paragraph at the top of the map menu. In actual fact, the .mapWrapper was set to position: absolute causing the map menu contents to be shifted over to the right. Removing this property (so it will be set back to the browser default of static) re-aligns the menu & removes the issue from the map content. It also allows the accordion to span the width of the container.
Note: This fix goes hand in hand with line 143 of maps.less - the position property of .map-accordion has already been set to relative on the staging branch (live site is currently set to position: absolute). So once this fix along with the fix on line 143 is pushed, it should all sit properly again.
Tested this fix using Chrome Dev Tools with the Nexus 6P, Neuxs 5X and Galaxy S5.
Before

After

:sparkles: :sparkles: LGTM :+1: :sparkles: :sparkles:
:tada: Thanks for contributing @imjamesdotme and congratulations on your first PR submission to FreeCodeCamp! :tada:

@theflametrooper Thank you very much! 😄 🎉


Why did ss report errors?

root@MY20171111:~# ss
Netid  State      Recv-Q Send-Q          Local Address:Port                           Peer Address:Port
Cannot open netlink socket: Protocol not supported
Cannot open netlink socket: Protocol not supported
Cannot open netlink socket: Protocol not supported
Cannot open netlink socket: Protocol not supported

#2249


updated mapping and paper content

@hmenager - maybe this one should be merged first? https://github.com/bio-tools/biohackathon2022/pull/12

Hi @supernord, this is great! I've also been adding few things to the paper but we should merge this first (and the #12 as well) so I can add my few lines afterwards, I'm reaching out to Hervé.


launch: program '' does not exist.

Hi
I can not install any program on my windows for network policies.
So I downloaded VS Code portable and open it.
Also I downloaded dot net core library zip file and extract it in : C:\dotnet
I opened vs code and open folder C:\MVC and type c:\dotnet\dotnet.exe new mvc in terminal.
Then dotnet run command and project run perfectly.
Now I just want to start Debugging in vs code but give me error : launch: program '' does not exist.
launch.json :
{
"version": "0.2.0",
"configurations": [
{
"name": ".NET Core Launch (console)",
"type": "coreclr",
"request": "launch",
"preLaunchTask": "build",
"program": "${workspaceFolder}/bin/Debug/netcoreapp2.2/MVC.dll",
"args": [],
"cwd": "${workspaceFolder}",
"console": "internalConsole",
"stopAtEntry": false,
"internalConsoleOptions": "openOnSessionStart"
},
{
"name": ".NET Core Launch (web)",
"type": "coreclr",
"request": "launch",
"preLaunchTask": "build",
"program": "${workspaceFolder}/bin/Debug/netcoreapp2.2/MVC.dll",
"args": [],
"cwd": "${workspaceFolder}",
"stopAtEntry": false,
"internalConsoleOptions": "openOnSessionStart",
"launchBrowser": {
"enabled": true,
"args": "${auto-detect-url}",
"windows": {
"command": "cmd.exe",
"args": "/C start ${auto-detect-url}"
},
"osx": {
"command": "open"
},
"linux": {
"command": "xdg-open"
}
},
"env": {
"ASPNETCORE_ENVIRONMENT": "Development"
},
"sourceFileMap": {
"/Views": "${workspaceFolder}/Views"
}
},
{
"name": ".NET Core Attach",
"type": "coreclr",
"request": "attach",
"processId": "${command:pickProcess}"
}
]
}

No VS Code can not find dot net core cli because I can not install any program and I can not copy dotnet folder to C:\Program Files because I have not the admin user pass.
Just tell me how can I set a setting for VS Code to define dot net code path to C:\dotnet?

@mammadkoma actually c# vs code extension is provided by Omnisharp you could try to ask also here https://github.com/OmniSharp/omnisharp-vscode/issues

https://github.com/OmniSharp/omnisharp-vscode/issues/2772


Webpack migration

Is there any tool to migrate webpack configuration into fusebox ?
This would be very useful tool.

@NN--- i think @aretecode is working on one?

FuseBox v4 Major Upgrade
As of today, the latest major version of FuseBox v4 is available in fuse-box@next. FuseBox has undergone a full re-write from scratch, learning from its mistakes in the previous versions.
There are a lot of improvements, speed and config wise.
Get started here
We are still working on the documentation and polishing before the major release. We are getting very close. And we need help testing.
npm install fuse-box@next

Most of the issues that I am closing have been resolved, however, if you find that your issue is still relevant in v4 don't hesitate to re-open or (better) create a new one with v4 configuration.


Checklist keyboard accessibility.

I'm submitting a ...

[ x ] feature request

As tested on: https://jsfiddle.net/icebob/0mg1v81e/

What is the current behavior?
When keyboard tabbing through a form, focus will skip the checklist field. It is not able to be expanded / collapsed.

What is the expected behavior?
Focus is given to the checklist and on a keypress (Enter), the list can be expanded collapsed to access the fields within.

The current checklist is not accessible via keyboard. Changing the .info <div> to a <button> (and adding an aria-expanded tag) would solve this issue.

Yep, I can confirm that is the case, the VFG is not fully accessible. Can this be looked into, please?


Transient zombie failure on Travis

Builds fail periodically with An error occurred running zombie tests TypeError: connect ECONNREFUSED <IP_ADDRESS>:8770 on travis.  The build works if you kick it off again, but its not a very good experience for contributors.
see also
https://github.com/redfin/react-server/pull/156
https://github.com/redfin/react-server/pull/151
https://github.com/redfin/react-server/pull/164

Lovely.  Thanks for filing @doug-wade.

@gigabo didn't you fix this in https://github.com/redfin/react-server/pull/456 ?

Yes.  Fixed... forever. 😐


Support for Vector Processor (RVV 1.0)

Hi, Is this OS has support for RVV 1.0 or in other words I can compile the code for vector extension using the risc-v toolchain compiler available in it?
Thank you

TH1520 only has its "THead extension", the RVV 0.7.1 draft.
the official GCC toolchain do not support this extension, so this flake do not have support for it too.
As for  RVV1.0, just use a GCC toolchain(the latest version of gcc13 maybe?) that supports RVV1.0 and the system will support it.

Thanks for the clarification regarding TH1520 and RVV support.
So, that meant, currently no commercial hardware is available with rvv 1.0 and as of for risc-v toolchain gcc, it has vector rvv 1.0 available, I am already using gcc 13.2 (ubuntu glibc 22.04) for cross compiling rvv 1.0 and running it on qemu-riscv
Thank you


[Doc][Improvement] Follow the order of user clicks in navigation steps

Doc: https://docs.wso2.com/display/IS540/Self+Sign+Up+and+Account+Confirmation
Please refer to step 3 under "Self sign up for super tenant users"
This should be updated as,
"Navigate to : Main tab -> Identity Providers -> Resident -> expand Account Management Policies tab"
or a similar format which follows the order of user clicks.

updated the docs [1], [2], [3] and [4]
[1] https://docs.wso2.com/display/IS550/Self+Sign+Up+and+Account+Confirmation
[2] https://docs.wso2.com/display/IS541/Self+Sign+Up+and+Account+Confirmation
[3] https://docs.wso2.com/display/IS540/Self+Sign+Up+and+Account+Confirmation
[4] https://docs.wso2.com/display/IS530/Self+Sign+Up+and+Account+Confirmation


Add an iptables rule for letting weaveDNS requests in

Fixes #1578
(continued from #1581)

There are a couple of bugs:

the rules should also be added when attaching a router, i.e. really the addition should happen in attach_router. That also means it should be triggered on -z "$NO_DNS_OPT", since $DNS_PORT_MAPPING is not always available in attach_router.
the rules should be removed on weave reset

An alternative would be to install the rules in create_bridge and remove them in destroy_bridge. That has the advantage of keeping all the iptabling (except for expose) in one place. The disadvantage is that the rule may not be necessary.

An alternative would be to install the rules in create_bridge and remove them in destroy_bridge.

+1

Issue  #1631 is also affected by this: pot 6784 is not open for connections coming from other containers. Should I also open this port? Any other ports?

port 6784 is not open for connections coming from other containers. Should I also open this port?

That would be a spectacularly bad idea. See #1632.

That would be a spectacularly bad idea. See #1632.

:joy: ok, I'll take that as a no...

@inercia Has this been accidentally rebased on master? There are 19 commits showing in this PR that look like they come from there...

@inercia Has this been accidentally rebased on master? There are 19 commits showing in this PR that look like they come from there...

Damn! Fixing it...


disable org.owasp dependency-check-maven upon mvn clean install but enable during mvn clean site

What I am trying to do:

I want to avoid running org.owasp dependency-check-maven when I run mvn clean install. On the other hand, I would also like it to run on mvn clean site.

What I tried so far:

In my pom xml file, I have this code:
 <plugin>
                <groupId>org.owasp</groupId>
                <artifactId>dependency-check-maven</artifactId>
                <version>8.4.2</version>
                <configuration>
                    <assemblyAnalyzerEnabled>false</assemblyAnalyzerEnabled>
                    <formats>
                        <format>html</format>
                    </formats>
                </configuration>
                <executions>
                    <execution>
                        <goals>
                            <goal>check</goal>
                        </goals>
                    </execution>
                </executions>
            </plugin>

Issue:

As of right now, during mvn clean install a whole dependency check is run
[INFO] --- dependency-check:8.4.2:check (default) @ project ---
[INFO] Checking for updates
[INFO] Skipping NVD check since last check was within 4 hours.
[INFO] Skipping RetireJS update since last update was within 24 hours.
[INFO] Skipping Hosted Suppressions file update since last update was within 2 hours.
[INFO] Skipping Known Exploited Vulnerabilities update check since last check was within 24 hours.
[INFO] Check for updates complete (18 ms)

Which is lengthy, and only needed when the whole site is needed.

Question:

How to run dependency check only for mvn site, and not mvn clean install

See example 2: https://jeremylong.github.io/DependencyCheck/dependency-check-maven/index.html
The plugin should only be defined in the reporting section.

Thank you @jeremylong , very clear. Good day!


Fix docs for server logging

That got me past the recompilation, and autocomplete is working great. Thanks!
I did manage to dump the entire command used for the version check (classpath and all), and it's working correctly when I run it manually. Not sure why the stdout event handler isn't getting called, but I'm happy to run with that line commented out.


Kolnovel new update

Steps to reproduce
I don't know why kolnovel new update got lines mixed
Expected behavior
It should be evry sentence in it's right place
Actual behavior
Lines mixed with each other
LNReader version
2.0.0
Android version
Android 14
Device
Oppo Reno 11
Other details
Nothing really
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 2.0.0.
[X] I will fill out all of the requested information in this form.

can you send screenshot and what novel/chapter you are seeing it?

Sentence that supposed to be in the end of the chapter now it's in the middle like that date you see in the middle of the chpater


Make "onlyBreakTreesWithLeaves" config default to "true", or add an in-game toggle.

I understand it was deliberately left as false to keep with the original behavior of the mod (as per the comments in #2), but I think the commonly expected behavior of a tree-cutting mod is to only work on leaved logs by default.
I went to change the setting immediately after trying to deconstruct a wooden structure.
Sneaking or using the toggle keybind lets you work around this while keeping the setting enabled, but perhaps a better solution might be to implement a third state or separate toggle to enable/disable this functionality specifically.

Hey, thanks for posting here!
I definitely don't like the current setup, where you need to restart the game to change some client-side configs. I think I'll take this opportunity to make an in-game interface to handle client-side options. This will also let the mod inform to users when server settings override certain options, like in #50, rather than leaving players confused about why their settings aren't working.

Sorry for the wait! As of 0.14.0, onlyBreakTreesWithLeaves has been renamed to treesMustHaveLeaves and now defaults to true. I've also added an in-game configuration screen (bound to the n key by default) that lets you change this option without restarting.
It's only for 1.16.3 - 1.16.5 right now, but I'll update the earlier versions eventually


sendMessage doesn't work for sending files

It looks like SendMessageData doesn't account for any of the file (and related) properties (e.g., url, file_name, etc) which then means that the serializer drops the properties before request.setBody is called

Thanks for the report @jwhitmarsh
Let me take a look at this and see get it fixed for you.
I will circle back here at the start of next week with a new release of the sdk.

Hi @jwhitmarsh 👋🏼
I have just released version 0.0.15 of this sdk with support for sending messages with a file.
I have done some local testing and successfully send file messages. Let me know if you have any issues. Thanks


DX: automatically generate priorities basing on priority tests

The algorithm for calculating priorities is quite simple:

set all priorities to 0,
for each relation (based on integration priority tests) update the priority of fixer that must be run earlier and update all priorities of fixers that must run before it recursively (logic is in Priority class).

Main questions for this RFC are:

Is the PriorityFixer good approach or we only want AutoReview test, something similar to FixerFactoryTest::testFixerPriorityComment?
In the PrioritiesCalculator should I remove the lines:    $priorities['final_class'] = $priorities['final_internal_class'];
    $priorities['no_multiline_whitespace_before_semicolons'] = $priorities['multiline_whitespace_before_semicolons'];
    $priorities['no_extra_consecutive_blank_lines'] = $priorities['no_extra_blank_lines'];

or they could stay? They are because of using AbstractProxyFixer.
3. Any hint on how to make fabbot.io happy?

This might prove to be a pain for merge conflicts?

Great idea :)
I remember a talk with @keradus on this topic as well. For 3.0 I think it might be worth to change the priority system to something not number based. For example, some thoughts;
Lets introduce an interface like;
<?php

interface FixerPriority
{
    /**
     * @return string[]
     */
    public function getBeforeRules(): array;

    /**
     * @return string[]
     */
    public function getAfterRules(): array;
}

Update FixerInterface

public function getPriority();

to

public function getPriority(): FixerPriority

Introduce a default implementation;
final class DefaultFixerPriority implements FixerPriority
{
    /** @var string[] */
    private array $after;

    /** @var string[] */
    private array $before;

    /**
     * @var string[] $before
     * @var string[] $after
     */
    public function __construct(array $before, array $after)
    {
         $this->before = $before;
         $this->after = $after;
    }

    public function getBeforeRules(): array
    {
        return $this->before;
    }

    public function getAfterRules(): array
    {
        return $this->after;
    }
}

Update all the fixers to return the priorities, for example for TernaryOperatorSpacesFixer;
    public function getPriority(): FixerPriority
    {
        return new DefaultFixerPriority([], ['array_syntax', 'list_syntax']);
    }

So now; no more numbers.
Update the abstract fixer test to see if the priorities returned make sense, i.e. each value is a string and of a known fixer, it is not itself and not in both the before and after list and there is a priority integration test fixture for it.
Also test that for each before rule that rule has the other one in its after and v.v., so in the example we expect array_syntax to return ternary_operator_spaces in its before list.
Now we no longer need the list in the AutoReview\FixerFactoryTest for a lot of test cases.
The hard part if of course sorting a list of rules based on this newly returned FixerPriority values.
For the known fixers in the project we can create the list each time we add or remove a fixer or when updating the priority values. This will speed up things during run time.
However we must also be able to deal with 3rd party fixers, i.e. mix those in to figure out the priority list.
We must also be able to detect priority conflicts, for example rule A, B, C with priorities Ap, Bp and Cp, if Ap > Bp and Bp > Cp but Cp > Ap will not work.


LoStik: needs driver in raspberry PI?

Hi, i want to use the lostik with my raspberry 3. The raspberry seems to recognize the lostik but i cant connect to serial port.
Mi question is if lostik needs driver installation in raspberry.
Greetings

	What distro are you using?

On Mon, Mar 25, 2019 at 8:49 AM -0700, "neypalma"<EMAIL_ADDRESS>wrote:
Hi, i want to use the lostik with my raspberry 3. The raspberry seems to recognize the lostik but i cant connect to serial port.
Mi question is if lostik needs driver installation in raspberry.
Greetings
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

What distro are you using?
Im using rasbian stretch

So I installed rasbian-strech-full on SD card, booted it and the device mounted to /dev/ttyUSB0
Unfortunately I didn't get any further before my RPI A croaked. I'm not sure what happened, but it's no longer with us.
Are you not seeing a device at /dev/ttyUSB0
On Mon, Mar 25, 2019, at 9:14 AM, neypalma wrote:

What distro are you using?
Im using rasbian stretch

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ronoth/LoStik/issues/11#issuecomment-476269270, or mute the thread https://github.com/notifications/unsubscribe-auth/AADshLC8w90V-xTv48kZnInBeb-xqJwnks5vaPX5gaJpZM4cHOdN.

Hi, yes I see TTYUSB0, I connect to it but can interact with the board. There is no message. The device seems to be hanged after making the connection

What terminal program are you using to connect?
On Mon, Mar 25, 2019, at 1:20 PM, neypalma wrote:

Hi, yes I see TTYUSB0, I connect to it but can interact with the board. There is no message. The device seems to be hanged after making the connection

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ronoth/LoStik/issues/11#issuecomment-476361262, or mute the thread https://github.com/notifications/unsubscribe-auth/AADshLQsoiz6fHpuyw-gmZKWJUrlx6-kks5vaS-IgaJpZM4cHOdN.

I try putty and minicom

Putty is known to have issues because the RN2903 requires \r\n endings.
I'm not really familiar with minicom. I would recommend trying the miniterm.py script in the examples directory first. There is also a gui program released by Microchip, but requires Java to run.
There is a bit more info on how to run miniterm and where to download the LoRa Development Suite Gui here.
https://github.com/ronoth/LoStik
On Mon, Mar 25, 2019, at 2:02 PM, neypalma wrote:

I try putty and minicom

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ronoth/LoStik/issues/11#issuecomment-476376205, or mute the thread https://github.com/notifications/unsubscribe-auth/AADshBlG82puP1Db75PO8vWjiiwAiiZdks5vaTlqgaJpZM4cHOdN.

@lolsborn  i have had success using picocom program and running it like this

picocom --omap crcrlf,delbs --imap crcrlf,delbs -c -b 57600 -r -l /dev/ttyUSB0

Greetings


[bugfix] fix final exporter not work bug

fix final exporter not working bug.

 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.杨熙 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix incorrect max_gen_toks generation kwarg default in generative Bigbench

cc @lintangsutawika
closes #1513 .

Hi @haileyschoelkopf, thank you for the pr regarding the issue #1513. Can you please explain how this can help to resolve the issue? If input_ids is more than the defined max_token, will the input be truncated to only 128 or something else will happen entirely?
Also, I am getting same error in all code2_text too. Thank you!

Well, I ran with bigbench_code_line_description_generate_until and it runs successfully.

Taking this as reference, I have made changes in code2_text. Can I make a PR for this? @haileyschoelkopf

Yep, go for it!


Cosine metric calculation uses different precision in default provider than in Panama and native

Cosine metric calculation uses different precision in final calculation in default and Panama/native providers so they produce slightly different results. Usually that should not be important, but we had some test failures after enabling Panama because different search results were returned.
The following test (after adjusting visibility for simplicity):
    @Test
    public void testCosinePrecision() {
        VectorUtilSupport dvus = new DefaultVectorUtilSupport();
        VectorUtilSupport pvus = new PanamaVectorUtilSupport();
        ArrayVectorFloat v1 = new ArrayVectorFloat(new float[]{99, 150});
        ArrayVectorFloat v2 = new ArrayVectorFloat(new float[]{99, 149});
        assertThat(pvus.cosine(v1, v2)).isEqualTo(dvus.cosine(v1, v2));
    }

produces
expected: 0.99999523f
 but was: 0.9999953f

The cause is last line of cosine metric calculation. In default provider it is:
    return (float) (sum / Math.sqrt((double) norm1 * (double) norm2));

In Panama and native provider it lacks cast to double and loses some precision:
    // float aMagnitude, bMagnitude;
    return (float) (sum / Math.sqrt(aMagnitude * bMagnitude));

For consistency and lack of surprises, all providers should cast to double and sqrt requires double anyway.

Thanks for pointing this out.  I'm pretty sure the right fix is the other way around, to use float everywhere.  @jkni ?

I do note though that if you're hoping to get 100% consistent results from floating point math, you're probably going to be disappointed anyway.  See discussion here https://twitter.com/benwtrent/status/1820946252361568630

I did not really think about order of additions (thanks for the link!), but that can be another source of surprising differences between providers. There is not much we can do with that.
I agree that using only float seems to make more sense. I created PR with that approach.


Issue installing on Ubuntu Mate 16.04

I get the following issue when running backdoor.py. I ran the install.py as well.
[root:...entest/the-backdoor-factory]# ./backdoor.py                 (master✱)
Traceback (most recent call last):
File "./backdoor.py", line 52, in
from pebin import pebin
File "/opt/pentest/the-backdoor-factory/pebin.py", line 50, in
from intel.intelCore import intelCore
File "/opt/pentest/the-backdoor-factory/intel/intelCore.py", line 38, in
from capstone import *
File "/usr/local/lib/python2.7/dist-packages/capstone/init.py", line 230, in
raise ImportError("ERROR: fail to load the dynamic library.")
ImportError: ERROR: fail to load the dynamic library.
[root:...entest/the-backdoor-factory]#                               (master✱)

Capstone did not fully install. Good luck!

On Oct 19, 2016, at 9:58 PM, mother2110<EMAIL_ADDRESS>wrote:
I get the following issue when running backdoor.py. I ran the install.py as well.
[root:...entest/the-backdoor-factory]# ./backdoor.py (master✱)
Traceback (most recent call last):
File "./backdoor.py", line 52, in
from pebin import pebin
File "/opt/pentest/the-backdoor-factory/pebin.py", line 50, in
from intel.intelCore import intelCore
File "/opt/pentest/the-backdoor-factory/intel/intelCore.py", line 38, in
from capstone import *
File "/usr/local/lib/python2.7/dist-packages/capstone/init.py", line 230, in
raise ImportError("ERROR: fail to load the dynamic library.")
ImportError: ERROR: fail to load the dynamic library.
[root:...entest/the-backdoor-factory]# (master✱)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks. You were correct. It is working now.


Fixing Issues with Terraform Integ readme

Fixing Issues with Terraform Integ readme

@nabuskey @nimakaviani Pls approve and merge.

The URL format seems to be correct as is.
https://github.com/cnoe-io/idpbuilder/pull/255


Support inferModulePath for test suites as automatic modules

I have several large legacy test suites that require JavaFX dependencies to be loaded via the module path instead of the classpath. These tests also have dependencies on the unnamed module, so I can't readily make the tests themselves named modules by adding a 'module-info.java' file to the test source set.
I could --add-reads ALL-UNNAMED each named test module, but that requires adding args for the forked test VM, or leveraging the Gradle Modules Plugin. From the docs I got the sense that I could use inferModulePath instead by making the tests an automatic module:

As discussed in the previous section, an automatic module can be used as an adapter between your real modules and a traditional library on the classpath.

I tried configuring an Automatic-Module-Name Jar manifest entry for my test Jar, but that doesn't seem to trigger Gradle to put dependency Jars that are also named or automatic modules on the module path for the test execution. It would be nice if it did.
Expected Behavior

Making a manually confiugured or JVM Test Suite Plugin configured test source set an automatic module, by configuring an Automatic-Module-Name Jar manifest attribute for its test Jar, triggers the inferModulePath behavior of adding direct and transitive test runtime classpath Jars that are also modules (named or automatic) to the module path for the Test task execution.

Current Behavior

Making a manual or JVM Test Suite configured test source set a named module, by adding a 'module-info.java' file, triggers the inferModulePath behavior, but making it an automatic module, via an Automatic-Module-Name test Jar manifest attribute, apparently does not. (As determined by running with --debug, locating and inspecting the temporary @args file used for the forked test VM, and noting that it only contains a -cp entry, not a --module-path entry. When configured as a named module, `--module-path is populated.)

Context
Large legacy test suites that do not well qualify to be made directly into named modules (e.g. due to dependencies on the unnamed module), but that need to do blackbox integration testing (e.g. due to JavaFX requirements), could be migrated more incrementally as automatic modules.
I made a reproducer by modifying the "Building Java Modules with Blackbox Tests Sample":

https://github.com/ianbrandt/java-modules-multi-project-with-integration-tests/tree/automatic-test-module-path-inference.

I modified the sample as follows:

I changed all the existing named modules to automatic modules by renaming their 'module-info.java' files to 'module-info.java.orig'.
I added Automatic-Module-Name attribute configurations for their Jar manifests.
I attempted to make the utilities subproject's integration tests an automatic module by configuring its corresponding Jar manifest.

No sure if I got the complete picture, but Gradle always puts Jars with Automatic-Module-Name on the Module Path.
More thoughts on that here: https://github.com/gradle/gradle/issues/20075#issuecomment-1125082613
In you reproducer, you should configure the Automatic-Module-Name for the test Jar (not only the main Jar) which would be the task registered here: https://github.com/ianbrandt/java-modules-multi-project-with-integration-tests/blob/main/buildSrc/src/main/kotlin/myproject.java-conventions.gradle.kts#L21

Hi @jjohannes,
Thanks for the comment, and for all your contributions to Gradle and the community.
I believe I did set the Automatic-Module-Name for the test Jar here:
https://github.com/ianbrandt/java-modules-multi-project-with-integration-tests/blob/automatic-test-module-path-inference/utilities/build.gradle.kts#L17
My observation though is that it did not result in the integration test dependencies being put on the module path. The temp args file created for the forked test run showed all dependencies added to -cp only. Is my approach to setting the Automatic-Module-Name there incorrect?
If I make the same suite a named module by adding a module-info.java instead, I do observe --module-path in the temp args file for the forked test run. The issue with that is I'm trying to migrate some legacy tests that need access to the unnamed module, so I need the tests to be an automatic module, not a named module.
An alternative approach I tried was to make the tests a named module, but give them access to the unnamed module via --add-reads ALL-UNNAMED. I couldn't get that to work, however. The test failed to compile for unresolved symbols from the unnamed module (and yes, I configured the --add-reads for the compile task in addition to the test task). I'm attempting to create a reproducer for that now, and will follow up.

@ianbrandt sorry my bad. I did not look close enough at the reproducer.
What I did not think about is that there are two mechanisms at play:

Should the test "run as module" - i.e. is there a Module Path at all?
What to put on the Module Path?

For 2. you did everything correct, but 1. is a bit more tricky (and maybe improvable in Gradle). This is done specifically inside the Test task. The task looks at the testClassesDirs to discover which tests exist. It also uses this to determine if the tests "are a module" (maybe this is not good enough). The testClassesDirs is different from the classpath, which is used later to construct the classpath (and possibly the Module Path) to run the tests that have been discovered.
It seems that you can add your Jar to the testClassesDirs and then the Automatic-Module-Name is picked up and Gradle runs the tests "as module". (Note that you still need the classes dir for test discovery, as Gradle cannot discover tests inside a Jar.)
In your example in myproject.java-conventions.gradle.kts replace:
testClassesDirs = integrationTest.output.classesDirs
by
testClassesDirs = integrationTest.output.classesDirs + files(integrationTestJarTask)
And I think then it works as you would like to have it.

Thank you @jjohannes, That seems to have done the trick.
It would be nice if Gradle did this (or somehow achieved equivalent behavior) without user configuration, so perhaps this issue could be for that. Between this and #17271, I'm finding there's a fair amount of undocumented requirements and configuration necessary to work with Kotlin and the Java Module System.


Fixed Broken/dead link in Arabic file.

See: #154
Made changes to the Arabic file.

@BatoolMM still waiting for your appeoval
thank you.

Fantastic, thank you 🙏

my pleasure.


fixproject should be done inside inlineview

Proposed changes
this is a follow up pr of https://github.com/apache/doris/pull/17035
Problem summary
The previous pr has defect, this pr make the correct changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.65 seconds
stream load tsv:          469 seconds loaded<PHONE_NUMBER>9 Bytes, about 152 MB/s
stream load json:         38 seconds loaded<PHONE_NUMBER> Bytes, about 59 MB/s
stream load orc:          74 seconds loaded<PHONE_NUMBER> Bytes, about 14 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230315151444_clickbench_pr_114815.html

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


How its work?

I have a file: test.gql
query GetAuthState {  authState }
I have an import:
import { GetAuthState } from './graphql-operations/test.gql'
And its produce this error:
Uncaught SyntaxError: The requested module '/src/graphql-operations/test.gql?t=1676882148580&import' does not provide an export named 'GetAuthState' (at apollo-client.js?t=1676882156504:7:10)
Whats wrong?
P.S. SandBox to reproduce: https://codesandbox.io/p/sandbox/rough-dream-2jsm9q?selection=[{"endColumn"%3A1%2C"endLineNumber"%3A9%2C"startColumn"%3A1%2C"startLineNumber"%3A9}]

I think its configuration problem. Its should be investigated.
Can you also add to project section 'Example' with minimal full featured solution.
Thx!

I am running into this same issue. I have realised that vite is assuming that the env is esm and not commonjs. So this package is creating a commonjs virtual package. Can't this be changed to just generate esm modules instead, where it generates module.exports = doc and module.exports['...'] = ... to export default doc and export const '...' = ... making it an esm module.
However, it should be interesting to see the actual configuration that makes this work with vite without doing this as @TomKein suggested.

@TomKein if you still working on this, try version 2.0.0 which includes @emahuni's fixes.
If you are using TypeScript also include a file in your project called graphql.d.ts:
declare module "*.gql" {
    const Query: import("graphql").DocumentNode;
    export default Query;
}
// Or `declare module "*.gql";` to be able to export fragments

and change your import from
import { GetAuthState } from './graphql-operations/test.gql'

to
  import GetAuthState from './graphql-operations/test.gql'

I've added these instructions to the README.


Set buffer localvars for channel and server

These are the same as what the irc plugin sets, and can be used in the
logger masks among other things.
This fixes #188.

Looks fine to me. @rawdigits, ok to merge?

Hm, with this, the server var is set to <team>.slack.com. If my suggestion in https://github.com/rawdigits/wee-slack/issues/144#issuecomment-202512332 is done, it would be set to slack.<team>.
Maybe it should be set to only the team name instead? This would make it more like to the irc plugin, where it is only set to the server name (e.g. "EFNet").

Just the team name would be fine, it'll be namespaced by the plugin name anyway, won't it? (For logging.)

Yeah, it makes more sense for logging as well. That way, you can choose if you want slack as part of the file name or not. I changed it.


[en]added explanation for unprivileged access

Part of: https://github.com/kubernetes/website/issues/38436
The preview page:

Link

👋 @Saliha067 .
I am closing this pull request. If you have questions, feel free to reopen it.
/close


Change PHP-FPM User and Group

Also would recommend changing the process manager to 'ondemand' rather than static. That's up to you whether or not you want to add that. This fixes the permission issues with nginx and wordpress.

Hi @marshki! Looks good. :+1:
I reproduced the issue by attempting to install a theme and confirmed this resolves it.
Thanks so much for following up on this. Sorry for letting it drop in the first place.

No problem @adaml1037! See what I did there :D

Opps! :flushed:


Boxes are not displaying

I have included it's css, jaevascript and all the necessary parts but boxes are not displaying on the page. It's showing only white area and i was also unable to find the css "level-1".
Please help me, i will be very thankful to you.

You might take a look on the getting started example:
http://vnjs.net//www/project/freewall/example/getting-started.html
Best


RTL support

Hello Lee
When I export to PDF everything is left aligned.
I'm not using the RTL build in option that obsidian has developed.
Instead, I'm using "RTL Support" becuase it keeps every language at his own direction. Exporting to pdf with "RTL Support"
plugin has no problems (in 1.2.1 version).

Version 1.6.3 was released. It seems the problem has been fixed

That's great! I noticed that one of the highlights of the 1.6 version update is improved RTL support. Thank you for your feedback!


[Scribble] Internal Error: attempt to index a nil value

Description
Error at launch
Metadata

Key
Value

Title
Scribble

Source URL
https://archive.org/download/3-libjune-05/3LIBJUNE05.iso/library%2Fepocgraphics%2Fscribble%2FScribble.sis

Details
attempt to index a nil value
SCRIBBLE\APPINIT:00002568: FF [NextOpcodeTable] 18 CallOpxFunc 0 45 (SYSTEM ReadRsc)
SCRIBBLE\START:00000739: 53 [RunProcedure] 0x06BC (name="APPINIT" nargs=0)
stack traceback:
	[C]: in for iterator 'for iterator'
	opl.lua:1514: in function <opl.lua:1512>
	(...tail calls...)
	opx/system.lua:385: in function 'opx.system.ReadRsc'
	ops.lua:2454: in function 'ops.CallOpxFunc'
	ops.lua:2332: in function 'ops.NextOpcodeTable'
	runtime.lua:989: in function <runtime.lua:974>
	[C]: in function 'xpcall'
	runtime.lua:1108: in method 'pcallProc'
	runtime.lua:1426: in function 'runtime.runOpo'

That's an interesting one... it's trying to read a resource from eikon.rsc, which apparently is implicitly opened by OPL programs. (the nil index is due to our impl of ReadRsc not expecting someone to call it without ever having called LoadRsc).
The resource in question appears to be the text "Busy" so I guess this application is using it as a way to leverage existing locallisations.

Nice find!


Removed old Codec/ByteChannel/MsgChannel and Framing

Not waiting for a review, this just removes obsolete code.


Change material for entire model

Is there an idiomatic way to change the material for an entire model?
I have tried:
model.items.forEach((itm) => { itm.mesh.material = [new three.MeshBasicMaterial({ color: "#fff", opacity: 0.5 })] })

but obviously, this is quite slow for an IFC with many elements.
My use case is for loading multiple models into a scene and being able to selectively set their opacity.

@tymokvo all the items of the same BIM model can be in 2 groups: transparent and opaque. All the items of the same group share the same material for memory saving's sake.
So if you pick any opaque item, or any transparent item, and change its material properties (like opacity and transparency), all the items of the same group will be affected. Of course, you can create a material per mesh, but that's going to be inefficient. Not because of our library, but because of three.js works.
If you want to make all the model transparent except for some items, I would do it like this:

pick any opaque material of the model and set its properties transparent=true, opacity=0.5.
create a clone of the items you want to set as opaque. You can clone a subset of the model.
add a new material to the cloned items.


Update browser output for front end pipeline

These updates were made to address the GitHub issues outlined here. The association output has been modified so that it is easier to read into the front-end workflow, as well as reduce computational cost by combining into one script. *Note some code may change once Hail Batch is updated tomorrow morning (see this post).
@daniaki and @danielreti , I've addressed the changes in the doc, but please let me know if there's something that should be changed or added. I also haven't added in the LD information section, as this will be run on the unique union set of significant SNPs across all cell types after running the association analysis on the whole genome.

@katiedelange @daniaki @danielreti I removed the filtering step on the ld calculation and reverted the chromosome field in the association output back to the 'chr' prefix. Tests are also all running sucessfully (including the wrapper script 🎉). Let me know what you think.

Thanks Kat! In reference to @katiedelange's point above, my preference goes to TSV (gzipped to save space for large files) as outputs to avoid issues when commas appear in column values.

great, thanks @daniaki. Maybe we can set that as the default then? I'll make sure to use TSV going forward


fixfill up miss slot of order having project

Proposed changes
fill up miss slot of order having project such as
select a + 1 as c from t having by c > 2 order by a

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


[Bug]: After a short time after (re)start of ADOT pod, metrics flow to managed Prometheus stops for most of metrics

Welcome to the AWS Observability Accelerator

[X] Yes, I've searched similar issues on GitHub and didn't find any.

AWS Observability Accelerator Release version
2.4.0
What is your environment, configuration and the example used?
EKS API v1.24.13-eks-0a21954
Managed Grafana:  version 9.4
Managed Prometheus:
ADOT: v0.74.0 - eksbuild.1
What did you do and What did you see instead?
I have deployed the AWS Observability accelerator blueprint example following the instructions. Right after the restart ALL metrics are showing up OK, then in about 4 min they stop and exception like the following are observed in the ADOT pod (namespace adot-collector-kubeprometheus) :
2023-06-02T23:27:24.158Z	warn	internal/transaction.go:121	Failed to scrape Prometheus endpoint	{"kind": "receiver", "name": "prometheus", "data_type": "metrics", "scrape_timestamp":<PHONE_NUMBER>141, "target_labels": "{__name__=\"up\", cluster=\"do-eks-tf-dz\", instance=\"<IP_ADDRESS>:9100\", job=\"node-exporter\", nodename=\"ip-10-11-3-29.ec2.internal\", region=\"us-east-1\"}"} 2023-06-02T23:27:25.083Z	warn	scrape/scrape.go:1372	Append failed	{"kind": "receiver", "name": "prometheus", "data_type": "metrics", "scrape_pool": "kubelet", "target": "https://kubernetes.default.svc.cluster.local:443/api/v1/nodes/ip-10-11-11-126.ec2.internal/proxy/metrics/cadvisor", "error": "invalid sample: non-unique label names: \"cluster\""} 2023-06-02T23:27:25.083Z	warn	internal/transaction.go:121	Failed to scrape Prometheus endpoint	{"kind": "receiver", "name": "prometheus", "data_type": "metrics", "scrape_timestamp":<PHONE_NUMBER>038, "target_labels": "{__name__=\"up\", beta_kubernetes_io_arch=\"arm64\", beta_kubernetes_io_instance_type=\"c7g.4xlarge\", beta_kubernetes_io_os=\"linux\", cluster=\"do-eks-tf-dz\", eks_amazonaws_com_capacityType=\"ON_DEMAND\", eks_amazonaws_com_nodegroup=\"cpu-graviton-man\", eks_amazonaws_com_nodegroup_image=\"ami-06326b7ef5c114349\", failure_domain_beta_kubernetes_io_region=\"us-east-1\", failure_domain_beta_kubernetes_io_zone=\"us-east-1a\", instance=\"ip-10-11-11-126.ec2.internal\", job=\"kubelet\", k8s_io_cloud_provider_aws=\"51d0ed1b12453098a108c272e71e962f\", kubernetes_io_arch=\"arm64\", kubernetes_io_hostname=\"ip-10-11-11-126.ec2.internal\", kubernetes_io_os=\"linux\", node_cpu=\"graviton\", node_kubernetes_io_instance_type=\"c7g.4xlarge\", node_role=\"compute\", region=\"us-east-1\", scale_model=\"bert\", topology_kubernetes_io_region=\"us-east-1\", topology_kubernetes_io_zone=
Additional Information
The root cause is likely in the following [source file]:(https://github.com/aws-observability/terraform-aws-observability-accelerator/blob/main/modules/eks-monitoring/otel-config/templates/opentelemetrycollector.yaml) There are the sections for job_name: 'kube-state-metrics' and job_name: 'kubernetes-kubelet'  particularly in the subsection of relabel_configs:

Source file location: modules/eks-monitoring/otel-config/templates/opentelemetrycollector.yaml

We will find a workaround for that in a PR shortly, but thanks for opening on the ADOT repo @ktibi

PR submitted, might need to run this for a few days to confirm @dzilbermanvmw @ktibi

@bonclay7 I deployed your fix since this morning. For the moment all is good.
2023-06-07T11:25:37.708Z	info	MetricsExporter	{"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 3, "metrics": 286, "data points": 624}
2023-06-07T11:25:47.127Z	info	MetricsExporter	{"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 3, "metrics": 316, "data points": 2181}
2023-06-07T11:25:54.482Z	info	MetricsExporter	{"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 1, "metrics": 72, "data points": 2055}
2023-06-07T11:25:59.680Z	info	MetricsExporter	{"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 1, "metrics": 336, "data points": 909}

Awesome https://github.com/aws-observability/terraform-aws-observability-accelerator/pull/174#issuecomment-1580610500

We merged a fix https://github.com/aws-observability/terraform-aws-observability-accelerator/pull/174, this will go in the next release. Please reopen if you observe the same behaviour

@bonclay7 I deployed your fix since this morning. For the moment all is good.
2023-06-07T11:25:37.708Z	info	MetricsExporter	{"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 3, "metrics": 286, "data points": 624}
2023-06-07T11:25:47.127Z	info	MetricsExporter	{"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 3, "metrics": 316, "data points": 2181}
2023-06-07T11:25:54.482Z	info	MetricsExporter	{"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 1, "metrics": 72, "data points": 2055}
2023-06-07T11:25:59.680Z	info	MetricsExporter	{"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 1, "metrics": 336, "data points": 909}

Many thanks @bonclay7 and team for addressing this issue, happy to confirm it is working just fine and I'm seeing uninterrupted stream of metrics all the way from ADOT to AMP and AMG!