Remove transmutes with hand-written structs

Fixes #207.
There's more than one possible way to fix the UB coming from transmute. This one tries using hand-written structs plus the zerocopy crate.
There's some ugliness there: because it's not possible to reinterpret padding bytes as a byte buffer, I added manual padding. No code should ever need to interact with the padding fields, because the Default trait is implemented and it's possible to do partial initialization and fall back on ..Default::default() for everything else.
As long as I'm trying this solution out, push the code to be more idiomatic:

add placeholders for documentation
convert DBState to an enum (only partially successful)
convert field names to snake case

Using serde might still be an improvement over this, because it would allow us to use a wider range of types while maintaining the same serialization format. But this should be far enough implemented to get a sense of whether we like this or not.

I do not think that explicit definition in Rust structure mapped to correspondent Postres C structure (rather then rely on FFI) is really good idea. I do not expect that format of such stucts pg_control file or WAL page header will change frequently.
Most likely not. But i any case it seems to be much safer and convenient to rely on autogeneration rather than duplicate structure definition.
Frankly speaking I do not completely understand the problem with #207. Miri reports UB? But it is so important for use to use Miri? And if so, is it possible to provide some small fix? For example just initiale buffer with zeros:
pub fn encode_foo(rec: FooStruct) -> Bytes {
    let b = [0u8; XLOG_SIZE_OF_FOO];
    b = unsafe { std::mem::transmute::<FooStruct, [u8; XLOG_SIZE_OF_FOO]>(rec) };
    Bytes::copy_from_slice(&b[..])
}

I thing that if FFI is avaible for Rust and we are really manged to make it work, then we should use this mechanism, rather than invent something else, requiring duplicated definitions of Postgres structs.
I also wan to notice that there can not be 100% solution of this problem. Postres is storing data in native format which depends on target system, compiler and compiler options. In theory you can build Postgres in such way (using some specific compiler or compiler options) that format of the data will not be compatible with one generated by FFI.  But I expect that  probability of it is very small and definitely much smaller than i case if we define structure ourselves (using some dummy  padding fields) and provide our own serializer for it.
Fortunately, there very few places in Postres where are "holes" in the struct which are persisted to the disk. Acgtually I didn't know any other except XLogPageHeaderData.

Frankly speaking I do not completely understand the problem with #207. Miri reports UB? But it is so important for use to use Miri?

I've been waiting for this question to rise :) Miri is an embodiment of the Rust language standard (which is a work in progress). It's used in the rustc for const evaluation and some other stuff, which makes it credible in my eyes. If it points to something, we'd better take a closer look at it. The problem won't go away if we ignore it.
I encourage you to take a look at this list of bugs found by Miri.

Frankly speaking I do not completely understand the problem with #207. Miri reports UB? But it is so important for use to use Miri?

I've been waiting for this question :) Miri is an embodiment of the Rust language standard (which is a work in progress). It's used in the rustc for const evaluation and some other stuff, which makes it credible in my eyes. If it points to something, we'd better take a closer look at it. The problem won't go away if we ignore it.
I encourage you to take a look at this list of bugs found by Miri.

Sorry, I have never used Miri but because I am dummy in Rust world. But I have widely used valgrind and similar tools for detecting problems in C/C++. Is it true that Miri is useful only in case of presence of unsafe code? May be I am wrong, but looks like all listed problems should not happen with safe language...
In any case, I think that it is necessary to clarify what we mean by UB here. If you have uninitiallized variable (in C) and use it in some expression, then we definitely get undefined behavior. If you have some gap between structure fields, than UB is possible only if you are comparing this structure as sequence of bytes. But nobody does it with XLOG page headers! So IMHO it is not quite correct to say about UB here. I prefer to classify this problem as "make compiler happy", so there is no real problem but  we have some useful tool and we do not want to get any error massages from it. The obvious solution of the problem with uninitialized memory is ... to initialize it. I wonder if we can do something like I proposed in #207 to make Miri happy?

May be I am wrong, but looks like all listed problems should not happen with safe language...

That's true, but low-level primitives (e.g. io_uring, pthread mutexes, etc) might require writing some inherently unsafe code in order to provide safe abstractions. Rust can't validate this code for us, but the contracts must still be upheld at all times, even if a user comes up with a meaningless piece of code which abuses the api in question.

In any case, I think that it is necessary to clarify what we mean by UB here.

There's a (very) incomplete list of things that are considered UB. The rest is up to Miri.

I'm sad if we have to make a copy of all the structs that bindgen will create us. Let's try to find a way to do this without hand-writing code that we will need to keep in sync with Postgres sources.

BTW, regardless of how we deal with the undefined behavior when writing the control file, it's a good idea to replace transmute with zerocopy::FromBytes when reading it.

BTW, regardless of how we deal with the undefined behavior when writing the control file, it's a good idea to replace transmute with zerocopy::FromBytes when reading it.

May be I missed something, but how it is possible to use zerocopy::FromBytes for struct generated by FFI?
Documentation says that it is necessary to mark struct with #[derive(FromBytes)]) but how it can be done if struct is generated?

May be I missed something, but how it is possible to use zerocopy::FromBytes for struct generated by FFI?
Documentation says that it is necessary to mark struct with #[derive(FromBytes)]) but how it can be done if struct is generated?

I think @hlinnaka is assuming a theoretical version of bindgen that allows us to add derive attributes to the auto-generated struct. The maintainers have said in the past they're open to a change that allows this...

Closing this, let's do https://github.com/zenithdb/zenith/pull/444 instead.


Create DJL Time Series Dataset

Description
A time series dataset contains a sequence of events happening across time. Some examples are climate, stocks, and forecasting. This issue is to add any time series dataset to the DJL basicdatasets.
References

Possible datasets to implement

Daily climate time series dataset
DJIA 30 Stock Time Series

Some of the DJL NLP examples use a sequence of text similarly to a sequence of time series dataset, and can be used as a reference

Hello, I want to fix this issue and can you assign it to me? Thanks!

@WHALEEYE just did, thanks for your contribution

@lanking520 @zachgk Hello, I'm trying to add Daily climate time series dataset into the project, but since the dataset is on kaggle, the user may need to login first to download this. So I guess maybe the dataset should be stored on somewhere else to allow users to automatically download it when they are using DJL?

Yeah @WHALEEYE, it would be good to not have to log in to get the dataset. What we can do is store the dataset along with the metadata file in S3 and distribute it that way.
Of course, the prerequisite is that the license permits us to redistribute the dataset. Some licenses like for mnist or imagenet do not. Fortunately, the climate dataset follows the CC0 license which does say that we can distribute it.
Here's what you should do. When you are creating the metadata file and testing it locally, set up a directory structure something like this:
/path/to/climate/dataset
    metadata.json (here is the metadata file for the climate dataset)
    1.0 (assuming dataset version is 1.0)
        datasetFile1
        datasetFile2
        ...

Then, in the metadata you can use a relative uri such as "1.0/datasetFile1". An example of this is the banana dataset. After it is working and you make the PR, don't add the dataset files to git. Just give us a reminder in the PR message to upload the files with the metadata and we will try to get it to match the metadata format. You can also compress any of the dataset files individually with gzip and it will be automatically extracted

Thanks! I also get one question: this dataset seems not have labels, but the Record class is expected contain both feature and labels, so what should I place at the position of labels in the returned records?

For unlabeled data like time series, you can just use an empty NDList for the labels

Ok, I got it.


Live editor

As expected, when running the plugin the live editor seems to have no effect, presumably as it's serving a static page. Is there any workaround for this or plan to not run live edits through the cache?

It should be disabled for previews, it uses the request service to check if
its a livepreview or not..
Op vr 8 apr. 2016 11:55 schreef nathobson<EMAIL_ADDRESS>
As expected, when running the plugin the live editor seems to have no
effect, presumably as it's serving a static page. Is there any workaround
for this or plan to not run live edits through the cache?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/craftapi/htmlcache/issues/11

--
Met vriendelijke groeten,
Chris

FYI: it's unexpected behaviour as it's defined here:
https://github.com/craftapi/htmlcache/blob/master/services/Htmlcache_HtmlcacheService.php#L44

OK, I'll run some more tests to see if it's an issue my end, which it sounds like it might be.

OK, a different environment and all appears well. I believe it might might be down to cache headers that were being added automatically by another host's setup which would stop live edit working regardless of having htmlcache active. I'm a dope :)

No problem, appreciate the feedback nonetheless!

Appreciate the crazy fast responses and thanks for releasing the plugin :+1:


Support for DDL statements?

Based on pull request #10, it looks like DDL statements are supported. Is that correct?

DDL are supported, but their result/output is somewhat broken since we skip the first line of Athena results (usually contains header). That PR would fix this but lacks tests.Feel free to pick up on it!

Ok, thanks.

Recently, we released speee/go-athena.
We have already merged the PR related to this issue and can keep maintaining this library and review / merge your PRs.
In addition, we added the following new features.

Add Docker environment
Add reviewdog configuration for linting
Remove Result Header when executing DDL
Use workgroup
Update tests (make each package up to date)
Introduce Result Mode (Query result acquisition mode)

Especially, we think the Result Mode is a cool feature, and we summarized the feature here.
Please check and try to use it!

Recently, we released speee/go-athena.
We have already merged the PR related to this issue and can keep maintaining this library and review / merge your PRs.
In addition, we added the following new features.

Add Docker environment
Add reviewdog configuration for linting
Remove Result Header when executing DDL
Use workgroup
Update tests (make each package up to date)
Introduce Result Mode (Query result acquisition mode)

Especially, we think the Result Mode is a cool feature, and we summarized the feature here.
Please check and try to use it!

Summarizing work done since back in the day:

Original fix without tests, not merged: https://github.com/segmentio/go-athena/pull/10
Subsequent merged update with ability to skip headers, but not addressing DDL specifically: https://github.com/segmentio/go-athena/pull/13
Second attempt at fix building on #13 but most likely the reverse of what's needed: https://github.com/segmentio/go-athena/pull/31


Exception on quit

My Setup

Operating System: macOS 10.13.6 (17G39b)
App Version: 2.11.0

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description
Quit the app.

Current Behavior
Exception is thrown.
Expected Behavior
No exception.

This is old.


serialization?

I may be missing something in the docs, but how do you serialize a draft.js editor, send it to the server, and then load it again?
I know there are third party things for creating html, but I don't see how you'd load that back into the editor.
This is half a support question, and half a request for docs. I can write them once I figure it out.
I'm guessing you can't just .toJS/fromJS on the EditorState because there are sub-objects with special methods.

You can use these:
http://facebook.github.io/draft-js/docs/api-reference-data-conversion.html#content

Thanks! Sorry I missed that.

Don't apologize! Feel free to send a PR to make this clearer if you looked elsewhere in the docs.

I wrote a guide for doing it, but I'm not sure if it should be its own page, or part of getting started. Part of it is copied from the reference page. Also, feedback?

Serialization
Because a text editor doesn't exist in a vacuum and it's important to save
contents for storage or transmission, you will want to be able to convert
ContentState into a plain JS object, and vice versa.
import {Editor, EditorState, convertToRaw} from 'draft-js';

class MyEditor extends React.Component {
  constructor(props) {
    super(props);
    this.state = {editorState: EditorState.createEmpty()};
    this.onChange = (editorState) => this.setState({editorState});
    this.onSave = () => {
      var content = this.state.editorState.getCurrentContent();
      var raw = convertToRaw(content);
      $.post('/api/comment', {comment: raw}, () => {
        alert('Saved');
      });
    };
  }
  render() {
    return (

        <Editor editorState={this.state.editorState} onChange={this.onChange} />
        <button onClick={this.onSave}>Save</button>

    );
  }
}

You can then load it back into an EditorState.
var editorState = EditorState.createWithContent(convertFromRaw(raw));

To display this elsewhere in your site, you can create an Editor component with
the readOnly prop set to true.
<Editor editorState={editorState} readOnly />

Note that the Draft library does not currently provide utilities to convert to and from markdown or markup, since different clients may have different requirements for these formats. We instead provide JavaScript objects that can be converted to other formats as needed.

@brigand your guide it explains it pretty well.
However I have some questions regarding saving and reading from a database. So, say I write a blog post and get its content using convertToRaw. Should I save and read JSON in/from the database?

@barbu110 yeah, and since you don't need to query it, you can just store it as a string instead of using a json data field.

This should definitely be accessible somewhere in the doc. I feel like as soon as you use a rich text editor you want to input/output datas from it and this is lacking of informations on how/what can/cannot do Draft about this.
I saw loads of issues related to html conversion too, having a line saying that Draft doesn't support HTML export (and the import is very basic) somewhere on the Data Conversion page would help.
Also to complete this guide, I would say that would may need to serialise this as JSON like so:
Sending to DB:
JSON.stringify(convertToRaw(content))
Reading from DB:
Draft.convertFromRaw(JSON.parse(content))

@brigand, What if we want to get the raw data after reading it from database and render it as HTML? Is there any clean way you might be aware of to accomplish this? I know stateToHTML from draft-js-export-html but that doesn't, for example, support iframes. It simply removes them! Thoughts?

@kevinguard did you try <Editor editorState={editorState} readOnly />?

@brigand Thanks for prompt response. It worked like a charm! 👍

Definitely a little tricky to find in the docs, "data conversion" -> "import & export" or something maybe.

thanks a lot!

I wrote a guide for doing it, but I'm not sure if it should be its own page, or part of getting started. Part of it is copied from the reference page. Also, feedback?
Serialization
Because a text editor doesn't exist in a vacuum and it's important to save
contents for storage or transmission, you will want to be able to convert
ContentState into a plain JS object, and vice versa.
import {Editor, EditorState, convertToRaw} from 'draft-js';

class MyEditor extends React.Component {
  constructor(props) {
    super(props);
    this.state = {editorState: EditorState.createEmpty()};
    this.onChange = (editorState) => this.setState({editorState});
    this.onSave = () => {
      var content = this.state.editorState.getCurrentContent();
      var raw = convertToRaw(content);
      $.post('/api/comment', {comment: raw}, () => {
        alert('Saved');
      });
    };
  }
  render() {
    return (

        <Editor editorState={this.state.editorState} onChange={this.onChange} />
        <button onClick={this.onSave}>Save</button>

    );
  }
}

You can then load it back into an EditorState.
var editorState = EditorState.createWithContent(convertFromRaw(raw));

To display this elsewhere in your site, you can create an Editor component with
the readOnly prop set to true.
<Editor editorState={editorState} readOnly />

helps me,thanks a lot!

What if I want to query and modify the data in the database?
I need to somehow persist it normalized in a relational database so I can benefit from using foreign keys, do joins.
What is the general approach for that case?


[FLINK-8608] [flip6] Implement MiniDispatcher for job mode

What is the purpose of the change
The MiniDispatcher is responsible for submitting the single job with which
a job mode cluster is started. Once the job has completed and if the cluster
has been started in detached mode, the MiniDispatcher will terminate.
In order to reduce code duplication, the MiniDispatcher is a sub class of the
Dispatcher which is started with a single job submitted job graph store.
Brief change log

Introduce SingleJobSubmittedJobGraphStore for the MiniDispatcher
Refactored JobClusterEntrypoint and SessionClusterEntrypoint such that most of the cluster component logic moved to ClusterEntrypoint
Renamed JobMasterRestEndpoint into MiniDispatcherRestEndpoint
Initialize MiniDispatcher with single job retrieved by the JobClusterEntrypoint
Terminate MiniDispatcher after job completion if the its execution mode is detached

Verifying this change

Added MiniDispatcherTest
The tests of MiniDispatcher are also valid for the MiniDispatcher

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

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks for your review @GJL. I've addressed your comments with cb3fca3 and rebased onto the latest master.

Thanks a lot for the review @GJL. Merging this PR once Travis gives green light.


fix: mainnet infura rpc url

:tada: This PR is included in version 2.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add VScode Python Settings

Pull Request
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

#98
#97  👈
main

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

Join @JackPlowman and the rest of your teammates on  Graphite


bosh-agent for Windows!

The Greenhouse team proudly presents months of effort and cross-continent collaboration!

So excite.

and... opened this to the wrong branch (should be to develop, not master)


samples: up_squared: gpio_counter: Remove sample code

The gpio_counter does not build, the code references defines that don't
exist anymore in the tree.  Remove the test since its doesn't compile
and isn't every built by sanitycheck.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Codecov Report

Merging #10983 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #10983   +/-   ##
=======================================
  Coverage   53.21%   53.21%
=======================================
  Files         217      217
  Lines       26664    26664
  Branches     5913     5913
=======================================
  Hits        14189    14189
  Misses      10058    10058
  Partials     2417     2417

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d70eb98...1919ded. Read the comment docs.

Opened issues #10993 and #10994

PR #11006 should fix the build errors.

lets close this one now that we have a fix.


Add library for APA102 LEDs

The new APA102 LEDs are arguably easier to use than WS2812s.  Add a library that supports the APA102s.
See also:

https://cdn-shop.adafruit.com/datasheets/APA102.pdf
https://cpldcpu.com/2014/08/27/apa102/

Implementation provided.


Allow apps define components

I'm just new to unicorn, and faced the first problem I see as newcomer. First there was only the "unicorn" app where components could live. Then, thanks to #15  there is a setting where you can define apps with components to look for.
But in a large scale application this is bad(TM).
Why can't unicorn just look at all installed apps and see if there is a components folder, hence load components from there? And any app could provide templates as well.
Is there any technical problem that prevents such a feature?
Background: I wrote a Django plugin system (GDAPS) which aims for installing apps dynamically (more or less) - but my main application does not know which plugin app is going to be installed. So adding an app to an array in settings.py does not work here. and doesn't have to.
I use this feature in GDAPS as well. GDAPS looks for the urlsmodule of each installed app and loads them dynamically, assembling all urlpatterns together. Unicorn could do that with components.

Definitely not a technical problem to implement this. It was mostly a performance optimization (to skip checking every installed app) and to provide an "easy" on-ramp with the startunicorn management command (by defaulting to a unicorn app).
Would this work for your use-case?
UNICORN = {
    ...
    "APPS": INSTALLED_APPS,
    ...

"APPS": INSTALLED_APPS, - this would definitely do it. Performance shouldn't be an issue, as it is only done once at server start. Yes, the server starts a little bit slower, but who cares about that...?
I would make this even more easy: Why not per default search in apps' components directories instead of a "unicorn" app?
This would remove the awkwardly needed  "unicorn" app and the adding of "unicorn" to INSTALLED_APPS. Many Django additions do that, even Django itself has the convention: search in the models module for models. Why not search in the components module for components? Would be even more django'ish :-)

... and to provide an "easy" on-ramp with the startunicorn management command (by defaulting to a unicorn app).
Oh, and this could also be dont with a mgmt cmd. Sockpuppet does this, you just need to add a parameter to the command which specifies the app where the scaffolding has to be placed. I do that in GDAPS mgmt cmd too.

My performance optimization comment was because (currently) all apps are checked the first time a component is loaded (it happens here: https://github.com/adamghill/django-unicorn/blob/c836bf18b6883c3bb7fbbb05e88ad9114cb311bf/django_unicorn/components/unicorn_view.py#L51). I have some caching around that section of code, but I could probably offset any potential performance hit by being a little smarter about how this works.

This would remove the awkwardly needed "unicorn" app and the adding of "unicorn" to INSTALLED_APPS
I completely agree with this. The unicorn app install step is a little confusing in the documentation. "Why do I need django_unicorn and unicorn?!"

I like this approach and it seems reasonably straight-forward to implement, and potentially less confusing for end-users. I'll try to whip up a PR when I get a chance and will attach it to this issue.

I started looking into how to get this working. Loading the components from INSTALLED_APPS is working in the attached PR.
I think I'm going to still default templates to a "unicorn" folder underneath the application's template folder. I think it makes sense to keep Unicorn templates separate by default. If someone really wants them somewhere else you can manually set template_name in the component view.

This is released in https://github.com/adamghill/django-unicorn/releases/tag/0.30.0.


[Bug]: Log does not seem to write to stdout

Describe the issue
I can not see infos/warnings, no matter what I set --minloglevel to. Setting --stderrthreshold=0 works as expected.
Steps to reproduce the problem
Code:
int main(int argc, char** argv) {
  absl::SetProgramUsageMessage("Echo server");
  absl::ParseCommandLine(argc, argv);
  absl::InitializeLog();
  std::cout << "Hello, world!" << std::endl;
  LOG(INFO) << "Info";
  LOG(WARNING) << "Warning";
  LOG(ERROR) << "Error";
  return 0;
}

Bazel build:
cc_binary(
  name = "echo_server",
  srcs = [
    "echo_server.cc",
  ],
  deps = [
    "@com_google_absl//absl/flags:parse",
    "@com_google_absl//absl/log",
    "@com_google_absl//absl/log:flags",
    "@com_google_absl//absl/log:initialize",
  ],
)

Command line:
bazel run //src/tools:echo_server -- --minloglevel=0
What version of Abseil are you using?
abseil-cpp-20230802.0
What operating system and version are you using?
Ubuntu 22.04.3 LTS x86/64
What compiler and version are you using?
gcc version 11.4.0 (Ubuntu 11.4.0-1ubuntu1~22.04)
What build system are you using?
Bazel 6.3.2
Additional context
No response

I'm not sure what you think the expected behavior is here. Logging writes to stderr (not stdout) when --stderrthreshold is set to the appropriate level, and you say that is working.
--minloglevel controls whether the logging infrastructure runs at all for a particular logging level.
If you want logs to go somewhere besides stderr, you need to use absl::AddLogSink().


Arguments to webdriver are not being pass to the selenium server

Hi there, it seems that the args pass to the webdriver are not being used later on.
For example I tried to use --seleniumPort 4445 and it seems that it works at the beginning, but at the end the selenium server uses 4444
this works:
./node_modules/webdriver-manager/bin/webdriver-manager start --seleniumPort 4445
Configuration:
webDriverStart: {
    args: ['--seleniumPort 4445']
},

Here are the logs:
[18:36:52] gulp-angular-protractor - The plugin is retrieved and will start soon
[18:36:52] gulp-angular-protractor - We will try to start and stop automatically the WebDriver server
[18:36:52] gulp-angular-protractor - The selenium address is: http://localhost:4445/wd/hub
[18:36:52] gulp-angular-protractor - The selenium address used is: http://localhost:4445/wd/hub
[18:36:52] gulp-angular-protractor - The selenium shutdown address used is: http://localhost:4445/selenium-server/driver/?cmd=shutDownSeleniumServer
[18:36:52] gulp-angular-protractor - We have the config file to the following path: protractor.conf.js
[18:36:52] gulp-angular-protractor - Webdriver standalone will be updated
webdriver-manager: using global installed version 10.2.8
[18:36:53] I/file_manager - creating folder node_modules\webdriver-manager\selenium
[18:36:53] I/downloader - selenium standalone: downloading version 2.53.1
[18:36:53] I/downloader - curl -o node_modules\webdriver-manager\selenium/selenium-server-standalone-2.53.1.jar https://selenium-release.storage.googleapis.com/2.53/selenium-server-standalone-2.53.1.jar
[18:36:53] I/downloader - chromedriver: downloading version 2.25
[18:36:53] I/downloader - curl -o node_modules\webdriver-manager\selenium/chromedriver_2.25win32.zip https://chromedriver.storage.googleapis.com/2.25/chromedriver_win32.zip
[18:36:53] I/downloader - geckodriver: downloading version v0.11.1
[18:36:53] I/downloader - curl -o node_modules\webdriver-manager\selenium/geckodriver-v0.11.1-win64.zip https://github.com/mozilla/geckodriver/releases/download/v0.11.1/geckodriver-v0.11.1-win64.zip
[18:36:58] I/update - chromedriver: unzipping node_modules\webdriver-manager\selenium\chromedriver_2.25win32.zip
[18:37:03] I/update - geckodriver: unzipping node_modules\webdriver-manager\selenium\geckodriver-v0.11.1-win64.zip
[18:37:12] gulp-angular-protractor - Webdriver standalone is updated
[18:37:12] gulp-angular-protractor - Webdriver standalone server will be started
[18:37:13] webdriver-manager: using global installed version 10.2.8

[18:37:13] [18:37:13] I/start - java -Dwebdriver.chrome.driver=node_modules\webdriver-manager\selenium\chromedriver_2.25.exe -Dwebdriver.gecko.driver=node_modules\webdriver-manager\selenium\geckodriver-v0.11.1.exe -jar node_modules\webdriver-manager\selenium\selenium-server-standalone-2.53.1.jar -port 4444

[18:37:13] [18:37:13] I/start - seleniumProcess.pid: 17964

[18:37:14] 18:37:14.433 INFO - Launching a standalone Selenium Server

[18:37:14] 18:37:14.856 INFO - Java: Oracle Corporation 25.121-b13

[18:37:14] 18:37:14.856 INFO - OS: Windows 10 10.0 x86

[18:37:14] 18:37:14.864 INFO - v2.53.1, with Core v2.53.1. Built from revision a36b8b1

[18:37:14] 18:37:14.937 INFO - Driver class not found: com.opera.core.systems.OperaDriver

[18:37:14] 18:37:14.938 INFO - Driver provider com.opera.core.systems.OperaDriver is not registered

[18:37:14] 18:37:14.944 INFO - Driver provider org.openqa.selenium.safari.SafariDriver registration is skipped:
registration capabilities Capabilities [{browserName=safari, version=, platform=MAC}] does not match the current platform WIN10

[18:37:14] 18:37:14.945 INFO - Driver class not found: org.openqa.selenium.htmlunit.HtmlUnitDriver
18:37:14.945 INFO - Driver provider org.openqa.selenium.htmlunit.HtmlUnitDriver is not registered

[18:37:15] 18:37:15.606 INFO - RemoteWebDriver instances should connect to: http://<IP_ADDRESS>:4444/wd/hub

[18:37:15] 18:37:15.606 INFO - Selenium Server is up and running

thanks!

Hi
Could you tell me on which version are you using the plugin ?
Can you give me a project as attachment to check it please ?
Many thanks
Regards

I'm using the latest protractor 5.1.1. Either way I checked with the webdriver version 10, the one your plugin installs, and with the command line works.
https://github.com/mribichich/angular-protractor

Hummm can you try please:
webDriverStart: {
    args: ['--seleniumPort', '4445']
},

Instead
webDriverStart: {
    args: ['--seleniumPort 4445']
},

Hi
Have you try to change the configuration like the example below please ?
Many thanks

Sorry I've been out of town. I'll try them later and let you know
Thanks!
On Mon, Feb 20, 2017, 16:49 Roche julien<EMAIL_ADDRESS>wrote:

Closed #35 https://github.com/rochejul/gulp-angular-protractor/issues/35
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rochejul/gulp-angular-protractor/issues/35#event-969717338,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFe3SlMs0qyiH4LSW69nCtyESs7GoBA5ks5ree5JgaJpZM4L8uBR
.


Add FeatherBB Installers

Write test, please. Example:
https://github.com/composer/installers/blob/master/tests/Composer/Installers/Test/InstallerTest.php#L87
https://github.com/composer/installers/blob/master/tests/Composer/Installers/Test/InstallerTest.php#L205

@beaver-dev will finish PR?

Yep, sorry for the late answer but for now you can close the PR, we found another way to handle plugins and themes. Maybe we'll reopen one if the project evolves, or use our fork of the repo, we'll see that later!
Thanks for your time anyway.


FIX #2093 warn in case we think VT-X is not enabled.

Signed-off-by: David Gageot<EMAIL_ADDRESS>

LGTM

A few nitpicks but looks pretty good.

@nathanleclaire Thanks for your feedback. Ready to merge.

LGTM


udp is not a valid protocol for HttpAdapter

tl;dr HttpAdapter does not actually send API requests via UDP, so it should not accept udp:// URIs.
udp:// URIs are only valid as neighbor URIs (i.e., as inputs for addNeighbors and removeNeighbors), because the node uses UDP for communication with other nodes.
The API, however, is an HTTP service; it doesn't use UDP, so HttpAdapter should not accept udp:// URIs.

Scheduled for release: 1.1.0


Delayed job not working in background while in production

When we start delayed job in background with below commant, it shows that delayed_job process has started.
RAILS_ENV=production bin/delayed_job start
If we see the logs file log/delayed_job.log it also says Starting job worker and doesn;t executes the job.
But when I try to execute delayed job in foregqound with RAILS_ENV=production rake jobs:work then it properly executes the all pending jobs.
Is anyone familiar with this kind of scenario?
Note: When I ran same command as RAILS_ENV=production bin/delayed_job start --exit-on-complete it says No more jobs available. Exiting
I am using,
Ruby 2.3.1
Rails 4.2.7
delayed_job (4.1.5)
activesupport (>= 3.0, < 5.3)
delayed_job_active_record (4.1.3)
activerecord (>= 3.0, < 5.3)
delayed_job (>= 3.0, < 5)

@amit-mhetre Did you solve this issue?

@amit-mhetre Did you solve this issue?

i am also facing the same issue. Did you solved it ?

Closing stale


ISPN-5839 Fix unfinished transaction log message

https://issues.jboss.org/browse/ISPN-5839

Stream.toString does not go and convert the Streams elements into a
String. Instead, to convert a Stream to String, a String joiner needs
to be used.

Do not integrate this yet, something else found...

Actually, this particular issue fixed. So all good :)

@danberindei Fixed

@galderz Can't we just use https://docs.oracle.com/javase/8/docs/api/java/util/stream/Collectors.html#joining-java.lang.CharSequence-java.lang.CharSequence-java.lang.CharSequence- with the bracket before and after?  This way we don't have to create another collection just to make the string.

@wburns Good idea, fixed it accordingly.

Integrated, thanks Galder!


[bug] [tauri-cli] --config argument stopped accepting files

Describe the bug
With 2.0.0-beta the tauri build --config argument won't accept files anymore.
Reproduction
Works: npm run tauri -- build -c "{\"version\": \"1.2.3\"}"
Won't work: npm run tauri -- build -c ./dev.conf.json
The latter will result in a failed to parse config to merge: expected value at line 1 column 1 regardless of the content of the file (full config or partial config to be merged). Up until late alpha builds, both commands worked.
Expected behavior
No response
Full tauri info output
[✔] Environment
    - OS: Mac OS 12.7.3 X64
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.74.1 (a28077b28 2023-12-04)
    ✔ cargo: 1.74.1 (ecb9851af 2023-10-18)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-x86_64-apple-darwin (default)
    - node: 18.8.0
    - npm: 8.18.0

[-] Packages
    - tauri [RUST]: 2.0.0-beta.2
    - tauri-build [RUST]: 2.0.0-beta.1
    - wry [RUST]: 0.35.2
    - tao [RUST]: 0.25.0
    - @tauri-apps/api : not installed!
    - @tauri-apps/cli [NPM]: 2.0.0-beta.1

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../frontend/prod
    - bundler: Webpack

Stack trace
No response
Additional context
zsh 5.8.1 (x86_64-apple-darwin21.0)
> cat dev.conf.json
{
  "build": {
    "frontendDist": "../frontend/dev"
  }
}

Closing my ticket. Fixed with tauri-cli beta.2: https://www.github.com/tauri-apps/tauri/commit/fb0d997117367e3387896bcd0fba004579475c40


User may add second user with same email (not unique)

This causes the user then not to be able to log in (since login is using email address in place of username)...
This seems like something django should have thought about in their backend for using email in place of the username field.
Fix: We need to subclass AbstractUser anyway later for OAuth so we should abstract it with uniqueness checking for email addresses.
Doesn't really need to be handled for MVP

What is the plan for the login?
oauth? user + email etc?

user & email, but we have to limit to one email, because if you add two with the same email it won't let you log in with either.


JsonFactory.toJson doesn't work with @JsonProperty

Hi,
Deserialization of beans with @JsonProperty annotation works but for deserialization behavior is very strange.
In the test below serialization ouput from toJson is : {"name":"name","host":"localhost","host":"localhost","name":"name"}
Which is incorrect. Fields are not named according to @JsonProperty and are duplicated, it should be :
{"Name":"name","Address":"localhost"}
Thx
public class Registration {
    @JsonProperty("Name")
    private String name;
    @JsonProperty("Address")
    private String host;
}

public class TestSerializeRegistration {
    @Test
    public void test() {
        Registration registration = new Registration();
        registration.setName("name");
        registration.setHost("localhost");

        ObjectMapper mapper = JsonFactory.create();
        String jsonSource = "{\"Name\":\"name\",\"Address\":\"localhost\"}";
        Registration unserializedRegistration = mapper.fromJson(jsonSource, Registration.class);

        String json = mapper.toJson(unserializedRegistration);
        Assert.assertEquals(json, jsonSource);
    }
}

Couple of things... QBit uses a forked version of Boon.. https://github.com/advantageous/boon I don't think JsonFactory supports annotations by default. You would have to build a JsonMapper that supported it. I will move this issue to https://github.com/advantageous/boon and look at it.


⚠ TorrentLeech has degraded performance

In 2ab972f, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 3077 ms

Resolved: TorrentLeech performance has improved in 1c19ab1 after 5 minutes.


Add accuracy metric to 001a_nn_basics notebook.

@sjaz24 could you please be so kind as to sign the CLA? Sorry we merged your PR without checking that! https://www.clahub.com/agreements/fastai/fastai_v1


Save shortcut

Not sure if this has already been submitted. Adding a CTRL+S (CMD+S for Mac) shortcut to save would be a good idea, I'm not sure if this already exists in the desktop version, but it doesn't in the browser version.

Hi !
Issue already exists and is almost implemented : #277 :)
I would be interested to know if you are usually saving to a piskelapp account or to a different target.
My current implementation automatically saves to the gallery if possible.
Otherwise it falls back to a local browser save. I'm not sure if users would prefer local browser save vs file (.piskel) download.
Any input ?

I usually save to the gallery through the side menu. CTRL+S opens the save dialogue and tries to save the page as an MHTML file, and CTRL+SHIFT+S does nothing as far as I'm aware

It is "almost" implemented, and not released yet.
For CTRL SHIFT S it would be cool if it could save as a New Piskel directly but that will be more complex.
Thanks for the info, I will close this issue now. You can comment on #277 if needed !


Use different layout for Account/API keys and preferences

This PR adds a new 'nonav' layout and uses this for accounts/api keys and preferences.
(note this can also be used for the about page when added)
These pages currently show the cluster explorer side nav which is not really applicable - and there is a bug when going to cluster tools from this.

nonav layout looks effectively the same as what the existing plain one would look like now, if it had been been updated (e.g. simple=true in the header).  Don't think we need two of these.
Remembering one page back doesn't really work.  If you go more than one page deep (Prefs -> Account or vice versa, or Account -> Create API Key) then the Back button turns into an infinite loop of the last 2 pages you were in.

Thanks @vincent99  - let me have a think - I left plain there - I'll dig into where its used and see if we can combine nonav into plain.


UI: Add deleting state for backup volumes and backup

The backup volume and backup has a new field named deleting to indicate it's deleting

Cancel the work. Backend cannot pass integration test.


Add parquet datasource and datasink connectors

Story
Define these connectors in external. These connectors should be able to read from local or hdfs (any valid uri)

migrated to https://issues.apache.org/jira/browse/GEARPUMP-95


updated url

Updated the URL of my personal web page. This PR is ready to be pulled.

thanks @tscheypidi!


Is there syntax to custom validate

Example:
validates_datetime :public_date, :on_or_after => :release_date

Error is not good enough for client understand :
Public date must be on or after 2015-07-01 09:33:00

I recommend it should be :)
Public date must be on or after 2015-07-01 09:33:00 (release date)

You can override any validation message in the locale file per attribute or with the message option key
validates_datetime :public_date, :on_or_after => :release_date, :on_or_after_message => "must be on or after %{restriction} (release date)"
We wouldn't use the symbol as anything such as a name in the message. The symbol could be any method which may not semantically represent the value to the user.


Gradient visualization

@WeiChengTseng
Dear Wei-Cheng,
Is there any way to visualize the gradient change? I want to visualize them to further illustrate the superiority.
Best wishes,
Jianhong Cheng

I think you can directly use the following code to see the scale of the gradient.
for p in model.parameters():
    param_norm = p.grad.data.norm(2)
    total_norm += param_norm.item() ** 2
total_norm = total_norm ** (1. / 2)

I close this issue since there is no follow-up information.

Dear Wei-Cheng,
Thanks for your interesting work. For example , i have two losses. How should i draw picture like Figure 1 in the paper?
Best reagrds，
Bizhe


Faceting Annotation Bugs

[x] When annotation is defined, changing the facet filters through the facet APIs, make the actual annotation definition in ermrestjs to change. This is most probably happening because of shallow copying an object.
[x] When annotation has preselected values, you cannot clear those filters.

A fixed has pushed to master for this bug.


Create Open_Programmable_Infrastructure_Funding_Charter_final.md

Creating a draft for review as proper markdown etc.
Would like to wait until LF has chimed in on non-content text changes
before merging
Signed-off-by: Paul Pindell<EMAIL_ADDRESS>

@opiproject/opi-maintainers Sridhar indicated in Slack that the formatting differences are ok when using GH. I've set the to ready for review.

@glimchb @mestery Boris has reviewed this. Kyle can you review and let me know if this is merge worthy.

Ping @opiproject/opi-maintainers Can we get a second maintainer approval and have this merged. it is not available anywhere publicly at this point.
Paul

Ping @opiproject/opi-maintainers Can we get a second maintainer approval and have this merged. it is not available anywhere publicly at this point.
Paul

The PDF has been available for months here: https://github.com/opiproject/opi/blob/main/Open_Programmable_Infrastructure_Technical_Charter_Final-06-9-2022.pdf


Repair broken version fetching

No updates in 5 months, and it looks related to this:
Run export TOKEN=$(curl -fsSL "https://auth.docker.io/token?service=registry.docker.io&scope=repository:library/caddy:pull" | jq --raw-output '.token')
jq: error (at <stdin>:1): Cannot iterate over null (null)

That also explains why the action was disabled ...

Weirdly, it works on my local machine
url -fsSL "https://auth.docker.io/token?service=registry.docker.io&scope=repository:library/caddy:pull" | jq --raw-output '.token'
eyJhbGciOiJSUzI1...

 Looks like it can't handle the multi-architecture manifest.

Fixed in https://github.com/ShaddyDC/caddy-cloudflare/commit/8fbd616c1e180fbe883583e94309f9cca6d37b26


Implicitly consider an extensionless file in "includes" to be a recursive directory glob

Fixes #10578
There is a lot of refactoring in this PR but the actual changes just have to do with uses of the isImpicitGlob function. If this always returns false then behavior will be the same as before. If wanted I could break this into 2 PRs, one which refactors and one which adds isImplicitGlob.
It would also be a good idea to someday rewrite this functionality into a) parsing globs, then b) doing things with parsed globs. Then we would pass around strings less often.

I tested this on the command-line with a real project and it worked. I'd still like to get @rbuckton's review before merging.


Frontend code quality improvement: separate test into test folder

It's better to put all tests under the "tests" folder.
For unit test, we could use Vitest and Vue test util, and test scenarios could target on:

receive props correctly
use computed properties correctly if has
emit events correctly if has

It's better to put all tests under the "tests" folder.
For unit test, we could use Vitest and Vue test util, and test scenarios could target on:
receive props correctly
use computed properties correctly if has
emit events correctly if has


is there an import/export function

would be nice to be able to sync shopping list, eg via import and export of shared list in txt format. Any way to implement this?Thanks

Hello archlute,
At the moment you can at least export in txt format (will not take attached pictures into account) using the share function (via email, sms, etc.)

Duplicate of #8

This I saw, but how to reimport en edited shoppinglist as txt file without compromising the database of purchased items? The exported file does not show any purchased items.
How and where in the root folder system are these data stored?  E.g. it could be synced via syncthing at least for rooted devices...
thanks again

As for where the data is stored, I read the app is using of an ORMlite database. So I guess the data is stored in there, namely in ShippingList.db (the filename might help you to find its full path).
I know nothing about ORMlite but I don't believe this can be edited via a text editor.
Now if your issue refers to backing up & syncing your lists (nevermind the file format), then this relates to the already existing topic I linked to in my previous answer.
However if your main concern is how to edit a list outside of the app (via a text editor using a keyboard for productivity reasons for instance), then you may want to dig in ORMlite documentation and/or (just an idea) experiment some hackish conversion from .txt or .csv to .db with a software such as SQLite Browser via a computer on a copy of your .db file...
Or maybe someone else could help you better...

The full file name would be /data/data/privacyfriendlyshoppinglist.secuso.org.privacyfriendlyshoppinglist/databases/ShippingList.db but it is only accessible with root permissions because it is saved in the internal storage of the application.
We are currently working on this feature, but because we are only a small team working on many apps at the same time, I can not give a timeline as to when this feature will be finished.

Hi, any news on that feature? I have not seen it in any of the release notes, but I might just have missed its implementation.


Wrap Exception messages with translation function __()

__() for better localization support of Exceptions. Now you have to overwrite all those Exceptions, which is unnecessary when this PR is merged.

None of this is supposed to be displayed to the user.


Adds missing live implementation of monitoringSignificantLocationChanges

Hey @myurieff, thanks for the PR!
It looks like we've already got support for startMonitoringSignificantLocationChanges and stopMonitoringSignificantLocationChanges but it just isn't exposed to macOS:
https://github.com/pointfreeco/composable-core-location/blob/ed26538db41294e56a76ddd869c8143d3a3f3bab/Sources/ComposableCoreLocation/Live.swift#L130-L134
https://github.com/pointfreeco/composable-core-location/blob/ed26538db41294e56a76ddd869c8143d3a3f3bab/Sources/ComposableCoreLocation/Live.swift#L158-L162
Is the main purpose of your change to expose it to macOS? If so I think you could update this PR to just add a || os(macOS) to the lines above and it should be work.

Hm, I was using the live client on iOS and it crashed when invoking startMonitoringSignificant... with the unimplemented message 🤔 Perhaps there's something else to it.

@mbrandonw Based on the snippet you linked, you do indeed have this, but it's on a branch you haven't released (unless I'm missing something?).
@myurieff Did you find a work-around? Appears they aren't releasing anything for a while (quite a shame, considering the one and only thing I need is already done in there!), and absent this one capability sadly I'll have to abandon this otherwise handy little library from my project.


Implement support for WebAuthn

This PR removes U2F as an option for two-factor authentication, and replaces it with it's successor WebAuthn.
WebAuthn allows for multiple tokens to be registered, and for those tokens to be either "cross-platform" (ie: portable between devices such as a YubiKey), or "platform-specific" (ie: locked to a specific device, such as using Android's lockscreen or Windows Hello).
This is really quite a complex one to digest, as it drops between JS and PHP on a fairly regular basis. I'll try and write up some documentation here regarding the implementation to better explain it, but as this is (by necessity!) touching authentication code, I'd appreciate a goooooood long look at this from someone else. If you have devices available which support WebAuthn (modern Android, modern Windows 10, or a U2F token such as a YubiKey or probably some iDevice), it's probably a very good idea to test this too - I've tested this on Chrome/Linux, Firefox/Linux, Chrome/Win10, and Chrome/Android so far. Firefox/Android is known to not support this yet

Codecov Report

Merging #666 (d91fdcb) into master (5e68295) will decrease coverage by 0.07%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             master    #666      +/-   ##
===========================================
- Coverage      5.06%   4.99%   -0.08%
- Complexity     2349    2387      +38
===========================================
  Files           172     173       +1
  Lines         11641   11817     +176
===========================================
  Hits            590     590
- Misses        11051   11227     +176

Impacted Files
Coverage Δ
Complexity Δ

includes/Fragments/TemplateOutput.php
0.00% <ø> (ø)
3.00 <0.00> (-1.00)

...s/Pages/UserAuth/Login/LoginCredentialPageBase.php
0.00% <0.00%> (ø)
42.00 <0.00> (ø)

...ncludes/Pages/UserAuth/Login/PagePasswordLogin.php
0.00% <0.00%> (ø)
6.00 <0.00> (ø)

...ncludes/Pages/UserAuth/Login/PageWebAuthnLogin.php
0.00% <0.00%> (ø)
5.00 <5.00> (?)

...des/Pages/UserAuth/MultiFactor/PageMultiFactor.php
0.00% <0.00%> (ø)
42.00 <17.00> (+20.00)

includes/Router/RequestRouter.php
98.00% <ø> (ø)
15.00 <0.00> (ø)

includes/Security/AuthenticationManager.php
0.00% <0.00%> (ø)
6.00 <0.00> (ø)

...oviders/WebAuthn/PublicKeyCredentialUserEntity.php
0.00% <0.00%> (ø)
2.00 <2.00> (?)

...CredentialProviders/WebAuthnCredentialProvider.php
0.00% <0.00%> (ø)
30.00 <30.00> (?)

includes/Security/RoleConfiguration.php
84.84% <ø> (ø)
14.00 <0.00> (ø)

... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5e68295...d91fdcb. Read the comment docs.

Codecov Report

Merging #666 (d91fdcb) into master (5e68295) will decrease coverage by 0.07%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             master    #666      +/-   ##
===========================================
- Coverage      5.06%   4.99%   -0.08%
- Complexity     2349    2387      +38
===========================================
  Files           172     173       +1
  Lines         11641   11817     +176
===========================================
  Hits            590     590
- Misses        11051   11227     +176

Impacted Files
Coverage Δ
Complexity Δ

includes/Fragments/TemplateOutput.php
0.00% <ø> (ø)
3.00 <0.00> (-1.00)

...s/Pages/UserAuth/Login/LoginCredentialPageBase.php
0.00% <0.00%> (ø)
42.00 <0.00> (ø)

...ncludes/Pages/UserAuth/Login/PagePasswordLogin.php
0.00% <0.00%> (ø)
6.00 <0.00> (ø)

...ncludes/Pages/UserAuth/Login/PageWebAuthnLogin.php
0.00% <0.00%> (ø)
5.00 <5.00> (?)

...des/Pages/UserAuth/MultiFactor/PageMultiFactor.php
0.00% <0.00%> (ø)
42.00 <17.00> (+20.00)

includes/Router/RequestRouter.php
98.00% <ø> (ø)
15.00 <0.00> (ø)

includes/Security/AuthenticationManager.php
0.00% <0.00%> (ø)
6.00 <0.00> (ø)

...oviders/WebAuthn/PublicKeyCredentialUserEntity.php
0.00% <0.00%> (ø)
2.00 <2.00> (?)

...CredentialProviders/WebAuthnCredentialProvider.php
0.00% <0.00%> (ø)
30.00 <30.00> (?)

includes/Security/RoleConfiguration.php
84.84% <ø> (ø)
14.00 <0.00> (ø)

... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5e68295...d91fdcb. Read the comment docs.

Draftifying as (realisticly) this needs work to bring it back up to where the master branch is.

Given a) the complexity of this PR, b) the apparent general reluctance to review it or keep it up-to-date, and c) considerations on changes to our authentication strategy pending the WMCS ToS updates, I'm going to drop this entirely for the time being. If we want to revisit WebAuthn at a later date (and if it's appropriate/reasonable to do so), at least this PR might act as good guidance.


null check for generated types not properly working

Having this datamodel, Order products should never be null
type ProductOrder {
  id: ID! @id
  product: Product! @relation(link: INLINE)
  order: Order! @relation(link: INLINE)
  price: Float! @default(value: 0)
  quantity: Float! @default(value: 0)
}
type Order {
  id: Int! @id
  createdAt: DateTime! @createdAt
  updatedAt: DateTime! @updatedAt
  products: [ProductOrder!]! @relation(onDelete: CASCADE)
  total: Float
  user: User!
}

But my generated schema looks like this
type Order {
  createdAt: DateTime!
  id: Int!
  products(after: String, before: String, first: Int, last: Int, orderBy: ProductOrderOrderByInput, skip: Int, where: ProductOrderWhereInput): [ProductOrder!]
  total: Float
  updatedAt: DateTime!
  user: User!
}

The ProductOrder is generated as [ProductOrder!] instead of [ProductOrder!]!

Yes I'm using prisma 1

@tafelito no need to create, I moved.


AFNetworking with REST api

How can I use AFNetworking with this kind of API.
http://localhost:4567/things/3

3 is the number of things id, I would like to use the AFHTTPSessionManager and the GET method, pass nil to the parameter and things/3 to its method, but is there a neat way that I do not needs to initialize a string such as things/3 each time I have to get json from server.

I'm not sure I follow the question. If you are using a REST API, you should not have to construct this "string", instead you should be getting this from your REST API. REST is about following hyperlinks to resources and not knowing about implementation details such as IDs and constructing URIs.
This is also not a bug and actually a usage question and doesn't belong in the AFNetworking issue tracker. You should take discussions like these to an appropriate forum such as Stack Overflow.

Thanks, @kylef.


Task 3 Wazuh task completed

Added relevant files for Wazuh deployment and created task instructions for students.

Some typo fixes and formatting, merge when you want!


Create one-liner to run Markcop

Makes further progress on hackedu/meta#163.

OUR TESTS NEED TESTS!


Golint comment

Coverage remained the same at 100.0% when pulling 7b9fda35d7a143fee18410231457cb4e7f82ed5f on golint_comment into 1c9fa8bc48f3be6d3c17c742294730420e05c54f on master.

Coverage remained the same at 100.0% when pulling 7b9fda35d7a143fee18410231457cb4e7f82ed5f on golint_comment into 1c9fa8bc48f3be6d3c17c742294730420e05c54f on master.

Coverage remained the same at 100.0% when pulling ba74b66cd58e73307b74d2556288ef4c001efe36 on golint_comment into 1c9fa8bc48f3be6d3c17c742294730420e05c54f on master.

Coverage remained the same at 100.0% when pulling ba74b66cd58e73307b74d2556288ef4c001efe36 on golint_comment into 1c9fa8bc48f3be6d3c17c742294730420e05c54f on master.


Fix "missing dependency resolver" in Windows

Fixes #3794

As far as I remember the file is closed when the ResoucreInteractiveLoader
is deleted..
On Sun, Feb 21, 2016 at 6:54 PM, eska014<EMAIL_ADDRESS>wrote:

Fixes #3794 https://github.com/godotengine/godot/issues/3794
You can view, comment on, or merge this pull request online at:
https://github.com/godotengine/godot/pull/3795
Commit Summary

Fix missing dependency resolver in Windows

File Changes

M core/io/resource_format_xml.cpp
https://github.com/godotengine/godot/pull/3795/files#diff-0 (1)
M scene/resources/scene_format_text.cpp
https://github.com/godotengine/godot/pull/3795/files#diff-1 (1)

Patch Links:

https://github.com/godotengine/godot/pull/3795.patch
https://github.com/godotengine/godot/pull/3795.diff

—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/pull/3795.

ResourceInteractiveLoaderText::~ResourceInteractiveLoaderText() {
memdelete(f);

}
so something strange is going on. I assume you are on Windows, so you can't
change a file that is opened
On Sun, Feb 21, 2016 at 6:59 PM, Juan Linietsky<EMAIL_ADDRESS>wrote:

As far as I remember the file is closed when the ResoucreInteractiveLoader
is deleted..
On Sun, Feb 21, 2016 at 6:54 PM, eska014<EMAIL_ADDRESS>wrote:

Fixes #3794 https://github.com/godotengine/godot/issues/3794
You can view, comment on, or merge this pull request online at:
https://github.com/godotengine/godot/pull/3795
Commit Summary

Fix missing dependency resolver in Windows

File Changes

M core/io/resource_format_xml.cpp
https://github.com/godotengine/godot/pull/3795/files#diff-0 (1)
M scene/resources/scene_format_text.cpp
https://github.com/godotengine/godot/pull/3795/files#diff-1 (1)

Patch Links:

https://github.com/godotengine/godot/pull/3795.patch
https://github.com/godotengine/godot/pull/3795.diff

—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/pull/3795.

I think only binary scenes open an InteractiveLoader to rename dependencies, tscn and xml just use FileAccess and modify the raw text

The InteractiveLoader that manages f is never deleted (within the function), since it's the loader that the function is called on.
In the binary loader, two loaders are created for the broken file and the fixed file, both of which go out of scope before file renaming happens.
But non-binary loaders don't create new loaders at all, they just open per FileAccess and modify the raw text.
FileAccess* f is a member of the loader that rename_dependencies is being called on, so it's never deleted before the function is done. It needs to be closed, or Windows will consider the file to be read-only

I would merge this before 2.0 if it's fine with you @reduz.

ah I misunderstood your fix, sorry, this is perfect


Create Theme Reader

This PR intends to create a new API to use the styled components yoga theme.
// Before:
import styled from 'styled-components';

const MyCoolComponent = styled.div`
  padding: ${(props) => props.theme.yoga.spacing.large}px;
  background-color: ${(props) => props.theme.yoga.colors.primary};
`;

// After:
import styled from 'styled-components';
import { theme } from '@gympass/yoga';

const { spacing, colors } = theme;

const MyCoolComponent = styled.div`
  padding: ${spacing.large}px;
  background-color: ${colors.primary};
`;

Important:
GoogleChrome/proxy-polyfill was added to make it compatible with IE.

The cool next step would be to create a custom babel plugin to replace the values.

Hi @tcK1, I'm extremely grateful for your contribution! Thanks! 🤩
I'd like to open a discussion here about this enhancement so I can hear you and the team's opinion, about it.
Although I did find your Proxy solution very elegant used in order to simplify the code writing on a styled-component I'm a lil bit concerned about its use on a complex component, thus your solution executes a callback on each line that needs a prop from theme
Take this Button for instance, each line use a property from theme but it only calls one callback, instead of 27 callbacks using the themeReader
So, what everybody thinks? how can we solve that?
PS: Nice job on the GoogleChrome/proxy-polyfill

Hey @alan-oliv! Calling a function multiple times could be troublesome. One thing to remember is that this approach does not invalidate the other.
I created this "helper" with the intuit to "help" developers use the yoga theme inside their on components, which is much simpler then the components inside our system. Here's an example.
If we wanted to use the helper on our components, we also could do that with a single function call. Here's how:
Before:
const Button = styled.button`
  ${({
    full,
    small,
    inverted,
    theme: {
      yoga: {
        baseFont,
        components: { button },
      },
    },
  }) => `
    /* css goes here... also, this indentation is so weird haha. */
    `}
`;

After:
const Button = styled.button`
  ${props => {
    const { full, small, inverted } = props;

    const {
      baseFont,
      components: { button },
    } = theme(props); // The single function call.

    return css`
      /* css goes here... */
    `;
  }
`;

One thing that could be possible, is to create a "cache" for the proxy. I will try to come up with something.

I've committed a change that adds cache to the Proxy generator. Here are the results:
The first spec was:
  const obj = {
      theme: {
        yoga: {
          breakpoints: {
            xs: {
              margin: 'value',
            },
          },
        },
      },
    };

    theme(obj);
    theme.breakpoints(obj);
    theme.breakpoints.xs(obj);
    theme.breakpoints.xs.margin(obj);

Before:
  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints', 'xs' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints', 'xs' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints', 'xs', 'margin' ]

After:
  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints', 'xs' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints', 'xs', 'margin' ]

The second result is the behavior that we should expect.
The second spec was:
  const obj = {
      theme: {
        yoga: {
          breakpoints: {
            xs: {
              margin: 'value',
            },
          },
        },
      },
    };

    theme.breakpoints.xs.margin(obj);
    theme.breakpoints.xs.margin(obj);
    theme.breakpoints.xs.margin(obj);
    theme.breakpoints.xs.margin(obj);
    theme.breakpoints.xs.margin(obj);
    theme.breakpoints.xs.margin(obj);
    theme.breakpoints.xs.margin(obj);
    theme.breakpoints.xs.margin(obj);
    theme.breakpoints.xs.margin(obj);
    theme.breakpoints.xs.margin(obj);

Before:
  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints', 'xs' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints', 'xs', 'margin' ]

 # Repeat 10 times.

After:
  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints', 'xs' ]

  console.log packages/yoga/src/Theme/helpers/themeReader/index.js:17
    Created Proxy for [ 'theme', 'yoga', 'breakpoints', 'xs', 'margin' ]

  # Only create the Proxy a single time.

Since the theme Proxy is application wide, if multiple components use the same values (theme.breakpoints.xs.margin in this example), it will only create the Proxy a single time. This is also valid if used inside the yoga components.
I hope that this can add some ideas to the discussion.

Thanks for your suggestion, @tcK1, and for the clever implementation as well.
Taking your example, we could simplify it by doing this:
export const LottieWrapper = styled.div`
  width: 208px;
  height: 162px;
- margin:
-   0
-   0
-   ${({ theme: { yoga } }) => yoga.spacing.small}px
-   0;
- @media (min-width: 1024px) {
-   width: 296px;
-   height: 231px;
-   margin:
-     0
-     ${({ theme: { yoga } }) => yoga.spacing.xxlarge}px
-     0
-     0;
- }

// I would suggest you to only put styles which depends on a theme inside the callback
+  margin: 0;
+
+  @media (min-width: 1024px) {
+    width: 296px;
+    height: 231px;
+  }
+
+  ${({
+    theme: {
+      yoga: {
+        spacing
+      }
+    }
+  }) => `
+  margin-bottom: ${spacing.small}px;
+
+  @media (min-width: 1024px) {
+    width: 296px;
+    height: 231px;
+    margin-right: ${spacing.xxlarge}px;
+  }
`;

Personally, I think it is easy to read in this way.
Taking another component from your example, to ilustrate what @alan-oliv is talking about, we have something like this:
// The original
export const Title = styled(Text.H1)`
  margin-bottom: ${({ theme: { yoga } }) => yoga.spacing.xsmall}px; // 1º callback
  @media (min-width: 1024px) {
    margin-bottom: ${({ theme: { yoga } }) => yoga.spacing.xxlarge}px; // 2º callback
  }
`;

// The Yoga way (as we do)
export const Title = styled(Text.H1)`
  /* 1º callback */
  ${({ theme: { yoga: { spacing } } }) => `
    margin-bottom: ${spacing.xsmall}px;

    @media (min-width: 1024px) {
      margin-bottom: ${spacing.xxlarge}px;
    }
  `}
`;

// Using your solution (without destructuring)
export const Title = styled(Text.H1)`
  /* 1º callback */
  margin-bottom: ${theme.spacing.xsmall}px;

  /* 2º callback */
  @media (min-width: 1024px) {
    margin-bottom: ${theme.spacing.xxlarge}px;
  }
`;

// Using your solution (with destructuring)
export const Title = styled(Text.H1)`
  /* 1º callback */
 ${props => {
  const {
    spacing,
  } = theme(props); /* 2º callback */
  margin-bottom: ${spacing.xsmall}px;

  @media (min-width: 1024px) {
    margin-bottom: ${spacing.xxlarge}px;
  }
`;

IMO, the way we do here at Yoga is easier to read and it scales very well as we can see by the example above, and despite of any CSS property, we could have just one callback.
I want to know more about why you need this. There is too many ${({ theme: { yoga } }) => yoga.*} in your code?

@ggdaltoso

I want to know more about why you need this. There is too many ${({ theme: { yoga } }) => yoga.*} in your code?

Yeah. Not only that, we could do something like we do inside the Yoga components (single function call) but we lose syntax highlight and eslint+prettier

@ggdaltoso @alan-oliv
I decide to run a little test. Compare the render time of the ways we could write the component, these were the results:
  console.time packages/yoga/src/Theme/helpers/themeReader/web/web.test.jsx:259
    Render StyledButtonWithoutHelper: 18ms

  console.time packages/yoga/src/Theme/helpers/themeReader/web/web.test.jsx:267
    Render StyledButtonWithHelperSingleFunction: 9ms

  console.time packages/yoga/src/Theme/helpers/themeReader/web/web.test.jsx:275
    Render StyledButtonWithHelperMultipleFunctions: 9ms

And this was the spec I ran: https://pastebin.com/5LVVe3Qr (too big to paste here hahah)
Also, while doing this I found an error on the Button component. There's a : missing in this line. The eslint+prettier found the error for me (another vouch for in my previous comment).

@tcK1 you had me with the cache implementation and the performance tests 👏🏻
It took away my concerns, so I have no reasons not to approve this PR if will help people out on using Yoga
I had a lot a fun with all the tests and the discussions, thank you for your contribution and effort

Another solution, could be using css variables instead (we would need the ponyfill). I don't know what would be better, but I kinda prefer the Proxy since it give more flexibility (and the user can't overwrite it).

@all-contributors please add @tcK1 for code, ideas, doc


S3ObjectMetaData initializer crashes

When fetching meta data for an S3 object the S3ObjectMetaData initializer crashes with a bad access exception. This is most likely due to the representation parameter that is of typ Any. This apparently leads to a memory problem because the representation object won't be retained until the end of the initializer.

@sbhklr I've pushed some changes to develop that I believe will fix this issue. I am trying a different approach to what you've done in #37.
There is now a responseS3MetaData serialization method that can be used.
I was unable to reproduce the crash. Could you confirm if this fixes the crash you are experiencing or not?
Thanks.

Your changes indeed fix it. A couple of notes:

Shouldn't the initializer of S3ObjectMetaData return nil in case there are no headers available?
Shouldn't the completion handler be of type completionHandler: Response<S3ObjectMetaData, NSError> -> Void because Result.value already returns an optional type. Otherwise I'd need to call it like if let metaData = response.result.value?!.metaData {} Suggestion:

public func responseS3MetaData(completionHandler: Response<S3ObjectMetaData, NSError> -> Void) -> Self {
    return response(responseSerializer: Request.s3MetaDataResponseSerializer(), completionHandler: completionHandler)
}

static func s3MetaDataResponseSerializer() -> ResponseSerializer<S3ObjectMetaData, NSError> {
    return ResponseSerializer { request, response, data, error in
        guard error == nil else { return .Failure(error!) }

        guard let response = response, metaData = S3ObjectMetaData(response: response) else {
            let failureReason = "No response data was found."
            let error = Error.errorWithCode(.DataSerializationFailed, failureReason: failureReason)
            return .Failure(error)
        }

        return .Success(metaData)
    }
}

Glad to hear this fixed the issue.
I'm sort of alright with the meta data being empty, but I can see your argument. I'll make that change as well. Hold tight. New version on the way.
I agree with your second point. I'm making a change right now to fix this. I'm also doing the same thing with the data serializer currently, so I'm going to fix that. This will cause minor API breaking changes.

👍 Alrighty. I'll update the PRs accordingly.

Pushing the new version right now. Thanks for your comments and your initial PR on this. If the new version does not fix the issue, please feel free to re-open this issue.


proposal: time/tick.go: Add C() method to expose the Ticker Channel

Proposal Details
Background
The Ticker struct currently have exporter C channel which is used to send ticks to.
type Ticker struct {
	C          <-chan Time // The channel on which the ticks are delivered.
	initTicker bool
}

Problem
Currently caller has to access the C channel directly from the struct, which makes it tricky to have a mock implementation of the Ticker and put it behind a custom interface.
Proposal
Make the C channel inside of the Ticker struct unexported and have a new method func C() <- time.Time which basically returns the same C channel.
The overall implementation of Ticker would look like below:
type Ticker struct {
	c          <-chan Time // unexport it
	initTicker bool
}

// C would return the time channel which receives the ticks
func (t *Ticker) C() <-chan Time {
      return t.c
}

Related Issues and Documentation

Package time > type Ticker 				¶ 				 				 			 > func NewTicker 					¶
Feature request: time.Ticker.Stop should close 'C' channel as well.  #2650 (closed)
Package time > type Ticker 				¶
Package time > func Tick 				¶
proposal: time/tick.go: Ticker provide api to allow shared C among runtimeTimer objects #59920 (closed)
proposal: time: allow Ticker to change duration #36544 (closed)
proposal: time: add a channel to time.Ticker to detect stopped tickers #35643 (closed)
proposal: time: Add TickerFunc #68482

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

this is not a backwards compatible change that we could make.

Can we just additional method C() <-time.Time without unexport of C attribute of Ticker?

ticker.C has to have one meaning.  It can't mean both the channel field and a method that returns the channel.


RangeError in /clan/reddit-dynasty-ugjpvjr

Error in ClashLeaders
RangeError in /clan/reddit-dynasty-ugjpvjr
Maximum call stack size exceeded
View on Bugsnag
Stacktrace
webpack:///node_modules/vue/dist/vue.esm.js:869 - new we

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
RangeError in /clan/guo-shi-neng-li-zhe-gong-hui-298cv8j8


Add yml flag to skip deletion of oldest image for selected images

Previously, old images were rotated and finally deleted after a certain number of newer images were released.
This commit adds a flag for the yml definition to stop automatic deletion.

[ ] Add unit test

Fixes #342

Renaming is done here: https://github.com/osism/openstack-image-manager/pull/640


Typos and Video embed fixes

Category

[x] Content fix

What's in this Pull Request?
Fixed typos and Video embed in article

Docs Build status updates of commit bf9cc8a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
:bulb:Suggestion
View (sharepoint-ps)
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit bf9cc8a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
:bulb:Suggestion
View (sharepoint-ps)
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit fae4616:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
:bulb:Suggestion
View (sharepoint-ps)
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit fae4616:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
:bulb:Suggestion
View (sharepoint-ps)
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Two things I would add to this:

Show how to install the module without local admin rights (-Scope CurrentUser).
Remove the variables in the examples; they're superfluous and increase the number of characters typed

Two things I would add to this:

Show how to install the module without local admin rights (-Scope CurrentUser).
Remove the variables in the examples; they're superfluous and increase the number of characters typed

@ganesh-sanap

@ganesh-sanap

@ganesh-sanap

I have made changes suggested by @get-itips to "Video" tag in article. Do you want me to make changes suggested by @tseward?

@ganesh-sanap

I have made changes suggested by @get-itips to "Video" tag in article. Do you want me to make changes suggested by @tseward?

yes :)

yes :)

Docs Build status updates of commit c597064:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit c597064:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@yogkumgit Updated article with suggested changes, please review.

@yogkumgit Updated article with suggested changes, please review.

@get-itips

@get-itips

Docs Build status updates of commit 578ce98:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
:bulb:Suggestion
View (sharepoint-ps)
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 578ce98:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
:bulb:Suggestion
View (sharepoint-ps)
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 06b072c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
:bulb:Suggestion
View (sharepoint-ps)
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 06b072c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
:bulb:Suggestion
View (sharepoint-ps)
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Update so correct role name is searched for to assign

function AssignRolesAsync' was just being passed ADMINfor each tenant - but tenants have role-naming likealphaadmin`

No, I was planning to do tenant wise separation for the Roles table too.
Role Name - Admin
Tenant - alpha
Role Name - Admin
Tenant - beta
alphaAdmin looks more of a workaround.
Maybe, can you implement it the way I mentioned? I feel it would be more cleaner and will follow tenancy as such.
@27

problem with that is the we will get duplicate keys - violates unique constraint "RoleNameIndex"

https://github.com/fullstackhero/dotnet-webapi-boilerplate/pull/40


Custom BeanFactory for all DAOs

Instead of the current HangarDao to supply instances of any DAO, we should implement a custom BeanFactory to simplify the injection of required DAOs.

I assume this got closed by #496?

Oh yeah, forgot I had an issue open for this.


la-go-live

Docs Build status updates of commit 3329e66:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/group-activities-1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/group-activities-2.png
:white_check_mark:Succeeded
View

msteams-platform/concepts/extensibility-points.md
:white_check_mark:Succeeded
View

msteams-platform/includes/landing-page.md
:white_check_mark:Succeeded
View

msteams-platform/resources/team-chat-member-api-changes.md
:white_check_mark:Succeeded
View

msteams-platform/samples/app-templates.md
:white_check_mark:Succeeded
View

msteams-platform/tabs/how-to/native-device-permissions.md
:white_check_mark:Succeeded
View

msteams-platform/tutorials/get-started-dotnet-app-studio.md
:white_check_mark:Succeeded
View

msteams-platform/tutorials/get-started-nodejs-app-studio.md
:white_check_mark:Succeeded
View

msteams-platform/tutorials/get-started-yeoman.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


No options selected on any preset after installation

Greetings, I installed the presets and the shaders and there is not any option selected inside of them, look empty on my end

This means GShade can't find the shaders, so double check to make sure my shader files are properly in one of the folders defined in the shader search path directories in the GShade Settings tab.

This means GShade can't find the shaders, so double check to make sure my shader files are properly in one of the folders defined in the shader search path directories in the GShade Settings tab.

Oh, okay I saw the path, it is different than the "Custom Shader" path specified on the guide though... should I Just put your custom shaders there with the rest of the GShade ones?

FIXED: Okay I managed to add the custom shader path to GSHADE, problem is solved. Thank you!


Upgrade to WF 19.1.0.Final

@mariofusco I think you can close this one
I split WF19 upgrade from Weld and other org.jboss.spec upgrades to be independent and possible to merge WF19 one https://github.com/errai/errai/pull/389 or both if Weld upgrade https://github.com/errai/errai/pull/388 will work at some point

meanwhile WF errai distro was released as it is standalone module not relying on other parts of Errai

the PR https://github.com/errai/errai/pull/390 was merged


ghdl not found in .tcl-interpreter during yosys-synth

I'm new to hardware design in general, so please let me know if I am the error.
Describe the bug
openlane2 fails when executed in /foss/designs with
openlane -f VHDLClassic --design-dir /foss/designs/*design_name*/ /foss/designs/*design_name*/config.json
Yosys fails, because some step.tcl cannot find ghdl.
ERROR: TCL interpreter returned an error: invalid command name "ghdl"
Logs attached.
The tclsh and the normal bash execute ghdl just fine.
Verilog flows work like a charme, so I know how to work around by manually translating with ghdl.
I don't know tcl at all, but it seems odd that global variables can get forgotten when executed from other folders.
Thats the reason I figured there might be a problem with Paths or env, hence I post here.
Please let me also know, if you think thats a problem with openlane itself so I can post there.
To Reproduce
Steps to reproduce the behavior:

Setup a container with the start_vnc.bat
connect to the container via your webbrowser
create a folder inside /foss/designs
create a basic config.json (mine config.json)
Open a Terminal in /foss/designs
execute faulty command from above (without the asterix')
openlane -f VHDLClassic --design-dir /foss/designs/design_name/ /foss/designs/design_name/config.json

Expected behavior
The flow should continue at least.
Environment:

OS: Windows, Docker Desktop
Operating mode: VNC
Version tag latest[03.07.2024]

Additional context
Error log of the run:
error.log
Log of the 01-yosys-vhdlsynthesis step:
yosys-vhdlsynthesis.log
COMMANDS of this step-folder contains:
yosys -c /usr/local/lib/python3.10/dist-packages/openlane/scripts/yosys/synthesize.tcl

This seems to be an issue with OpenLane and how it calls Yosys to enable the GHDL module. I have opened an issue with OpenLane for clarification, see https://github.com/efabless/openlane2/issues/514.

We need a wrapper for yosys to properly load the modules like (see https://github.com/efabless/openlane2/issues/514#issuecomment-2232835506)
#!/bin/sh
exec -a "$0" <path/to/yosys-bin> -m ghdl "$@"

This is the version that has been implemented. @donn
#!/bin/bash
if [[ $1 == "-h" ]]; then
    exec -a "$0" "$TOOLS/yosys/bin/yosys" "$@"
else
    exec -a "$0" "$TOOLS/yosys/bin/yosys" -m ghdl -m systemverilog "$@"
fi

Implemented in 79a3942263b5babccfc01fa6ab550124875c706b


CellProcessors as Map?

Hi,
Is it possible to have the CellProcessors passed as a Map? (As as an alternative to an Array?)
The key could be the column name (like defined in the header) and the value as already defined in the Array?
This would be very usable when there are allot of columns and the defaults are good, except a few columns?
e.g.
Instead of:
public static final CellProcessor[] PROCESSORS = new CellProcessor[] {
    // other
    //      50
    //           columns...
    new ParseDate("dd/MM/yyyy") };

something like:
public Map<String, CellProcessor> map = new HashMap<>();
map.put("myDate", new ParseDate("dd/MM/yyyy") ;

Also as an alternative would be useful to map based on field type? E.g. in many cases only the Date
fields require handling.
E.g.:
public Map<Class, CellProcessor> map = new HashMap<>();
map.put(Date.class, new ParseDate("dd/MM/yyyy");

Thank you.

Hello,
my intention is a little different. If someone add additional columns to the CSV file It should still work with some old program versions (they don't need this new value at logic). At the moment there is a hard restriction that all columns needs a value inside CellProcessor[] (at least a NULL value). That means if a column is added to CSV files you have to change the old app (even if this additional value is not needed at this old version).
Therefore I think it's really useful to add the CellProcessors by Map. Of cause - you also don't need to define the NULL values.
Your second suggestion is very NICE! But this will need more changes to the code. For now I added a pull request to add CellProcessors with a Map<String, CellProcessor> ...


User API doesn't support changing password without providing old password by admin user

User API doesn't support changing password without providing old password by admin user

Due to security compliance, let's defer this feature


Delimiter for THORNames memo in Thorchain-query

As per https://github.com/xchainjs/xchainjs-lib/issues/900
There should be a trailing '::' at the end of the memo

This is fixed now.
Some other issues and points.

Load screen, type in a new thorname and press enter. you are prompted for a password.
If I put in a name, select year then press estimate, it works but get an an error as I didn't select alais. When i do buttons are greyed out for like 5 seconds.
Estimate should have a USD amount
As the estimate is predetermined based on year, what is a button needed? Why can't the fee just updated based on selection?

Done for now


[HUDI-2171] Add parallelism conf for bootstrap operator

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

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

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #3268 (aff1a1e) into master (b0089b8) will decrease coverage by 44.86%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #3268       +/-   ##
============================================
- Coverage     47.72%   2.86%   -44.87%
+ Complexity     5529      85     -5444
============================================
  Files           934     283      -651
  Lines         41457   11711    -29746
  Branches       4167     961     -3206
============================================
- Hits          19787     335    -19452
+ Misses        19908   11350     -8558
+ Partials       1762      26     -1736

Flag
Coverage Δ

hudicli
?

hudiclient
0.00% <ø> (-34.47%)
:arrow_down:

hudicommon
?

hudiflink
?

hudihadoopmr
?

hudisparkdatasource
?

hudisync
5.37% <ø> (-49.15%)
:arrow_down:

huditimelineservice
?

hudiutilities
9.11% <ø> (-50.15%)
:arrow_down:

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

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b0089b8...aff1a1e. Read the comment docs.