odo dev --debug

What type of PR is this:
/kind feature
What does this PR do / why we need it:
Which issue(s) this PR fixes:
Fixes #5583
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

@feloy in the past when we were working on debug mode, we used to consider testing things using VS Code launch.json and connecting to the port on which the app's debug mode port is listening.
For this PR, I downloaded the devfile in the tests of this PR, but can't really connect to it. Or, maybe, VS Code is connecting to it, but I don't know what to do. Also, I have never worked with websockets myself, so maybe I'm doing it completely wrong.
Do you think you can share the steps you followed to test the debug mode yourself?
I spent some time on it, but still feel dumb about this thing (how to test debug mode functionality.)

@feloy in the past when we were working on debug mode, we used to consider testing things using VS Code launch.json and connecting to the port on which the app's debug mode port is listening.
For this PR, I downloaded the devfile in the tests of this PR, but can't really connect to it. Or, maybe, VS Code is connecting to it, but I don't know what to do. Also, I have never worked with websockets myself, so maybe I'm doing it completely wrong.
Do you think you can share the steps you followed to test the debug mode yourself?
I spent some time on it, but still feel dumb about this thing (how to test debug mode functionality.)

@dharmit It should be the exact same way as with odo v2. If you follow the steps described in this blog post (Debugging the micro-service), it should work: https://dev.to/feloy/odo-a-tool-to-simplify-development-on-kubernetes-5c9a
To test with the files used in the integration test, you can get the launch.json from the blog post, then run a Debug session and set a Breakpoint where a request is replied (on the res.send('Hello from Node.js Starter Application!'); for example). Then you execute a request on your app endpoint (curl <IP_ADDRESS>:40001) and vscode should pop up and show your program interrupted on your breakpoint

To test with the files used in the integration test, you can get the launch.json from the blog post, then run a Debug session and set a Breakpoint where a request is replied (on the res.send('Hello from Node.js Starter Application!'); for example). Then you execute a request on your app endpoint (curl <IP_ADDRESS>:40001) and vscode should pop up and show your program interrupted on your breakpoint

@feloy I am doing something wrong, it seems.
launch.json:
{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "node",
            "request": "attach",
            "name": "odo dev debug",
            "program": "${workspaceFolder}/server.js",
            "address": "localhost",
            "port": 40002,
            "localRoot": "${workspaceFolder}",
        }
    ]
}

I have added breakpoints at both the lines in the server.js that have Hello from Node.js  in them. I run odo dev --debug, then I start debugging session from VS Code, and finally do curl localhost:40001/curl localhost:40002. Neither of them takes me to the breakpoint.
The feeling of dumbness is getting real bad at this point, so I'll approve since I don't want my limitation to block your PR.
/approve
/lgtm

To test with the files used in the integration test, you can get the launch.json from the blog post, then run a Debug session and set a Breakpoint where a request is replied (on the res.send('Hello from Node.js Starter Application!'); for example). Then you execute a request on your app endpoint (curl <IP_ADDRESS>:40001) and vscode should pop up and show your program interrupted on your breakpoint

@feloy I am doing something wrong, it seems.
launch.json:
{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "node",
            "request": "attach",
            "name": "odo dev debug",
            "program": "${workspaceFolder}/server.js",
            "address": "localhost",
            "port": 40002,
            "localRoot": "${workspaceFolder}",
        }
    ]
}

I have added breakpoints at both the lines in the server.js that have Hello from Node.js  in them. I run odo dev --debug, then I start debugging session from VS Code, and finally do curl localhost:40001/curl localhost:40002. Neither of them takes me to the breakpoint.
The feeling of dumbness is getting real bad at this point, so I'll approve since I don't want my limitation to block your PR.
/approve /lgtm

Can you try with this launch.json (remoteRoot added):
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Attach to remote 5858",
            "type": "node",
            "request": "attach",
            "address": "localhost",
            "port": 40002,
            "localRoot": "${workspaceFolder}",
            "remoteRoot": "/projects"
        }
    ]
}

/override ci/prow/v4.10-integration-e2e
IBM Could tests pass

Can you try with this launch.json:

That works. Thanks!
It works when I curl the port 40001, not 40002. However, the debug port 5858 is forwarded to 40002:
Your application is now running on the cluster
 - Forwarding from <IP_ADDRESS>:40001 -> 3000
 - Forwarding from <IP_ADDRESS>:40002 -> 5858

I think I need to use websockets instead of http when trying to work with 40002, right?

Port 5858 (and 40002) are for the debugger only, for it to be able to communicate with the process inside the container. You, as a developer, don't need to use it directly.

OK, thanks for guiding me patiently. I surely need some debugging 101 class/course. 😞


cut_by_quality3

I am trying to do 3' trimming based on the  quality of the 3' end. I would like to know how exactly --cut_by_quality3  work? Does it start by looking at the quality per base from the 3' end, and it keeps trimming the 3' tail until it reaches a good quality read?
Right now I'm using a command like this:
(my data is PE)
fastp -i  mse-1-CTL_R1_001.fastq -I mse-1-CTL_R2_001.fastq -q 20 --cut_by_quality3 20 -t 20 -l 50 -o mse-1-CTL_R1_001_Q_A.fastq -O mse-1-CTL_R2_001_Q_A.fastq -h mse-1-CTL_001.html

It's based on sliding window.
See https://github.com/OpenGene/fastp#per-read-cutting-by-quality-score


HHH-12996 - Reproducer for missing property paths of IdClass attributes

https://hibernate.atlassian.net/browse/HHH-12996

Rebased on master to see whether the issue persisted after https://github.com/hibernate/hibernate-orm/commit/8535adc05736e9ae93d818d9aff25bf9255a1bbd . The issue is still there, and I think it's related to https://hibernate.atlassian.net/browse/HHH-12969 .


Code seems correct, but giving me error. Not recognizing  that anchor is wrapped.

Challenge Nest an Anchor Element within a Paragraph has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 Your code passes for me. I've checked on Chrome and Firefox and it passes for both. Have you tried resetting your cache and trying again?

closing issue, no update since 21 days.


cqut-thesis-typst:0.1.0

This package provides a Typst template for Chongqing University of Technology (CQUT) thesis. It aims to help students quickly generate PDF format graduation theses that comply with university requirements. Key features include:

Support for undergraduate and postgraduate thesis formats
Customizable cover pages, declaration page, abstracts, table of contents, and acknowledgments
Configurable font settings and double-sided printing mode
Roman numeral pagination for preface and appendix sections
Numbered environments for tables, equations, and theorems
Local editing and preview support

Users can easily customize their thesis by modifying the thesis.typ file. The template is designed to be extensible and can be adapted for various CQUT academic document needs.
This submission includes all necessary files, documentation, and license information as per the package submission guidelines.

emm... how can I fix this pull request problem? In fact. I'm a newer for it...


[FEATURE] Toggle barrier texture and particle

So im pretty sure anyone doesnt want to see the barrier 24/7 so maybe add a keybind / command / gui to toggle the barrier texture and also to toggle particle
Because who want see the barrier particle when you have the textures? (yes i know, I doesn't put the resourcepack yet, I just show the barrier particle)

I will not personally be adding this, but if someone wants to implement this PRs are open.


[Feature]Get plain type legend width and height number.

What problem does this feature solve?
Let developer can auto adjust legend (plain type) position when user resize chart.
What does the proposed API look like?

getLegendWidgh
getLegendHeight

Currently, you can get the bounding rect like this,
const legendModel = myChart.getModel().getComponent('legend');
const legendView = myChart.getViewOfComponentModel(legendModel )
// get the bounding rect of the legend content
const { width, height } = legendView.getContentGroup().getBoundingRect();

It works! Thanks~


Syncronization is incomplete

It looks like most of my phone isn't getting synced to DAV. It's probably because of permissions. Is it possible to grant EasySync permission to access all files on internal storage? Would root be required?

Root is not required.
When you start the app, it will show you a warning if you did not grant permissions.
What's your version of android? And phone model?

I'm running the latest Android 14 on a Pixel 7. I granted all the permissions it asked for. The sync got to 100%, I didn't see any errors. I even went in and renamed a file that was synced and it was resynced.
Would it help if I collected a list of which folders were backed up vs skipped? Eg. In Android, media was backed up but data and obb were skipped.
I uninstalled the app for now but I'd be happy to reinstall and try again. Is there a way to generate a log?

Thanks for your feedback.
That's the way it is intended to work: only media and downloads are synced.
This app is not a backup app, it synchronizes "regular" user files, ie media and downloads, but not internal android data/obb files.

Documentation was updated to reflect precisely what is synced or not: https://github.com/phpbg/easysync#what-is-synchronized
Feel free to comment if it is not clear enough.

Hello,
AIFF (.aif or .aiff) audio files seems not synchronized.


Dukemtmc is unavailable

I have download the dataset,but it's unavailable

I found that the error lies in the MD5 value，would U mind offer the correct md5?

i have finish this problem

However I also found the pre-trained models are damaged


Upgrade to 6217b002ba779aae93f340838d9166253c5407bf

Isn't in nightly yet, leaving PR open to be merged when it becomes nightly

Ah, still a few hours for the next nightly


fix: handle gap for empty text

Problem:
If there is a component returning an empty string, the renderer will take it as non-empty and will add a gap:
windows = {
  autocomplete = {
    min_width = 1,
    draw = {
      columns = {
        { "label" },
        { "kind", "label_description", gap = 1 },
      },
    },
  },
}

In this case, label_description is empty but the renderer is adding a gap

Solution:

I have also removed the manual spacing within the kind and kind_icon components as it should be handled by the gap

Great work as always :)


Fix issue with specifying release notes

Fixed a bug where it wasn't possible to submit release notes.
Unfortunately this solution doesn't handle ' very elegantly if it's written in the release notes.

Updated the tpa action to be based on the RestClient in order to gracefully handle release notes with either " or ' inside of them.
This means that the progress bar has been deprecated. Since the official upload_to_testflight action doesn't show a progress bar neither, then I believe this is an okay solution.
The status messages written in the console to the user matches the one in upload_to_testflight:
https://github.com/fastlane/fastlane/blob/1af33c0884e783867bdc07a324280ec6bef124ad/fastlane_core/lib/fastlane_core/itunes_transporter.rb#L371

Perhaps rename tpa to upload_to_tpa or make it an alias.
It reads much better in your Fastfile.


OpenQA.Selenium.WebDriverException while creating an object for Windows Driver

I am facing "OpenQA.Selenium.WebDriverException" very frequently when I execute the unit test.
I had executed WinAppDriver.exe with admin rights and when I execute the unit test it throws an exception. This exception is primarily observed after adding the Desired Capabilities.
Below is the code snippet where this exception is observed.
AppiumOptions capabilities = new AppiumOptions();  capabilities.AddAdditionalCapability("app", "Root");  WindowsDriver<WindowsElement> localSession = new WindowsDriver<WindowsElement>(new Uri("http://<IP_ADDRESS>:4723"), capabilities);
Issue is observed when creating WindowsDriver object.
Exception Details:
OpenQA.Selenium.WebDriverException: 'Unexpected error. System.Net.WebException: Unable to connect to the remote server ---> System.Net.Sockets.SocketException: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond <IP_ADDRESS>:4723
at System.Net.Sockets.Socket.DoConnect(EndPoint endPointSnapshot, SocketAddress socketAddress)
at System.Net.ServicePoint.ConnectSocketInternal(Boolean connectFailure, Socket s4, Socket s6, Socket& socket, IPAddress& address, ConnectSocketState state, IAsyncResult asyncResult, Exception& exception)
--- End of inner exception stack trace ---
at OpenQA.Selenium.Appium.Service.AppiumCommandExecutor.Execute(Command commandToExecute)
at OpenQA.Selenium.Remote.RemoteWebDriver.Execute(String driverCommandToExecute, Dictionary`2 parameters)'
NOTE:

Added WinAppDriver to the Firewall exception list.
It opens the application but throws the above exception on line where we create object localSession.

Kindly let me know where the issue is and how do I fix it permanently.
I am new to Automation Development environment.
Thanks in advance.

WindowsDriver localSession = new WindowsDriver(new Uri("http://<IP_ADDRESS>:4723"), capabilities,TimeSpan.FromMinutes(3));

Can you try the above?

Thanks @anunay1 for suggestion.
I realized that splash screen was blocking my way. Hence I added below code which fixed the issue.
var currentWindowHandle = session.CurrentWindowHandle; Thread.Sleep(TimeSpan.FromSeconds(5)); var allWindowHandles = session.WindowHandles;  session.SwitchTo().Window(allWindowHandles[0]);
But there is another issue which I have updated in my question.
It will be grateful if you can suggest some fix for that.

Whats the other issue?

Below is the details of the issue which I am facing post fixing the above one.
I want to click on the first menu item (Options) and then select the first sub menu item (One). Below is the code snippet.
WebDriverWait wdv = new WebDriverWait(session, TimeSpan.FromSeconds(10));
foreach (var element in menuItem)
{
if (!string.IsNullOrEmpty(element.Text))
{
if (element.GetAttribute("Name").Equals("Options"))
{
element.Click();
var optionsMenuItem = element.FindElementByName("One");
wdv.Until(x => element.Displayed);
optionsMenuItem.Click();
}
}
}
Here it is able to find the 'Options' menu item but when I perform Click() operation on it, it throws the below exception.

Nothing wrong with your code, what is the value of IsKeyboardFocusable: in inspect.

Thanks @anunay1 for pointing out to this.
The IsKeyboardFocusable property is True but it was my mistake that I accidentally moved the mouse pointer before it tracks and clicks the menu item.


user switching with sddm not possible on unstable, potentially master/17.03/

Issue description
i just switched from gdm to sddm * plasma5,
so i tried to switch users, and viola, unlike with gdm i dont get to choose the 2nd user anymore, so user-switching turned not doable

im closing this one as pebcac, the active session indicator is quite a confusing ux when one is used user switching from gnome


[issue] make p3_maybe_rayon parallel feature non-default

Addresses issue #201, to allow p3_maybe_rayon parallel feature to be non-default. Also remove p3_maybe_rayon dependency from mersenne-31 crate, as currently it is not needed.

As I understand it, the implicit goal of #201 was to ensure end users (like the authors of binaries which use various p3 libs) can control whether rayon is enabled or not.
Since cargo unions features when merging dependencies, I think crates such as dft need to avoid (unconditionally) enabling parallel. So it seems there are two options for crates like dft,

Leave parallel off; have the end user turn it on
Have their own parallel features to control on/off

Both are weird in a way - (1) because the end user might need a dependency on p3-maybe-rayon just to enable parallel, even though they don't directly interact with its code, and (2) suggests a false sense of control, since a crate like dft could have its parallel feature off, but still use parallelization because something else enabled the underlying p3-maybe-rayon feature.
In my opinion (1) seems a little nicer, so that there's still a single parallel feature rather than different features in different crates.

superseded by a slightly different approach in 7965966bf3a308d838c8ecad5b6462509cb150a6


onBeforeClientSignUp event is not being called

onBeforeClientSignUp doesn't appear to be called and as a result the spam checker module does not work.
I'm not sure what is causing this and i haven't checked if the other event calls are being ignored. I'm going to keep digging and try to figure it out. If anyone has any insight, feel free to comment.

i will try too

Ok, so if i have figured this out correctly, the event manager looks in the database and grabs a list of event hooks to be called from the various modules.
But looking at a freshly installed database, the information that it is looking for is not there. Now to find out why..
here is the extract of code that does this lookup:
private function _connectDatabaseHooks(&$disp, $event)
{
    $sql="SELECT id, rel_id, meta_value
        FROM extension_meta
        WHERE extension = 'mod_hook'
        AND rel_type = 'mod'
        AND meta_key = 'listener'
        AND meta_value = :event
    ";
    $list = $this->di['db']->getAll($sql, array('event'=>$event));

    // no need to connect listeners
    if(empty($list)) {
        return ;
    }

    foreach($list as $listener) {
        $mod = $listener['rel_id'];
        $event = $listener['meta_value'];
        try {
            $s = $this->di['mod_service']($mod);
            if(method_exists($s, $event)) {
                $disp->connect($event, array(get_class($s), $event));
            }
        } catch(Exception $e) {
            error_log($e->getMessage());
        }
    }
}

In case anyone else has this issue, this happens when you don't setup the cron job, as @samapraku brought to my attention:

The hooks are initialised at the first cron job run and gets updated if it needs to be, each time the cron job executes.

If like me you use Boxbilling for development only and don't require an active cron job, you can manually run the cron job once after installation.


Not registerTreeDataProvider to default explorer

  vscode.window.registerTreeDataProvider(     "explorer",     new FolderAliasTreeProvider()   );
or
  vscode.window.registerTreeDataProvider(     "Folders",     new FolderAliasTreeProvider()   );

@Muromi-Rikka I don't understand the issue, can you please provide more details about what you expect to happen and what's actually happening?

@alexr00 I want to achieve this effect in the default explorer

The docs show how to contribute a view to the "Explorer" view container: https://code.visualstudio.com/api/references/contribution-points#contributes.views

@alexr00 Instead of adding a new view, I want to modify the default explorer

@Muromi-Rikka we do not support modifying existing views, but you can always duplicate some of the functionality of the default explorer with a tree view, use the menu contribution point to add more actions to the explorer, or use icon themes to change the icons!


update backbone dependency to latest backbone

update backbone dependency to latest backbone

Thanks!

I'll cut a release tomorrow AM.

Woops. Can't release from work 'cuz we use a private npm and I haven't yet figured out a good system to switch back and forth between the two. I'll release when I get home :see_no_evil:

Forgot about this until just now 🤕
Just released it on npm :tada:


[Enhancement]Support  last offset of kafka partition in routine load task

Add the latest offset of the Kafka partition in the routine load task to observe the lag between Kafka and starrocks

What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)
[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:heart_eyes: pass : 9 / 10 (90.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/routineload/KafkaTaskInfo.java
7
8
87.50%
[205]

:large_blue_circle:
com/starrocks/load/routineload/KafkaRoutineLoadJob.java
2
2
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 9 / 10 (90.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/routineload/KafkaTaskInfo.java
7
8
87.50%
[205]

:large_blue_circle:
com/starrocks/load/routineload/KafkaRoutineLoadJob.java
2
2
100.00%
[]


Issues with Logger

Stdout logger shows wrong timestep
Tensorboard logger saves one event file for each episode. Is this normal?

Also, beautify the printing format. Right now it just prints numbers directly. It would be good to control precision of numbers like episode rewards, etc which have a lot of decimals

Stdout logger shows wrong timestep

For which trainer?

Also, beautify the printing format. Right now it just prints numbers directly. It would be good to control precision of numbers like episode rewards, etc which have a lot of decimals

We are logging only reward i guess as of now for train and for evaluate method we have mean and reward. I'll set all of those with precision 4.

The timestep thing isn't fixed yet right?

Is the timesteps issue only with DQN or do you face this issue with any other agent?

Its probably better to just use episodes than timesteps
timesteps is always confusing and depends on a lot of thing.
Any thoughts?

Since episode sizes vary across games, it'll be hard to compare different
games on difficulty level, if we use number of episodes. I think this is
also why most papers use timesteps
On Fri, 29 May 2020, 08:45 Sharad Chitlangia<EMAIL_ADDRESS>wrote:

Its probably better to just use episodes than timesteps
timesteps is always confusing and depends on a lot of thing.
Any thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SforAiDl/genrl/issues/134#issuecomment-635732781, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AGQWIY64X6I6C47RNQUCLCLRT4SFFANCNFSM4NI32F7Q
.

Hmm yeah, could also get confusing when there an agent has >1 lives

I don't really think there is an error with the timesteps. I printed the whole training process seems good to me.

I'm going to close this, reopen if still an issue


[BUG] Error when creating plot

Describe the bug
When creating plot number 2 the log reports a repeating error, and never finishes:
Only read 0 of 4194304 bytes at offset<PHONE_NUMBER>2 from "E:\temp\plot-k32-2021-02-22-17-54-5cb69645d57826c3c49e7a6461025002ec617399e5dd234286096b0ffd588d38.plot.table7.tmp"with length<PHONE_NUMBER>2. Error 1. Retrying in five minutes._
this keeps repeating
Steps to reproduce the behavior:
Create a new plot.
Wait for phase 2/4 to begin
Expected behavior
The plot should finish, it doesnt.
Desktop (please complete the following information):

OS: Win10 pro
OS Version/Flavor: 10.0.19041 build 19041
CPU: i5-9400F
Chia version: 0.1.27

An Error 1 is 80% of the time caused by a problem with your filesystem and 20% a problem with your RAM.


Icon is oddly pixelated on Linux

Here's a screenshot:

Fixed in https://github.com/zulip/zulip-electron/commit/8410769fdd107946e6b351b589ab828a40ac38b9


Certificate Revocation Validation with CRL and OCSP in X509 Authenticator

@IndunilRathnayake Is this same as #3178 ?

Closing since this issue is duplicated in #3178


edit search.md: in Note that describes LINQ Contains and SQLite case (in)sensitivity, include links in the Note instead of outside

Before this change:

^ Links are outside purple box.
After this change (what I intend):

^ Links are inside purple box.
Because the links are part of the note's message, I claim the links should be inside the note's box (if possible).

Internal previews

📄 File
🔗 Preview link

aspnetcore/tutorials/razor-pages/search.md
Part 6, add search to ASP.NET Core Razor Pages

Neat! Thank you for telling me about the improved text for screen readers and the link text format for GH issues. :)


DBZ-2693 using db2 image with ocp entry-point

Complementary PR to https://github.com/debezium/debezium-examples/pull/135

@jpechane  can the 1.2 image be updated or should I switch to some other tag? e.g. 1.2-testing to sth.

@jcechace The 1.2 image can be updated


[RFC] Deprecating ntree_limit in favor of best_iteration.

The use of ntree_limit is error prone in the context of multi-class and random forest.  The calculation is as followed:
ntree_limit = num_class * num_parallel_tree * iteration

This looks simple, but due to some historical reasons, the num_class is handled internally while num_parallel_tree is handled in language bindings.  The behavior generates unnecessary confusion.  Right now there are 2 alternatives in xgboost, first one is the inplace_predict function, which accepts iteration number directly, second one is model slicing, which allows user to extract a part of model using iteration number.
The deprecation might be a huge breaking change, so I would like to hear some suggestions on how we move forward.  Many thanks!

cc @hcho3 @RAMitchell

cc @hcho3 @RAMitchell

Closing in favor of https://github.com/dmlc/xgboost/issues/6632 .


Fix LFS Locks over SSH

Whilst looking at adding more tests for #6993 I've noticed that LFS locking was not-preauthenticating  correctly over SSH.
This PR fixes this.
Fixes #3084

It seems there is an issue reported that.

@lunny do you know which issue it is because I can't seem to easily find it.

Maybe it is https://github.com/go-gitea/gitea/issues/5478

It's possible that this is part of the fix for that, but I think #6993 is probably more likely to fix that.
I'm going to incorporate this in #6993 in any case.

Codecov Report

Merging #6999 into master will decrease coverage by 0.01%.
The diff coverage is 54.32%.

@@            Coverage Diff             @@
##           master    #6999      +/-   ##
==========================================
- Coverage   41.44%   41.42%   -0.02%
==========================================
  Files         441      441
  Lines       59696    59746      +50
==========================================
+ Hits        24739    24750      +11
- Misses      31732    31766      +34
- Partials     3225     3230       +5

Impacted Files
Coverage Δ

routers/routes/routes.go
82.68% <100%> (ø)
:arrow_up:

modules/lfs/locks.go
48.59% <53.75%> (+1.35%)
:arrow_up:

models/lfs_lock.go
56.81% <0%> (-6.82%)
:arrow_down:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

models/error.go
31.86% <0%> (-1.54%)
:arrow_down:

routers/repo/view.go
42.02% <0%> (-1.02%)
:arrow_down:

modules/lfs/server.go
45.16% <0%> (+0.74%)
: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 8de76b6...8b320fa. Read the comment docs.

@sapk have you been able to test this yet? git_test.go does test the locking now btw.

@zeripath not much time to do so lately. So if someone other validate it should be good as my requested changes are done.

@zeripath  I think this should resolve #7219, could you send back port to v1.8 .


facebook: handle responses on shares of your posts

i recently switched FB API endpoints from /posts to /feed because FB says it returns a superset of the data.. i thought in practice it was the same, but one interesting difference is that we now see (and try to backfeed) other people's shares of your posts.
it almost works, but OPD isn't finding the original post links. not surprising. example mf2; example log.

here's the example share: https://www.facebook.com/10101031520017413/posts/10101612230134683
i can't fetch this from the API with /10101612230134683, /10101031520017413_10101612230134683, or (of course) /212038_10101612230134683. soooo...not sure what to do. maybe shares just aren't doable and i should filter them out?

one possible approach is to fetch the Response object from the datastore, since we can construct its key from the (underscoreless) post id and use the urls in its sent + unset + error + failed + skipped fields. i'm a little reluctant, but it would probably work.
i'm going to deprioritize this for now.

Facebook support is dead (#817, #826), so I'm closing this as obsolete. feel free to reopen if you have any concerns!


Licensing

Hi @kayeeyeung @evan3334 and @rgschmitz1 ,
that's a super amazing project you're working on here! I'm glad to see that clij is useful.
May I make a small suggestion. If people wanted to use your project for their science, they need the one way or another make a copy of it. Therefore, a copyright statement (license file) would be good in your project. Otherwise it's kind-of illegal to copy your work. And with a license file you can also make sure you're not liable if others do wrong things with your tools.
https://choosealicense.com
Thanks!
Best,
Robert

Also another minor issue: Some files in this folder may originate from this repository which is BSD licensed. This means, you're ok to copy it, but you should also copy the license file from the source to make sure the original copyright holders are not forgotten. Read more in our license:
https://github.com/clij/clij-docs/blob/master/license.txt

Hi @haesleinhuepf
Thank you for your feedback. We will add licensing information to our repo.
We are excited that you find our project interesting. Can I invite you to test drive our tools and provide feedback in our usability study?
This should take ~30 minutes and would let you try our tools on AWS.
We are expanding the scope of our work from genomics to bioimage data, and would greatly appreciate input from the bioimage community to guide our future work.
Thanks
Ka Yee

Hi @kayeeyeung @evan3334 and @rgschmitz1 ,
that's a super amazing project you're working on here! I'm glad to see that clij is useful.
May I make a small suggestion. If people wanted to use your project for their science, they need the one way or another make a copy of it. Therefore, a copyright statement (license file) would be good in your project. Otherwise it's kind-of illegal to copy your work. And with a license file you can also make sure you're not liable if others do wrong things with your tools.
https://choosealicense.com
Thanks!
Best, Robert


[RO] Added context and response testing

Added context for sentences (e.g. "turn on the light" said in the kitchen).
Added response tests
This PR depends on https://github.com/home-assistant/intents/pull/756 as it fails tests without it
Known issues to be solved even after #756:

context information is not treated as slots and populated into responses
state context is unavailable for climate_HassClimateGetTemperature - see https://discord.com/channels/330944238910963714/1054846248374505572/1067557531590856734

God, I hate merges


Secure Configuration for Podman

We have done basics such as Securing the Operating System per Openscap, Central Logging, Auditing, SELinux Enforcing, FIPS mode.  We also configured rootless containers per RedHat Documentation [1].  We have setup signed images as a requirement to pull containers although I understand this does not block manual local container file import. [2]  I've had trouble finding security recommendations for podman itself (not the creation of secure containers).
How can podman's configuration be secured?
We are running RedHat RHEL7 Latest packaged version of Podman:
podman.x86_64                         1.4.4-4.el7            @rhel-7-server-extras-rpms
What podman configuration files should we setup auditing against?
-w /etc/containers/policy.json -p wa -k podman-config-monitor
-w /usr/lib/tmpfiles.d/podman.conf -p wa -k podman-config-monitor
-w /usr/share/containers/libpod.conf -p wa -k podman-config-monitor
Is it helpful to audit all root-less user's podman config files and if yes how could this be done?
podman command auditing
-a always,exit -F arch=b64 podman -k podman-command
[1] https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux_atomic_host/7/html-single/managing_containers/index#set_up_for_rootless_containers
[2] https://access.redhat.com/articles/3116561 Verifying image signing for Red Hat Container Registry
Cliff notes:
atomic trust add
--pubkeys /etc/pki/rpm-gpg/RPM-GPG-KEY-redhat-release
--sigstore https://access.redhat.com/webassets/docker/content/sigstore
registry.access.redhat.com
Adds a section to the /etc/containers/policy.json file.
...
Adds signature server files in directory /etc/containers/registries.d/.
    docker:
      registry.access.redhat.com:
        sigstore: https://access.redhat.com/webassets/docker/content/sigstore

And the registry.redhat.io.yaml file:
    docker:
      registry.redhat.io:
        sigstore: https://access.redhat.com/webassets/docker/content/sigstore

@mtrmac @vrothberg @mheon PTAL
I think you should watch /etc/containers/registries.conf
Perhaps storage.conf

Thank you, good idea I added those files and the entire directory with better labels.

-w /etc/containers/registries.json -p wa -k apodman-registries
-w /etc/containers/storage.json -p wa -k apodman-storage
-w /etc/containers/policy.json -p wa -k apodman-policy
-w /etc/containers/ -p wa -k apodman-containers
-w /usr/lib/tmpfiles.d/podman.conf -p wa -k apodman-config
-w /usr/share/containers/libpod.conf -p wa -k apodman-config
-w /usr/bin/podman -p xar -k apodman-command

I kind of want to push against the whole idea of “how can we make it secure”, that should be the default expectation; non-default hardening should only be necessary for users with unusual requirements. If we are not living up to that, that’s, by default, a bug (or, maybe, rarely, a known painful compromise).
WRT auditing global configuration files, search for RootForImplicitAbsolutePaths in vendor/containers/image; the list of such files can expand over time.  Also consider the case of configuration (and credentials) that are expected to exist in per-user files (in home directories and /run).

@Michael-Angel-Sec, do you have further questions?

Thank you, I appreciate your secure by default position.   I appreciate these areas to focus on.


Trilha dev

Хорошо

Исправим

package 2.json

My asset somehow after I bridged from base to ETHERIUM through MetaMask is now in its previous contract address and if only I could access code to reverse it to my address the miner sent to originally I can recover my maga trump 112 shares of it bought early End of March signed a message on 11th of April which I don’t understand where it went I tried too many maneuvering to get it back and need real help.


add --data-values-file-yaml to load data values from yaml files that do not have @data/values annotation

currently following works
$ ytt --data-values-inspect -f values.yml=<(echo -e "#@data/values\n---\nstores: [1, 2, 3]")
stores:
- 1
- 2
- 3

however, it would be nice to have following work as well:
$ ytt --data-values-inspect --data-values-file-yaml <(echo "stores: [1, 2, 3]")
stores:
- 1
- 2
- 3

Related Conversations:

https://kubernetes.slack.com/archives/CH8KCCKA5/p1567035583060800
https://kubernetes.slack.com/archives/CH8KCCKA5/p1605718554315200

note to self: downside of this feature is that it becomes less friendly to library module inclusion since files wont be annotated as data/values.

What if the file extension .data implied the #@data/values annotation?
ytt --data-values-inspect -f values.data.yml=<(echo -e "stores: [1, 2, 3]")

I think it also patches up the problem with libraries.

This was implemented as the --data-values-file and library.with_data_values(plain=True) features.

Hi John, is there an example for this feature?


Fixing 415 - Converting the typing function to async

Summary
When using RTMClient#typing(), the typing indicator was not being sent immediately because the function was not asynchronous. This caused the erroneous behavior described in #415. In order to immediately send any data over the websocket with aiohttp we must use an async function.
This PR changes both #typing() and #ping() functions on the RTMClient class into async functions.
Here's an example of what this would look like implemented:
@slack.RTMClient.run_on(event="open")
async def typing_message(**payload):
    rtm_client = payload["rtm_client"]
    await rtm_client.typing(channel="C01234567")

It's a minor but still backwards-incompatible change.
Requirements

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

This looks good to me. Its definitely unfortunate this was missed the first time around but it looks good


Id provider

Generate a new provider that gives APIs to access Id values for layout items.  Also generate mocking utility for Ids and add a sample.

 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<EMAIL_ADDRESS>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.

This PR addresses https://github.com/Comcast/resourceprovider/issues/20


pull request for e2e test at 01/20/2017 15:45:09

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add Windows service support

Do not block on HTTP server
Add support for installing and uninstalling service

Closes #83

@patriksvensson ah, see that you already merged this :-)  Nothing major in the review.

@gep13 Not to late to change things. What was your suggestion?

@gep13 Didn’t see your comment for some reason. Will change! Thanks a lot! 😊


insomnia: 5.15.0 -> 5.16.0

Motivation for this change
Update insomnia.  Fixes #39542, which does not add the required ssl library and would lead to a broken package.
Unfortunately the binary does not respond to any commandline flags, so I found no good way to quickly check for this regression.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@grahamcofborg build insomnia

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: insomnia
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: insomnia
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/v90v2kmq96nxyn1740j2gqvw1nia6hz4-insomnia-5.16.0
shrinking /nix/store/v90v2kmq96nxyn1740j2gqvw1nia6hz4-insomnia-5.16.0/share/insomnia/insomnia
shrinking /nix/store/v90v2kmq96nxyn1740j2gqvw1nia6hz4-insomnia-5.16.0/lib/libnode.so
shrinking /nix/store/v90v2kmq96nxyn1740j2gqvw1nia6hz4-insomnia-5.16.0/lib/libffmpeg.so
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/v90v2kmq96nxyn1740j2gqvw1nia6hz4-insomnia-5.16.0/lib  /nix/store/v90v2kmq96nxyn1740j2gqvw1nia6hz4-insomnia-5.16.0/bin
patching script interpreter paths in /nix/store/v90v2kmq96nxyn1740j2gqvw1nia6hz4-insomnia-5.16.0
head: error writing 'standard output': Broken pipe
checking for references to /build in /nix/store/v90v2kmq96nxyn1740j2gqvw1nia6hz4-insomnia-5.16.0...
/nix/store/v90v2kmq96nxyn1740j2gqvw1nia6hz4-insomnia-5.16.0


AWS Kinesis Delivery Stream ingest issues

AWS has introduced a new tool connected to the Kinesis Product,  the "Kinesis Firehose Delivery Stream".     This tool is designed to allow AWS to Deliver Data from a source,  such as the Kinesis Data Stream or a Direct PUT into the Delivery pathway,  to a Destination such as AWS Open Search, S3 Bucket, or Redshift.  It also supports 3rd party Delivery to locations such as Datadog, Splunk, Dynatrace, LogicMonitor, MongoDB Cloud, NewRelic, Sumo Logic,  Or a custom HTTP Endpoint.
Our Existing Kinesis Ingester is designed to work with the Kinesis Data Streams,    however for existing workflows using a Delivery Stream or some AWS products which use a Direct PUT into the Delivery stream to export data,  This ingester will not work.
Unfortunately our HTTP Ingester also does not work due to some custom request/response requirements that AWS has.
If we attempt to use AWS' Splunk ingester with the HEC Listener,  it fails due to AWS Requiring Splunk's HEC Ack functionality being enabled which we do not support within the ingester currently.     This functionality may actually be a preferred method to implement due to wider potential support/desire for a direct HEC plugin replacement.

Dump of Demo stock ticker type data attempted to import into Gravwell via the http ingester with authentication disabled:
5976165362.zip
AWS Reported the delivery as a failure:
The response received from the endpoint is invalid. See Troubleshooting HTTP Endpoints in the Firehose documentation for more information. Reason:. Response for request ccbeeb8e-b473-4064-86d5-b50fac45b29d must contain a 'content-type: application/json' header. Raw response received: 200:
Firehose Request/Response Specs:   https://docs.aws.amazon.com/firehose/latest/dev/httpdeliveryrequestresponse.html
HTTP Ingester Verbose output:
HEADERS map[Content-Length:[314] Content-Type:[application/json] User-Agent:[Amazon Kinesis Data Firehose Agent/1.0] X-Amz-Firehose-Access-Key:[Gravwell on6O6ekLYixvlgDXdV7dprvapHIzxcQK] X-Amz-Firehose-Protocol-Version:[1.0] X-Amz-Firehose-Request-Id:[5d342fea-527d-444a-8e1b-0ac15efdc14f] X-Amz-Firehose-Source-Arn:[arn:aws:firehose:us-west-2:400526119735:deliverystream/Gravwell_http_Ingest]]

Splunk Documentation on the HEC Acknowledgement functionality:
https://docs.splunk.com/Documentation/Splunk/8.2.4/Data/AboutHECIDXAck

tag=Test1 json -x records requestId timestamp records | json -e records data | base64 -t decoded -d data | table requestId timestamp decoded cracks the data cleanly and the responses expected are pretty straight forward.
We can fix this


Gem version doesn't match Ceedling version

When running gem install ceedling, the 0.31.0 gem is properly installed, but ceedling version reports that the ceedling version is still 0.28.3. This is also true when installing the 0.30.0 gem (i.e. gem install ceedling -v "<0.31.0"). Both gems seem to contain the 0.28.3 version of ceeedling.
This has been reproduced both in windows and linux. Here's some outputs from the install testing:
$ sudo gem install ceedling
Fetching deep_merge-1.2.1.gem
Fetching constructor-2.0.0.gem
Fetching ceedling-0.31.0.gem
Fetching rake-12.3.3.gem
Fetching thor-0.20.3.gem
Successfully installed thor-0.20.3
Successfully installed rake-12.3.3
Successfully installed deep_merge-1.2.1
Successfully installed constructor-2.0.0
Successfully installed ceedling-0.31.0
Parsing documentation for thor-0.20.3
Installing ri documentation for thor-0.20.3
Parsing documentation for rake-12.3.3
Installing ri documentation for rake-12.3.3
Parsing documentation for deep_merge-1.2.1
Installing ri documentation for deep_merge-1.2.1
Parsing documentation for constructor-2.0.0
Installing ri documentation for constructor-2.0.0
Parsing documentation for ceedling-0.31.0
Installing ri documentation for ceedling-0.31.0
Done installing documentation for thor, rake, deep_merge, constructor, ceedling after 1 seconds
5 gems installed

$ ceedling version
  Ceedling:: 0.28.3
CException:: <IP_ADDRESS>
     CMock:: <IP_ADDRESS>
     Unity:: <IP_ADDRESS>

This has been quite frustrating, as version 0.28.3 has very inferior reporting capabilities for gcov.

Chris -- my best guess is that if you look at your project file, you'll find that you're using a vendor-installed ceedling instead of the latest gem. If you'd like to switch to the new Ceedling, the best way (at the moment... this is getting improved is to do the following):
Update your project.yml file:
:project:
  :which_ceedling: gem

If you have :unity and/or :cmock sections and they have :vendor_path: lines, remove them.
If you have a rakefile, this is no longer necessary and it can also be removed.
In the :plugins: and :load_paths: section, replace vendor/ceedling/plugins with "#{Ceedling.load_path}"
You should now be using the gem directly... you can remove the folder vendor/ceedling from your project once you're sure it's working again.

Ah-ha! That did it! Perhaps a message could be added to ceedling version to list the path of the ceedling version it is reporting?
In any case, my coverage report is now working!! Awesome. Now my only challenge is that I'm seeing a blast of "Could not find coverage results for..." messages. I'll open a separate issue for that.


Fixed incorrect url containing a stray ) character

As per title, just removed the ).

Thank you for your contribution! I'm merging this in, and I'll roll out a version to NuGet in just a moment. I appreciate it!


[BUG] ListResourceUsage throws an exception on CdnProfiles

Describe the bug
I am trying using dotnet sdk manage my CDN profiles and endpoints, I know that a subscription can have no more than 25 CDN profiles and I want to get this number with sdk api, instead of hard coding 25 in my program.
My code looks like this:
 var creds = new AzureCredentialsFactory().FromServicePrincipal(client, secret, tenantId, AzureEnvironment.AzureGlobalCloud);
 var azure = Microsoft.Azure.Management.Fluent.Azure.Authenticate(creds)
                .WithDefaultSubscription();
 var usage = azure.CdnProfiles.ListResourceUsage();

When I call ListResourceUsage(), it throws an ErrorResponseException with message Operation returned an invalid status code 'NotFound'. I can call other method on CdnProfiles without any issue. Besides, I notice the ListResourceUsage() doesn't work on CdnProfile neither.
Expected behavior
I expect ListResourceUsage() returns a IEnumerable<ResourceUsage> object as the document says so.
Actual behavior (include Exception or Stack Trace)
The trace look like this:
Microsoft.Azure.Management.Cdn.Fluent.Models.ErrorResponseException: Operation returned an invalid status code 'NotFound'
   at Microsoft.Azure.Management.Cdn.Fluent.ResourceUsageInnerOperations.ListWithHttpMessagesAsync(Dictionary`2 customHeaders, CancellationToken cancellationToken)
   at Microsoft.Azure.Management.Cdn.Fluent.ResourceUsageInnerOperationsExtensions.ListAsync(IResourceUsageInnerOperations operations, CancellationToken cancellationToken)
   at Microsoft.Azure.Management.ResourceManager.Fluent.Core.Extensions.Synchronize[TResult](Func`1 function)
   at Microsoft.Azure.Management.Cdn.Fluent.CdnProfilesImpl.ListResourceUsage()
   at Microsoft.Azure.Management.Cdn.Fluent.CdnProfilesImpl.Microsoft.Azure.Management.Cdn.Fluent.ICdnProfiles.ListResourceUsage()

To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)
Run the code above.
Environment:

Name and version of the Library package used: [Microsoft.Azure.Management.Cdn.Fluent 1.34.0] and [Microsoft.Azure.Management.Fluent 1.34.0]
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): [Windows 10 ASP.NET Core 3.1.401]
IDE and version : [Visual Studio 16.7.1]

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

add @xccc-msft to help

add @xccc-msft to help

@hund030 I'm unable to reproduce this. For me, the listResourceUsage works. Are you still facing the same issue? If so, can you please try to capture the request/response by tool like Fiddler.

@hund030 I'm unable to reproduce this. For me, the listResourceUsage works. Are you still facing the same issue? If so, can you please try to capture the request/response by tool like Fiddler.

@xccc-msft I update my Microsoft.Azure.Management.Fluent package from v1.34.0 to latest v1.36.1, then it works.
Thanks for updates!

@xccc-msft I update my Microsoft.Azure.Management.Fluent package from v1.34.0 to latest v1.36.1, then it works.
Thanks for updates!


UserRegistration Routes

As we are refactoring a lot of the server side to use new models and such, we will look to split the User model into two models:

User: The User model will focus purely on authentication and user account details
UserRegistration: This model will focus on things relevant to a student and his/her semester at RCOS.

For reference, this is the preliminary model in mind:
  - user_id (ref)
  - project_id (ref)
  - semester_id (ref)
  - rin (string)
  - credits (string)
  - status (string - student, external mentor, alumnus, RPI faculty, etc.)
  - schedule_conflict (boolean)
  - has_completed_urp_form (boolean)
  - requires_wheelchair_accessible_room (boolean)
  - mentor_preferences ([ref])

For this issue, please make the rudimentary UserRegistration routes. This will entail making the index.js file like before. The following routes should be implemented

GET /api/user_registration
POST /api/user_registration
GET /api/user_registration/:userid
PUT /api/user_registration/:id
DELETE /api/user_registration/:userid

We should probably find UserRegistrations not by the ID of the registration, but by the user since it is heavily tied to one particular user (hence the routes above).

Resolved


Fix known failure suffixes

This is similar to my fix in https://github.com/facebookgo/httpcontrol/pull/2 for no such host, but I since realized that the issue with the trailing periods affects other errors from being retried as well. This does the following:

Removes trailing periods from other errors. As far as I can tell, none of these will have trailing periods.
Replaces strings with constants from where the errors originate where possible. The others are private fields or generated within private functions in other packages.
Adds a retry for io.EOF with a test show how this can happen. (This has happened to me a few times in production so far.)

I've seen an error that might be retryable as well net/http: request canceled while waiting for connection – would you accept a PR to include that in the knownFailureSuffixes?

@dvdplm should probably open a new issue for it, but it looks like that is returned when https://golang.org/pkg/net/http/#Transport.CancelRequest is used. Either your code is calling that directly or is being triggered on account of a timeout -- either way I'm not sure if retrying is necessary in that scenario.

@daaku yes, that seems to be the scenario. The request cancellation is probably triggered by the timeout setup on the httpcontrol.Transport I'm using (I'm not explicitly cancelling the req). Re-trying it works and solves the issue I'm having, but I'm not sure if it's ok to add here: if I were calling CancelRequest explicitly I would not want it retried I guess.
Is there a way to check if the cancellation was caused by the timeout set on the transport or by something else (and only retry in the former case)?

Is there a way to check if the cancellation was caused by the timeout set on the transport or by something else (and only retry in the former case)?

Not sure - but a quick check seems to indicate no.

Bummer. I'll dig around and see what I can come up with. Thanks.
On Tue, Mar 24, 2015 at 7:37 PM, Naitik Shah<EMAIL_ADDRESS>wrote:

Is there a way to check if the cancellation was caused by the timeout set
on the transport or by something else (and only retry in the former case)?
Not sure - but a quick check seems to indicate no.
—
Reply to this email directly or view it on GitHub
https://github.com/facebookgo/httpcontrol/pull/3#issuecomment-85636075.


[dspace-7_x] Fix Collection Page e2e test. Wait on Recent Submissions

Description
Frequently, the collection-page.cy.ts is failing because the e2e test times out before the Collection Page fully loads.  This simply updates the test to wait on the list of Recent Submissions before performing any other tests.
Instructions for Reviewers

If e2e tests succeed, I'll merge this immediately.  This failure is occurring frequently on the 7.x branch (only)

Success! This seems to fix the annoying issues we are having on dspace-7_x with collection-page.cy.ts failing frequently


"exit" terminal doesn't respond for a while

Environment
Microsoft Windows [Version 10.0.18362.207]
Steps to reproduce
type "exit" for close terminal (with one tab opened)
Expected behavior
Close the terminal quickly (like when you're closing the tab)
Actual behavior
Terminal doesn't respond for a while (second or two) with just window borders displayed and then exits.

To me thats normal, but I'm sure it will be improved in the future.

#1380


Use latest hexkit

Update dependencies, particularly hexkit.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
629

Relevant Lines:
720

💛  - Coveralls


Refactor 'aeon' modules

In particular, './aeon/analysis/' should be removed, and modules within should be moved into './aeon/processing/' and the 'aeon_analysis' repo as appropriate.
e.g. 'aeon/processing/init.py' :
# This is where functions for common preprocessing done on raw data files will live
# (e.g. 'calc_forage_dist', 'calc_forage_bout', 'collate_movie', 'calc_tube_test', etc.)

Related to https://github.com/SainsburyWellcomeCentre/aeon_analysis/discussions/8

Actually, 'processing' could mirror 'schema', where we have a 'core' module, and then modules for different types of experiments (e.g. 'foraging', 'social', etc.)

We could even do the same with 'qc' - 'qc' functions dedicated to streams used in specific experiments.
Somehow I like the idea of this mirroring between 'schema', 'processing', and 'qc'

@glopesdev related to module refactoring - what is the differentiation between functions currently in 'aeon.io.video' and those in 'aeon.analysis.movies' ? Is there a reason you wanted to keep them separate rather than combining them into one module?

Just brainstorming here, but now that we've agreed with steering committee that 'analysis' package can live in aeon_mecha, the repo structure could like something like this (note the core structure stays the same as it is currently):
| aeon/
|-- analysis/
|---- ...
|-- dj_pipelines/
|---- ...
|-- io/
|---- api.py
|---- device.py
|---- reader.py    # abstract classes only
|-- schema/    # modules here correspond to general exp types
|------ core.py
|------ foraging.py
|------ social.py
|------ ...
|-- processing/
|---- core.py
|---- foraging.py
|---- social.py
|---- ...
|-- qc/
|---- core.py
|---- foraging.py
|---- social.py
|---- ...

Some notes:

In aeon.schema, each module can contain Readers, binder functions, and helper functions, all with the goal of creating versioned schemas for the given experiment name. The __main__  in these modules can return different versions of a schema, depending on input args.

aeon.io.schema.core can contain things that will get reused by all other schemas, including a potential raw Video reader.

I would move the Pose reader and associated code from aeon.schema.social to aeon.schema.core


Docs for mocks

Codecov Report

Merging #21 (cbf7840) into gomock-integration (92390fc) will not change coverage.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##           gomock-integration      #21   +/-   ##
===================================================
  Coverage               98.93%   98.93%
===================================================
  Files                       5        5
  Lines                     283      283
===================================================
  Hits                      280      280
  Misses                      2        2
  Partials                    1        1

Impacted Files
Coverage Δ

ensurepkg/run_table.go
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #21 (cbf7840) into gomock-integration (92390fc) will not change coverage.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##           gomock-integration      #21   +/-   ##
===================================================
  Coverage               98.93%   98.93%
===================================================
  Files                       5        5
  Lines                     283      283
===================================================
  Hits                      280      280
  Misses                      2        2
  Partials                    1        1

Impacted Files
Coverage Δ

ensurepkg/run_table.go
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Fix for navbar and logo position

Closes #1046

The navbar has been positioned to the right
The position of logo has been centered in navbar

Sure @wesleytodd
Current appearance of site:

Appearance of site after changes:

Also, I have checked that the proposed changes does not break the responsiveness of the the website.

It does feel a bit harder to use, at least for me:
Before:

After:

The navigation now ends up way on the far right, but the content of the site does not move to be in the middle or anything.

@dougwilson This does seem to be an UX issue for screens with width higher than 2080px (2K and 4K displays). The website seems to have been made initially for screen width 1080px as after 1080px the white space on website increases drastically and the content does not fill it.
I can suggest two solutions:-

Let the changes be same. It might require extra effort for users with displays >= 2K, but the number of these users are still less and most of the users still use 1366p or 1080p displays (source). But it will enable greater feasibility and UI advantage for users with less than 2K displays. The website behaviour will be similar to that of Gmail in 4K displays with logout button in far top right

We can center the whole content of the website when the screen dimension is greater than 1440px.
Example:

This is the approach followed by most of the websites but would require major changes to the stylesheet.

@dougwilson I will work on overhauling the design to center the content after 1024px.


Jupyter Notebook files not identified

A rather simple issue: .ipynb files are not registered by the extension. Whenever the active file is a Jupyter Notebook, the extension indicates the idle state.

Duplicate of #120


Auto corrected by following Lint Ruby Style/CaseLikeIf

Auto corrected by following Lint Ruby Style/CaseLikeIf
Click here to configure it on awesomecode.io

Coverage remained the same at 97.025% when pulling 4669ea0982378ba060410bb961929b5c5572faf9 on awesomecode-lint-ruby-style-caselikeif-47828 into cd32a52b407596d75541c69b729af6b1d4bd7e99 on master.

Coverage remained the same at 97.025% when pulling 4669ea0982378ba060410bb961929b5c5572faf9 on awesomecode-lint-ruby-style-caselikeif-47828 into cd32a52b407596d75541c69b729af6b1d4bd7e99 on master.


Index updates to add additional links to concept topics and Oqtane Blog

Updates 2 index.md files in the guides section to add additional links to Oqtane Concept topics and Oqtane Blog.

uid: Guides.Themes.Index
Is this something we are going to include inside index.md pages going forward?

---
uid: Guides.Themes.Index
---

guides/themes/index.md
Is this something we are going to include inside index.md pages going forward?

The UID is like an internal ID for linking to the page.
It doesn't show in the final output - You are probably looking at some kind of preview in your screenshot
With UIDs we can create links like [some title](xref:Guides.Themes.Index) which is more convenient for pages which will get a lot of links, but overkill for sub-pages which are linked just once or twice.


Migrate to Payload 2.x

Should use mongodb, lexical, and webpack adapters.

Planning on it 👍

Id love to see a port


Update v4 doc links

Please update the links on the front page:

APIs: https://github.com/ReactTraining/react-router/tree/v4/packages/react-router-website/modules/api
Examples: https://github.com/ReactTraining/react-router/tree/v4/packages/react-router-website/modules/examples

Can you also update the doc with what npm package@version to install for v4?

react-router@next. We've been through this before with 2.0 and 3.0, so it'll get done soon enough :)

Sure. I just feel as a new user who wants to try v4 it's hard to figure out what version to install. Even the npm website doesn't show the v4 versions.

you can find info here https://reacttraining.com/react-router/


Fix lfs locks

I've noticed that TestAPILFSLocksLogged() is recently failing due to a small time difference in the expected values.
Some time ago (#3167) the allowed margin was extended from 1 to 3 seconds. With this PR I'm extending it to 10, as the servers seem to be somewhat slower lately.
EDIT: this is the error produced in the automated tests:

I re-started the build that failed on a unrelated test.

Codecov Report

Merging #8361 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8361      +/-   ##
==========================================
- Coverage   41.76%   41.75%   -0.01%
==========================================
  Files         496      496
  Lines       65585    65585
==========================================
- Hits        27391    27385       -6
- Misses      34681    34687       +6
  Partials     3513     3513

Impacted Files
Coverage Δ

modules/sync/unique_queue.go
78.57% <0%> (-10.72%)
:arrow_down:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

routers/repo/view.go
42.53% <0%> (-1.02%)
:arrow_down:

models/repo_indexer.go
68.21% <0%> (+1.93%)
: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 c9f819e...ea7dc3b. Read the comment docs.


Figure out how to make facets for fields that contain arrays

Modifications and treatments immediately come to mind...

Changed calculated properties so that this would work. As an example, check out these two biosample inserts: 231111bc-8535-4448-903e-854af460c13f and 231111bc-8535-4448-903e-854af460b255. Facet behavior works correctly, with no need to concatenate objects in arrays.


[C++] Use S3 conditional writes to avoid some explicit checks

Describe the enhancement requested
We should investigate whether S3 conditional writes can be used to reduce some of the checks we do when creating files or pseudo-directories.
We'll also have to be careful to remain compatible with non-AWS S3 implementations such as Minio, as well. This might complicate testing.
Component(s)
C++

This was initially suggested by @felipecrv in https://github.com/apache/arrow/pull/44225#discussion_r1783055536


More Obvious Look For Active Heading

Just adding bold to active heading (in auto expand mode) is not enough. Can you please add a option to making it more obvious like highlighting it or something.
Also the headings color in light mode shown white too. I usually use Dark mode but still a issue. Not sure it is theme related or note.
Thanks

For problem 1, you can change to desired appearance at location section in styles.css.

like background-color:yellow;

For problem 2 , same as #34

For problem 1, you can change to desired appearance at location section in styles.css.

like background-color:yellow;

For problem 2 , same as #34

Thanks, it is going to get deleted on a new update if I modify it? how i can keep a change if i make one?

I think it wiil. When updating plugin, old CSS file may be overrideed by new one.
You can create a custom CSS file in obsidian appearance setting and add above rules into it.
Custom css files have the highest priority and doesn't change after plugin updating.

I think it wiil. When updating plugin, old CSS file may be overrideed by new one.
You can create a custom CSS file in obsidian appearance setting and add above rules into it.
Custom css files have the highest priority and doesn't change after plugin updating.

Thank you, works like a charm


letting center_position be None

Let center_position be None, indicating that the MPS is not canonical

added more testing (mostly covering raise statements)


PHP 8.1 compatibility

Hello!
Please, resolve this:
https://github.com/protocolbuffers/protobuf/issues/9293

It is resolved in https://github.com/protocolbuffers/protobuf/pull/9370 .


Allow organism in search form to be deleted (set to null after the user had previously selected an organism)

Done


Extra values are not copied into clipboard

In version 2.2.1 not all extra values are copied to clipboard. For example, in http://paste.ubuntu.com/12070915/ extra 3 value contains URL to file, which is shown in Intent Intercept interface, but not in the clipboard.
EXTRA 3:
Class: android.net.Uri$HierarchicalUri
Key: android.intent.extra.STREAM

The intent is generated by "Graph Paper" Export PDF event.

Should be fixed in 2.2.2 just pushed to Google Play Store.

Works like a charm. =)


local install in level3 failed

Dupe of #3281
Although the engine is open source and free, the level contents have rights reserved (so CodeCombat can sell subscriptions, after all). Only a handful of levels are included in the public DB dump so you can work on the engine and test it.

thanks for your anwser! @UltCombo


Adding screen utili package #106 enhancement level3

ADD screen unit package in the project and change every widget and screen size using pkg

@krishitapandey good work


Support NGFF using n5-universe

Overview
This PR implements loaders for the NGFF formats using n5-universe.
Supported formats

Format
Metadata
Storage
Confirmed converter

HDF5
N/A
local only
Plugins › BigDataViewer › Export Current Image as XML/HDF5 in Fiji

OME-Zarr
OME-NGFF v0.4
local or remote
bioformats2raw

N5
COSEM or N5 Viewer
local or remote
File › Save As › HDF5/N5/Zarr/OME-NGFF ... in Fiji

Usage

Select new project from URL in Mastodon launcher.

Press the browse button to open N5 dialog.

Click the Browse button to load local dataset, or input a URL and and press the Detect datasets button.
Please use the URLs starting with http: or https: because the s3: protocol is not fully supported at the moment.

Select a (multiscale)  dataset and press the OK button.

Save a .xml file that contains the SpimData information.

Press the create button.

Visualize the data in the bdv window.

Work with authentication of the remote storage
If the remote storage requires authentication or fails to load data, a dialog will appear, allowing you to enter the username (or AWS_ACCESS_KEY_ID) and password (or AWS_SECRET_ACCESS_KEY).

Alternatively, the user can set the credentials permanently on the system using the AWS credentials provider chain.
https://docs.aws.amazon.com/AWSJavaSDK/latest/javadoc/com/amazonaws/auth/DefaultAWSCredentialsProviderChain.html

AWS credentials provider chain that looks for credentials in this order:

Environment Variables - AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY (RECOMMENDED since they are recognized by all the AWS SDKs and CLI except for .NET), or AWS_ACCESS_KEY and AWS_SECRET_KEY (only recognized by Java SDK)
Java System Properties - aws.accessKeyId and aws.secretKey
Web Identity Token credentials from the environment or container
Credential profiles file at the default location (~/.aws/credentials) shared by all AWS SDKs and the AWS CLI
Credentials delivered through the Amazon EC2 container service if AWS_CONTAINER_CREDENTIALS_RELATIVE_URI" environment variable is set and security manager has permission to access the variable,
Instance profile credentials delivered through the Amazon EC2 metadata service

@ksugar Awesome! I will make basic tests then merge. All in time!
Thank you so much Ko

Doc for the N5 dialog is here:
https://github.com/saalfeldlab/n5-ij?tab=readme-ov-file#open-n5


[SPARK-22664] The logs about "Connected to Zookeeper" in ReliableKafkaReceiver.scala are in wrong position

https://issues.apache.org/jira/browse/SPARK-22664
The logs about "Connecting to Zookeeper" and "Connected to Zookeeper" in ReliableKafkaReceiver.scala should be printed when new a zkClient() in line 122.

Can one of the admins verify this patch?

You shouldn't create a JIRA for items like this, where the description is the same as the PR. I wouldn't move the initial log, as creating the Consumer is part of connecting. It won't matter much. But in case of error you will have logged what you were connecting to first.

@srowen
Thanks for your reply.Could you help me review it?

Test build #4002 has started for PR 19856 at commit 652068b.

Test build #4002 has finished for PR 19856 at commit 652068b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

@srowen
Please help merge this PR as it has passed all tests.

I guess the original purpose of such log is to reflect the behavior of consumer connection. It is not super necessary to do such trivial change. Also ReliableKafkaReceiver is not recommended any more, let's keep as it is.

@jerryshao
I think the log can't reflect the  behavior of consumer connection,because consumer.create doesn't do any connect,it only construct a ZookeeperConsumerConnector instance,so the right position is where zkclient connect to zk. ReliableKafkaReceiver is not marked deprecated,so users will use it.To avoid misleading by this log when a problem of connect to zk occur ,I think we should fix it.

I think the log can't reflect the behavior of consumer connection,because consumer.create doesn't do any connect,it only construct a ZookeeperConsumerConnector instance

That's not true, ZookeeperConsumerConnector will also connect to ZK during initialization, please see here (https://github.com/apache/kafka/blob/c9f930e3fe25e5675e64550c8b396356c5349ca7/core/src/main/scala/kafka/consumer/ZookeeperConsumerConnector.scala#L126).

Actually there's no issue here, IMHO I think your understanding of this log is slightly different from the original purpose.

@jerryshao
Yes,you are right,but this log is not accurate,I think it should log like this "consumerconnector has been created",it is too ambiguity now.

This is trivial and not obviously a useful change, so it should be closed


Unknown websocket error

Hi,
I´m creating an app to communicate with my meteor app. But I  am not able to connect to my meteor app or demoparties.meteor.com.
I'm using Android Studio 0.8.6
Android-DDP-Client <IP_ADDRESS>
Java-DDP-client <IP_ADDRESS>
This is the error I recieve
I/DDPStateSingleton﹕ createDDPClient on server: demoparties.meteor.com  port: 80
I/DDPStateSingleton﹕ createDDPClient mDDP: com.keysolutions.ddpclient.DDPClient@426043f8
V/DDPStateSingleton﹕ isConnected - state:Disconnected
V/DDPStateSingleton﹕ broadcastDDPError - errorMsg = [Unknown websocket error (exception in callback?)]
V/DDPStateSingleton﹕ getState: Closed
D/Connection Issue﹕ Error connecting to server...please try again
I/DDPStateSingleton﹕ connectIfNeeded
I/DDPStateSingleton﹕ connectIfNeeded - currently closed connection > Re-connect
I/DDPStateSingleton﹕ createDDPClient on server: demoparties.meteor.com  port: 80
I/DDPStateSingleton﹕ createDDPClient mDDP: com.keysolutions.ddpclient.DDPClient@425f7fe0
D/Login Error﹕ Unknown websocket error (exception in callback?)
What can I do?
/Pelle

Hi @kenyee
I am getting the same error. Any idea about this?


Problem with UTF8 in CA certificate

Hello. I have installed openxpki on a clean Debian Jessie system by quickstart guide, generated CA, Intermediate CA, Vault and CSEP certificates. This certificates contains UTF8 characters in OU, L, N, S fields.
OpenXPKI status shows "System status is good"  but when I open Information - CA Certificates I got this error:

2017/09/05 13:28:59 I18N_OPENXPKI_CLIENT_SEND_RECEIVE_SERVICE_MSG_ERROR_DURING_COLLECT; __EVAL_ERROR__ => I18N_OPENXPKI_TRANSPORT_SIMPLE_CLIENT_READ_CLOSED_CONNECTION at /usr/lib/x86_64-linux-gnu/perl5/5.20/OpenXPKI/Client.pm line 148.
; __STATE__ => can_send

How can I fix this problem? I've tried this patch https://github.com/openxpki/openxpki/pull/224 but got the same error.
My locale: ru_RU.utf8. en_US.utf8 is installed. Output of locale -a:

C
C.UTF-8
en_US.utf8
POSIX
ru_RU.utf8

Default language in server.yaml: C
Database created with charset utf8:

show table status where name ='aliases';
+---------+--------+---------+------------+------+----------------+-------------+-----------------+--------------+-----------+----------------+---------------------+-------------+------------+-----------------+----------+----------------+---------+
| Name    | Engine | Version | Row_format | Rows | Avg_row_length | Data_length | Max_data_length | Index_length | Data_free | Auto_increment | Create_time         | Update_time | Check_time | Collation       | Checksum | Create_options | Comment |
+---------+--------+---------+------------+------+----------------+-------------+-----------------+--------------+-----------+----------------+---------------------+-------------+------------+-----------------+----------+----------------+---------+
| aliases | InnoDB |      10 | Compact    |    4 |           4096 |       16384 |               0 |            0 |   5242880 |           NULL | 2017-09-04 14:21:50 | NULL        | NULL       | utf8_general_ci |     NULL |                |         |
+---------+--------+---------+------------+------+----------------+-------------+-----------------+--------------+-----------+----------------+---------------------+-------------+------------+-----------------+----------+----------------+---------+

openxpkiadm version 1.18.1
perl version 5.20.2

Please use the Mailinglists for support. You might try the current development head which has the first draft of the new transport layer that hopefully solves that.


determine why lbry://2048 game fails / doesn't play

The Issue
lbry://2048#44064cadfc22dc13419843999ec80e30cf13262b
get this first, which crashes the app.

reopening it leads to:

System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Might be related to Node v8 not supporting { recursive: true } in fs.mkdirSync(), see:
https://nodejs.org/docs/latest-v8.x/api/fs.html#fs_fs_mkdirsync_path_mode
I'll make a patch to support earlier versions of Node.


Example of blowfish theme project

What I want to ask is, after downloading the theme, the theme contains many document tutorials, and I want to delete them all, that is, the articles of docs and samples. After all, building a real website means that these document tutorials are useless. However, I found that deleting the articles of docs and samples would cause errors. So, what I want to ask is, can't the document tutorials in this be deleted? If it can be deleted, which document articles should be kept？

Hello @jawaylog, let me see if I can help. You are 100% right, everything in the exampleSite folder can be deleted.
Can you clarify how did you install the theme so that I can try to reproduce the error?

Hello @nunocoracao , the installation method I use is git, and then I copy and replace the files in the exampleSite directory in the theme to hugo root directory, so that the website can run. But in hugo's root directory, delete some articles of docs and samples in the content or delete the docs and samples folder. The mistake I mentioned above will happen. In the end, my approach is to delete the Tags folder and users.md, and the website can run.

ah got it, you are using the configs from exampleSite as your starting point and cleaning everything, is that it? Check the docs for the menu configuration and taxonomies for what needs to be changed. The exampleSite uses almost all feaures available in Blowfish so it's hard to clean up. Will close this issue now.


Could support to run in browerser enviroment?

Could support to run in browerser enviroment?

You can bundle by yourself.


Update Kronos.podspec version to specify Swift 5

Addresses issue #70

Thanks, but I ended up tagging 4.1.0 because of other changes. It's live on CocoaPods now

Great, thanks @keith !


Update README.md with Fortran-Lang Discourse

Fixes #485
Added GitHub Discussions & Discourse link
TODO:

[x] Add CSS in SVG for hover Not possible with GitHub Markdown

Best convert the text to a path, otherwise the svg will look strange if the correct fonts are not available.

Completely forgot about that (and that is why I had a pngs before), nice catch. This is basically done (after converting to paths). Do you think we need to add anything more?
PS I accidentally edited your comment apologies for that

I just realised that VSCE i.e. VS Code Marketplace, has banned SVGs since they can be a security vulnerability so I will have to convert all the SVGs to high quality PNGs anyway.


March 2022 Release Plan

Prerequisites

Python 3.7 and higher
run python3 -m pip install --user -r news/requirements.txt

Release candidate (Friday of VS Code endgame week, XXX XX)

[x] Review Component Governance (Click on "microsoft/vscode-jupyter" on that page) and resolve all High/Severe issues.

[x] Manually add any repository dependencies (if you can't add manually, refer here). Only add a cgmanifest.json if the components are not NPM or are not dev only.

[x] Create new release branch with format release/release-YYYY.MM.100

[x] Create a pull request against release/release-YYYY.MM for changes
[ ] Run npm install to verify package-lock.json did not get updated.
[ ] Update CHANGELOG.md

[ ] Run news (typically python news --final --update CHANGELOG.md | code-insiders -)
[ ] Copy over the "Thanks" section from the previous release into the "Thanks" section for the new release
[ ] Make sure the "Thanks" section is up-to-date (e.g. compare to versions in requirements.txt)
[ ] Touch up news entries (e.g. add missing periods)
[ ] Check the Markdown rendering to make sure everything looks good

[ ] Update ThirdPartyNotices-Repository.txt as appropriate. This file is manually edited so you can check with the teams if anything needs to be added here.
[ ] Merge pull request into release/release-YYYY.MM

[ ] Update the release branch

[ ] If there are release branches that are two versions old (e.g. release-2020.[current month - 2]) you can delete them at this time (you would have to un-protect the release branches temporarily to delete them).

[ ] Update main after creating the release branch. (Warning: this should happen right after creating the release branch. If this is deferred till later, the main and release branches can diverge significantly, which may cause merge conflicts.)

[ ] Merge the changes from release (Changelog, delete news, ThirdPartyNotices) into main branch
[ ] Bump the version number to the next monthly ("YYYY.M.100") release in the main branch

If latest is 2022.2.100, them bump it to 2022.3.100,
This ensures the new new pre-release targets the next version.
[ ] package.json
[ ] package-lock.json
[ ] Confirm the 3rd part of the version ends with 100.

[ ] Create a pull request against main
[ ] Merge pull request into main

[ ] GDPR bookkeeping (@greazer) (🤖; Notes in OneNote under Python VS Code -> Dev Process -> GDPR)

[ ] Open appropriate documentation issues

new features
settings changes
etc. (ask the team)

[ ] Release notes from VS code have something about jupyter

[ ] Schedule a sanity test. Aim for close after freeze so there is still time to fix release bugs before release. Ask teams before bash for specific areas that need testing.

[ ] Is the validation pipeline clear? If not, drive to make sure that it is clear for release. Sanity test can be used to help with this.

[ ] Ask CTI to test the release candidate

Testing (Monday of VS Code release week, XXX XX)

[ ] Obtain VS Code prebuild for sanity testing
[ ] Sanity test release candidate VSIX against VS Code prebuild
[ ] Candidate bug fixes found from sanity test should be checked into main and cherry-picked to release branch

Release (Tuesday or day before VS Code publishes, whichever is later)
Preparation

[ ] Make sure the appropriate pull requests for the release notes(https://github.com/microsoft/vscode-docs/blob/vnext/release-notes/v.md, make note of the branch and file name).

[ ] Include all new new features under a section named Jupyter with screenshots or animated gifs.
[ ] Fixes, code health should remain in the Jupyter change log.

[ ] Make sure component governance is happy
[ ] Make sure there is nothing targeting this release that still needs to be validated
(is:issue sort:updated-desc milestone:"<milestone>" label:verification-needed -label:verified)

Release

[ ] Publish the release

[ ] Verify the PR Pipeline on Github actions is green against the release branch.
[ ] Manually run the Stable pipeline against the release/release-xxxx.xx branch
[ ] Approve the Publish stage
[ ] For manual (if needed as automatic should be tried first)
[ ] Go to https://github.com/microsoft/vscode-jupyter/releases and add a new release

[ ] Tag is version number
[ ] Branch is release branch
[ ] Copy contents of release branch changelog into the release (just copy the markdown)
[ ] Save

[ ] Determine if a hotfix is needed

[ ] Ensure the version in package.json is updated as follows:

If released version is YYYY.MM.100, then hot fix will be YYYY.MM.110
If released version is YYYY.MM.110, then hot fix will be YYYY.MM.120

Day of VS Code publishing (Wednesday, XXX XX)

[ ] Go through info needed issues and close any that have no activity for over a month (🤖)
[ ] GDPR bookkeeping (🤖) ((@greazer does regular classification every Monday evening))
[ ] If any steps were unclear or changed in this release plan please update the release_plan.md file to make it clear for the next release

Day of VS Code releasing the next insider version (Wednesday, XXX XX)

[ ] Bump the engines.vscode version on the main branch to point to the next insider version. For example, from 1.58.0-insider to 1.59.0-insider

Prep for the next release

[ ] Create a new release plan (🤖)

Title <Month> <year> Release Plan
Add the labels endgame-plan to this issue
Pin this issue

[ ] (Un-)pin release plan issues (🤖)

Notes.

I'm skipping the npm install step for package-lock.json as we currently have the issue that our lock file is getting totally reorganized. Just npm ci.


Child accessory fetches name from main accessory after adding to HomeKit

I encountered the issue that when you add a bridge to the Home application, the accessory is added normally, but takes the name of the main device and overwrites it to all child controls. See here what accessories look like in homebridge. Humidifier is the main device and then there are switches of its modes.

Next this is what a grouped accessory looks like after being added.

Look - the name of the child device is taken from the main one, though...

although... if I erase the name, then seeing the placeholder showing is a correct name.

It seems this is some kind of bug in the bridge structure, the name of the Home Kit child should be the same as in homebridge.
{                     "name": "Air humidifier",                     "ip": "<IP_ADDRESS>",                     "token": "-hidden-",                     "deviceId": "530142419",                     "model": "zhimi.humidifier.ca4",                     "pollingInterval": 10,                     "deepDebugLog": false,                     "buzzerControl": true,                     "ledControl": true,                     "childLockControl": true,                     "modeControl": true,                     "fanLevelControl": true,                     "screenControl": true                 }
Thank you.

Hi,
this is a know issue with ios16.
Please read about it here:  https://github.com/merdok/homebridge-miot/issues/338#issuecomment-1249729272
As far as i know there is still nothing that i can do about so as a temporary solution you would need to rename your accessories manually.


assignment-1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Adding code based on the instructions in the assignment.
What did you learn from the changes you have made?
Make new directory, make/copy/delete files, move files from one directory to another, use wildcards, and output re-direction.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
Changes were tested by executing the commands individually in the terminal and at the end by running them together as a bash script.
A reference to a related issue in your repository (if applicable)
Checklist

[✓] I can confirm that my changes are working as intended

You did a great job here!
Just keep in mind that the branch name should be assignment not assignment-1.
Please go ahead and do the modification and then let me know.
. Thanks!


Too many issues

Problem

We have 199 issues and a happy one ain't one

Solution

Make another issue. Make it a happy one

Welp, the issue's closed


License error

After registering for the free PostSharp Express, I still get the following licensing error upon building:

License error. MyProject.dll uses non-licensed features (Portable Class Library Support). Please visit  > http://www.postsharp.net/purchase to acquire a license of PostSharp. The installed user license is: "..."

I am using Visual Studio 2015 and DNX projects.

We haven't looked at the licensing for .NET Core yet. I suggest you apply for an open-source license at: https://www.postsharp.net/purchase/discounts

@gfraiteur We are not using .NET Core.  I'm referencing PostSharp.Dnx in a DNX project (targeting dnx451) and attempting to create my own OnMethodBoundaryAspect but I cannot get this project to build due to the above error.

I understand. But we are planning to work on licensing in 4.3 so if getting a free OSS license would fix the problem for you, we would save some time addressing this issue in 4.2. That's why I'm proposing you this license.

Thanks, I have applied.

Okay, the license is on the way. Write if it does not come within 15 min (or check your spambox).


Is the timeout setting working with this gem?

I have this setting:
default: &default
  adapter: postgresql
  encoding: unicode
  pool: 5
  host: <%= ENV['DB_HOST'] || 'localhost' %>
  port: <%= ENV['DB_PORT'] %>
  database: <%= ENV['DB_NAME'] %>
  username: <%= ENV['DB_USERNAME'] %>
  password: <%= ENV['DB_PASSWORD'] %>

staging:
  <<: *default
  pool: 1
  timeout: 5
  multidb:
    databases:
      master:
        - host: <%= ENV['DB_HOST'] %>
          port: <%= ENV['DB_PORT'] %>
      slave:
        - host: <%= ENV['REPLICA_DB_HOST'] %>
          port: <%= ENV['REPLICA_DB_PORT'] %>

with a timeout of only 5sec
On my staging machine I add an iptables rule:
iptables -A OUTPUT -d <IP_ADDRESS> -j DROP

Now I am visiting a page of a controller that uses Multidb.use(:slave, &block)
The browser hang until I get a 504
The rails log just does not show much:
Started GET "/j?q=java&l=&button=&sp=search" for .... at 2017-06-28 05:39:18 +0000
Processing by SearchController#index as HTML
  Parameters: {"q"=>"java", "l"=>"", "button"=>"", "sp"=>"search"}

I tried adding timeout to master and slave but did not work
staging:
  <<: *default
  pool: 1
  timeout: 5
  multidb:
    databases:
      master:
        - host: <%= ENV['DB_HOST'] %>
          port: <%= ENV['DB_PORT'] %>
          timeout: 5
      slave:
        - host: <%= ENV['REPLICA_DB_HOST'] %>
          port: <%= ENV['REPLICA_DB_PORT'] %>
          timeout: 5

@casertap This doesn't look related to the gem. Multidb does not care about timeouts, it only reconfigures ActiveRecord.


how to set the proxy

i want to set the proxy with this library

please help

@elsyms I can take this up as I have a requirement for this. Do you take PRs?

@vedhavyas Of course. If you cannot do it I'll happily pick it up.

@elsyms go ahead 👍


Embed plunkers are broken

All < and >  are replaced by < and > in the HTML of every embed plunker. It breaks everything.
You can see examples in the embed plunkers on the official documentation here:
https://ggoodman.gitbooks.io/plunker/content/embed.html
Thanks!

I created similar issue in a different repo by mistake:  see https://github.com/filearts/plunker/issues/139

The HTML problem has been fix but another issue seems to be remaining, see:
#140
Thank you for your awesome work!

Fixed earlier today. Thanks for reporting!


Blinking cursor in mobile when using DateRangePicker

Hi,
I am using "react-dates": "^18.1.1"
I am using DateRangePicker with withFullScreenPortal option. Once touch the input I am gettting a blinking cursor. It doesnt create a problem in desktop but it is creating a problem in mobile.
when I am using readOnly attribute, Touchinput is not working. But it works Old version only.( "react-dates": "12.2.2" version)
How to disable keyboard with calendar using DateRangePicker?
Is there any way to avoid the blinking cursor in mobile when using latest version of react-dates?

@yaswakarthi does setting the readOnly prop to true solve your issue?

Actually I'm going to mark this as a duplicate of #1476 and continue the conversation on that issue.


feat: custom cursor added (issue : #772)

Title and Issue number

Title :Custom Cursor
Issue No. : #772
Code Stack : HTML, CSS , JS
Close #772

Description

I have added the custom cursor.
Video/Screenshots (mandatory)

https://github.com/apu52/Travel_Website/assets/131477601/d560a39c-296a-4b7f-b977-e1504a4e5e30
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

[x] I have performed a self-review of my code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have gone through the  contributing.md file before contributing
<!- [X] - put a cross/X inside [] to check the box -->

Additional context:

##Are you contributing under any Open-source programme?

GSSOC'24

Let me know if there are any changes to be made.

would do please review this PR.

Please review this PR.

Sure

@apu52  kindly review this PR.

sure


feat: (co)cones as elements of limit of hom functor

We exhibit the natural isomorphism between cones on F with cone point X and the set lim Hom(X, F·) and similarly the natural isomorphism between cocones on F with cocone point X and the set lim Hom(F·, X).

I considered putting these results in their own file, but I didn't know what to call it.

Thanks!
bors merge


[form recognizer] added samples

fixes #10193

can you add 2 more sphinx snippets for creating FormTrainingClient and FormRecognizerClient?


Fix typo in parameter t56

As discussed on the openforcefield repository and cross posted here in issue #89 I've fixed the typo in t56 to be !#1 (not a hydrogen atom) as opposed to the previous !1 (not an isotope with mass 1).

See also https://github.com/openforcefield/openforcefield/pull/268
Waiting on merging temporarily.

@j-wags do we have versioning info sorted out now so that we can merge this?


Duplicate images for Gaia Sky

We see duplicate images for https://flathub.org/apps/de.uni_heidelberg.zah.GaiaSky.  Effectively, the website shows the same number of images as we have https://github.com/flathub/de.uni_heidelberg.zah.GaiaSky/blob/master/de.uni_heidelberg.zah.GaiaSky.metainfo.xml  but duplicates some of them.
What images are duplicated appears to change on some builds.  This time it is the 2nd and 3rd image (among others).
I couldn't find any apps that have a similar number of screenshots as us.  Could we be exposing a bug others  haven't found yet?

Hey,
I've downloaded https://hub.flathub.org/repo/appstream/x86_64/appstream.xml.gz and checked the file and it's already wrong there. So it happens before the website even starts.
I can also see the bug in gnome software, showing the flatpak. So this is likely a problem with https://github.com/hughsie/appstream-glib which we hopefully switch away from somewhen this year(ish?)
@barthalion


Adding new token, Workflows approval

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ x ] question about how to use this project

Summary
2 workflows awaiting approval - cannot seem to solve this issue

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)
Pull Request

@keone sorry for tag you seem to be the man with the answers for everything lol

https://github.com/solana-labs/token-list#warning-about-the-last-element-in-the-list

https://github.com/solana-labs/token-list#warning-about-the-last-element-in-the-list

WOW hah right in front of my face the whole time!!! thank youu

So I did exactly as stated, still getting the same issue, even on a new pull request from a new fork, restarting doesnt seem to work either. Build and codeQL dont run and require maintainer approval?


Unable to retrieve secret from Kubernetes secret store

Note: If you have a general support question and are looking for a quicker response, please checkout our discord channel for answers from the community:
https://aka.ms/dapr-discord
In what area(s)?

/area operator

Ask your question here
I'm having issues retreiving secrets from a Kubernetes secret store.
I created the k8s secret:
kubectl create secret generic secret-store --from-literal=hello=world
I defined the following Dapr secret store component:
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: secret-store
  namespace: default
spec:
  type: secretstores.kubernetes
  version: v1
  metadata:
  - name: ""

And when I try to retrieve the secret:
curl http://localhost:3500/v1.0/secrets/secret-store/hello
I receive an error that the secret doesn't exist:
{"errorCode":"ERR_SECRET_GET","message":"failed getting secret with key hello from secret store secret-store: secrets \"hello\" not found"}
I'm running it all on minikube, with Dapr deployed using the Helm chart.

Dapr defaults to k8s as a secret store when running in that env, can you try removing the component and see what you get?
Also, try naming the k8s secret hello, so: kubectl create secret generic hello --from-literal=hello=world

Yes! It worked!
The Dapr documentation should be updated to reflect this usage, it's currently very confusing.

Here is the problem with your original approach @guyzyl:
Your secret was called secret-store (this is a very confusing name for a secret). To access that secret with your defined secretstore (named secret-store) you would have to:
curl http://localhost:3500/v1.0/secrets/secret-store/secret-store
This would return a JSON which includes the hello key.

@berndverst thanks for the reply!
I was already able to solve this thanks to @yaron2 's suggestion.
The problem was with the secret creation.
I created a secret using kubectl create secret generic secret-store --from-literal=hello=world
When what was needed was: kubectl create secret generic hello --from-literal=hello=world
The problem is that this isn't documented anywhere, but at least now this ticket exists with some insights into this.

@yaron2 Is it required by dapr that we name the secret hello?