select team page still trigger to load more even loaded all the teams

Summary
select team page still trying to load more even no more team.
it seems still trigger load more if no more data.
Environment Information

Device Name: Android
OS Version: 8.0.0
Mattermost App Version: 1.34.1
Mattermost Server Version:

Steps to reproduce

go to select team page
scroll down to the end after loaded,
and scroll down again

Expected behavior
it should not trigger load more if loaded all the teams.
Observed behavior
still trying to load more even loaded all teams.

@bill2004158 Can you expand? Why is this a problem?

the problem is:
it may send many load requst to server, while which is not required.
not a big issue, but it is better avoid send useless request.

Although I totally agree with you in this scenario a new team can be created after the app fetched then “all” thus the newly created one will not appear.
What you suggest we do in other places like posts in a channels as we know for sure there are no more to load as we reached the older one.
If you have any suggestions on how to improve here that’ll be great. Even better if you want to contribute with the code as well ;)

in this scenario a new team can be created after the app fetched them “all” thus the newly created one will not appear.

agree.
so when scroll to bottom, always trying to check if any new data.
how about add "throttle" for loading more?
because when scrolling, it may trigger multiple loading more request at same time.
when I test, I saw at least two requests were sent out.
I think only 1 is enough.

Yes, throtteling would be nice, are you willing to make that improvement and submit a PR?

This should not be applicable any more, since the way the team list works in V2 is slightly different.


Add default policy support / React to options changes

New default policy added to AuthorizationOptions
            options.DefaultPolicy = new AuthorizationPolicyBuilder("default").RequireClaim("default").Build();

The default policy is used only when combining default AuthorizeAttributes (no policy / role specified)
cc @divega @blowdart

cc @Tratcher I accidentaly applied my Options/cleanup changes to this branch too, Can you review as well?
I nuked the AuthenticationHandler base that didn't take TOptions since it wasn't used.
I changed CookieAuthenticationOptions to implement IOptions and added a UseCookie overload that can be used via an instance which will ignore the shared cookie options in DI.
We could do something similar for the social auth middleware if we wanted.

Merged changes from https://github.com/aspnet/Security/pull/370
Also pulled over replacing IsStringOrWhiteSpace => IsStringOrEmpty
and sugar for configuring sharedAuthenticationOptions inline with AddAuthentication

Small change in plans, instead of removing usage of IOptions, the new pattern for options is simply to make each options instance implement IOptions of itself, so they can be directly passed in, which will bypass whatever shared options were in DI. Still working through the details of exactly how the new UseXyzAuth methods will be most likely:

UseXyzAuth(options => options.Foo = "whatever") // same as today, action tweaks options from DI
UseXyzAuth(new XyzOptions() { options.Foo = "whatever" }) // will use only this instance, ignores DI options

Naive question: if we configure - as suggested - some global options in ConfigureServices and add a middleware in Configure that overrides the global options, won't it also override the options used by the next middleware in the pipeline, since the options instance is a singleton?
public void ConfigureServices(IServiceCollection services) {
    services.ConfigureCookieAuthentication(options => {
        options.CookieName = "Global_cookie_name";
    });
}

public void Configure(IApplicationBuilder app) {
    // This middleware will use Local_cookie_name as the cookie name.
    app.UseCookieAuthentication(options => {
        options.CookieName = "Local_cookie_name";
    });

    // This one should use Global_cookie_name.
    app.UseCookieAuthentication();
}

Doesn't override so much as each updates the shared options, so yes the next middleware will 'see it'.  Really want to use the instanced options for situations like that (i.e. cookies)

Basically singleton shared options are exactly that, its a bit weird but everyone who is using that shared instance will see any updates to it.  Not the best thing to use when placing multiples of the same middleware in the pipeline (should use the non shared overloads for that via newing up a stand alone options for each middleware)  I still need to finish updating identity to see what that looks like...  its not pretty either :/

Named options had at least one merit then :laughing:
I guess it would be better to avoid offering a way to "override" the options from the app.Use*Authentication if it has such unpredictable side-effects.
The pattern I had suggested in my prototype sounds more robust (https://github.com/PinpointTownes/Security/commit/26b511f3d294f4b9545e1fbc57e4ef5226672118):

Basically, app.Use*Authentication extensions have 2 versions: one that relies on the options defined by ConfigureServices and is defined by authenticationScheme (the current vNext solution), and another one that directly takes the options instance and wraps it in a Options wrapper instead of using the options defined by ConfigureServices (Katana's approach).

yeah we'll need to figure out something that works consistently everywhere

Replaced by https://github.com/aspnet/Security/pull/425


No cursor themes in Xfce

Issue description
In the Xfce "Mouse and Touchpad" settings, in the Theme tab, no cursor themes are shown other than Default. The cursor size option has no effect. Even after installing pkgs.vanilla-dmz (a set of cursor themes) via environment.systemPackages, no cursor themes are available.
Technical details

System: NixOS 16.09

I have the same problem with the Windows Manager Only setup. NO desktop environment.
LXappearance did show the mouse themes not after I copy some Cursor themes manually into ~/.icons. but still refuse to change the theme.
also I am not using XCURSOR_PATH environment variable.
if there that is the solution how should i implement it?

This is a serious issue for me, as it's very hard to see the mouse cursor on a retina display with the default size.
If there is any temporary workaround to change the size/theme, please let me know, that would be very useful.

@aloiscochard The workaround is to add these to your configuration.nix (this is what my patch makes the default):
environment.variables.XCURSOR_PATH = "$HOME/.icons";
environment.profileRelativeEnvVars.XCURSOR_PATH = [ "/share/icons" ];

@ambrop72 "error: The option value environment.variables.profileRelativeEnvVars' in /etc/nixos/configuration.nix' is not a string or list of strings."

@taohansen the error suggests you didn't enter it right,
it's environment.profileRelativeEnvVars
not environment.variables.profileRelativeEnvVars.
On Sun, Apr 9, 2017 at 9:19 PM, taohansen<EMAIL_ADDRESS>wrote:

@ambrop72 https://github.com/ambrop72 "error: The option value
environment.variables.profileRelativeEnvVars' in/etc/nixos/configuration.nix'
is not a string or list of strings."
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/issues/21442#issuecomment-292806461,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACgTse7EKMcBoM8R008PWXWvnyfe6VIBks5ruS8qgaJpZM4LWZXD
.

@ambrop72 I've made your necessary changes which allowed a rebuild however my mouse cursor still does not adopt a new cursor theme.

Just to say that I got my custom cursor working  by following @ambrop72 advice, thanks a lot for the help!

My cursor is only custom in over some applications. Any of the Xfce applications launched as user display the old default but qutebrowser and emacs shows my new cursor. Here's my configuration.nix.

Adding environment.sessionVariables.GTK_PATH = "${config.system.path}/lib/gtk-3.0:${config.system.path}/lib/gtk-2.0"; in conjunction with @ambrop72's two lines has fixed this issue for me.

This has been fixed in 8a039ff219a9d7630679803c1cbf82c82741b801.
I can't reproduce the issue (tried nix-env -iA nixos.vanilla-dmz):

system: "x86_64-linux"
host os: Linux 4.14.5, NixOS, 18.03.git.ab1d2a5 (Impala)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 1.11.16
channels(root): "overlay, nixos-18.03pre122666.afe9649210c"
channels(chronos): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

This has been fixed in 8a039ff219a9d7630679803c1cbf82c82741b801.
I can't reproduce the issue (tried nix-env -iA nixos.vanilla-dmz):
* system: `"x86_64-linux"`

* host os: `Linux 4.14.5, NixOS, 18.03.git.ab1d2a5 (Impala)`

* multi-user?: `yes`

* sandbox: `yes`

* version: `nix-env (Nix) 1.11.16`

* channels(root): `"overlay, nixos-18.03pre122666.afe9649210c"`

* channels(chronos): `""`

* nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs`

Same problem in Gnome.
My Icon Themes placed in ~/.icons are listed and defined, but not applied.


Enter a DAO detail page, click Creator address, can not open explorer to display address detail

Description
Enter a DAO detail page, click Creator address, can not open explorer to display address detail
Steps To Reproduce

Enter a DAO detail page https://test.tmrwdao.com/dao/telegram-apps-dao
Click Creator address

Current Behavior

Can not open explorer to display address detail

Expected Behavior

After clicking creator address, should open explorer to display address detail

Environment

Platform: PC
Node: v18.18.0
Browser: Chrome 126.0.6478.56

fixed confirm


Dropdown not validating on blur or change events

I dont know if this is for a particular reason, but i cant seem to get a dropdown box to validate on blur or change events. It does validate correctly upon form submit.
All semantic-ui demo code for form validation also has this, so no demo code needed i suppose.

Any update on the issue ?
Here is a jsfiddle that duplicates the error : http://jsfiddle.net/8wm6uoth/18/
All elements validate trough blur, only dropdown doesn't, still validates on submit tho.

I'll try to sneak this in 1.9

awesome, thanks :+1:

everything seems fine now, thanks for the fix :+1:

I am still having this issue with Semantic-ui  2.1.5. only dropdown doesn't validate onBlur.
I am using semantic ui with React framework.
Demo site seems to have same issue too.
http://semantic-ui.com/behaviors/form.html#/examples

Echoing @utsavig, dropdown doesn't validate on blur.

I'm using semantic-ui-dropdown npm package. To fix @utsavig problem, I added :
 $input.trigger('blur');
to the blur method on line 927
blur: function(event) {
            $input.trigger('blur');
            pageLostFocus = (document.activeElement === this);
            if(!activated && !pageLostFocus) {
              module.remove.activeLabel();
              module.hide();
            }
          }

Still seeing the 'blur' problem in 2.3.1 and on the examples page.
Im using this for now:
  $('.ui.dropdown').on('blur', function () {
    setTimeout(function ($e) { $e.find('input,select').trigger('blur') }, 1, $(this))
  })

The setTimeout is there to avoid a flash of the error prompt when selecting a value. Hope someone else finds it useful.


Fix float amounts for NEAR attached deposit

Describe the bug
When a user chooses to input an attached deposit amount in NEAR unit, and specifies a float amount like 0.5 the UI will actually put 0.5 in the proposal JSON (instead of converting to yocto). For an example, see this failed proposal
To Reproduce
Steps to reproduce the behavior:

Make a multicall
go to export tab
in 'Total attached deposit' choose NEAR as the unit
input 0.5
fill the rest of info and propose
see how proposal fails

Expected behavior
UI should convert the amount to yocto
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Browser: chrome, brave

code fixing this was already merged with #23


Bug Fixes and changes required for Advisor portal publishing

@tsubasaxZZZ need your final review and approval before merge

@infosatheesh2020 Thank you for your quick respond. I've checked through first pane to end pane quickly. It seems to be working correctly.


Items rework

Category:

Item (weapon, armor, consumable, resource, crafting recipe, etc.)

Feature description
Many times players ask for more inventory slots. This is not an unreasonable request, but I don't think people will be satisfied until they can carry most items at once, so they would just keep asking for more and more slots in the future, and I think it gets away from the real problem; You often want to have options available to you for whatever you are doing, and it is kind of annoying to need to keep going to the bank to get access to what you want.
I think items and crafting can be improved by changing from a set 10 slot limit in the inventory bar, to a "bag" type system where you can store as many items as you want, but each item would have a weight, and you can only carry so much weight.
For example, with a carry weight limit of 100, and if an individual item, i.e. iron ore, weighs 2, then you can carry 50 iron ores at once. If one iron hammer weighs 20, you can carry 5 of them.
This would separate the kind of item from how many items you can carry, as currently you can only carry the same amount no matter what it is, i.e. carry 10 cotton, or 10 noctis armours, even though they have vastly different values.
This would make resource gathering less annoying, as currently you can only gather 10 resources per trip (also assuming you dropped/destroyed your pickaxe too).
The problem with being able to hold so many different kind of items at once (if each type of item has a weight of 1 with a max weight of 100, you could carry 100 different kinds of items), is that it hard to present that on the UI cleanly. So, this main "bag" where you can access all of your items, would be accompanied by a "hotbar" (which would be the current main inventory slots bar) for quick access to items in your inventory. You can add items from your bag to the hotbar and select/use them from there.
Notes
This would be similar to how the inventory and item carrying works in The Elder Scrolls games.

Storage panel (i.e. bank) changes to accommodate this new item system.

All individual tasks completed. Added to update 8.


'org.pl' fails with ovh hook.

Hi,
Got a problem with 'org.pl' domain with OVH hook.
OVH api claims that such resource does not exist.
All my other domains work fine - but all other domains are .com .org .studio .pl ones.
Only one domain of mine is org.pl and that particular one is failing.

This must be an issue with the hook script, doesn't sound like an issue with dehydrated.


Complete carousel refactor adding new functionality

Summary
This is a complete overhaul of the carousel components (code-wise) as well as adding a new structure for it.
I decided to update the thumbnail highlight to the new design (with a full thin border rather than just a fat top border) as the old design looked weird with the new changes.
Relevant Technical Choices

To-do

[x] Refactor carousel
[x] Change thumbnail sizing and layout
[x] Retire old compact indicator
[x] Change scroll arrow behavior
[x] Change carousel menu layout
[x] Update thumbnail highlight to new design
[ ] Make sure tests still run

User-facing changes

Testing Instructions

Make sure thumbnail carousel, grid view and keyboard shortcuts still work as intended.

Fixes #5692
Fixes #5077

@barklund Seems to work great. One question from testing:

If there is only one Page, or generally very few pages so that the scrolling is disabled from both sides, should the arrows be shown at all?

@barklund Seems to work great. One question from testing:

If there is only one Page, or generally very few pages so that the scrolling is disabled from both sides, should the arrows be shown at all?

@barklund One more finding:
When deleting a page via keyboard, the focus "gets lost" and is not in the carousel anymore.
Steps:

Add a bunch of pages
Use arrows to select one of the pages and hit "Delete"
Try to delete another page via "Delete" -- not happening, the focus is not in the carousel anymore.

Thoughts?

@barklund One more finding:
When deleting a page via keyboard, the focus "gets lost" and is not in the carousel anymore.
Steps:

Add a bunch of pages
Use arrows to select one of the pages and hit "Delete"
Try to delete another page via "Delete" -- not happening, the focus is not in the carousel anymore.

Thoughts?

@barklund One more finding:
When deleting a page via keyboard, the focus "gets lost" and is not in the carousel anymore.

Good find! Fixed in d063d7423.

@barklund One more finding:
When deleting a page via keyboard, the focus "gets lost" and is not in the carousel anymore.

Good find! Fixed in d063d7423.

This is a lot of new code, are we're confident it works as intended without any new tests? (Where's the code coverage report when you need it?)

Aside:

assets/js/stories-dashboard.js	466 kB	+3.78 kB (+1%)

Sounds like I'll need to do some digging into bundle generation again 🤔

This is a lot of new code, are we're confident it works as intended without any new tests? (Where's the code coverage report when you need it?)

Aside:

assets/js/stories-dashboard.js	466 kB	+3.78 kB (+1%)

Sounds like I'll need to do some digging into bundle generation again 🤔


Make include_type_name mapping param optional

The include_type_name mapping parameter was added in 6.8 (defaults to true), kept in 7.x (defaults to false) and will be removed in 8.x (Schedule for removal of mapping types). In Elasticsearch versions which don't know about this parameter (5.x, for example), the save method in Mapping is raising an exception: elasticsearch.exceptions.RequestError: TransportError(400, 'illegal_argument_exception', 'request [/testing] contains unrecognized parameter: [include_type_name]').
This PR doesn't change default behavior but allows a class to pass a include_type_name=False to mapping to avoid this error.

Usage example:
import toute
from elasticsearch import Elasticsearch

class Person(toute.Document):
    _doctype = 'person'
    _index = 'testing'
    _es = Elasticsearch(host='elasticsearch', port=9200)
    name = toute.KeywordField()

Person.init(include_type_name=False)

Thank you!!!


tsconfig's rootDirs overridden by synthesized rootDir in compiler.ts

Expected behavior:
If I use rootDirs (plural) and not rootDir (singular) in tsconfig.json, only rootDirs should be passed to the compiler.
Actual behavior:
compiler.ts synthesizes a value for rootDir, which is then passed to the compiler.  This overrides rootDirs and causes my project to fail with gulp-typescript, even though it builds with tsc.
Your gulpfile:
Include your gulpfile, or only the related task (with ts.createProject).
import gulp from 'gulp';
import ts from 'gulp-typescript';

const mainProject = ts.createProject('tsconfig.json');

function buildTypeScriptProject(project) {
  const projectSources = project.src();
  const projectBuilder = project();
  const outDir = project.options.outDir;
  const buildResults = projectSources.pipe(projectBuilder);
  return buildResults.js.pipe(gulp.dest(outDir));
}

// Omitting the rather complicated generate-sources task...

gulp.task('build', gulp.series('generate-sources', () => {
  return buildTypeScriptProject(mainProject);
}));

tsconfig.json
Include your tsconfig, if related to this issue.
{
  // Use these TypeScript settings as defaults:
  "extends": "./tsconfig-base.json",

  "compilerOptions": {
    // output JS to dist/
    "outDir": "dist",

    // Some generated modules should be found in generated/ as well.
    // Treat src/ and generated/ as equivalent during module resolution.
    "rootDirs": ["src/", "generated/"],
  },
  "include": [
    "src/**/*.ts",
  ],
}

Code
Include your TypeScript code, if necessary.
Not strictly necessary, given:
I have a simple fix in mind that works for me locally.  In compiler.ts, rootDir is created on this.project.options, when it should instead be created on this.  It should fall back through options.rootDir, options.rootDirs[0], and then finally the computed version that exists today.  With this change made locally, everything works correctly.  Either rootDir or rootDirs[0] can be used to determine the locations of things, but the config from tsconfig.json won't be modified inappropriately before being passed to the compiler.
I will send a PR shortly.

This fails with TypeScript 4.9.4, but doesn't fail with the 3.6 or 3.7 versions that you test with in gulp-typescript.

It looks like you haven't updated typescript/dev in 3 years, but a regression test for this will not be effective without testing against TypeScript v4.

The minimum version I can find that fails in this case is v4.3.x.


Allow null score for retrieved documents (redo)

Fix of PR #1966 that was reverted by #1970

There was a missing "return" in the sort by score statement. It didn't show when I tested because I was locally set to do query=none retrievals---and the last conversations in my history were such instances too
:bow: for the mistake

That does not explain that some retrievals worked? They should have all failed no?

The return was there when scores were both null; it was omitted when scores were both there => as such it didn't fail for query=none retrievals

Right but you mentioned "some retrieval worked" during the incident I believe. Do you confirm that none worked at all?

yes. As I saw multiple conversations working on my end, I worte this as part of the investigation process I was on. Retrospectively, in production, all retrievals failed since there are no retrieval with query = none


Vagrant is out of date

Environment: (put here arch, model, OpenWrt version)
Vagrant
Description:
$ vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Box 'vladimir-babichev/openwrt-19.07.3' could not be found. Attempting to find and install...
    default: Box Provider: virtualbox
    default: Box Version: >= 0
The box 'vladimir-babichev/openwrt-19.07.3' could not be found or
could not be accessed in the remote catalog. If this is a private
box on HashiCorp's Vagrant Cloud, please verify you're logged in via
`vagrant login`. Also, please double-check the name. The expanded
URL and error message are shown below:

URL: ["https://vagrantcloud.com/vladimir-babichev/openwrt-19.07.3"]
Error: The requested URL returned error: 404 Not Found

https://app.vagrantup.com/vladimir-babichev/boxes/openwrt-19.07

- config.vm.box = "vladimir-babichev/openwrt-19.07.3"
+ config.vm.box = "vladimir-babichev/openwrt-19.07"

Vue3 + Nginx + Lua is coming...


Resources

Planets now have resources, which means that the people won't starve.

The process is simple, but easy to extend in future sprints.
Also:

wide improvements in UI and query speed.
Stability and bugs
Removed redundant code

https://github.com/BillmanH/exoplanets/issues/95


When accessing interface resources through gateway, invalid version format: UNSUPPORTED exception and normal response appear alternately

i have the same question! nacos 2.0.3 + spring-cloud-starter-gateway 3.1.1 spring-boot 2.6.7

me too!

If you'd like us to spend some time investigating, please take the time to provide a complete, minimal, verifiable sample (something that we can unzip attached to this issue or git clone, build, and deploy) that reproduces the problem.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Alpha Ember - and Alpha Blood - and Alpha Tropical Wyvern

I cant find this in drop downlist.

I guess these are from a mod? If so they'll appear with their internal class name (something that looks a bit like CrystalWyvern_Character_BP_WS. There's no way to extract proper species names from the save file and it would be a never-ending huge task to try to gather species names from all the mods people play.
But it's also possible that there are simply none of that species currently spawned - only species are that currently spawned on the map can be listed.


add document for application properties guide

Description
I added application properteis gruide using asciidoc.
Related Issue : #2246
How Has This Been Tested?
None.
Need additional checks?
None.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

@ufoscw @kyungtaak I've added some explanations for data preparation. Sorry for repeated approval requests.


A fatal error has been detected by the Java Runtime Environment: SIGSEGV

Please provide a brief summary of the bug
We've experienced the following issue:
A fatal error has been detected by the Java Runtime Environment:
SIGSEGV (0xb) at pc=0x00007f0cde41a5d0, pid=1964, tid=2180
JRE version: OpenJDK Runtime Environment Temurin-11.0.13+8 (11.0.13+8) (build 11.0.13+8)
Java VM: OpenJDK 64-Bit Server VM Temurin-11.0.13+8 (11.0.13+8, mixed mode, tiered, compressed oops, g1 gc, linux-amd64)
Problematic frame:
V  [libjvm.so+0x7f05d0]
Core dump will be written. Default location: Core dumps may be processed with "/bin/sh /usr/sbin/coredump.sh %p %P %t %e %h" (or dumping to /opt/core.1964)
If you would like to submit a bug report, please visit:
https://github.com/adoptium/adoptium-support/issues
Please provide steps to reproduce where possible
No response
Expected Results
No crash during the JVM runtime
Actual Results
JVM stopped working caused by a SIGSEGV error. See the following file for more details:
hs_err_pid1964.log
What Java Version are you using?
OpenJDK 64-Bit Server VM Temurin-11.0.13+8 (11.0.13+8, mixed mode, tiered, compressed oops, g1 gc, linux-amd64)
What is your operating system and platform?
Rocky Linux release 8.7 (Green Obsidian)
How did you install Java?
RPM installation from custom built rpm from https://github.com/adoptium/temurin11-binaries/releases/download/jdk-11.0.13%2B8/OpenJDK11U-jdk_x64_linux_hotspot_11.0.13_8.tar.gz
Did it work before?
No response
Did you test with the latest update version?
No response
Did you test with other Java versions?
No response
Relevant log output
No response

Please upgrade to 11.0.20 and report back here with any crashlogs.


repeated matrix game

Hi, thanks for the contribution to the community!
I didn't find the environments of repeated matrix games. Although they are simple and may be created by users, it would be great if we can use them directly from this repo since the results from the paper show the suboptimal performance of many algorithms. Thus, I think these matrix games are worth a try.

We used this repo:
https://github.com/semitable/matrix-games

Thanks very much!


Added auto generated doc headers placeholders

@SD10 I left the placeholders empty as you can fill them better than me

Thanks for doing this 👍


Add type hints and reformat with Python black

Hi @FangjinhuaWang, thanks for sharing your great work and congrats on your CVPR oral. I've added a few changes to the module.py file:

Added type hints for clarity
Reformatted with Python black for improved readability
Added docstrings per the Google Python style guide

Is this something you would consider pulling into your repo?

Thanks for your contribution!

Thanks!


Facet errors

This PR addressed #1332
It has made changes to common/errros.js to catch these newly created errors (InvalidFacetSorting and InvalidPageCriteria.) and generate error modal accordingly.

Type
Message
Link formation

Invalid sort criteria
Generated at Ermrestjs
Strip sort and after, before

Invalid page criteria
Generated at Ermrestjs
Strip after, before

Invalid facet
Generated at Ermrestjs
Strip facet and other conditions can co-exist

Invalid filter
Generated at Ermrestjs
Strip filter and other conditions can co-exist

Depending upon which error is being thrown the modal header changes and asks user to click on the OK button to reload the page without the faulty condition.
You can play around with the condition here to see the behavior in error modal.
Minor modifications are done in common/modal.js and utils.js to accommodate modal body changes.
Test cases are written in errors.spec.js

Test cases were added. Could not test at Travis since timestamp issue is failing.

@amitjha21 If you remember in the meeting we mentioned that we don't want this logic to be in catch all. You cannot assume this error is being thrown for the read request on the main entity. It should be on the catch function (or error callback) for the read request on the main entity. More specifically these reads: recordset, recordset-with-faceting, recordset-re-read, record, recordedit
Also the timestamp issue is in a PR and test cases should pass. You can see that the build in this branch is failing on new test cases that you have added.


Merging latest testcontainers with our fork

Merging the latest code from testcontainers to our fork

@dannywieser-1p @myusuf3


Readme: prefetch-deps: example of usage of configfile

Add example how config file should be used
Before you complete this pull request ...
Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

Not sure if it's just for me, but when I check the README I don't see the link in available configuration parameters being rendered:
https://github.com/konflux-ci/build-definitions/blob/6d5f1541435af4b363dc3914594cfc21978abb35/task/prefetch-dependencies-oci-ta/0.1/README.md

Not sure if it's just for me, but when I check the README I don't see the link in available configuration parameters being rendered: https://github.com/konflux-ci/build-definitions/blob/6d5f1541435af4b363dc3914594cfc21978abb35/task/prefetch-dependencies-oci-ta/0.1/README.md

you are right, it worked only in my IDE :D


can customize the data source?

I hope to show the data that like the result of mongodb's group method, can not through find, meteor now also does not support group method, so I wrote a method to realize the classification, I think this how to customize the data source from my custom methods results?

I don't think it will currently be possible, but some of the other datatables packages may support it. This package is specifically for large collections that need optimized subscriptions. I may add support for other data sources in the future.

ok


Invalid color value and unnecessaty BOM header.

Color channel values should be separated by commas (standard_surface_copper.mtlx).
Unnecessary BOM file header (lib/mx_microfacet_diffuse.glsl).

mx_latlong_map_lookup() function is not required for custom multi-environment texture lookups. It's better to move this function declaration to corresponded mx_environment_fis.glsl and mx_environment_prefilter.glsl files.
Multiple environment lookups require an additional argument for all _bsdf_indirect( functions that will be passed to mx_environment_radiance() and mx_environment_irradiance() functions.
We are using a simple structure for it:
struct IBL { mat3x4 transform; uint diffuse; uint specular; };
It's possible to manually patch shaders during generation, but it would be better to have a custom argument that can be used for multiple IBL support.

Thanks for this proposal, @Tellusim, and two fixes listed in the title (color value and BOM header) look like clear improvements.
I'm less sure about the move of mx_latlong_projection and mx_latlong_map_lookup into implementation-specific files, since this has the unfortunate side effect of duplicating code that was previously centralized, making it more challenging to maintain this code over time.  Perhaps we should consider this change separately, so that it doesn't hold up the integration of your original two fixes?

The problem with mx_latlong_map_lookup is the sampler2D type. Combined samplers only exist in OpenGL and Vulkan. Another good solution for us would be a macro block for mx_latlong_projection()/mx_latlong_map_lookup() functions or move all environment texture lookup functions into the separate file. It will not duplicate code. Please tell us what works best for you, and we will create a pool request.

Ah, that makes sense, and I wasn't aware of that restriction.  In that case, the best approach might be to move these functions to a new environment.glsl file in the same pbrlib/genglsl/lib folder, and this new file would be included directly by dependent files such as environment_fis.glsl and environment_prefilter.glsl.
If that approach works for you, then I'd recommend putting this in a new pull request dedicated to that change, in order to make tracking more straightforward in the future.

Here is the commit with dedicated mx_environemt.glsl file with mx_latlong_projection()/mx_latlong_map_lookup() functions inside.

Might I suggest to move the USD change to another PR?
Perhaps this:

USDUvTexture is the color variant of the spec and the spec has colorspace attributes (which still need some resolution).
A vector3 variant would be useful.
Seems the vec3 variant should only be a normal map so the scale, bias and default can be set appropriately to perform *2 -1. w/ default value { 0.5, 0.5, 1.0 }.
It is not easy to just route a different set of normals in tangent space as no utility exists for this AFAIK, so 0,0,1 is "safe" as a default. There is no "tangent" space to specify input normals with (e.g. "Ntangent").

BTW @jstone-lucasfilm, I think the translation graph also needs to be fixed up so if it's a normalmap input to std surface then it needs to be removed and replaced with a USDUvTexture or an image node with additional scale + bias (which is what the graph of USDUvTexture basically is). But worth checking with Spiff if about how much to pursue USDUvTexture ?

@pablode Thanks for the explanation. This makes this material much different from StandardSurface and Gltf PBR. We will fix the UsdPrevewSurface tangent space orientation on our side.
@kwokcb The modification is reverted back. But this difference is a good place for making mistakes.

@Tellusim, Yes, within the MaterialX ecosystem, the interpretation for the normal input makes it an "outlier".
Will follow-up on ASWF to see what folks have to say on this and update existing issues.

@kwokcb Let's leave aside questions about improving upon the UsdPreviewSurface shading model, since that's a separate specification maintained by the USD team at Pixar, and focus on the changes proposed here.
@Tellusim It may be valuable to split this pull request into independent pieces (e.g. environment map refactor, additional normalize operations for tangent frames, and syntax fixes), so that we can focus on the impact of each one separately.
For the additional normalize operations, it would help to have some visual motivation for this change, to get a sense of what artifacts it is designed to address.  Usually I wouldn't expect two separate normalize calls to be required when generating a tangent frame, and more background on this specific proposal would be valuable.

@jstone-lucasfilm Here is the comparison of normalized and non-normalized tangent spaces:
https://youtu.be/4OvuNRB-A_M https://youtu.be/-EfCR-NzbFI
PS: yes, we will split pool requests.

That comparison looks pretty compelling, @Tellusim, and I'd be open to analyzing this proposal in a separate changelist.  In your example renders, are there relatively strong normal maps on the surface?  That could potentially be the property that differentiates it from the standard examples in our test suite.

@Tellusim Just a reminder that this pull request should be split into separate proposals for each related set of changes (e.g. data library fixes, GLSL file reorganization, renormalization of tangent frames).

Yes, sorry, it's a little bit out of schedule. It should be done soon.


alias not working

I cannot set alias for the latest deploy

config
failure

This works:
$ node_modules/.bin/now alias coverthabit-ywvzrdnsel.now.sh coverthabit
> Assigning alias coverthabit to deployment coverthabit-ywvzrdnsel.now.sh
> Success! coverthabit.now.sh now points to coverthabit-ywvzrdnsel.now.sh [2s]

This does not
$ node_modules/.bin/now alias
> Error! Couldn't find a deployment to alias. Please provide one as an argument.

Not sure what I am doing wrong.

We are also having this issue now, we weren't before...

Same with alias with parameter:
now alias https://build-jkloxylm.now.sh mysitebaseurl
Error! Response error

My errror went away after updating the CLI


Reasonable way to do parallelism on GPU

Do we need to (optionally?) allow for separate udtfs for the same function sig for cpu and gpu?

This will depend on whether UDTFs are expected to be CUDA-aware or not, or stated differently, do we expect that users (who provide UDTF definitions) write CUDA-specific code?

Wondering if the stride logic could be pushed into the Column field accessor, which could keep track of the thread id and stride in the case of the GPU. (May not work though for cases where row order/neighbors matter)

Slack discussion https://quansight.slack.com/archives/C01CAU0U1FH/p1605528413113400 :
Currently, rbc is able to make a decision if a UDF/UDTFs can be run on GPU or not based on the functionality used in LLVM IR. For instance, if a UDF/UDTF uses stdlib functions like malloc etc, these will be run on CPU only.
One can define UDF/UDTFs to run on CPU or GPU or both by using appropriate suffix. For instance, a UDF definition with a name foo__gpu_1  will tell to omniscidb that the UDF foo  can be executed on GPU only. Similarly, foo__cpu_0  would be executed on CPU only. If the suffix does not contain neiter _cpu_  nor _gpu_  substrings, the corresponding UDF/UDTF will be available for execution on both CPU and GPU.


the order of execution?

is there a control of the order of execution?

Currently, no.
Every aspect in XAspect should be independent of each other. If you care about the order of execution, you should declare the order explicitly in the original method.

thanks for reply.


just a question

Perhaps a silly question but I just wanted to verify before doing something irreversible, this extension doesn't actually redeem keys when it populates your library does it?
I have so many junk keys in my humble library which I would be horrified to see polluting my steam library. Thanks

That's a great question! No, it doesn't redeem anything. I'm not even sure if Humble exposes an API to do so. It uses an endpoint that just enumerates all of your keys. I use it to build a database so I can stuff I own before picking up a game a third time in a new bundle.


feat(repeater): add PROXY and NO_PROXY env variables

PROXY is an alias to --proxy flag
NO_PROXY is an alias to --proxy-domains-bypass flag

Thanks @maksadbek.
Maybe the PROXY flag should be HTTP_PROXY / HTTPS_PROXY?
https://superuser.com/questions/944958/are-http-proxy-https-proxy-and-no-proxy-environment-variables-standard

Thanks @maksadbek. Maybe the PROXY flag should be HTTP_PROXY / HTTPS_PROXY? https://superuser.com/questions/944958/are-http-proxy-https-proxy-and-no-proxy-environment-variables-standard

Also: https://about.gitlab.com/blog/2021/01/27/we-need-to-talk-no-proxy/#background


Could not initialize class com.ecwid.maleorang.util.ClassUtil

Hi,
Been trying to use your maven version 3.0-0.9.5 in my Java 8 Spring Boot project but am getting the following error when trying to run it on a AWS Tomcat instance:
java.lang.NoClassDefFoundError: Could not initialize class com.ecwid.maleorang.util.ClassUtil
Any ideas?

Hi @jammyclanger ,
Can you provide the complete stacktrace of the error you get?

Hi
I have had a similar issue as the one mentioned above.
occurs when i make the following call within a service method.
EditMemberMethod.CreateOrUpdate method = new EditMemberMethod.CreateOrUpdate(listId,email);
throws
java.lang.NoClassDefFoundError: Could not initialize class com.ecwid.maleorang.util.ClassUtil
stacktrace appears to be empty.
I suspect its a problem with my integration of the project with maven
i have added

com.ecwid
maleorang
${mailChimpWrapper.verson}

within my maven build file
from the example
https://github.com/Ecwid/maleorang/blob/master/src/test/java/com/ecwid/maleorang/examples/ExistingMethodExample.java
I assumed it was just a matter of importing the correct classes and following the outlined structure.

List<MailchimpMapping> mailMappings = mailchimpMappingDao.findMailchimpMappingByWebhookId(webhookSettingsId);
			if(mailchimpProperties.getApiKey()!="" && mailchimpProperties.getMailChimpListId()!=null){
				final String apiKey = mailchimpProperties.getApiKey();
				MailchimpClient client = new MailchimpClient(apiKey);
				try{
					final String listId =  mailchimpProperties.getMailChimpListId();
					for(Details person: details){
						final String email =  person.getEmail();
						EditMemberMethod.CreateOrUpdate method = new EditMemberMethod.CreateOrUpdate(listId,email);
						method.status = "subscribed";
						method.merge_fields = new MailchimpObject();

						for(MailchimpMapping mapping : mailMappings ){
							method.merge_fields.mapping.put(mapping.getMailChimpVarName(), person.getReflectionValue(mapping.getLocalVarName()));
						}

				        MemberInfo member = client.execute(method);
					}
				} finally{
					client.close();
				}

			}

any help would be greater appreciated.

Hi @shaunMccloskey ,
Can you provide access to a sample project, where the problem can be reproduced?


Blog: Show more related links at bottom of each post

What's your idea?
Currently, the only external link (opens in a new tab) awaiting the reader at the bottom of each blog post is our .txt easteregg,

I want to add at least one more link to the original discussion on GitHub. Looking for more ideas of links to add to the bottom of each blog post.
Mock-up (messing with the HTML in my browser):

Ok I've been thinking and I just realized after looking through some example blogs/news sites, we never indicate when these posts are published! (or maybe we do and I'm blind so correct me if I'm wrong haha).
My idea is something of the sorts:

Ok I've been thinking and I just realized after looking through some example blogs/news sites, we never indicate when these posts are published! (or maybe we do and I'm blind so correct me if I'm wrong haha).
My idea is something of the sorts:

I added the publish date to https://github.com/EthanThatOneKid/acmcsuf.com/pull/481

I was thinking maybe we could add a link to the github discussion so people can comment on it "Discuss on Github" or something?

oh yeah ethan informed me. that's a cool addition too maybe


ERROR: Get API Key Permissions

E/flutter (17177): [ERROR:flutter/lib/ui/ui_dart_state.cc(209)] Unhandled Exception: type 'Null' is not a subtype of type 'int'
E/flutter (17177): #0      new ApiRestrictions.fromMap (package:binance_spot/src/rest/classes.dart:1412:40)
E/flutter (17177): #1      WalletEndpoints.apiKeyPermission. (package:binance_spot/src/rest/endpoints/wallet.dart:487:57)
E/flutter (17177): #2      _rootRunUnary (dart:async/zone.dart:1436:47)
E/flutter (17177): #3      _CustomZone.runUnary (dart:async/zone.dart:1335:19)
E/flutter (17177):
Please fix it!

should be fixed now (in the last commit). The error comes when Binance doesn't send "tradingAuthorityExpirationTime" (didn't know it could do that since it is not written as optional), that is why I changed this class member to be an int? instead of an int


Ability to select the button orientation

provide options to choose if the button is centered, right or left

Thank you for the proposal.
Will add

Fixed and will be part of the next release


Async / await: strange things happen

I'm using CQ with async/await (i.e. the async procedure awaits the string to be downloaded, then processes it with CQ) called simultaneously, like this (pseudocode):
var tasks = urls.Select(ProcessUrl);
Task.WhenAll(tasks).Wait();
...

async Task<Detail> ProcessUrl(string url)
{
 var s = await DownloadToString(url);
 CQ dom = s;
 var link = dom["a.nextPrevLink"];
 // some DOM processing here, resulting in
 return new Detail { ... };
}

it works fine ... in theory.
Sometimes strange bugs happened.
When debugging I found out it randomly happens that the CsQuery selectors return malformed elements (like, an element with random "class" attribute which CERTAINLY is not in the document).
To make sure I've inspected the html content, and made sure that CsQuery is wrong.
An example: the "link" element (see above pseudocode) with class="ad_1byf"  attribute.
When inspecting s variable there's NO "ad_1byf" occurance AT ALL.
in the Immediate Window of Visual Studio I've tried to instantiate new CQ element, but running the same selector returns the same malformed code.
Now I suppose it might be related to relying on ThreadStatic or something similar, but I'm no expert, neither I know anything about CsQuery...
any suggestions how to fix that?

Hi, it's been a while, any updates here?
I'm about to choose HTML parsing engine for a new project, would love to use CsQuery again but I'm afraid of being affected by this issue again


Migrate to ReactiveMP 3.9.0

Flux is no longer needed for the CVI. Use Optimisers instead in tests.
Flux is still used in one of the examples

The CI most likely will fail unless a new release of ReactiveMP is merged in the General registry.

I'll restart the CI as soon as ReactiveMP 3.9.0 is released.

Codecov Report
Patch coverage has no change and project coverage change: -0.83 :warning:

Comparison is base (7a4a5e6) 80.29% compared to head (811f413) 79.47%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
- Coverage   80.29%   79.47%   -0.83%
==========================================
  Files          11       11
  Lines        1208     1208
==========================================
- Hits          970      960      -10
- Misses        238      248      +10

see 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


config: allowlist for tasks that can be configured as Jobs

We need a JOBS_ALLOWED_TASKS (or JOBS_TASK_ALLOWLIST?) list config variable to control what tasks can be eligible for being configured in a Job.

This is done in the sense that you now require to declare a JobType object, and only those are selectable in the UI when setting up a job.


WARNING: consider updating your call to make use of computation

Description of feature
Hey,
Sorry this is not particulary a feature request.
I would be grateful if you could shed some light on the warning: WARNING: consider updating your call to make use of computation
thanking you
shobhit

This is a warning from bbknn and they have updated their internal API. This will be soon'ish addressed when we release the next version. I have to conduct some additional tests for this version.


Inlet: Add Sphinx documentation generation option

Add a Sphinx documentation writer class that is over writable by the users to write basic Sphinx files that output tables that contain name, description, and other information about Tables and Fields defined by the user.

#290 provides this functionality


Simple step by step tutorial on how to get the manager and example tests setup & running

Is your feature request related to a problem? Please describe.
The readme does not mention the patch required to flutter test binding for tests to run and it also assumes a user knows how to navigate Github to get the build artefacts. It also mentions the example project, but there is no step-by-step tutorial on how to get this running alongside the manager.
Describe the solution you'd like
A simple step by step tutorial with screenshots on all the steps involved with getting the manager built/downloaded & running and the example tests running. Things I would also love to see documented are why and how to apply the patch from https://github.com/fzyzcjy/flutter_convenient_test/issues/337 and where to find the artefacts from the CI (screenshots on how to navigate the Github UI). Being able to follow this tutorial would give users more confidence that they know what they are doing and everything is set-up and working correctly 😄
Describe alternatives you've considered
Update the description of https://github.com/fzyzcjy/flutter_convenient_test/issues/337 with an example of what the binding file looks like after the changes as a git-diff is confusing to read for someone who doesn't use this feature of git.

I am happy to PR this once I am sure I know how to apply the patch correctly and get all the example tests passing :)

Looks pretty reasonable and looking forward to your PR!


[keyring-controller] Lock controller mutex on write operations

Explanation

Since all changes to the state of any keyring held by KeyringController should result in an update to the vault, we need to ensure that what we save into the vault is the result of the latest executed operation. In other words, we need to:

Ensure that each operation that changes the state is mutually exclusive. That is, if an operation is made of multiple steps, these steps are guaranteed to be executed before any other operation is started
Ensure that each operation is persisted to the vault before another operation can start

To do this, this PR adds the KeyringController.#withControllerLock helper function, to be used on any write operation that can potentially mutate the controller state (e.g. any function that calls this.persistAllKeyrings at the end).
References

Related to #4154

Changelog

@metamask/keyring-controller

CHANGED: KeyringController method calls that change state are now mutually exclusive

Checklist

[ ] I've updated the test suite for new or updated code as appropriate
[ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate

There are still some methods left out:

addNewAccountForKeyring
QR related methods

These are internally called by other methods that are already behind a mutex, so putting the same mutex also on them would mean ending up in a deadlock.
A solution could be to refactor some methods so that they don't call each other internally

Perhaps we could move the persistAllKeyring logic along with its vault lock to an internal private function, so that we can put a controller lock on the public persistAllKeyrings without having a deadlock, since the public function is currently called internally by many other methods

PR Reopened here: https://github.com/MetaMask/core/pull/4182


[ENH] predict_proba for forecasters, tensorflow-probability dependency

Relates to #1746, proposes interface and technical solution.
Solves interface/framework for predict_var and predict_proba in #1687.
This PR adds predict_proba and predict_var interface and default behaviour to BaseForecaster:

predict_var returns a pd.DataFrame of variance predictions
predict_proba returns a tfp Distribution with batch indices corresponding to fh indices

The default behaviour is as follows:

predict_var first looks if predict_proba is implemented, and gets variance there if it is. Otherwise, if intervals/quantiles are implemented, returns variance of a normal with same IQR as the predictions
predict_proba uses predict as location and square root of predict_var as scale parameter. If predict_var is not directly implemented, leverages its default behaviour to get this from intervals/quantiles.

This PR also introduces a soft dependency on tensorflow and tensorflow-probability, localized to a dl dependency set.
This dependency set is added to dependency specifications where necessary; a function _check_dl_dependencies to check and raise error if the dependency is not installed is introduced.
Further, this PR turns on a number of tests to work for predict_var and predict_proba, by adding defaults for these methods to the scenarios.
Depends on the following PR:

https://github.com/alan-turing-institute/sktime/pull/2105
https://github.com/alan-turing-institute/sktime/pull/2108

I think [all_extras] should still install all soft-dependencies. We can have the dl option as a subset of it. You can look here how this can be done: https://github.com/pandera-dev/pandera/blob/master/setup.py

You can look here how this can be done:

I can´t copy this, because we use pyproject.toml and not a python file to define the dependency sets?
Or do I misunderstand sth?
I can, of course, just copy the dl dependencies into the all_extras.
@lmmentel, @freddyaboulton, any opinions on this from the maintenance perspective?

I can, of course, just copy the dl dependencies into the all_extras.

@aiwalter, I just did that.


#147: fix npm install dependency issue

test pull request to investigate failing builds via github actions

Being previously a test pull request, this pull request does now contain the fix for the failing builds.


[Bug] Library Update process shuts down after a couple titles

Device information

Tachiyomi version: 2.3.2
Android phone & version: moto g power, android 11

Steps to reproduce
Slide down screen in Recents to update library
Expected behavior
The process should run and go through the library and update all of the titles
Actual behavior
The process crashes after 2   ##titles

Other details
Additional details and attachments.

Please update to 2.5.2 then try

My apologies, I mistyped there. I am on 2.5.2

Can you try again in ten mins or so. They have been making some API bug fixes so that might have solved this

Yep, it still is having the same issue. I'm not sure if it has anything to
do on my end or what.
On Wed, Jul 7, 2021, 6:03 PM Carlos @.***> wrote:

Can you try again in ten mins or so. They have been making some API bug
fixes so that might have solved this
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CarlosEsco/Neko/issues/563#issuecomment-875964320,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUYQBSXNZNVY54B7TGXGSY3TWTFMBANCNFSM477NX7NA
.

This should be the full log from the start of the library update, if it helps at all

1625698640180|D|||NEKO-TIMER|fetching chapters from Dex took 680 ms
1625698640189|D|||NEKO-TIMER|Fetch Chapters for  10,000 Light-Year Binoculars took 698 ms
1625698640712|D|||NEKO-TIMER|Manga Detail for  10,000 Light-Year Binoculars took 1220 ms
1625698640713|D|||NEKO-TIMER|Chapter and Manga Details for  10,000 Light-Year Binoculars took 1223 ms
1625698640740|D|||NEKO-TIMER|library manga 10,000 Light-Year Binoculars took 1254 ms
1625698640742|D|||NEKO|	├ eu.kanade.tachiyomi.source.online.handlers.MangaHandler.fetchMangaAndChapterDetails(MangaHandler.kt:26)
└ eu.kanade.tachiyomi.source.online.MangaDex.fetchMangaAndChapterDetails$suspendImpl(MangaDex.kt:81)
fetch manga and chapter details
1625698640767|I|||NEKO|	├ eu.kanade.tachiyomi.source.online.MangaDexLoginHelper.isAuthenticated(MangaDexLoginHelper.kt:44)
└ eu.kanade.tachiyomi.source.online.MangaDexLoginHelper$isAuthenticated$1.invokeSuspend(Unknown Source:14)
check token is authenticated false
1625698640768|I|||NEKO|	├ eu.kanade.tachiyomi.network.TokenAuthenticator$refreshToken$1.invokeSuspend(TokenAuthenticator.kt:36)
└ kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
Token is invalid trying to refresh
1625698641830|D|||NEKO-TIMER|Manga Detail for  30-sai made Doutei de Itara Mahou Shoujo ni Narimashita took 1085 ms
1625698641838|D|||NEKO-TIMER|fetching chapters from Dex took 1094 ms
1625698641840|I|||NEKO|	├ eu.kanade.tachiyomi.source.online.MangaDexLoginHelper.refreshToken(MangaDexLoginHelper.kt:55)
└ eu.kanade.tachiyomi.source.online.MangaDexLoginHelper$refreshToken$1.invokeSuspend(Unknown Source:14)
refreshing token
1625698641882|D|||NEKO-TIMER|Fetch Chapters for  30-sai made Doutei de Itara Mahou Shoujo ni Narimashita took 1138 ms
1625698641883|D|||NEKO-TIMER|Chapter and Manga Details for  30-sai made Doutei de Itara Mahou Shoujo ni Narimashita took 1140 ms
1625698641936|D|||NEKO-TIMER|library manga 30-sai made Doutei de Itara Mahou Shoujo ni Narimashita took 1195 ms
1625698641937|D|||NEKO|	├ eu.kanade.tachiyomi.source.online.handlers.MangaHandler.fetchMangaAndChapterDetails(MangaHandler.kt:26)
└ eu.kanade.tachiyomi.source.online.MangaDex.fetchMangaAndChapterDetails$suspendImpl(MangaDex.kt:81)
fetch manga and chapter details
1625698642011|E|||NEKO|	├ eu.kanade.tachiyomi.data.library.LibraryUpdateService$updateChaptersJob$3.invokeSuspend(LibraryUpdateService.kt:320)
└ kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
kotlinx.serialization.json.internal.JsonDecodingException: Unexpected JSON token at offset 25: Expected start of the object '{', but had ':' instead
JSON input: {"result":"ok","statuses":[]}
1625698642013|D|||NEKO|	├ eu.kanade.tachiyomi.source.online.handlers.MangaHandler.fetchMangaAndChapterDetails(MangaHandler.kt:26)
└ eu.kanade.tachiyomi.source.online.MangaDex.fetchMangaAndChapterDetails$suspendImpl(MangaDex.kt:81)
fetch manga and chapter details
1625698643742|E|||NEKO|	├ eu.kanade.tachiyomi.data.library.LibraryUpdateService$updateChaptersJob$3.invokeSuspend(LibraryUpdateService.kt:320)
└ kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
kotlinx.serialization.json.internal.JsonDecodingException: Unexpected JSON token at offset 25: Expected start of the object '{', but had ':' instead
JSON input: {"result":"ok","statuses":[]}
1625698656919|D|||NEKO|	├ eu.kanade.tachiyomi.ui.base.controller.BaseController$1.preDetach(BaseController.kt:47)
└ com.bluelinelabs.conductor.Controller.detach(Controller.java:984)
Detach view for RecentsController@26165b9
1625698661123|D|||NEKO|	├ eu.kanade.tachiyomi.ui.base.controller.BaseController$1.preAttach(BaseController.kt:43)
└ com.bluelinelabs.conductor.Controller.attach(Controller.java:942)
Attach view for RecentsController@26165b9
1625698664520|D|||NEKO|	├ eu.kanade.tachiyomi.ui.base.controller.BaseController$1.preDetach(BaseController.kt:47)
└ com.bluelinelabs.conductor.Controller.detach(Controller.java:984)
Detach view for RecentsController@26165b9

Also, I just checked to see if I could update individual titles manually, and it seems like for some titles it does work. I'm not sure if the search is just crashing at an early step or what. I have noticed that with the regular Tachiyomi app that it will crash if I search for some titles across multiple sources, though I'm not sure how that might relate to this issue

fixed in 2.5.3

Same issue and persist after update.

Same issue and persist after update.

Can you message on the neko channel on discord


Route through TOR won't connect at all.  All network traffic stops

the route through TOR button on the network manager does not work.
pkg install TOR (it's missing during install)
If TOR is not installed the menu item should not show up.
However when TOR is installed after pressing the route through TOR
it launches the browser and won't connect.
No TCP traffic works, not pings not DNS nothing.
Uncheck the route through tor network and everything works again.
This has been a problem for 2 years and no one wants to fix it.

by the way if you run TOR from the command line it works great and builds a circuit. However nothing will route through it.  So while tor is running if you press route through TOR once again nothing.

expected behavior: if you run openvpn (or any vpn) then route through TOR it should send the tor traffic through the VPN.  This has never worked in TRUEOS but works great in linux or mac.  It is a "best practice" to route tor through a VPN first to hide that you are using TOR.

Closing this stale issue.
The Tor mode scripts were re-written a few months ago specifically for Project Trident and appear to be working fine now.


Driver Drowsiness Detection

Is your feature request related to a problem? Please describe.
Since countless people drive on the highway day and night. Taxi drivers, bus drivers, truck drivers, and people traveling long-distance who suffer from lack of sleep. Due to this, it becomes very dangerous to drive resulting in the majority of accidents happening due to the drowsiness of the driver.
Describe the solution you'd like
A project that aims to build a safety technology that will detect if the driver’s eyes are closed for a few seconds and will alert the driver when drowsiness is detected.
Describe alternatives you've considered
Track traffic police officers at intervals with flashlights to keep the drivers in check. Track systems for any signs of improperly driven vehicles.
Approach to be followed (optional)
The project focuses on warning the drivers of any vehicle at any time of the day/night in case of drowsiness while driving. It can majorly help prevent accidents and make the driver retrieve consciousness to focus on the path properly. The requirement for this project is a web cam through which we will capture images of the vehicles along with the drivers.
Additional context
In this project, I'll be making a Driver Drowsiness Detection System built using Python, Open CV, AND Dlib which will help reduce accidents and their impacts by warning the drivers depending upon their physical state.

Please assign this project to me and allow me to contribute my hard work to GSSoC'23.

Hi Team, I would love to contribute to this issue, Please assign me for enhancement.

Hello i m well expirenced in ml . please assign this issue to me

i would like to contribute to this issue. Please assign this task to me.

Hey @akshitagupta15june I have worked on various computer vision problems and have a good experience with computer vision algorithms related to faces.I would be very happy to work on this issue and contribute to this awesome repo.
Thank You!

Hey! I would like to contribute to this project under SSOC. Can you please assign it to me?


Test installation of stable OBS packages

See: https://github.com/kata-containers/documentation/issues/625.
I think the simplest solution is to update the kata-manager script to accept an optional branch name so that if you wanted to install a specific branch version of Kata, you could run, say:
$ bash -c "$(curl -fsSL https://raw.githubusercontent.com/kata-containers/tests/master/cmd/kata-manager/kata-manager.sh) install-docker-system stable-1.10"

(see https://github.com/kata-containers/documentation/blob/master/install/installing-with-kata-manager.md#full-installation)
The default should remain master.
We can then update the test below to test master,  and the last 2-3 stable branches:

https://github.com/kata-containers/documentation/blob/master/.ci/test-install-docs.sh

/cc @cmaf - this might be an interesting issue for you to look at?

@jodh-intel I'll look into it

@jodh-intel  thanks for confirm, @grahamwhaley  I was not aware recent release was driven by @bergwolf probably he has more information

actually looking to https://build.opensuse.org/package/show/home:katacontainers:releases:x86_64:stable-1.9/linux-container seems that actually the build was broken due to problems getting the new tarball for the kernel, I restarted the services from OBS to see if that helps

@jcvenegas Thanks, just checked and seeing this error on
https://build.opensuse.org/package/show/home:katacontainers:releases:x86_64:stable-1.9/linux-container
Files could not be expanded: service error: Running /usr/lib/obs/service//download_url --protocol https --host cdn.kernel.org --path /pub/linux/kernel/v5.x/linux-4.19.75.tar.xz --outdir /var/cache/obs/cI9aRdTaTjKP/out more info

Looks like kernel 4.19 is being downloaded with  "v5.x" in its path, which does not seem right.
Can you take a look. cc @chavafg
Going forward, I also think, the status for whether the packages have been generated and tested should be visible and easily accessible when we make a release similar to github build status badge maybe, so that we can mark a release as complete.
(Does azure pipeline support this?)

In addition, other users have reported in https://github.com/kata-containers/runtime/issues/2574#issuecomment-606147304 that latest alpha1 is not in OBS master directory, which currently has the alpha0 release. The alpha1 packages can be found in alpha directory.
Seems confusing to me, is this correct?
And do we have documentation about the different directories (master, head, alpha) in OBS?

@amshinde I fixed the packages manually still need to to package testing, I found that the packaging tag for 1.9.6 was in master and not in the stable branch. @bergwolf did you had any issue with tagging with the script? Or probably was done the tagging by hand. Feel free to open an issue if you found something wrong.
@chavafg  I started both jobs for packages againg 1.9 and master to get it in the correct place.

@jodh-intel Is your original suggestion still relevant?

@cmaf - yes, I think so. Once we can run kata-manager.sh specifying a branch/release, we can automate testing the last 'n' stable releases in Azure/Jenkins/whatever.

Nice! Now that @cmaf's https://github.com/kata-containers/tests/pull/2446 has landed, we can create a test in this repo I think to test the both master and the last "s" stable releases (where s=2 - see https://github.com/kata-containers/documentation/blob/master/Stable-Branch-Strategy.md#branch-management).
The question is which approach do we take to convert this into a test?
a) Create s * d Jenkins jobs, one for each stable branch we want to perform install tests for (where s is the number of stable branches and d is the number of different distro versions we use in our CI).
b) Create d Jenkins jobs, where d is the number of different distro versions we use in our CI, and make the 9documentation repo CI test](https://github.com/kata-containers/documentation/blob/master/.ci/test-install-docs.sh) do the following:

Determine the list of stable releases to run the tests for.
This is going to require a small amount of manual intervention, but I'd suggest we use a YAML database like this:

 supported-branches:
   - master
   - stable-1.11
   - stable-1.10

Note: the CI script should always query the version of this file from the master branch.

Run the install tests for master plus each of the last s stable branches from the database above.

I prefer option (b) since:

There is less crazy-mad Jenkins XML hacking involved.
It almost maintains itself - we just need to update supported-branches.yaml.

And once we have a supported-branches.yaml database in this repo, we can hook it into all sorts of other interesting areas such as:

Our release process.
We do need to think carefully about OBS though - how are we going to make the docs CI test pull binary packages from OBS before we've officially announced a release? Do we use a staging area on OBS that the docs can pull from? But if so, the install docs would be incorrect when we announce the release. Sounds like we may need another variable in the install docs in addition to BRANCH possible? /cc @jcvenegas.
When we create a new stable branch, we add it to the YAML file, and delete the oldest stable branch from it.

The stable branch strategy doc.

OBS packaging.

/cc @amshinde, @gnawux, @grahamwhaley, @jcvenegas.

@jodh-intel in option (b), would test_install_docs.sh test the installation of all 3 branches in 1 vm or container instance? (using the remove-packages or remove-docker-system option)

Hi @cmaf - option (b) would test for example master, stable-1.11 and stable-1.10 in a single Jenkins VM "node", yes. That script would installing a version of Kata, test it, then remove it before installing the next version. We'd need to ensure that /etc/kata-containers/ doesn't exist before the tests to ensure we're only testing with the packaged configuration (/usr/share/defaults/kata-containers/).
What we do need to ensure though is that we have a Jenkins job for the documentation repo for each distro version we provide an install guide for. Looking at http://jenkins.katacontainers.io/, we currently have atleast one version [1] of:

Ubuntu
Fedora
OpenSUSE
CentOS

But we don't have the following:

debian
RHEL
SLES

This raises the issue of our CI resources again. If there are any community folk who could help us out by providing CI resources for distros / distro versions they care about, please do get in touch!!

[1] - Ideally, we'd have two versions in most instances: the latest LTS version (if applicable) plus the latest version (if different). But for distro versions without an LTS version, the last two supported versions would make sense (so CentOS 7 + 8).

@jodh-intel Ok, in that case, I ran into some problems (such as broken packages on Ubuntu, and not properly downgrading on CentOS) using kata-manager.sh uninstalling and reinstalling packages, both in increasing and decreasing order.
Some of it can probably be handled in cmd_remove_packages like using autoremove. Otherwise, using downgrade or installing in increasing order.

@cmaf - ah - sorry, I should have been clearer. I don't think these tests should be testing upgrade / downgrade (atleast at this stage). W should install each version, perform a basic test to ensure it "works", then totally remove all traces of it before installing the next version. Essentially do something like the following in pseudo-code:
for version in master stable-1.10 stable-1.9
do
    # also removes qemu packages
    remove_kata_packages()

    # removes /etc/kata-containers/
    delete_old_config()

    install_and_test_kata_version("$version")
done

If the kata manager script isn't correctly cleaning up, that's obviously a bug. We do need to be careful with that script given it's run by users for the automated install.

@chavafg I vote for the second option

I guess I'd vote for option (2) as it avoids constant Jenkins updates. But although I like the idea of automatically determining the previous set of stable branches, I worry slightly that something "odd" may happen to cause it to silently fail. For example, if a new strangely named stable branch was created (even by mistake), we may end up testing that rather than the correct one, giving a false sense of security. Hence, although both these options introduce a small amount of human interaction, I think we should do one of:

Require a manual update to the supported-branches.yaml  file.
I say "manual", but maybe "explicit" would be a better word: it could be updated automatically as part of the tooling we have to create a release. But atleast doing it that way, we've stated explicitly which branches/tags we consider to be stable releases. And anyone (or any other tool) can go and inspect that list.

Use @jcvenegas's magic method, but require the CI to mail the publish those test results somehow / somewhere and also require that we get a human to check those results before announcing a release (again, to ensure there are no mistakes).

I'd vote for the 1st option above personally.

Closing this since:

The focus is now on Kata 2.x.

We are no longer producing OBS packages for Kata 2.x.

Most distros are now producing their own Kata packages, which reduces the importance of this issue:
https://github.com/kata-containers/kata-containers/tree/2.0-dev/docs/install#official-packages


The primary key field will be automatically appended

Description
When the field is a primary key but not self increasing, using create will automatically append the primary key field, resulting in a statement error
Screenshot

let GORM know this is not an auto increment property

let GORM know this is not an auto increment property
I clearly informed him that it is a primary key, not a self increasing primary key, but he did not consider whether the primary key is self increasing


Spyder was unable to receive the value of this variable from the console error after renaming a class name

Problem Description
After renaming a class name in a module, browsing structures containing class type elements in the variable explorer fails with the error Spyder was unable to receive the value of this variable from the console. The error message was: AttributeError("Can't get attribute 'Correctname' on '').
The error is gone after either closing and re-opening Spyder or renaming the module/filename. Therefore it looks like some caching problem to me (Tools - Update modules names list or Consoles - Restart kernel does not fix the problem though).
What steps reproduce the problem?

Create a class Typoname in a module mytools
Write a script creating a list filled with Typoname-type elements
Run the script and browse the list and its sub-elements in the variable explorer

Rename the class Typoname to Correctname in both the module and the executed script
Re-run the script (Spyder notifies about Reloaded modules: mytools)

What is the expected output? What do you see instead?
The variable explorer should still allow browsing the list and its sub-items with the new type name Correctname. Instead the error appears:

Also, when inspecting the module/variable mytools in the variable explorer, it still holds the old name Typoname as attribute.
Versions

Spyder version: 3.3.3
Python version: 3.7.2 (Anaconda 4.6.7)
Operating System name/version: Windows 10

Dependencies
IPython >=4.0     :  7.3.0 (OK)
cython >=0.21     :  None (NOK)
jedi >=0.9.0      :  0.13.3 (OK)
matplotlib >=2.0.0:  3.0.2 (OK)
nbconvert >=4.0   :  5.3.1 (OK)
numpy >=1.7       :  1.15.4 (OK)
pandas >=0.13.1   :  None (NOK)
pycodestyle >=2.3 :  2.5.0 (OK)
pyflakes >=0.6.0  :  2.1.0 (OK)
pygments >=2.0    :  2.3.1 (OK)
pylint >=0.25     :  2.2.2 (OK)
qtconsole >=4.2.0 :  4.4.3 (OK)
rope >=0.9.4      :  0.12.0 (OK)
sphinx >=0.6.6    :  1.8.4 (OK)
sympy >=0.7.3     :  None (NOK)

Thanks for reporting. We'll take a look at this for Spyder 4.

In my case I need to run file which contain objects definition. By this I mean that I have two files:
-the_one_with_definitions.py
-the_one_using_definitions.py
Inside of the_one_with_definitions.py:
class TimePoint:
    def __init__(self):
        self.a=1

Inside of the_one_using_definitions.py:
import the_one_with_definitions
tp= the_one_with_definitions.TimePoint()

When I run the_one_using_definitions.py Variable explorer works fine.

I have also similar issues to retrieve the value from console. Its work perfectly when I do not call any DLL in my program, but getting error when I call dll. I would appreciate your help to solve this problem


The new and improved auro-input

General Support Request
Given the current architecture of auro-input, its code base is way too complex and the single component is responsible for too many individual interactions. This breaks all the rules of single responsibility and is simply an anti-pattern we need to avoid.
Support request
For the next generation of auro-input we should create a basic input element that only addresses the common needs a consumer expects to happen within an input element.
Possible Solution
Let's create a new base element within this repo which is the basic features of auro-input. A suggestion is to also break up base-input.js into its individual functions as individual .js files. That way we can see what is specifically needed for this basic input and what will be abstracted into other input interfaces.
At this time, do not extend classes. Go with a simple architecture that is less confusing.

create a new auro-inputBase.js file
create a ./js directory for all functional dependencies
Add new auro-baseInput to demo view to eval side:side

Consider: better or worse... creating an individual <auro-label> element and possibly a <auro-helptext> element as to decouple these content parts away from the element as a whole.
Context
It's also worth looking into this issue as it pertains to passing attributes down from element to the template

https://github.com/AlaskaAirlines/auro-button/issues/176

Exit criteria
This issue will be considered complete once there is a new base input element that is considered shippable. Lessons learned from that will allow us to make decisions moving forward for either extending or creating dependencies on the base input for the other custom support features.

Additionally we need to consider adding phone number support/validation within the component family. Please reference @v-ukaranji for any updates pertaining to this request.

First story ready for work

https://github.com/AlaskaAirlines/auro-input/issues/219


Port relay_field from ROS1

Description
Would be nice to have https://github.com/ros/ros_comm/pull/639 in ROS2

@christophebedard @emersonknapp could you please assign this to me? I'm working on it over at #42 Thanks.


PI value on Grant Detail table still has "loading... " value

Should be solved now

will wait to see behavior once user service is integrated.


Added event status STREAMING

New STREAMING event which passes back current progress while audio is playing or paused.

How is it behaving with audio stream without duration like the one in the example app ?

I had only tested with fixed length mp3 files. I will try to test with the live stream example today.

Normally, the status for STREAMING will have duration and progress where duration is a fixed float number representing the length of the audio (in seconds) and the progress is a float that increases as you play the audio.
With a live stream, the duration stays at 0 and the progress increases from 0 to however long you listen.
As for the seekToTime, goBack and goForward functions - they are simply ignored when using a live stream. We could add code in the example where IF the duration is zero, then hide (or disable) the goBack and goForward buttons).
See the screenshot below for a live stream from the console:

👍 , thanks !

How can I use this event? Is there any documentation for it?

It is not documented yet because it's only available on iOS.

Can you give me some reference? I need this feature very desperately. :)


feat: add messages and service functions for cursor based paginated queries

Closes #418

I'm going to close this PR because it looks like I messed up the rebasing. I'll resolve the issues and then put up a new PR


Add aspect-ratio support

This does add a bit of space to the core file, but seems to be used often enough in FINN it should be supported

With this approach we kinda can't. It just adds X and Y rules, so the user can cook their own AR.
e.g.
aspect-w-16 aspect-h-9

So it's pretty difficult to eliminate any single number from the support. 🤷

Right yea, you can only filter to/out specific numbers not really combos. I suppose if you only allowed ratios it might work, or maybe it wouldn't. Wasn't clear to me from the docs. Probs not a big size increase?

Before
30.5 kB: fabric.min.css.br
460.81 kB: fabric.min.css.min.css

After:
31.1 kB: fabric.min.css.br
468.06 kB: fabric.min.css.min.css

Pretty sizeable hit, so I'm OK if we wanna support this another way - but does seem to be used enough it might be worth just eating it?

Yea, we could end up dying from a thousand paper cuts as we add features back in 😆
I don't have much to add regarding whether its worth it or not in this case. I've asked all my questions 🤣 🚀


License

Hiya - LICENSE file is missing. It'd be great to know if this is under Apache license.
Thanks!

Good point, we'll use the same as Kubernetes (Apache).


router crash when used with Solid 1.7

Describe the bug
After upgrade to the latest version of solidjs and @solidjs/router, the router crashes with the following stack trace:
Uncaught TypeError: route.outlet is not a function
    at get children [as children] (components.jsx:65:68)
    at Object.fn (dev.js:994:35)
    at runComputation (dev.js:695:22)
    at updateComputation (dev.js:678:3)
    at createMemo (dev.js:250:10)
    at children (dev.js:576:20)
    at dev.js:994:14
    at untrack (dev.js:434:12)
    at Object.fn (dev.js:990:36)
    at runComputation (dev.js:695:22)

As I understand Solid 1.7 introduce the ability for passing "child" signals without the keyed property in Show component
I think that the following commits will solve it, but I'm not sure if it is released or not:

https://github.com/solidjs/solid-router/commit/a8a32b97ba47128f47e316455e65fde4b1c6f126
https://github.com/solidjs/solid-router/commit/2212e2cd0f6d657c5f353dd2916c4526931d10cb

Your Example Website or App
.
Steps to Reproduce the Bug or Issue
example code:
<Routes>
    <Route path="/home" element={Hello} />
    <Route path="*" element={World} />
</Routes>

Expected behavior
As a user, i expected it to work 😝
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome
"solid-js": "^1.7.1"
"@solidjs/router": "^0.7.1"

Additional context
No response

My bad, I thought 0.71 is the latest, and it is currently 0.82


🐛 Bug - type error in nextjs 14.2.4

@google-translate-select bug report
Summary

this is the type error am getting when i run it on nextjs

TypeError: (0 , index3_mjs__WEBPACK_IMPORTED_MODULE_3_.jsx) is not a function
Environment
Next.js 14.2.4

I tested using both chrome and brave but it safe to assume that the problem is simply next.js
Steps to reproduce

install on nextjs

npm install npm install @google-translate-select/react

import and use it

"use client"

import GoogleTranslateSelect from "@google-translate-select/react";

export default test = () => {
return (
  <div>
          <GoogleTranslateSelect
            defaultLanguageCode="en"
            defaultPageLanguageCode="en"
            fetchBrowserLanguage={false}
            trigger="click"
            select={handleGoogleTranslateSelect}
          />
        </div>
);
}

Run it

npm run dev

Expected results
to work like how it works on the projects i have used on vue

Actual results

Marking issue as stale since there was no activity for 30 days


CSV Delimiter error

The map "Caer Blaidd, Wolfpack's Den" will cause an error when loading the CSV. Suggest wrapping map column in quotation marks. Alternatively use semicolon as delimiter, should be more robust.
Loaded into Excel

Displayed in VSCode

Ah, right. Thanks for reporting.


Use of System.exit() on parser errors in CSSParser

Hi Phil,
thanks for the jCSS-Parser! It works very well and has been very easy to use! :smile:
However, there is one strange thing. On parser errors (in my case I had a regular "//" comment that is not allowed in CSS somewhere) the handler.handleError() is correctly called but then immediatly after that a System.exit(0) is performed.
I think this is a bit harsh :wink:
Is there a special reason for that or can this be changed into a throw new IOException() or something similar instead?
I've found it on three spots in the CSSParser class here:
https://github.com/phil-brown/jCSS-Parser/blob/master/CSS Parser/src/self/philbrown/cssparser/CSSParser.java#L83
https://github.com/phil-brown/jCSS-Parser/blob/master/CSS Parser/src/self/philbrown/cssparser/CSSParser.java#L304
https://github.com/phil-brown/jCSS-Parser/blob/master/CSS Parser/src/self/philbrown/cssparser/CSSParser.java#L460
I can provide a pull request if your time is limited but just wanted to ask if this has to be that way - although I can't think of a reason, right now.
void

@void256 great suggestion and thanks for the positive feedback! I'll get a fix in very soon.

@void256 I have not touched this code in quite some time, but the reasoning that the exception is handled as-is, is that a syntax error might cause the rest of the parser to work incorrectly. The // syntax is - from what I have read - not the correct way to specify a comment. According to w3schools.com, /*comment*/ is the correct syntax. It does seem harsh though, considering that it would cause applications (such as integration in Android, etc) to have issues, so I definitely agree with that part. So to approach the issue, I plan to add a custom ParseError that will stop all parsing, but not call System.exit(). What are your thoughts? Does using the /* syntax work for the issue described above? If not, the correct place to add code to include this sort of comments would be line 262 of Scanner.java. I'll add an improved exception handling later today, and thanks for pointing this out!
Also, I am really excited that someone is getting some use out of this, and that it is working well :smile: Please continue to report any issues you find. I will be happy to help.
Best,
Phil

It's not necessary to add support for // comments since the correct way in CSS is to use /* ... */. That is ok! :smile:
I think it's okay to stop parsing in that case. Why not simply throw an IOException to notify the caller of the error? The caller of the parse() method already has to take care of that Exception and in case of a SyntaxError he's kinda doomed anyway :wink:
And yes, I like your project because it does what it says, it simply parses the CSS and that's it. Most other projects I've checked are created with HTML/CSS processing in mind so they all do more than what I needed.
So far, I did not have other issues :smile: The only thing I'd eventually like to see would be the packaging of your lib as a Maven dependeny to make it available in the central Maven Repository. It would make re-using your lib for Maven users easier! But that is a minor thing really.

@void256 I just made some minor changes, and created a new release, available here. This now throws a ParseException if the parsing fails, but no longer calls System.exit(0). Again - great find! I'm currently looking into how to add Maven support, I definitely would like to add that option in the near future.

Will try it as soon as I can. Thanks already!

Works as expected! Thanks for the quick fix! :smile:


Fix semver lt

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

18 of 18 (100.0%)  changed or added relevant lines in 3 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.2%) to 83.197%

Files with Coverage Reduction
New Missed Lines
%

src/hooks/useConfig.ts
1
81.46%

src/plumbing/link.ts
5
55.07%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
2226

Relevant Lines:
2600

💛  - Coveralls


if the screen is resized and the prompt no longer fits on one line the prompt is corrupted

if the amount of text that can fit on a line is changed that causes the current prompt to reflow across multiple lines then the existing prompt gets corrupted and multiple items are repeated.
could be a clink issue also?
clink version  : <IP_ADDRESS>3e64
flexprompt 0.9
Can be reproduced by any way that changes how much text can fit on a line, e.g. zooming (changing font size in terminal or changing terminal window width.
video shows zooming as it is easier to capture
https://user-images.githubusercontent.com/494726/177745015-eff15105-dcc5-4c19-8570-36eb0ed806d1.mp4

Please share your flexprompt_autoconfig.lua file, and/or any Lua scripts that configure the prompt.
I don't experience that degree of mangling.  The OS resizes the console asynchronously, which means it's impossible for any program to handle resize entirely correctly.  But I've never seen the kind of explosion of text that your video shows.  You may be right that flexprompt is accidentally appending content repeatedly.  But I can't reproduce it, so it might be related to your specific prompt configuration.

Well, my best guess so far is that it is flexprompt at fault.
Because:

I made a prompt filter to write the resulting prompt string to a file after flexprompt built it.
I made an alternative prompt filter that simply uses that pre-made string, instead of using flexprompt.
The identical prompt string works fine outside of flexprompt, but runs amok inside of flexprompt.

But so far I don't see how flexprompt could be causing this.  Still looking...

Looks like an issue in Clink:  It's caused by the presence of a right-sided prompt string + wrapping the prompt.  When the prompt wraps onto a second (etc) line, the logic for showing the right-sided prompt puts the cursor on the wrong line.


Edit TypeScript style guideline to ban engineers to write TS code at the initial phase of the migration

Add an instruction to ban new code to be written in TypeScript in the initial phase of the migration.
As par this Slack conversation (internal link), we don't want any engineers outside the migration team to write TypeScript code at this point.

This doesn't need payment or bug zero checklist. Closing this issue


Android

Somewhere in the past few releases I broke Android, probably while I was making the window-management less irritating. Find+fix.

Apparently I routinely trigger this bug: on Android

Oh, that was foolish, it's because of this: https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/tabs/create compatibility issue, which would require fixing this issue: https://bugzilla.mozilla.org/show_bug.cgi?id=1398097, which someone else has mostly fixed already. Not sure why they don't want to allow contextual identities on Android but I'm going to raise the issue and see if they'll let me add it to the existing patch.


support incremental command navigation in the terminal accessible buffer

fix #179272

I need to test on Windows because on MacOS with voiceover, just positioning the cursor isn't enough for the screen reader to read the content as I navigate.
We might also need to set the selection here. Or can await feedback...

not sure why the tests are failing, they look unrelated

started failing here, so reverting that
https://github.com/microsoft/vscode/pull/179415/commits/1945e48fad744ce87600c007ad1a44cc12796d21

there was a cyclic dependency


Trigger monitor messages on VM state changes

Description
IM probes in kvm (collectd) report each monitor_interval.
Use case
The idea is to be able to relax the monitoring time and be notified of changes of state:

for instance, each 5 minutes report to oned
but if there is a change in the state of any of the VMs, report. Also, a MONITOR_PERIOD can be defined per VM

Interface Changes
Only conf files

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Done in new monitor


Forms to Contacts not working

We're using Umbraco Forms and the HubSpot CMS integration, but we're not receiving any of the contacts after form submissions. I’ve already authorised the workflow and connected it through our HubSpot account. I’ve also added in the field mappings, is there something we’re missing?

@LawrenceWorks could i have an update? Thanks

Hi @LawrenceWorks ,
Could you tell me which versions of Umbraco/Forms/Hubspot you are using? I've tested against a version 10 with private access tokens and OAuth, and could not replicate the issue.
Thanks

Hi guys,
Apologies for the late reply, in the end, I couldn't get it to work so I've just gone with creating a custom workflow and used their API documentation to hardcode the fields we need from our forms to Hubspot contacts.


medical_ts_datasets is 3 commits behind master

The changes that @ExpectationMax made in August are not reflected in the commit id of the submodule. Is this safe to update?

Hey maresb,
I checked the changes you mentioned. 3 buggy patients without time series observations were removed from the physionet 2019 dataset, otherwise an update is safe thus I bumped the version in the repo.
Cheers


add line-fill plot calls

implements the line fill plot codes in blocks 0x48, 0x58, 0x68 and 0x78

this implementation is not efficient, but it works and provides compatibility for these previously missing plot codes
a more efficient implementation will likely come at some point - essentially that will involve moving this functionality into vdp-gl.
(this version has been integrated into the Console8 VDP variant and is present in the 2.2.0 release)


DRILL-8008: Add Config Option to HTTP Plugin to Skip SSL Validation

DRILL-8008: Add Config Option to HTTP Plugin to Skip SSL Validation
Description
In the current implementation, Drill validates all SSL certificates when querying REST APIs.  In some circumstances, such as a corporate network, or for testing, a user might want to disable this functionality.  This PR adds a config option to the HTTP plugin to disable SSL validation.
Documentation
Update README with the following:
verifySSLCert
Default is true, but when set to false, Drill will trust all SSL certificates.  Useful for debugging or on internal corporate networks using self-signed certificates or
private certificate authorities.
Testing
Manually tested.

@vvysotskyi Thanks for your quick review.  I addressed your comments.  Anything else?


Add category Feature/#64

Improvements:

Ability to archive categories (same logic as PATCH /api/questions/{id}/archive and PATCH /api/questions/{id}/unarchive);
Endpoint for getting single category by id.


Articles disappear when moving feed to Feedbin

I added a feed (blog.halide.cam) to "On my Mac", it fetched 10 Articles.
When moving the feed into a Feedbin-Folder, all but the newest article disappear.
They reappear if the feed is moved back.
In the Feedbin web interface, they all appear, even though only one is shown in NNW.

We only sync the last 90 days of articles for Feedbin and the other syncing services.  We do this so that we aren't syncing too much data and over taxing the syncing service.
The On My Mac account works differently and will sync any article still in the feed regardless of how old it is.  That is why you are seeing a different number of articles between the two accounts.
This is normal behavior, so I'm going to go ahead and close this issue.  Feel free to reply if you have any questions you would like answered.

On iOS the timespan seems to be 30 or 31 days, or am I mistaken?

iOS is 90 days as well.

I saw, that one can only switch read/unread on articles which are at most a month old, so I guessed those things were connected.

You should only have the read/unread marking restriction of 30 days on Feedly accounts.  Did you observe this on a Feedbin account?

Ah, no, on a Feedly account. Sorry for the confusion.


IsAdmin Util

We need a helper function for admin only mutations and queries.  checkout https://github.com/get-convex/convex-helpers/blob/main/packages/convex-helpers/README.md#custom-functions make a custom query and mutation

on it


Added kraken assignment to creation of hvreads.

@jeffkaufman, as discussed, I slightly edited the run.py code to add Kraken assignment information to *.hvreads.json.

Thanks!  This looks correct, but there are other things we'll need to do after this.

Right now run.py file has:

existing_outputs = get_files(args, "hvreads",
                                # date we added quality scores
                                min_date='2023-09-18')

which will need to be updated to the current date so old hvreads are regenerated.

Everyone will need to rm -r dashboard/hvreads/

Everything downstream that reads hvreads will need to be updated to use the new format.  I think the easiest way to do this is just make the change and then fix what crashes.


Change ats exec check from traffic_cop to traffic_manager.

Traffic_cop is removed in 8x and manager now handles its duties. Manager was always the one to handle traffic_ctl communication though so this should be a valid change
for 7x as well.

What does this PR (Pull Request) do?

[ ] This PR fixes #3458
Which Traffic Control components are affected by this PR?

CDN in a Box
Documentation
Grove
Traffic Control Client
Traffic Monitor
Traffic Ops
Traffic Ops ORT
Traffic Portal
Traffic Router
Traffic Stats
Traffic Vault

What is the best way to verify this PR?

If this is a bug fix, what versions of Traffic Ops are affected?

The following criteria are ALL met by this PR

[ ] This PR includes tests OR I have explained why tests are unnecessary
[ ] This PR includes documentation OR I have explained why documentation is unnecessary
[ ] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[ ] This PR includes any and all required license headers
[ ] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[ ] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Can one of the admins verify this patch?

@ocket8888 yea I dont know why the formatting got all jacked up, but I think I got what i could now

You SHOULD be able to check every box. It's been changed recently, so they're truly items on a checklist.


bump up the version to 7.1.0 for nuget publish to work

Hi @skomarag, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (skomar). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Datalake store tests are failing and I see a note that BeGoldSm will be fixing it (test failing by design)
As skomar has a demo Tuesday 10 AM.
Merging this simple change for version bump.