select2/selectize - ajax fetch options

Describe the bug
when the indicated option is selected and you have a table related via key to another table, in the edit view the selection list does not respond to clicking. Selecting an option is not possible either with the mouse or with the keyboard. does not work when displayed as a single view or when the edit view is opened in a modal
To Reproduce
Steps to reproduce the behavior:

Go to: in the edit view there must be an input of type select2 with the option "ajax fetch options" checked
Click on: input select2, start searching for content and try to select an item from the result list (list larger than 1 item)

Expected behavior
ability to select items that were searched for when the ajax fetch option is enabled
Additional context
saltorn 1.0.0 running in docker with postgresql 15. checked in chrome and firefox
selectize also does not work when ajax fetch options is checked

Hi, I just prepared the plugin for mobile and didn't see that problem.
Can you create an example app on saltcorn.com (perhaps all with public roles)

I Just want to say that this problem exists regardless ajax, If you are using embedded edit view (sub form) that include select2 field.
Steps to reproduce:

Create Table A for groups (or cities or departments), and table B for related members (like group members, or streets in the city or department members) and table C for common names for members or streets.
Create edit view for table B, the edit view includes a select2 input field to select the name from table C.
Create edit view for table A and embed the edit view for table B - to create\edit the group and its members.
Press the + sign to have the forms of 3 members in the main form of the group.
Now you find that you can't edit the names of the members, only the name in the top form.

Why the + to add sub form is duplicating the existing filled form, instead of creating empty fresh sub form?

sorry, I can't upload my project to saltcorn.com correctly. It's too big (nginx returns entity too large error 413). but I've discovered what the problem is. my tables contain data from external sources, which have unique identifiers as unique key (id). tables are imported (discovered) in saltcorn. they are not created in it. however, when I create a table from a csv file (or do it manually and then import), the structure changes relative to the original data. instead of the original unique_number_id field, id is set, and then my two fields are added. the problem is probably the strictly expected database structure with the id field, which is different for me.
steps to reproduce the problem:
create a table with data as below (but not via saltcorn! use psql or your favorite database management program for this). discover (search) the table in saltcorn. let this table be related to another one via its primary key. create a view to edit the parent table with the input select2.
I am attaching the sql code of one table and a screenshot of the relationships between the tables. To reproduce the problem, you only need one table that was not created in saltocrn.
sql code
-- public.voivodeship definition

-- Drop table

-- DROP TABLE public.voivodeship;

CREATE TABLE public.voivodeship (
	voivodeship_id varchar(32) NOT NULL,
	voivodeship_name varchar(128) NOT NULL,
	CONSTRAINT voivodeship_pk PRIMARY KEY (voivodeship_id)
);
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('02', 'dolnośląskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('04', 'kujawsko-pomorskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('06', 'lubelskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('08', 'lubuskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('10', 'łódzkie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('12', 'małopolskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('14', 'mazowieckie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('16', 'opolskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('18', 'podkarpackie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('20', 'podlaskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('22', 'pomorskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('24', 'śląskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('26', 'świętokrzyskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('28', 'warmińsko-mazurskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('30', 'wielkopolskie');
INSERT INTO public.voivodeship
(voivodeship_id, voivodeship_name)
VALUES('32', 'zachodniopomorskie');

screenshot

what the data looks like in the original table (where select2 doesn't work)

what does the data look like in a table created in saltcorn (where select2 works)

thanks, I can re-create it with an external table and ajax


feat: support array_dims function

Support of array_dims() from postgres:
This function returning a text representation of an array's dimensions
Syntax:：
array_dims ( anyarray ) → text
SQL Example:
INPUT:
SELECT array_dims(ARRAY[ [1,2,3],[4,5,6],[7,8,9] ]);
OUTPUT:
 array_dims
------------
 [1:3][1:3]
(1 row)

Thanks for your interest. However, array_dims may not be an easy or proper one. As stated in ths tracking page:

A significant difference between PostgreSQL array and RisingWave array/list is that the former requires multi-dimensional ones to be rectangular (like in C), while the latter allows jagged ones (like vector or list).

That means, we have no proper output to return for this query:
SELECT array_dims(ARRAY[ [1,2,3],[4,5],[6] ]);

We could report an error, but this also means we would have to recursively check all subarrays at each layer. At the same time, it just outputs a text representation, which is not easy to be combined with other functions and less frequently used.
If you are interested in extending the expression support, do you mind taking a look at #8937?

Thanks for your interest. However, array_dims may not be an easy or proper one. As stated in the tracking page:

A significant difference between PostgreSQL array and RisingWave array/list is that the former requires multi-dimensional ones to be rectangular (like in C), while the latter allows jagged ones (like vector or list).

That means, we have no proper output to return for this query:
SELECT array_dims(ARRAY[ [1,2,3],[4,5],[6] ]);

We could report an error, but this also means we would have to recursively check all subarrays at each layer. At the same time, it just outputs a text representation, which is not easy to be combined with other functions and less frequently used.
If you are interested in extending the expression support, do you mind taking a look at #8937?

Thinks, LGTM.
I will try  https://github.com/risingwavelabs/risingwave/issues/8937 first.
And for this issue, I will try it later after https://github.com/risingwavelabs/risingwave/issues/8937, or others can also assign to themselves

#10197


Free Microsoft-hosted parallel jobs can only run for up 60 minutes

We created our Azure DevOps organization almost 2 years ago. We used it since then to host Public Projects code and build them using Microsoft-hosted parallel jobs. Based on our observations, Public Project free tier parallel jobs CANNOT run for more than 60 minutes at a time. This goes against what is stated on this page.
"For Microsoft-hosted parallel jobs, you can get up to 10 free Microsoft-hosted parallel jobs that can run for up to 360 minutes (6 hours) each time for public projects. When you create a new Azure DevOps organization, you are not given this free grant by default."
or further down in the table
"Up to 10 free Microsoft-hosted parallel jobs that can run for up to 360 minutes (6 hours) each time"
Hence, I believe the documentation doesn't reflect the reality.

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

ID: 43d1ea36-caf8-c3f6-95f3-2d40ead86f5a
Version Independent ID: 46755386-7c55-bd3a-7984-30687a4e94f3
Content: Configure and pay for parallel jobs - Azure DevOps
Content Source: docs/pipelines/licensing/concurrent-jobs.md
Product: devops
Technology: devops-cicd-licensing
GitHub Login: @steved0x
Microsoft Alias: jukullam

@PatrickMessierGoC -- thank you for your thorough feedback.
@steved0x -- please look into this issue.
cc @juliakm for awareness

@PatrickMessierGoC I will verify the 360 minutes for free tier public project jobs, but there can be a shorter timeout configured on a per-pipeline basis. This is most common on classic pipelines, where the default is 60 minutes. If your pipeline is a classic pipeline, can you verify the timeout in the pipeline?
https://docs.microsoft.com/en-us/azure/devops/pipelines/process/phases?view=azure-devops&tabs=yaml#timeouts
For a YAML pipeline, there is a per job setting, timeoutInMinutes, and according to the article I linked, it defaults to 60 minutes (I didn't know that). If your pipeline is a YAML pipeline, could you try raising that to 360 (or set to 0 for max timeout). Let me verify that, and I will make an update in the docs to call that out, as I have seen folks with a classic pipeline get caught by the 60 minute default, but I didn't know it was also for YAML pipelines.

@steved0x We used YAML pipelines. I wasn't aware of the default value for the timeoutInMinutes parameter. I will test it and report when done. Stay tuned...

@steved0x Successfully tested with timeoutInMinutes: 0. The YAML pipeline can now run beyond 60 minutes. That being said, I would recommand adding a note/link to the documentation to clarify this point.
Thanks for your quick replies.

@PatrickMessierGoC I am reopening this issue until I can make an update to the docs to clarify this issue. Thanks for getting back so quickly!
Steve

AB#1924531


UITableViewCell with UILabel drawing text improperly

UITableViewCells using UILabels to draw text in CTCatalog are being drawn partially off screen.

@aballway please add more details/repro steps.

Sorry, I missed that comment.  This can be repro'd by running CTCatalog and opening the Font Functions page.  Scrolling down, the text in the right column will be drawn partially off screen.

Fixed with 78259bd9686dc0e8a741ecec6a67f35505b3b7d9


A link failed

There is a link pointing to this apge failed

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

ID: 509335de-3776-7c83-8693-946f32451c53
Version Independent ID: d3a3d52f-a559-2888-958f-5224bd508814
Content: CMFCToolBarEditBoxButton Class
Content Source: docs/mfc/reference/cmfctoolbareditboxbutton-class.md
Product: visual-cpp
Technology: cpp-mfc
GitHub Login: @mikeblome
Microsoft Alias: mblome

@songxia233 that hierarchy chart ends with the type documented on the current page, so no need to link. These are all by design. Thanks!


problem with running dev version

hey,
first off all, you wrote great article at medium, it was a pleasure to read.
now for my problem,
I'm trying to play a bit with the project template for maybe a future use, and after i installed the dependencies and try run npm run dev, there is an error and the server is returning error.
i did exactly as i should.
can you please help me ?

you can close, i fix it while using css and not scss


62 run bacalhau from golang

Main changes were allowing diffdock to always run in batch mode.
Also, Golang automatically does a lot of the checking the validate funcs were doing, so I removed those.
Not completely polished, but diffdock has been working reliably for me. Tested with 4 complexes.
export WEB3STORAGE_TOKEN=<your_token>
cd plex
go build

# to run one diffdock complexes
./plex -app diffdock -input-dir ./testdata/pdbbind_processed_size1

# then run the printed bacalhau command

# to run 4 diffdock complexes
./plex -app diffdock -input-dir ./testdata/pdbbind_processed_size2

# to run custom complexes just point to any directory with .pdb & .mol2 or .sdf files

ToDos not covered in this PR

[ ] allow override passthroughs to get passed through CLI
[ ] allow custom index.csv (right now it does all possible combinations)

good to review?

LGTM!

should we combine instruction_template and app-config.jsonl?- I think this would make things easier for the user to maintain.

I was thinking about that too. My current thoughts are that app-config is for the local client side setting up to send the job and the instructions.jsonl. is the more generalized info about the job. Deserves some more deep thinking at some point.

we can probably move some of the functions from utils to parse

Agree, some of the organization was biased by the co-current development.

can you add some text about the canarytest - I wonder what would be the best place for us to create that level of deep documentation

it's not working, want me to delete it? The goal was just to have a hello world type job for debugging

Would it make sense for us to simply run bacalhau for the user, instead of displaying the command?

Definitely. Especially if we want to do other actions with job submission like post to the exchange for tracking.

Do you want me to do some of the above changes in this PR or merge and have us plan this weeks changes as separate tickets?

Let's merge this and do the QoL over the next 24h


[Question] How many startup trials in distributed optimization

❓ Question
I'm distributing optimization across 6 processes.
I know if I use for each of 6 processes --n-trials 100 then i will get 600 trials.
But what with --n-startup-trials 10? Will there be 10 or 60 startup trials?
Checklist

[X] I have checked that there is no similar issue in the repo
[X] I have read the SB3 documentation
[X] I have read the RL Zoo documentation
[X] If code there is, it is minimal and working
[X] If code there is, it is formatted using the markdown code blocks for both code and stack traces.

hen i will get 600 trials.

correct.

Will there be 10 or 60 startup trials?

I'm not sure about that one, but I would assume 10 if you use a database.
I also would say it's more an optuna question than a RL Zoo one.

I have used database indeed.
I added print function next to random sampler and it appeard that the answer is 10 startup trials for 6 processes on --n-startup-trials 10 .
Thank you for answer.


rename CQ config options

Instead of
compute-runs-per-interval = 10
compute-no-more-than = "2m0s"
recompute-previous-n = 2
recompute-no-older-than = "10m0s"

I propose
incremental-queries-per-interval = 10
min-time-between-batches = "2m0s"
max-resampled-queries-per-batch = 2
max-age-of-queried-data = "10m0s"

@rkuchan @gunnaraasen @dgnorton thoughts, improvements, or suggestions?

🎉👍🎉

I don't think changing the names makes them easier to understand but don't have strong feelings about it if everyone else agrees that the change makes it easier for users.  Also, @pauldix my veto breaking config change.
I think the real solution is to rethink how it computes when CQs should run.  It might be simpler for users if the query language allowed per-CQ settings.  I think we could eliminate two of the four settings that way.  E.g.,
CREATE CONTINUOUS QUERY mycq ON mydb
   PREVIOUS 1
   PERINTERVAL 1
   BEGIN
      SELECT mean(value) INTO cpu_mean FROM cpu GROUP BY time(1m)
   END

The parser could default PREVIOUS to 1 and PERINTERVAL to 1.  For the example above, the query would run once for the current and previous interval.  That makes the query simpler to reason about and defaulting to 1 for both of those would reduce the chances of an unexpectedly expensive CQ.
Another example:
CREATE CONTINUOUS QUERY mycq ON mydb
   PERINTERVAL 4
   BEGIN
      SELECT mean(value) INTO cpu_mean FROM cpu GROUP BY time(1h)
   END

In this example, we've taken the default PREVIOUS 1 and just specified how many times we want this CQ to run for the current interval and previous interval.  1h / 4 = 15m...so it will run once every 15 minutes.
Even better would be running only the CQs whose intervals have changed.  Then we probably wouldn't need any settings.  But that's a hard problem.

I agree with @dgnorton's suggestions. All of these settings should be pulled from the config file in favor of a couple of easier to understand settings tied to each CQ. This goes inline with our use of config files: they're for host settings only. Anything that is applicable in the cluster should be in the metastore (thus the config should be specified via the query language).
I think that @dgnorton's two suggested settings cover all the bases and it's much more flexible since it's on a per CQ basis. What do you guys think?

I'm strongly in favor of @dgnorton's suggestion to set these config options per CQ.
Would it make sense to decouple the runs per interval setting for the current and previous intervals? Say there's the following CQ.
CREATE CONTINUOUS QUERY mycq ON mydb
   PREVIOUS 3
   PERINTERVAL 60
   BEGIN
      SELECT mean(value) INTO cpu_mean FROM cpu GROUP BY time(1h)
   END

The goal would be to have the latest CQ number up to date every minute. However, this would also end up recomputing the previous period every minute when we maybe only want it to recompute once or four times. Not sure if that's a real issue people will run into or even an issue that's worth solving.
Also, if we're going to move per-CQ scheduling out of the config, I think it makes sense to make the continuous querier service RunInterval configurable. RunInterval is currently hard-coded to one second, which is unnecessarily high for some low power/perf machines.

I have laid out the initial ground work for allowing the scanner and parser to understand the currently proposed syntax and pushed it to the js-5136-per-cq-options branch.
I did run into one small issue with the use of PREVIOUS. The ident string "previous" is already used in one place for the fill(previous) function. When PREVIOUS gets turned into a reserved keyword, this breaks fill(previous). I was able to work around the issue by teaching the ParseExpr function to treat the PREVIOUS token in an expression context differently than in other contexts, but that may be too hacky of a solution. Is there a better name we can use?

It would be great to have these settings per CQ, like @dgnorton 's suggestion :+1:

I've been thinking about what the name of the token should be. I don't think the word PREVIOUS helps describe what the option does. To my understanding, the option is supposed to change the number of previous intervals to resample. Maybe we should consider using RESAMPLE instead? The word PREVIOUS can refer to anything related to previous results, while RESAMPLE seems to be more specific to me.
@dgnorton @pauldix any thoughts on what word we should use in the syntax?

RESAMPLE works for me

RESAMPLE isn't clear to me.  It doesn't necessarily mean we're going into the past and (to me) almost sounds like a better replacement for PERINTERVAL.  How about...
INTERVALS
PREVIOUS INTERVALS
RESAMPLE INTERVALS
PAST INTERVALS
PAST
PAST GROUPS
GROUPS
?

Yeah, now that you mention it, RESAMPLE works better for the current interval.
Having a two word phrase feels clunky to me too. Although I don't have any better suggestions.

How about changing the options from integers to durations? Then the syntax could be something like RESAMPLE EVERY 10m FOR 2h.

I think RESAMPLE EVERY 10m FOR 2h might be the best option. It seems to describe exactly what this does and is easier to read than the other options.
It might require too many reserved keywords though. @pauldix @dgnorton thoughts?

@gunnaraasen, @jsternberg I like it. As long as the EVERY or FOR clauses are optional. Well, at least one must be present if you have the RESAMPLE.

I like RESAMPLE EVERY 10m FOR 2h but how should it handle it if the FOR duration isn't a multiple of the GROUP BY duration?

@dgnorton I was just going to not check for that and the math would just end up working itself out. So if you had RESAMPLE EVERY 2m FOR 9m, it would resample on 2m, 4m, 6m, and 8m, but not 10m.
Likely worth a test case though to ensure that's true.

@dgnorton @jsternberg I vote for being inclusive to the nearest interval. That way you know at least all CQ-generated measurements are correct for data written within the FOR duration. Applied to @jsternberg's example, RESAMPLE EVERY 2m FOR 9m would resample the 0m-2m, 2m-4m, 4m-6m, 6m-8m, and 8m-10m intervals.

I've pushed the latest on this to js-5136-per-cq-options. It includes what I believe to be a working implementation. It doesn't do error checking to see if the EVERY and FOR values make any sense, but it should still do the correct thing anyway.
I don't think we should have to deal with strange durations as we'll provide some error messages around them instead to prevent the kinds of edge cases we're talking about here, but for now, RESAMPLE EVERY 2m FOR 9m would resample on the 2, 4, 6, and 8 minute intervals.

can you create a PR so we can comment on it there?
On Mon, Dec 21, 2015 at 5:26 PM, Jonathan A. Sternberg <
<EMAIL_ADDRESS>wrote:

I've pushed the latest on this to js-5136-per-cq-options. It includes
what I believe to be a working implementation. It doesn't do error checking
to see if the EVERY and FOR values make any sense, but it should still do
the correct thing anyway.
I don't think we should have to deal with strange durations as we'll
provide some error messages around them instead to prevent the kinds of
edge cases we're talking about here, but for now, RESAMPLE EVERY 2m FOR 9m
would resample on the 2, 4, 6, and 8 minute intervals.
—
Reply to this email directly or view it on GitHub
https://github.com/influxdb/influxdb/issues/5136#issuecomment-166440660.

Pushed as part of #5194. It still needs unit tests and then likely a cleanup of the stuff that isn't relevant anymore, at least for a first iteration.

Merged. This will be included in 0.10.0.


Vmmem starts at boot and even after terminating it starts again by its own

Windows Build Number
[Version 10.0.19043.1081]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.4.72
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps
Vmmem starts automatically when I boot my system and even if I terminated via PowerShell its starts again by its selfe.
I already uninstalled wsl2 and my Ubuntu distro, did a Disk Clean up to remove any temp files but it keeps starting again by itself.
Expected Behavior
Vmmem should not start on its own. I had it on another machine and it would only start if I opened the Windows Terminal with my bash shell.
Actual Behavior
Vmmem starts automatically when I boot my system and even if I terminated via PowerShell its starts again by its selfe.
I already uninstalled wsl2 and my Ubuntu distro, did a Disk Clean up to remove any temp files but it keeps starting again by itself.
Diagnostic Logs
No response

I suspect you have Linux file (something in \\wsl$\) in your recent files list.  Can you try clearing your recent files list?

I suspect you have Linux file (something in \wsl$) in your recent files list. Can you try clearing your recent files list?

I just did what you suggested plus, I uninstalled wsl2, Ubuntu-20.04, deleted all temps files from C:\Users\user\AppData\Local\Temp and C:\Users\alicr\AppData\Local\Packages\CanonicalGroupLimited.Ubuntu20.04onWindows_79rhkp1fndgsc
Did a Disk Clean up on both my drives
Then did a clean install of Ubuntu-20.04 and wsl2 and after restarting...
Vmmem started running again by its own

I ended up doing the same above and installing Kali instead of Ubuntu
I guess there might be some sort of temp file somewhere very well hidden that when we install same distro as before it holds to some profile or setting that kind of restores from last one.
Not sure if its the best solution, but I leave it here in case someone else has the same issue.
It would be nice to have this revised and addressed though.
Thanks for the help.

I suspect you have Linux file (something in \wsl$) in your recent files list. Can you try clearing your recent files list?

thanks, it is helpful.

We'll close this for now as we don't have anything actionable for the team. This is currently by design, if there are references to files in areas like File Explorer's recent access etc. then the distro will be spun up to query that file. In the future we can explore improving this behaviour to not spin up the distro as frequently.

I have the same issue and every time it spins up it also does this, which stutters my sound in windows. I have linked my sound stuttering to wsl doing this every couple of minutes and I have no idea why or how it can be fixed.

I suspect you have Linux file (something in \wsl$) in your recent files list. Can you try clearing your recent files list?

Also, I just cleared my recent files and it spun up after a wsl --shutdown automatically, also a wsl -t kali-linux has the same behaviour.

I suspect you have Linux file (something in \wsl$) in your recent files list. Can you try clearing your recent files list?

Also, I just cleared my recent files and it spun up after a wsl --shutdown automatically, also a wsl -t kali-linux has the same behavior.

What worked for me after hours of trying everything I found online was to:

Uninstall WSL completely
Uninstall my linux distro/s (backup as needed)
Cleared the recent files list (Open File Explorer>click on File>Change folder and search options>Privacy - click Clear.
Did a Disk Cleanup (WindowsKey>type Disc Cleanup and run as Administrator>I cleared everything in there, also had made sure to backup anything if needed >
Restart at this point there should be no hyper-v or Vmmem spun
Clean install of wsl2
Clean install of a different distro

This got rid of the auto-spun of hyper-v and/or Vmmem completely.
Not sure is the best solution, as it could be a pain to change distro but, luckily I only had Ubunto20 and now I'm on Kali(loving it btw).
Wish you luck.


Caching is not working properly with Repositories.GetContents

Not sure if it's go-github issue or httpcache one.
I'm trying to cache Repositories.GetContents responses to avoid my rate limit's burning.
package main

import (
	"context"
	"fmt"

	"github.com/google/go-github/github"
	"github.com/gregjones/httpcache"
	"github.com/gregjones/httpcache/diskcache"
)

func main() {
	cl := github.NewClient(httpcache.NewTransport(diskcache.New(".")).Client())
	_, _, r, _ := cl.Repositories.GetContents(context.Background(), "google", "go-github", "LICENSE", nil)
	fmt.Println(r)
}

At first run (diskcache file is going to be created for the first time), it works
$ date && go run main.go
Thu 23 Apr 2020 02:01:04 PM CEST
&{0xc00014c090 0 0 0 0  github.Rate{Limit:60, Remaining:5, Reset:github.Timestamp{2020-04-23 14:20:15 +0200 CEST}}}
...
$ date && go run main.go
Thu 23 Apr 2020 02:01:27 PM CEST
&{0xc000114000 0 0 0 0  github.Rate{Limit:60, Remaining:5, Reset:github.Timestamp{2020-04-23 14:20:15 +0200 CEST}}}

But if you run it again, after 60s (Github API's max-age), it doesn't work anymore
$ date && go run main.go
Thu 23 Apr 2020 02:02:17 PM CEST
&{0xc000112000 0 0 0 0  github.Rate{Limit:60, Remaining:4, Reset:github.Timestamp{2020-04-23 14:20:14 +0200 CEST}}}
...
$ date && go run main.go
Thu 23 Apr 2020 02:02:19 PM CEST
&{0xc00018a000 0 0 0 0  github.Rate{Limit:60, Remaining:3, Reset:github.Timestamp{2020-04-23 14:20:14 +0200 CEST}}}

I see you already found #242 (where I probably would have put this issue).
I've never personally used httpcache and I see in #242 that others have had challenges with it.
I would suggest contacting the httpcache author and see what he recommends, then please report back here (or close this issue and report back in #242... either way is fine with me).

@gmlewis sure! let me try httpcache.
I'll keep this open since #242 could be different.

Update:
httpcache is lazy, and invalidates the cache after full http.body read. The issue is, we never get EOF here in code snippet even though go-github reads the full body.
At first place it was really confusing because it works occasionally! then I noticed, sometimes Github API server sends Transfer-encoding: chunked. Whenever we have chunked, it uses chunkedReader, gets EOF, and invalidates the cache. But without chunked, it uses bufio reader, and misses EOF.
An easy workaround for httpcache would be checking the length beside EOF here.
func (r *cachingReadCloser) Read(p []byte) (n int, err error) {
	n, err = r.R.Read(p)
	r.buf.Write(p[:n])
	if err == io.EOF || n < len(p) {
		r.OnEOF(bytes.NewReader(r.buf.Bytes()))
	}
	return n, err
}

But anyways we should check why bufio doesn't handle EOF.

Maybe you could submit a PR to the httpcache repo with the workaround?
Alternatively (or in addition :smile:), you could submit a PR to this repo with a unit test demonstrating the bufio EOF problem along with the fix (if I understand you correctly).
Third option - if you could put a tiny-as-possible test case into this issue that demonstrates the bufio EOF problem, I could start looking into a fix.
Thanks for the investigation.

The bufio issue could be in Go, not in go-github (I'm looking into it).
With fixing bufio there is no need to send any PR to httpcache.

It's not a Go issue because document says EOF is optional. I'm sending a PR to fix the issue in httpcache.
Since issue is happening inside JSON decoder (where we read the body), we cannot have a test/PR in this repo, sorry :)

OK, sounds good to me.  Thanks, @m4ns0ur.
Closing issue.

For reference: https://github.com/gregjones/httpcache/pull/104

@m4ns0ur - would that PR you reference also resolve our #242?

If so, would you mind referencing your PR in that issue as well?

@gmlewis I don't think so.


Change Windows Message Box to Windows System Tray Notifications

Currently, some notes are output as windows message box, but requires additional clicks to close, which is not ideal.
To Do:

Investigate whether it is possible to output a system tray notification with powershell
Change all messages from message box to system tray notifications
Add new system-tray menu setting to turn notifications on and off

Set notifications on by default

The initial message box will remain, which raises an error when the tool is not run as administrator initially. This is required to install the AudioDeviceCmdlets module. Because this occurs prior to creating the system tray application, there is no object to output a balloon tip.


Accept multidimentional atomic indexes

Previous implementation used to work only with flat indexes. This one accept multidimentional indexes.
Question: Should we remove default value for the index? It will cause TypeError on 2- and 3-dimentional arrays?

[x] Have you provided a meaningful PR description?
[x] Have you added a test, reproducer or referred to an issue with a reproducer?
[x] Have you tested your changes locally for CPU and GPU devices?
[x] Have you made sure that new changes do not introduce compiler warnings?
[ ] If this PR is a work in progress, are you filing the PR as a draft?

Question: Should we remove default value for the index? It will cause TypeError on 2- and 3-dimentional arrays?

Yes, I think so. It was a bad design initially also. We should force people to be explicit with creation of atomic ref.


Creating Repo from Template for Private Repositories Fail

When creating a GitHub repository from a template for a private repository, the apply step failed. The underlying code looks to be trying to also enable GitHub pages (which in my case was not specified).
Expected Behavior
When creating a private GitHub repository from a template the apply step should complete without error.
Current Behavior
When creating a private GitHub repository from a template the apply step fails with the following:

│ Error: POST https://api.github.com/repos/TheYorkshireDev/test_github_repo_from_template/pages: 422 Your current plan does not support GitHub Pages for this repository. []
│
│   with module.test_github_repo_from_template.github_repository.this,
│   on .terraform/modules/test_github_repo_from_template/main.tf line 1, in resource "github_repository" "this":
│    1: resource "github_repository" "this" {
│
╵
Error: Terraform exited with code 1.
Error: Process completed with exit code 1.

Steps to Reproduce
Try and create a repository with the following code block. The code does not specify the repository has the public visibility, therefore, is defaulted to private which does not support GitHub pages.
module "test_github_repo_from_template" {
  source  = "operatehappy/repository/github"
  version = "3.0.0"

  name = "test_github_repo_from_template"

  template = {
    owner      = "TheYorkshireDev",
    repository = "application-template"
  }
}

The output of the plan gives:
Terraform will perform the following actions:

  # module.test_github_repo_from_template.github_branch_default.this will be created
  + resource "github_branch_default" "this" {
      + branch     = "main"
      + id         = (known after apply)
      + repository = "test_github_repo_from_template"
    }

  # module.test_github_repo_from_template.github_repository.this will be created
  + resource "github_repository" "this" {
      + allow_merge_commit     = true
      + allow_rebase_merge     = true
      + allow_squash_merge     = true
      + archive_on_destroy     = false
      + archived               = false
      + auto_init              = false
      + default_branch         = (known after apply)
      + delete_branch_on_merge = false
      + etag                   = (known after apply)
      + full_name              = (known after apply)
      + git_clone_url          = (known after apply)
      + has_downloads          = false
      + has_issues             = true
      + has_projects           = false
      + has_wiki               = true
      + html_url               = (known after apply)
      + http_clone_url         = (known after apply)
      + id                     = (known after apply)
      + is_template            = false
      + name                   = "test_github_repo_from_template"
      + node_id                = (known after apply)
      + private                = (known after apply)
      + repo_id                = (known after apply)
      + ssh_clone_url          = (known after apply)
      + svn_url                = (known after apply)
      + visibility             = "private"
      + vulnerability_alerts   = true

      + pages {
          + custom_404 = (known after apply)
          + html_url   = (known after apply)
          + status     = (known after apply)
          + url        = (known after apply)

          + source {
              + branch = "main"
              + path   = "/"
            }
        }

      + template {
          + owner      = "TheYorkshireDev"
          + repository = "application-template"
        }
    }

Plan: 2 to add, 0 to change, 0 to destroy.

It clearly shows an attempt to enable GitHub pages. Then the apply step fails (see below)
module.test_github_repo_from_template.github_repository.this: Creating...
╷
│ Error: POST https://api.github.com/repos/TheYorkshireDev/test_github_repo_from_template/pages: 422 Your current plan does not support GitHub Pages for this repository. []
│
│   with module.test_github_repo_from_template.github_repository.this,
│   on .terraform/modules/test_github_repo_from_template/main.tf line 1, in resource "github_repository" "this":
│    1: resource "github_repository" "this" {
│
╵
Error: Terraform exited with code 1.
Error: Process completed with exit code 1.

Other Information
I believe the underlying cause is because the module is trying to create GitHub pages from var.template as seen on line 28 of main.tf rather than var.pages https://github.com/operatehappy/terraform-github-repository/blob/74a7d355e4b4735110676a78093874de32719938/main.tf#L28

Fixed in #29


Select, Edit Data, and Scripting fail for databases with special characters in name

SQL Operations Studio Version: master

Steps to Reproduce:

Create a database with special characters in the name like dbo."'``'"@test
View it in Object Explorer and try to run Select, Edit Data, or any script action on it

Expected behavior: Select, Edit Data, and scripting work as normal
Actual behavior: Nothing happens and an error appears in the developer console: "Scripting failed. error: An error occurred while scripting the objects."

Verify this is fixed by Microsoft/sqltoolsservice#701 and investigate adding automated tests

Verified this is fixed now


Release version 0.6

Hi Lotus has a dependency on Lotus Utils version 0.6 https://github.com/lotus/lotus/blob/master/lotusrb.gemspec#L22
But Gem Version is currently:
Can you please release 0.6?

@DanielBlanco lotus-utils 0.6 will be released when lotusrb will hit 0.6.
I think you're using lotusrb master if you're asking this question :wink: If so, please add this line to your Gemfile.
gem 'lotus-utils', github: 'lotus/utils', branch: '0.6.x'


WIP: Use SKR Module List as single source of Truth

Description
Changes proposed in this pull request:

Implements https://github.com/kyma-project/lifecycle-manager/issues/756
Related issue(s)

Lets wait for https://github.com/kyma-project/lifecycle-manager/pull/834 to be merged, since the tests introduced in https://github.com/kyma-project/lifecycle-manager/pull/834 needs to be adapted with this change in the current PR.

/test pull-lifecycle-mgr-tests


Newtmgr logs show

Hey gordon, this isn't compiling for me --
log_nmgr.c:47:6: error: 'LOGS_NMGR_OP_CLEAR' undeclared here (not in a function)
[LOGS_NMGR_OP_CLEAR] = {NULL, NULL}, /* Stub */
^
Can you take a look, I don't see where LOGS_NMGR_OP_CLEAR is defined.
Thanks,
Sterling

Sorry about that. Looks like I missed adding that change.
It is missing a
#define LOGS_NMGR_OP_CLEAR (1)
I'll fix this tonight when my laptop is on a network.
Gordon
On Feb 22, 2016 3:08 PM, "Sterling Hughes"<EMAIL_ADDRESS>wrote:

Hey gordon, this isn't compiling for me --
log_nmgr.c:47:6: error: 'LOGS_NMGR_OP_CLEAR' undeclared here (not in a
function)
[LOGS_NMGR_OP_CLEAR] = {NULL, NULL}, /* Stub */
^
Can you take a look, I don't see where LOGS_NMGR_OP_CLEAR is defined.
Thanks,
Sterling
—
Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-mynewt-larva/pull/6#issuecomment-187430642
.

@sterlinghughes: a91d0a9 addresses the issue I introduced.


Update CSharp.lua

This is actually an issue for your fork of CSharp.lua, but I think issues are disabled there, so I'm posting here. Could you update it with the latest from yanghuan/CSharp.lua:master? It has a few bug fixes for issues I posted (here and here). It would be great to have those merged in with your WC3-specific transpiler. Thanks - I appreciate the help!

Hello! Unfortunately, you are talking about War3Api, not WCSharp. WCSharp is just a set of libraries to make programming maps in C# easier, but the functionality of compiling the map, transpiling the code, etc. is handled by War3Api, which in turn is managed by Drake53. You'll want to open an issue on his CSharp.lua repo for this.


{Identity} Merge release 2.10.0 to identity

Description
Merge release 2.10.0 to identity for 2.10.10 beta release.
⚠ Do not squash merge!
⚠ Do not solve conflicts on GitHub, otherwise release branch will be updated.

Identity


Missing v1.3.0rel tag/release

The release is advertised in the changelog but is not present under either releases nor tags.

Wow, cannot believe how synchronized we where (I had the update to 1.3.0 on my todo list since November 20th)… I’ve received the notification email for the release exactly after clicking “Comment”.
That being said, the release was now tagged with a changelog file mentionning the 1.3.1 release and the corresponding #212 PR that is indeed included. Is that expected?

fixed now

Does not seems so, e.g. https://github.com/Unidata/cftime/blob/v1.3.0rel/cftime/_cftime.pyx#L56 this does not look right to me: the v1.3.0rel tag is pointing to a state that is 1.3.1 (also in the changelog).

sorry, it's really fixed now.


Simplify next-release.sh

Test plan

N/A

Why change this now?

@dominiccooney, we improved the versioning recently by "disconnecting" it from the sourcegraph relases. Now we realised we can go further. The former buildnum versioning does not look good, cannot be controlled, and is very complicated. Now, with a new release script created by Peter we can easily create new releases and bump major/minor/patch one by one.


[Merge] CL:0002609 (neuron of cerebral cortex) into CL:0010012 (cerebral cortex neuron)

@dosumis @shawntanzk
CL term (ID and Label)
CL:0002609 (neuron of cerebral cortex)
Reason for deprecation
The parent CL:0010012 (cerebral cortex neuron) is Equivalent to: neuron and has soma location some cerebral cortex
The child CL:0002609 (neuron of cerebral cortex)is Equivalent to: CNS neuron (sensu Vertebrata) and has soma location some cerebral cortex
but cerebral cortex is only found in vertebrates, right? (I have a vague recollection of a recent discussion on related sensu terms)
If yes, the two terms are effectively the same. CL:0002609 has children itself, hence my suggestion to merge rather than obsolete.
If the two terms are not the same, the labels should be disambiguated please.
Term to be merged with (ID and Label)
CL:0010012 (cerebral cortex neuron)
Thanks,
Paola

I'll handle this - we actively use one of these terms so will be careful with my merge. Thanks for pointing this out @paola-scibite

@shawntanzk from an editing POV, I'm not sure of the full ideal strategy to avoid breaking anything, other than triple-checking that any occurrence of the relevant IDs is addressed/accounted for.

I think that "breaking" something is inevitable with obsoleting things like that, some dependencies will be affected no matter what. But I guess in our ontologies, replacing terms are pretty easy (eg BDS can change to using whichever term quite easily, and I assume GO is the same). I'm just unsure about the etiquette on this - is un-deprecating a term ok?
@hdrabkin does MGI use the term itself for annotation or is it through GO? cause if it is through GO, a solution might be to get GO to replace CL:0002609 with CL:0010012, which would be a trivial thing for them to do. Though not sure what GO release cycle is like, whereas in CL (assuming nothing is broken >.<) I can probably do a release immediately after the change.
Happy to do either, but would like input on this. Thanks!

Undeprecating a term is rare, but has happened on occasion.
I assume Harold would not have brought up this concern if it were an easy fix on MGI's side.  In any case, we (the CL community) need to have better protocols for announcing term obsoletion events to the community, and as GO is a major user of CL, we might include the GO-annotation mailing list as one target for emails announcing future CL term obsoletions.
In general, we should not favor creating new classes to replace existing CL classes when the only change is adding details to the definition or adding logical axioms that do not fundamentally change the meaning of the class.

I see now that apparently, we have used both ids in extensions for GO annotation. we've used CL:0010012 much less (8 genes) vs CL:0002609 (87).  I could easily fix changing the CL:0010012 to CL:0002609 if the latter became the winning term, but the other way around is a problem.  If it were annotations coming only from our in house interface, we might be able to have an SE do a quick replace wherever CL:0002609 is found (part_of; occurs_in, etc.).
However, there is no such quick fix for the annotations made in Noctua; I think David Hill said there was a ticket in to do something to make correcting such things easier.
We were alerted when I looked at qc reports for 'Invalid "Properties" Values in GO Annotations' .
I don't know yet whether once we officially load the cell ontology in the MGI db whether this at least will stop the error report, but of course it would leave a deprecated term in the fields scanned. Might not be a problem if the link-outs would still work.  I originally thought based on look-ups in Ontobee and Obo foundry that the 2609 term was obsoleted, but from the ticket here I see that it was merged; Gurus on this thing tell me that in Protege, a merge is a kind of obsoleting of the losing term, so this is why it showed up like that when I searched.
Anyways, it appears that the annotations flagged are not prevented from loading into the MGI db yet. However, when we finally do a list import into Noctua and shut down our interface, I wonder in Noctua will reject all of the annotations flagged;
Sorry for blathering.

Undeprecating a term is rare, but has happened on occasion.

Are we happy to undeprecate CL:0002609 and merge CL:0010012 into it instead then? (or like basically literally do a switch in ID in the file lol) - if so, let me know and I can do it and make a release :)

Few quick points:

If this has been released, I would avoid reverting.
We should have an SOP for checking GO usage and taking that into account for winning term in future.
GO has never pre-announced trivial merges like his (presumably MGI copes with these just fine?), so I 'd be reluctant to follow that route as it could slow things down considerably.  We should be announcing any obsoletion that is non-trivial.  We probably need to write some guidance on this.  GO examples might be useful to consider.
We are planing to introduce a simple obsolete term mapping system in OxO later this year.  Would this be of help in future?

Just got off the phone with a GXD curator about the version of CL loaded into our backend.
Apparently, GO:0002609 is not al alt id for CL:0010012 and neuron of cerebral cortex is not a synonym for cerebral cortex neuron. They loaded from an obo file and I'm not sure where they get it.
Since the use of CL for GXD is very recent (this week), they haven't any backlog of terms used.
When we load the GO (nightly), we handle merges because of the new term having the alt-id, so this gets 'fixed' except in text fields where we would now get an error report (like a GO id in the with-field of an IC annotation).

PS please give me a thumbs up and a go ahead or whatever if we are all agreeable to the above

And for what it's worth
Noctua has CL:0002609 obsolete neuron of cerebral cortext  (at least I can add it to a model as an individual), and CL:0010012 as cerebral cortex neuron so at least the old term can't be used any further. I can find annotations using CL:0010012; MGI has two models using the term. However, I can't do a search for use of the obsoleted term in the search dropdown.

@shawntanzk, I wonder if the lack of alt_id and the synonym is why it can't be searched on in Noctua
In GO when we do a merge using protege,  we have to make the losing term an alt id and the old term name a synonym. If we forget we can't push the change.

synonym is totally my fault - sorry about that, totally missed it. I've not really done alt_id though, always thought we just added item_replaced_by. But if its important to have alt_id we should have it in our docs.

We do add the replaced by when we obsolete a term if there is a clear id to use instead.

To answer @shawntanzk ' MGI use the term itself for annotation or is it through GO? cause if it is through GO, a solution might be to get GO to replace CL:0002609 with CL:0010012, which would be a trivial thing for them to do. Though not sure what GO release cycle is like, whereas in CL (assuming nothing is broken >.<) I can probably do a release immediately after the change.'
At the moment MGI is using through GO, but we have just started loading the CL in our backend for use with GXD (gene expression db) and Alleles and Phenotype.
However note that it is difficult to search on the older term. I am wondering if the key is to add the alt id, so that when I search on 2609, I get back annotations that are using it? I can't tell at the moment. As for our in-house interface, I put in a work order to try to replace all instances of 2609 with the 0012 term. That request is being triaged now.

@hdrabkin we'll add the alt_id in this case

@hdrabkin - I've made a new release for CL, hopefully that helps solve your problem, if not do feel free to open a ticket again. Thanks :)


Initial version of a simple VerneMQ clustering status page

The status page shows the following information:

Node information

Peercount, how many cluster members this node can communicate with
Connect Rate (sampled and averaged at 10 second intervals)
Publish In/Out Rate (sampled and averaged at 10 second intervals)
Msg drop Rate (sampled and averaged at 10 second intervals)
Nr. of clients (online/offline)
Nr. of queued messages
Nr. of subscriptions
Nr. of retained messages
Running Protocols (a selection of MQTT, MQTTS, MQTTWS)
VerneMQ Release version

Cluster information

Cluster size
Aggregated Node information (see above)

Cluster issues, shows issues, that might degrade overall stability

Warns if cluster communication uses <IP_ADDRESS> interface
Warns if VerneMQ release version differs among cluster nodes
Warns if a cluster node isn't reachable by any other node
Informs if a MQTT(S|WS) listener isn't running

Technology:

Bootstrap, Mustache.js, JQuery

I think this looks great!
Do we have to declare something (beta/not api) to be able to change this in the future?  Or is it the general expectation that we can change this however we like in a minor version?
One enhancement idea would be to explain why it is a warning to use the <IP_ADDRESS> interface and why that is a potential problem. Maybe we could add links to the wiki or docs for explaining this?
Should it be possible to disable this feature?
A changelog item should be added.

Is this merged into the new release?

No, as today's release was a bugfix it was not included. It'll be part of 1.4.0.

I'll should be in the nightly build though (which is just 1.3.1 + this PR).


GridField Translate

Hello , I wonder if there is a possibility to create version for each language , grid inserts make Silverstripe

Hi, you can add the FluentExtension to any dataobject and you can translate them in the gridfield. You will still need to use the left-hand menu dropdown to toggle between languages though.

I will do that , thank you.


Ruby on Rails capstone project - Budget app

Hey there
Welcome to my RoR capstone project.
Here are the Project Requirements:
You will create a Ruby on Rails application that allows the user to:

Register and log in, so that the data is private to them.
Introduce new transactions associated with a category.
See the money spent on each category.

Design

Follow these design guidelines, including:

Colors.
Typography: font face, size and weight.
Layout: composition and space between elements.

Interactions

Splash screen.

A simple page with the name of your app (yes, you need to choose one), and links to the sign up and log in pages.

Sign up and log in pages

The user should be able to register in the app with full name, email and password (all mandatory).
The user can log into the app using email and password.
If the user is not logged in, they can't access pages that require the user to be logged in (all the pages described below).

Home page (categories page)

When the user logs in, they are presented with the categories page.
For each category, the user can see their name, icon and the total amount of all the transactions that belongs to that category.
When the user clicks (or taps) on a category item, the application navigates to the transactions page for that category.
There is a button "add a new category" at the bottom that brings the user to the page to create a new category.

Transactions page

For a given category, the list of transactions is presented, ordered by the most recent.
At the top of the page the user could see the total amount for the category (sum of all of the amounts of the transactions in that category).
There is a button "add a new transaction" at the bottom that brings the user to the page to create a new transaction.
When the user clicks on the "Back" button (<), the user navigates to the home page.

"Add a new category" page

The user fills out a form to create a new category, indicating their name and icon (both mandatory).
The user clicks (or taps) the "Save" button to create the new category, and is taken to the home page on success.
When the user clicks on the "Back" button (<), the user navigates to the home page.

-"Add a new transaction" page.
- The user fills out a form to create a new transaction with:
- name (mandatory)
- amount (mandatory)
- categories (mandatory at least one)
- The user click (or taps) the "Save" button to create the new transaction, and is taken to the transactions page for that category.
- When the user clicks on the "Back" button (<), the user navigates to the transactions page for that category.
Testing requirements

Create unit and integration tests for all the most important components of your RoR application.

Technical requirements

You should use Postgres as your database.
You should use devise for authentication.
You should validate all user input to make sure that anyone with bad intentions cannot compromise your app.
You can use a view template engine of your choice (.erb, .slim, .haml).
The project should be deployed and accessible online.
Your database schema should reflect the following structure:

Thanks for your kind review and feedback

Thanks @DuaneDave !!


dynamic_reconfigure: 1.5.48-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository dynamic_reconfigure to 1.5.48-0:

upstream repository: https://github.com/ros/dynamic_reconfigure.git
release repository: https://github.com/ros-gbp/dynamic_reconfigure-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.26
previous version for package: 1.5.47-0

dynamic_reconfigure
* [Bugfix] dont enforce ROS names for constants (#84 <https://github.com/ros/dynamic_reconfigure/issues/84>)
* [Compiler warnings] avoid unused-parameter compiler warnings in specialized ParamDescription<std::string>::clamp() (#83 <https://github.com/ros/dynamic_reconfigure/issues/83>)
* Contributors: Johannes Meyer, Mikael Arguedas

fixes regression introduced by 1.5.47 on several leaf packages


VerneMQ and X509 services integragion - Implement an authentication plugin for VerneMQ using Redis Database

The plugin must validate if the fingerprint is associated with the corresponding Client ID in the Redis database.

#2067

done!


Feature absorber doing nothing

rftools-1.1x-6.00.jar
rftoolsdim-1.1x-4.55.jar
Created several test dims in ssp and smp, set a feature absorber (or 9) in each dim, and they did not absorb anything. Tried giving them a redstone signal as well and that had no effect.

Were there features in that dimension?


Grid layout breaks auto-expanding editor panes.

VSCode Version: 1.25.0
OS Version: Windows 10

Steps to Reproduce:

Open three editors in horizontal arrangement fashion:

Squeeze the right-most one to minimal allowed size
Switch focus to the middle one, then to the squeezed one
Editor sizes DO NOT CHANGE

This didn't happen prior to 1.25 - editor panes, when one was contracted to a minimum, used to expand the one that's focused and squeeze those that were not in focus. This change (statically sized editor panes) completely breaks my workflow :(
What's worse - when going to "Zen" mode it leaves BIG WIDE STRIPES OF NOTHINGNESS on the sides!

Are we going for a hip & cool new look of new-wave messengers suck as Viber and WhatsApp over getting the job done in a functional manner now?

This is the behaviour I came to absolutely love and utilize heavily:

That way I don't lose track of the files I jump between while working on some feature, but when I jump between them I always have the one in focus maximized. 1.25-way, where the sizes don't change - no bueno.

Duplicate of https://github.com/Microsoft/vscode/issues/51614

@Taugeshtu

What's worse - when going to "Zen" mode it leaves BIG WIDE STRIPES OF NOTHINGNESS on the sides!

You can disable this by configuring "zenMode.centerLayout": false


Log on request instead of on response

Hi, It would be great if I'd had the option to configure expressWinston in a way that it would log messages on request instead of on response.
This is important for cases such when a server crashes and does not respond with any response (timeout for example).
Similar behavior is implemented in morgan when initiatlized with immediate: true.

Hm, that's a pretty fundamental change to the module as a whole, but I'm interested in your scenario. In the case where the server crashes or times out, isn't that a scenario that the express-winston error logger should already be handling?

if server crashed it doesn't reach the errorLogger and the request that caused this is lost

would also be interested in this scenario...

@Ijmir explained it perfectly.

@Ijmir's explanation makes sense.
PRs are welcome. I may get to this myself, but it's not my highest priority.

Forgive me for commenting when I'm just getting started with this package, but would it be possible to just call express-winston from the uncaughtException event?
https://nodejs.org/docs/latest/api/process.html#process_event_uncaughtexception
Instead of redesigning the whole product just to catch exceptions, can't we just add a stub into the node event handlers and do it that way on demand?

@graham-sportsmgmt You're right that uncaughtException would handle logging for most server-crash scenarios. I think the problem with that in @oleiba's use case is that there wouldn't be any way to identify which request caused an unhandled exception.

My use case is, it makes it easier when I am reading the logs in case of debugging to see when the request came in, then see what other things happened before the response or before a certain part of code. Adding this feature would be really helpful. Thanks


Bug When deleting an error in API-Publisher(UI)

Description:
When trying to delete a published API which has subscriptions  no error message  shown to the end-user following logs in the console
"ERROR - ApisApiServiceImpl Cannot remove the API b1915d42-44ee-4ce6-a404-42ae193b90e3 as active subscriptions exist"

Fixed


[Rancher2]  Update GKE docs for GKEv2

Specifics Include:

logs viewer permissions is needed
How to use setting interval

Once docs are in place, this will help with additional UI pointers: https://github.com/rancher/rancher/issues/32030

@btat I've added partial docs in staging to update the information about GKE cluster configuration.
The changes are in this PR https://github.com/rancher/docs/pull/3200

The new configuration reference page is at: https://staging.rancher.com/docs/rancher/v2.5/en/cluster-provisioning/hosted-kubernetes-clusters/gke/config-reference/

On this page I added a tab for v2.5.8 that explains the new cluster provisioning. It now says you need a project ID and it breaks out creating cloud credentials as a separate step.

And the updated GKE cluster provisioning page is at: https://staging.rancher.com/docs/rancher/v2.5/en/cluster-provisioning/hosted-kubernetes-clusters/gke/

On this page I created a new GKE cluster configuration reference because it didn't already exist for Rancher before v2.5.8. So I created both tabs. I filled in the second half of the fields.

Can you please create a new PR that fills in the incomplete information in the configuration reference? I've added placeholder section titles for the fields. (Or at least, I think I got all the fields. Let me know if you notice anything missing.)
There will also need to be changes to the docs regarding registered GKE clusters, but that's a different page and can be handled with a separate PR.
I've organized the configuration reference based on screenshots from GKE cluster provisioning before and after v2.5.8. This is how I got the organization to break down the reference options by cluster options, node pools etc.
What still needs to be documented

First half of configuration reference (the ones that I added placeholders for)
Logs viewer permissions is needed
How to use setting interval
Registered cluster docs for GKE

Screenshots of GKE provisioning in v2.5.8

Screenshots from GKE provisioning before v2.5.8


efficientnet_bN and densenet161 backpropogation

Calculation of backprop fails for these nets:
Testing  densenet161
    FAIL     od=0.00001 md=5.86444
Testing  efficientnet_b0
    FAIL     od=0.00001 md=1.40656
Testing  efficientnet_b4
    FAIL     od=0.00001 md=0.20620

Fixed in https://github.com/artyom-beilis/dlprimitives/commit/8b159bb4737827c15ecc661b2a3918ddded167f1 and in https://github.com/artyom-beilis/pytorch_dlprim/commit/8a4dc83918a6b9e81c02d3a95c0527d7ac90688e


Adding DefaultPlugins to the App panics with default-features = false

Bevy version
0.8
What you did
Use Bevy without default features and add the DefaultPlugins to the App
cargo.toml:
[dependencies.bevy]
version = "0.8.0"
default-features = false
features = ["dynamic"]

main.rs:
use bevy::prelude::*;

fn main() {
    App::new().add_plugins(DefaultPlugins).run();
}

What went wrong
The code panics with the following backtrace:
thread 'main' panicked at 'Requested resource bevy_asset::asset_server::AssetServer does not exist in the `World`.
                Did you forget to add it using `app.insert_resource` / `app.init_resource`?
                Resources are also implicitly added via `app.add_event`,
                and can be added by plugins.', /home/felix/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_asset-0.8.0/src/assets.rs:324:43
stack backtrace:
   0: rust_begin_unwind
             at /rustc/1b57946a405d5b2a87e612335db033edb2c3427f/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/1b57946a405d5b2a87e612335db033edb2c3427f/library/core/src/panicking.rs:142:14
   2: bevy_ecs::world::World::resource
             at /home/felix/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_ecs-0.8.0/src/world/mod.rs:817:21
   3: <bevy_app::app::App as bevy_asset::assets::AddAsset>::add_asset
             at /home/felix/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_asset-0.8.0/src/assets.rs:324:32
   4: <bevy_render::RenderPlugin as bevy_app::plugin::Plugin>::build
             at /home/felix/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_render-0.8.0/src/lib.rs:130:9
   5: bevy_app::plugin_group::PluginGroupBuilder::finish
             at /home/felix/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_app-0.8.0/src/plugin_group.rs:135:21
   6: bevy_app::app::App::add_plugins
             at /home/felix/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_app-0.8.0/src/app.rs:810:9
   7: viewer::main
             at ./bin/viewer/main.rs:20:5
   8: core::ops::function::FnOnce::call_once
             at /rustc/1b57946a405d5b2a87e612335db033edb2c3427f/library/core/src/ops/function.rs:248:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Additional information
It looks like the recent change that removed bevy_asset from no-default-features caused this issue as adding bevy_asset to the features functions as a workaround.

Ping @mockersf with complaints ;)

We should really be testing this specific combination in CI IMO. It's natural and important.

I can't reproduce locally, default-features = false is working correctly.
Given your stack trace, you have at least the feature bevy_render enabled. Enabling that one, I can reproduce. Are you sure about your features?

Hey, sorry for the late response. Turns out one of my dependencies did enable the render feature. I can confirm that if I remove all dependencies except for bevy and disable the default features it does not panic unless I add render. Again if I add the bevy_asset feature that fixes the problem.
I haven't checked if there are any other features also cause it to panic.


Create Cloud Function to write FHIR conversion failures to storage

Resolves https://app.clickup.com/t/3gmwjz3
This adds a Cloud Function which takes a JSON request (the failure output of the FHIR Conversion service), saves it to a file, and uploads it to a storage bucket. Includes the terraform configuration for the function and the new storage bucket. Also includes some modifications to the FHIR Conversion service to include the original filename so that it can be included in the new file saved to the Failed FHIR Conversion bucket.

Task linked: CU-3gmwjz3 Create Cloud Function to write FHIR conversion failures to storage


Hide FullScreen icon

How to hide fullscreen icon in right botton

You can edit it after clone/ download.


fix(browser-bookmarks): sort by score

Description

Screencast

Checklist

[ ] I read the extension guidelines
[ ] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

done

@pernielsentikaer Ready to merge for me.


chore: Change Query/DenomTrace gRPC

Description

Change the DenomTrace grpc so that it takes the whole ibc denom instead of just the hash
closes: #101

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #1312 (ed299c6) into main (cc4cd7b) will decrease coverage by 0.02%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main    #1312      +/-   ##
==========================================
- Coverage   80.26%   80.24%   -0.03%
==========================================
  Files         166      166
  Lines       12017    12020       +3
==========================================
  Hits         9646     9646
- Misses       1916     1918       +2
- Partials      455      456       +1

Impacted Files
Coverage Δ

modules/apps/transfer/keeper/grpc_query.go
75.00% <83.33%> (-3.27%)
:arrow_down:

I'll close this PR and make a new one


I can't run pyfolio in a virtualenv on OSX

I think the root issue here is that virtualenv on OSX by default does not use a framework build of python & seaborn needs that at import time.
There are some work arounds here, but none of them seem like an ideal solution, has anyone worked around this in a virtualenv?
http://matplotlib.org/faq/virtualenv_faq.html
import seaborn as sns

Results in:
Traceback (most recent call last):
  File "/Users/adzuci/Documents/code/pyfolio/pyfolio/plotting.py", line 20, in <module>
    import seaborn as sns
  File "/Users/adzuci/.virtualenvs/pyfolio/lib/python2.7/site-packages/seaborn/__init__.py", line 1, in <module>
    from .rcmod import *
  File "/Users/adzuci/.virtualenvs/pyfolio/lib/python2.7/site-packages/seaborn/rcmod.py", line 5, in <module>
    from . import palettes
  File "/Users/adzuci/.virtualenvs/pyfolio/lib/python2.7/site-packages/seaborn/palettes.py", line 7, in <module>
    import matplotlib.pyplot as plt
  File "/Users/adzuci/.virtualenvs/pyfolio/lib/python2.7/site-packages/matplotlib/pyplot.py", line 114, in <module>
    _backend_mod, new_figure_manager, draw_if_interactive, _show = pylab_setup()
  File "/Users/adzuci/.virtualenvs/pyfolio/lib/python2.7/site-packages/matplotlib/backends/__init__.py", line 32, in pylab_setup
    globals(),locals(),[backend_name],0)
  File "/Users/adzuci/.virtualenvs/pyfolio/lib/python2.7/site-packages/matplotlib/backends/backend_macosx.py", line 24, in <module>
    from matplotlib.backends import _macosx
RuntimeError: Python is not installed as a framework. The Mac OS X backend will not be able to function correctly if Python is not installed as a framework. See the Python documentation for more information on installing Python as a framework on Mac OS X. Please either reinstall Python as a framework, or try one of the other backends. If you are Working with Matplotlib in a virtual enviroment see 'Working with Matplotlib in Virtual environments' in the Matplotlib FAQ

I'm using these versions:
pip freeze | grep "seaborn\|matplotlib"
matplotlib==1.5.0
seaborn==0.6.0

Related: https://github.com/mwaskom/seaborn/issues/790

I think it's a matplotlib issue, not a pyfolio one.

I think the framework issue is a matplotlib+osx issue, but when the matplotlib backend is instantiated in the __init__.py file of pyfolio I can't just set the backend in the test files with matplotlib.use('TkAgg').  I was looking for a way to make pyfolio more osx friendly that didn't involve changing the matplotlib backend, but I think that might require a change to seaborn or matplotlib.  My thought was that if seaborn stopped calling out to the backend at import time I would be able to work around it better, but maybe that's the wrong idea?
@gwulfs helped me work around the issue for now by overriding my local matplotlib backend:
http://stackoverflow.com/questions/21784641/installation-issue-with-matplotlib-python
echo "backend: TkAgg" > ~/.matplotlib/matplotlibrc

I should have included that I was running into this issue running tests & that I reprod it on another osx machine:
workon pyfolio
pip install -e .
pip install patsy nose nose_parameterized runipy
nosetests pyfolio/tests

Thanks posting the work-around.

@adzuci @twiecki can we add this to the README? I just ran into this same issue

I put up a PR for https://github.com/quantopian/pyfolio/pull/257/files, I don't like the idea of adding a work around like this though unless there is no way to make it "just work".


Side Menu loses focus after navigating links with keyboard

Bug Report
Description
When I navigate in the sidebar with a keyboard and go into a submenu, I lose focus on the sidebar, and have to re tab through the page again to get to the navigation. It seems as though we should always be focused on the sidebar when navigating with it.
Steps to Replicate

Tab through the page until you are at the sidebar.
Tab to a submenu.
Activate the submenu link.

Expected Behavior
Focus is still in the menu
Actual Behavior
Focus leaves the menu

This is for the old terra-application navigation and outdated.


Not receiving audio from Web SDK after v4.x upgrade

Describe the bug
Unable to receive audio from WebSDK after upgrading to RN Agora v4.x
To Reproduce

Create a Web Client (agora-rtc-sdk-ng 4.20.0 or v3X)
Set a channel, Join the channel, and publish a local track audio track (as host or any other)
Create a React Native Client v4.x (iOS and/or Android)
Join the channel created using the Web Client SDK
Speak through the Web Client mic
Expected behavior
The React Native Clients v4.x (Android and iPhone) should receive the audio from the Web Client SDK but audio does not come through.

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
WEB: Initializing AgoraRTC client v4.20.0 build: v4.20.0-0-g334e514b-dirty(12/8/2023, 3:48:29 PM), mode: live, codec: vp9
React Native:  react-native-agora: 4.2.6
Smartphone (please complete the following information):

Device: iPhone 15, Samsum Galaxy
OS: iOS 17 and One UI

Additional context
If the React Native Client is v3.x it it can receive audio from the Web Client SDK (v3.x and v4.x)l; however, after upgrading the RN Clients to v4.x, they are not able to receive audio from the Web Client SDK (v3x or v4.x). Communication between RN Clients (v3.x and v4.x) works.
NOTE: I only use the audio portion for "live" streaming to large audiences (10+ listeners)

@jfbourne change the Reproduce step 3 client to https://github.com/AgoraIO-Extensions/react-native-agora/tree/main/example , is that work on the demo?

@guoxianzhe after many hours of troubleshooting I narrowed down the issue to be related to joinChannelWithUserAccount and the  RtcTokenBuilder.buildTokenWithAccount. Let me explain
It appears that it can successfully join the call as you can see below

However; it does not receive any audio if I join with joinChannelWithUserAccount and a token generated by RtcTokenBuilder.buildTokenWithAccount
The only time it will receive audio is if I generate a token with RtcTokenBuilder.buildTokenWithUid and join with joinChannel
This behavior is present in my app as well as the example app as you referenced above ONLY for audio (using live streaming) sent using Agora Web SDK 4.20.0 as a 'host' and the React Native Clients joining as listener audience.

@jfbourne If you use joinChannelWithUserAccount, the Agora Web SDK 4.20.0 side should also use string uid too.
eg:
let client = AgoraRTC.createClient();
client.join(appid,channel,token,'12345') // '12345' is uid

@guoxianzhe Thanks.. just a few hours before your reply I came across a warning message here describing exactly what you said

Attention:
To ensure smooth communication, use the same parameter type to identify the user. For example, if a user joins the channel with a user ID, then ensure all the other users use the user ID too. The same applies to the user account. If a user joins the channel with the Agora Web SDK, ensure that the ID of the user is set to the same parameter type.

https://api-ref.agora.io/en/voice-sdk/react-native/4.x/API/class_irtcengine.html#api_irtcengine_joinchannel2
Thank you that solved my issue.


Missing onlyoffice.socket in "/var/run/onlyoffice/" after .deb installation

I have performed a .deb installation (following Installing Community Server for Linux on a local server guide) on my Debian 8 server.
I get the following errors in the Nginx server logs:
[crit] 17700#0: *14 connect() to unix:/var/run/onlyoffice/onlyoffice.socket failed (2: No such file or directory) while connecting to upstream, client: <IP_ADDRESS>, server: docs.example.com, request: "GET / HTTP/1.1", upstream: "fastcgi://unix:/var/run/onlyoffice/onlyoffice.socket:", host: "docs.example.com"
[crit] 17700#0: *14 connect() to unix:/var/run/onlyoffice/onlyoffice2.socket failed (2: No such file or directory) while connecting to upstream, client: <IP_ADDRESS>, server: docs.example.com, request: "GET / HTTP/1.1", upstream: "fastcgi://unix:/var/run/onlyoffice/onlyoffice2.socket:", host: "docs.example.com"
[error] 17700#0: *14 no live upstreams while connecting to upstream, client: <IP_ADDRESS>, server: docs.example.com, request: "GET /favicon.ico HTTP/1.1", upstream: "fastcgi://fastcgi_backend", host: "docs.example.com", referrer: "http://docs.example.com/"

It seems that sockets in /var/run/onlyoffice/ are missing.
/var/run/onlyoffice/ is indeed empty, is owned by onlyoffice:www-data and has the following permissions: drwxr-sr-x.
I tried restarting monoserve* and onlyoffice* in hope theses sockets would be recreated: no luck.
How can I manually create them?

No creation need, you may link the two sockets between with performing the following command from the inside of a docker-container:
ln -s /var/run/onlyoffice/onlyoffice2.socket /var/run/onlyoffice/onlyoffice3.socket


Add "=" sign to the initialization value of Tau

"=" symbol missing in tau = random_state.normal((3 * prob) + 1 * (1 - prob), 0.1).

And thanks again @Tanmay-Kulkarni101. We appreciate your help!


MTB: Add test for Block modification of resource quotas (MTB-PL1-CC-TI-1)

Add automated test case for:
https://github.com/kubernetes-sigs/multi-tenancy/tree/master/benchmarks/e2e/tests/block_ns_quotas

/remove-lifecycle stale

/assign @phoenixking25

closing as the project will be restructured


Return Id of new posted object

Hello guys!
Thank you for such great project.
But I stuck on one thing: I want to post object and I want to have id in response(because id is auto incremented). I found one way - creating of stored procedures. But is not good, because I will need to create that procedures for all tables.
How can I resolve my issue(getting posted object id on callback from posting process)?
Thanks!

@HorunS Use the Prefer: return=representation header.
This is mentioned in http://postgrest.org/en/v5.2/api.html#insertions-updates.

Great!! Thank you!!


Need a help

Dear Support
I was installed the pH7-Social-Dating-CMS nsoftware on my website, but something is wrong. Can you help me?
Details of the problem can be found at the following links:
http://szimmath.com/aphrodite
http://szimmath.com/aphrodite/admin123/main/login
Regards
Robert Ven

@szimmath Hi !
First
https://github.com/pH7Software/pH7-Social-Dating-CMS/wiki/How-to-ask-for-help
Configure the base folder for your installation.

This is not how you ask for help. We need a full description of the issue, full server setup specs and explain the steps you have already taken before you came here.

@Undefined-Variables The links I tried did work but showed either a gZip/CSS Minifier problem or something alike.

Oh okay. I never trust links so always open them using Tor so might be due to that.
@szimmath This issue can normally be solved by disabling the gzip cache by signing in to admin then go tools->cache settings and disable it there. If you cant get to this page via a browser open
_protected/app/configs/config.ini

and change the 1's to 0's on the cache settings. If this doesn't solve your issue please supply us with the requested info explained on the page that @polynamaude gave you the link to above.

@Undefined-Variables It makes non-sense what you say about trusting links...
Your browser shall be secured by itself... The only thing that you do by running thru the Tor options is hiding your IP and possibly disabling many things...

JavaScript is disabled so no JavaScript on the page can run. I get what your saying about anything that runs in the browser will stay in the browser but I have seen stranger things happen. Plus hidding your IP is never a bad thing IMO.

@Undefined-Variables You see things that you don't understand and presume at wrong that they are harmful... Take it easy

No we are working openly on a platform that is used by developers and people who are super smart and are on the cutting edge of  computers, a subject still in its infancy and many things that are created and discovered everyday. Where there is good there is also bad and smart people are defined by good or bad they are in both groups.
This in mind it makes sense to take every precaution you possibly can, Clicking a link exposes a lot of information to the server the page is hosted on not only your IP address.


LFS-715: Display the questionnaire description when available

Includes 3/4 subtasks:

LFS-716: Questionnaire selector displays questionnaire description
LFS-718: Questionnaires table displays questionnaire description
LFS-719: Questionnaire description can be viewed and updated in the questionnaire wizard

LFS-717 (Subject chart displays questionnaire description) should be implemented together with or after LFS-698 (Restructure the patient chart).
To test, first use the questionnaire wizard to add descriptions to any existing questionnaires, then check it gets displayed in the questionnaire table and questionnaire selector.

3 out of 4 requested changes are done. The missing one is On the Subject chart: under the Questionnaire Titles (after LFS-698: Restructure the patient chartIN PROGRESS) , using subtitle typography


[types] Add missing overload for set

The set function can also take an object of coordinates to modify, as described in the README: https://github.com/color-js/color.js#manipulating-colors
This PR changes the types to reflect this.
The reason that the definition on the Color class had to be changed is because of how TypeScript handles overloaded functions in other types (i.e. ToColorPrototype). Without this, only the last signature of the set function would be present on Color.

(Force pushed because of accidental formatting changes)


Reorder menu in service create modal

Before to the left; after to the right:

retest this please

run integration tests

run integration tests 💃

run integration tests

Looks good, works great! One minor nit, but nothing that stops this PR

@philipnrmn I pushed a commit, hope you don't mind! :ship:


Broken link to documentation

When I click on the documentation link in the README (or the main site), I get:

Thanks! API is back up. Just to give you a heads up, we are working on releasing 0.5.0 with revamped simplified docs and much simpler API. You can follow this on the milestone page


Distance request - Wrong track on map for request made offline

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.48-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/cases/view/2730099
Issue reported by: Applause - Internal Team
Action Performed:

Go Offline
Make Distance request in any chat
Place Start/Finish waypoints like 'Los Angeles, USA' and 'Las Vegas, USA' > complete request
Go online
Open report details

Expected Result:
Expect to see track on map from Start to Finish for selected waypoints
Actual Result:
Track on map is not correspond to Start and Finish waypoints
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/94ec7fad-a810-452a-8c99-e6ea44c65a6b

View all open jobs on GitHub

We think that this bug might be related to #wave5
CC @dylanexpensify

@sakluger  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

I think this should be internal because the backend has a problem with differentiated locations with the same name.
For Example, The address for Las Vegas, USA needs to be Las Vegas, CA, USA , So it does not respond with a different location.

Las Vegas, USA should be Las Vegas, NV, USA, and Los Angeles, USA should be Los Angeles, CA, USA, so the backend does not respond with a different location.
Ipresume this issue should be fixed internally because the backend has a problem with differentiated locations with the same name.

Las Vegas, USA should be Las Vegas, NV, USA, and Los Angeles, USA should be Los Angeles, CA, USA, so the backend does not respond with a different location.
I presume this issue should be fixed internally because the backend has a problem with differentiated locations with the same name.

I agree that this is a bug. Both the original bug description, plus the detail added here are unexpected behaviors.

I'm putting up a fix

The PR is deployed to prod and no contributors reviewed the PR. We're good to close this one.


docker network inspect <network> does not contains containers

Hi I am having trouble listing the containers of a network. Here are my docker-engine info (on the manager) and after the steps to follow to reproduce.
docker version
Client:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8eab29e
 Built:        Thu Jul 28 22:00:36 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8eab29e
 Built:        Thu Jul 28 22:00:36 2016
 OS/Arch:      linux/amd64

docker info
Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 8
Server Version: 1.12.0
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 66
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: overlay bridge host null
Swarm: active
 NodeID: 1rd3z1fvgils8ta332bmosc53
 Is Manager: true
 ClusterID: 0krcc9l12q1fpqfysca80icwa
 Managers: 1
 Nodes: 3
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot interval: 10000
  Heartbeat tick: 1
  Election tick: 3
 Dispatcher:
  Heartbeat period: 5 seconds
 CA configuration:
  Expiry duration: 3 months
 Node Address: <IP_ADDRESS>
Runtimes: runc
Default Runtime: runc
Security Options: apparmor
Kernel Version: 3.19.0-66-generic
Operating System: Ubuntu 14.04.5 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 487.5 MiB
Name: sw-manager-0
ID: IMQS:2RE5:3VUU:XD5B:YHB2:7XHX:TWNL:S633:CLEZ:BWTL:2RUH:MXJ2
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Insecure Registries:
 <IP_ADDRESS>/8

First I created a manager and two nodes with the following steps
docker swarm init --advertise-addr eth0

then I executed this on both of my nodes
docker swarm join \
    --token SWMTKN-1-18l5iyraef9z743ub9du0gs21vst94m9aza7utavscam8i4hqj-0nv5zvrnkvrdyxcxmwb2vkpvv \
    <IP_ADDRESS>:2377

when I do docker node ls on the manager I got the expected output
ID                           HOSTNAME           STATUS  AVAILABILITY  MANAGER STATUS
1rd3z1fvgils8ta332bmosc53 *  sw-manager-0       Ready   Active        Leader
3y27h5s9c82zt028hytc9aslj    sw-node-web-0      Ready   Active
b6e139kamrvo89h3kqoz52vgn    sw-node-backend-0  Ready   Active

I also created a test network
docker network create -d overlay test_network

which has the id 4j5wtsm4za7gw40cdf0re43di
then I create a first service :
docker service create --name whoami -p 80:8000 --network test_network jwilder/whoami
which has the id 3kv2zk8g0dmd0fyxyfxdw36sq
In this case the container has been attached to one of the worker (not the manager)
then if run on the manager : docker network inspect test_network
I got the following result:
[
    {
        "Name": "test_network",
        "Id": "4j5wtsm4za7gw40cdf0re43di",
        "Scope": "swarm",
        "Driver": "overlay",
        "EnableIPv6": false,
        "IPAM": {
            "Driver": "default",
            "Options": null,
            "Config": []
        },
        "Internal": false,
        "Containers": null,
        "Options": {
            "com.docker.network.driver.overlay.vxlanid_list": "257"
        },
        "Labels": null
    }
]

It is strange that the containers is empty
If i run the same command on the worker where the service is running i got the following
[
    {
        "Name": "test_network",
        "Id": "4j5wtsm4za7gw40cdf0re43di",
        "Scope": "swarm",
        "Driver": "overlay",
        "EnableIPv6": false,
        "IPAM": {
            "Driver": "default",
            "Options": null,
            "Config": [
                {
                    "Subnet": "<IP_ADDRESS>/24",
                    "Gateway": "<IP_ADDRESS>"
                }
            ]
        },
        "Internal": false,
        "Containers": {
            "d0b4a9aa7fe79c4ef30520b2ef63d018c9da764ffde80848040598dd2d8a5e5a": {
                "Name": "whoami.1.djp6m3izdxubho02qwbafynyc",
                "EndpointID": "cad773b7986ab912fcda400f897081928e2be69d77c2016270f26da6273f2f66",
                "MacAddress": "02:42:0a:00:00:03",
                "IPv4Address": "<IP_ADDRESS>/24",
                "IPv6Address": ""
            }
        },
        "Options": {
            "com.docker.network.driver.overlay.vxlanid_list": "257"
        },
        "Labels": {}
    }
]

This is a demonstration, in my real use case I use jwilder/nginx-proxy and I guess because the containers fields are empty it cannot detect the service across nodes.
So my question is : is it normal that the containers field does not contain containers regarding the fact that it is an overlay network I should see the container right ?

So do I understand, you're trying to use jwilder/nginx-proxy to proxy several web services, and the way that works is by listing containers on a network and then automagically adding them to an nginx reverse proxy? And it doesn't work because when you try to list containers on a network, they're not listed properly?
I'm not sure if Containers being empty is a bug or not, but I think that jwilder/nginx-proxy is going to be incompatible with swarm-mode architecture. In swarm mode, we address things by the VIP of their service, not the IP of their container. There's a layer of abstraction there.
Does this help any?

Yes this exactly the case. There is an issue about this on the jwilder/nginx-proxy on https://github.com/jwilder/nginx-proxy/issues/520. but I am still wondering is

Yes, docker/docker#24186. Containers being empty is a known issue. Does that answer your question?

Thanks, indeed it does, did not found this issue before.
I will follow the upcoming update about this.

@dperny I was planning on doing the exact same thing using jwiler/nginx-proxy so I don't have to manually manage a conf file every time we deploy a microservice.
If this solutions is "wrong", what would you suggest the "right" solution to be?  I personally don't care what  tech we use for the reverse proxy as long as it's automatic.

I am having the same issue. Two containers assigned to same created overlay network, docker network inspect  shows null for containers.


[gradle] signing and jenkins cleanup

Gradle signing wasnt actually working. Have added v1-v3. I was under the impression you needed v2+ to submit these days, but either way, it doesnt change anything for your playstore submission stuff.
Added the zipalign to the gradle build file.
Removed the apksign script, as its no longer needed. Ive removed all the variables pertaining to it from jenkins.
Ran this test job https://jenkins.kodi.tv/view/Android/job/Kore/4871/ to confirm signing. Below snippet to verify
static:Kore administrator$ jarsigner -verify -verbose -certs kore-20231119-1c03adc-gradle_signing.apk
<snipped for brevity>
- Signed by "CN=XBMC Foundation, OU=Android platform, O=XBMC Foundation, L=US, ST=US, C=us"
    Digest algorithm: SHA-256
    Signature algorithm: SHA256withRSA, 2048-bit key

jar verified.

Warning:
This jar contains entries whose certificate chain is invalid. Reason: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
This jar contains entries whose signer certificate is self-signed.
This jar contains signatures that do not include a timestamp. Without a timestamp, users may not be able to validate this jar after any of the signer certificates expire (as early as 2042-03-13).
POSIX file permission and/or symlink attributes detected. These attributes are ignored when signing and are not protected by the signature.

The signer certificate will expire on 2042-03-13.

Feel free to just commit your own variant of this, just wanted to provide the info to save you hunting around.

Thanks.


cli: ensure support for every sequencer action type & abci query

Currently astria-cli doesn't support all of our action types which is a hindrance to running manual and adding smoke tests. We want to support all action types, and abci query such that they are all easily tested through our domain types logic.
┆Issue Number: ENG-804

This issue is stale because it has been open 45 days with no activity. Remove stale label or this issue
be closed in 7 days.

This issue was closed because it was stale


Update to use template files

Pull Request (PR) description

Update appveyor.yml to use the default template.
Added default template files .gitattributes, and .vscode settings.

This Pull Request (PR) fixes the following issues

Task list

[x] Added an entry under the Unreleased section of the change log in the README.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[ ] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #24 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #24   +/-   ##
=======================================
  Coverage   26.08%   26.08%
=======================================
  Files           3        3
  Lines          92       92
=======================================
  Hits           24       24
  Misses         68       68

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 8e65927...99fdbe1. Read the comment docs.


Allow XOR on byte sequences or fix bug in XORed \x representation in strings

I'd like to search for XORed byte sequences and couldn't find a nice way to do that.
      /* not possible */
      $x0 = { E9 6C 0A 01 01 C4 56 8C ED 52 52 84 66 F9 01 8C } xor
      /* not working */
      $x1 = "\xE9\x6C\x0A\x01\x01\xC4\x56\x8C\xED\x52\x52\x84\x66\xF9\x01\x8C" xor

The $x1 doesn't work, although I believe that it should. It could be a bug or something that I miss.
Sample to test it on:
https://www.virustotal.com/gui/file/601a02b81e3bd134c2cf681ac03d696b446e10bf267b11b91517db1b233fec74/content/hex

I've used YARA 4.1.3 for my tests.

I just did some testing and this does indeed seem like a bug, but based upon my testing it only happens if the entire string is escaped. I'll see if I can dig a bit deeper tonight...

Thanks for the review.
Based on your testing, I tried to find an XORed version of that byte sequence that contains many ASCII characters.
$x1 = "W\xD9wnn2\xc3\xF9\x5B\xBF\xBF\xF1\xD3gn\xF9" xor

I tried shorted version of that string and it also fails.
$x2 = "W\xD9wnn2" xor

$x2 = "\xc3\xF9\x5B\xBF\xBF\xF1\xD3gn\xF9" xor

$x2 = "wnn2\xc3" xor

The rule that I tried to improve was one from a report of a German authority

https://www.verfassungsschutz.de/SharedDocs/publikationen/DE/cyberabwehr/2022-01-bfv-cyber-brief.pdf

Upon closer inspection I don't think is a bug.
The bytes from the file are EA 6D 0B 02 02 C5 57 8D EE 53 53 85 67 FA 02 8D.
The string you are searching for is E9 6C 0A 01 01 C4 56 8C ED 52 52 84 66 F9 01 8C.
If we xor the first byte from the file (0xEA) with the first byte from your search string (0xE9) we get a key byte of 0x03. Repeating for the next byte (0x6D and 0x6C) we get a key byte of 0x01, so YARA is not going to find it because YARA is only able to search for a single byte key across your entire string, and the key byte is changing in your case.
Also, I haven't read the report but the description of that rule says it is looking for a sequence of bytes generated by some ADD/SUB encoding routine, and they generated all the possible values from it. This is not an xor encoded blob that they are searching for.

Oh, my. I should have XORed it myself. Since every byte is just one decimal value lower than the other I assumed that this would be a simple 1 byte XOR without confirming it. Sorry for the confusion.


Add crash case (see #587) for fixed bug to crashers_fixed/

Sorry, this fails on both OS X and Linux.  Reverting.
Could you check if it is non-deterministic?  I ran the test 20 times, and it fails every time.

@gribozavr Confirmed, it fails for me as well with swift-ide-test compiled from current master and run under Linux. I tested both with and without assertions enabled.
I ran the test 1000 times and it failed every single time, so it appears to be a determinic crash. I'll re-add it as a crash case in another PR.


NameError: uninitialized constant Sinatra::Sprockets::Manifest when precompiling assets

Hi, I had the same error with precompiling assets like this one described in the issue
https://github.com/kalasjocke/sinatra-asset-pipeline/pull/62

NameError: uninitialized constant Sinatra::Sprockets::Manifest

I found this changes https://github.com/kalasjocke/sinatra-asset-pipeline/commit/364069d1b4adc5261183b699e141ecfffe7c4460#diff-88eb5ff86459193ecce7995b8773fcb9R24 deletes :: in ::Sprockets::Manifest
I don't really know is it real issue, but I got one and adding this :: again fixed it for me:
https://github.com/designcitypl/sinatra-asset-pipeline/commit/d63604251a72a3c4125ef3f3b47fade6a6f412e1

That's strange. I'm not sure why it happens for you, I would love if you could spend some more time digging into this and if you find anything submit a pull request. I can't reproduce it locally. Closing this as a wont fix since it seems to be working for most people using this gem.


Add possibility see which crashes are regressions

We can mark crashes as closed in one version of an app. Then fix is published in next version but bug still is occurring. There is no way to see in app center that this is a regression, you have to manually search for crash reports.
In Hockeyapp crashes were linked between app versions and you could easly checked on which app version bug was fixed and on which version it started to occur again.
It would be nice to have these feature in app center as well.

Not done


Support for Windows

First of all, thanks for your great dependency!
I ran some example on Linux OS.
Is it possible to run dependency on Windows OS?
As I know, Chrome 59 beta is released on Windows.

Hi @starwarsjk ,
thanks for your comment. I'll check that until Saturday.
Have a nice day,
Lenny

The main problem is that the latest Chromedriver 2.29 supports Chrome up to version 58 but the headless mode is available in version 59+ and there is currently no windows version of the dev sources.

Thanks for checking. I am waiting now for support from https://sites.google.com/a/chromium.org/chromedriver/downloads

Hi @ldaume
there is already a new version of ChromeDriver with support for Chrome v58-60!
Can you try to run dependency on Windows OS?
Thank you in advance!