validation package rewrite

This is the rewrite of the validation package I mentioned in #13.
I opened a new PR so you can view the approach. If you think its correct we can continue with it in this style. If not I am happy to just contribute the env parser and we can forget about this approach to validation.
However I do think it has quite a lot of advantages
Here is an example schema
	schema := Schema{
               // I have added support for slices
		"emails":     Slice(String().Email()).Min(1),
		"name":      String().Min(3).Max(10),
		"age":       Int().GT(18),
		"isMarried": Bool().True(),
		"lights":    Bool().True(),
                 // each data type has typesafe methods. You can't add email validation to a float for example
		"cash":      Float().GT(10.0),
		// refine allows the implementation of a custom rule by just providing a rule name, error message and a func
		"swagger": String().Refine("swagger", "should be doweird", func(rule p.Rule) bool {
			return rule.FieldValue.(string) == "doweird"
		}),
	}

I won't say the approach is perfect, it does have some trade-offs:

I have had to write quite a lot more code. Since each type has its own struct with its own methods. A lot of the checking logic is reused in but its still more code
For optional overriding of default validation messages (not yet implemented) we have to do something like this which is a little nasty

func (v *StringValidator) Min(n int, msg ...string) *StringValidator {
	defaultMsg := fmt.Sprintf("should be at least %d characters long", n)
	if len(msg) > 0 {
		defaultMsg = msg[0]
	}
	v.Rules = append(v.Rules,
		p.LenMin[string](n, defaultMsg))
	return v
}

For more examples of validation I suggest you look at the tests. I have also implemented a requestParams validator for example.
WARNING: code won't compile because I have not changed every instance of the use of the previous methods. If you decide to move forward with this implementation I'll make a quick update to the PR

I am going to close this for now as I realised there were quite a lot of problems with the architecture and I am modifying it quite a bit. I'll bring it back once I think its in a more stable state


cmd/wasm-dump: implement -x, -s and -d

This change is

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Incorrect List behavior when current page is bigger than pages count

Page handling code should be synchronized with ScrollingGroup

add pr #414


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.4.133",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=bdf5334250c69fabf555b7322c75249ea7d5f148"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "bdf5334250c69fabf555b7322c75249ea7d5f148",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "atm: nicstar: Fix possible use-after-free in nicstar_cleanup()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.4.133 by commit bdf5334250c69fabf555b7322c75249ea7d5f148. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001357


Multicore-aware executors

Thank you!
Thank you for your contribution.
Please make sure that your submission includes the following:
Must

[x] The code compiles without errors or warnings.
[x] All examples work.
[x] cargo fmt was run.
[x] Your changes were added to the CHANGELOG.md in the proper section.
[x] You updated existing examples
[x] You or added examples: interrupt-mode, cross-core thread-mode

[x] esp32
[x] esp32s2
[x] esp32s3

[x] Added examples are checked in CI

[x] Make sure to check examples with both timg0 and systimer where available.

Nice to have

[x] You add a description of your work to this PR.
[x] You added proper docs for your newly added features and code.

The Xtensa executor in embassy-executor is not safe on multiple cores. There are two reasons for this:

there exists a single global "there's more work" flag
it is not possible for one core to wake the other

This PR remedies this issue, implementing:

a thread-mode executor using FROM_CPU_0
an interrupt executor, which can use the rest of the FROM_CPU_x interrupts (one per executor)

On single-core (S2) we could probably use other interrupts as well. I'll probably need to add a list of crosscore-compatible, sw pendable interrupts for ESP32 and S3 (which is probably just the 4 FROM_CPU interrupt), so that we can safely disallow arbitrary interrupts on multicore.

We can't implement this in embassy, because there is no cross-core communication possibility in the Xtensa ISA. However, for consistency's sake I've also included non-multicore support (S2), which is basically identical to what's in embassy.

No, this isn't an executor that spreads load to two cores. Instead, this is an executor that is safe to use on both cores where one core's tasks can wake up the other core's tasks.

So, I've added an example which immediately deadlocks. The multicore example seems to work so I assume the bug is not in the multicore lock, but my executor code. It will be interesting to debug.

The added examples are now working on ESP32. The commit history could use some cleanup, but the pull request is now ready to be tested.
One quirky thing I experienced is:
It looks like the timers are started when the first alarm is started. This might cause some interesting behavior when one tries to wait for some duration while spinning in a loop, checking instant.elapsed(). Since the timer is not yet running, the loop does not terminate. While this is probably very rare, it's not correct either and I think we should start the timebase in embassy::init to fix this issue.

Testing in my project, the interrupt executor currently hangs when I spawn a task on it.

Who may I curse with this monster? @MabezDev I hope you don't mind if I ping you for a review?

Bleh, global copy-and-replace just made this worse 🤣

Ok I ended up going all in against StaticCell<Executor>, as static cells are unusable anyway after initialization.


[cleanup] return Promise.reject(new Error(...)) gets turned into throw new Error(new Error(...))

:tada: This issue has been resolved in version 1.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Renaming CreateGroupRequest to CreateManagementGroupRequest

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

@mcardosos The model validation should now run fine. I have also updated the readme since we want to keep using the previous version for SDK generation for now. Thanks!!


fix: if avaliable requestVideoFrameCallback() instead of request animation frame for better subtitle sync

Chrome and other browsers now offer the use of requestVideoFrameCallback() which will allow a callback function to be run on every video frame. This should be more consistent then something like requestAnimationFrame. In my experience when dealing with sync issues using requestVideoFrameCallback() helped greatly for a move smooth experience.

Most anime video frame is 24 fps, while requestAnimationFrame match the display refresh rate, most common is 60 fps.
Isn't requestAnimationFrame better than requestVideoFrameCallback ?

And requestAnimationFrame is only used for checking when a subtitle should display, not used for animations.
Animations (\t()) is implemented with Web Animation API, which may be same as requestAnimationFrame, but we have no way to change Web Animation API to use requestVideoFrameCallback.

requestVideoFrameCallback() is meant to be more accurate in terms of getting a time on the video, chrome's currentTime is purposely inaccurate to avoid fingerprinting, but requestVideoFrameCallback includes metadata.mediaTime, which should be more frame perfect. At least in my testing it appears to be more accurate then just using requestAnimationFrame(), but your mileage may vary.

Oh, I got your point.
media.currentTime precision might be 100ms, only 10 fps, requestVideoFrameCallback is a better than it.

Precisely, sorry if I didn't make myself very clear, but yes, it should be a fair bit more accurate then using requestAnimationFrame for displaying subtitles at the right time.


AttributeError: module 'jumpscale.tools.git' has no attribute 'get_latest_remote_tag'

Traceback (most recent call last):
  File "src/gevent/greenlet.py", line 906, in gevent._gevent_cgreenlet.Greenlet.run
  File "/root/js-sdk/jumpscale/packages/threebot_deployer/bottle/utils.py", line 118, in get_threebot_info
    solution_info = build_solution_info(workloads, threebot)
  File "/root/js-sdk/jumpscale/packages/threebot_deployer/bottle/utils.py", line 63, in build_solution_info
    release = j.tools.git.get_latest_remote_tag(sdk_path)
AttributeError: module 'jumpscale.tools.git' has no attribute 'get_latest_remote_tag'
2021-12-19T09:55:43Z <Greenlet at 0x7f8c92c82ae0: get_threebot_info(UserThreebot(
  instance_name='threebot_f2a90c0da6)> failed with AttributeError

Traceback (most recent call last):
  File "src/gevent/greenlet.py", line 906, in gevent._gevent_cgreenlet.Greenlet.run
  File "/root/js-sdk/jumpscale/packages/threebot_deployer/bottle/utils.py", line 118, in get_threebot_info          solution_info = build_solution_info(workloads, threebot)
  File "/root/js-sdk/jumpscale/packages/threebot_deployer/bottle/utils.py", line 63, in build_solution_info         release = j.tools.git.get_latest_remote_tag(sdk_path)                                                       AttributeError: module 'jumpscale.tools.git' has no attribute 'get_latest_remote_tag'                           2021-12-19T09:55:43Z <Greenlet at 0x7f8c912b66a0: get_threebot_info(UserThreebot(                                 instance_name='threebot_e4c670dc4d)> failed with AttributeError                                                                                Traceback (most recent call last):                                                                                File "src/gevent/greenlet.py", line 906, in gevent._gevent_cgreenlet.Gre...

happening on deploy3bot.grid.tf

fixed, seems the server wasn't updated and restarted properly.


Jest testing: listeners not working

I can't seem to get action listeners to work at all when testing with Jest. I made this really basic example:
'use strict';

jest.autoMockOff();

describe('Test', function () {
  it('Tests listenables', function () {
    var Reflux = require('../node_modules/reflux/index');

    var actions = Reflux.createActions([
      'action1',
    ]);

    var mockFn = jest.genMockFn();

    var store = Reflux.createStore({
      listenables: actions,
      onAction1: function () {
        mockFn();
      }
    });

    actions.action1('Hello World');
    expect(mockFn).toBeCalled();
  });
});

And it always gives me
Expected Function to be called.

Am I doing something wrong?

Same problems here :(

yeah... for now I've been getting around it by doing stuff like
var someAction = jest.genMockFn().mockImpl(myStore.onSomeAction);

...but it would be nice to have a way to test that action listeners are all actually hooked up correctly.

I figured it out! You just have to fast forward all the timers before any expect calls. For my example, it was as easy as adding this:
jest.runAllTimers();


[feat]: Added react-icons and removed primeicons

Fixes Issue
closes #819

Changes proposed

[X] Having react-icons included amongst the packages installed on running npm install.
[X] Replacement of primereact icons (or integrating those with react-icons, if possible), wherever we're using them.
[X] Adding support for the social link types that don't have any icons at the moment.

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Looks good to me !
Dose milestones icons looks good without applying custom size/CSS styles?

Dose milestones icons looks good without applying custom size/CSS styles?

I think it looks good

Also you can add graduation hat icon which will close some other issues as well with this one PR, Imo

Added


rncp: add blocking function to receive

Problem: right now it is needed

add callback function for link_established
do sleep in a loop waiting for callback function triggered

I dont know, if Reticulum API offers such a possibility, but if not, it would be worth to do. Some blocking function, that wait for some maximum time till link is established.

I don't understand exactly what you mean here. Can you describe what you are trying to do, and how it goes wrong?


[WIP] Adds dialog field association circular reference checker to main vali…

…dation for ui creation
Dialog field associations are checked for circular references on dialog import but this check needs to also be included in the main dialog validation on save for association creation via the classic ui.

@miq-bot assign @gmcculloug
tagging @eclarizio for review 🖊


Add Documentation for the Emoji class

Describe
A clear and concise description of Emojis in Zenora.

I will be working on this issue after I get to my destination. As I'm typing, I'm in my family car going to another district in an 8 hour journey. Puts a lot of stress on me. But as soon as I get there and take some rest, I will be back working with full speed so don't worry 😄

Done!


How to inject code in all windows?

I'm making an application where the user can create windows using their own html files, but it would require the windows to have a default behavior, so I developed a simple code that could be manually added by the user, but if there was a way to inject that code would make it a lot easier, I searched and found RuntimeOptions::InitScript, I tried calling a native function and nothing happened.

There's an answer here: https://sciter.com/forums/topic/how-to-inject-code-in-all-windows/

I tried calling a native function and nothing happened.

Could you show the code? It's not clear yet.

Could you show the code? It's not clear yet.

This is what i tried (still doesn't work)
#[macro_use]
extern crate sciter;

struct Handler;

impl Handler {
  fn calc_sum(&self, a: i32, b: i32) {
    print!("{} + {} = {}", a, b, a + b);
  }
}

impl sciter::EventHandler for Handler {
  dispatch_script_call! {
    fn calc_sum(i32, i32);
  }
}

fn main() {
  sciter::set_options(sciter::RuntimeOptions::DebugMode(true)).unwrap();
  sciter::set_options(sciter::RuntimeOptions::InitScript("
    Window.this.xcall('calc_sum', 1, 2);
  ")).unwrap();
  let mut frame = sciter::Window::new();
  frame.event_handler(Handler);
  frame.load_html(b"
Hello
", None);
  frame.run_app();
}

I believe, it's too early to call. I wonder if timers work in this case.

I believe, it's too early to call. I wonder if timers work in this case.

I tried several possible ways and nothing works, my app doesn't need to call native functions too early, I just need to create some functions to manipulate the window.
extern crate sciter;

fn main() {
  sciter::set_options(sciter::RuntimeOptions::DebugMode(true)).unwrap();
  sciter::set_options(sciter::RuntimeOptions::InitScript("setTimeout(() => console.log('Hello World1'), 5000)")).unwrap();
  let mut frame = sciter::Window::new();
  frame.load_html(b"
:)
", None);
  frame.run_app();
}

cc @c-smile

@WesleiRamos Try to ask on http://sciter.com/forums/ - I think, it's a Sciter related issue.

@WesleiRamos Try to ask on http://sciter.com/forums/ - I think, it's a Sciter related issue.

InitScript should work in <IP_ADDRESS>, and see: https://sciter.com/forums/topic/how-to-inject-code-in-all-windows/

It's working now, thanks guys


Fixed issue with sensitive swipe item

Before, when click on item in recycler view, click event fires not immediately, 'cause reveal layout was very sensitive and he has try to swipe. Click event fires from 3-rd or 5-th time. Now this problem is fixed

@DenisMakovskyi very nice work and handy solution, really I need that!

@chthai64 accept this PR plz


Respect enums alias attribute

https://github.com/siniarskimar/zglgen/blob/main/src/glregistry.zig#L671
Currently zglgen generates multiple names for the same value and zig doesn't like that.

Resolved by 474b2fb


Proxy of device is ignored.

This is a known issue in flutter: https://github.com/flutter/flutter/issues/26359
This can be fixed with the following plugin:
https://pub.dev/packages/osproxysetting

It would seem that there is now something for this in io, other libraries have not been migrated to null safety...
    (dio.httpClientAdapter as DefaultHttpClientAdapter).onHttpClientCreate = (client) async {
      client.findProxy = HttpClient.findProxyFromEnvironment;
    };

This feels like something for the template project, not the lib

Yes indeed

Won't fix in this lib


USHIFT-4078: Wait for vg-manager to exist before waiting for availability

/retest-required

/test e2e-aws-tests

/hold cancel


 I don’t recall any case where it helped while I remember many ones where it was just an nuisance. I propose to get rid of this check completely. Any objections, @amimas ?

Originally posted by @gdubicki in https://github.com/gitlabform/gitlabform/pull/585#discussion_r1284414736

I like the concept or idea of type checking and validation. But, I'm really new to Python and don't have enough experience to suggest whether to keep using mypy for this project or not. I'd go with your expertise @gdubicki . Hopefully we hear from other experienced contributors as well.

I use type checking, and mypy, generally without issue, with the type annotation settings turned on in vs-code and a bit of policing from ruff.
I'd be more inclined to use pydantic on anything new though.

Isn’t pydantic use-case a bit different, @adam-moss? I haven’t used it but based on what I read is more of a data validation solution than a type-checker.

I don't think it should be removed. Use it extensively at work. It has several pros.
There was a survey made about typing in python (Pylance team at Microsoft and PyCharm at JetBrains):
Discussion: https://discuss.python.org/t/2024-python-typing-survey-analysis/61456
Results (check out the slides): https://discuss.python.org/t/python-typing-survey-results/71821


Support for "SingleTile" WMS layers

WMS layers are tiled in leaflet. It's perfect for basemaps and/or for public access.
But it's not good for some use cases :

realtime business layers
private access with custom WMS servers
layers animations (it's difficult to synchronize tiles animations)

It's implement in openlayers as a "singleTile" option :
http://openlayers.org/dev/examples/single-tile.html
In this mode, the GetMap WMS request must request :

the exact bounding box of the visible map
the exact width, height of the visible map

Thanks :-)

@sheppard :+1:

Thank you @fnicollet That  SingleTileWMSLayer  is exactly what I need and I have made some change on the "onRemove" method.I think we shoud reset "_imagesCreated "status for the next adding. Here is the code.
onRemove: function (map) {
        // super()
        L.ImageOverlay.prototype.onRemove.call(this, map);
        // add
        if (this._imageSwap){
            map.getPanes().overlayPane.removeChild(this._imageSwap);
        }
        //reset _imagesCreated status
        this._imagesCreated = false;
        map.off('moveend', this._onViewReset, this);
        map.off('zoomanim', this._onZoomAnim, this);
    }

By the way, I use

Leaflet

<EMAIL_ADDRESS>


Minor change to workflow doc based on feedback

According to lionbridge feedback the NVcloth gem is also required to complete some of these areas. Updating the doc to include it as a requirement.

According to lionbridge feedback the NVcloth gem is also required to complete some of these areas. Updating the doc to include it as a requirement.

Please ask Lionbridge to provide more information, because there is no relation between ragdolls and cloth. All ragdolls workflow can be done without having NVIDIA Cloth gem enabled.

@michleza Were you able to verify the requirement with Lionbridge?

@michleza could you please comment on the feedback?

@michleza if you could respond by 10.09.2023, it would be great. Otherwise we will close this issue.


Change Request: Update RuleVisitor interface

Which packages would you like to change?

[ ] @eslint/compat
[ ] @eslint/config-array
[x] @eslint/core
[ ] @eslint/migrate-config
[ ] @eslint/object-schema
[ ] @eslint/plugin-kit

What problem do you want to solve?
The RuleVisitor interface is currently defined as follows:
export interface RuleVisitor {
	/**
	 * Called for each node in the AST or at specific times during the traversal.
	 */
	[key: string]: (...args: any[]) => void;
}

However, this is causing problems when trying to extend RuleVisitor under strict TypeScript mode, as seen in these PRs:

https://github.com/eslint/json/pull/66
https://github.com/eslint/eslint/pull/19157

The errors take the form of the following:
error TS2411: Property 'Document' of type '((node: DocumentNode$1) => void) | undefined' is not assignable to 'string' index type '(...args: any[]) => void'.

What do you think is the correct solution?
The problem is that the method defined in RuleVisitor is not optional.
I can see a few different options:

Remove all methods. We can remove any predefined methods from RuleVisitor and leave essentially a blank interface:

export interface RuleVisitor {
	/**
	 * Called for each node in the AST or at specific times during the traversal.
	 */
-	[key: string]: (...args: any[]) => void;
}

That would give consumers something to extend from so that it logically looks like there's some relationship between the core types and consumer types. I'm not sure it's valuable.

Convert to a mapped type. We can change it to a mapped type as suggested by @fasttime here:

- export interface RuleVisitor {
+ export type RuleVisitor = {
	/**
	 * Called for each node in the AST or at specific times during the traversal.
	 */
-	[key: string]: (...args: any[]) => void;
+	[key in string]?: (...args: any[]) => void;
}

Defining all methods to be optional also seems to make RuleVisitor mostly useless.

Eliminate RuleVisitor interface completely. Because we don't know what the visitor will look like for any particular language, maybe it's a bad idea to try and enforce anything in particular. We could just define the RuleDefinitionTypeOptions#Visitor as Object, which would allow any non-primitive type to be used, and then we don't need RuleVisitor at all:

export interface RuleDefinitionTypeOptions {
	LangOptions: LanguageOptions;
	Code: SourceCode;
	RuleOptions: unknown[];
-	Visitor: RuleVisitor;
+	Visitor: Object;
	Node: unknown;
	MessageIds: string;
	ExtRuleDocs: unknown;
}

I think this is my favorite approach, but I'd like some insights from @JoshuaKGoldberg and @fasttime on all of the options and their ramifications.
Participation

[ ] I am willing to submit a pull request for this change.

Additional comments
No response

The problem with Visitor: object is that it allows any object to be passed in. So you could pass { value: "gotcha!" } and the types would be fine with it.
I poked around a bit in eslint/json and think the [key in string]?: mapped type option is where I'd go. It makes sense as treating the root issue:

Per strictNullChecks, null and undefined can't be used in places that don't say they can take them in (common phrasing: are a nullable union)
The current interface RuleVisitor explicitly says that for any given key, the value must be a non-nullable function
Rule visitors such as JSONRuleVisitor are saying that for some keys, the value may be undefined

Mismatch!
As I understand it, *RuleVisitor interfaces aren't meant to say that all defined keys must exist. They define the set of allowed keys and then implementers can choose some of them to exist, right?

Because we don't know what the visitor will look like for any particular language, maybe it's a bad idea to try and enforce anything in particular. We could just define the RuleDefinitionTypeOptions#Visitor as Object, which would allow any non-primitive type to be used

I think it's object (with a small o) that can be assigned any non-primitive type, whereas Object allows any value other than null or undefined in TS strict mode, or any value in non-strict mode.
It isn't clear to me how a Visitor would work if its properties aren't functions. Currently ESLint uses property keys of a Visitor object as selectors and property values as listeners. The listeners are called at some point:
const ruleListenerReturn = ruleListener(...listenerArgs);

https://github.com/eslint/eslint/blob/01557cec24203be72222858a3912da0a474ac75c/lib/linter/linter.js#L1098
So I was under the impression that the expectation was for a Visitor to be an object with callable functions as property values. Are you planning to change the way Visitors work?

@JoshuaKGoldberg

As I understand it, *RuleVisitor interfaces aren't meant to say that all defined keys must exist. They define the set of allowed keys and then implementers can choose some of them to exist, right?

That's correct. So for JSONRuleVisitor, we want to define all of the known possible methods that rules could use but rule authors are not required to implement any of them.
@fasttime

So I was under the impression that the expectation was for a Visitor to be an object with callable functions as property values.

That's correct. The RuleVisitor interface is only really intended for use by language plugins authors, who will in turn extend that type to define any actual methods that the language will accept on rule objects. If a rule returns an object with non-function key values, that will result in an error if the key is an expected method name or otherwise it will be ignored.

It looks like you're both in the mapped type camp, so put together a PR here:
https://github.com/eslint/rewrite/pull/135


Add CCAFS Unique Project Tag

CCAFS is redesigning their corporate website and would like to make use of a new metadata field to show outputs corresponding to certain projects. They need a new field for this, which will be added retroactively to their existing items, as well as all future items. We've decided on using cg.identifier.ccafsproject. There will be one input dropdown with about 60 projects initially.
We need to:

[x] Add the vocabulary to the input forms
[x] Add cg.identifier.ccafsproject to the metadata registry

Added the project tags to the input forms in: https://github.com/alanorth/DSpace/commit/fa9f5d644a7c85b2850b01c21aac2680091ffd8b

Merged.


adding basedir option

Is it possible to add a basedir option which will do something like this
src: [        'public/js/libs/file1.min.js',        'public/js/libs/file2.min.js',        'public/js/libs/file3.min.js',        'public/js/libs/file4.min.js' ]
instead of carying around all this public/js/libs/ to put it inside basedir and specify only file1.min.js, ... file4.min.js  in src.

It is not work. The config can copy the files, but can not concat.


Support passing third party connection instance to Flask-sqlalchemy db constructor or objects

SQLAlchemy engine construction
SQLAlchemy allows users to pass connection created by third-party clients instead of specifying database url to construct a connection to remote.
For example, if I want to connect to Hive, of which configuration is a little bit complicated. SQLAlchemy allows users to create engine by
engine = sqlalchemy.create_engine('impala://', creator=conn)

Where "impala" from cloudera is an implementation of Hive2 client.
However in the current implementation, only URL string accepted.
More Problems
The buggy implementation (not aligned to original ORM specification) will also create errors when people try to bind to multiple backends:
[] hadoop,
[] hive
[] nosql
[] sql
We have to seek different suppliers solution to trackle that problem.

The solution I propose at https://github.com/pallets/flask-sqlalchemy/issues/166#issuecomment-470711632 will solve this as it will give you complete control over how to create your engine.


Raw messages beginning with a number are incorrectly parsed as JSON

It looks as though a message of the form <raw JSON value><some other stuff> is incorrectly parsed as the JSON contents <raw JSON value>, so the raw value 123foo would be parsed as 123. This is a problem when using processors that attempt to extract a structured form from messages (such as the branch processor), as it'll result in the message being mutated.

Fixed: https://github.com/Jeffail/benthos/commit/e11a2bb03e73725bed07353f3a6c184d4227664b


Restrict instrument/plan change

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Some tests need to be updated.


Uncaught Error: Bootstrap's JavaScript requires jQuery version 1.9.1 or higher, but lower than version 3

When using ember version 2.11.0+ (ember-cli version 2.11.0)

Upgrading bootstrap to 3.3.7 resolves this issue.
Need to update the dependency version in the blueprint - https://github.com/softlayer/sl-ember-components/blob/master/blueprints/sl-ember-components/index.js#L6

I'm getting Uncaught Error: Could not find module ember-resolver imported from npm-test/resolver but my Bootstrap is 3.3.7. Any ideas?

@tinyweasel Is the version only changed in your bower.json file or have you done a bower install as well?
Whenever I run into these types of errors the first thing I do is run npm set progress=false && rm -rf node_modules && rm -rf bower_components && npm cache clean && bower cache clean && npm install && bower install as it will often clear such things up.  If this problem still exists after this then we will have to dig into it further.

I'd done all of that except npm set progress=false. I'm trying it again, I'll let you know if the problem persists.


Updating the ABCI grammar to handle an uncovered scenario caused by finalize-block commit from an older round.

ABCI Grammar Change Proposal
Summary
We found a possible scenario which is not captured by the current ABCI grammar.
Problem Definition
The scenario exploits the fact that finalize-block and commit can be called at any time in rounds $r>0$.
Specifically, below is the possible scenario:
Round 0:
-any sequence of ABCI requests but no finalize-block and commit
Round 1:
got-vote process-proposal got-vote finalize-block commit

Proposal
To fix this, we need to:

[ ] fix the ABCI grammar to cover this scenario
[ ] update e2e tests to account for the change in grammar
[ ] write a paragraph in the grammar description that highlights the peculiarity of this scenario

I thought it only applied to ABCI 2.0, with the introduction of ExtendVote.

The referred grammar: https://docs.cometbft.com/v0.38/spec/abci/abci++_comet_expected_behavior

Yes, It applies to ABCI 2.0 only.


Matched with same person 3x in 5weeks

With Team members around 50, can Icebreaker identify that the person was matched this week then it should find a new match next week or the succeeding weeks? Can we control this logic? Thanks!

But 3x with the same person in 5weeks? The algorithm should be reviewed I think.

bot, my post which is a question and didn't get an answer. Update your logic please.

The matching process in Icebreaker is performed using a random algorithm that cannot be manually controlled. The overall purpose of Icebreaker is to match pairs that are not predetermined. As there are 50 members in the channel, the likelihood of getting the same matching pair is very minimal.

3x with the same person in 5weeks? The algorithm should be reviewed I think. Feedback?


['install_vars']

Cookbook version
2.0.1
Chef-client version
14.13.11
Platform Details
Windows Server 2016
Scenario:
Install Chocolatey from Artifactory repo instead of Chocolatey.org
Steps to Reproduce:
We are running the Chocolatey Cookbook using the following code:
node['chocolatey']['install_vars']['chocolateyDownloadUrl'] = "https://URL/TO/ARTIFACTORY/REPO"
include_recipe 'chocolatey::default'
Expected Result:
Expect the Chocolatey agent to be pulled from"https://URL/TO/ARTIFACTORY/REPO" and then installed.
Actual Result:
Instead, agent is being installed from Chocolatey.org.  Syntax to add URL variable looks correct but doesn't seem to be getting used.

I want to say it's likely there is something incorrect, but I'm not sure what you are installing - there is the chocolatey client, the chocolatey agent is a Windows service that is used in C4B (Chocolatey for Business) scenarios only. So I'm guessing you mean chocolatey client here.

Seeing that this is a bit older, were you able to resolve your issue or is it still active?


perf: reuse splines on UVBeam objects

Re-uses splines when using beam_list instead of bm_cube. This should give a healthy speedup.

Thanks @r-pascua and @piyanatk -- I've made the relevant changes.
To your question, @r-pascua, yes the update is the check_azza_domain parameter, which will become available in pyuvdata soon.


App crash(the fga button suddenly disappears during quest) also when reopening, after the crash, also crashes for a while before opening

Preparation

[X] I tested the latest release
[X] I looked at other issues (even the closed ones)
[X] I read the Troubleshooting Guide

FGO server
NA
FGA build number
1908
Describe the bug

Start the app
Start FGO
Go to the quest screen
Start the FGA
wait until the wave 1
suddenly the fga crashes(the fga button suddenly gone)

sometimes the app just suddenly crashes at wave 1 or while loading into wave 1
crashing when opening fga log (this is on the first video)
crashing when opening fga log.txt
fga gone suddenly log (this is on the second video)
fga gone suddenly log.txt
Other log files that might help, also captured during their own crashes but no video of their crashes
fga log.txt
fga log2.txt
Video
The fga button suddenly gone
https://youtu.be/wXKNEqyLgsY
When opening will crash
https://www.youtube.com/watch?v=Zs2pif9WLQs
Device model
Xiaomi Redmi note 7 (Pixel Experience rom)
Android version
11
Screen size
2336x1080
RAM
3GB

one of the crashes
java.io.FileNotFoundException: No resource found for: android.resource://com.mathewsachin.fategrandautomata.debug/0
But I even did again the Extract Default support images just to be sure. but it still crashes

For future documentation
Assuming you already connected the devices to the adb via adb devices
this is the adb command I use to get the crash
adb logcat -e com.mathewsachin.fategrandautomata > log.txt

this is for the recording of the file
scrcpy -r file.mp4

Thought I was the only one. One thing I did to make it not crash is to just set the game to 30fps instead of 60. It will still crash but after like an hour of use. After that, I need to reset my phone because fga is bugged now and won't start again.

Thought I was the only one. One thing I did to make it not crash is to just set the game to 30fps instead of 60. It will still crash but after like an hour of use. After that, I need to reset my phone because fga is bugged now and won't start again.

yea it used to be like that for me. But nowadays, the crashing is more frequent

@ArthurKun021 Can you share the log? I want to check which part fails.

Forgot to say that I have Xiaomi Redmi Note 7 too but with stock rom and android 10.

"Accessibility service not running" is the cause of the crashes.
Is battery optimization disabled? Is the app "locked" in the "recently opened apps" screen? (Square button in the navigation bar)
If yes, is there any error before the first "Accessibility service is not running" message?

Is battery optimization disabled?

I already have my battery optimization for the FGA turn off

Is the app "locked" in the "recently opened apps" screen? (Square button in the navigation bar)

On my phone, it is not pause app

If yes, is there any error before the first "Accessibility service is not running" message?

Based from the suggestion here, I did turn it off and then turn it on and then the accessibility is now on again. After the crash upon going to the app again you'll see the state of the accessibility service the same as the 3rd picture

Please check if this build works better: https://github.com/Fate-Grand-Automata/FGA/suites/9999667043/artifacts/485474291
FGA shouldn't crash repeatedly anymore, but it will still crash from time to time when the Accessibility Service was killed by Android.

Please check if this build works better: https://github.com/Fate-Grand-Automata/FGA/suites/9999667043/artifacts/485474291
FGA shouldn't crash repeatedly anymore, but it will still crash from time to time when the Accessibility Service was killed by Android.

log.txt

Good news and bad news I guess.
Bad news.
I can't seem to find a fix for it. I check some post and comments from stackoverflow and reddit and it seems like the Tasker app, which similarly uses the accessibility settings, faces the same issue. I forgot where I saw it but it seems like this is related to bug in Android. What happened in the Tasker is that they uses a webiew as dependency and when it is updated causes the bug.
I don't know if this links might help but here
From the official android platform which states the problem with the accessibility services
https://cs.android.com/android/_/android/platform/frameworks/base/+/ec8cff62610d0964b015e7e0918ba060a29bb617
Relevant links to this
https://www.reddit.com/r/tasker/comments/syhzr7/comment/hxyc37o
https://www.reddit.com/r/tasker/comments/sz2kg7/investigation_accessibility_services_getting/
Good news.
I updated to Android 12 and facing no problem so far but that might be also due to writing additional adb commands.
But this might be also due to freshly installed android 12 hence no issues so far.
I will still keep checking on it due to the lotto ongoing right now I need to catch up
You might want to run some adb commands
For disabling the battery optimization to the specific app. just change - to + if you want to re-enable
adb shell dumpsys deviceidle whitelist -=com.mathewsachin.fategrandautomata

To help with the accessibility settings
adb shell settings put secure enabled_accessibility_services com.mathewsachin.fategrandautomata/com.mathewsachin.fategrandautomata


feat: add URL at button and remove active in navbar

Related Issue

Information about the related issue
N/A

Closes: #
Describe the changes you've made
I added URL of start-here repository at button and remove active text in navbar and change it to Home text.

Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)

Original
Updated

!

Can you share some ss maybe ?

Can you share some ss maybe ?

Added at top section

LGTM :)


Failing tests related to -in-element in Ember 3.17.0-beta

See https://github.com/emberjs/ember.js/issues/18696

Fixed upstream.


[Bug] Mod doesn't work, at all

Checks

[X] I confirm that I have searched for existing issues / pull requests before reporting to avoid duplicate reporting.
[X] I confirm that I followed the MCDC Docs and read the Changelog, but this problem still exists.
[x] I confirm that I noted that if I don't follow the instructions, the issue may be closed directly.

Java Version
19.0.2 (openjdk version 19.0.2+7, mixed build)
Minecraft Version
1.19.3
MCDiscordChat Version
2.1.2
Description
Fabric loads mod, but none of the commands work, the discord bot doesn't come online and nothing related to MCDC in logs. In fact, the MCDC commands don't even get registered.
I've tried with both java 19 and 17
openjdk 19.0.2 2023-01-17
OpenJDK Runtime Environment (build 19.0.2+7)
OpenJDK 64-Bit Server VM (build 19.0.2+7, mixed mode)

openjdk 17.0.6 2023-01-17
OpenJDK Runtime Environment (build 17.0.6+10)
OpenJDK 64-Bit Server VM (build 17.0.6+10, mixed mode)

I've tried 2.1.2 from the modrinth page, 2.1.2 from releases, 2.1.2 from actions and 2.1.1 from releases, none of them seem to work

To Reproduce

Start server
no message relating to mcdc in logs
configuration file doesn't get created
mcdc commands dont work

Log
Starting net.fabricmc.loader.impl.game.minecraft.BundlerClassPathCapture
[05:16:50] [main/INFO]: Loading Minecraft 1.19.3 with Fabric Loader 0.14.14
[05:16:50] [main/INFO]: Loading 53 mods:
	- c2me 0.2.0+alpha.10.0
	- c2me-base 0.2.0+alpha.10.0 via c2me
	- c2me-fixes-chunkio-threading-issues 0.2.0+alpha.10.0 via c2me
	- c2me-fixes-general-threading-issues 0.2.0+alpha.10.0 via c2me
	- c2me-fixes-worldgen-threading-issues 0.2.0+alpha.10.0 via c2me
	- c2me-fixes-worldgen-vanilla-bugs 0.2.0+alpha.10.0 via c2me
	- c2me-notickvd 0.2.0+alpha.10.0 via c2me
	- c2me-opts-allocs 0.2.0+alpha.10.0 via c2me
	- c2me-opts-chunk-access 0.2.0+alpha.10.0 via c2me
	- c2me-opts-chunkio 0.2.0+alpha.10.0 via c2me
	- c2me-opts-math 0.2.0+alpha.10.0 via c2me
	- c2me-opts-scheduling 0.2.0+alpha.10.0 via c2me
	- c2me-opts-worldgen-general 0.2.0+alpha.10.0 via c2me
	- c2me-opts-worldgen-vanilla 0.2.0+alpha.10.0 via c2me
	- c2me-rewrites-chunkio 0.2.0+alpha.10.0 via c2me
	- c2me-server-utils 0.2.0+alpha.10.0 via c2me
	- c2me-threading-chunkio 0.2.0+alpha.10.0 via c2me
	- c2me-threading-lighting 0.2.0+alpha.10.0 via c2me
	- c2me-threading-scheduling 0.2.0+alpha.10.0 via c2me
	- c2me-threading-worldgen 0.2.0+alpha.10.0 via c2me
	- com_electronwill_night-config_core 3.6.5 via c2me
	- com_electronwill_night-config_toml 3.6.5 via c2me
	- com_fasterxml_jackson_core_jackson-annotations 2.14.1 via mcdiscordchat
	- com_fasterxml_jackson_core_jackson-core 2.14.1 via mcdiscordchat
	- com_fasterxml_jackson_core_jackson-databind 2.14.1 via mcdiscordchat
	- com_google_code_gson_gson 2.10 via mcdiscordchat
	- com_ibm_async_asyncutil 0.1.0 via c2me
	- com_neovisionaries_nv-websocket-client 2.14 via mcdiscordchat
	- com_squareup_okhttp3_okhttp 4.10.0 via mcdiscordchat
	- com_squareup_okio_okio 2.10.0 via mcdiscordchat
	- com_vdurmont_emoji-java 5.1.1 via mcdiscordchat
	- com_velocitypowered_velocity-native 3.1.2-SNAPSHOT via krypton
	- fabric-api-base 0.4.20+afca2f3e85 via nochatreports
	- fabric-networking-api-v1 1.2.15+99f9db8085 via nochatreports
	- fabric-resource-loader-v0 0.10.4+a91e48b785 via nochatreports
	- fabricloader 0.14.14
	- ferritecore 5.1.0
	- java 19
	- krypton 0.2.1
	- lazydfu 0.1.3
	- lithium 0.10.4
	- mcdiscordchat 2.1.2
	- minecraft 1.19.3
	- net_dv8tion_jda 5.0.0-beta.2 via mcdiscordchat
	- net_objecthunter_exp4j 0.4.8 via c2me
	- net_sf_trove4j_trove4j 3.0.3 via mcdiscordchat
	- nochatreports 1.19.3-v2.0.0
	- org_apache_commons_commons-collections4 4.4 via mcdiscordchat
	- org_jetbrains_kotlin_kotlin-stdlib 1.8.0 via mcdiscordchat
	- org_json_json 20220924 via mcdiscordchat
	- org_slf4j_slf4j-api 2.0.6 via mcdiscordchat
	- org_threadly_threadly 7.0 via c2me
	- starlight 1.1.1+fabric.ae22326
[05:16:50] [main/INFO]: SpongePowered MIXIN Subsystem Version=0.8.5 Source=file:/home/afy/Desktop/mc/pexoserver/libraries/net/fabricmc/sponge-mixin/0.11.4+mixin.0.8.5/sponge-mixin-0.11.4+mixin.0.8.5.jar Service=Knot/Fabric Env=SERVER
[05:16:50] [main/INFO]: Compatibility level set to JAVA_16
[05:16:50] [main/INFO]: Compatibility level set to JAVA_17
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.base.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.fixes.chunkio.threading_issues.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.fixes.general.threading_issues.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.fixes.worldgen.threading_issues.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.fixes.worldgen.vanilla_bugs.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.notickvd.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.opts.allocs.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.opts.chunk_access.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.opts.chunkio.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.opts.math.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.opts.scheduling.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.opts.worldgen.general.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.opts.worldgen.vanilla.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.rewrites.chunkio.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.server.utils.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.threading.chunkio.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.threading.lighting.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.threading.scheduling.mixin
[05:16:50] [main/INFO]: Initializing com.ishland.c2me.threading.worldgen.mixin
[05:16:50] [main/INFO]: Loaded configuration file for Lithium: 106 options available, 2 override(s) found
[05:16:50] [main/WARN]: Force-disabling mixin 'alloc.blockstate.StateMixin' as rule 'mixin.alloc.blockstate' (added by mods [ferritecore]) disables it and children
[05:16:50] [main/WARN]: Force-disabling mixin 'world.player_chunk_tick.ThreadedAnvilChunkStorageMixin' as rule 'mixin.world.player_chunk_tick' (added by mods [c2me, krypton]) disables it and children
[05:16:52] [main/INFO]: Building unoptimized datafixer
minecraft:empty task radius: -1 -> -1
minecraft:structure_starts task radius: 0 -> 0
minecraft:structure_references task radius: 8 -> 0
minecraft:biomes task radius: 8 -> 0
minecraft:noise task radius: 8 -> 0
minecraft:surface task radius: 8 -> 0
minecraft:carvers task radius: 8 -> 0
minecraft:liquid_carvers task radius: 8 -> 0
minecraft:features task radius: 8 -> 1
minecraft:light task radius: 1 -> 1
minecraft:spawn task radius: 0 -> 0
minecraft:heightmaps task radius: 0 -> 0
minecraft:full task radius: 0 -> 0
[05:16:53] [main/INFO]: Compression will use libdeflate (Linux x86_64), encryption will use OpenSSL 1.1.x (Linux x86_64)
[05:16:53] [main/INFO]: KONNICHIWA ZA WARUDO!
[05:16:53] [main/INFO]: Default JVM text encoding is: UTF-8
[05:16:53] [main/INFO]: Reading config file NoChatReports/NCR-Common.json...
[05:16:53] [main/INFO]: Writing config file NoChatReports/NCR-Common.json...
[05:16:53] [main/INFO]: Krypton is now accelerating your Minecraft server's networking stack 🚀
[05:16:53] [main/INFO]: Environment: authHost='https://authserver.mojang.com', accountsHost='https://api.mojang.com', sessionHost='https://sessionserver.mojang.com', servicesHost='https://api.minecraftservices.com', name='PROD'
[05:16:54] [main/INFO]: Loaded 7 recipes
[05:16:54] [main/INFO]: Loaded 1179 advancements
[05:16:54] [main/INFO]: Mixing server.MixinDedicatedServer from mixins/common/nochatreports.mixins.json into net.minecraft.class_3176
[05:16:54] [main/INFO]: Mixing server.MixinPlayerList from mixins/common/nochatreports.mixins.json into net.minecraft.class_3324
[05:16:54] [main/INFO]: Mixing common.MixinServerStatus from mixins/common/nochatreports.mixins.json into net.minecraft.class_2926
[05:16:54] [Server thread/INFO]: Starting minecraft server version 1.19.3
[05:16:54] [Server thread/INFO]: Loading properties
[05:16:54] [Server thread/INFO]: Default game type: SURVIVAL
[05:16:54] [Server thread/INFO]: Generating keypair
[05:16:54] [Server thread/INFO]: Starting Minecraft server on *:25565
[05:16:54] [Server thread/INFO]: Using epoll channel type
[05:16:54] [Server thread/WARN]: **** SERVER IS RUNNING IN OFFLINE/INSECURE MODE!
[05:16:54] [Server thread/WARN]: The server will make no attempt to authenticate usernames. Beware.
[05:16:54] [Server thread/WARN]: While this makes the game possible to play without internet access, it also opens up the ability for hackers to connect with any username they choose.
[05:16:54] [Server thread/WARN]: To change this, set "online-mode" to "true" in the server.properties file.
[05:16:54] [Server thread/INFO]: Preparing level "world"
[05:16:55] [Server thread/INFO]: Preparing start region for dimension minecraft:overworld
[05:16:55] [C2ME worker #1/INFO]: Preparing spawn area: 0%
[05:16:55] [C2ME worker #3/INFO]: Preparing spawn area: 0%
[05:16:55] [C2ME worker #6/INFO]: Preparing spawn area: 0%
[05:16:55] [C2ME worker #2/INFO]: Preparing spawn area: 0%
[05:16:56] [C2ME worker #5/INFO]: Preparing spawn area: 8%
[05:16:56] [C2ME worker #5/INFO]: Preparing spawn area: 21%
[05:16:57] [C2ME worker #2/INFO]: Preparing spawn area: 42%
[05:16:57] [minecraft_overworld - Light/INFO]: Preparing spawn area: 71%
[05:16:58] [Netty Epoll Server IO #1/INFO]: Mixing server.MixinServerboundChatCommandPacket from mixins/common/nochatreports.mixins.json into net.minecraft.class_7472
[05:16:58] [Netty Epoll Server IO #1/INFO]: Mixing server.MixinServerboundChatPacket from mixins/common/nochatreports.mixins.json into net.minecraft.class_2797
[05:16:58] [Netty Epoll Server IO #1/INFO]: Mixing server.MixinServerboundChatSessionUpdatePacket from mixins/common/nochatreports.mixins.json into net.minecraft.class_7861
[05:16:58] [Netty Epoll Server IO #1/INFO]: Mixing common.MixinServerStatusSerializer from mixins/common/nochatreports.mixins.json into net.minecraft.class_2926$class_2929
[05:16:58] [minecraft_overworld - Light/INFO]: Preparing spawn area: 91%
[05:16:58] [Server thread/INFO]: Time elapsed: 3302 ms
[05:16:58] [Server thread/INFO]: Done (4.024s)! For help, type "help"

Config
{
    "botToken": "REDACTED",
    "webhookUrl": "REDACTED",
    "channelId": "1075945072400531526",
    "updateNotificationChannelId": "1075945072400531526",
    "useUuuidInsteadOfName": false,
    "broadcastCommandExecution": false,
    "adminsIds": ["276363003270791168"]
}

Try the latest build from GitHub Actions?

I did, issue persists

I found the problem now. Install Fabric API should solve the problem.

Alright thank you


Removed support for Citrix XenServer

The change LGTM, perhaps you could refer to the PR or some info about the change?
Also, is there a note in release notes? How should people with xenserver + nuage that upgrades handle it?

Is this for a part of a particular version of CloudStack, or it never been supported?

@FFincannon you mean, support of the nuage-plugin for XenServer, right?
I think the PR title is a little misleading. When I read I thought it was support of ACS for the XenServer.

Yes, Rafael, I did mean support of the Nuage plugin for XenServer.
In answer to Erik Weber's question, "How should people with xenserver +
nuage that upgrades handle it?" the product manager, Sanjeev Singh, says: "As
of now we do not have any customers on Xen and do not see any customer
demand for that. If we do see customer traction - we will re-evaluate Xen
support."
Thank you,
Fiona
On Mon, Aug 1, 2016 at 10:00 AM, Rafael Weingärtner <
<EMAIL_ADDRESS>wrote:

@FFincannon https://github.com/FFincannon you mean, support of the
nuage-plugin for XenServer, right?
I think the PR title is a little misleading. When I read I thought it was
support of ACS for the XenServer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/cloudstack-docs/pull/18#issuecomment-236641298,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APkn5ztp9PeCIzfG795RQr1Tz-gygqsIks5qbiY3gaJpZM4JIeyv
.

The changes here seemed to have been applied in some other PR. I will close this one then.


Facet sizing broken if the facet panel is not open when the facet is created

Creating a facet while the Undo / Redo tab is active makes the facet initially collapsed, breaking the automatic sizing introduced in #5514.
Current results

Expected Behavior
The facet should be properly sized automatically
Versions

Browser Version: Firefox 102.8.0esr
OpenRefine: 3.8-SNAPSHOT

Datasets

Additional context
Ideally, the facet panel should also be switched to automatically, because when I create a facet I want to see it directly (either to look at its statistics or to use it to filter rows / records). This means that the fix should be rather simple and should not worry about resizing an element that is not currently rendered: just switch to the facet panel whenever new facets are created. That being said, I am not completely sure about what to do when facets are automatically created without an action from the user, for instance after a reconciliation operation is run.

I have not checked but I would expect that all facets get refreshed automatically whenever you create a new facet, no? If not, it should really be automatic and I don't think we should need to hint the user at manually clicking a button for that to happen.


Add ERC: Chain-specific addresses using ENS

My apologies if I'm in the wrong place, I've not done this before. I'm here to comment on this proposal. My feedback is simply to think about how it might map to blockchain agnostic representations, notably this one:
https://github.com/ChainAgnostic/CAIPs/blob/main/CAIPs/caip-10.md
Thank you.

@lucemans I believe ENSIP-9 doesn’t actually solve the issue, but gives redundancy in case this only gets applied to the stricter Ethereum ecosystem. The whole reason to do this multi-chain resolution is to allow for ‘clean names’ based on a widely used primitive: chainID. ENSIP-9 builds on top of this with another identification system, while this ERC is an alternative.
BTW thank you for mentioning, it’s actually a great feature of ENS that could be publicized more!

Please keep technical discussion on the forum: https://ethereum-magicians.org/t/erc-7828-chain-specific-addresses-using-ens/21930


cmd/cue: cue get go doesn't support uintptr type

What version of CUE are you using (cue version)?

cue version v0.4.0 linux/amd64

Does this issue reproduce with the latest release?
yes, on master
What did you do?
go mod init tmp.com
go get istio.io/api/networking/v1alpha3
cue get go istio.io/api/networking/v1alpha3
cue eval ./...

What did you expect to see?
success
What did you see instead?
_#field: reference "uintptr" not found:
    ./cue.mod/gen/github.com/gogo/protobuf/proto/pointer_unsafe_go_gen.cue:11:10
package "reflect" not found:
    ./cue.mod/gen/github.com/gogo/protobuf/proto/properties_go_gen.cue:7:8

I'm seeing this too, with cue get go istio.io/api/networking/v1beta1. I get the same error message.

reference "uintptr" not found is for the same reason as, and can be worked around by, what was described in https://github.com/cue-lang/cue/issues/1336.

I just had this with 0.4.2 also.

Hi 👋– I have this "issue" with github.com/cilium/cilium/pkg/k8s/apis/cilium.io/v2 too and I found this issue here. I use cue 0.4.2.

It seems like this is a simple omission of the uintptr type from the Go import logic.


noob here, but i couldn't import asrta vector search from store

what exactly i've expreienced, was that i couldn;t find out astra-vector-search component in locally installed langflow(maybe version issues), but only at stores. so i made choice to use it,  then i import and findout that it now could be seen at my project but only confronted sudden error happened. glad if floks provide some supports

@6921min what kind of environment were you using?
could you try to run
pip install langflow -U --force-reinstall

or use the docker image
docker run --rm -p 7860:7860 langflowai/langflow:latest

i just found that in the lastest model, it such vector DB search fuction were been merged in to vector store components, coresponding to each single DBs.
And i got managed to sucessfully use as what i meant to do so.
anyway thx fir atention! ㅇㅅㅇ


Add Register page on the edge-agent wallet

Reference: https://wallet.interop.digitalbazaar.com/register

not required anymore


fix(deps): bump poetry-core

Issue number: #1646
Summary
Changes

Please provide a summary of what's being changed

This bumps the build system to use poetry-core.
Please note that I'm way out of my confort zone here, so this PR might not be the right solution for the symptoms.
But locally it fixed the problem for me.
User experience

Please share what the user experience looks like before and after this change

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[ ] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

No idea why tests are not running here.

No idea why tests are not running here.

it's because it's not changing any python file.

Thanks everyone! My two cents from what I've seen in the wild: The switch only involves that and should work out of the box.


Update Image in graph

Is there a method to produce an image (in my case QR Code which is rendered to a png) and then have it displayed via the normal fill method (fill: {:img}) currently i can get it to render once but it takes ~4secs for the png to render correctly so nothing is displayed on the second attempt
Alternatively, I have the output of the QR Code as binary, is there a method to render a group of rectangles given a list? i.e i have a list:
 [1,1,1,0] it should render three white squares and one black square next to it. Digging around in the core library yesterday i played with grouping etc, but i think i need a little more info for it

It should work in theory... but isn't an area I've really dived into yet. That is... dynamic generation of image content.
The key is to put image data of a supported format into the cache so that the renderer can use it. What in-memory image format are you generating for the QR code?
Or, if you like, what library are you using to generate the image...

Before diving too deep into this, please take a look at how @wstucco did it in this repo:
https://gitlab.com/wstucco/scenic-color-cycling
He is dynamically editing an image and pushing it out into the graph. Works really well.

Thank you very much, I was using png, now playing with placing the rectangles and altering the color codes on a update based on the Id of the block. I'll take a gander at the above library.

Thanks for that, I tweaked a few things based on the above project and it now works perfectly.


Attempt to fix Spanner integration tests.

Currently failing https://scans.gradle.com/s/ajlgofucflem2/tests/failed

No longer relevant


[#2292] improvement(PyClient): Fix unstable Python client integration test

What changes were proposed in this pull request?

Add skipPyClientITs param in Gradle build script. Let's Python client integration test not running in the backend ITs
Combined Gradle build command of Python client test and integration test
improvement FilesetCatalog integration test codes.

Why are the changes needed?
Fix: #2292
Does this PR introduce any user-facing change?
N/A
How was this patch tested?
Through CI check.

@xunliu
Is it ready for review?

@jerryshao Please help me review this PR, Thanks.


VAE part of model

Hi, it looks like that  this code actually train not VAE model but simple auto-encoder model. Here are reasons:

Epsilon std is 0.01 https://github.com/maxhodak/keras-molecules/blob/master/molecules/model.py#L58 when it should be 1. I assume that it is safe to say that  there is almost no sampling.
KL loss should be very small because there is mean operation https://github.com/maxhodak/keras-molecules/blob/master/molecules/model.py#L78. In that case there is mean along feature and sequence shape. But both of this should be summed to obtain right KL loss relative to crossentropy loss.
The picture in readme also indicates that, because not all regions in latent space are covered by points. And authors wrote in paper that they observed this when they trained simple auto-encoder model.

May be it makes sense to simple train autoencoder model and compare results.

Hi,
I'm working on a similar problem but with protein sequences rather than molecules
You mention epsilon_std is not 1, which also seems quite strange to me
Yet, I found it was often the case in other codes (for example the Keras tutorial on VAE)
When I changed mine from 1.0 to 1e-3 some months ago, it allowed the model to learn (it didnt before)
Would you say VAE arent suited for that problem ?
Regards,
Sebastien

Hi Sebastien, do you have any update one the issue regarding epsilon_std?
I am trying to implement the same model in PyTorch and encountered the save problem. If I set epsilon_std to 1, the model refuses to learn anything (loss stagnates at very high values.)
If I change this value to 0, the VAE effectively degenerates to a simple AE. It learns very fast, recovering input sequences almost perfectly. But just like with any other simple AEs, the latent space it produces is sparse and it generates garbage when interpolating or decoding randomly sampled latent variables.
If I pick small, non-zero epsilon_std values, the result is between the two scenarios - the model learns better than when epsilon_std is set to 1 but not as good as when it is set to zero. In none of the cases the model works as good as described in Aspuru-Guzik's paper.

@lyu18
I encountered the same problem as you. And I looked into the code of original paper and found out that they anneal the epsilon_std.  Maybe this can help the model training. I will try it quickly.

@allenallen1037 That makes a lot of sense! Please let me know if you get any results. Thanks

@lyu18 It helps to improve the reconstruction accuracy when training. This is expected since it is some kind of tradeoff between AE and VAE. But the KL divergence loss is quite large, which means the latent space may not be smooth. I will do more investigation when finishing training.


Refactoring for packaging purposes

I have added updated some metadata, and restructured the package directories (for detailed description check commit summary of Restructured src)
For additional corroboration of this being a required and reasonable approach I attach two images of the pytest implementation:

Other things to consider:

I used relative imports
We can append files which we don't want to be part of the API with _

Nice commit message - What do you think of moving the files for pyparser, expressions and scanner into a directory called something like _internal? (and also prepending _ to tokens?)
src/
  pychart/
     runner.py
     _internals/
         scanner.py
         expression.py
         pyparser.py
     _token/
        ...

In regards to the relative imports - I did it last time cus of a circular dependency (maybe this fixes that problem?) that caused pytest to crash, so I'm happy as long as it passes the testing suite

Nice commit message - What do you think of moving the files for pyparser, expressions and scanner into a directory called something like _internal? (and also prepending _ to token_type?)
src/
  pychart/
     runner.py
     _internals/
         scanner.py
         expression.py
         pyparser.py
     _token/
        ...

I think just appending _ to everything that is internal is better

Idk to me its very messy to have lots of files in the same directory, and it makes it harder to read/reason with. So if we can split files into nice little chunks why not?

Idk to me its very messy to have lots of files in the same directory, and it makes it harder to read/reason with. So if we can split files into nice little chunks why not? (Theres nothing worse than scrolling through a huge list of files cus they're not segmented nicely lol)
You know what I mean?

@miningape
Sure but I don't think we should group things based on them

In regards to the relative imports - I did it last time cus of a circular dependency (maybe this fixes that problem?) that caused pytest to crash, so I'm happy as long as it passes the testing suite

@miningape
I also once had a problem with circular imports, but I think they can only happen with from . import *

@spralja "based on them" wym?

@spralja "based on them" wym? (Just saw u removed it nvm)

@miningape I don't like the name _internal because it feels like a 'forced' grouping maybe _interpreter?

@spralja go with _interpreter it fits better tbh (unless u can think of something even better)


Windows >2GB file write issues

Fixes an annoying issue on windows where writes greater than 2GB do not complete.
This is probably due to the underlying windows API being 32-bit.
The solution is to cut up the write buffer and write 1GB at a time.

+1 verified on windows


updating readme from branch example

##updates
I updated the readme file please review and accept

Thoughts
I should probabll +1


Add LIMIT param to Character endpoint

Add the LIMIT parameter to the character endpoint.
Users can limit the number of responses from the API
Something like this
https://finalspaceapi.com/api/v0/character?limit=4
This will result in first 4 characters in the Database
Resources
https://stackabuse.com/get-query-strings-and-parameters-in-express-js/
https://expressjs.com/en/resources/middleware.html
https://masteringjs.io/tutorials/express/req

Can you assign me to this task?

@PSmigielski
Sure
Let me know if you need any help.
Thanks

i can't connect to remote mongodb database

Hey
You can use a local database with test or dummy data.

Hey @PSmigielski
#49 and #51 #50  are somewhat related to this
See this comment:
https://github.com/lelouchB/final-space-api/pull/51#pullrequestreview-503771740
Both these code blocks will go in helpers/options.js  file.
Let me know if you have any questions

Hey @PSmigielski
#51 is finished, you can follow it and add a limit field to helpers/options.js.
Should be pretty straightforward.

Ok

would you like me to do this one?


103-animation-frame

#103
I think that sync only rerender with animation frame is enough:

It saves battery+cpu if animations are stopped in inactive browser tabs .
Animations / rerenders are smoother and less resource-intensive in active tabs.
There won't be some hard-to-debug problems with old models.
Implementation is relatively simple.
We don't lose render control (render / skip, ...).

So I've implementated only sync rerender with AF in this PR.

I've tried to reproduce problems with jumping cursor in Elm and it seems working.
I've tried to send chat messages to inactive window and it invokes rerender on tab activation (websocket example).

Changes:

Added examples/animation_frame
Added variant ForceRenderNow into enum ShouldRender + updated comments and added associated mehod into Orders
request_animation_frame modified and it's accessible with related types from user scope now

Ideas:
We can add some subscriptions (?) like onAnimationFrame or onAnimationFrameDelta in the future to remove boilerplate for animations.

Awesome.


fix(chat): add error message and border if dot at the end of prompt name appears while renaming (Issue #1103)

Description:
Add error message and border if dot at the end of prompt name appears while renaming
Issues:

https://github.com/epam/ai-dial-chat/issues/1103

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review

/deploy-review


repetitive factor(ifelse...) statements

There are many mutate staments of the form
factor(if_else(SexCd == 1, uni_s[1], uni_s[2]), uni_s)
They should be generalized via a function, something like
fct_if(uni_s, 1,2)

moved to Gitlab


Wrong string for invalid command

As we can see on the picture down below, when I enter unsupported command, it just prints out that command. I would like to see some string like "Invalid command: ".
(Sorry for using Windows, I am planning to play with Nvidia CUDA)

test is command
sorry


Dont count items that are previously clicked in hitrate calc as default

Tested on a couple of cases I had handy and the hitrate is lowered by a reasonable amount.
Breaking change: Changes the previous default behaviour to not including previous clicks, but left as an option in the hitrate module.

Couldn't you just count hits like normal, and then subtract the sum of "noCount hits" at the end? That would make it a bit simpler, and make the need to worry about masking noCounts and stuff fall away

Unfortunately, no. What we really want to do is to exclude the items clicked in training data from being allowed to be recommended. For example, if a user clicked on item 5 in training and in test period, we cannot first count all matches (2) and then subtract the training match (1). That would leave us with 1 match, but the desired result is 0 matches.


Firefly Application should say App Builder application in graphic

Is there an existing issue for this?

[X] I have searched the existing issues

Which topic?
https://developer-stage.adobe.com/uix/docs/getting-started/app-builder/#what-is-app-builder
What's wrong with the content?
Text is all about App Builder, but the graphic on page says App Builder application. The graphic should change this to App Builder application. Firefly was the pre-release code name for App Builder and should not be used unless talking about Adobe Firefly, our generative AI solution.
What changes do you propose?
Please update the graphic to rename the box Firefly application to App Builder application.
Anything else that can help to cover this?
that's it.

Finally done.


How to make vehicles that are added through Unreal Engine run?

Hi , I want to achieve automatic overtaking training without going through the reverse lane. Since the simulator currently does not support multiple lanes, I widened the width of one-way lanes. And added vehicles through the Unreal Engine in the widening lane. But the added vehicle is static. And how to make Non-player agents to run on the extended road?
I want to know whether the current simulator supports my idea. If it can be realized, what do I need to do?Thanks for your help in advance.

Hi @zhangfree2018,
What you are trying to do is very difficult to achieve with Carla. Our vehicles require a URoadMap to drive, this is a sort of image that tells them the direction of the road at any point. Our road generator already generates and serializes this map in editor, but if you use your own roads you'll have to adapt it. Then, the autopilot of the vehicles is very simple and was written with the assumption that there was only one lane, most probably, the vehicles will drive through the center of the widened lane. You probably need to adapt this too.
For the vehicles to start moving you need to spawn their controller too right after spawning the vehicle. The easiest is to put some player starts around and the Carla vehicle spawner should put vehicles with the right controller.

Hi @nsubiron ,
Sorry for my delayed reply. Your answer means that I can't modifiy Twon1 and Town2 you provided. How is the image that  tells them the direction of the road at any point obtained? Currently, my demand for maps requires multiple lanes. Because I don't have the experience of building a simulation environment. Can you provid some simple operation documents? I have read the documents about Map customization you provided, but I can't understand.


Basic welcome page

Home page should direct to this page

Hey team! Please add your planning poker estimate with ZenHub @AnnieTran13 @bcobo341 @luigibesani

maybe implement upcoming events in calendar form

Very nice design. The color theme is looking good. Having upcoming events on the home page is a great idea.


Create the transmission tasks that do the work of sending data to vendors

Gather the pre-processed marc files.
Initialize a connection (FTP, API, etc.)
Transmit the data (marc or record id)
Archive the transmitted data files.

Blocks #869


fix fix single compaction input rowsets

Proposed changes
Issue Number: close #xxx
pick from master #33053

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall


Command precedence should be honored irrespective of whether an auto-loading module is currently already imported or not

As described in about_Command_Precedence, cmdlets take precedence over external programs with the same name.
However, this currently only applies to cmdlets whose containing module has already been imported into the session at hand.
This makes the behavior hard to predict.
Given that the commands exported by auto-loading modules are known in advance, before actual import, there's no reason to consistently give them precedence, even if that entails import on demand.
Steps to reproduce
Run the following on Windows.
 pwsh -noprofile -c '$null>$env:TEMP\Get-Service; $env:PATH += '';'' + $env:TEMP; (Get-Command Get-Service).Source; Remove-Item $env:TEMP\Get-Service' |
  Should -Not -BeLike "$env:TEMP\*"

Expected behavior
The test should succeed, because submitting Get-Service should find the cmdlet of that name in the auto-loading Microsoft.PowerShell.Management module - even though that module hasn't been loaded yet - not the external file in $env:PATH by that name.
Actual behavior
The test fails, because the external file in $env:PATH unexpectedly takes precedence.
Environment data
PowerShell Core 7.0.0-preview.4

This is a good find. The problem is of course that proving a negative takes longer. We should look into fixing this, but it's probably a breaking change from Windows PowerShell and would also cause significant command invocation slowdown.

The whole point of the cmdlet cache is to automatically load cmdlets that aren't present and this works today. I therefore assume that the issue with Get-Command is that it's using a different code path than CommandDiscovery which is weird.

cause significant command invocation slowdown

I would much rather live with a minor, obscure inconsistency (if there is one) than take a big perf trying to fix it.

it's probably a breaking change from Windows PowerShell

Technically, yes, but it's hard to imagine that someone actively relied on this behavior.

would also cause significant command invocation slowdown.

Why would there be a significant slowdown?
The session already knows about the command names from not-yet-auto-loaded modules, so it sounds like you simply need to consider those too when looking up by name, which I wouldn't expect to have a tangible impact (though I know little about the plumbing).
Or are you specifically concerned that  immediately after session start-up it takes a while to collect all names from auto-loading modules, which would slow down CLI invocations with -Command?

Why would there be a significant slowdown?

Say you have a script called build.ps1 on the filesystem in the current directory.
When you run build, it must look for commands matching that before falling back to the script.
First we look for loaded commands. That's fast.
Then we look in the module analysis cache. Slower, but still acceptable. But still no result.
So now, we're forced to look on the module path for any module exporting a command or alias called build. This is because the cache might not capture modules newly installed or added. This is very slow, because:

We must find every module on the module path
We must inspect all manifests
In the cases where manifests use wildcards, we must do module analysis (where we analyse script modules to see if we can work out their exports).

Only after we have looked at every module on the module path can we fall back to invoking the local script file. And because the filesystem can always have a new module plonked onto it, the cache can never prove the negative, so we always must look. Meaning script invocation now gets much much slower.

But I also agree that autoloading should be transparent and that it shouldn't affect command precedence. Invoking a script without the slash in it is something I personally wouldn't recommend and haven't seen much of. But I can imagine not everyone agrees with that second point.

I therefore assume that the issue with Get-Command is that it's using a different code path than CommandDiscovery which is weird.

We first try to resolve the command with CommandSearcher and then try autoloading:
https://github.com/PowerShell/PowerShell/blob/49e906bc3bd302e54ab1f5ad00c5bda8f8fba07f/src/System.Management.Automation/engine/CommandDiscovery.cs#L793-L822
CommandSearcher looks like this:
https://github.com/PowerShell/PowerShell/blob/49e906bc3bd302e54ab1f5ad00c5bda8f8fba07f/src/System.Management.Automation/engine/CommandSearcher.cs#L76-L243

I appreciate the background information, @rjmholt - sounds like there is indeed a serious performance concern; let me mull that over a bit.
@BrucePay:

inconsistency (if there is one)

If the OP leaves any doubt as to whether there is an inconsistency, let me know.

How many name conflicts do we have? It seems it is an edge case.
For interactive session it is not critical too - user can fix his mistake by adding extension, or path to the command name, or install/import a module.
For reliable scripts we would use full qualified cmdlet names and full path to scripts to avoid name conflicts. Otherwise, there is always a way to break the script. (Even just changing PATH env variable)

Given the performance implications, it makes sense to live with this inconsistency and simply document it - see https://github.com/MicrosoftDocs/PowerShell-Docs/issues/4976 - so I'm closing this.
@iSazonov, @rjmholt
I don't expect it to be common either, but naming conflicts strike me as most likely with *.ps1 scripts placed in $env:PATH.
I use this technique personally to avoid having to cram too many functions into my $PROFILE (and when creating a whole module would be overkill), but, more importantly, the technique makes sense for convenient invocation of PSGallery-installed scripts (Install-Script).
With other kinds of external executables, naming conflicts with cmdlets are unlikely, though more so with functions with non-standard names and especially aliases.
Yes, you can work around the issue, but the points is that you may not be aware of the need to do so, especially given that the behavior differs situationally.

I don't expect it to be common either, but naming conflicts strike me as most likely with *.ps1 scripts placed in $env:PATH.

We could continue (async) search after we found first command and write warnings about possible name conflicts.

Thanks, @iSazonov - I personally think it's sufficient to document the issue.
Just to prevent confusing future readers, @rjmholt - your helpful comment with background information
mentions falling back to the script in the current directory if just build is submitted, but PowerShell by design doesn't allow invoking scripts/executables in the current directory by mere name; the issue at hand only arises with executables/scripts in the path.

I'll make an edit just so there's no jumping around required


CentralAdministration - SecureSocketsLayer

Details of the scenario you try and problem that is occurring:
CentralAdministration is created with http
The DSC configuration that is using the resource:
SPCreateFarm
Version of the Operating System and PowerShell the DSC Target Node is running:
Windows Server 2016, SharePointDSC <IP_ADDRESS>
Version of the DSC module you're using:
Would like it to be possible to create the CentralAdministration with SSL

@bardtomm You can do this through combining a couple of resources

Create the farm with SPCreateFarm
Add a HTTPS alternate URL with SPAlternateUrl
(optional) Bind your SSL certificate to the site in IIS using xWebsite in the xWebAdministration module

This is the approach I've taken with a number of customers and it works a treat. The reason we do it like this is that the central admin site always comes out of the New-SPConfigurationDatabase command as HTTP, so in the same way as you would normally need to add the URL we do the same with DSC. Does that make sense?
I'm going to close this issue off as it's not something we can change in the DSC resources, but if you need help using the alternate URL approach let me know.


Funny business is happening with the territory side panel

Describe the bug
For some reason (this is maybe on purpose) but census tracts in Guam are no longer selectable. When I was able to select them, tracts were showing up as disadvantaged but there was no data being used to make that designation. I will also posit that the side panel design can be improved for the territories.
To Reproduce
Steps to reproduce the behavior:

Go to 'Explore the tool'
Click on 'Guam'
Zoom in 'Can't select census tract + designation is off.'
See error

Expected behavior
Census tracts should be selectable and there should be some better way to understand / represent the data beyond just -- especially when it is not expected to be there (like the whole category)
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Mac OS
Browser Safari
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@KameronKerger - are we still seeing this?

no because we don't have data for Guam @vim-usds


[docs-only] Running oCIS on Kubernetes

@rhafer @wkloucek could you give this first draft a try to the setup with minikube and replicate the output?
Mind that the charts I made half a year ago are using a fairly old ocis image: owncloud/ocis:1.0.0-rc8-linux-amd64, I would need to update them with a latest review.
Also mind that the WebUI is not entirely loading because we need a deployment with an external IDP, perhaps a Keycloak local setup with a reverse tunnel, so we don't have to deploy things elsewhere.
Comments and feedback are more than welcome!

could you give this first draft a try to the setup with minikube and replicate the output?

After a bit of a fight with my minikube setup I got this to run. :fireworks: . I still wonder a bit whether that mdns based registry is really a good fit for a kubernetes deployment. Will that really work in a multi-node setup (especially when using cni plugins a little fancier than the default)? But I guess we can work with it for a start.
Also, I guess, in a real-world setup you'd need to expose the grpc endpoint also via service objects wouldn't you? Currently those only exist for the http endpoints.
BTW, you might wanna update your minikube to get a somewhat newer kubernetes version. 1.18 is EOL :slightly_smiling_face:

still wonder a bit whether that mdns based registry is really a good fit for a kubernetes deployment

No, not at all.

Also, I guess, in a real-world setup you'd need to expose the grpc endpoint also via service objects wouldn't you? Currently those only exist for the http endpoints.

Because there is a service registry being handled by Micro, we find these backends by their names. I'm smelling the kubernetes service registry uses service objects to expose the backends, but have not read that part just yet 🤷

BTW, you might wanna update your minikube to get a somewhat newer kubernetes version. 1.18 is EOL 🙂

Thanks! will do

As a shameless plug of self-promotion (and also to keep myself from repeating the same things at various points), I have written a little bit more in depth explanation (and with less lying about the magic happening there) of k8s, which might have a place near the https://dev.to/miguelmota/comment/filh reference/end of the k8s overview section.
It's a 4 part series, beginning with the following article:
http://deaddy.net/introduction-to-kubernetes-pt-1.html
In particular the networking piece might be of interest with regards to the mdns registry question and in my experience it actually often helpful to clear up some confusion about services and ingresses, as most of the documentation seems to be written for the big cloud providers or stuff like minikube, but not what you might see in production on prem deployments:
http://deaddy.net/introduction-to-kubernetes-pt-2-ingress-networking.html
I guess at some later time we will find time to have a little less patchy and more integrated documentation.

@Deaddy I updated the docs to also take into consideration your blog series, as it adds value for further read. Pretty good write up! Thanks for sharing with it with us!


test: add existence property to leaf_stage testing

Checking that effectively every created leaf contains all the expected values increases the execution time of the leaf_stage test by 1 second (on my machine). However, this increase is not noticeable when the entire test suite is executed. More importantly, it really checks the correctness of the extension range protocol

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#505  👈
#504
#503
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gabriele-0201 and the rest of your teammates on  Graphite

Merge activity

Oct 24, 6:32 PM EDT: A user started a stack merge that includes this pull request via Graphite.


Icl tests

This adds unit tests for several tokenizer + ICL dataset pairs. It also updates the readme for eval and updates jeopardy to include the new line in the context!

should we change the relative filepaths in the yamls so that we can run composer eval.py YAML rather than the updated composer eval/eval.py YAML?

should we change the relative filepaths in the yamls so that we can run composer eval.py YAML rather than the updated composer eval/eval.py YAML?

We are going to keep all paths relative to scripts so that training YAMLs can also refer to eval/yamls/tasks.yaml


allow chained futures to be created immediately

Use case

I have:
List<Future> lst = [];
lst.add(
    somethingThatReturnsAFuture().then((){
        //do stuff here
    })
);

The intent of the above code snippet is to add the future corresponding to the then call to the list of futures.
However this future does not get added to the list until somethingThatReturnsAFuture() completes successfully. So if somewhere else in the code you have something like await lst.first, you may encounter an exception because lst doesn't actually have any futures in it.
Proposal
I think it would make sense if the future that corresponds to then is added to lst right away, even though it itself cannot get started until somethingThatReturnsAFuture() completes. To do this would require "chained" futures to all be created immediately. The workaround that I see in the current state of flutter is for you to first ensure that somethingThatReturnsAFuture() has completed, and only then add the future you're actually interested in to the list, which isn't as clean.

Hi @Fnalsk
From what I can see, the issue is related to Dart rather than to Flutter itself. Please open the issue in the dedicated repository.
I think Future.wait(futures) is what you are looking for.
Closing, as this isn't an issue with Flutter itself.
If you disagree, please write in the comments,
providing your flutter doctor -v, your flutter run -v,  your pubspec.yaml
a minimal reproducible code sample that does not use 3rd party plugins,
and I will reopen it.
Thank you


Support for max_execution_time and max_input_time

max_input_time and max_execution_time are currently disable as they conflict with ZTS.
PHP will need to be improved if we want to support these options: https://externals.io/message/118859

I wonder if we could emulate this?
Provide a function in go (maybe go_register_execution_timers?) that is called during initialization in C by reading the settings from php.ini:
go_register_execution_timers(PG(max_execution_time));

Then in go, we create a timer and listen for the end of the request:
select {
  case <- httpRequest.Context().Done()
  case <- time.After(maxExecutionTime)
}

// tell PHP to die if it is still alive

WDYT?

Done for Linux. It will probably not be possible for mac and Windows.

hi @dunglas where can we set the max execution time and what is the default?

In your php.ini, as usual. The default is 30s.


Don't retry flakey specs whilst in a local environment

Context
We don't need rspec-retry to run locally. This changes the config to only retry failing specs whilst inside a PR build
Changes proposed in this pull request
Wrap the rspec-retry config around a conditional that checks for the CI environment variable.
Guidance to review

Link to Trello card

Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

Tested this with a flakey spec:
it 'should randomly succeed', :retry => 3 do
  expect(rand(2)).to eq(1)
end

Can confirm it didn't retry locally but triggered the flakey message whilst on the build.


Feature Request - resetting environment from python

Current simulator features are fine for behaviour cloning.
However if you would like to train agents with algorithms that require interaction with environment you're out of luck - if agent get's stuck somewhere out of the road the environment will be useless without manual intervention.

Such functionality already presents in term2_collection branch. Check script CommandServer_pid.cs


well-known serde for Vec<OffsetDateTime>

I am porting my program from chrono to time. One thing I am stuck with is serializing a Vec<OffsetDateTime> with the rfc3339 format.
This seems like a small but very useful addition. Is this something that would be considered to be included?

This is what just tried to use in the serialize_with attribute:
fn date_vec_ser<S: Serializer>(
    vec: &Vec<OffsetDateTime>,
    serializer: S,
) -> Result<S::Ok, S::Error> {
    let stringvec = vec
        .iter()
        .map(|odt| odt.format(&Rfc3339))
        .collect::<Result<Vec<String>, time::error::Format>>()
        .map_err(S::Error::custom)?;
    stringvec.serialize(serializer)
}

I'm not going to say it's 100%, but I think this is reasonable.

There is support for the well-known formats in serde_with, such that you can write this code and support other nestings.
#[serde_with::serde_as]
#[derive(Debug, serde::Deserialize)]
struct Data(
    #[serde_as(as = "Vec<time::format_description::well_known::Rfc3339>")]
    Vec<time::OffsetDateTime>
);

let v = serde_json::json!([
    "1997-11-21T01:55:06-06:00",
    "2000-01-21T05:55:06-06:00",
    "2020-03-21T09:55:06-06:00",
    "1997-11-21T23:55:06-06:00",
]);
let d: Data = serde_json::from_value(v).unwrap();
dbg!(d);

https://www.rustexplorer.com/b/kdutp7


Impossible to raise both flippers simultaneously in portrait mode

With the current control mapping for Portrait mode, it is impossible to raise both flippers simultaneously, as they are mapped to opposite ends of the + Pad, and have no alternative inputs available. Please consider adding another input to activate both flippers, or adding a flipped portrait mode that uses the ABXY buttons instead of the + Pad for flipper control


feat(control-interface)!: wrap all operations in CtlResponse

Feature or Problem
This PR tackles a large refactor to better wrap errors and responses for the control interface. After #1108 our topics are much more predictable, but it still remained difficult to differentiate between protocol errors, deserialization errors, and control interface handling errors. This PR makes this much easier to do by more strongly typing handlers in the host and the control interface, so as a user you can tell the difference between these issues.
Note that a good bit of the handlers use helpers like CtlResponse::success() or return CtlResponse<()>, which is because of the generic type. Even if we return None as a response, we still need to "type" it just in case it was a Some. This works pretty well though.
Related Issues
This is based off of #1575 and moved into its own PR because it's sufficiently large.
Release Information

Consumer Impact

Testing

Unit Test(s)
A big part of this PR is the ctl_response_comprehensive unit test which is ignored right now for general logistics, but passes perfectly well with a running NATS server and host. I wanted to ensure that each and every one of our control interface operations is working properly, albeit with one or two error cases that I'm taking advantage of. This is a large enough change that ensuring that it's working end-to-end is important so we know the control interface is sound.
Acceptance or Integration
We'll fully exercise this functionality with wRPC integration tests, but it will be important to handle error cases well on the client side depending on the operation's outer Result and inner success member.
Manual Verification

Will merge once I can confirm it's not going to disrupt @vados-cosmonic too hard

Hey go ahead and merge this @brooksmtownsend ! I'm ready to rebase once you do.

@vados-cosmonic Done!


Screen cheese with SCM main button

have an incoming change
resize the panel
:bug: the label is duplicated

https://user-images.githubusercontent.com/1794099/208411390-cee36c5f-7e80-4815-a6fd-b5998eeca3aa.mov

I believe that this is a regression introduced by https://github.com/microsoft/vscode/commit/3fcbb1cc96376cd4aba91695138bc8801ed2b64b. //cc @daviddossett


Use RAW GP as standard for currency conversions

I'm the maintainer of World Currency 5e, a module that allows for currency customization. This pull request changes the currency conversion schema so that all currencies are defined in terms of GP (according to rules as written).
There are a few problems with the current schema:

Module creators often hard code their own conversion logic because it's convoluted to use the system conversion numbers to convert a currency to/from GP.
Completely omitting a currency like Electrum can't be achieved while keeping copper and silver.
We can't have more than one currency that's worth more than GP.
Defining new currencies in terms of each other isn't intuitive for most GMs.

I believe that the current schema was designed to make the "Currency Converter" logic straightforward. I have updated the currency converter logic so that it'll work exactly the same as before.

I'd propose that the current iteration be merged. Refactoring working code is outside the scope of my original goal. Am I correct in understanding that deepClone here doesn't produce any measurable problem?

It doesn't. I don't think that's a problem which is preventing this PR from being merged though.

Thanks. Any other changes you'd like me to consider at this point?

Thanks for the above. I'm still not as familiar with Javascript as I'd like to be, so I appreciate the formatting suggestions. Anything else that can be tightened up?  Also, are you in agreement with the proposed currency conversion schema in config.mjs?

Is the conversion the same as it was previously?

Yes, the values of each currency are the same relative to each other.

Then the only thing I can really think of is adding to the documentation on config.msj, line 620, mentioning that the conversions are relative to a single "base" unit (which in this case has a one-to-one equivalency to GP).

Good call. I've updated the documentation, as you suggested.

See https://github.com/foundryvtt/dnd5e/commit/7153fce3def063d7e7d53498e3b1be6cadc86fd2 for follow-up tidying.