make full utilisation of GPU during inference

Related to #213 #863 #932
Thanks for sharing your work, when running the inference script for video which looks like this
from mmdet.apis import init_detector, inference_detector, show_result
import time
config_file = 'configs/faster_rcnn_r50_fpn_1x.py'
checkpoint_file = 'checkpoints/faster_rcnn_r50_fpn_1x_20181010-3d1b3351.pth'

# build the model from a config file and a checkpoint file
# model = init_detector(config_file, checkpoint_file, device='cuda:0')
model = init_detector(config_file, checkpoint_file, device='cuda:0')

# test a single image and show the results
import cv2
import numpy as np

# Create a VideoCapture object and read from input file
cap = cv2.VideoCapture('video.mp4')

# Check if camera opened successfully
if (cap.isOpened()== False):
  print("Error opening video  file")

# Read until video is completed
count =0
start = time.time()
while(cap.isOpened()):

  # Capture frame-by-frame
  ret, frame = cap.read()
  if ret == True:
    # img = 'test.jpg'  # or img = mmcv.imread(img), which will only load it once
    result = inference_detector(model, frame)
    show_result(frame, result, model.CLASSES, out_file='video_prediction/result{}.jpg'.format(count))
    count+=1
    print('Count ',count, 'Time ',time.time() - start)
end = time.time() - start

print('_______', end)
    # test a list of images and write the results to image files
    # imgs = ['test1.jpg', 'test2.jpg']
    # for i, result in enumerate(inference_detector(model, imgs)):
        # show_result(imgs[i], result, model.CLASSES, out_file='result_{}.jpg'.format(i))

The GPU utilisation is very low

Is there a way to use all the memory, if someone can point me in a direction ill do this myself
Thanks in advance.

I don't think this code was designed to be a video processing pipeline. To
properly handle video you need the IO (both read and write) to be done in
parallel to processing as well and NMS.
to check the actual GPU utilization, try changing your code to:
frame = mmcv.imread('test.jpg') #don't do and read in the loop

while True:

  result = inference_detector(model, frame)

#  show_result(frame, result, model.CLASSES,
out_file='video_prediction/result{}.jpg'.format(count))
  count+=1
  print('Count ',count, 'Time ',time.time() - start)

On Tue, Jul 16, 2019 at 5:34 AM Ryan Dsouza<EMAIL_ADDRESS>wrote:

Related to #213 https://github.com/open-mmlab/mmdetection/issues/213
#863 https://github.com/open-mmlab/mmdetection/issues/863 #932
https://github.com/open-mmlab/mmdetection/issues/932
Thanks for sharing your work, when running the inference script for video
which looks like this
from mmdet.apis import init_detector, inference_detector, show_result
import time
config_file = 'configs/faster_rcnn_r50_fpn_1x.py'
checkpoint_file =
'checkpoints/faster_rcnn_r50_fpn_1x_20181010-3d1b3351.pth'
build the model from a config file and a checkpoint file model =
init_detector(config_file, checkpoint_file, device='cuda:0')
model = init_detector(config_file, checkpoint_file, device='cuda:0')
test a single image and show the results
import cv2
import numpy as np
Create a VideoCapture object and read from input file
cap = cv2.VideoCapture('video.mp4')
Check if camera opened successfully
if (cap.isOpened()== False):
print("Error opening video file")
Read until video is completed
count =0
start = time.time()
while(cap.isOpened()):
Capture frame-by-frame
ret, frame = cap.read()
if ret == True:
img = 'test.jpg'  # or img = mmcv.imread(img), which will only load it once
result = inference_detector(model, frame)
show_result(frame, result, model.CLASSES, out_file='video_prediction/result{}.jpg'.format(count))
count+=1
print('Count ',count, 'Time ',time.time() - start)
end = time.time() - start
print('______
', end) # test a list of images and write the results to image files #
imgs = ['test1.jpg', 'test2.jpg'] # for i, result in
enumerate(inference_detector(model, imgs)): # show_result(imgs[i], result,
model.CLASSES, out_file='result{}.jpg'.format(i))
The GPU utilisation is very low
[image: usage]
https://user-images.githubusercontent.com/25679895/61236073-c1ade000-a754-11e9-96c7-993d53abc7ab.png
Is there a way to use all the memory, if someone can point me in a
direction ill do this myself
Thanks in advance.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/open-mmlab/mmdetection/issues/1003?email_source=notifications&email_token=AC7IWCYM3PO4P427FMAB6K3P7SYKBA5CNFSM4IDZDCS2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G7I5IFA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC7IWCYYZGSFIJJEEJYXMA3P7SYKBANCNFSM4IDZDCSQ
.

@mosheliv  Thanks for the suggestion let me check this


Cannot connect to MySQL 5.x instance, CLIENT_PLUGIN_AUTH is required

Description
As of DBeaver 23.2.5, I can no longer connect to MySQL 5.x instances. I get a connection error dialog box, with the message "CLIENT_PLUGIN_AUTH is required".
I believe this is due to the following change, from the release notes:

Databases:
MySQL:
Driver version was upgraded to 8.2
Legacy MySQL 5.x driver was deprecated and replaced with MySQL 8 driver

Is there a workaround for this, or must I stay on DBeaver 23.2.4 or earlier?
DBeaver Version
Community Edition 23.2.5
Operating System
Linux
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Same here. After the last update, I lost connectivity with many hosts running 5.x MySQL versions. I think that removing MySQL5 connector-j was not a good idea since connector-j 8 is not compatible with versións of MySQL minor than 5.6.

Another confirmed case here.

MySQL 5 driver will return in the next version.
EA version will be published Nov 22.

Thank you, meanwhile, I have downloaded connector-j 5 directly from MySQL created a new driver from Driver Manager, and recreated some connections that I urgently needed.
You can download this from: https://downloads.mysql.com/archives/c-j/ and select Product Version 5.1.49

The current EA access version didn't seem to fix this issue for me. Is it still being worked on?

Hello @AllisonF-23 Could you please provide more info about your case?

Sure thing! I use dBeaver for a wide range of SQL servers of various ages, and when I updated to dBeaver 23.2.5 last week, all of the older servers I connect to began giving me an error that says "CLIENT_PLUGIN_AUTH is required". The servers I can no longer connect to are on MySQL 5.1.73 . I tried reverting to dBeaver version 23.2.4 and had the same problem, and then lastly I have updated to the latest early access version I see which is 23.3.0 and the behavior still exists. I'm utilizing the Debian version of dBeaver as well if that is relevant. Is there any other information I can provide?

Hello @AllisonF-23 Could you please provide more info about your case?

Sure thing! I use dBeaver for a wide range of SQL servers of various ages, and when I updated to dBeaver 23.2.5 last week, all of the older servers I connect to began giving me an error that says "CLIENT_PLUGIN_AUTH is required". The servers I can no longer connect to are on MySQL 5.1.73 . I tried reverting to dBeaver version 23.2.4 and had the same problem, and then lastly I have updated to the latest early access version I see which is 23.3.0 and the behavior still exists. I'm utilizing the Debian version of dBeaver as well if that is relevant. Is there any other information I can provide?

So an update to this, I managed to get this working. What I ended up doing was exporting my dbeaver settings, nuking my install of dbeaver completely from my PC, installing 23.3.0, importing my settings. When I did this, I was prompted to download the various drivers for each server and it now works as expected.

We updated EA version, could you please take a look one more time?
https://dbeaver.io/files/ea/

@AllisonF-23
Thanks for the update!

Used to work, broken since last update...

I can confirm that DBeaver 23.3.0 works fine again with connections to MySQL 5.x instances.

Even so, it's necessary to update the driver properties for the MySQL 5 connections since they were marked to use the MySQL 8 driver in the previous update.

Note that you only need to do this if you upgraded to one of the affected versions in the mean time (>= 23.2.5, <23.3.0). If you go from 23.2.4 to 23.3.0 then everything will continue to work with no changes.
Thanks for the fix.


[Bug]: quoted attributes from an imported CSV aren't respected

What happened?
When importing a CSV, every comma is treated like a data separator even when quoted / escaped.
For example, with the following CSV:
File, col2
"object 1, with note", object 2

I would expect
---
File: object 1, with note
col2: object 2
---

instead I get
---
File: object 1
col2: with note
undefined: object 2
---

What platform were you using?
Desktop
Version of the plugin
3.2.0
Relevant log output
No response
Are you using the latest version of the plugin in your Obsidian vault?

[X] I have verified that I am on the latest version

Are you check if there is a similar issue?

[X] I have verified that there are not similar issues

Hello @Obsidian-user, Thanks for the report
I will check this issue

I will change the actual csv libraryper a custom solution we can parse better.
I am working on a plugin API for another Input/Output solutions and this tutorial could be an aprox https://refine.dev/blog/how-to-import-csv/

This is the first step of a huge refactor Commit 41480453
It will fix this issue


relevant is not working properly for the state variables which are created in MultieStep components

Describe the bug
In the below code sandbox link my use case is, if the isLoggedIn value is true then the second step shouldn't be rendered. For that I am returning the isLoggedIn value in the relevant function. But here even though the variable isLoggedIn value is changed to true the second step is still rendering. I have tried with relevancedeps also which also didn't give me any change.
To Reproduce
https://codesandbox.io/s/multie-step-practice-forked-1zbtd?file=/src/App.js

Can you not open another issue for the same issue please. You already have an issue for this, lets keep conversation over there.


rtl support

Hi guys,
I needed a swipe component for a small calendar application I'm building and this dragend plugin works perfectly for my needs. I implemented it successfully for my LTR version of my application, but unfortunately I have to support RTL as well.
When I implement the same code for my RTL version, the swipeable-space becomes empty, as if non of the pages are visible.
Anyone else found any issues implementing it with right-to-left languages? Anyone got it to play nicely?
Thanks,
Joseph.

Just start at one of the right hand pages.


[CI] Update the Component generator workflow.

Current Behavior

The Components generator file https://github.com/meshery/meshery-linkerd/blob/master/.github/workflows/update-oam-defs.yml using bash which isn't in uniformity with Other adapters.
Desired Behavior

Edit the workflow to maintain uniformity with other adapters
Implementation

Acceptance Tests

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

++ @Revolyssup,

Excellent, @karnatisrinivas


Visual improvements and issues.

A catch-all task for all little visual bugs/improvements.
### Tasks
- [ ] Add dark background when CB is opened.
- [ ] Align (+) button in Vector widget with closing bracket
- [ ] https://github.com/enso-org/enso/issues/8995
- [ ] Remove "halo" from output enabled indicator.

Halo on output enabled indicator:


Fix the AssertionError when calling masker in coco_eval.py

After applying the changes in #207.
I got the same AssertionError as described in #185 when I executing
python tools/test_net.py --config-file "configs/caffe2/e2e_mask_rcnn_R_50_FPN_1x_caffe2.yaml" TEST.IMS_PER_BATCH 2

It could be fixed by doing the same way as #187 .

The performance seems to become bad due to pass masks to Masker, so I decide to drop the PR currently.

So the code is not working for you, is that right?

I got the AssertionError, so I made some changes, and it worked.
But later, I found the performance drops when evaluating the results, so I decided to check it again and cancel the PR.
I think the below command could reproduce the error,
python tools/test_net.py --config-file "configs/caffe2/e2e_mask_rcnn_R_50_FPN_1x_caffe2.yaml" TEST.IMS_PER_BATCH 2

Also, can you tell me what is the AssertionError that you got, just to double check?

Hi @fmassa
Thanks for your reply.
Here is the AssertionError.
2018-11-26 19:46:20,913 maskrcnn_benchmark.inference INFO: Total inference time: 0:01:23.692130 (0.13390740776062013 s / img per device, on 8 devices)
2018-11-26 19:46:23,992 maskrcnn_benchmark.inference INFO: Preparing results for COCO format
2018-11-26 19:46:23,992 maskrcnn_benchmark.inference INFO: Preparing bbox results
2018-11-26 19:46:24,971 maskrcnn_benchmark.inference INFO: Preparing segm results
0it [00:00, ?it/s]
Traceback (most recent call last):
  File "tools/test_net.py", line 94, in <module>
    main()
  File "tools/test_net.py", line 88, in main
    output_folder=output_folder,
  File "/home/henrywang/github/maskrcnn-benchmark/maskrcnn_benchmark/engine/inference.py", line 104, in inference
    **extra_args)
  File "/home/henrywang/github/maskrcnn-benchmark/maskrcnn_benchmark/data/datasets/evaluation/__init__.py", line 22, in evaluate
    return coco_evaluation(**args)
  File "/home/henrywang/github/maskrcnn-benchmark/maskrcnn_benchmark/data/datasets/evaluation/coco/__init__.py", line 20, in coco_evaluation
    expected_results_sigma_tol=expected_results_sigma_tol,
  File "/home/henrywang/github/maskrcnn-benchmark/maskrcnn_benchmark/data/datasets/evaluation/coco/coco_eval.py", line 47, in do_coco_evaluation
    coco_results["segm"] = prepare_for_coco_segmentation(predictions, dataset)
  File "/home/henrywang/github/maskrcnn-benchmark/maskrcnn_benchmark/data/datasets/evaluation/coco/coco_eval.py", line 119, in prepare_for_coco_segmentation
    masks = masker(masks, prediction)
  File "/home/henrywang/github/maskrcnn-benchmark/maskrcnn_benchmark/modeling/roi_heads/mask_head/inference.py", line 185, in __call__
    assert len(boxes) == len(masks), "Masks and boxes should have the same length."
AssertionError: Masks and boxes should have the same length.

After checking it again, I decide to reopen the PR and wait for your review.
The performance is bad because I was testing on some large images. (2048*1024)
and it will make the Masker spending more time.
I just found there are some discussions about this before.

Oh, I just realized where the problem is actually: when I rebased the Pascal commit, I forgot to add this part of the code.
Can you please update your PR with the aforementioned fix, and I'll merge this?
Thanks!


Feature/#498

Removing inner classes to reduce the complexity of the problem.
Now the design is easier to understand.

Before merge t o #443, I will test with a EV3 Brick tomorrow.
So much changes.

NOOOOOO

Hi,
the modeswitch was intentionally moved to the beginning of the fetchSample method. This is because otherwise there cannot be multiple active sensor modes at the same time (the possible bug I wrote about above). Aswin wrote about this in his comment here: https://github.com/ev3dev-lang-java/ev3dev-lang-java/issues/121#issuecomment-205161182

Another approach is to have a different class for each mode that a sensor supports. This is the solution leJOS adapted. The RedMode class always measures reflected light, the ambientMode class always measures ambient light. This solution does not have the drawback of the previous solution. One can even have the two classes active at the same time.

Also, if we look in the leJOS source code, taking EV3ColorSensor#ModeProvider as an example:
        @Override
        public void fetchSample(float[] sample, int offset)
        {
            switchMode(mode, SWITCH_DELAY);
            if (sampleSize == 1)
                sample[offset] = (float) (port.getByte() & 0xff)/100.0f;
            else
            {
                port.getShorts(raw, 0, raw.length);
                for(int i = 0; i < sampleSize; i++)
                    sample[offset+i] = (float)raw[i]/1020.0f;
            }
        }

We can see that switchMode is called in the fetchSample function.
If necessary, we can optimize switchMode() implementation to cache the configured mode again.
Regards

Hi @JakubVanek,
I see your point of view, but in what scenarios do you think that it is possible to have multiple active sensor modes at the same time? Is it a edge case?

[continuation] There is though at least one remaining problem with the current solution (switching in fetchSample). If two calls to fetchSample() on the same sensor are occuring in two threads in parallel, it can happen that the sensor mode is switched during of the data fetch. This would result in incorrect reading. The solution would be to add a mutex to BaseSensor. However I think reading from different modes from the sensor in parallel is already a bad idea, so we do not have to do guard against this.
[reply] OK, you got me on this. :smile: I do not have a case where this would be necessary and could not be solved using switchMode. I can imagine however the user would not have an instance of EV3ColorSensor (for example) at all and would just store instances of sensor modes (I found one such code here, but without using multiple modes). If the sensor was in red reflection mode and the user wanted to know a color of a brick in front of the robot, he'd call fetchSample() of the correct mode in advance to eliminate switch delay (if necessary) and then simply fetch the data. Also, there is a possible advantage that fetchSample() propagates through filters, so you only need to call fetchSample() on the top-level SampleProvider, without needing to call into EV3ColorSensor.

IMHO it also helps when we implement "getters" in sensor-specific classes through GenericMode (or measurement functions in general). You do not have to repeat switchMode everywhere, it is automatically assumed.

I don't say no, give me the opportunity to reply you with arguments. :)

Ok, I'm probably triggered :smile: . I'm listening now.

Hmmm, the problem is with performance reduction due to string manipulation & repeated reading of the sensor mode from the kernel. I'll optimize it tomorrow.

Good morning,
the method fetchSample is method to read from the physical device but previously you change the mode, in this moment, you execute switchMode internally. Normally the developer switch the mode and later execute fetchSample many times.
This is the reason, I think that execute switchSample when we getMode is the right place. Indeed a sensor has a unique responsability. In EV3 appear sensors with multiple modes but not before. One LIDAR returns samples, a Camera returns frames an Ultrasonic Sensor returns distances, single responsability. It is true that one EV3 Ultrasonic sensor is able to put in listen mode or distance mode but the developer or use the sensor to listen or use the sensor to receive distances.
I agree with you about the idea to not have multiple threads to for the same sensor, always Single responsability.
https://en.wikipedia.org/wiki/Single_responsibility_principle
This video show multiple threads reading from different sensors:
https://www.youtube.com/watch?v=3hXTeK8lFMc
I think that we should maintain the commit:
https://github.com/ev3dev-lang-java/ev3dev-lang-java/commit/dbdec953fcf1596049ca530637893bac51e3fcc1
This simple example show the idea:
import lejos.hardware.port.SensorPort;
import lejos.robotics.SampleProvider;
import lejos.utility.Delay;

public class USSensorDemo {

	private static EV3UltrasonicSensor us1 = new EV3UltrasonicSensor(SensorPort.S1);

	public static void main(String[] args) {

		final SampleProvider sp = us1.getDistanceMode();
		int distanceValue = 0;

        final int iteration_threshold = 100;
        for(int i = 0; i <= iteration_threshold; i++) {

        	float [] sample = new float[sp.sampleSize()];
            sp.fetchSample(sample, 0);
            distanceValue = (int)sample[0];

			System.out.println("Iteration: " + i + ", Distance: " + distanceValue);

			Delay.msDelay(500);
        }

	}

}

In the other hand, switchMode is executed in the right place so we don`t need more optimizations at the moment. Today, I will do a full regression with all EV3 Sensors.
Can you put the commit again? :)

Hi,
I see your point and partly agree with it. In a good program the sensor mode switch should happen before even calling fetchSample() Though, theoretically, the function would be better called switchTo*Mode() rather than get*Mode().
If we move switchMode back to get*Mode(), I think it would be good to do at least one of the following things, because we are potentially breaking code that would work on leJOS:

Explicitly declare in documentation that either:

A: reading from GenericMode when other switchMode() was called from anywhere is an undefined behavior (readings might not make sense), or
B: reading from GenericMode when the sensor was switched to a different mode is an undefined behavior. I.e. you are allowed to call the original GenericMode for mode A after you switch to a different mode B and then back to mode A.

Similarly declare that the gyro reset in the current form switches modes and potentially causes both variants of undefined behavior.
(as an optional extension to the first point) Add one explicit check to GenericMode#fetchSample() for one of the following circumstances, and if it is not fulfilled, throw some RuntimeException with "invalidated sensor mode" message or alike.

A: check whether switchMode() was called from anywhere from the point in time when we called get*Mode(), by using a counter, or
B: check whether the mode that we last switchMode()d to is the same as the one the GenericMode is associated with. This would be probably done the same way as the optimization for switchMode-in-fetchSample I wanted to do, just that we wouldn't switch the mode, but just throw an exception.

But, if we go opt for both mitigations and variant B, we are essentially doing what we would be doing with switchMode-in-fetchSample, but the result would be that the mode won't be switched, rather an exception would be thrown. This is probably acceptable too, as this would eliminate the switch delay from first fetchSample() after a mode switch.
Regarding the commit itself: what is COL-RESET mode? Does it exist and work? It is not in ev3dev sensor database, though it is possible that the kernel will interpret "RESET" as a command to reinitialize the sensor. But, then the floodlight would eventually turn red, if I understand sensor mode switching correctly.
Regards

Hmmm, it scares me that now the code seems a little bit more complex. Maybe I'm tired, but it seems to me the mode cache is making it more convoluted.

I am in a train with no Internet. In few hours we continue the conversation. Please, try not complicated the solution with the original commit is enough. I have to test the setModes behaviour only. Please wait few hours. Not code... wait cheers

We can revert my commits or alternatively rebase them out (~ erase them from history) later, when we decide on which path ve will follow. I don't think I'm going to add more things now.

Reviewing latest commits

git show dbdec95
git reset --hard dbdec95
git commit -m "reverting changes for Sensors"
git push -f

Executing full regression for EV3:
Actuators:

[x] SoundDemo
[x] LCDDrawIconsTest
[x] LCDDrawImagesTest
[x] LCDDrawLinesTest
[x] LCDDrawRectanglesAnimationTest
[x] LCDFontTest
[x] LCDTest
[x] LCDWriteTextTest
[x] LEDExample
[x] LargeMotorDemo
[x] LargeMotorDemo2
[x] LargeMotorStopModesExample
[x] MediumMotorDemo
[x] MediumMotorStopModesExample
[x] MotorEncoderTest
[x] MotorEventTest
[x] MultipleMotorsDemo
[x] NXTRegulatedMotorDemo
[x] RegulatedMotorRotateDemo
[x] RegulatedMotorRotateToDemo
[x] UnregulatedMotorDemo PENDING
[x] UnregulatedMotorDemo2 PENDING
[x] UnregulatedMutilpleMotorDemo PENDING

I continue the process to execute the regression tests at sensor level for EV3.

Sensors:

[ ] BatteryDemo
[ ] ButtonExample
[ ] ButtonExample2
[ ] ButtonListenersExample
[ ] ColorSensorDemo

Finished regression.


Repository avatars

Description
Back in June 2018, Gogs added repository avatars to it. (see https://github.com/gogs/gogs/pull/5221 for more details.)
Can we get this added to Gitea?
Screenshots
Shots from Gogs:

Well, gogs not included all of my changes.
Repo icons available only at:

repos lists
repo page

Right pannel missing repo and org icons.
I can try to adjust my previous work for gitea.

@sergey-dryabzhinsky Thanks. Please send your PRs and we will review that.

And @rapardington1 @sergey-dryabzhinsky, it seems this is duplicated with #694. Let's discuss further there and I will close this one.


Code improvement: Grep Search / Editor Improvements

Hey all, love the engine.  But, I've been having to use other software while using Godot to do things that i am accustomed to doing in other IDEs (IntelliJ, Eclipse, XCode, etc.)
One of the biggest ones has been "Find Occurences".  I've been using Grep to find all occurrences of a code or the uses of a method.  Being able to quickly find these sorts of relations in the editor would be fantastic, like right clicking a method/variable and "find all usages".  (I know Godot doesn't have right-click, but you get the idea.)
Another would be multi-pane editing.  Only being able to have one GDScript pane open at a time (instead of side-by-side) cuts down on some productivity.  I have been having to keep a Notepad++/TextMate open at the same time with multiple scripts to look at them side-by-side.
Additionally, the quick open (ctrl + shit + o) is great for Scenes, but it would be nice if it could search for file names of GDScripts as well.
Thanks for all you do, guys.  You're amazing.
For mods: tags to use on this post are probably  "feature proposal", "topic:gdscript", "topic:editor"

Please avoid opening a single issue for different topics, it makes it hard to discuss and fix the single points.
For an overview of all such proposed improvements, see #2982.

Additionally, the quick open (ctrl + shit + o) is great for Scenes, but it would be nice if it could search for file names of GDScripts as well.

This is implemented already, via "Quick Open Script" / Ctrl+Alt+O.

I didn't know about that command!  Good to know. :)
I'll add future notes to the thread you posted and close this one, thanks.


optaplanner on android doc

As I promised - short instructions how to use OptaPlanner on the Android platform. Also I send you an email.

Great work, thanks :)
I improved the text a bit with active wording ("should be added" => "add") and motivation of certain commands, see http://github.com/droolsjbpm/optaplanner/commit/0a17a2ce1
Also mentioned in release notes:
http://github.com/droolsjbpm/optaplanner-website/commit/265aa527f


fix(font-tokens): set scaling param to true for expressive-heading-05

Related Ticket(s)
Expressive type token value errors #5554
[Lead space - with search] React experimental, sub-headline should be $expressive-heading-05 #5545
Description
Pass in true to scaling parameter so the font size scales up/down according to breakpoint
Changelog
Changed

set true to expressive-heading-05 tokens

LGTM, does this need design approval too?

Yeah I'll add design just to make sure.

Unable to QA the result coz I cannot see Lead space with Search. Turning on feature flag...


Dark mode colors have low contrast

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
The new darkmode colors make text very hard to read due to bad contrast. Compare these screenshots:
Unmodified:

Text Color #eee:

Text Color #eee and Body Color #111 (bootstrap-darkmode):

I propose changing the $body-color-dark to something brighter like #eee for improved readability.
Reduced test cases

What operating system(s) are you seeing the problem on?
Windows, macOS, Android, iOS, Linux
What browser(s) are you seeing the problem on?
Chrome, Safari, Firefox, Microsoft Edge, Opera
What version of Bootstrap are you using?
v5.3.0-alpha3

FWIW, we've using the dark-mode (based on preference with JS) on docs.mesa3d.org, and we've gotten complaints from users about the lack of contrast in the dark-mode as well.

The color contrast in dark mode (#adb5bd to #1a1c20) is 8.26. I could see us going a little darker on the background and perhaps brighter on the foreground.

I see that the contrast works in theory, but as mentioned by @kusma and in my opinion, in practice it is still not readable for everyday text. Compare the comments here on GitHub, with a contrast ratio of 12.26:1.

Fixed by #38815.


StaticSite: fileOptions uploader does not mimic the S3 CLI and is limited

Issue 1
I think there is an issue with how the current fileOptions works in StaticSite  since exclude and include are setup as static properties (where includes always override excludes) there is no way to reproduce this command that the s3 sync CLI could…
aws s3 sync ./public/ "s3://${BUCKET_NAME}/" \
  --exclude "*" \
  --include "static/*" \
  --include "*.css" \
  --include "*.js" \
  --exclude "sw.js" \
  --cache-control 'public,max-age=31536000,immutable' --delete

Notice the second exclude comes after a bunch of includes and another exclude. Perhaps laying out these fileOptions would have been better this way…
{
  fileOptions: [
    {
      matching: [
        { exclude: '*' },
        { include: 'static/*' },
        { include: '*.css' },
        { include: '*.js' },
        { exclude: 'sw.js' },
      ],
      cacheControl: 'public,max-age=31536000,immutable',
    },
  ],
}

or perhaps even shorter something like
{
  fileOptions: [
    {
      matching: [ '!*', 'static/*', '*.css', '*.js', '!sw.js'],
      cacheControl: 'public,max-age=31536000,immutable',
    },
  ],
}

Need to check that it doesn’t conflict with other ! in standard rules… but it appears you have to use [!thingy] if you want to use negate.
Issue 2
It also seems like being able to use ONLY exclude is also missing and is a perfectly acceptable command with the CLI.
Property 'include' is missing in type '{ exclude: string[]; cacheControl: string; }' but required in type 'StaticSiteFileOption'
The following command would upload all files, except for the excluded, and not set any caching options on them.
aws s3 sync ./public/ "s3://${BUCKET_NAME}/" \
  --exclude "*.html" \
  --exclude "page-data/*.json" \
  --exclude "*.js" \
  --exclude "*.css" \
  --exclude "static/*" \
  --exclude "robot.txt" \
  --exclude "sitemap.xml" \
  --delete

For reproduction steps
The following is a working list of CLI commands that sets up any GatsbyJS site will officially recommended cache control options.
 If it is still relevant please reopen it there - sorry for the trouble


application pruning not working as expected

$ rollappd version
v2.2.0-hotfix.3

even though pruning seems to be working with these settings applied:
/usr/local/bin/rollappd start --log_level debug --pruning=custom  --pruning-keep-recent=100 --pruning-interval=10 --min-retain-blocks=10000

application db size doesn't change over time
du -sh ~/.rollapp_evm/data/application.db/

656G    /home/ubuntu/.rollapp_evm/data/application.db/

also, the expected behaviour is that since the starting blocks are already pruned, queries at old height shouldn't return any result, in reality they do:
$ rollappd q staking validators --height 1
pagination:
  next_key: null
  total: "0"
validators:
- commission:
    commission_rates:
...

the unknowns

node isn't completely synced with the sequencer, hence the syncing speed might be roughly the same as pruning speed and the db size doesn't change ( doesn't explain why the query at old heights return values )

the application db size does go down when the process is being restarted

closing as works as expected (we just didn't know what expected means here)


Add package-lock.json.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.00%. Comparing base (ff4d33d) to head (17c87f9).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #373   +/-   ##
=======================================
  Coverage   92.00%   92.00%
=======================================
  Files          53       53
  Lines        3792     3792
  Branches      352      352
=======================================
  Hits         3489     3489
  Misses        302      302
  Partials        1        1

Flag
Coverage Δ

unittests
92.00% <ø> (ø)

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

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


Use preconstruct for monorepo local linking

https://github.com/preconstruct/preconstruct
https://github.com/Andarist/lerna-alias/issues/11#issuecomment-841800943

actually not sure this will bring any benefit now, and we don't need the rollup builds from preconstruct

We need something to build type definitions for each package, preconstuct may be the best option here

Closed in  #992


Update Web-LLM.py for windows

Reconfigured a few things, please have a look!

Hey please check the discussions tab, and talk to me about this I made a new discussion about this so we can collaborate a bit!

Update Web-LLM.py to ensure compatibility with Windows #10. Implement any necessary adjustments to address system-specific issues. Include changes or enhancements as requested by Jason Koepke, Principal, to improve functionality and meet project requirements.

Update Web-LLM.py to ensure compatibility with Windows . Implement any necessary adjustments to address system-specific issues. Include changes or enhancements as requested by Jason Koepke, Principal, to improve functionality and meet project requirements.

@gillshaman what are you on about?

I made some changes will have to close all current pull requests unfortunately, please reopen if still needed Also Added your windows version to it's own brain thanks for your help!


Remove deprecated typings package

It's unused and jasmine migrated to @types already
npm install gives the following warning
npm WARN deprecated<EMAIL_ADDRESS>Typings is deprecated in favor of NPM @types -- see README for more information

@berickson1  Thanks for the PR. This was already in our backlog. We will review and merge it soon.

Rebased to pick up removal of package-lock.json from dev branch

This is addressed on the latest dev branch.


Better help text for sync

This will displays a better help text when an environment do not have child environments
should we add an Error: 🤔 ?
Test

pull the branch and switch to it
run npm link
run vip sync --app=[ID] where ID is the ID of an app without childs (try 636)

fixes #88

should we add an Error: 🤔 ?

If you do, I would recommend trying ${ chalk.red( 'Error:' ) } to see what it looks like.
I wonder if it's worth creating a helper function that prefixes errors with red 'Error' text.


[Feature Request] Versioning

Problem description
Versioning is not yet implemented
Desired solution
We should have a clear process to track the version of the UI.
Probably (but not necessarily) it should be in line with the version of the server API. (Pros: it is clear for everyone which to use, cons: we need to sync the code even when there are no changes to one of the two)

Fixed in #41.
We can now push a new tag (same as in infrastructure repo) and the deployment pipeline will automatically update it in the code and release it.


there is no margin for multiple page document

there is no margin for multiple page document.
the second page starts immediately and no settings available to change that.It is already an open issue in previous versions and it exists in new release as well.

Do we have any fix for this issue ?

Could you share some screenshots of where you might require margins for the pages? Most of the templates have some sort of background that pans across the entire page, which is why they don't have a specific margin. But I'd like to understand where exactly you expect margins to be.


Projects with a single module won't open in the source viewer

Description
I have a couple of projects which have a single module under src/ (https://github.com/data61/foldl-statistics/ is an example). When running ghcitui, it compiles and loads the project, but pressing M then Enter or o doesn't load the module into the source viewer. I've tried a couple of other projects with multiple source modules and they seem to work fine.
To Reproduce
$ git clone https://github.com/data61/foldl-statistics/
$ cd foldl-statistics
$ ghcitui
# press M then Enter or o

Expected behaviour
The source should be loaded in the source viewer.
Screenshots

Environment

OS: macOS 14.7.1
GHCiTUI Version: <IP_ADDRESS>, installed via cabal install ghcitui)

I confirm the same problem, which I've seen in larger projects too depending how you start GHCI. ghcitui sometimes lists modules but won't display their code. Here's one attempt, with debug output.
~/src/foldl-statistics$ ghcitui -vvv --cmd 'stack exec -- ghci -isrc -isrc/Control/Foldl src/Control/Foldl/Statistics.hs'
Starting up GHCiTUI with: `stack exec -- ghci -isrc -isrc/Control/Foldl src/Control/Foldl/Statistics.hs`...
This can take a while...
[ghcid startup:out] GHCi, version 9.8.2: https://www.haskell.org/ghc/  :? for help
[ghcid startup:out] Loaded GHCi configuration from /Users/simon/.ghci
[ghcid startup:out] [1 of 1] Compiling Control.Foldl.Statistics ( src/Control/Foldl/Statistics.hs, interpreted )
[ghcid startup:out] Ok, one module loaded.
[DEBUG]: |startup| GHCi Daemon initted

[DEBUG]: |updateContext| CMD: :show context

[DEBUG]: |updateContext| OUT:
[DEBUG]:

[DEBUG]: |updateBindings| CMD: :show bindings

[DEBUG]: |updateBindings| OUT:
[DEBUG]: it :: () = ()
[DEBUG]:

[DEBUG]: updateModuleFileMap|: CMD: :show modules

[DEBUG]: |updateModuleFileMap|: OUT: Control.Foldl.Statistics ( src/Control/Foldl/Statistics.hs, interpreted )
[DEBUG]:

[DEBUG]: |updateModuleFileMap| modules: [("Control.Foldl.Statistics","src/Control/Foldl/Statistics.hs")]

[DEBUG]: |updateBreakList| CMD: :show breaks

[DEBUG]: |updateBreakList| OUT:
[DEBUG]: No active breakpoints.

[DEBUG]: |history| OUT:
[DEBUG]: Not stopped at a breakpoint

GHCiTUI has shut down; have a nice day :)

Related: #30

I have identified the issue.
When there's only one module, we auto-select that module to load into the source viewer. However, the source viewer isn't refreshing first time it loads, so it shows the default splash screen. Additionally, the source viewer skips refreshing if the new module is the same as the old module path.
So two solutions I'm thinking:

Refresh the source on first load, so that it displays the source immediately if available.
Always refresh the source on module select.

Either of these seem fine solutions, though I'm leaning towards 1. Perhaps do both.

This should be fixed in c06c1fa. Feel free to build from develop if you wish to test it yourself. I am unlikely to release a new version until the weekend (or next weekend), so apologise for the delay.


Open gists with only github username and gist id

we all know gists url base is https://gist.github.com
would be nice if gistpad could open gists with only github user name and gist id provided.
for example: cjbassin/4b34001dcda4d20ce57b410a89096681
mainly b/c most online dev tools that work with gists will have github username and gist id in their gist preview url & it's a pain to add gist base url when clipping them into gistpad.
try blocks for example: https://blockbuilder.org/search
or vega editor that shows gists with target file name too :) https://vega.github.io/editor/#/gist/455e1c7872c4b38a58b90df0c3d7b1b9/bar.vl.json

If you run the “GistPad: Open Gist Files” command, you can already specify just the ID (in addition to the full URL). So you specifically want to be able to use username and ID?

I think it should do both. Also, I was not aware of Open Gist Files.
I tried it from Gists -> Open Gist... explorer link, which uses GistPad: Open Gist as Workspace command by default I believe.
Can you assign some keyboard shortcuts to both of those commands too?
Thanks!

1 more, somewhat related:
That gist url quick input box goes away when vscode looses focus.
which is annoying when you open to enter gist and switch to browser to copy gist id :)

Sounds good. I’ll make these changes later today 👍

The following fixes have been pushed to master, and I'll ship an update with them this afternoon. Thanks again for the feedback!

Opening a gist via username/id format (in addition to by URL or by ID)
Added a keyboard shortcut for opening a gist: ctrl+shift+o/cmd+shift+o
Prevent the open gist quick pick from closing until you explicitly hit <ESC>


Is this specific to angular?

Great library and for anyone that understand the core/concept of redux, they will understand that Rx offers that and more. My only question is, couldn't a redux store implementation using Rx have been achieved without coupling it with angular. Ex. redux isn't glued to react or any other framework. Its just a state management lib. Why not implement this the same way as another option to redux and offer specific(if any) binds to ng as an integration project just like redux offer a react integration.
I'm asking because I think there is fatigue with the whole angular/react ++ and its always great to have these libraries that focus on their core disciplines without hinging to a framework. Just a thought. Users might just assume this is just for angular which is not the case. Or is it?

as of today, yes - that's mainly because the focus of ngrx as an org is stuff for angular2. that said, there's a PR pending with a bit of a refactor to loosen that restriction, and i'd certainly consider publishing a standalone package once that's ready.


When a user doesn't have a username in paas-accounts, the username is shown as blank

In such a circumstance I think it'd be good to show the raw username from UAA (or something) so that the user is clickable and editable.

I can't replicate this. Can you show me how when you're back, @46bit ?

I think it’s already been fixed
On Wed, 4 Sep 2019 at 11:02, Andy Hunt<EMAIL_ADDRESS>wrote:

I can't replicate this. Can you show me how when you're back, @46bit
https://github.com/46bit ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alphagov/paas-admin/issues/379?email_source=notifications&email_token=AABH2J4KYSKMFEXHDCI65ATQH6BRRA5CNFSM4IKUHN52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD53BTEQ#issuecomment-527833490,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABH2J5UZYSJSOYD7H6Y3YDQH6BRRANCNFSM4IKUHN5Q
.

--
Cheers,
Miki
N.B. I prefer people use they/them pronouns for me. You can read more
about that here
https://lifehacker.com/how-to-use-gender-neutral-pronouns-1821239054, or
please ask if you'd like to find out more.
[image: image003]
Michael (Miki) Mokrysz
Senior Developer in Reliability Engineering
*Government Digital Service *

Ok, thanks, Miki. Closing this then.


Fix micro size label typo

What this PR does / why we need it:
Quick typo fix missed in #1
/cc @0xFelix
/cc @ksimon1
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm


minor doc updates to clarify extensions using accessors

Super minor docs updates that try to clarify that "accessors" are the tool used to "extend" xarray (for those of us who have spent time trying to figure out where/when to use which term).

nice!


Add ScrollView Indicator Overlay style

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[x] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary
NSScrollView can have overlay style explicitly set.
https://developer.apple.com/documentation/appkit/nsscrollerstyle/nsscrollerstyleoverlay
Overlay style will not take up any space in the ContentView so content can be flush with insets.
Default uses the legacy style, not the overlay.
Changelog
[macOS] [ADDED] - Add ScrollView Indicator Overlay style
Test Plan
Overlay style can be enable/disabled
https://user-images.githubusercontent.com/96719/213046667-0b09b0f9-a0ad-4758-9ad8-7dfefa729d71.mp4

This PR would allow you to override the system preference and set your own default? @chiuam sounds useful 😅

This PR would allow you to override the system preference and set your own default? @chiuam sounds useful 😅

This does allow user to override the system preference and always have overlay style enabled.
https://user-images.githubusercontent.com/96719/213052899-fa625570-eaf5-4482-9f86-5ed8889dbbdb.mp4


Proposal: flag: add support for int32 and uint32 options

Currently many core libraries like grpc have configurable flags and options that are either int32 or uint32, e.g.

HTTP2 Server Max Concurrent Streams
Internal Poll Socket Opt in Windows

This forces the userland code for any configurable system to look something like this:
...
maxConcurrentStreams := f.Int("max_streams", 100, "Number of maximum concurrent streams")
f.Parse()
...
maxConcurrentConnectionsOption := grpc.MaxConcurrentStreams(uint32(maxConcurrentStreams))
...

Such casting (and any other workaround) results in error prone implementations sometimes, since the error handling has to be done by the developer, which I'm not particulary against, but it's better if we could trust Go's core flag parsing library to support and validate this.
In this manner, we could allow the implementations to simply look like this:
maxConcurrentStreams := f.Uint32("max_streams", 100, "Number of maximum concurrent streams")
f.Parse()
...
maxConcurrentConnectionsOption := grpc.MaxConcurrentStreams(maxConcurrentStreams)
...

I know it may look simple but flag already supports int, int64 and float64, so imho, adding this makes sense and allows for better looking and more secure code.

I've opened https://github.com/golang/go/pull/26465 with the necessary implementation to add this to flag.

I'm not sure that the presence of int64 and uint64 sets a precedent. I imagine they were added so that one can have numerical flags that accept very large values without having to worry about 32-bit builds.

@mvdan the reason for adding uint32 and int32 is not due to the precedent of other types.
The main reason behinf all of this is to allow core libraries users to be able to produce homogeneous code that is less error prone in basic implementations.
I do agree with the thought of adding *64 types for build-related reasons tho.

You can already use flag.Var to declare a flag variable with an arbitrary validation function (the Set method), and flags in general may require different validation constraints: for example, some must be nonzero, or must fit in a uint32 after adding 12 (to account for some prefix added by the package), etc.
So what's special about the uint32 and int32 constraints? (If you believe they're more common than other validators, some statistics or examples would help.)

int/uint and int64/uint64 should be enough. We don't need the entire set. Note that if uint32 is a very important type (such as in this particular package) it's easy to build a uint32-accepting flag.Var, as @bcmills pointed out above.


Update governance, split out on-/offboarding

Signed-off-by: Richard Hartmann<EMAIL_ADDRESS>

I will let this sit here for feedback before calling for a formal vote.

Previous members is nicer

Very valid points; I honestly had forgotten about Conor. Last I talked to
him, Prometheus was a job, nothing more.
Should we ask Matt?
On Sun, Feb 16, 2020 at 9:40 PM Björn Rabenstein<EMAIL_ADDRESS>wrote:

@beorn7 approved this pull request.
BTW: There is one former prometheus-team member already: Conor Broderick.
As we didn't have an alumni/ae list back then, he wasn't asked if he wants
to be on it. When we actually create the list, we should probably ask him
about it. Also, that list should clarify that there have been very active
contributors (and even leaders/founders) who are not on the list of former
team members because there was no formal team back in their days.
But that just as a comment. The changes here look OK to me.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/docs/pull/1552?email_source=notifications&email_token=AAFYII4WFALICVAAETEQ4RLRDGQE5A5CNFSM4KVFRWNKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCVWG4ZI#pullrequestreview-359427685,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFYII7IPNM4NNWQ7INCRI3RDGQE5ANCNFSM4KVFRWNA
.

Should we ask Matt?

Matt is kind of an easy case as he is explicitly acknowledged in the Acknowledgement section on https://prometheus.io/community/ . I'm more concerned about other people that were never on prometheus-team because they contributed before prometheus-team existed. Might become a rabbit hole.
I suggest to think about it once a team member has left and actually wishes to be listed. Then we can decide about the correct format of the alumni/ae list.

On 17 Feb 02:31, Richard Hartmann wrote:

RichiH commented on this pull request.

Upon death of a member, their team membership ends automatically.
+In case the member leaves, the offboarding procedure is applied.
This was a serious question, btw. Still think we don't need to, just went exploring corner cases as "someone in a coma" would mean someone else gaining access to their accounts, passwords, etc.

I think that this should not be automated. A forced removal vote would be enough.
--
(o-    Julien Pivotto
//\    Open-Source Consultant
V_/_   Inuits - https://www.inuits.eu

No as that is always the fallback. I had half a mind to restructure the
whole consensus and voting references, but decided to tackle that in the
future.
Sent by mobile; please excuse my brevity.
On Wed, Feb 19, 2020, 21:53 Brian Brazil<EMAIL_ADDRESS>wrote:

@brian-brazil commented on this pull request.
In content/governance.md
https://github.com/prometheus/docs/pull/1552#discussion_r381537867:

@@ -95,6 +94,12 @@ Decisions are usually made by lazy consensus. If no consensus can

Material changes to this document are discussed publicly on the [developer mailing mailing list][devs]. Any change requires a supermajority in favor. Editorial changes may be made by lazy consensus unless challenged.
+### Editorial changes
+
+Editorial changes are changes which fix spelling or grammar, update work affilition, or similar; they update style or reflect an outside and obvious reality.
+They do not change the intention or meaning of anything in this document.
+They must be made via PR and accepted via lazy consensus.
Could this prevent a majority vote on these?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/docs/pull/1552?email_source=notifications&email_token=AAFYII5OZ3BXGILX7JT7ZN3RDWL6DA5CNFSM4KVFRWNKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCWFPMRQ#pullrequestreview-361428550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFYII777XFALXLEERM2LDTRDWL6DANCNFSM4KVFRWNA
.

For back-reference, we are now voting on this in "[VOTE] New governance document" on prometheus-dev

Quorum has been reached and 16 in favour votes are documented at https://groups.google.com/forum/#!topic/prometheus-developers/4mHqIUlloA0 .
23 * 0.66.. = 15.3.., so 16 is a 2/3 majority.


Enhance the latest blog

Add an image to the latest blog post and bold some of the bullet points for consistent formatting.
Blog - https://meshery.io/blog/simplifying-cloud-infrastructure-and-kubernetes-management
Contributor Resources
The meshery.io website uses Jekyll and GitHub Pages. Site content is found under the master branch.

See the Contributing to Meshery.io Website section of the readme.md and other contributing instructions, too.
See Meshery site designs in this Figma designfile. Fill-in the Layer5 Community Member Form and join the Layer5 Community Slack for access.

@iArchitSharma I would like to work on this issue can you assign it to me

@Satakshijain sure go on


Block buttons should highlight depending on which block type you are currently in

When you are in a Heading 1 block, that box should be highlighted (in bold) and the rest should not be. etc.

Hey, @tcr. Has this been taken up?

@briankabiro Hey! Unfortuantely it has (live on http://sandbox.edit.io/) and I've been bad at issue maintenance. Thanks for surfacing this.
I opened another good first issue task, in case that's what you were looking for: #44
There is also a follow up task to this available about making the "highlight button when caret is inside it" logic less expensive. Right now it's called for every document change and this computes some things twice. By avoiding the recomputation, it should speed up all client changes.

Oh, thanks. I was looking for a good first issue task. I'll checkout #44 to get acquainted with the codebase and then ease my way to the follow-up task.

Hey, @tcr. Referencing There is also a follow up task to this available about making the "highlight button when caret is inside it" logic less expensive.. Could you provide more context on that, or would it be better to open an issue for it? 😃

@briankabiro I've opened a new issue with first steps!
https://github.com/tcr/edit-text/issues/51
Making the "figure out the button state logic" where identify_block is called in client.rs is still expensive, but the first step is to reduce how much data we're sending to the client. This should drop the WebAssembly <-> JavaScript traffic by half.
I'll follow up with other tasks related to this shortly, but hopefully this is a good start. Thanks for your first PR also :)

Thanks, @tcr for the prompt response. I've checked out the issue and can take it up.
Also about the first PR, no biggie. 👍


feature request: use site's favicon by default

It would make sense to, by default, use URL site's favicon.ico, unless specified otherwise.
At the moment, I have to visit the site, fetch the favicon, save it locally and then specify it in the configuration.

Thank you for your suggestions on my project ! 🙂
This one requires only minor adjustments and should be available pretty soon.


Loss value

I am wondering how loss value looks like. Could you give some pictures of the loss during training?

Sorry for the dense calculation of the MLE loss...
I'll let you know when I clean up the clutter in the code.
Temporarily, I'll explain the loss one by one.
l_mle_sum = l_mle_normal0.5 * math.log(2 * math.pi) + (torch.sum(y_logs) + 0.5 * torch.sum(torch.exp(-2 * y_logs) * (z - y_m)**2) - torch.sum(logdet)) / (torch.sum(y_lengths // hps.model.n_sqz) * hps.model.n_sqz * hps.data.n_mel_channels)

Sorry for the dense calculation of the MLE loss...
I'll let you know when I clean up the clutter in the code.
Temporarily, I'll explain the loss one by one.
The original line I implemented was:
l_mle = 0.5 * math.log(2 * math.pi)
    + (torch.sum(y_logs) + 0.5 * torch.sum(torch.exp(-2 * y_logs) * (z - y_m)**2) - torch.sum(logdet))
    / (torch.sum(y_lengths // hps.model.n_sqz) * hps.model.n_sqz * hps.data.n_mel_channels)

It can be decomposed as
l_mle_normal = torch.sum(y_logs) + 0.5 * torch.sum(torch.exp(-2 * y_logs) * (z - y_m)**2)
l_mle_jacob = -torch.sum(logdet)
l_mle_sum = l_mle_normal + l_mle_jacob
denom = torch.sum(y_lengths // hps.model.n_sqz) * hps.model.n_sqz * hps.data.n_mel_channels
l_mle = 0.5 * math.log(2 * math.pi)  + l_mle_sum / denom

l_mle_normal is the negative log likelihood of normal distribution N(z| y_m, y_logs) (except the constant term: 0.5*log(2pi)), where y_m and y_logs are the mean and logarithm of standard deviation of the prior distribution. Please see Equation 2 in the paper.

l_mle_normal = torch.sum(y_logs) + 0.5 * torch.sum(torch.exp(-2 * y_logs) * (z - y_m)**2)

l_mle_jacob denotes the negative log determinant of jacobian of flows. Please see Equation 1 in the paper.

l_mle_jacob = -torch.sum(logdet)

l_mle_sum denotes the total negative log likelihood of the model, and denom is a denominator to average the total negative log likelihood across time steps and mel channels (Our model force a mel-spectrogram length y_lengths to be a multiple of n_sqz.).

l_mle_sum = l_mle_normal + l_mle_jacob
denom = torch.sum(y_lengths // hps.model.n_sqz) * hps.model.n_sqz * hps.data.n_mel_channels

Add the the constant term, 0.5*log(2pi), excluded in step 1.

l_mle = 0.5 * math.log(2 * math.pi)  + l_mle_sum / denom

Thanks for your detailed explanation. I think you could ignore the constant term, it does not contribute to backpropagation. Btw, I found another paper that has the same idea of learning implicitly duration of each character but in a different approach AlignTTS.

Yes the constant term is ignored in backpropgation. I just left it for exact calculation of log likelihood. And I saw AlignTTS, which also proposes an alignment search algorithm similar to Glow-TTS. I think it is clever, thanks for the heads up! Btw, I hope you enjoy the interesting characteristics of our model such as manipulating the latent representation of speech :)

Just wanted to say amazing work! Love the controllability of length and expressiveness. I wanted to try a few ideas of my own using your repository as a codebase by I've run into a strange phenomena. It's related to loss function so maybe you could help me understand what is the cause. The strange thing is that the value of l_mle(g0) loss depends on value range of melspectrograms.
Orange - LJSpeech wavs transformed into melspecs using default paramters. Melspec values range from 0.5 to -11.5
Pink - My data transformed the same way as LJSpeech
Blue - My data transformed to melspecs with different sfft parameters and then scaled to 0.5 to -11.5 range
Gray - My data transformed to melspecs with different sfft parameters. Value range from 0. to 0.76 (the same results if multiplied by -1)

From what I was able to check in the case of data in the range of 0 to 0.76 values differ in the following way
l_mle_jacob - is bigger for melspectrograms with smaller absolute values. I think it make sense because jacobian is calculated based on weigths and they have to be bigger to result in the same values.
l_mle_normal -  about the same
denom - obviusly the same
l_mle - with different proportion of l_mle_sum and denom l_mle no longer normalizes to 1. I think it's a problem because balance between g0 and g1 is disturbed and alignment gets worse


RecordCollection

Record Collection:
//my code here//
function updateRecords(id, prop, value) {
if(prop === "tracks" && value !== "") {
collection[id][prop] = value;
}else if(value !== "") {
collection[id][prop]=value;
}//delete
else {
delete collection[id][prop];
}
return collection;
}
Issue Description

should pass all tests... but doesn't.
the console is displaying all the information requested to pass the given tests.
Browser Information

this is a Chrome browser on a linux system

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:
5.0 (X11; Linux i686) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/53.0.2785.143 Chrome/53.0.2785.143 Safari/537.36

record 5439 tracks should be an array.. Pls use the help room next time you have this kinda issue.

Hi @theman0123
Thanks for the report.
The issue tracker is for reporting bugs only. Please use the help chat room or try looking through the forum for help with a specific challenge.
Happy Coding.


subcommand doesn't support custom usage

When I defined UsageText in subcommand, I found it doesn't work.
Is that subcommand doesn't support custom usage?

I don't know the answer to this offhand, but my experience is that the help text could definitely use a lil bit of cleanup 👍

Unable to reproduce. We have unit tests for this and the template engine supports UsageText from App & Command as well.


Better explanation of "ReferenceError: SYMBOL_ITERATOR is not defined" error

This is related to #9 (which was resolved) -- essentially it seems that the @Componenent annotation is calling out to jQuery but doing so using an anonymous function that doesn't really give a helpful stack trace.
For example, I get error messages like this in my console:
ReferenceError: SYMBOL_ITERATOR is not defined
    at checkIter (eval at <anonymous> (http://localhost:50017/packages/jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:372:22), <anonymous>:1090:7)
    at getCollection (eval at <anonymous> (http://localhost:50017/packages/jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:372:22), <anonymous>:1119:33)
    at Object.23../$ (eval at <anonymous> (http://localhost:50017/packages/jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:372:22), <anonymous>:1263:11)
    at s (eval at <anonymous> (http://localhost:50017/packages/jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:372:22), <anonymous>:40:254)
    at eval (eval at <anonymous> (http://localhost:50017/packages/jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:372:22), <anonymous>:40:305)
    at Object.1../src/es5 (eval at <anonymous> (http://localhost:50017/packages/jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:372:22), <anonymous>:55:1)
    at s (eval at <anonymous> (http://localhost:50017/packages/jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:372:22), <anonymous>:40:254)
    at e (eval at <anonymous> (http://localhost:50017/packages/jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:372:22), <anonymous>:40:425)
    at eval (eval at <anonymous> (http://localhost:50017/packages/jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:372:22), <anonymous>:40:443)
    at eval (eval at <anonymous> (http://localhost:50017/packages/jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:372:22), <anonymous>:2263:2)

I was hoping that we could either try to work on improving the documentation around the @Component (for example, the different "bind" options of '=', '&', and '@' are not documented), and maybe also provide a more helpful error message when the "selector" fails (which I think is the origin of my error).

@jacobdr I'm open to documentation improvements. Perhaps you would like to help with a PR?
I have not documented the bind options "=", "@" and "&", because they are exactly the same as those documented here https://docs.angularjs.org/api/ng/service/$compile under the heading Directive Definition Object, sub heading scope. Perhaps we could add the link to the relevant AngularJS documentation.
Also, what do you mean when you say?

more helpful error message when the "selector" fails

Could you provide me an example of the actual problem or a link to a repo?

The work that you have done so far is really awesome -- and until the Angular2 docs get updated (they are really pretty sparse right now...) your project allows our team to experiment with the new syntax while being able to fall back on the old API without penalty.
I am very happy to help out. I am actually pretty new to Angular2 (and even Angular1), and so am reading through your source and trying to get more familiar with it. I forked the repo and will try to submit a PR in a couple of days when I feel more comfortable.
With regard to that error, I think it is related to the fact that I am mixing syntaxes in my project. For example, I am registering one controller via the Ang1 module("x").controller("y",function(){}) and a second controller via the new syntax. I don't use the bootstrap function since I already have an ng-app corresponding to those older modules at the top of my source.
I thought that I could just register a new controller using SetModule(), then use the @Component tag, and then drop in a corresponding "" tag in my index.html and everything would just work. Seems like I am running into slightly more trouble, which I think relates to the "selector" property in the @Component annotation, since it is jQuery that is showing up in that stack trace, but I could be totally wrong. Anyway, it would be ideal if that stack trace would mention the line in the Angular2 now file that is the root cause of the exception. At the end of the day I am fairly certain it is an error in my use of the library and not a bug, but descriptive errors are still a useful end to achieve.
At the end of the day this is all related to the fact that I am mixing syntaxes, and so was hoping you might have some examples pages or best practice suggestions for how to do this.

Of course my error was a typo in the @View{templateUrl:path}.... Not sure if you have any ideas for how to throw an error that would indicate as much, but I'll mull over the source and submit a PR if you think that it is appropriate

This error is a known issue with Angular-Meteor. You'd also get this error if you weren't using angular2-now and you mistyped the name/path of a template in a templateUrl. So, a typo in the template path results in some weird angular error, which doesn't seem to relate to anything. However, it is not actually to do with angular2-now.
The solution, as I see it, would be to verify that a template path actually exists. To do that you'd have to try to pull the template itself in using XHR ($http) or something similar. Also, the template could already be in the template cache, if your build workflow bundles your templates into JS files, like Angular-Meteor's does.
I might look into angular-meteor on the weekend, if I get some time, to see if I can find/fix this issue.
But the good news for now is that this is not an angular2-now bug :)

Regarding mixing normal code with angular2-now, the best advice I have is don't do it.
Also, try to not use stand alone controllers at all.
I have some examples listed at the top of the README, which show how to put together applications with my library. Perhaps those would help?
At work we are using angular2-now in production on some larger projects. I was thinking about extracting some best-practices from those, but there never seems to be enough time. Again, perhaps this weekend :)


Add link for configuring Emacs

Hi @bram209 is there anything wrong with Github actions on this repo?

I am not sure why, but they are not running for first time contributors (even after approving a PR...).
That aside,  thanks for your PR.
The documentation that you wrote for Emacs sets the LSP server setting globally, however the note about VSCode and Neovim are describing how to do project-local LSP configuration. If you set it globally, it will call leptosfmt even for projects that don't use leptos at all. Do you know if this is easy to manage in emacs as well? If so, could you update the documentation?

Ah okay. I think at this point (having docs for 3+ editors) it makes sense to restructure the readme a bit so that each editor has its own heading.
For example:
# Usage with Rust Analyzer
...

## VSCode
...

## NeoVim
...

## Emacs
...

## Helix
...

I will restructuring the readme a bit and piggybacking on the Emacs instructions that you provided in this PR. Thanks again for taking the time to contribute : )


Add support for requiring json

Fixes #519

It's so dumb that end-users needs to this manually. It should just be handled by Rollup by default. Rollup should support the Node.js require algorithm out of the box... Anyways.

Yeah rollup literally only takes care of es modules and tree shaking. That's it. To replace browserify it literally needs 5+ plugins (node-resolve, commonjs, json, globals, builtins, maybe more) and they often have issues as you start requiring modules that have less common configurations.
I'll try webpack@3 at some point, once ModuleConcatenationPlugin gets more docs.

I'll try webpack@3 at some point, once ModuleConcatenationPlugin gets more docs.

Even Webpack forces you do to the JSON thing manually...

Note: Since webpack v2, importing of JSON files will work by default. You might still want to use this if you use a custom file extension.
https://github.com/webpack-contrib/json-loader

🤗


Updating Readme Quickstart instructions to include required python ve…

What this Pull Request (PR) does
When I attempted to follow these instructions in a windows environment using WSL, I kept running into issues because my python version was too low (3.8). I then was going through hoops trying to upgrade to version 3.12 as the process seems more complicated on windows OS.
To avoid these headaches, I thought it best to warn potential users ahead of time to ensure their environment is running the latest version of Python or at least python 3.10, which seemed to work for me finally.
Related issues
Please reference any open issues this PR relates to in here.
If it closes an issue, type closes #[ISSUE_NUMBER].
Screenshots
Provide any screenshots you may find relevant to facilitate us understanding your PR.


Intermittent error: Requested 'h11<0.13,>=0.11', but h11==0.13.0 is already installed

Just got that on this page: https://lite.datasette.io/?sql=https%3A%2F%2Fgist.githubusercontent.com%2Fsimonw%2Fac4e19920b4b360752ac0f3ce85ba238%2Fraw%2F90d31cf93bf1d97bb496de78559798f849b17e85%2Fdemo.sql
  File "/lib/python3.10/asyncio/tasks.py", line 232, in __step
    result = coro.send(None)
  File "/lib/python3.10/site-packages/micropip/_micropip.py", line 310, in add_requirement
    return await self.add_requirement_inner(req)
  File "/lib/python3.10/site-packages/micropip/_micropip.py", line 391, in add_requirement_inner
    if self.check_version_satisfied(req):
  File "/lib/python3.10/site-packages/micropip/_micropip.py", line 337, in check_version_satisfied
    raise ValueError(
ValueError: Requested 'h11<0.13,>=0.11', but h11==0.13.0 is already installed

BUT... sometimes when I load that page it works OK. So maybe a timing error of some sort?

Trying to spot the conflict:
https://github.com/simonw/latest-datasette-with-all-plugins/blob/main/pipdeptree.txt
    - httpx [required: >=0.20, installed: 0.23.0]
      - certifi [required: Any, installed: 2022.6.15]
      - httpcore [required: >=0.15.0,<0.16.0, installed: 0.15.0]
        - anyio [required: ==3.*, installed: 3.6.1]
          - idna [required: >=2.8, installed: 3.3]
          - sniffio [required: >=1.1, installed: 1.2.0]
        - certifi [required: Any, installed: 2022.6.15]
        - h11 [required: >=0.11,<0.13, installed: 0.12.0]
        - sniffio [required: ==1.*, installed: 1.2.0]
      - rfc3986 [required: >=1.3,<2, installed: 1.5.0]
      - sniffio [required: Any, installed: 1.2.0]
    - hupper [required: ~=1.9, installed: 1.10.3]
    - itsdangerous [required: >=1.1,<3.0, installed: 2.1.2]
    - janus [required: >=0.6.2,<1.1, installed: 1.0.0]
      - typing-extensions [required: >=<IP_ADDRESS>, installed: 4.3.0]
    - Jinja2 [required: >=2.10.3,<3.1.0, installed: 3.0.3]
      - MarkupSafe [required: >=2.0, installed: 2.1.1]
    - mergedeep [required: >=1.1.1,<1.4.0, installed: 1.3.4]
    - pint [required: ~=0.9, installed: 0.19.2]
    - pluggy [required: >=1.0,<1.1, installed: 1.0.0]
    - PyYAML [required: >=5.3,<7.0, installed: 6.0]
    - uvicorn [required: ~=0.11, installed: 0.18.2]
      - click [required: >=7.0, installed: 8.1.3]
      - h11 [required: >=0.8, installed: 0.12.0]

So uvicorn requires h11>=0.8, but httpx requires h11>=0.11,<0.13
Uvicorn isn't used by Datasette Lite though. And it still doesn't explain what caused h11==0.13 to be installed.

httpx: https://github.com/encode/httpx/blob/dd8a1d6cda72bb37883b8e4315f58b6d2acb7e19/setup.py#L62
        "httpcore>=0.15.0,<0.16.0",

httpcore: https://github.com/encode/httpcore/blob/225a067ee5727b7468c1ad4bf9e954cc0ab7cfc9/setup.py#L57
        "h11>=0.11,<0.13",

There's a PR against httpcore to upgrade that dependency here:

https://github.com/encode/httpcore/pull/526

I've only started seeing this error after I upgraded to Pyodide 0.21.0. I'm going to downgrade to 0.20.0 for the moment.

No longer a problem with 0.22.1.


haunted mesh not working

Do you supply heat to the Air Blower with the Haunted Mesh?
I might make it so it doesn't require heat by default, since it seems what most people are assuming.

Maybe add it as a tool tip? Putting a tool tip on the meshes that indicates that they go in the industrial air blower might also help.


Any plan to support weights quantization?

Thanks for the great project
I see that there is currently a separate project about converting frozen model but do we have any plan to support weights quantization during checkpoint convertion? Or it's not necessary since we can reuse quantization in tensorflow toco? Especially when the frozen model support is ready.

@pyu10055 and @nkreeger for more details.

The plan is to use grappler instead of TOCO for graph optimization,  weight quantization will not be in the initial release. It will be high in our radar, since it is crucial for reducing the weight file size.

Wow, have never heard of grappler module before. That sounds like a good plan, thanks!


Added options for localization

For a multi language site I needed the plugin to handle different languages. Added an extra group of options for all the labels and moved the two exisiting message options also to this group. Also updated the documention for the new options. Hope you like it and can merge it into your branch. Potentially a lot of labels needed in the HTML. Maybe future improvement could be to load translations through a translation/resource json file.

Hey! Super!
Tested and it seems to be working.
Thanks a lot.

Nice, thanks!

Bumped up the version https://github.com/Trustroots/trustpass/releases/tag/0.2.0


export ValidationError and MissingRefError

What issue does this pull request resolve?
#1832
What changes did you make?
Exported 2 Error subclasses.
Is there anything that requires more attention while reviewing?

Never saw this export format before :)
It should be added in other ajv classes too - core.ts, jtd.ts, 2019.ts and 2020.ts

Probably no need to export from core.ts.

What is required to get this over the line? Im hitting alot of issues trying to upgrade a codebase. Happy to get involved if possible

@dannyb648: It should be added in other ajv classes too - jtd.ts, 2019.ts and 2020.ts

@dannyb648: It should be added in other ajv classes too - jtd.ts, 2019.ts and 2020.ts

I've added these changes and the original ones to PR #2020 - Any chance of a review? Additionally looks like something is broken with the coverage plugin on all the PRs atm?


Disable FindBugs NPE checking / Fix possible NPE

#241.

Ignore and disable FindBugs' possible NPE checking.

Add // NOPMD to suppress empty if-statement.

Fix one point where NPE seems possible.

LGTM


[wip] tests: add browser tests

Time to add some tests so regressions don't occur.

[x] happy path
[x] send()
[ ] namespaces
[ ] effects
[ ] subscriptions

After this we probably want write some specs for the demos, but this is unit testing I guess :sparkles:

[Current coverage][cc-pull] is 44.04%

Merging [#24][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #24   diff @@
==========================================
  Files             1          1
  Lines           109        109
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits             48         48
  Misses           61         61
  Partials          0          0

Powered by Codecov. Last updated by [bd28e93...0e90f8f][cc-compare]
[cc-base-branch]: https://codecov.io/gh/yoshuawuyts/choo/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/yoshuawuyts/choo/compare/bd28e93cd19335d3ca69f4cbda884e6faaa160ea...0e90f8f7b72e00404677668eac1f734ecafac96a
[cc-pull]: https://codecov.io/gh/yoshuawuyts/choo/pull/24?src=pr

Closing in favor of #72


Setup release v0.0.28

#2834 is blocking this release

https://github.com/google/kythe/releases/tag/untagged-15bf83bf9d8cbde5c010


Added a col 'TYPE', renamed previous col as 'Name'

Please double check your table formatting. It is not rendering properly

you can 'preview' the file to see how the markdown will render. It should show up as a table and right now it is just misformed text

Sorry but somebody else already fixed issue #17  without any errors. You can work on another issue if you'd like.


ASSIGNMENTS QUERY

I don't see where we submit assignments for feedback

I don't see where we submit assignments for feedback

me too. and I just started this course


Make bitvector ABC

We should create a BitVector ABC to aid abstraction of the BV types.
This ideally would provide mixin methods where appropriate, e.g. define  bvnot and bvand get bvnand for free.
Crazy idea: I think we could get away with requiring only __getitem__, concat, and bvnand  for all of the bv* methods but that would lead to inefficient code so we probably don't want to encourage that.
We might want to add a make_constant method, this would make abstracting over Bit Vector types easier as their constructors are not guaranteed to be uniform (e.g. a smt_bitvector needs to bound to a solver and Bit is already bound to numbits https://github.com/phanrahan/peak/blob/mapper/peak/mapper/SMT_bit_vector.py)
def make_constant(self, value:int, num_bits:typing.Optional[int] = None) -> type(self):
  if num_bits is None:
    num_bits = self.num_bits
  #else:
  #  if a type with a bound size (e.g. Bit) construct a type of an appropriate size
  return type(self)(value, num_bits)

Closed by https://github.com/leonardt/bit_vector/pull/17, let's open more specific issues as they arise.


Why ship generated files?

OpenSSL ships objects.txt and all the files that are built from it.  Is that still necessary?  It's hard to see the rationale for doing so, since so many other files are built at configure time now.
Ping @levitte since this might have out out-of-tree implications.
There are many PR's that have been bigger than they needed to be because of this, for example.

And also many merge problems...
Not sure we can totally get rid of OIDs and NIDs,  though.

I don't want to get rid of them, I just don't want to ship the files; run the perl scripts at the same time we build CA.pl, configdata, etc.

Just like other numeral files (util/*.num, crypto/err/openssl.txt, to name a few), they contain numbers that people expect to be constant over time. Among others, NIDs are shared between libcrypto and engines, and applications can use them too, so if those numbers would change from one openssl build to another, we would get complete chaos.
This is a very tangible reason why we try to avoid NIDs as much as possible in the new provider interface, so we don't fall into that same maintenance trap (and also, so provider vendors don't have to ask us to insert NIDs for their stuff, like we currently have to do with, for example, gost).

That's a good comment, @levitte, but as long as OpenSSL controls objects.txt, the numbers won't change.  Your comment doesn't explain why the dependant files could not be built at configuration time.

The NIDs are public, constant and generated. They are not present in objects.txt. You are mixing them up with OIDs.

Hmm. Sure, enforcing make update for freshly added objects does the trick.

In theory, we could exclude autogenerated files from PRs to reduce the probability of merge conflicts, but it will make the final merge and travis tests more complicated.

Yeah, sure, it's perfectly possible to wait with the make update until release time...  that is, unless something new in the code uses the newly generated NIDs.

The NIDs are public constant and generated.
OpenSSL controls objects.txt and the scripts that parse it and generate all the other things.
Why not do it at config time?

In theory, we could exclude autogenerated files from PRs to reduce the probability of merge conflicts, but it will make the final merge and travis tests more complicated.

No final merge.  The only file in the tree is objects.txt and the scripts that generate the files.
It gets even EASIER.

Why not do it at config time?

Because existing NIDs need to remain constant across minor versions, let alone from one build to another.  It's the exact same problem as the numbers generated for _R_ macros, do you want to generate those on the fly too?

Because existing NIDs need to remain constant across minor versions, let alone from one build to another.

Except that OpenSSL makes no checks to ensure that this happens.  If we wanted to do that, we should have a separate "state" file, just like the error macros.  (And, of course, the project has no tests to see if the error number state file changed anything, as opposed to adding anything.)  I might do a test across various versions to see if the NID's changed.

The "state" file you're looking for is crypto/objects/obj_mac.num

Why is everything in "make generate" and "make update" done at make time, and not at Configuration time?  Configure is around 3200 lines of perl.  Do we not trust all the other perl scripts to run properly?

I'm afraid the answer is the same. If software refers to NIDs, they should be constant. Do we have a way to make them constant without storing them in this or that form?

Please see https://github.com/openssl/openssl/issues/9966#issuecomment-533818815

If we ship the same state file and the same perl scripts, why do we not trust that everyone will get the same output?  And why do we trust it for Configure and various .in files, but not the other autogenerated files?

Got it. If we ship the same state file, it works. It let us to regenerate of obj_dat.h and obj_mac.h in during Configure (or maybe, make) phase.

Aaaaah, that's what you were after. Yeah, good point

Marking as inactive, to be closed at the end of 3.4 dev, barring further input


ftype missing

Is this expected behaviour? Perhaps there is something else I need to require?
$ bin/pry
File.respond_to? :ftype
=> true
require 'fakefs/safe'
=> true

FakeFS.activate!
=> true
File.respond_to? :ftype
=> false
FakeFS.deactivate!
=> true
File.respond_to? :ftype
=> true

Version of the lib is 0.6.5.
Regards,
iain

Forgive me, I see there is a pull request with this feature pending. Github's search foils me once again (o_O) :)


LinkedIn recommendation templates

We need to write the recommendation for the LinkedIn exchange network.

Closed by https://github.com/shu8/mlh-fellowship-0.2.1-halfway-hackathon/pull/41


Date table has 0 rows

In the 1.06 release, the Date table appears empty using the default settings.

Thanks for reporting it. I updated the ZIP file on sqlbi.com and I released a 1.06b version here with the updated files.


ci, docker: Disaggregate Clang and LLVM versions, Clang 12, LLVM 10+11

Towards #12

Oh whoops, the dev container needs a version of Clang that is no newer than the version of LLVM, since it compiles the C programs in the test suite... Or we could check in the LLVM IR files...

I think the right answer is to limit the version of Clang to that of the highest supported version of LLVM, and only run the golden tests when using the highest supported versions. That way, we keep just one copy of the golden tests and save on CI minutes.


X11: add held modifier info on mouse events when window isn't focused

Before this, moving the mouse or scrolling in JWM window while it isn't focused would always give no held modifiers. With this patch, the modifiers are read from the appropriate field in addition to the key state logic. (mouse/key presses don't need changes, as they only happen when the window is already focused)

Thanks!


Not compatible with laravel 5.4.15?

Hi, I am trying to use this Middleware in my laravel 5.4 project, but the installation process stop with this error
`Your requirements could not be resolved to an installable set of packages.
Problem 1
- Conclusion: remove laravel/framework v5.4.15
- Conclusion: don't install laravel/framework v5.4.15
- laravelcollective/html v5.2 requires illuminate/view 5.2.* -> satisfiable by illuminate/view[v5.2.0, v5.2.19, v5.2.21, v5.2.24, v5.2.25, v5.2.26, v5.2.27, v5.2.28, v5.2.31, v5.2.
32, v5.2.37, v5.2.43, v5.2.45, v5.2.6, v5.2.7].
- laravelcollective/html v5.2.1 requires illuminate/view 5.2.* -> satisfiable by illuminate/view[v5.2.0, v5.2.19, v5.2.21, v5.2.24, v5.2.25, v5.2.26, v5.2.27, v5.2.28, v5.2.31, v5.
2.32, v5.2.37, v5.2.43, v5.2.45, v5.2.6, v5.2.7].
- laravelcollective/html v5.2.2 requires illuminate/view 5.2.* -> satisfiable by illuminate/view[v5.2.0, v5.2.19, v5.2.21, v5.2.24, v5.2.25, v5.2.26, v5.2.27, v5.2.28, v5.2.31, v5.
2.32, v5.2.37, v5.2.43, v5.2.45, v5.2.6, v5.2.7].
- laravelcollective/html v5.2.3 requires illuminate/view 5.2.* -> satisfiable by illuminate/view[v5.2.0, v5.2.19, v5.2.21, v5.2.24, v5.2.25, v5.2.26, v5.2.27, v5.2.28, v5.2.31, v5.
2.32, v5.2.37, v5.2.43, v5.2.45, v5.2.6, v5.2.7].
- laravelcollective/html v5.2.4 requires illuminate/view 5.2.* -> satisfiable by illuminate/view[v5.2.0, v5.2.19, v5.2.21, v5.2.24, v5.2.25, v5.2.26, v5.2.27, v5.2.28, v5.2.31, v5.
2.32, v5.2.37, v5.2.43, v5.2.45, v5.2.6, v5.2.7].
- laravelcollective/html v5.2.5 requires illuminate/view 5.2.* -> satisfiable by illuminate/view[v5.2.0, v5.2.19, v5.2.21, v5.2.24, v5.2.25, v5.2.26, v5.2.27, v5.2.28, v5.2.31, v5.
2.32, v5.2.37, v5.2.43, v5.2.45, v5.2.6, v5.2.7].
- don't install illuminate/view v5.2.0|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.19|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.21|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.24|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.25|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.26|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.27|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.28|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.31|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.32|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.37|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.43|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.45|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.6|don't install laravel/framework v5.4.15
- don't install illuminate/view v5.2.7|don't install laravel/framework v5.4.15
- Installation request for laravel/framework (locked at v5.4.15, required as 5.4.) -> satisfiable by laravel/framework[v5.4.15].
- Installation request for laravelcollective/html 5.2. -> satisfiable by laravelcollective/html[v5.2, v5.2.1, v5.2.2, v5.2.3, v5.2.4, v5.2.5].
Installation failed, reverting ./composer.json to its original content.`

Not really sure why you think this is an issue with my package? It seems that your require laravelcollective/html 5.2 in your 5.4 project?


Purifycss and ">" css operator

Hi,
does Purifycss remove lines with ">" css operator ?
I have this code on my project (from zurb/foundation) :
.grid-margin-x > .cell {...}
and i never used '.grid-margin-x', but i used '.cell'. After purify, the previous css rule still present.
purify-css : 1.2.5
node : 8.11.3

please suggest me
i try to remove unuse css from my side with Purifycss
https://www.asguardplus.com/%e0%b9%80%e0%b8%aa%e0%b8%b7%e0%b9%89%e0%b8%ad%e0%b8%84%e0%b8%99%e0%b8%87%e0%b8%b2%e0%b8%99/
but side bar navigator is not activated in mobile.


hCaptcha Support

This PR aims to add support for hCaptcha
Related to #12582, but may not completely close it.
Draft for a few reason.

Our reCaptcha implementation is....interesting. It appears to unmarshal the entire response, but only uses Success
Should hCaptcha follow this practice as well?

Do we want to use the same approach as reCaptcha and just running a simple POST, or use a library such as shameless self-plug?

Option 2 makes sense, perhaps we could also use the lessons learned from your library and apply them to the recaptcha one.

@zeripath Done. :slightly_smiling_face:

Golangci-lint demands a sacrifice...

I'd argue this could go into 1.13 as it's ready to be merged

A few things to note with the latest changes...

challenge_ts was sometimes returning a timestamp that Go choked on parsing, so both hcaptcha and recaptcha now just marshal it to a string. We weren't using the timestamp, but if it was needed in the future we can look at parsing it then.
There was a potential slice index issue that has been fixed because a successful check meant the ErrorCodes slice was empty.


Documentation base structure

Create the Basis so that others  can write documentation.
The plan is to host documentation on readthedocs.org and build it with sphinx.
Therefore documentation will be written in restructured text.
A good example is http://docs.pyroute2.org/usage.html
If one clicks on show source, one can easily learn how to write docu.

Done.
See http://miniworld-core.readthedocs.io/en/documentation/contribute/documentation.html


fix: enable wallet on connect

while calling connect() I think the wallet should actually call .enable() on the underlying wallet to complete the wallet object

we'll probably add this behavior in future iterations, closed for now


See also link misleading

The link for - Manage the Site Collection App Catalog - is poorly named. It sounds like it is speaking of site app catalogs which are related to this document. That article is talking about creating the app-catalog site collection for the tenant/organization.
Please consider changing the name of the link, and possibly the article link name:
...article/Manage-the-Site-Collection-App-Catalog-...
Possibly it should say Manage App Catalog Site Collection, or Manage Organizational App-Catalog. We need to drive home the differences between site (or site collection) app-catalogs, and the central app catalog site collection for the org.
Category

[ ] Content fix
[ ] New article
[x] Example checked item (delete this line)

For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
(DELETE THIS PARAGRAPH AFTER READING)

Related issues:

fixes #issuenumber
partially #issuenumber
mentioned in #issuenumber

If this fixes (aka: closes) or references an issue, please reference it here. This helps maintaining the issue list as it will (1) link the PR to the issue & (2) automatically close the issue when this PR is merged in.
(DELETE THIS PARAGRAPH AFTER READING)

What's in this Pull Request?

Please describe the changes in this PR. Sample description or details around bugs which are being fixed.
(DELETE THIS PARAGRAPH AFTER READING)

Guidance

Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
Please target your PR to 'master' branch. Released documents are in live branch.
(DELETE THIS PARAGRAPH AFTER READING)

Couple things - the link is apparently retired and redirected to an article with a different title. The link should either be removed or updated to use the title & URL of the final destination.
Also, please the sections in the new PR template provided when creating a new PR. It looks like you ignored it... this is added and required for all PRs.


Please verify azure search service is up, restart the WebApp and try again

We are trying to deploy a KB on production and QA (Jenkins) and we are getting this error:
"error": {
"code": "BadArgument",
"message": "Please verify azure search service is up, restart the WebApp and try again"
On Dev environment works fine. We restarted the service, but it fails again.

hi @Dubawi
Can you please link to the documentation you were following when you received this error?

Also; does the KB work in the QnA portal?

Hi @dmvtech, I don't have documentation about this error, we can deploy using pipelines only. In dev environment, the KB works fine.
When the pipeline executes this line:

we get this:

The service is on (started) and we restated it before running the script, but we got the error,


Implements Oracle RENAME oldTable TO newTable Statement

Fixes #1253

Coverage decreased (-0.2%) to 88.515% when pulling 3943591c80d8e19ea389c270275f3d9b65e5b8ac on manticore-projects:RenameTableStatement into f85b4b630008d9ecbf707b534336460784ed6ea6 on JSQLParser:master.


Revert "Disable docker releases"

Reverts anchore/syft#493
Testing the docker install process on a fork shows that this issue has been resolved. There is the larger issue of this occurring again in the future --it would be ideal to solve this by removing the dependency on docker-practice/actions-setup-docker@v1 and docker desktop for mac in general. (Migrating to builds via podman + qemu?)

It looks like it has https://github.com/docker-practice/actions-setup-docker/commit/51fd5009666ea5e3382fe125abf5d73d31386b59 a few days after the problem was noticed on our end. I tried something like this with a different brew formula commit without luck.


:) Enhancement of watching auto

@shiye515
Hi. Here i want to change the option of "auto" then using a button to control the carousel if auto play.
So I added new watch function at Carousel.vue (line to 76)
auto () {
   this.setTimer()
}

It works  perfectly !

Thanks for advice, and i merged your code in version 1.0.6


[FR] Plant sensor implementation

🚀 Feature Request
Flower sensors/plant sensor implementation

Awaiting future improved & reliable sensors


Using NSwagStudio with a relative Output file path

How would I use a relative output filepath in NSwagStudio.
Since the solution is in source control - each developer could have a different location for their output file.
I tried using a relative path syntax and cannot get it to generate.
I would expect it to be relative to the .nswag config file

The output paths are stored as relative pathes in the document but displayed ansolute in the ui

Graying out the relative part would be helpful


Manage other (in-person or virtual meeting), recruited via community organization

Manage engagement - empty: https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=3326%3A102887
Manage engagement -filled: https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=3326%3A102917
Add CO:
https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=2718%3A71173
https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=2782%3A73645
https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=2782%3A75350

If this engagement wasn't using the CO recruitment method, and instead used any other (matching service, open call, CC) it would require the FRO to add engagement details. The page would look like this: https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=2730%3A74255


Navbar: version 2

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
Version 2 of the Navbar component.
Includes:

navbar-profile component
mobile responsiveness
mobile sidebar

📦 Published PR as canary version: 21.0.0--canary.941.fe4968516b56dfb2bb1fa26128b9151ec21980b1.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

There is some spacing to the left of Menu Item 1.
Is this desired?

There is some spacing to the left of Menu Item 1. Is this desired?

No. Can you describe how to reproduce it?

@tishoyanchev Can you remove the menu items for the user? or else they are hidden in the mobile menu

I removed the sub menu items in the User menu. The first layer menu items are not hidden in the mobile menu. The profile menu should have items inside. That's the purpose of the menu.


Reusable Workflows

Shamelessly pulled from the GitHub Docs:
Rather than copying and pasting from one workflow to another, you can make workflows reusable. You and anyone with access to the reusable workflow can then call the reusable workflow from another workflow.
Reusing workflows avoids duplication. This makes workflows easier to maintain and allows you to create new workflows more quickly by building on the work of others, just as you do with actions. Workflow reuse also promotes best practice by helping you to use workflows that are well designed, have already been tested, and have been proven to be effective. Your organization can build up a library of reusable workflows that can be centrally maintained.

@gduncklee @hkeeler
Is there more work to do on this ticket or can it be closed?


anonymous access

Is it possible to get post information, from a public page such as this:
https://www.instagram.com/p/CT-cnxGhvvO/
without login?

It is possible with one way for sure, using the web api instead of the mobile API. I don't think you can do it on the mobile API as you have to be authenticated to use the app whereas in the browser you can check public posts without authenticating.
I think the best solution would be to define a new AnonymousClient-Type where you can retrieve info without logging in

Closing this in favor of my own package:
https://github.com/89z/mech/tree/master/instagram


Date fixes

Actually attempt to fix the dates to keep local time (MTN) for posts. This should also fix the issue with the RSS feed to show the correct time and date.
Will close #25

Also fixes sitemap missing trailing slashes on urls