Application error: a client-side exception has occurred

Intermittent error, first reported yesterday,
Unable to reproduce at first, I've since been able to with the prompt: show me an es6 generator in js
Error on page:
Application error: a client-side exception has occurred (see the browser console for more information).
Error content from browser console:
Error: marked(): input parameter is undefined or null
Please report this to https://github.com/markedjs/marked.
    at Marked.parse (689-9b6a1ec9ea44ea98.js:30:7486)
    at marked (689-9b6a1ec9ea44ea98.js:30:8403)
    at markdownToText (page-8ba03c29f0c57a5b.js:1:9907)
    at pre (page-8ba03c29f0c57a5b.js:1:13456)
    at Mf (fd9d1056-fc215812d5e2fa90.js:9:39686)
    at rC (fd9d1056-fc215812d5e2fa90.js:9:112346)
    at Jk (fd9d1056-fc215812d5e2fa90.js:9:102245)
    at Ik (fd9d1056-fc215812d5e2fa90.js:9:101422)
    at df (fd9d1056-fc215812d5e2fa90.js:9:101240)
    at ef (fd9d1056-fc215812d5e2fa90.js:9:98669)
window.console.error @ 2472-7918ce866bb273fc.js:1

In the recently added copy to clipboard button on individual code blocks, it seems there's times no content gets passed in during renders. I think this can probably by short-circuiting the markdownToText helper method if no content is supplied.


attaching a router to an existing Vue instance (set the router property)

It appears that router property must be present at the time of Vue instance construction
new Vue({router}).$mount("#app")
I like to do the following
var App = Vue.extend();

// later in the code...
App.router = router;

new App().$mount("#app")

Getting an error
Vue warn]: Error when rendering root instance:
Uncaught TypeError: Cannot read property 'matched' of undefined

Why do you need to add the router later on?

I have a server rendered multi tenant pager with SPA part.  And that page contain Vue instantiation code common for all clients (including ajax wireups).  Each client specific SPA parts are loaded dynamically.  And routing is only applicable in some of those cases.

The extended component is a constructor. You can just do:
App.options.router = router

Thanks! Didn't notice that indirection!!

I would like to do this too, except after the creation of the component.
@posva, I'm looking to adopt vue-router incrementally. I've been looking through the docs and searching, trying to find a way to inject the router after component creation.
We've implementing the router in an admin dashboard for an application, and there's a component acting as the root component of the admin dashboard. That component is bundled in along with the rest of the app's code.
Something like this:
<template>
    <div class="admin-dashboard">

    </div>
</template>

<script>
    import routes from 'routes/dashboard';
    import VueRouter from 'vue-router';

    export default {
        name: 'admin-dashboard',

        router: new VueRouter({ routes })
    }
</script>

Now, due to the VueRouter being instantiated at time of import, that VueRouter is instantiated even if the component is not used. So on every other page, VueRouter automatically changes the page's hash to #/ when it's instantiated.
I could set a global, or check the page URL etc. before instantiating VueRouter, but I'd prefer if I could set up VueRouter in some lifecycle hook so that the instantiation doesn't depend on a global, or the URL, but instead whether or not the component is even used. It just feels like a better solution.
I've tried doing:
beforeCreate() {
    // Dynamically add in router
    this.$options.router = new VueRouter({
        routes
    });
}

But doing that presents me with the following error in the console:
TypeError: Cannot read property 'matched' of undefined
    at render (vue-router.esm.js:68)
    at createFunctionalComponent (vue.js:4054)
    at createComponent (vue.js:4244)
    at _createElement (vue.js:4414)
    at createElement (vue.js:4351)
    at vm._c (vue.js:4483)
    at Proxy.render (kiosk.vue?8edf:131)
    at VueComponent.Vue._render (vue.js:4535)
    at VueComponent.updateComponent (vue.js:2788)
    at Watcher.get (vue.js:3140)

Is there a way to do this? If not, checking the URL or a global is fine, I'd just rather the creation of VueRouter be implicit as a result of creating the admin-dashboard component in the first place.
Thanks in advance!

I'm using https://single-spa.js.org/ and I want to dynamically load a router into the root Vue instance. I'm doing something similiar to @CaelanStewart and I'm getting:
TypeError: Cannot read property 'matched' of undefined
Caelan, did you manage to get it to work?

Has anyone managed to do this? Our Vue app is packaged as a web component that gets included on a separate web page. We're looking for a way to init the router in the connectedCallback on our web component (which I believe equates to mounted in Vue). Doesn't seem to be a way to do this.

I managed to get this to work by setting: app.$options.router directly on the wrapper application

@espiegel I've tried your suggestion but I'm getting the same error as @CaelanStewart .
I've tried in my sub component that wants a router by trying this:
    beforeCreate() {
      this.$parent.$options.router = getRouter();
    }

I've tried both this.$parent and using a reference to the wrapper Vue instance.
The getRouter function looks like this (in other module);
let router;

export function getRouter() {
  if (router) { return router; }
  Vue.use(VueRouter);
  router = new VueRouter();
  return router;
};

Did you put that on the root vue instance?
I also managed to put an empty new VueRouter({}) on the root instance and then dynamically call:
app.$router.addRoutes(routes); with a routes array

@espiegel The question is; when in the lifecycle did you assign app.$options.router ?
I tried this basically;
// app.js
var app = new Vue({ ... });
window.$app = app;
app.$mount('#app');

// in component registered into app.
beforeCreate() {
  console.log(window.$app); // jackpot!
  window$app.$options.router = getRouter();
}

@espiegel  Basically from what I've gathered; the issue is that <router-view> is trying to use $route-property of the parent component that renders it.
And $route and $router gets assigned as  properties here; https://github.com/vuejs/vue-router/blob/dev/src/install.js#L38-L44
The issue is that if you don't assign the router to the options before instantiating the root Vue instance those properties will point to the value assigned by this line https://github.com/vuejs/vue-router/blob/dev/src/install.js#L29 which will be this meaning the component itself. Which will not have any _routerRoot property.
But since this is a closed ticket, I'll stop here! :)

@CaelanStewart I was wrestling with the same thing for a bit today. It's interesting that all tutorials assume you want the router on your root instance and you're not building a mini SPA that's part of a larger site. My main problem was the undesired hash in the URL on pages that weren't using vue-router, although I get what you're saying about not wanting to load it at all if it's not even used on the current page. My solution for the hash was just to use the mode: 'history' option. But I think you could solve your problem using an async component. I realize you've probably moved on by now so I guess this is for posterity.
components: {
    'component-with-router': () => import('./component-with-router')
}

@regularmike
That's great, thanks for the suggestion.
If I'm honest we opted for the lazy route and forgot about it, since there's so much on the backlog still to get through.
I'll look into the async component option, though.
Thanks

@regularmike
could u please show an example how exactly you are doing this async thing? I dont get where goes what.
I remove router from main vue instance
    this.vueInstance = new Vue({       el: this.rootElement,       i18n,       store,       // router     });
and then how do I add it to a component on a specific page load ?
I have checked the docs but they also show it like one line
components: {
'my-component': () => import('./my-async-component')
}
but where does this code go? and how do I append router to vue instance in './my-async-component'?
I didnt know my understanding of vue was this bad :S
thanks


Security warning (real!)

If you change the code of the Assembler, you can make it insecure
I know you are going to say "yeah but thats for all open source projects!", and then you are right. But I think I should say this just in case.


feat: add bionic compatibillity

add bionic as supported distro
test install all tools for bionic

Leaving this as draft until i  fixed the binary builders


Package request: ocamlPackages.raylib

Project description
Ocaml bindings for the raylib graphics library.
Metadata

homepage URL:  https://github.com/tjammer/raylib-ocaml
source URL: https://opam.ocaml.org/packages/raylib/
license: mit
platforms: linux, macOS

@vakiyama I have opened PR for raylib ocam here #368926


Use "reason" field for Summarizer logs instead of "message"

Let's try to converge on "reason" as a common field across events, and avoid using "message" since it's vague/antiquated (now we have structured logs!)

can this move to next?

I think @arinwt fixed this with some recent refactoring?


Create Pyi parser

There is parser in progress for parsing PYI ast: https://github.com/alexpantyukhin/PythonParser
From tests it seems that now it's possible to parse some simple modules without 'IF's.

There are still some problems with parsing type's. But going to fix that soon.

Great effort! Keep up the good work!

Parser in progress. Tested it today on whole pyi files in https://github.com/python/typeshed/tree/master/stdlib . it raises error in 175 cases from 885 files. Keep working with this.

That is really great progress!! Sorry I haven't had much time to look at it yet since I have been busy with the compiler (and Jupyter).

Now there are 12 files from whole typeshed  raises error (some except situation for stdlib and left for protobuf from stubs). Now I'm thinking how it could be best way to generate the Fable AST and compile it to files. I guess It's the time where I can move forward with this.

@alexpantyukhin Not sure I understand what you mean about generate Fable AST? Why Fable AST? Did you mean F# AST to generate F#?
Btw please note the new [<NamedParams(fromIndex=1)>], e.g: https://github.com/fable-compiler/Fable.Python/blob/main/src/stdlib/Builtins.fs#L123 on how to handle e.g named only arguments and methods with optional arguments.


Need support for manual update for the test case status as Passed, failed and Skipped

Area -> Testman
New Feature - Yes
Describe the bug - NA
Version - 3.1.0
Expected behaviour - Need support for manual update of the test case status as Passed, failed and Skipped. This option will help to track the test case status for manual validation and tracking. Option to mention the Version tested as text field will be helpful to know with what version the test case is tested and updated
Actual behaviour - No support available
How to Reproduce? - NA

Tested OK
Tested Version: 3.2.5
Need support for manual update for the test case status as Passed, failed and Skipped --Issue Fixed
https://jam.dev/c/e762b12e-e354-47f6-837f-31c1f00d6de6

Issue fixed and Tested ok


Update mb_vsprintf method

Phalcon 4.2+ debug returns an error when a simple warning occurs.
The list function returns a warning when the array created by the preg_split does not correspond to the number of parameters of the PHP list function.
On vanilla PHP, this does not cause any problem, because it assigns null to the unsetted parameters by default. Unfortunately, on a newer version of Phalcon, 4.2+, or Zephir, it catches the warning and returns an error, even when using @.
To avoid this error, simply assign the variables manually instead of using the PHP list function.
While this fix is not the prettiest or most advanced, it allows us to avoid having to modify the rest of the method which does not cause any problem.

Thank you @fchenier


Add destroy method?

It would be good (also good plugin practice) to add a destroy() method to remove the hoverIntent bindings on an element or set of elements.
This would take a minimimal amount (3 unminified lines) of code, and the code has already been written here:

http://stackoverflow.com/questions/8151278/jquery-unbind-or-rebind-hoverintent

While it's possible to just direct users to the StackOverflow question, I think it would be better to include the method inside the plugin to avoid versioning issues in the future, as well as for the sake of promoting plugin modularity (allow for revokable methods).
thanks, this is a great plugin!

Agree!

With the newest (yet-unreleased) version of the plugin, the instructions found on stackoverflow are actually no longer valid, so perhaps a destroy() method (or similar) is definitely under consideration. Thanks for the suggestion!


Issue 4224 duration bug

Description
Fixes #4224

longer default duration (1000 hours)
default duration is 2 / C-rate hours for C-rate steps

Currently working on adding a warning if maximum duration is reached
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[ ] No style issues: $ pre-commit run (or $ nox -s pre-commit) (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)
[ ] All tests pass: $ python run-tests.py --all (or $ nox -s tests)
[ ] The documentation builds: $ python run-tests.py --doctest (or $ nox -s doctests)

You can run integration tests, unit tests, and doctests together at once, using $ python run-tests.py --quick (or $ nox -s quick).
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

@valentinsulzer You need to take abs(value) in CRate.default_duration(), because value comes through negative for charge steps.

@brosaplanella @valentinsulzer To confirm, this is working as expected for the use cases on which I was testing.

Changing default duration to 1000 hours caused a severe performance degradation, investigating further

Thanks! I just tried it and though I'd prefer a longer duration barring a substantial performance hit, in my opinion, the bug is fixed by the warning alone.
I wonder if 100 + ε hours might be a happy medium?

This PR adds the warning and also fixes the C/100 case which I assume is more common than specifying a small current (backed up by a n=2 sample size from @aabills and @ejfdickinson ). The default duration is 24h and we can reconsider once @MarcBerliner is done with updates to solvers, which will make solve times much less sensitive to things like duration, period, etc


Separate migration generator

This actually builds on top of my other PR, but didn't want to do this refactor in that PR.
Seeing as Migrator is no longer tied to the filesystem, I thought I would pull the make() function and all supporting helpers into it's own class. Will make it easier to ensure no fs related dependencies sneak back into Migrator.
This is just an idea, happy for it to be closed if you folks don't think it's a good idea :) It's just the last commit which is new in addition to the last PR - https://github.com/tgriesser/knex/pull/2786/commits/76ebc1e458b726f8ec9a67194cd1f0d8fb7fd757

https://travis-ci.org/tgriesser/knex/jobs/424648921#L3481

@elhigu Probably makes sense to ignore this one for now, it is closely related to https://github.com/tgriesser/knex/pull/2775 and probably should be reassessed after that one lands.

Yeah, I will fix this one up now the other PR has been merged. Want me to close for now until I do that?

No need to close, take your time 👍

@elhigu I have updated the PR. the coverage has decreased, not really sure why, thoughts on what tests I should add?

@JakeGinnivan I don't think issue is with your code, see https://github.com/tgriesser/knex/pull/2810 -> after it is merged, I think that rerunning CI on PR should pass.

@JakeGinnivan Please pull changes from tgriesser/knex master, that should resolve the issue

Also, if you don't agree with separating these classes, happy for this one to just close. Is just an idea

Personally, I think that this is very much a step in the right direction.
Huh, coverage calculation is still off. Wth?..

We shouldn't care that much about coverage... those calculations did seem pretty strange like there were some files, where it wasn't able to calculate any coverages.

Any chance of a @next tagged release so I can give all this a spin :)

@elhigu Would be great to also include https://github.com/tgriesser/knex/pull/2802 @next release to unblock people waiting for it :)

Everything that is in master is going to be there. I also document how to do the next release so that people can easily update it to have code from latest master.

or afterwards and do another release

Or that :)

Awesome! Thanks heaps :)

And knex@next is now available.

There goes my sleep 😂
runs off to add shiny new stuff to work project


Give the user an error "The file appears to be corrupted" if the head…

…er at the end of the .skyd file has impossible values.
An AutoQC user had a .skyd file that was corrupted.
They were getting an error message "Array dimensions exceeded supported range".
With this change, they will get an error message which looks like this:

(Is that the best wording for this message? Should it say anything more? Should it tell them to delete the file?)

Yes, I think letting the user know that the only way to recover is to delete the .skyd file and reimport the raw data would be helpful.
Thanks a lot for adding a check for corrupted data!

Yes, I think letting the user know that the only way to recover is to delete the .skyd file and reimport the raw data would be helpful.
Thanks a lot for adding a check for corrupted data!


Update huggingface-hub dependency to 0.15.1 version

Close #1291.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

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

Comparison is base (b17c73b) 90.43% compared to head (6cc92b7) 85.45%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1294      +/-   ##
==========================================
- Coverage   90.43%   85.45%   -4.98%
==========================================
  Files         183       59     -124
  Lines       10796     1767    -9029
==========================================
- Hits         9763     1510    -8253
+ Misses       1033      257     -776

Flag
Coverage Δ

jobs_cache_maintenance
99.08% <ø> (ø)

jobs_mongodb_migration
83.49% <ø> (ø)

libs_libcommon
?

services_admin
85.99% <ø> (ø)

services_api
?

services_worker
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 124 files with indirect coverage changes

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


Draft: Bazel recursive stubgen example

I made this branch mainly for another MR I am trying to merge with the help of @nicholasjng. If/when the nanobind-bazel MR can land, would you maintainers want this example merged as well?
It's meant to be a slight modification of the existing example to create an example sub-module and makes a small modification to the setup.py to copy files whilst preserving the module structure.

Hey @cemlyn007 , since rules_python just released v1.0.0 with a fix for the Windows library locations, I updated the setup.py (and some other files) accordingly to test if we can build nanobind on Windows with only a hermetic interpreter.
Could I ask you to rebase on current bazel? Then I'll merge this right after.

If @wjakob agrees, I can take over all Bazel-related PRs on this repo.

I agree.
A tangent : would it make sense to advertise more broadly that there is a bazel branch on the master branch? This may not be obvious to visitors.

Thanks! And sure, I can add a note to the readme or something similar.

Note: I declared the dependency on nanobind_bazel to be against a stable release from BCR, since that is how most users will use the package. Instead of keeping the local_path_override in this repo, I will add it in nanobind-bazel CI.
Currently CI is all red, since we're passing arguments to stubgen that have not been implemented yet (PR is open already). Once nanobind-bazel v2.4.0 is out on the BCR, this will be fixed.


[Service Bus] Prevent batching from being used on entities with large message support

Idea to fix https://github.com/Azure/azure-sdk-for-net/issues/44261
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

@danielmarbach:  Thanks very much for your contribution.   As discussed in #44272, hardcoding assumptions about service limits in the client isn't something that we're comfortable doing.   There's an existing request filed on the Service Bus service to provide additional metadata to the client that will allow us to understand the actual limits that the service enforces on batches when large message support is enabled.   Once that is available, the client will be updated to take advantage.
As this isn't a change that we'll be moving forward with, I'm going to close this out.   Thank you, again, for your efforts and contribution.

That perfectly fine and reasonable decision. I slapped this together in roughly 30 min and it was meant as a discussion starter only. Thanks for following through


No way to follow a channel

I want to follow a known channel<EMAIL_ADDRESS>but am unable to just type it in someone on the client.

The android client v1.1.0 allows to follow either known or unknown channels through channel search.


Reflection on TimeSpan.FromSeconds needs to specify the parameter types

The TimeSpan type includes the method FromSeconds(double), and in .NET 9.0, a new overload FromSeconds(long) was introduced. The reflection logic for FromSeconds needs to be updated to specify the exact parameter types. Without this fix, the code will throw exceptions such as:
 System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
  ---> Jil.DeserializationException: The type initializer for 'Jil.Deserialize.InlineDeserializer`1' threw an exception.
  ---> System.TypeInitializationException: The type initializer for 'Jil.Deserialize.InlineDeserializer`1' threw an exception.
  ---> System.Reflection.AmbiguousMatchException: Ambiguous match found for 'System.TimeSpan System.TimeSpan FromSeconds(Int64)'.
    at System.RuntimeType.GetMethodImplCommon(String name, Int32 genericParameterCount, BindingFlags bindingAttr, Binder binder, CallingConventions callConv, Type[] types, ParameterModifier[] modifiers)
    at System.Type.GetMethod(String name, BindingFlags bindingAttr)
    at Jil.Deserialize.InlineDeserializer`1..cctor()

@kevin-montrose please approve PR or at least confirm that this package is out of support, so that I could have a proof reason to spend effort on replacing Jill with System.text.json.


[experimental] An example usage of REST API

Retrieve the account sequence number via the REST API instead of going through the old json rpc client.
We should probably have a REST API client with a number of util functions, like retrieving the resource or event, sending a transaction, retrieving transaction result, etc.

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/diem/diem/tree/main/developers.diem.com, and link to your PR here.)
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

This PR https://github.com/diem/diem/pull/9823 introduces a new rest client which can handle some of the rest APIs. Its still WIP but should be basic enough for people to start using it for querying account resources and such.


[Bug?]: Dev server crashes when there is a syntax error solidjs/solid-start#1222

https://github.com/solidjs/solid-start/issues/1222

I'd like to see the server revert and remain running until the change is corrected.
This is particularly frustrating when using Auto Save in VSCode, as the code isn't necessary correct on each code (e.g. might be awaiting a closing tag).

Fair. This needs to be fixed

@nksaraf this is a deal breaker for my team, maybe you can point me to the problematic code and I will make a PR

one issue is that because there are a few places your files get parsed, we have to handle the error case everywhere, which is fine, just needs some thorough trial and fixing.
I had found some cases, for example the fsRouter parses files to check for exports, this probably errors out esbuild which is not handled gracefully.


[9.x] Use Faker within seeders

This exposes the Faker API within seeders.
Example
<?php

class TestSeeder extends Seeder {
    public function run()
    {
         Test::create([
             'title' => $this->faker->title
         ]);
    }
}

Can't this PR go to 8.x instead?

I suggest putting this in your factory.


fix(app): response object was not accessible if app verification threw request exception

Fixes #682

nicely done on the unit tests.

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

Merging [#685][cc-pull] into [master][cc-base-branch] will increase coverage by +0.07%

@@             master       #685   diff @@
==========================================
  Files            29         29
  Lines          2547       2550     +3
  Methods           0          0
  Messages          0          0
  Branches        396        396
==========================================
+ Hits           2093       2099     +6
+ Misses          325        322     -3
  Partials        129        129

Powered by Codecov. Last updated by [5821726...03f859f][cc-compare]
[cc-base-branch]: https://codecov.io/gh/deis/controller/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/deis/controller/compare/58217262072343fd76a94fe35606e7deba1453be...03f859f69d56031ea69d1e69f64481eabd31f1a2
[cc-pull]: https://codecov.io/gh/deis/controller/pull/685?src=pr


latest also copies other tag folders

Cannot reproduce.

Need to re-create the latest folder each time.


Support for onnx export

@AyushExel I am working on exporting trained model to onnx format. I have taken some references from here and here. I was able to export the model successfully but predictions are not working.


Makefile: Add redirect support

I'm seeing that visiting http://<IP_ADDRESS>:4000/latest/ after running this is printing the contents of the rendered _site/latest directory. We must be missing something here...

@tomdee I'm willing to discuss solutions here. For example, that .html can probably be made a .md, especially now that we can use redirect_to in it.

@djosborne I updated the Makefile to cover .html files too


Add an option to emit errors via webpack

probably a bit like this: https://github.com/MoOx/eslint-loader#errors-and-warning
see context here: https://github.com/necolas/postcss-bem-linter/issues/26

But what if user want to hide linter warnings but emit a error from postcss-mixins?
I think we need this settings in linter plugins.


Low Level Retries Interval

What is your current rclone version (output from rclone version)?
A flag like the --retries-sleep could be useful for the low-level retries too. This would be helpful in making the vfs cache more reliable too since we can simply set the interval to 24 hours or so in case of failures due to rate-limiting.
What problem are you are trying to solve?
Backoff between low-level retries.
How do you think rclone should be changed to solve that?
Introduce a flag to set the --low-level-retries-interval

The retries already do that as they back off each time they get failure.
// consecutiveRetries starts at 1 so go from 1,2,3,4,5,5 => 1,2,4,8,16,16
		// maxSleep is 2**(consecutiveRetries-1) seconds + random milliseconds

Once it gets a successful connection, it resets.


Branch bernard ppp draft

Codecov Report

Merging #279 (0717676) into master (9427549) will increase coverage by 1.39%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #279      +/-   ##
============================================
+ Coverage     67.80%   69.19%   +1.39%
- Complexity      763      779      +16
============================================
  Files           127      127
  Lines          2587     2594       +7
  Branches        311      314       +3
============================================
+ Hits           1754     1795      +41
+ Misses          743      702      -41
- Partials         90       97       +7

Impacted Files
Coverage Δ

...commands/classcommands/RemoveFromClassCommand.java
83.72% <0.00%> (-13.51%)
:arrow_down:

src/main/java/seedu/address/model/AddressBook.java
95.34% <0.00%> (+4.65%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
86.58% <0.00%> (+4.87%)
:arrow_up:

...gic/commands/classcommands/DeleteClassCommand.java
73.33% <0.00%> (+73.33%)
:arrow_up:

.../classcommandparsers/DeleteClassCommandParser.java
100.00% <0.00%> (+100.00%)
:arrow_up:

...sscommandparsers/RemoveFromClassCommandParser.java
100.00% <0.00%> (+100.00%)
:arrow_up:

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 9427549...0717676. Read the comment docs.


UI is blocking when getting the "session timed out" error

Hello
We built an app with Amplify 2.18.0 and AmplifyUILiveness 1.1.1.
It worked well if we followed the hints to finish the flow.
When we simulated some failure cases like time-out, we started the face check flow and didn't finish it.
The UI was blocking for about 5 - 10 seconds when getting the first error.
The console log showed the error but there was no "TTT test" string in the first time.
After that, we got the next error.
The console log showed the "TTT test" string and the error.
It is weird that the UI is blocking when getting the "session timed out" error.
Please help with this case.
Thank you!
Sample code we used:
FaceLivenessDetectorView(sessionID: viewModel.sessionID, region: "us-east-1", disableStartView: true, isPresented: $isPresentingLiveness) { result in
                switch result {
                case .success:
                    let _ = debugPrint("TTT success")
                case .failure(let error):
                    DispatchQueue.main.async {
                        let _ = debugPrint("TTT test")
                    }
                    let detectError = error as FaceLivenessDetectionError
                    let _ = debugPrint("TTT \(String(describing: detectError))")
                }
            }

Console logs:

@wind15973 Thanks for submitting the issue, we'll try to reproduce the issue and investigate further.

Hi @phantumcode, I created a PR to fix this issue #64. I will also try the latest release. Thanks for your help.

@wind15973 Can you provide confirmation that the latest release fixes your issue?

I will test it in these two days. Will back to you soon. Thank you

@wind15973 Fix should be in the latest release 1.1.4
Your fix in #64 does not resolve the root cause.  In the original issue submitted, the error message was delayed 5-10 seconds because the error was being set on a background thread (this was fixed in the previous release).  The error messages are getting set twice because the timeout handler was getting call even after the timeout error was already set as the current state, this should be fixed in the latest release 1.1.4.

Hi @phantumcode, this issue still has not been fixed in the latest release 1.2.1.


Improve documentation about concurrency and *Gui.Execute()

Explain how concurrency works in gocui under the hood. For instance, write a bit about channels for keyboard/mouse and user events and put more emphasis on how to modify views from goroutines.
Relates to #77

When a used gocui recently, I made the mistake of calling SetKeybinding from a Layout method. That made my system works weird, binding the same key multiple times. Saying that, maybe the best way for the documentation is to define the best practices or patterns to use gocui's methods in general.

I agree with @gbitten, but what should be considered is that _good practices _ can be found in the examples directory. However, they only teach people how they should use something, not how it works or why that way. The documentation is quite obscure in that respect.

Function renamed to *Gui.Update() + more docs in 6564cfcacb01db61e3a4e983405a598db3e91982.


Update lxml version to 4.9.4

Breaking change for the new release? @DeltaFox0018

after updating to version 2024.1.0 of home assistant and restarting the server, the error in the logs was that it couldn't find lxml:4.9.3 even after trying to reinstall integration. the only one was to update the lxml version

Is it possible to create release version 1.2.4 so as to also update HACS?

Yes, but currently not near my laptop and can only do this on a laptop.


Fix API.AddApplication to return NotFound if inexistence repo given

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Nice.
/lgtm

Nice catch!
/approve


improve text for searching prior knowledge

This PR changes the text when searching for prior knowledge so that it is more clear this is not yet the screening phase.

Thanks, hopefully, this resolves the confusion for some users!

Thanks, hopefully, this resolves the confusion for some users!


Mandelbrot only Terrible works

I'm probably doing this wrong but only the "Terrible" Mandelbrot code gives me a graphic output. The others don't seem to do anything? Or maybe they do?
If I call them from the Code/ directory I get the following Stacktrace:
Traceback (most recent call last):
  File "MandelbrotCode_(Uncommented).py", line 41, in <module>
    plot_mandelbrot(x, y)
  File "MandelbrotCode_(Uncommented).py", line 33, in plot_mandelbrot
    plt.savefig('Results/Figures/Mandelbrot.png', facecolor=None)
  File "/home/robin/.local/lib/python3.8/site-packages/matplotlib/pyplot.py", line 958, in savefig
    res = fig.savefig(*args, **kwargs)
  File "/home/robin/.local/lib/python3.8/site-packages/matplotlib/figure.py", line 3019, in savefig
    self.canvas.print_figure(fname, **kwargs)
  File "/home/robin/.local/lib/python3.8/site-packages/matplotlib/backends/backend_qtagg.py", line 85, in print_figure
    super().print_figure(*args, **kwargs)
  File "/home/robin/.local/lib/python3.8/site-packages/matplotlib/backend_bases.py", line 2319, in print_figure
    result = print_method(
  File "/home/robin/.local/lib/python3.8/site-packages/matplotlib/backend_bases.py", line 1648, in wrapper
    return func(*args, **kwargs)
  File "/home/robin/.local/lib/python3.8/site-packages/matplotlib/_api/deprecation.py", line 412, in wrapper
    return func(*inner_args, **inner_kwargs)
  File "/home/robin/.local/lib/python3.8/site-packages/matplotlib/backends/backend_agg.py", line 541, in print_png
    mpl.image.imsave(
  File "/home/robin/.local/lib/python3.8/site-packages/matplotlib/image.py", line 1675, in imsave
    image.save(fname, **pil_kwargs)
  File "/home/robin/.local/lib/python3.8/site-packages/PIL/Image.py", line 2297, in save
    fp = builtins.open(filename, "w+b")
FileNotFoundError: [Errno 2] Datei oder Verzeichnis nicht gefunden: 'Results/Figures/Mandelbrot.png'

But I guess that's because it's supposed to be run from the main folder?
Do they write into Results/Figures and overwrite the already present figures there?

Your guess is correct. For some reason on my system the current working directory is always "ProjectTemplate" instead of "ProjectTemplate/Code". I'm still trying to find out why this actually happens.


Read request body on POST to changes feed

Prevents to return https://github.com/apache/couchdb-couch/pull/68 error for every POST request.

cc @rnewson

Hrm. Will look in the week. Not sure this is the right fix

Ok. I also not sure if this one is correct, but I took handle_view_list_req/3 as an example for that. Also, for couch_http, it actually does the same, but through couch_httpd:json_body_obj (but now it's broken because this branch will be never reached).

FWIW, this does seem to fix the problem for me locally.

Yeh also as mentioned on irc, this fixes the pouchdb test suite that is currently broken with anything that filters by doc_ids

Ok, I took a look. I think the right fix is to revert https://github.com/apache/couchdb-couch/pull/68 and but the validating logic right after the original couch_httpd:json_body_obj(Req). PR shortly.

Reverting apache/couchdb-couch#68 seems to be enough, I'm unable to make a post request that causes the function_clause, can someone explain?

vote to close this now apache/couchdb-couch#68 is reverted.
Clearly there's a way to get a function_clause error, @jaydoane or @kxepal are you able to induce one with latest code?

@rnewson
$ curl -v -XPOST http://localhost:59840/db/_changes?filter=_doc_ids  -H 'Content-Type:application/json'
*   Trying ::1...
* connect to ::1 port 59840 failed: Connection refused
*   Trying <IP_ADDRESS>...
* Connected to localhost (<IP_ADDRESS>) port 59840 (#0)
> POST /db/_changes?filter=_doc_ids HTTP/1.1
> Host: localhost:59840
> User-Agent: curl/7.43.0
> Accept: */*
> Content-Type:application/json
>
< HTTP/1.1 500 Internal Server Error
< X-CouchDB-Body-Time: 0
< X-Couch-Stack-Hash: 564179134
< X-Couch-Request-ID: cde7e6ea4c
< Server: CouchDB/bcb7c1a (Erlang OTP/18)
< Date: Sun, 12 Jul 2015 13:01:01 GMT
< Content-Type: text/plain; charset=utf-8
< Content-Length: 69
< Cache-Control: must-revalidate
<
{"error":"unknown_error","reason":"function_clause","ref":564179134}
* Connection #0 to host localhost left intact

Logs:
2015-07-12 16:01:01.280 [error]<EMAIL_ADDRESS><0.358.0> req_err(564179134) unknown_error : function_clause
    [<<"jiffy:decode/1 L10">>,<<"couch_util:json_decode/1 L383">>,<<"couch_httpd:json_body_obj/1 L551">>,<<"couch_changes:get_doc_ids/1 L313">>,<<"couch_changes:configure_filter/4 L196">>,<<"chttpd_db:handle_changes_req1/2 L75">>,<<"chttpd:handle_request/1 L212">>,<<"mochiweb_http:headers/5 L93">>]
2015-07-12 16:01:01.280 [error]<EMAIL_ADDRESS><0.358.0> httpd 500 error response:
 {"error":"unknown_error","reason":"function_clause","ref":564179134}
2015-07-12 16:01:01.281 [notice]<EMAIL_ADDRESS><0.358.0> cde7e6ea4c <IP_ADDRESS> localhost:59840 POST /db/_changes?filter=_doc_ids 500 ok 2

thanks.

fixed by enhancing couch_httpd's json_body function to match chttpd's which checks for undefined and throws a clean 400 bad request.

@rnewson thank you!


cmake: add lvgl cmake support

Summary
Add LVGL cmake support.
Impact
Testing
cmake -Bbuild -DBOARD_CONFIG=boards/sim/sim/sim/configs/lvgl_fb -GNinja

There's still one issue for cmake is the CONFIG_INIT_ARGS must be set to empty string for config lvgl_fb. The generated config.h is wrong #define CONFIG_INIT_ARGS \widgets\


take new events into account for awaitWriteFinish

I was having issues when listening to a local path which is mounted as a samba share for others to copy files. The problem is that samba reserves the filesize for that transfer and the stats function returns always the full size even with 1% of the transfer completed.
I understand this solution might not be suitable for some other users where the file size is the key point. Probably another config option instead.

Coverage decreased (-0.05%) to 98.255% when pulling e44c17be318617909413f771bcce31e8362efabb on ijliebemij:master into 69a549315cd862f54942c042edfd42edfabadb55 on paulmillr:master.

Actually this seems reasonable to me. Can't think of why it would cause problems for anyone using awaitWriteFinish.
@alexpusch @nono any thoughts?

Looks good to me

Then, can we assume any file size change will always be preceded by an event? If that's the case, we can probably remove the fs.stat on the awaitWriteFinish function, or just the file size comparison in case you want to keep the error check.
If any event on a file is considered and not just the filesize, we can even remove the pollInterval option, as any event could reset the timeout, and unless any other event resets the timeout, the emit will be triggered after the stabilityThreshold.

@ijliebemij
I've done a simple experiment. I copied a 2Gb file into a watched dir:
fs = require('fs')

fs.watch('dir', function(event){
  console.log('event', event);
})

var checkFileSize = function(){
  fs.stat('dir/file', function(err, stats){
    if(stats){
      console.log('stats.size', stats.size)
    }
    setTimeout(checkFileSize, 100)
  })
}

checkFileSize()

Here are the result on OS X:
stats.size 0
event rename
stats.size 59768832
stats.size 97517568
stats.size 135266304

... more size changes

stats.size<PHONE_NUMBER>
stats.size<PHONE_NUMBER>
stats.size<PHONE_NUMBER>
event rename

Results on Ubuntu 15.04 (On Virtual Box)
event change
event change
event change
stats.size 17174528
event change

... more event change

event change
stats.size 22040576
event change

... more event change

event change
stats.size 27394048
event change

... more event change

goes on like this

As we can see the change event triggering is dependent upon the platform, on OS X there is a single event for file addition, and write end (awf is not necessary), and on Ubuntu there seems to be a change event for each filesize change (dough I can't tell for sure)
Maybe there could be a strategy that relay only on fs events, but it will probably won't be very reliable.
Anyway, I think the PR addition is good.

Nice test indeed. Ok, my assumption was not correct then. Current checks seem to be the most consistent.


neo-cli is not showing any logs in v<IP_ADDRESS>

I am running neo-cli using the docker image: microsoft/dotnet:2.0-sdk
my config.json is:
{
  "ApplicationConfiguration": {
    "Paths": {
      "Chain": "/data/chain",
      "ApplicationLogs": "/data/logs"
    },
    "P2P": {
      "Port": 10333,
      "WsPort": 10334
    },
    "RPC": {
      "Port": 10332,
      "SslCert": "",
      "SslCertPassword": ""
    },
    "UnlockWallet": {
      "Path": "",
      "Password": "",
      "StartConsensus": false,
      "IsActive": false
    }
  }
}

where the dir is owned by the user.
I am using the command:
dotnet neo-cli.dll --rpc --log
and there are no logs in the ApplicationLogs directory.
is there a setting for the log level, or a way to make the process log to stdout?

Are you in a private net created by docker? @InfinitePest

Logs was moved to a plugin
https://github.com/neo-project/neo-plugins/blob/eabfd782242bda946311b8016ce4c8979fce05e2/SimplePolicy/SimplePolicyPlugin.cs#L83


openssl dhparam -C generates incompatible code

I am on Debian testing, trying to build Pound against libssl1.0.
Debian testing has helpfully installed the latest openssl command-line tool, version 1.1.0d.
Part of the Pound build process invokes openssl dhparam to generate a couple of .h files. Unfortunately, the code generated is only compatible with libssl1.1, so the Pound build fails. I can make the build work by generating DH parameters on a machine with an older OpenSSL and copying the files over.
Would it be possible/reasonable for openssl dhparam to generate code which uses some clever #define magic to remain backwards compatible with older versions of OpenSSL?

No, the generate command is only compatible with the same runtime.  Sorry.

Fair enough. It seemed like an easy fix to just keep old code around in the template, but I might well be underestimating the maintenance burden.
I'll see about filing a bug against the Debian libssl1.0-dev package then, it's not fully usable without the appropriate openssl binary.
Thank you for the prompt reply!


pkgx.yaml support

is https://docs.pkgx.sh/using-dev/dev#pkgx.yaml supported? afaik it doesnt support shell integration. is it possible to install from a pkgx.yaml and have those binaries usable in your github bin path?

Within a github action? Or some other way? pkgx.yaml files work fine; you need pkgx installed, and you need to call the dev function after pkgx is integrated in your shell.
If you mean using the github actions, you'd want to use the with.+: yaml input attribute to the setup action, or the dev action.
Both are shown in our documentation here: https://docs.pkgx.sh/run-anywhere/ci-cd

Yes use the dev action

so having a pkgx.yaml in my repo
dependencies:
  terraform

this action:
- uses: actions/checkout@v3
- uses: pkgxdev/dev@v1

will pick it up and i can use
steps:
  - run: terraform --version

?

is it possible to have different pkgx.yaml ?


[CNCF]  Is your project in its own separate neutral github organization?

Parts of the CNCF onboarding issue tracked by this issue
This is the list of all bullet points from the CNCF onboarding issue that this issue will track:

[ ] Is your project in its own separate neutral github organization?

So with  -> https://github.com/confidential-containers
My view is we do indeed have a separate neutral github organization so suggesting we tick this task off and move on. I will leave this issue open for a while to allow anyone to correct me or agree.

Agreed.


[FEATURE] add sort_by to ArgillaTrainer

Is your feature request related to a problem? Please describe.
From this PR, it is possible to filter the dataset.
Describe the solution you'd like
I want to use this to filter on submitted records by default and allow for changing this via a sort_by arg during trainer definition and the train method.
Describe alternatives you've considered
N.A.
Additional context
N.A.

@plaguss Are you interested in having a shot at this too?

Sure!


Display number of active cases in the infobox.

Hello. I think it would be valuable to have also a number of active cases displayed, as that's what really determines an immediate impact of the disease on local society. It looks like this:

As you see in the screenshot, I have also made some minor visual adjustments so that the numbers fit into the info column and have thousands separators. Judging by the latest development, it seems we would have needed this additional space for additional orders of magnitude anyway :(

@jakobzhao Given your last commits, I interpret it as not accepting the pull request.

Thanks for your contribution,  similar functions has been provided


chore(grafana): configurable datastore

In grafana dashboard, prometheus datasource is hardcoded. This does fit every installation.
This PR make prometheus datasource a templating variable.

Lgtm, let's align to the semantic commit messages to get it merged!
I'd say chore(grafana): configurable datastore would be good.

I generally see the ${DS_PROMETHEUS} notation but I guess it's ok without brackets

True. Let me check.

@h4wkmoon can we convert this from draft to ready for getting merged?


Invalid FCM token received on iOS

Everything works as expected on android and ios, sending the notification from the firebase console to all devices
(ios and android app).
Trying to address a single device using its token fail due to invalidity:
{   results: [ { error: [FirebaseMessagingError] } ],   canonicalRegistrationTokenCount: 0,   failureCount: 1,   successCount: 0,   multicastId: 172160598... } ERROR:  FirebaseMessagingError: Invalid registration token provided. Make sure it matches the registration token the client app receives from registering with FCM.
Sending to android devices works as expected (both using firebase or my own backend). However sending to iOS devices from my backend throws the error above.
Also I observed, that the device tokens are different. The length of tokens in android is 163, however ios token sizes are 64 characters (this seems to be correct according to other google results).
Edit: I've changed my backend from using the firebase admin-sdk with legacy api to v1 API, with same result =>
{
code: 400,
  errors: [
    {
      message: 'The registration token is not a valid FCM registration token',
      domain: 'global',
      reason: 'badRequest'
    }
  ]
}

react-native: 0.71.2
react-native-notifications: ^4.3.2
Device: iPhone 13 Pro (iOS 16.1.1)

how do translate IOS APNS token loaded by iOs device  to FCM token format ?

@couchounou The token that you retrieve from FCM is translated to the APNS token in Firebase itself. So you don't need to handle any translations from the FCM token to APNS token. Just sending your notification should do the trick.

@seco35 ios devices registration from react-native-notifications get a apns token not a fcm ?

Is it so? Haven't read that in the docs?


Optimus version 2 building

@argenisleon please create a new PR with the changes so travis can do the testing


searchPreferences not working?

Hello, I'm trying to run a CustomSearch with the subtitles filter EgIoAQ%253D%253D
Youtube example (First result have subtitles, second dont)
customSearch = CustomSearch(query = 'test',searchPreferences = 'EgIoAQ%253D%253D',limit = 20)

But the results aren't subtitled. Am I doing something wrong?

Hi there @jpmcarrilho !
I know what you are doing wrong. You need to "unescape" that string before passing.
i.e. "EgIoAQ%253D%253D" is actually "EgIoAQ%3D%3D".


Sprint 2 - Nisarg Shah

Assignment
Points
Grade
Evidence

Pull Request (Integration)
2
1.7

Relevancy Checklist (Peer)
2
0.9

Lesson (Group)
1
0.95

Homework, Popcorn Hacks
1 x 8
0.9

Individual Contribution
1
0.9

Personal Notebooks / Blogs
1
0.9

Total
15

Skill
Points
Grade
Evidence

Work Habits (Analytics)
1

Team Planning (Issue)
1

Presentation Memories
1

Grading and Feedback
1

Beyond Perfunctory
1

Total
5

Looks like you want a bad grade on this.

Looks like you want a bad grade on this.

issue created was just a placeholder as I was working on the finance app combining all java lessons.


Archive failed, error: exit status 65

Last lines of the Xcode's build log:
Code Signing Error: No profile for team 'XXXXXXXXXX' matching 'ABC' found:  Xcode couldn't find any provisioning profiles matching 'XXXXXXXXXX/ABC'. Install the profile (by dragging and dropping it onto Xcode's dock item) or select a different one in the General tab of the target editor.
Warning: The Copy Bundle Resources build phase contains this target's Info.plist file 'ABC/Info.plist'.

Hi @ShaileshTiwari ,
Please run our codesigndoc tool to collect all the signing file on your Mac, and upload all the collected files to bitrise.io , that will help! ;) Related guide: https://devcenter.bitrise.io/ios/code-signing/#collect-the-required-files-with-codesigndoc
Please let us know how it goes or if you'd have any questions! :)

Hi @ShaileshTiwari ,
Did you try the steps I suggested above? Did that work or do you still have this issue?

I got a similar problem since the 2018-04-11.
I think it was around the time I switched to Xcode 9.3.
The certificates still are valid until 2018-05-31.
Archiving, exporting and uploading to the App Store manually / locally with Xcode still works.
The status code back then was 70.
Using codesigndoc didn't help.
Did follow the steps described in the Bitrise documentation.
Occurs with both 1.9.2 and 1.9.3 of the Certificate and Profile installer and 2.4.7 of Xcode Archive & Export for iOS and Xcode 9.3 stack.
Also re-uploading the Distribution and Dev certs and all provisioning profiles I could find on the Apple Dev Center, did not help.
The status code recently changed to 65.
This is the most current log.
Replaced the productname and domain.
The log hints that the automatic signing got deactivated. Maybe it got something to do with that. In my Xcode project, it is enabled though.
+------------------------------------------------------------------------------+

| (0<EMAIL_ADDRESS>                                                    |
+------------------------------------------------------------------------------+
| id: xcode-archive                                                            |
| version: 2.4.7                                                               |
| collection: https://github.com/bitrise-io/bitrise-steplib.git                |
| toolkit: go                                                                  |
| time: 2018-05-08T00:08:52-07:00                                              |
+------------------------------------------------------------------------------+
|                                                                              |
INFO[00:08:52]  * [OK] Step dependency (xcode) installed, available.
INFO[00:08:52]  * [OK] Step dependency (go) installed, available.
ipa export configs:
- ExportMethod: app-store
- UploadBitcode: yes
- CompileBitcode: yes
- TeamID:
- UseDeprecatedExport: no
- CustomExportOptionsPlistContent:
xcodebuild configs:
- OutputTool: xcpretty
- Workdir: /Users/vagrant/git
- ProjectPath: ProductName.xcodeproj
- Scheme: ProductNameStaging
- Configuration:
- OutputDir: /Users/vagrant/deploy
- IsCleanBuild: no
- XcodebuildOptions:
- ForceTeamID:
- ForceProvisioningProfileSpecifier:
- ForceProvisioningProfile:
- ForceCodeSignIdentity:
step output configs:
- IsExportXcarchiveZip: no
- ExportAllDsyms: yes
- ArtifactName: ProductNameStaging
- VerboseLog: yes
step determined configs:
- xcodebuildVersion: 9.3.0 (9E145)
- xcprettyVersion: 0.2.8
Create the Archive ...
[00:08:58] $ set -o pipefail && xcodebuild "-project" "ProductName.xcodeproj" "-scheme" "ProductNameStaging" "archive" "-archivePath" "/var/folders/90/5stft2v13fb_m_gv3c8x9nwc0000gn/T/__archive__540395444/ProductNameStaging.xcarchive" | xcpretty
▸ Building ProductName/ProductName [Staging Release]
▸ Check Dependencies
Last lines of the Xcode's build log:
Check dependencies
Code Signing Error: No profiles for 'com.some-domain.ProductNameStaging' were found:  Xcode couldn't find any iOS App Development provisioning profiles matching 'com.some-domain.ProductNameStaging'. Automatic signing is disabled and unable to generate a profile. To enable automatic signing, pass -allowProvisioningUpdates to xcodebuild.
Code Signing Error: Code signing is required for product type 'Application' in SDK 'iOS 11.3'
** ARCHIVE FAILED **
The following build commands failed:
	Check dependencies
(1 failure)
You can find the last couple of lines of Xcode's build log above, but the full log is also available in the raw-xcodebuild-output.log
The log file is stored in $BITRISE_DEPLOY_DIR, and its full path is available in the $BITRISE_XCODE_RAW_RESULT_TEXT_PATH environment variable
(value: /Users/vagrant/deploy/raw-xcodebuild-output.log)
Archive failed, error: exit status 65
|                                                                              |
+---+---------------------------------------------------------------+----------+
| x |<EMAIL_ADDRESS>(exit code: 1)                            | 12 sec   |
+---+---------------------------------------------------------------+----------+
| Issue tracker: https://github.com/bitrise-io/steps-xcode-archive/issues      |
| Source: https://github.com/bitrise-io/steps-xcode-archive                    |
+---+---------------------------------------------------------------+----------+

After adding the suggested option, I got the following log.
Now it states that my account details are missing.
Still exit status 65
+------------------------------------------------------------------------------+

| (0<EMAIL_ADDRESS>                                                    |
+------------------------------------------------------------------------------+
| id: xcode-archive                                                            |
| version: 2.4.7                                                               |
| collection: https://github.com/bitrise-io/bitrise-steplib.git                |
| toolkit: go                                                                  |
| time: 2018-05-08T00:43:47-07:00                                              |
+------------------------------------------------------------------------------+
|                                                                              |
INFO[00:43:48]  * [OK] Step dependency (xcode) installed, available.
INFO[00:43:48]  * [OK] Step dependency (go) installed, available.
ipa export configs:
- ExportMethod: app-store
- UploadBitcode: yes
- CompileBitcode: yes
- TeamID:
- UseDeprecatedExport: no
- CustomExportOptionsPlistContent:
xcodebuild configs:
- OutputTool: xcpretty
- Workdir: /Users/vagrant/git
- ProjectPath: ProductName.xcodeproj
- Scheme: ProductNameStaging
- Configuration:
- OutputDir: /Users/vagrant/deploy
- IsCleanBuild: no
- XcodebuildOptions: -allowProvisioningUpdates
- ForceTeamID:
- ForceProvisioningProfileSpecifier:
- ForceProvisioningProfile:
- ForceCodeSignIdentity:
step output configs:
- IsExportXcarchiveZip: no
- ExportAllDsyms: yes
- ArtifactName: ProductNameStaging
- VerboseLog: yes
step determined configs:
- xcodebuildVersion: 9.3.0 (9E145)
- xcprettyVersion: 0.2.8
Create the Archive ...
[00:43:54] $ set -o pipefail && xcodebuild "-project" "ProductName.xcodeproj" "-scheme" "ProductNameStaging" "archive" "-archivePath" "/var/folders/90/5stft2v13fb_m_gv3c8x9nwc0000gn/T/__archive__072373204/ProductNameStaging.xcarchive" "-allowProvisioningUpdates" | xcpretty
▸ Building ProductName/ProductName [Staging Release]
▸ Check Dependencies
Last lines of the Xcode's build log:
Code Signing Error: There are no accounts registered with Xcode. Add your developer account to Xcode
Code Signing Error: No profiles for 'com.some-domain.ProductNameStaging' were found:  Xcode couldn't find any iOS App Development provisioning profiles matching 'com.some-domain.ProductNameStaging'.
Code Signing Error: Code signing is required for product type 'Application' in SDK 'iOS 11.3'
** ARCHIVE FAILED **
The following build commands failed:
	Check dependencies
(1 failure)
You can find the last couple of lines of Xcode's build log above, but the full log is also available in the raw-xcodebuild-output.log
The log file is stored in $BITRISE_DEPLOY_DIR, and its full path is available in the $BITRISE_XCODE_RAW_RESULT_TEXT_PATH environment variable
(value: /Users/vagrant/deploy/raw-xcodebuild-output.log)
Archive failed, error: exit status 65
|                                                                              |
+---+---------------------------------------------------------------+----------+
| x |<EMAIL_ADDRESS>(exit code: 1)                            | 14 sec   |
+---+---------------------------------------------------------------+----------+
| Issue tracker: https://github.com/bitrise-io/steps-xcode-archive/issues      |
| Source: https://github.com/bitrise-io/steps-xcode-archive                    |
+---+---------------------------------------------------------------+----------+

I have similar issue:

Last lines of the Xcode's build log:
Check dependencies
Code Signing Error: There are no accounts registered with Xcode. Add your developer account to Xcode
Code Signing Error: No signing certificate "iOS Development" found:  No "iOS Development" signing certificate matching team ID "XXXXXXX" with a private key was found.
** ARCHIVE FAILED **

Hi @bclausdorff  & @malashkevich - can you please create separate issues?
Closing this thread as it starts to get confusing to follow. Sorry for the inconvenience.

Hey ! I have the same issue here as original issue mentionned. No idea.

Closing this thread as it starts to get confusing to follow and the original question does not have enough information to have a simple reply (see #106 (comment) above). Sorry for the inconvenience.

Sorry @viktorbenei your link is down :(

@jhanzo indeed that was moved - up to date guides can be found at: https://devcenter.bitrise.io/code-signing/ios-code-signing/collecting-files-with-codesigndoc/ (if you're referring to the link in https://github.com/bitrise-io/steps-xcode-archive/issues/106#issuecomment-379796067 )


Add Rule option "disallowedFor" for component prop

Suppose we have a component that accept a prop named string.
function Comp(props) {
    return
{props.string}
;
}

function App() {
    return <Comp string="213"  />;
}

As needs change, we need to replace string prop by another named number.

function Comp(props) {
    return
{props.number || props.string}
;
}
Comp.propTypes = {
    number: PropTypes.number,
    /**
     * @deprecated use number instead
     */
    string: PropTypes.string
};
Comp.defaultProps = {
    string: '123',
    number: 123
};

The string prop will works in a period of time, but it's better to replace it with number prop.
This rule option can be use to avoid using deprecated prop for Comp.
{
"react/forbid-component-props": [
    "error", { "forbid": [{"propName": "string", "disallowedList": ["Comp"]}] }
  ]
}

Since it's marked as deprecated with jsdoc, there's eslint-plugin-deprecated - and if you're using TS, it will catch it too.
Why can't you just remove it and make a major bump?

Hello, I would also like this: my use case is for discouraging the usage of props in components that I don't control, and thus can't annotate or use the component myself.
I could wrap it in an HOC and use that instead, but that doesn't prevent others from using the original component, so it doesn't really get me very far. I also don't want to create and HOC and than ban the original altogether, because that would be a much larger change for not a lot of benefit.

@cincodenada you can prevent them from using the original component using no-restricted-imports and/or no-restricted-properties, fwiw.
That said, thanks, that is a reasonable use case.


Fix unbound child bones of the PhysicalBone aren't modified

Fixes https://github.com/godotengine/godot/issues/91606.
PhysicalBoneSimulator caches all bones and modify only bound bones, but it stores only the global pose, so unbound bones will not be modified since it doesn’t process recursively.
There is no need to change the child bones global pose recursively to fix the problem. The issue is that the skeleton reflects bones that are not moving, so ignoring the unbound bones fixes the problem.

It seems that the problem is not completely solved.
https://github.com/godotengine/godot/assets/48926464/307b1ac0-4ec0-4fad-85cc-56681b55391c
testragdoll.zip

The problem appears to be dependent on the update order of bone.
At any rate, I knew how to make it work correctly, and since this is a bad regression, I applied it. But the known problem is that the current way of updating the global pose of the Skeleton is not good performance-wise, so a follow up is definitely needed later.
We have already sometimes discussed this within the animation team before, but ideally there should be a fundamental solution since any SkeletonModifier may have this performance issue in the future.

Thanks!


Using playground as base for ghpage

@all-contributors please add @MatthiasKainer as contributor for code and tests


How to inference based on export_hf result?

Results with export_hf_checkpoint.py:
├── config.json
├── generation_config.json
├── pytorch_model-00001-of-00007.bin
├── pytorch_model-00002-of-00007.bin
├── pytorch_model-00003-of-00007.bin
├── pytorch_model-00004-of-00007.bin
├── pytorch_model-00005-of-00007.bin
├── pytorch_model-00006-of-00007.bin
├── pytorch_model-00007-of-00007.bin
└── pytorch_model.bin.index.json
While convert llama to hf format as follows:
├── config.json
├── generation_config.json
├── pytorch_model-00001-of-00007.bin
├── pytorch_model-00002-of-00007.bin
├── pytorch_model-00003-of-00007.bin
├── pytorch_model-00004-of-00007.bin
├── pytorch_model-00005-of-00007.bin
├── pytorch_model-00006-of-00007.bin
├── pytorch_model-00007-of-00007.bin
├── pytorch_model.bin.index.json
├── special_tokens_map.json
├── tokenizer_config.json
└── tokenizer.model
Is the result of export_hf correct ? As missing special_tokens_map.json、tokenizer_config.json、tokenizer.model files，inference with from_pretrained from export_hf's result occurs error:
tokenizer = LlamaTokenizer.from_pretrained(export_hf_dir)
model =  LlamaForCausalLM.from_pretrained(export_hf_dir, torch_dtype="auto", device_map="auto")
OSError: Can't load tokenizer for '/xx/export_hf_dir/'. If you were trying to load it from 'https://huggingface.co/models', make sure you don't have a local directory with the same name. Otherwise, make sure '/xx/export_hf_dir/' is the correct path to a directory containing all relevant files for a LlamaTokenizer tokenizer
Should I copy special_tokens_map.json、tokenizer_config.json、tokenizer.model files from llama path to export_hf path?

The result appears correct.
You can convert it to f16 then to 4bit using llama.cpp if you want: https://github.com/ggerganov/llama.cpp (which can run the 4 bit model too)

@philwee Thanks for your replay! Is it possible to run with transforms's from_pretrained method  directory? Like  this:
tokenizer = LlamaTokenizer.from_pretrained(export_hf_dir)
model = LlamaForCausalLM.from_pretrained(export_hf_dir, torch_dtype="auto", device_map="auto")

Yup, you can:
For example:
import transformers
from transformers import LlamaForCausalLM, LlamaTokenizer

tokenizer = LlamaTokenizer.from_pretrained('hf_ckpt')
model = LlamaForCausalLM.from_pretrained('./hf_ckpt', torch_dtype="auto", device_map="auto")

input_text = "What is the meaning of life?"
input_ids = tokenizer.encode(input_text, return_tensors='pt')

output_ids = model.generate(input_ids)
output_text = tokenizer.decode(output_ids[0], skip_special_tokens=True)

print(output_text) #output: What is the meaning of life? This is a question that has been asked by many people throughout

@philwee I tried the code above, same problem as mentioned in the question:
OSError: Can't load tokenizer for './hf_ckpt/'. If you were trying to load it from 'https://huggingface.co/models', make sure you don't have a local directory with the same name. Otherwise, make sure './hf_ckpt/' is the correct path to a directory containing all relevant files for a LlamaTokenizer tokenizer
I think missing these files in hf_ckpt: special_tokens_map.json、tokenizer_config.json、tokenizer.model files. How can i get these files and put them into  result of export_hf_checkpoint.py?

I think the simplest way would be to download them from the huggingface repo where you got your llama from

@philwee I tried the code above, same problem as mentioned in the question: OSError: Can't load tokenizer for './hf_ckpt/'. If you were trying to load it from 'https://huggingface.co/models', make sure you don't have a local directory with the same name. Otherwise, make sure './hf_ckpt/' is the correct path to a directory containing all relevant files for a LlamaTokenizer tokenizer I think missing these files in hf_ckpt: special_tokens_map.json、tokenizer_config.json、tokenizer.model files. How can i get these files and put them into result of export_hf_checkpoint.py?

You need to copy these three files (tokenizer.model, tokenizer_config.json and special_tokens_map.json) from the BASE_MODEL to './hf_ckpt'. The BASE_MODEL is the one you set when executing 'export_hf_checkpoint.py' and its default value is ‘huggyllama/llama-7b’. When you use this default value, you can find its downloaded model in the following path:

for Linux user:    ~/.cache/huggingface/hub/models--huggyllama--llama-7b/snapshots/
for Windows user:    $HOME.cache\huggingface\hub\models--decapoda-research--llama-7b-hf\snapshots\

When you copy these three files to './hf_ckpt', you can run the finetune.py like this:
python finetune.py --base_model './hf_ckpt' --data_path '<path to data json file>' --output_dir './lora-alpaca'


Missing CFBundleVersion in Info.plist of frameworks

If framework dependencies are build by generating a Xcode project with the swift package manager, then the CFBundleVersion key will not be set which causes app store distribution problems.

@mrylmz Shouldn't we also introduce a similar handling for CFBundleShortVersionString in case it is missing? This key is also required for app store submissions...


removing unnecessary record overload

when passing in a record object to tropo.record an overload is completely unnecessary and breaks the ability to create records with transcription objects

@gradin Thank you your pull request, I think may be we could keep this overload for compatible with old codes of customers. And could you give an example of breaks the ability to create records with transcription objects?
How about the solution: Add an annotation to the method using the keyword Obsolete.

Close in almost 1 week no response.


Reduce the memory consumed by Gradle

.. so that Travis CI builds do not fail unexpectedly.

Hmm.. failed again :confused:

🤔🤔🤔

Problem not solved by this patch. Closing for now.. :cry:


Failed to establish a new connection: [Errno 111]

Hi,
I tried a simple command with python3 cloudfail.py --target  but it fails, here's a copy:
[10:25:17] Fetching initial information from: themebanks.com...
[10:25:17] No ipout file found, fetching data
[10:25:17] Just checking for updates, please wait...
[10:25:17] Updating CloudFlare subnet...
[10:25:17] Updating Crimeflare database...
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/urllib3/connection.py", line 159, in _new_conn
    (self._dns_host, self.port), self.timeout, **extra_kw)
  File "/usr/local/lib/python3.6/site-packages/urllib3/util/connection.py", line 80, in create_connection
    raise err
  File "/usr/local/lib/python3.6/site-packages/urllib3/util/connection.py", line 70, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 600, in urlopen
    chunked=chunked)
  File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 354, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/lib64/python3.6/http/client.py", line 1254, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib64/python3.6/http/client.py", line 1300, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib64/python3.6/http/client.py", line 1249, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib64/python3.6/http/client.py", line 1036, in _send_output
    self.send(msg)
  File "/usr/lib64/python3.6/http/client.py", line 974, in send
    self.connect()
  File "/usr/local/lib/python3.6/site-packages/urllib3/connection.py", line 181, in connect
    conn = self._new_conn()
  File "/usr/local/lib/python3.6/site-packages/urllib3/connection.py", line 168, in _new_conn
    self, "Failed to establish a new connection: %s" % e)
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPConnection object at 0x7f90fa9ed4e0>: Failed to establish a new connection: [Errno 111] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 638, in urlopen
    _stacktrace=sys.exc_info()[2])
  File "/usr/local/lib/python3.6/site-packages/urllib3/util/retry.py", line 399, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPConnectionPool(host='crimeflare.net', port=83): Max retries exceeded with url: /domains/ipout.zip (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f90fa9ed4e0>: Failed to establish a new connection: [Errno 111] Connection refused',))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "cloudfail.py", line 309, in <module>
    init(args.target)
  File "cloudfail.py", line 138, in init
    update()
  File "cloudfail.py", line 252, in update
    r = requests.get("http://crimeflare.net:83/domains/ipout.zip", stream=True)
  File "/usr/local/lib/python3.6/site-packages/requests/api.py", line 76, in get
    return request('get', url, params=params, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/requests/api.py", line 61, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.6/site-packages/requests/sessions.py", line 655, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/requests/adapters.py", line 516, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='crimeflare.net', port=83): Max retries exceeded with url: /domains/ipout.zip (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f90fa9ed4e0>: Failed to establish a new connection: [Errno 111] Connection refused',))```

Thanks for your help

Same issue here ... :(
Is the project dead?

I'm having the same issue. I doubt the project is dead. My guess is that some API key is misconfigured. I'll see what I can find.

Looking up crimeflare showed that the sites are no longer up.


Z-Order is broken

http://blogs.u2u.be/diederik/post/2014/02/23/A-Floating-Control-for-Windows-8-Store-apps.aspx

@felixrieseberg Is this related to the z-order we talked about the other day? I think I got it to work now.

That'd be awesome. Even if not, I'd love a sane PR. If you want to check if this issue will remain relevant, give the test app a background color. If the control disappears, we still need to fix it.
Don't let that hold you back on the icon/label position thing though!

This is now fixed.


Unwrapped AES keys have key_length set to 0

I'm not sure if I'm doing something wrong here, so this could be my fault.  I have some AES keys that were wrapped by another HSM that I'm trying to unwrap into SoftHSM.  I'm using RSA_PKCS_OAEP and AES_KEY_WRAP_PAD as the mechanisms for the wrapping.  The unwrap process seems to complete correctly and I get back a pkcs11.Key subclass, but key_length is set to 0:
            example_unwrapping_test = example_wrapping_softhsm.unwrap_key(pkcs11.ObjectClass.SECRET_KEY, pkcs11.KeyType.AES, example_blob, mechanism=pkcs11.Mechanism.RSA_PKCS_OAEP, store=True, label='example-unwrapping-test', template={
            pkcs11.Attribute.SENSITIVE: False,
            pkcs11.Attribute.EXTRACTABLE: True,
        })
        except Exception as error:
            print(error)
            print(vars(error))
            raise
        print(example_unwrapping_test)
        print(vars(example_unwrapping_test))
        print(len(example_unwrapping_test[pkcs11.Attribute.VALUE]))

<SecretKey label='example-unwrapping-test' id='' 0-bit AES>
{'session': <pkcs11._pkcs11.Session object at 0x7f0b2c8cd1d0>, '_handle': 8, 'label': 'example-unwrapping-test', 'id': b'', 'key_length': 0, 'key_type': <KeyType.AES>, '_key_description': '0-bit AES'}
32

Attribute.VALUE of the test key is the right length (32) and does have the same value as the test key before it was wrapped.  I tried passing in key_length to unwrap_key and got an error, so it doesn't seem like I can set it directly.  Do I have to set a value in the template somehow?

key_length comes from the Attribute VALUE_LEN, which could be wrong, but it could also be a caching bug. Can you test what Attribute.VALUE_LEN returns for you?

Attribute.VALUE_LEN is also 0.

Okay so that's coming from the HSM. There's not a lot to be done here from our side.

I posted this question to the SoftHSM people, https://github.com/opendnssec/SoftHSMv2/issues/628, and they told me to try passing CKA_VALUE_LEN in the template.  When I tried, I got this error:
                secret=self.private.unwrap_key(
                    # RSA OAEP can only wrap symmetric keys
                    pkcs11.ObjectClass.SECRET_KEY,
                    pkcs11.KeyType.AES,
                    blob_file.read(),
                    mechanism=pkcs11.Mechanism.RSA_PKCS_OAEP,
                    store=True,
                    label=file_name + "-unwrapping-test",
                    # Test template
                    template={
                        pkcs11.Attribute.SENSITIVE: False,
                        pkcs11.Attribute.EXTRACTABLE: True,
                        pkcs11.Attribute.VALUE_LEN: 256,
                    },

Traceback (most recent call last):
  File "offline_example.py", line 542, in <module>
    'example-wrapped',
  File "offline_example.py", line 307, in unwrap_key
    pkcs11.Attribute.VALUE_LEN: 256,
  File "pkcs11/_pkcs11.pyx", line 1276, in pkcs11._pkcs11.UnwrapMixin.unwrap_key
  File "pkcs11/_errors.pyx", line 88, in pkcs11._pkcs11.assertRV
pkcs11.exceptions.AttributeReadOnly

I don't think this seems like correct behavior, though if you think it's SoftHSM that's misbehaving, I can go back to them.

The response code also comes from the HSM.

Thank you for your patience.  I did get SoftHSM to confirm the problems are on their end.


Test informalsystems/cosmos.nix#64

Description
This is a test branch for running integration tests with the new Go packages built in https://github.com/informalsystems/cosmos.nix/pull/64.

PR author checklist:

[ ] Added changelog entry, using unclog.
[ ] Added tests: integration (for Hermes) or unit/mock tests (for modules).
[ ] Linked to GitHub issue.
[ ] Updated code comments and documentation (e.g., docs/).

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

@romac with https://github.com/informalsystems/cosmos.nix/pull/64 merged, we can now enable back Gaia v6 on the CI by pinning directly to v6.0.2 instead of v6.0.3.