Implementing From<&mut AsciiStr> for &mut str and &mut [u8] is unsound

They allow writing non-ASCII values to an AsciiStr which when read out as an AsciiChar will produce values outside the valid niche.
These impls were added by me in 4fbd0503, so 0.9, 0.8 and 0.7 are affected.
Here's an example using these impls to create out-of-bounds array indexing in safe code (when compiled in release mode):
let mut buf = [0u8; 1];
let ascii = buf.as_mut_ascii_str().unwrap();
let byte_view = <&mut[u8] as From<&mut AsciiStr>>::from(ascii);
let arr = [0b11011101u8; 128];
byte_view[0] = 180;
assert_ne!(arr[ascii[0] as u8 as usize], 0b11011101);

I don't see any good way to tell users of the crate to stop using these impls:
Deprecation notices on trait impls are ignored (by both Rust 1.38 and Rust 1.9).
Changing the impls to panic or return an empty slice could break working code (that never writes non-ASCII values) at run-time.
The only fix we could make appears to be to remove the impls, telling users of them to do the unsafe pointer casting explicitly. On one hand this will make any accidental users of it aware of the problem when they update Cargo.lock, but it will also break any use that happened to be OK with a minor release, and any reverse dependencies of these uses.
On the other hand doing nothing and hoping nobody accidentally uses these impls feels irresponsible. What do you think @tomprogrammer?
In any case I don't think we need to fix 0.7 and 0.8, as Rust didn't backport the security fix in 1.29.1 to previous affected versions.

I don't see any good way to tell users of the crate to stop using these impls

You can file a security advisory at https://github.com/RustSec/advisory-db
This will alert anyone running cargo-audit that they're using a vulnerable version and need to upgrade. You can even include the specific functions that need to be called to be affected by the issue.
Out-of-bounds indexing in safe code is usually an exploitable vulnerability, so I would strongly recommend filing a security advisory.

I think Shnatsels idea is good, we should file an advisory. I've yanked the affected versions already.

I've filed advisory, so this can be closed now :)

Out-of-bounds indexing in safe code is usually an exploitable vulnerability, so I would strongly recommend filing a security advisory.

@tomprogrammer hi, I am confused about the purpose of PoC provided here.
arr[ascii[0] as u8 as usize]

I understand that here ascii[0] should not have value larger than 127. However, in your PoC, the length of arr is user-defined. You can also define arr with an arbitrary length less than 127. In this case, even though ascii[0] can only ascii value, you still can trigger OOB above.
Therefore, I can't understand why your PoC can prove the unsoundness. Could you give some more explanation, thanks?

@shinmao:
If the length of the array that is indexed into is less than 128, then also ascii values can be out of bounds and the compiler has to insert a check that the index is within bounds. That check also catches the invalid values above 127.
(And for arrays longer than 255, all byte values are valid indexes, so the invalid ascii values are also within bounds.)
@KisaragiEffective:
Thanks.

@tomprogrammer thanks for the response. The reason why I feel confused is that this OOB case could never be checked by compiler statically. For example,
let arr = [0u8, 1, 2, 3, 4, 5];
println!("{}", arr[ arr[4] as u8 as usize ] );

This code can be compiled and have a runtime panic because compiler couldn't understand the value of arr[4] statically. and we won't say that arr has UB because it allows OOB indexing in safe code here.
However, maybe this is the only way to show unsoundness here?

In your example there is no OOB indexing or UB because the inserted check will panic and prevent the OOB indexing from occurring.
But in the ascii example, the check is optimized out which lets OOB indexing actually happen.
Why the compiler is allowed to optimize out the check:

AsciiChar is an enum whose variants all have discriminants /values below 128.
When the value from ascii[0] is cast to u8 and then usize, the compiler remembers that it is coming from an Asciichar and therefore will never be more than 127.
(While values up to 255 are physically possible, those occurring would already be UB and can therefore be assumed never to happen.)
The length of arr is greater than 127, so all valid AsciiChar values are valid indexes.

In my example the unsoundness happens in the From impl on line 3, the UB happens on line 5, and line 6 just shows that the UB can be used to make bad stuff happen. Putting the value into an Option<AsciiChar> as Some but getting None back might also show it, but would depend on how the compiler implements the variant checking.
I hope this explains it.
(And by the way, I'm @tormol, not tomprogrammer)

@tormol sorry I ping the wrong person hahaha
Thanks for your clear answer. It really helps solve my question!


Only allow USDT (default) or NEAR for solution funding requests

These days, DevHub can only provide funding in USDT or NEAR, so let's update the Solution funding request form with USDT (default) and NEAR.
Other notes:

bridged USDC is also available, but it is more cumbersome
fiat USD / EUR / CHF are available for very limited cases

@ori-near I will need your call here.

@frol could you please elaborate on issues with USDC.e?

@carina-akaia The legal team does not hold USDC.e and waiting for native USDC to arrive to avoid people getting confused.

Hi @frol – thanks for creating this ticket. I agree that we should simplify the solution funding options. Let's do USDT (Default) and NEAR. Note – right now there's a bug that won't even allow me to select the existing USDC option ... so whoever works on this ticket should make sure that both currency options are supported.

I can pick this one up @ori-near

@ori-near I've opened a PR ready for review: #222

@elliotBraem Looks great! Thanks so much for linking to the test cases of the AC too. Super helpful.


🐛Bug: Builder option for select does not work when using preprocessor

Describe the bug
A clear and concise description of what the bug is.
When using Select component, there is the option builder
{...$option({ value: category, label: category })} use:option
You put that line of code above to make a div a select option, but if you use the preprocessor use:melt from the docs
use:melt={$option({ value: item, label: item })}
it does not work, its null

By the way, this is in the latest version 0.47.6

Oh nvm, forgot to setup preprocessor lol


Seektime method is not working in iOS devices

In iOS devices it will start the video all over again, whether it works fine on android devices.
Is there any way to fixed this issue ?
Basically what we want, we have to sync media's current time to next state where we open the same media file.It perfectly fine on desktop browser and android mobile browser.But it has an issue with iOS device browsers.For first few second it shows the previous state updated time, but immediately it reset time to 0 and play video/audio all over again from start.
we are using angular ui-router for changing the state.

A plunkr or codepen with working example would be good to help you in this issue.

Here is example - http://codepen.io/eldarzakiryanov/pen/OXAPwb/
It just seekTime by 10 seconds. But unfortunately codepen.io doesn't work on Safari.
I double checked seekTime doesn't work on Safari both on MacOS and iOS.
My temporary solution is to detect browser Safari and add seconds to end of source file #t=20
For example: http://localhost:8080/file/movie.mp4#t=30

I have the same issue using Videogular with Ionic v1 project.
After executing seekTime on iOS device, it resets time to 0. Everything works fine with Android.
Any suggestions?


Set a database template as default

What makes the feature necessary?
When adding new items to a database where the template used is always or most often the same it is somewhat cumbersome to click multiple times to apply the template. Having the "New" button apply the set default template automatically would benefit a smoother workflow.
How would the feature work?
Preferably a toggle option, named "Make default" or something, inside this submenu

Then just click "New" and presto!

out of scope, the notion-enhancer can't override notion at that level

I see! That's too bad. :slightly_frowning_face:


maven: 3.5.3 -> 3.5.4

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/apache-maven/versions.
meta.description for maven is: '"Build automation tool (used primarily for Java projects)"'.

Checks done (click to expand)

built on NixOS
/nix/store/xhvfkghazvv8ynwawm7gidq8mpljn4yl-apache-maven-3.5.4/bin/mvn passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 3.5.4 with grep in /nix/store/xhvfkghazvv8ynwawm7gidq8mpljn4yl-apache-maven-3.5.4
directory tree listing: https://gist.github.com/488aeb1ef6f60137dd3f8bf047640dea
du listing: https://gist.github.com/2a2fc45de955c262570eea4898060a22

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout df126e2adf8b43a62a2dd8ee598002438afcf88e && nix-shell --pure -I nixpkgs=. -p maven

cc @cko for testing.

Success on x86_64-linux (full log)
Attempted: maven
Partial log (click to expand)

trying http://www.eu.apache.org/dist/maven/maven-3/3.5.4/binaries/apache-maven-3.5.4-bin.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 8635k  100 8635k    0     0  6051k      0  0:00:01  0:00:01 --:--:-- 6047k
building '/nix/store/r96fyxhd228cnw0f5lm314jgr5ln5amw-apache-maven-3.5.4.drv'...
unpacking source archive /nix/store/qp5mfwk4i3lvjzdz3df5vl43ciqhpfh6-apache-maven-3.5.4-bin.tar.gz
source root is apache-maven-3.5.4
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file apache-maven-3.5.4/lib/slf4j-api.license
/nix/store/xhvfkghazvv8ynwawm7gidq8mpljn4yl-apache-maven-3.5.4

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

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

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


Webpack gives error while bundling modules having "main", "module" or "js:next" attributes set to "." in their package.json

Let us assume I am using a module named do-something in my package.json as a dependency. This module has package.json with entry attributes like
"main": ".",
"module": ".",
"jsnext:main": ".",

While doing bundling for this module through webpack, I get this error
Module not found: Error: Recursion in resolving
I am in an assumption that, the "main": "." attribute path will resolve to "main": "index.js"
I am suspecting webpack is calling do-something module folder again and again.
I investigated more and it is coming from enhanced-resolve package that webpack uses for resolving paths. I use webpack major version 2 "webpack": "^2.4.0"
But I have tried with the latest webpack version as well, it still does not work. Is there a way in webpack to bundle these kinds of dependent modules?

@webpack-bot move to webpack/enhanced-resolve
Looks like a bug.


make:server error "The given data failed to pass validation."

Trying to spin up a Linode and get:
In MakesHttpRequests.php line 90:   The given data failed to pass validation.
With these settings:
forge make:server --provider=linode --credentials=12345 --region=london --ip=<IP_ADDRESS> --private-ip=<IP_ADDRESS> --php=php71 --database=mysql

Hiya James!
The --ip and --private-ip options are only required when using --provider=custom. Can you try without those two and see if this helps?
(also, I assume you've already added Linode credentials and are using the right ID in the command there? 🙂)

Thanks! I just tried it like this but get the same error?
`forge make:server --provider=linode --credentials=12345 --region=london --php=php71 --database=mysql```

Looks like you need to use --region=7 when creating a server on Linode:

Also; can you run forge credentials? Are you using the ID of the one where the "type" is Linode?
This package should really do a better job validating the request that's being made; I'll put that on my todo list 😄 For now though, you can see most of the valid options in Laravel Forge's API documentation here.

No worries, I wouldn't expect users of this package to go looking at some first-party API documentation honestly 👍
Did you get it working? If not, I'll do some further digging to get this figured out 😄

Can you remind me what forge.json should look like if I'm manually creating the file?

Ok time for an update! It's still not working I'm afraid, same error from this:
forge make:server --provider=linode --credentials=XXXXX --region=7 --php=php71 --database=mysql
where XXXXX matches an ID from forge credentials with type linode

@j-greig I know this is an old issue, but the problem I had was that the size and name parameters were not set up correctly.
I've submitted a PR which addresses this: #44

Closing because I am abandoning this package in favor of laravel/forge-cli.


Add base class in lib.scripthost.d.ts for Automation objects

TypeScript Version:  2.5.0 / nightly (2.6.0-dev.20170902)
Code
// in lib.scripthost.d.ts
class AutomationObject<T> {
  private constructor();
  private typekey: AutomationObject<T>;
}

// in activex-excel.d.ts
declare namespace Excel {
    class Application extends AutomationObject<Application> {
        Workbooks: any;
    }
}

interface ActiveXObject {
    new(progid: 'Excel.Application'): Excel.Application;
}

// usage
let x: Excel.Application = new ActiveXObject('Excel.Application');

This currently wouldn't compile, because of the private constructor in the base class; but pending resolution of #18283, it would be possible.
This would prevent assigning structurally matching objects to the inheriting types:
// Compiler error
x = {
    Workbooks: [];
};

and also prevent using new to create a new instance of the inheriting type:
// Compiler error
x = new Excel.Application();

I've opened an issue to define similar behavior for VarDate and SafeArray<T> in lib.scripthost.d.ts. If AutomationObject<T> is defined, then it could be used in lib.scripthost.d.ts as a base class for these types as well:
declare class VarDate extends AutomationObject<VarDate> { }
declare class SafeArray<T> extends AutomationObject<SafeArray<T>> { }

On further consideration, this suggestion would have value for a flat inheritance hierarchy, where all the declared classes inherit directly from AutomationObject<T>.
However, when one declared class inherits from another, inheriting from AutomationObject<T> would not prevent assigning an instance of the base class to a subclass typed variable or parameter:
declare class BaseAutomation extends AutomationObject<BaseAutomation> { }
declare class DerivedAutomation extends BaseAutomation { }
let x: BaseAutomation = ...
let y: DerivedAutomation = x; // both have the same private key, so it's a valid assignment

In the real world, this might not be an issue, because DerivedAutomation would probably have some additional members that would prevent this assignment.


Feature request: overwriteMode on unique indexes without _key

My Environment

ArangoDB Version:  3.9.2

Component, Query & Data
Affected feature:
Inserts with overwriteMode other than conflict
AQL query (if applicable):
LET data = [
   {
      "_from":"characters/NedStark",
      "_to":"characters/RobbStark"
   },
   {
      "_from":"characters/NedStark",
      "_to":"characters/SansaStark"
   },
]

FOR item IN data
INSERT item IN relationships OPTIONS {overwriteMode: "update"}

Dataset:
[
   {
      "_from":"characters/NedStark",
      "_to":"characters/RobbStark"
   },
   {
      "_from":"characters/NedStark",
      "_to":"characters/SansaStark"
   },
]

Steps to reproduce

Create a relationship edge collection
Set a unique persistent index on [_from, _to]
Insert above data twice with overwriteMode set to anything but conflict.

Problem:
In the above situation I don't want duplicate edges. However _key isn't set in the data. Thus overwriteMode doesn't kick in.
And the query execution throws a duplicate error. (as expected from docs)
Expected result:
For overwriteMode to work regardless of _key presence.
When the unique index triggers a match overwriteMode should kick into action.

That would make the insert operation significantly more expensive since we would have to check all indexes. However, you can achieve this yourself by using UPSERT.
FOR item IN data
  UPSERT { item._from, item._to }
  INSERT item UPDATE {} IN relationships

I think this should do exactly what you want. UPSERT item inserts a subquery the looks for a document matching the one specified. This subquery is optimized like any other (i.e., it will use appropriate indexes), but is also able to observe previous inserts performed within the same query. If no matching document is found we insert it, otherwise we perform an "empty update".


added option to enable-disable cache

type(scope): if this commit is applied, it will...
i.e. fix(login): [more detailed message]
Why was this change made?
Feature request? A bug was found? Doing some refactoring? Let us know why this should be in our codebase.
Links to any relevant tickets, articles, or other resources
closes https://github.com/SchemaApp/SchemaApp/issues/12813
Other Notes
Did you fix any additional issues or do you have special notes for the reviewer? Put them here.
Suggested Test for Reviewer

Login to Schema app dashboard
Go to
Do action
See result

Label "Dispatcher Cache Purge"
Tooltip "If enabled page updates will clear dispatcher cache within 30 minutes. If disabled markup is updated but the cache is not purged and only used when there is another cache clearing mechanism"

Like Marks phrasing, go with it.

We were waiting until the installer problem was resolved, and we're fairly sure we identified that problem. Would be ok to merge and release.

Checked with @nikhilajudiya on this and with our changes this will require some retesting. He will take that on.


[rozhlas.cz] Unsupported entity type "serialPart";

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
EU, USA
Provide a description that is worded well enough to be understood
Most URLs work, some specific ones throw Unsupported entity type "serialPart"
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', 'https://www.mujrozhlas.cz/povidka/gustavo-adolfo-becquer-hora-duchu']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-master-builds [e28e135d6] (linux_exe)
[debug] Python 3.10.13 (CPython x86_64 64bit) - Linux-6.1.0-18-amd64-x86_64-with-glibc2.36 (OpenSSL 3.2.1 30 Jan 2024, glibc 2.36)
[debug] exe versions: ffmpeg 5.1.4-0 (setts), ffprobe 5.1.4-0, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2024.02.02, mutagen-1.47.0, requests-2.31.0, secretstorage-3.3.3, sqlite3-3.45.1, urllib3-2.2.1, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1833 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-master-builds/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-master-builds
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp-master-builds)
[MujRozhlas] Extracting URL: https://www.mujrozhlas.cz/povidka/gustavo-adolfo-becquer-hora-duchu
[MujRozhlas] gustavo-adolfo-becquer-hora-duchu: Downloading webpage
ERROR: [MujRozhlas] gustavo-adolfo-becquer-hora-duchu: Unsupported entity type "serialPart"; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "yt_dlp/extractor/common.py", line 732, in extract
  File "yt_dlp/extractor/rozhlas.py", line 343, in _real_extract

diff --git a/yt_dlp/extractor/rozhlas.py b/yt_dlp/extractor/rozhlas.py
index 63134322d..52f9748ba 100644
--- a/yt_dlp/extractor/rozhlas.py
+++ b/yt_dlp/extractor/rozhlas.py
@@ -322,7 +322,7 @@ def _real_extract(self, url):

         entity = info['siteEntityBundle']

-        if entity == 'episode':
+        if entity in ('episode', 'serialPart'):
             return self._extract_audio_entry(self._call_api(
                 'episodes', info['contentId'], 'episode info API JSON'))


Working on code and prose at the same time?

I tried your theme and suggestions, putting all the config in a function to be used as hook as below.
However, all these settings still seem to be applied globally, and not just in the org-mode/markdown files.
I'm not actually sure it's possible at all to jump between one and the other just depending on the mode, I wonder if I should maybe just fire up another instance of Emacs with these settings when I just want to work on documentation for example.
Or are you using these settings also to work on code?
Thanks for the great suggestions!
(defun writer-mode ()
  (variable-pitch-mode 1)
  (set-face-attribute 'default nil :family "Iosevka" :height 130)
  (set-face-attribute 'fixed-pitch nil :family "Iosevka")
  (set-face-attribute 'variable-pitch nil :family "Baskerville")
  (olivetti-mode t)
  (typo-mode t)
  (blink-cursor-mode t)
  (linum-mode nil)
  (load-theme 'poet-dark))

(add-hook 'org-mode-hook 'writer-mode)
(add-hook 'markdown-mode-hook 'writer-mode)

Hmm, unfortunately the face attributes, and theme settings are not buffer local and will apply globally. While working on both prose and code I just have the fonts behave differently, but the theme stays the same  – variable pitch mode only in any mode deriving from a text-mode (text-mode-hook), while standard behaviour for the programming fonts.
I vaguely remember seeing a mode that also makes a theme buffer local, but I haven't tried it out or tested it.

I see, well I think I'll experiment with 2 different instances of Emacs running maybe, one for coding and the other for reading and writing documentation.
Thanks

It might be worth to mention that in the README, what do you think?

Fair, I'll reopen and update the readme.

Looking at this again: the global settings are not directly related to poet, I've also removed references to them because I don't use them anymore. Closing this out.


Is there a way to format a column  based on the value of another column?

Category

[X] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Hello,
I am building a list [of actions] in Share Point Online.
Is there a way to format a column (let's assume it is called "Action Item" ) based on the value of another column (let's assume is called "Owner")? For example, is there a way to format in bold characters the content of the "Action Item" column is the value of the "Owner" is @me? (or another string).
Thanks in advance,
Santiago.

Yes! This is covered in the documentation here: https://docs.microsoft.com/en-us/sharepoint/dev/declarative-customization/column-formatting#detailed-syntax-reference under the "[$FieldName]" section.
There are also several samples of doing this:

Example of using the @me: Highlight the current user
Example of formatting based on another column: Date Range Format

Fantastic. Many thanks.


SVG path and cutting time optimisation

As a longer term goal is there any chance that we can have code to optimise vector path operations. The optimisations I can think of are:
A. When creating the original path list (or when you execute the engrave / cut) deduplicate paths or part paths which are identical - some SVG files have, unbeknown to the user, duplicate paths (which is entirely different to when the user has deliberately created duplicate paths within Meerk40t), and these get unexpectedly doubly engraved / cut. The two use cases I have seen are entire paths which have been duplicated, and e.g. rectangles which are adjacent and share a side (partial or full).
Clearly deduplicating part paths is more complex than deduplicating identical full paths - if this is too complex or time-consuming, then you could perhaps limit this to straight line segments more than a minimum length. Coincident straight lines with different start and end points can be compared for coincidence by checking the following easy algorithm: slope angles the same, at least one end point of one line within the box made by the end points of the other line, that particular end point is on the other line - these are all extremely simple calculations and comparisons. Ask if you think a diagram can help.
B. Merge two paths into one where a first path has the same finish point is the same as either the start or finish point of another path. In other words rather than cut a continuous border which happens to be several segments out of order, try to merge them so that they are cut as one continuous burn.
C1. For vector engraves, start with the path with a start or end point nearest the origin (reversing the direction of the burn as necessary). For the next burn select the path with the nearest start or finish point to the end of the first burn. etc. This will minimise the distance of head moves.
C2. For cuts use a more complex enhancement of this algorithm which tries to do small cuts first and bigger cuts later i.e. cut the inside cut-outs first and cut the outside of the piece second. However you do not want a literal small-to-large cut sequence because that would take the head all over the place, so you might want an algorithm that e.g. sequences the paths by length, and then collects them into groups of approximately similar size (within a multiple of the smallest path length of the group) and then apply C1 to each of these groups.
I believe that these optimisations will reduce laser elapsed time, and improve vector engraving / cutting quality.

Yeah, the current work for the vector path is a bit strange. The instructions exist for the 0.6.x version:
https://www.youtube.com/watch?v=9DQpuke8j_w
There were a couple others in there but you can see the deficits pretty clearly. They require things be merged and they lose some stuff and look weird if they have a fill, #319. This does the main obvious optimization of reducing travel. In some cases the duplicate path would be a desire to go over that area twice. For some cuts it would be proper to remove the duplicated edge. If edges are really close they often get made to overlap with optimize travel and the routine won't stop there.
B. Gets a little more complex when there would be say 3 points that are coincident there. Since it gets a bit weirder.
C1. This doesn't actually minimize the distance the head moves. This is actually a pretty naive solution to the travelling salesman problem and generally results in a reduction of something like 30% or so. When a more properly optimized path drops by about 1/10th the previous path. You can fiddle with optimize travel a little bit. I don't have an obvious version of the latter one but you can see it sometimes has longer moves since it optimized all the moves together as a group.
C2. The general rule is you have sub criteria that fit into a larger set of constraints and heuristics. So cutting inner first is constraint since on the cut planning but the heuristic would be to order it such that it reduces head travel. So you make two-opt switches that minimize the heuristic. So if you want to reduce angle changes which is a thing some plotplanning does, and reduce travel you need to specify how much is a directional shift worth vs. an angle shift and then minimize that value.
Two-opt is an amazing algorithm for that stuff and works well with heuristics and can be made to work with constraints as well.
https://www.youtube.com/watch?v=w1hG1wv8tS4 <-- See lightburn's cut planning.
This set of routines is planned in part with a phase in jobs. In 0.7.0, however it's not done yet, and another feature that is somewhat required for this is also missing, namely preview. It's one thing to optimize things, it's another to show the end-user what was changed and what the current path is. Without doing the latter the former is a bit problematic. Though I'm likely going to do optimize travel as a step even without the preview. Which I think I labeled Project Foresight, but pulled some of those aspects since I wanted a better version of 0.7.0 published sooner. I think I got most of the stuff in there but only published a few buggy betas.

C1. I watched the first video and this looks great for vector engraves!! And if there is a GUI-driven or automatic implementation in 7.0 that will be exactly what I am asking for. And yes - travelling salesman is really what is needed.
My only issue with the first video (aside from having to use Terminal which we both understand) is the need to merge and split the paths to make it work.
C2. I noticed from the video that you have an Optimize cut_inner - but haven't found a video demonstrating that from the terminal. But I am guessing that is what I am asking for as well - or rather Optimize cut_travel. But the second Lightburn video illustrates my desires very well. And it sounds like you have it planned - though it is a big job.
(In the end, performance and GUI optimisations are just nice-to-have - Meerk40t seems to have all the basic functionality I need so far to do what I need - yes there is scope for functional improvement, but if I was going to prioritise it would be to improve the documentation - and that is something that I can easily help with.)
If you want help with testing specific parts of 7.0 please ask (though my time availability will be a constraint - who would think that retirement could be so busy even during a lockdown).

A couple of things I spotted:
a. I manually sorted the paths and yet a closed curve was not printed continuously because some segments were in the wrong direction. So an easy fix is to check each segment immediately before cutting / vector engraving and reversing the direction if that fixes it.
b. I tried the terminal optimisation and it created duplicate cuts somewhere in the merge / optimise cut_inside / subpath commands. I checked that the original did not have duplicate cuts. I will try to create a small test example with an easily counted set of paths if I get the chance and see if I can narrow this down.

Most of this has been implemented so I am closing it.
Path simplification is a whole separate issue to burn optimisation.


adding step to combine coverage

Pull Request Test Coverage Report for Build 306941391

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 95.858%

Totals

Change from base Build 286947600:
-0.07%

Covered Lines:
2106

Relevant Lines:
2197

💛  - Coveralls


Version number in min.js file

I have no idea which version I installed last time. All I know is that bower found and downloaded version 1.2.2.
It would be awesome if the version number could be inserted as a comment on the first line of the dist/layzr.min.js.
Autocompeter does this using gulp-header.

going to do some surveying of other prominent JS libs (not to say that mine is prominent haha) and try to follow the industry best practice on this one.
if you have some examples of other libs that include a comment indicating version or otherwise in the minified file, i'd appreciate you linking them. thanks for the gulp plugin recommendation as well

Other libs other than autocompeter?
I don't think it matters. Where you get the version number from might vary but that gulpfile is a great example of how you're supposed to get it in there.

Oh, another thing you should consider is to NOT work on a file in the dist/ directory. That directory is usually meant for computer generated things.
In Autocompeter I work on the files in src/ and my gulpfile.js creates the files in dist/.

what i was asking is: do major js libraries include version numbers in their minified versions?  i'd like to follow their lead, was what i was saying. based on what i'm seeing, it looks like they do, and so i've added this to the roadmap.
totally see your point about the src folder. an added benefit would be using something like gulp-umd to add the UMD wrapper, to keep that ugliness out of the source file. both changes coming soon, thanks dude

Examples:

jquery
angularjs
localforage

Yeah, I didn't know about gulp-umd but that's a really good point!

yep, seems to be standard. thanks man, will update you when i make some changes later today

@peterbe fixed the repo folder structure (added the src folder, dist folder is only processed output) in the new release.
part of this fix was adding a comment to the top of both dist files that includes version number. thanks again for the help bud


Task/ios load balancer

Soundtrack of this PR: link to song that really fits the mood of this PR
Motivation
load balancer task
In this PR

adds url load balancing support

iOS Implement Internal Load Balancer
Future Work

Additional tests for load balancer logic
Adding protocol abstraction for load balancer impl

Im having concern that were managing the rotation to far into the code-path for "creating" a new connection, and this could cause mismatch because we have to manage multiple downstream usage points.
One of my original ideas was to put the rotation in HttpProtocolConnectionFactory/GrpcProtocolConnectionFactory above the init for each particular connection. We know 100% this code path only gets called once each time a new connection is created.
Also, we could make this change lower impact by passing the RandomLoadBalancer in as a separate parameter from the config object. So the config remains mostly the same as it is now and we don't have to propagate all the changes in at each of its usage sites.
Ill work on a prototype implementation for my idea

suggested changes here - https://github.com/mobilecoinofficial/MobileCoin-Swift/pull/115

Closing this PR to replace with new PR after squashing to rebase.


runtime error

the error log:
2016-12-07 08:46:45 libevent: getaddrinfo: address family for nodename not supported
2016-12-07 08:46:45 Binding RPC on address :: port 18332 failed.
2016-12-07 08:46:45 Binding RPC on address <IP_ADDRESS> port 18332 failed.
is the libevent version low than needed?

Currently 0.13-dev uses port 18332 which bitcoind uses, maybe you have a bitcoind running.
You can try branch https://github.com/litecoin-project/litecore-litecoin/commits/0.13-dev-rebased,  which was rebased from https://github.com/thrasher-/litecoin/commits/0.13-dev


Calling multiple sync actions in order applies only first one

Hi,
I tried to create a mapping that would do 3 things for me at once:

Clear unused imports
Import missing imports
run fixAll

Mapping looks something like this:
  vim.keymap.set('n', '<F1>', function()
    local ts = require('typescript').actions
    ts.removeUnused({ sync = true })
    ts.addMissingImports({ sync = true })
    ts.fixAll({ sync = true })
    -- Also tried this with the bang, same thing
    -- vim.cmd('TypescriptRemoveUnused!')
    -- vim.cmd('TypescriptAddMissingImports!')
    -- vim.cmd('TypescriptFixAll!')
  end)

When I invoke it once, it does only first thing, then if I do it again, it does the second. This means that the functions are not really full sync, but this may be due to the lsp, I'm not sure.
If I add some timeout between the calls, it works, but only with a fairly big timeout (~1000 ms). For example:
  vim.keymap.set('n', '<F1>', function()
    local ts = require('typescript').actions
    ts.removeUnused({ sync = true })
    vim.defer_fn(function()
      ts.addMissingImports({ sync = true })
    end, 1000)
    vim.defer_fn(function()
      ts.fixAll({ sync = true })
    end, 2000)
  end)

Using something like 100ms doesn't work. I assume this depends on the server speed and project size.
Steps to reproduce:

Clone https://github.com/neoclide/coc.nvim
Run npm install
Open up src/attach.ts
Update line 7 to this: import { objectLiteral, vimCompleteItem } from './util/is'
Remove line 5: import Plugin from './plugin'
Press <F1>, it only removes vimCompleteItem
Press  <F1> again, it imports the Plugin class

If you set debug = true in this plugin's options and set your Neovim LSP log level to "debug", you can see that the requests are getting sent with the correct parameters, but for some mysterious reason, the second response is empty. Only the first request ever receives a proper response.
I've seen similar issues before, so I went to investigate, and I suspect this logic in typescript-language-server is related. Applying the edits from the first request triggers a diagnostic refresh, which is still in progress when the second request is sent. Since the code that adds source actions to the response is skipped, the response Neovim receives is empty.
This hypothesis is supported by the fact that this somewhat ugly workaround seems to do the trick 100% of the time:
-- add to on_attach
vim.keymap.set("n", "go", function()
    local ts = require("typescript").actions
    ts.removeUnused({ sync = true })
    api.nvim_create_autocmd("DiagnosticChanged", {
        buffer = bufnr,
        callback = function()
            ts.addMissingImports({ sync = true })
        end,
        once = true,
    })
end)

If I'm right, there's not much we can do about it here. The source code indicates that an upstream fix may be possible, but I imagine it would take some work to investigate and implement it.


Focus outline around extensions list as soon as I select an element

This is not happening in stable:

Themeing?

@joaomoreno probably, it does not happen for the tree though.


<11.0.1>   [YOUTUBEJS][Parser]: Error: Type mismatch, got FlexibleActionsView expected Button | ButtonView | SegmentedLikeDislikeButtonView.

Steps to reproduce
const lib = await innertube.getLibrary()
always throws this warning:
[YOUTUBEJS][Parser]: Error: Type mismatch, got FlexibleActionsView expected Button | ButtonView | SegmentedLikeDislikeButtonView.
Failure Logs
[YOUTUBEJS][Parser]: Error: Type mismatch, got FlexibleActionsView expected Button | ButtonView | SegmentedLikeDislikeButtonView.

Expected behavior
works without warnings
Current behavior
it's working but with warnings
Version
Default
Anything else?
No response
Checklist

[X] I am running the latest version.
[X] I checked the documentation and found no answer.
[X] I have searched the existing issues and made sure this is not a duplicate.
[X] I have provided sufficient information.

Don't worry about Parser warnings like this, because all it means is that YouTube is testing out some layout change(which they do all the time). As long as you have the data you want, it's not a problem.


Fix resource allocation

security-token-service is currently using way less than it is requesting
https://console.nav.cloud.nais.io/team/aura/prod-fss/app/security-token-service/resources?from=2023-11-13&to=2023-11-20

done


Add sales filter to filtering and columns

1. Why is this change necessary?
Adds a new filter for merchants so they can easily filter their best selling products
2. What does this change do, exactly?
Adds a new filter to the product filtering
3. Describe each step to reproduce the issue or behaviour.
N/A
4. Please link to the relevant issues (if any).
N/A
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-24029
Please use this issue to track the state of your pull request.

Hey @MelvinAchterhuis,
your PR looks good so far, but could you add the option to also display the sales in the listing (same as with currencies)?

Hey @MelvinAchterhuis, your PR looks good so far, but could you add the option to also display the sales in the listing (same as with currencies)?

@J-Rahe , how would you see this working with parent products that has no sales? A sum of al variant sales? (seems like me this could have an impact on the performance when the parent has a lot of variants)

Hey @MelvinAchterhuis, sorry that it took so long. I have now confirmed with the PO that:

The main listing should have the option to show the sales and just show an placeholder(e.g. "-") for variant products.
The variant listing (from the main listing and in the product tab) should have the same option to enable sales being shown, and will then show the sales for each variant.

@J-Rahe ,
I've updated the PR

Thanks again for your contribution, your PR got finally merged! :tada: :blue_heart:


No page after rejecting MS Account Login

After clicking to log in using my Microsoft Account, I decided to click "No" and not give all the permission asked for. I was redirected to:
http://allready-d.azurewebsites.net/signin-microsoft?error=access_denied&error_description=The user has denied access to the scope requested by the client application.&state=CfDJ8JpNXGetVhVOqpGUyBpOqdZZ3TrR65z1WHOFdaMcph3fhU9aivprGUyJEqwp65kQCDfu_dXOWW8Bx9nL82W5x5uJFTlBQTGfXl5aWAj-_Ue5Khf2ADPacc12EZ0o0HEh1Lpk7GEDLffQrlk0IPX3hm6s35g11ZspO4tUPkkB8H5zD3kWBQCqxKJBS_aBx7RIslagEAiJNR8WJM06FaSjjEcSXElu0LbMUdW63tXf47g-KH04QKf74bz43L_5hGXLzXhDXGrsySGZlM39jVF-l2VIUGvq22nE5Veo86P9qcoSHNcJGwLnIt1KrwqNFDPFag
but only got a blank page.

Similarly, after refusing to give all the permission requested from Twitter, got a blank page at this location:
http://allready-d.azurewebsites.net/signin-twitter?denied=u-V7uAAAAAAAikCZAAABUvU8uqQ

May also want to review what information is being required. Just getting the email address should be enough....

@shahiddev do you think that change you made earlier should have fixed this?

@stevejgordon Possibly, it depends on exactly what point of auth flow was cancelled. I'll test it to be sure so we can close this out if not required.

@stevejgordon Yes works as expected after fix #1707 and returns user to login screen so we can close this one off


How to make Rotated Table Column Headers

Thanks to all contributors to this great library for their time. ❤
Expected Behavior

I'd like to make rotated column headers.
This is what I want to build with mui-datatables.
Just column headers, no others.

Your Environment

Tech
Version

Material-UI
^4.4.3

MUI-datatables
^2.14.0

React
^16.9.0

browser
Chrome

etc

https://codesandbox.io/s/muidatatables-custom-toolbar-xcik6
using setCellHeaderProps


fix: Allow apostrophes in email addresses

Summary | Résumé
Fixes #4494
On login, registration, and reset-password we were using the valibot email validator which fails on email addresses with apostrophes, which should be valid.
In most places we were additionally using a custom validator, isValidGovEmail which checks for a valid government email domain, but also checks the name portion of the email address, and correctly passes addresses with apostrophes.
This PR removes the valibot check and just uses the isValidGovEmail check instead.

Would be worth taking a look at https://www.geeksforgeeks.org/how-to-write-email-validation-test-cases/ to see if there's other test cases we could pull in.

QA tested on the PR environment and it LGTM! I tested:

Signing up with a new email that included an apostrophe
Logging in (+ requesting a new 2FA code a couple times)
Resetting Password

Also double checked to see if the same email would work on Staging to validate the current issue (it doesn't, as expected - so the fix solves it woooo):


Tree performance improvements

concatenate urbit assets
use minified resources
preload more data
meeker dependency handling

kernel?

oh right, there was an eyre embedded js tweak


RHIDP-939: Fix typos and consistency in Getting Started with RHDH doc

Jira: https://issues.redhat.com/browse/RHIDP-939

/lgtm

@hmanwani-rh I added a comment in https://issues.redhat.com/browse/RHIDP-939
Please take a look at the discussion on https://issues.redhat.com/browse/RHIDP-978 for this part.

YAML snippets need to be formatted better

The current snippet has an error and the correction in https://issues.redhat.com/browse/RHIDP-978 looks to be correct.
All other changes in this PR look good though. Thanks,


Information/support request - uploading only missing blocks for a partially sent blob

Hi,
I'm using @azure/storage-blob_12.0.0-preview.4 library to upload large files direct from a user's browser to a blob container.  This process works fine, however, I would like to resume aborted uploads (when the browser is closed or the user pauses the upload) and I currently cant see how/if that is possible.
Using a separate API call I'm able to get a list of 'uncommittedblobs', and from that I can then see how many blocks were uploaded for a file, and how many of the blocks should be offset when attempting a re-upload.
I've tried slicing the Blob & re-uploading only the missing blocks using uploadBrowserData & UploadSeekableBlob, but looking directly at the code it seems that the list of block ids passed to commitBlockList is always initialised as a fresh array:
// line 995
const blockList: string[] = [];
....
// line 1026
return this.commitBlockList(blockList, { ...options, spanOptions });

therefore, only the blocks uploaded in the second attempt will be committed.  I've confirmed this is happening in the Azure portal admin UI.
Could you tell me, am I approaching this in the correct way?  If so, are you able to offer any information on merging the existing & new block lists?  Or, alternatively am I missing a function which can append data to an existing uncommittedblob already?
Thanks in advance.

You need to understand how a blockblob is, and how does a blockblob uploads, and leverage following APIs to do that. And also you need to implement a concurrency uploading, because SDK doesn't provide a concurrency upload API for an uncommitted blob.

BlockBlobURL.listBlocks to list all uncommitted blocks for a uncommitted blockblob
BlockBlobURL.stageBlock to upload a block into a blockblob as uncommitted block
BlockBlobURL.commitBlockList to commit uncommitted blocks.

Thanks for getting back to me. So if Ive understood correctly, the approach I'm using is correct but I will need to implement my own custom function (which will be based on UploadSeekableBlob) that uses stageBlock and commitBlock to complete the process.  I already have a concurrency server, of sorts, that I'm using to fetch the uncommitted block list.
That's enough for me to work with, thanks for the information!


erlangR19: 19.1.6 -> 19.2

Motivation for this change
Update erlangR19 from 19.1.6 to 19.2.
N.B. I've added a sw_vers substitution for darwin.
Things done

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

[ ] NixOS
[x] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Hey, @DerTim1! I appreciate the reviews you've been doing, thank you and keep up the good work!
This change LGTM but just want to get a darwin person (like LnL ) to look at the change. Can you explain what that substitute does?

@grahamc No problem. I thought a build von a linux-system is always useful. :)

@DerTim1 Definitely! Donating some cpu cycles is always appreciated.

Rebased.


Cant download data_input_v3.zip

Where can i find this file!

There is a torrent, is it not working?

No the torrent is saying 0 out of 0 connected peers on all computers/networks i have tried

I'm seeding now. Is it working?

still not working :/ http://imgur.com/a/8c6hd

I can torrent some other files but this one isnt working

Give me a minute. We'll sort it out! c;

ok thanks!

I've seeded it at multiple locations but it doesn't seem like anyone is connecting. Do you know how to verify a file has the correct hash against a .torrent file?

Also I just checked, it downloaded almost instantly for me when I deleted the file and torrent and started over.

How odd i will take another look at it the hash mine is showing is a1195c9f7ab650fa7c7bf020b51fc19ea8d9440c

Well, I was considering hopping on IRC with you and just sending it. But ideally you should verify the file I send you matches the hash of the torrent. I can just do it if you trust me but that is unwise. :c

I got it to start working my ISP was blocking the torrent connection than you for all your assistance!

No worries c;

Sorry for bothering you again but for some reason anytime I launch my SAFEA9lhinstaller from homebrew the screen does the normal startup app flash of red then proceeds to go black. While the device is in this state ive tried waiting and pressing select like troubleshoot suggests both without success

i have also tried reinstalling the files several times and have verified md5
im on version 11.1 fresh non modded ds using the cubic ninja hack to get the homebrew menu

-_- it can't be run on 11.1. You need to follow the guide

lol


[SPARK-21963][CORE][TEST]Create temp file should be delete after use

What changes were proposed in this pull request?
After you create a temporary table, you need to delete it, otherwise it will leave a file similar to the file name ‘SPARK194465907929586320484966temp’.
How was this patch tested?
N / A

Can one of the admins verify this patch?

This is too trivial to bother with. The JVM cleans this on shutdown anyway.

Is it better to use  DELETE_ON_CLOSE?

How about other test cases that use File.createTempFile? Did they clean the temp file correctly?

@kiszk thank you for review it.
I checked the other test cases that use  File.createTempFile and Files.createTempFile .
Except as described in this PR, others test cases that already use File.deleteOnExit or File.delete.
thanks.

This seems not a big problem, all the temp files are created under target/tmp, this can be cleaned by mvn clean or sbt clean.

Same, let's not bother with stuff this trivial @heary-cao please.
If it really makes the code consistent on this one point, I'm not against this, other than that it encourages more PRs this small.

Test build #3919 has started for PR 19174 at commit 0dafce7.

Test build #3919 has finished for PR 19174 at commit 0dafce7.

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

Merged to master anyway.


error when running "python demo/motion_imitate.py --gpu_ids 2"

see attached screenshot below, any help would be appreciated, Thanks!

it is usually caused by different cuda version between local cuda(cat /usr/local/cuda/version.txt) and  pytorch (pip show torch).


Clarification on Terminology

I am having trouble understanding how to use MockServer since the description is sparse
and terminology is not clear.

What is MockClient? If I am mocking a service that my system under tests uses, then the "client" of the mock is my system under test is it not? Yet, the description suggests that "client" is used to define expectations!

What is a MockProxy?

Regards,
Rk

The MockClient is used to configure the expectations that define how the mock will behave, i.e. what response to return when it receives a specific request, etc.
The proxy is used to intercept calls made from the SUT to third part non-mocked systems so you can either analyze how the SUT interacts with it dependencies or only mock specific dependencies.


Fix #5507

Add try,exception to getclassAds

Tapas, this can be closed, right?

@ticoann  Yes ! Have a look at the pycondorplugin.py and let me know if you think otherwise.

duplicate


ANTLRv4 grammar syntax error

Hello,
I'm using the ANTLR grammar to parse stix patterns but there seems to be a problem in the grammar.
The error I'm getting is:
error(156): STIXPattern.g4:233:7: invalid escape sequence

Thanks, @ShakedH. Am I correct in guessing this happened with ANTLR 4.6? If so, I'm guessing it's the same thing that was changed in oasis-open/cti-pattern-matcher#14. Should be an easy fix.

FYI, I reverted the generated grammar files in this repo to the version produced by ANTLR 4.5.3, until #5 is fixed. This problem should not recur, since the underlying grammar has been updated (and should work with both 4.5.3 and 4.6).


Handle withdrawn versions

Dependabot has (legitimately) created a merge request for a new version. But now the developer has noticed an error and deleted the version. Dependabot does not notice this and writes "is up to date" in the logs, because there is no newer version. It would be nice if Dependabot recognizes that the current version was removed and creates a new merge request on the older version.
We use pip for package management, but possibly this would be suitable for other services as well.

Thanks @nick-traeger, I'm going to close this out as a duplicate of https://github.com/dependabot/dependabot-core/issues/4379 - I realise you posted first, but we've started tracking that issue.
I don't have anything to share in terms of roadmap, but this a feature we are interested in.


grpc-web v1.2.0 appears to be broken

Hey,
It looks as though a recent release of grpc-web 1.2.0 broke something. This might be it: https://github.com/grpc/grpc-web/issues/865
It might be worth changing the dependencies in this package.json to something more specific like 1.1.0 which I believe works.

Hi there,
thanks for reporting!
The dependency in package.json is at "peerDependencies", so it does not really force anything; one should be able to use any matching version.
Need to investigate more on what's broken in 1.2.0. This lib doesn't use Promise implementation as far as I remember.

When I had 1.2.0 installed I would get a status event with code 0 and then nothing else would come through.
Out of curiosity, why are "peerDependencies" used rather than just a dependency?

When I had 1.2.0 installed I would get a status event with code 0 and then nothing else would come through.

Yep, I see the same on my projects now. Looks like some of their underlying API is changed.

Out of curiosity, why are "peerDependencies" used rather than just a dependency?

Well, a long story short: in frontend projects in general its better to use peerDependencies over dependencies to avoid client side code duplication (e.g. when you have two projects importing different versions of the same library). In addition, as an end-consumer of the library you can easily override any dependency in "peerDependencies" by installing the one you want; with "dependencies" it is impossible; that's just how npm works. As a downside - the dependencies from "peerDependencies" aren't automatically installed. However, it's better evil :)

This issue is a really good one, there are a lot of useful insides described in the connected issue on grpc-web side.
There is also an upcoming followup fix -> https://github.com/grpc/grpc-web/issues/868 .
Please use latest version that fixes the issue.

@smnbbrv thanks for digging into it, looks like they have been very helpful.


install webpack gem and run rails webpack:install

I rule and have completed some work on Case Manager that's ready for review!
I wanted to see how hard it would be to set up webpack in daria, and it turns out, not very hard. I'm tagging in a bunch of people for review because I'd like a wide range of opinions on this, particularly from people who I know work in JS a lot.
This PR sets up the webpacker gem to compile rails assets. The next steps from this PR would be porting over our existing coffeescript stuff to ES6 and depreciating the sprockets asset pipeline in favor of webpack stuff. This would get us some longer term stability and keep us using what are considered Sensible, Well-Established Standards. (webpack is not the rails default, but way more people know ES6 than coffeescript, so we're making a technical decision here in favor of the lingua franca of frontend engineers.)
I am not really a js specialist and I don't know what I don't know, so please check these assumptions and make sure that I'm not off base on this:

webpack is a reasonably stable and straightforward way of handling client side JS, which we should use instead of sprockets
we should port over our SCSS to be part of the webpack compiled assets as well
this would let us use stuff from npm pretty seamlessly
we still want to have rails ERB views be the primary way we handle html generating (e.g. this is not an indicator that we want to split the frontend and backend and move to react or vue)
porting our existing javascript and sass to be webpack friendly is not a pain in the behind
we're not scared of the security stuff, because we can be covered by good security tools in CI. (A thing that does worry me about JS generally is the fact that npm is the wild west, but I'm okay with this.
we can still do most of our testing of JS on the systemtests level, so we don't need to worry about setting up jest or anything like that.

what do you all think?
This pull request makes the following changes:

install webpack

no issues

I'm tagging in a bunch of people for review because I'd like a wide range of opinions on this, particularly from people who I know work in JS a lot.
This PR sets up the webpacker gem to compile rails assets. The next steps from this PR would be porting over our existing coffeescript stuff to ES6 and depreciating the sprockets asset pipeline in favor of webpack stuff. This would get us some longer term stability and keep us using what are considered Sensible, Well-Established Standards. (webpack is not the rails default, but way more people know ES6 than coffeescript, so we're making a technical decision here in favor of the lingua franca of frontend engineers.)
I am not really a js specialist and I don't know what I don't know, so please check these assumptions and make sure that I'm not off base on this:

webpack is a reasonably stable and straightforward way of handling client side JS, which we should use instead of sprockets
we should port over our SCSS to be part of the webpack compiled assets as well
this would let us use stuff from npm pretty seamlessly
we still want to have rails ERB views be the primary way we handle html generating (e.g. this is not an indicator that we want to split the frontend and backend and move to react or vue)
porting our existing javascript and sass to be webpack friendly is not a pain in the behind
we're not scared of the security stuff, because we can be covered by good security tools in CI. (A thing that does worry me about JS generally is the fact that npm is the wild west, but I'm okay with this.
we can still do most of our testing of JS on the systemtests level, so we don't need to worry about setting up jest or anything like that.

what do you all think?

Looks good. I can confirm we can starting using es6. Webpack definitely beats the alternatives.

thanks everyone for your insight and thoughtfulness here! I especially appreciate @geoffreyyip 's point about churn and @rudietuesdays 's experience with webpack. In the immediate term I'll be making some adjustments to our readme and build to reflect that this is a thing now, like adding some security tools and a static analyzer probably. The next steps after that are to gradually port over our existing JS to ES6 and get ourselves out of coffeescript.


Aspect ratio of images with shared colorbar

%%opts Image.B [colorbar=True]
hv.Image(np.random.rand(10,10)) + hv.Image(np.random.rand(10,10), group='B')

produces an image with unequal aspect ratios:

How can I make them the same?

What's happening here is that both A and B are assigned the same area. However B is resized slightly because of the colorbar. I'll have to think a little bit about how to fix this. Ideally plots which include colorbars would be assigned a bit more space similar to the way side histograms are handled but getting that just right might be pretty difficult.

Colorbars are often shared, wouldn't it make sense to include them as separate axes?

Yes absolutely, that's what we'll have to do.

I have now implemented colorbars correctly and they should work consistently irrespective of the plot aspects, the number of colorbars etc. Please reopen if there are any further issues.


Add documentation and a passthrough strategy to show the plugin system

Actually adding a passthrough was a bit too complex and would duplicate a lot of code, so it was decided instead to add a disclaimer and open another issue about it


Better Graph Visualization in Tensorboard (hide boilerplate)

Hiya, I absolutely love this library, however, it does produce quite messy visualizations in Tensorflow at time. I was wondering if there were any efforts to help re-mediate this. While this project makes great strides in using good naming scopes some operations (like the l2regularizer) make the graph very difficult to read. In particular, many of the GAN architectures can be difficult to visualize.

The graph is just so complex, and I don't know how tensorboard organizes a graph and how to make it looks better. Maybe I'll try adding some name scope here and there but I'm not sure.

Yeah, I wish there was a feature in Tensorflow for hidden namescopes or such. Regardless, I've figured out how to look at Generator and Discriminator based Tensorboards, but it can definitely be confusing at times. My biggest pete peave is the l2regularizers and conditionals which creates a long lines of disconnected ops and require zooming in a lot. Also a lot of the dataflow code could probably be a little better organized. All in all, the namescopes at last make it readable which is more than I can say for other libraries.

Until now tensorflow still doesn't have a nice way to reuse a name_scope (https://github.com/tensorflow/tensorflow/issues/6007). Without this feature it's hard to group relevant ops together in tensorboard.


Create Add the site - jollyvids.com

Please add the site in your directory and in the chrome extention as well

@Manup5u this repository has been abandoned years ago. Check out our forked and maintained one at https://github.com/jdm-contrib/jdm


Fix toc bug, resolves #939

Coverage increased (+0.2%) to 47.109% when pulling b2d3e918bf56b0bc917c019b8eabe3cc26df8388 on fix_toc_bug into 39eb024f830a41a77c603bcc141a1b6c35343ede on release2.1.

Coverage increased (+0.3%) to 47.129% when pulling 064d46da7b99e740ce4b9f8017991e4c29f014eb on fix_toc_bug into 39eb024f830a41a77c603bcc141a1b6c35343ede on release2.1.


Divide by zero error during page rank

When there are no parts to rank, should skip doing page rank.  This causes divide by zero errors in pagerank function in pagerank.py.

Fixed by #41


getting a column size error when using MySQL.

Describe the bug
I'm getting a column size error when using MySQL.
It seems to executed when initial setup starter_projects
sqlalchemy.exc.DataError: (raised as a result of Query-invoked autoflush; consider using a session.no_autoflush block if this flush is occurring prematurely)
                             (pymysql.err.DataError) (1406, "Data too long for column 'description' at row 1")
                             [SQL: INSERT INTO flow (name, description, icon, icon_bg_color, is_component, updated_at, folder, id, data, user_id) VALUES (%(name)s, %(description)s, %(icon)s,
                             %(icon_bg_color)s, %(is_component)s, %(updated_at)s, %(folder)s, %(id)s, %(data)s, %(user_id)s)]
                             [parameters: {'name': 'Basic Prompting (Hello, World)', 'description': 'This flow will get you experimenting with the basics of the UI, the Chat and the Prompt component.
                             \n\nTry changing the Template in it to see how the ... (26 characters truncated) ... change it to this and a Text Input into the `type_of_person` variable : "Answer the
                             user as if you were a pirate.\n\nUser: {user_input}\n\nAnswer: " ', 'icon': '', 'icon_bg_color': None, 'is_component': 0, 'updated_at': datetime.datetime(2024, 5, 14, 9,
                             36, 35, 63929, tzinfo=datetime.timezone.utc), 'folder': 'Starter Projects', 'id': '5d216c873f0f4c5a98ec1c85438a90f4', 'data': '{"nodes": [{"id": "Prompt-uxBqP", "type":
                             "genericNode", "position": {"x": 53.588791333410654, "y": -107.07318910019967}, "data": {"type": "Prompt",  ... (24099 characters truncated) ...
                             153Text\\u0153],\\u0153type\\u0153:\\u0153str\\u0153}"}], "viewport": {"x":<PHONE_NUMBER>5500563, "y":<PHONE_NUMBER>111936, "zoom": 0.43514115784696294}}', 'user_id': None}]
                             (Background on this error at: https://sqlalche.me/e/20/9h9h)

Screenshots
Additional context
plz help

@songjinu you're exceeding the description length of a flow. Please reduce it to be less than 1406 characters


Relationship: AND WHERE

Is it possible to have an additional WHERE statement on a relationship?  Intended behavior is to only get the comments that has a parent_id of this.id AND parent_type of status
A modifyEager is what I am looking for, however inside of the model itself vs on the query builder
https://vincit.github.io/objection.js/#modifyeager

See the relation mapping docs. You can add a filter for the mappings.

http://vincit.github.io/objection.js/#relationmappings


DontFragment unsupported SocketOption

I have an simple issue, I wanted to give the netstandard 2.0 Version of LiteNetLib a try, but I got an instant issue when starting.
Error:
System.Net.Sockets.SocketException (45): Operation not supported
   at System.Net.Sockets.Socket.UpdateStatusAfterSocketErrorAndThrowException(SocketError error, String callerName)
   at System.Net.Sockets.Socket.SetSocketOption(SocketOptionLevel optionLevel, SocketOptionName optionName, Int32 optionValue, Boolean silent)
   at System.Net.Sockets.Socket.SetSocketOption(SocketOptionLevel optionLevel, SocketOptionName optionName, Int32 optionValue)
   at System.Net.Sockets.Socket.set_DontFragment(Boolean value)
   at LiteNetLib.NetSocket.Bind(IPAddress addressIPv4, IPAddress addressIPv6, Int32 port, Boolean reuseAddress) in E:\Projects\LiteNetLib\LiteNetLib\NetSocket.cs:line 105
   at LiteNetLib.NetManager.Start(IPAddress addressIPv4, IPAddress addressIPv6, Int32 port) in E:\Projects\LiteNetLib\LiteNetLib\NetManager.cs:line 821
   at LiteNetLib.NetManager.Start(Int32 port) in E:\Projects\LiteNetLib\LiteNetLib\NetManager.cs:line 848

_netManager.Start();

Library version: <IP_ADDRESS>-beta
Framework: dotnetStandard 2.0, dotnet Core 2.1
OS: MacOs

To be quite honest, I don't understand why this exception even raises. Checking your code, you have a try catch around it.
            if (socket.AddressFamily == AddressFamily.InterNetwork)
            {
                socket.Ttl = NetConstants.SocketTTL;

                try { socket.DontFragment = true; }
                catch (SocketException e)
                {
                    NetUtils.DebugWriteError("[B]DontFragment error: {0}", e.SocketErrorCode);
                }

                try { socket.EnableBroadcast = true; }
                catch (SocketException e)
                {
                    NetUtils.DebugWriteError("[B]Broadcast error: {0}", e.SocketErrorCode);
                }
            }

Further Investigations, using the current nuget package <IP_ADDRESS>-beta, this issue occurs. Using the current version of the repo and using a project reference and not a nuget package, the behaviour is as expected. The try-catch block triggers and puts into the logger, that the dontFragment is not supported. It doesn't crash any further and keeps running. Any Idea on this strange behaviour?

@SynergiAios seems that this is some specific ios/netcore bug.

@SynergiAios fixed

@RevenantX just saw it, thx awesome.

It so sad, written in Net Standard, and still have to insert special rules for MacOS. Whom to blame here would be interesting for me :) (Apple or Microsoft?)

@SynergiAios i think both)


Add license and metadata from openfermion

for third_party representability

I am confused about why travis is complaining.  I am adding to non-code files....

@kevinsung any ideas?

Maybe try with the our new CI that uses GitHub Actions?


Help setting LSP Signature Help Floating window to have  round corners.

I had it that the floating window was round. This doesn't seem to be the case anymore.

I don't get it. Configure your borders to your liking in lsp hover

It was working fine until the upgrade.

From your screenshot I can clearly see that your float window borders are set to "none". This colorscheme does nothing to float win borders except setting the FloatBorder hlgroup


Prevent global variables references

The skeleton actually requires files through its index file:

config/config.php
config/container.php (dynamically created)

Those files expose few variables ($container, $cachedConfigFile, $config, ...etc), which remains accessible within the app, thought the global keyword.
This PR close the access of those global variables in expressive by running inclusions within an anonymous scope in the public/index.php, and discourage forever usage of the global keyword.

The PR has been updated according your comments

Thanks, @ronan-gloo!


⚠ Fortnite has degraded performance

In aecc9c1, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1642 ms

Resolved: Fortnite performance has improved in cc2c9e3.


build!: update to latest gts and TypeScript

BREAKING CHANGE: typescript 3.7.x introduced breaking changes in the generated code.

looking good 👍 one note, we've been setting the target language to es2018 in the tsconfig file, mind making this change too.

Updated, thanks


Update documentation

[x] Update paragraph: Running as a standalone server
[x] Update paragraph: Running in Docker
[x] Update paragraph: Configuration
[x] Update paragraph: StorageProvider

Closing with #7.


Update nf-fileapi-getvolumeinformationw.md

#define FILE_SUPPORTS_GHOSTING              0x40000000  in winnt.h

awesome. I did the same with "a" version.
seems there ar more wrong Flag's value.
Is there an updated version of winnt.h anywhere, to check all of them?
(the one here on github last changed 10y ago)


cms.js is not a CMS, please rename it.

Please, don't name it cms.
cms.js do not let the user manage it's content.
The content is managed outside (by a text editor).
Every CMS provide:

server side to store the data
backend to write the content
front end to let visitor view the content

CMS.js provide:

tools to transform text into web page using theme.

So it is not a CMS.
https://en.wikipedia.org/wiki/Content_management_system
I would recommend to change the name.

Definitely open to suggestions. Anyone have any ideas?

ops.js
one page static

I like the direction. I'm going to leave this open bit longer to give anyone else that would like the opportunity to voice their suggestions. Then, maybe setup a small Google poll to see what majority likes.

just suggesting
CMMS.js  :)
clientside Markdown markup system

CMMS.js!

:thumbsup: nice

Clientside Markdown Site?....doh

CMS
CMS is Markdown Site

I'm a fan of CMS, CMMS, or something like jsSG. When I heard about CMS.js I immediately thought of it as an alternative to Jekyll, not Wordpress, Drupal, etc. This is a really cool project. Thank you!

Voilà could be a nice name.

presto.js

@cdmedia  simplestatic.js

Thanks for all the suggestions all. I'm going to leave this open till Monday and then we can vote.

I've always thought a JS rendering engine would be really cool and so I went searching for it and finally found what you made here. Glad to catch it so early after you put it on Github so I can be a part of it. Here's a few name ideas. Note that I didn't look to see if any of them have already been taken or otherwise branded.
Copper.js (Client-side One Page Publishing Engine for Reading)
Raw.js (Render A Website, Render App Websites)
Web.js (Website Engine Brewer, brew in the sense of Javascript :D)
Mr.js (Markdown Router)
Chai.js (Client Helps Activate It - sorry, had to throw in another coffee shop word)

Should avoid M-for-markdown in the name, coz expanding to other markup languages is a fairly likely future evolution.
How about Wahlah.js?  wahlah dot io is available.

love what you are building here, i'll be playing with it this weekend :smiley_cat:
Not an english speaker but to me feels as a piece of code that allows us to share ideas on the web fast so something in the spirit of ship.js could work.

@DanielBaird Good point about M-for-markdown. I assume by Wahlah, you mean the onomatopoeia of voila. Lol, in search for this, I found some guy who was mad about people spelling it that way. Here's his quote, just for fun.
"I wish people would stop writing "wallah" instead of "voilà". It probably irks me the most out of all the misspellings out there. It’s pronounced VWAH-LAH. Not WAH-LAH." - (Source)
@polmoneys I like ship.js too. Plays on that include...
vessel.js (don't really like that as much)
Perhaps naming it after a ship like the fastest ship ever.
Francisco.js - (Source)
Concorde.js - Fast Jet
Perhaps a famous ship.
Ark.js - Obviously Noah's Ark from the Bible.
Titanic.js - Would probably get buried in search results, due to the popularity of the boat. Also wouldn't want to associate the project with a ship that ultimately met its doom at the hands of the ice cold Atlantic.
Or other ships...
kaya.js - Japanese Destroyer
Or in a totally different direction...
Agni.js - Name it after a messenger god (Source)

Thank you all for your suggestions! I've create a Google poll with your name ideas so feel free to vote for your favorite! We'll tally up the votes next week.
https://docs.google.com/forms/d/1tLA7PMJcqGEqQ0He0g7EttIoQxgzfUYGAMfmMOkk9Gw

http://m.youtube.com/watch?v=orybDrUj4vA
IMHO a different voting system should be used ;). I will completely not put
that link on Slack to skew the results.
On Jan 26, 2016 4:35 AM, "Chris Diana"<EMAIL_ADDRESS>wrote:

Thank you all for your suggestions! I've create a Google poll with your
name ideas so feel free to vote for your favorite! We'll tally up the votes
next week.
https://docs.google.com/forms/d/1tLA7PMJcqGEqQ0He0g7EttIoQxgzfUYGAMfmMOkk9Gw
—
Reply to this email directly or view it on GitHub
https://github.com/cdmedia/cms.js/issues/12#issuecomment-174801149.

Thanks @zamber for the link. Cool explanation. I never really thought if it like that. Since, it's already up I took a look at the options for voting and enabled multiple submissions.

Honestly, when it comes down to it, CMS.js is great so I voted for it.

My suggestion is MrHyde because is the good side of Dr Jekyll or hydejs

@matheus I'm sold. I actually really love that.

@barryanders I reckon not Hyde, because it's kind of defining the project in its relation to some other project which isn't so cool.  A more obscure character from the story, like Bradshaw, I think is fine though.
I still like wahlah, partly because it's a whole hearted adoption of the misspelling and mispronunciation of violá (implying pragmatism) but I also just realised that you can make it even more defiant AND web applicable by tripling the w to make it wwwahlah.
Wwwahlah
seems awesome to me.  I do like
Bradshaw
though, it's clean, pronounceable.. might be hard to get a domain for.

Results are in! Thank you all for taking the time to voice your suggestions/votes!
1.Keep it the same - CMS.js
2. Copper.js
3. Mr.js
Results`

If the name was decided in february, which i like, and awesome solution btw, why keep this issue open, link to it from the wiki pages and close this off, you did it already! A tidy house is a something... i dunno, good.


Error on \connect db_name while executing SQL-script

Description
I've done the SQL-script with DB_NAME > Tools > Backup.
When I want to restore my DB from this script, DBeaver stops with error on \connect <DB_NAME> line:
CREATE DATABASE XXXXX WITH TEMPLATE = template0 ENCODING = 'UTF8' LOCALE = 'C';

ALTER DATABASE XXXXX OWNER TO YYYYY;

\connect XXXXX

Error is:
SQL Error [42601]: ERROR: syntax error at or near "\"
  Позиция: 1
  ERROR: syntax error at or near "\"
  Позиция: 1
  ERROR: syntax error at or near "\"
  Позиция: 1

psql < ~/my_script.sql work normally.
Please, fix this.
DBeaver Version
Community Edition <IP_ADDRESS>405191503
Operating System
macOS Sonoma 14.5
Database and driver

DB: PostgreSQL 14.0
PostgreSQL JDBC driver 42.7.2

Steps to reproduce

Create DB backup in SQL format. "DROP if exists" & "CREATE if not exists" should be on.
Try to restore DB from this script: open the file, push Alt-X (Option-X).
Get an error.

Additional context
No response

Hello @dayfuaim,
The \connect is a meta-command available only in PSQL. DBeaver doesn't know anything about it.

@ShadelessFox Thanks for the answer.
This helped.
😎


fix rackaware policy is ineffective after delete zk rack info

Fixes #xyz

Main Issue: #xyz

PIP: #xyz

Motivation

Modifications

Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[ ] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Could you tune the test refer https://github.com/horizonzy/pulsar/commit/906416fc41120352cffd545a864a17ac22496d4a


Building native transport epoll linux-x86_64_fedora

Trying to build the native epoll transport. I cloned the repo, checked out the 4.0.38.Final tag, and issuing 'mvn clean install' in the netty/transport-native-epoll directory. It detects the operating system as:
[transport-native-epoll]$ mvn clean install
[INFO] Scanning for projects...
[INFO] ------------------------------------------------------------------------
[INFO] Detecting the operating system and CPU architecture
[INFO] ------------------------------------------------------------------------
[INFO] os.detected.name: linux
[INFO] os.detected.arch: x86_64
[INFO] os.detected.version: 2.6
[INFO] os.detected.version.major: 2
[INFO] os.detected.version.minor: 6
[INFO] os.detected.release: centos
[INFO] os.detected.release.version: 6
[INFO] os.detected.release.like.rhel: true
[INFO] os.detected.release.like.fedora: true
[INFO] os.detected.release.like.centos: true
[INFO] os.detected.classifier: linux-x86_64-fedora
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building Netty/Transport/Native/Epoll 4.0.38.Final
[INFO] ------------------------------------------------------------------------

However, the artifact is not named with the fedora classifier, it is simply named netty-transport-native-epoll-4.0.38.Final-linux-x86_64.jar. What am I missing?

Can you clarify the issue? Is it that you want to build the transport-native-epoll module on a fedora based system (see [1]), or that you are expecting a linux-x86_64-fedora to exist in maven central?
[1] http://netty.io/wiki/native-transports.html#building-the-native-transport

@Scottmitch The issue is that the built artifact is named linux-x86_64, not linux-x86_64-fedora. So, if I specify the dependency in my project as directed in [1], it won't find it in my local maven repo. Is there something I am missing to get the classifier added to the artifact name during the build process, or do I need to rename it manually?
[1] http://netty.io/wiki/native-transports.html#building-the-native-transport

or that you are expecting a linux-x86_64-fedora to exist in maven central?

That would be great. For now I am trying to build it myself.

try to use the same classifier as netty does <epoll.classifier>${os.detected.name}-${os.detected.arch}</epoll.classifier> for your dependency. The wiki may need to be updated as I don't think we need a different epoll artifact for fedora.

Thanks, I will try that tomorrow.

@dlmarion did it work ? ( I think it should )

@normanmaurer I got sidetracked with other issues. I will be able to try it mid week

@Scottmitch  @normanmaurer  changing the classifier as suggested works and all of my tests pass. Is it the case that I don't need a fedora-flavored native epoll jar? I will test further in the next day or so.

@dlmarion yes that is the case... We will update the wiki.

@dlmarion iirc the only fedora-flavored Netty jar is netty-tcnative and that's because the shared libraries have a different naming convention..
tmp $ ldd linux-x86_64-fedora/META-INF/native/libnetty-tcnative.so
ldd: warning: you do not have execution permission for linux-x86_64-fedora/META-INF/native/libnetty-tcnative.so' linux-vdso.so.1 =>  (0x00007ffcb0966000) libssl.so.10 => /usr/lib64/libssl.so.10 (0x00007f41ede72000) libcrypto.so.10 => /lib64/libcrypto.so.10 (0x00007f41eda8d000) tmp $ ldd linux-x86_64/META-INF/native/libnetty-tcnative.so ldd: warning: you do not have execution permission for linux-x86_64/META-INF/native/libnetty-tcnative.so'
linux-vdso.so.1 =>  (0x00007ffcbb9f9000)
libssl.so.1.0.0 => not found
libcrypto.so.1.0.0 => not found

I updated the wiki @normanmaurer PTAL.

@Scottmitch thanks! LGTM :)
Let me close this one now


Fix multiline comment for .sh files

For .sh files multiline comments are not working properly cause comments contain ':' which is currently used as a separator in the source code.

#19 will fix this issue aswell


Update S3 directions

Review to merge this request which

Looks to sperate incoming and outgoing requests into different guides
Removes internal guidelines from the public repo and makes a new internal guidance repo
Still needs updates, but i feel if we merge this in the set the structure we can then focus on each elements as the requests come in?

@LukeHauck FYI
Will eventually tie into the Transfer Family work


XML validation

Integrate real XSD validation option for transcription editor

Schemas should live in a folder schemas/ in the application's main folder
The editor.py main document properties should be expanded to contain an "XML schema" input under the "Git Repo" box (and in the DB, a field 'schema' should be added to the document table). The schema field is just the file name of the associated schema file.
On XML validation, the contents of the document's XML are checked against the schema and xmlint's output is given in the validate box (or in the dashboard as an abbreviated warning). If no schema file is specified, the validation box returns 'No schema' when validate is clicked, and in the dashboard, the XML icon stays gray with tooltip 'No schema'.

Better than input in editor.py: a select box based on the directory contents of schema/, but always including an empty option (=not specified)


Idea: support prioritization of tasks

We have a fairly large cluster now and several hundred applications running. Some of these applications are business critical - ie, they go down and we start to loose money while some of the applications are simple intern toys or hackathon projects.
We would be interested in potentially setting a "priority" level on applications - a simple weight of which applications get offer priority in the case of partial cluster outage or if a slave goes down and tasks need to be rescheduled.
We could do this with roles, but right now marathon only supports one additional role beyond the default '*' role
Another problem is preemption. If a subset of machines go down, we want to kill the super-low priority offers on other machines if room isn't available.

@andrewortman I agree with this proposal. Multiple roles and preemption is on the roadmap.
To workaround the current limitation of one additional role: you could use multiple Marathon instances (e.g. one for high prio and one for everything else).

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2435. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Does WebARonARCore support the similiar getPixelIntensity func of ARCore for lightEstimate?

Light estimation is not yet exposed for WebARonARCore/WebARonARKit. We're exploring this currently and is on our roadmap!

@jsantell, thanks for the reply, any idea about when lightEstimate will be ready?

We don't have any info to share on timeline for light estimation quite yet


Audio controls (play button) within the 'Anki export' modal

Is your feature request related to a problem? Please describe.
The subtitles aren't always lined up correctly with the audio. When exporting the audio words are cut off or the next subtitle line is included. I would then have to go back into the export window and play around with the scrubber to capture the exact audio segment to match the subtitles. Even then it's all guess work to capture the correct audio segment.
Describe the solution you'd like
Audio controls within the 'Anki export' modal. If there was a play button included near the scrubber at the bottom of the modal I can capture the exact audio which matches the subtitles without having to guess before exporting to Anki.
Additional context

@Azv121 Just want to confirm - you are using the asbplayer website with local video?

Yes, I'm using the asbplayer website with local video stored on my drive.

@Azv121 i guess the UI could be improved but you can already play the audio by clicking on the audio field of the popup

I wasn't aware of this, saves me a lot more time. Thanks for letting me know. :)


update docs for separate ECR repos and releases

Updates our documentation in order to have our new ECR Public
repositories exposed in appropriate links.
Removes the "Work in Progress" user documentation navigation and
adds our install, IRSA and Authorization user docs to the navigation
sidebar.
Updates the Services matrix with the most up-to-date links and statuses.
Finally, replaces the wording in the documentation index/homepage about
"Developer Preview" and "Not to use in production" with explanation
about our release process and maintenance phases and links to the
install docs.
Issue #829
By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

@A-Hilaly care to lgtm? :)

/lgtm


max-pages-shown doesn't seems to work

Hi,
thx for your plugin!
I'm using vue3 with vite and the parameter max-pages-shown it's not changing.. trying with 2,3,4 etc..
Thanks!

Hello @20
Can you please provide a sample or more clarifications on how it does not work?
Because on my side it seems fine and works

Closing because of no activity


Can't turn HotWater to OFF

I can set it to manual or schedule but if I try and turn it off the call just seems to be ignored.
I have tried -
hotwater.SetState(hotwater.State.Off);
hotwater.SetState(hotwater.Mode.Off);
hotwater.SetState(''OFF");
All of these seem to be ignored, no events are fired.
Thanks.

The underlying Hive api isn't very intuitive when it comes to setting the water to always off, it looks like the setState method can't do this but there is an onOffState method which can. After some investigating I found that the following code will do what you want.

var Hive = require('bg-hive-api');
var hive = new Hive('your login', 'your password');
var HotWaterControl = require('bg-hive-api/hotWaterControl');

hive.on('login', function (context) {

    // Create an instance of the hot water controller
    var water = new HotWaterControl(context);

    // Handle the on complete event.
    water.on('complete', function (response) {
        // write the response state object to the console.
        console.log(response);
        // log out
        //hive.Logout();
    });

    water.once('accepted', function (response) {
        console.log('Water is off');
        hive.Logout();
    });

    water.Call({
        'onOffState': {
            'PUT': {
                'onOffState': 'OFF'
            }
        }
    });

});

//Log in
hive.Login();

Counter-intuitively setting the onOffState to On will set the water back to Scheduled therefore to explicitly set the water to on use
water.SetState(water.Mode.Manual);
And back to schedule
water.SetState(water.Mode.Schedule);
as per the documentation. I'll need to handle this is in code so the setState method works as expected.

Thanks for getting back to me, really appreciate your work on this.


Using exit not only quits byebug but kills the Ruby process

Or at least it does when called from my local server - Thin 1.5.1
Unless I'm misunderstanding the documentation this is not suppose to happen.

same here with exit or quit - rails console / rails server also closes

I'm facing the issue again, it just doesn't exit the byebug but also does system exit.
`byebug) exit
Completed 401 Unauthorized in 1767641ms (ActiveRecord: 37.0ms)
SystemExit (exit):
(byebug):1:in exit' (byebug):1:in block in load_invoice'`

Hi! I misunderstood the original bug report. The correct command to use to exit byebug is quit, not exit. exit is not a byebug command, but a Kernel method. When whatever you typed in the byebug CLI is not a valid byebug command, byebug will evaluate whatever you typed in the current context. So, a system exit seems expected here.

Hi David, thank you for your prompt response.
I used the quit but instead of stopping the byebug, it exited completely and I had to restart the server.
(byebug) quit
Really quit? (y/n) y
Shikebs-MacBook-Pro:intra shikeb$ rails s
=> Booting Puma
=> Rails 5.1.7 application starting in development
=> Run rails server -h for more startup options
Puma starting in single mode...

Version 3.11.3 (ruby 2.6.5-p114), codename: Love Song
Min threads: 5, max threads: 5
Environment: development
Listening on tcp://<IP_ADDRESS>:3000
Use Ctrl-C to stop

https://jmp.sh/u1IT67q

Ah ok, so quit is doing the same thing. I'll reopen this then.

https://stackoverflow.com/questions/30627186/make-byebug-finish-executing-without-exiting-pry


Update Comparison with PVLIB MATLAB

The description contained in the Comparison with MATALB section is very outdated.
For example, it is stated that:

We'd all like to keep the core functionality of the Python and MATLAB projects synchronized,

This makes it sound like the MATLAB and python version are at somewhat similar stages, whereas my impression is that the MATLAB version receives very little maintenance and development and is thus far behind pvlib-python.
@cwhanse are you up for updating this description?
Alternatively, do we even want an entire page dedicated to pvlib MATLAB? Perhaps this section could be replaced by an additional sentence or two in the History and acknowledgement section?

We should update that. I'll draft a PR. I may end up removing that page and adding a sentence as you suggest.

It'd also be nice to remove the MATLAB comment in the contributing section:

If you have MATLAB experience, you can help us keep pvlib-python up to date with PVLIB_MATLAB or help us develop common unit tests. For more, see https://github.com/pvlib/pvlib-python/issues/2 and https://github.com/pvlib/pvlib-python/issues/3.


Vetiver docstrings use "Example" for sections, but it does not get parsed as a numpy example section

I think the numpy doc spec technically wants people to write "Examples" with an s, but also w/e does the parsing for sphinx autodoc + numpydoc style appears to be able to handle "Example".

Should we change the word "Example" to "Examples"?
Could also raise an issue on griffe and see if it's valuable to add "Example" to the parser / whether they want to require "Examples" verbatim.

Here's an example from vetiver.pin_metrics

opened an issue here: https://github.com/mkdocstrings/griffe/issues/118
I think the vetiver docs use sphinx-napolean for parsing. One thing I wonder about is whether sphinx-napolean always escapes chunks of code with >>>?


Update the game over message

Update the game over message so people know how to play again!

Interesting


Automation

Adds automated building with Python.
Note: please do not reject this PR, I have literally spent my entire morning working on this system.

@fkorotkov ready for initial review

@RDIL what kind of problem are you trying to solve with this change?

Boredom
Making an easy and centralized way to manage images

I plan to make it so that once its done, you can just run python3 build.py --image-variant base for example


Add edit nat spec

Description
Adds nat spec for VenusLens and edits natspec for governorBravo

please verify that all natspec return variables are with lowercase since docgen tool will throw errors

Done. I added a job to check this


Bump Dependencies #3001227762

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

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[vulkan] Remove mem::transmute usages in conv.rs

Now that conversion methods abound in ash and hal, mem::transmute may be a thing of the past?
PR checklist:

[ ] make succeeds (on *nix)
[ ] make reftests succeeds
[ ] tested examples with the following backends:
[x] rustfmt run on changed code

Thanks!
Bors r+

On Dec 29, 2018, at 21:06, Brian Merchant<EMAIL_ADDRESS>wrote:
@bzm3r commented on this pull request.
In src/hal/Cargo.toml:

@@ -23,6 +23,8 @@ mint = { version = "0.5", optional = true }
failure = "0.1"
serde = { version = "1", features = ["serde_derive"], optional = true }
fxhash = "0.2.1"
+num-traits = "0.2"
@kvark ok, sounds good! those should be removed now,

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Can you provide the dataset you are using?

Can you provide the dataset you are using?

Can you provide me with the dataset you are using?


Introducing a minimal devcontainer (for codespaces)

This PR adds the .devcontainer directory with a specification for how to create an F* devcontainer. This devcontainer is what's used to generate a 'codespace', which is just a devcontainer running on Github's servers.
The container is based on a Dockerfile that installs our usual dependencies, and will also run make as part of the creation process. Also, it automatically installs the F* vscode extension, so starting a codespace will give a usable F* out of the box.
The container can also be run locally within vscode by just using the "Reopen in container" command. VS Code will in fact automatically suggest that when opening the workspace.
I tried to base  this container on top of Github's codespace image (instead of a plain Ubuntu image), but I got different behaviors when running locally and on codespaces.. so I dropped that idea for now.
Note: when someone creates a codespace from the F* repo, the resource consumption (/ billing) runs on their tab. Having this file here should not use any resources from the FStarLang org.

BTW you can try this PR out by hitting the '<>Code' button above.

It will take a while to build but you should just see a VS code UI when it is done.

Wow, this is really nice. I was able to get a working environment with VS Code just clicking on the code button.
One nit: once the editor launched, I had to reload the window to get syntax highlighting and gutter icons. Not sure why. We might record that in a README somewhere as FAQ/workaround.


Can't use wildcards in package versions

Another little one. @benrudolph cc @gcapalbo

Strange missed that. Thanks


MOTECH-2639: Added input filter directive

Input filter group directive allows to filter a list with a set of checkboxes. It takes ng-model and a list of checkboxes which are controlled as an input group and wraps an input filter directive which holds a single checkbox behaviour.

@jredlarski -- this looks great.
I'm going to merge it and then we can get this QA'd