Never allow an empty password to validate

#6023 caused a regression.

Maybe add a test?

Does not seem to work, all tests fail

Yeah, so the reason tests failed is because I wrote:
// IsPasswordSet checks if the password is set or left empty
func (u *User) IsPasswordSet() bool {
       return u.ValidatePassword("")
}

instead of:
// IsPasswordSet checks if the password is set or left empty
func (u *User) IsPasswordSet() bool {
       return !u.ValidatePassword("")
}

yes, I'm an eejit.

Codecov Report

Merging #9682 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #9682      +/-   ##
=========================================
- Coverage    42.2%   42.2%   -0.01%
=========================================
  Files         591     591
  Lines       78085   78085
=========================================
- Hits        32959   32953       -6
- Misses      41080   41085       +5
- Partials     4046    4047       +1

Impacted Files
Coverage Δ

models/user.go
50.77% <100%> (ø)
:arrow_up:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

modules/process/manager.go
74.69% <0%> (-3.62%)
:arrow_down:

services/pull/temp_repo.go
31.62% <0%> (-2.57%)
:arrow_down:

modules/queue/workerpool.go
39.91% <0%> (-1.29%)
:arrow_down:

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

services/pull/check.go
53.37% <0%> (+4.72%)
: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 32fb813...465caef. Read the comment docs.


Implement stat

I'm working on this.

Are you still working on it? I'd like to take it up if you aren't.

Hi,
I was almost finished, but did not have the time to keep up with the rust
changes.
It's yours.
On Tue, Apr 28, 2015 at 8:25 AM, Divyanshu Kakwani<EMAIL_ADDRESS>
wrote:

Are you still working on it? I'd like to take it up if you aren't.
—
Reply to this email directly or view it on GitHub
https://github.com/uutils/coreutils/issues/378#issuecomment-96942648.

#893


Move Figure 55 (Historical assessments) to somewhere NOT next to retrospectives

Jaclyn noted that it is difficult to know that the historical analysis of previous assessments is not a retrospective yet when placed next to the retrospectives it is slightly difficult to interpret.

I think if we end up shortening an moving the Performance of Past Projections section to an appendix, we could then have section 3.10 by something like "Historical Performance" and just move the pertinent paragraphs from the Retrospective section to the new 3.10 section.

Figure 55 was (in the draft last year) this one

Jaclyn's correct that the text falls under the 'retrospective analyses' section, which is indeed confusing. So we should simply just add in a new section heading "Comparison with past assessments".
Regarding 'Performance of past projections', I can shorten the text, and remove Figures 60-63, which help the understanding of Figure 64 (but we can just refer to previous assessment for the more detailed explanation). I find them really interesting but not sure other readers do! Kind of think there's not really enough to move it back into an Appendix, once it's just four figures and some text (Trevor and Shannon appreciate them).

It's still a retrospective analysis in a sense, though perhaps not what most assessment scientists think of when they hear retrospective analysis.  Changing to a new section is fine too.


ACL Anthology RSS: Go through the setup, get familiar with Hugo

Description
For https://github.com/acl-org/acl-anthology/issues/358 I first need to get familiar with the codebase and the Hugo framework. This issue covers all the necessary steps to set up the repo and have a working pipeline from code to HTML pages.
DoD

[ ] Build the static HTML pages locally.
[ ] Run a local web server for testing.

Update: Using the Makefile, I get to the point where Hugo builds the pages, but then the process crashes. It might be a memory issue because I don't have enough swap space. I'll have to investigate this tomorrow.

Using a machine with more RAM, the setup worked fine out of the box.


Enable-WriteCore-feature-for-playwrighttesting

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.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.PlaywrightTesting


doc decorator in macros.rs has typo

https://github.com/SergioBenitez/Pear/blob/80112eedec7809c59e525f6852490c49856a6550/lib/src/macros.rs#L79
This should be "hidden", not "hiden".  This now fails on nightly rust.

Fixed in d137aa1df314e2e199470e1f54b931bf859ade89.


Owen Name

Owner name

Why:

Closes [issue link]
What's being changed:

Check off the following:

[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

Owner name
Why:
Closes [issue link]
What's being changed:
Check off the following:

[x]  I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x]  For content changes, I have reviewed the localization checklist
[x]  For content changes, I have reviewed the Content style guide for GitHub Docs.

-Help out

[x] ````


npm start stop running after a 404 for an image

A tweet I retweeted has an image, when I run npm start I get the following error and it won't continue (or at least it appears not to, I have left it quite a while):
Image request error Bad response for https://pbs.twimg.com/media/FCDPpZ4WYAE0QWi.jpg (404): Not Found
The tweet exists: https://twitter.com/prekesh/status/1450387420210417665 and the image is still there, but none of the media links in the JSON actually work.
The full tweet json:
{
    "tweet" : {
      "edit_info" : {
        "initial" : {
          "editTweetIds" : [
            "1450388063818076162"
          ],
          "editableUntil" : "2021-10-19T09:37:15.978Z",
          "editsRemaining" : "5",
          "isEditEligible" : true
        }
      },
      "retweeted" : false,
      "source" : "Twitter Web App",
      "entities" : {
        "user_mentions" : [
          {
            "name" : "prekesh",
            "screen_name" : "prekesh",
            "indices" : [
              "3",
              "11"
            ],
            "id_str" : "39297909",
            "id" : "39297909"
          }
        ],
        "urls" : [ ],
        "symbols" : [ ],
        "media" : [
          {
            "expanded_url" : "https://twitter.com/prekesh/status/1450387420210417665/photo/1",
            "source_status_id" : "1450387420210417665",
            "indices" : [
              "57",
              "80"
            ],
            "url" : "https://t.co/vzoxxogQuh",
            "media_url" : "http://pbs.twimg.com/media/FCDPpZ4WYAE0QWi.jpg",
            "id_str" : "1450387390242119681",
            "source_user_id" : "39297909",
            "id" : "1450387390242119681",
            "media_url_https" : "https://pbs.twimg.com/media/FCDPpZ4WYAE0QWi.jpg",
            "source_user_id_str" : "39297909",
            "sizes" : {
              "medium" : {
                "w" : "1156",
                "h" : "771",
                "resize" : "fit"
              },
              "thumb" : {
                "w" : "150",
                "h" : "150",
                "resize" : "crop"
              },
              "large" : {
                "w" : "1156",
                "h" : "771",
                "resize" : "fit"
              },
              "small" : {
                "w" : "680",
                "h" : "454",
                "resize" : "fit"
              }
            },
            "type" : "photo",
            "source_status_id_str" : "1450387420210417665",
            "display_url" : "pic.twitter.com/vzoxxogQuh"
          }
        ],
        "hashtags" : [ ]
      },
      "display_text_range" : [
        "0",
        "80"
      ],
      "favorite_count" : "0",
      "id_str" : "1450388063818076162",
      "truncated" : false,
      "retweet_count" : "0",
      "id" : "1450388063818076162",
      "possibly_sensitive" : false,
      "created_at" : "Tue Oct 19 09:07:15 +0000 2021",
      "favorited" : false,
      "full_text" : "RT @prekesh: me as the new macbooks were being announced https://t.co/vzoxxogQuh",
      "lang" : "en",
      "extended_entities" : {
        "media" : [
          {
            "expanded_url" : "https://twitter.com/prekesh/status/1450387420210417665/photo/1",
            "source_status_id" : "1450387420210417665",
            "indices" : [
              "57",
              "80"
            ],
            "url" : "https://t.co/vzoxxogQuh",
            "media_url" : "http://pbs.twimg.com/media/FCDPpZ4WYAE0QWi.jpg",
            "id_str" : "1450387390242119681",
            "source_user_id" : "39297909",
            "id" : "1450387390242119681",
            "media_url_https" : "https://pbs.twimg.com/media/FCDPpZ4WYAE0QWi.jpg",
            "source_user_id_str" : "39297909",
            "sizes" : {
              "medium" : {
                "w" : "1156",
                "h" : "771",
                "resize" : "fit"
              },
              "thumb" : {
                "w" : "150",
                "h" : "150",
                "resize" : "crop"
              },
              "large" : {
                "w" : "1156",
                "h" : "771",
                "resize" : "fit"
              },
              "small" : {
                "w" : "680",
                "h" : "454",
                "resize" : "fit"
              }
            },
            "type" : "photo",
            "source_status_id_str" : "1450387420210417665",
            "display_url" : "pic.twitter.com/vzoxxogQuh"
          }
        ]
      }
    }
  }

I stand corrected, it IS still running it's just taking a while (first run still downloading images). Apologies 👋.


Autocomplete shows snippet in last position

The autocomplete shows my snippets in last position. Is that normal?

Yes. Its an atom thing, not really and atom-typescript thing :rose:


Remove GroupBy for Find Count queries

An update for PR #150 as it wasn't clearing groupBy introduced by $modify queries

It fixed the tests params.modifierFiltersResults=true applies count from modify query and params.modifierFiltersResults=undefined applies count from modify query as the query response for both tests was as follow :
{
  total: 1,
  limit: 2,
  skip: 0,
  data: [
    Company {
      id: 41,
      name: 'Google',
      ceo: 1,
      size: 'medium',
      jsonObject: null,
      jsonArray: null,
      jsonbObject: null,
      jsonbArray: null,
      employees: [Array]
    },
    Company {
      id: 42,
      name: 'Apple',
      ceo: null,
      size: 'large',
      jsonObject: null,
      jsonArray: null,
      jsonbObject: null,
      jsonbArray: null,
      employees: [Array]
    }
  ]
}

Clearing the groupBy for the count query fixes the issue

Hi @alex-all3dp , can you please review this PR? it may break the behavior you've added here.
@gmercey I'm reverting the previous PR until this PR is resolved.

@alex-all3dp with which DB Engine do you get the error? Maybe we need to have different behaviour depending on the engine used?

@alex-all3dp with which DB Engine do you get the error? Maybe we need to have different behaviour depending on the engine used?

@gmercey PostgreSQL

We could wrap the clear('groupBy) around if (params.modifierFiltersResults !== false) to accommodate? Because the example you are giving is not a proper count query for pagination as it returns a non-aggregated column whereas the count query should return 1 value

We could wrap the clear('groupBy) around if (params.modifierFiltersResults !== false) to accommodate? Because the example you are giving is not a proper count query for pagination as it returns a non-aggregated column whereas the count query should return 1 value

I guess that could work :)

@dekelev @alex-all3dp PR updated

@gmercey The if clause does not fix the issue. My bad, I didn't look closely enough the first time.
So also without the if clause my find queries with modifiers still work as expected. The problem I am having on this branch is that service.get with a.modifier (GET /entity/:id?$modify=myModifier)  throws the error message. Any ideas?

@alex-all3dp is it failing because the service.get is a wrapper for _find :
  _get (id, params = {}) {
    // merge user query with the 'id' to get
    const findQuery = Object.assign({}, { $and: [] }, params.query);
    findQuery.$and.push(this.getIdsQuery(id)); // BUG will fail with composite primary key because table name will be missing

    return this._find(Object.assign({}, params, { query: findQuery }))
      .then(page => {
        const data = page.data || page;

        if (data.length !== 1) {
          throw new errors.NotFound(`No record found for id '${id}'`);
        }

        return data[0];
      });
  }

What we should do in this case, is to pass paginate=false as a param because we don't care about pagination in that case.


[ENH] Speed up read_raw_neuralynx() on large datasets with many gaps

Reference issue
Fixes #12370.
What does this implement/fix?
Drop np.where and compute onsets via np.cumsum Instead of using np.where to iterative over gap indices and search over the recording samples, in this PR I compute all segment start samples by cumulatively summing segment sample sizes (see this change: 5b40d2).
This gives starting indices for all segments, then I select only those corresponding to gaps. Doing it this way, yields substantial improvements over using np.where as confirmed on a a few local datasets with increasing numbers of gaps (see Figure in #12370)
Additional information
Any additional information you think is important.

I don't think a changelog entry is needed for this so I'll label it as such, but feel free to add one if you want @KristijanArmeni (probably as 12371.newfeature.rst since this is an enhancement)

Thanks @larsoner! Added a changelog entry as suggested.

pip-pre failures are unrelated, I opened #12372 to track it

@larsoner just pinging here that it looks like this was not auto-merged (it was automatically disabled after my changelog commit). But I see pip-pre fails are now sorted and main is passing so I think just this just needs an update from main and should be all green.

Thanks for the ping @KristijanArmeni, merged main into this branch reenabled auto-merge, and will keep an eye on it to make sure it lands today!


Refactor ensuring sync parent execution

This resolves #64 ensuring that the execution of synchronous modules, whose execution can only begin after deeper asynchronous dependencies have already completed, remains synchronous with their parent modules.
This ensures that adding an asynchronous dependency to a module does not change the execution of that module which could be a source of bugs during application development - adding a new and unrelated dependency should not change the timing of the modules already in the graph.
Consider the following example:
main.js
import './a.js';
import './b.js';
console.log('main');

where a.js contains:
console.log('a');
Promise.resolve().then(() => {
  console.log('a task');
});

and identically for b.js.
On execution we get "a", "b", "main" synchronously in order, followed by "a task" then "b task".
Now if we introduce a new dependency to this application which is async:
main.js
import './a.js';
import './b.js';
import './async.js';
console.log('main');

with async.js containing:
console.log('async start');
await new Promise(resolve => setImmediate(resolve));
console.log('async end');

where async.js contains a top-level await, the entire execution timing changes to instead read - "a", "b", "a task", "b task", "async start", "async end", "main".
The issue here is that introducing a dependency can now alter the interrelated timings of existing modules, which may introduce execution bugs. Typically we don't think of introducing a dependency as something that can drastically change the execution semantics of our existing code! This effect is also viral in that all modules importing this "main" file will also have microtasks now being run on their individual completions too. No module can execute synchronously with main anymore.
The approach taken in this PR is to defer the execution of synchronous dependencies to happen after the async modules have resolved, instead of immediately.
The result is that in the example above we get the following execution - "async start", "async end", "a", "b", "main", where the execution of "a", "b", "main" remains identical in semantics to the execution of "a", "b", "main" before we added the asynchronous dependency. In addition all parent importers of main will continue to execute according to their previous execution timing as well, avoiding the execution timing difference propagating up al modules in the graph. What this means is that introducing an asynchronous dependency will no longer risk drastically changing the execution semantics of our existing code from as it was before we added the asynchronous dependency.
The final approach turned out to be quite simple to implement.Circular references and error handling invariants are maintained in that cycles transition through all states together, and errors stop further execution of synchronous modules, while also throwing into both sync and async cycles as a single strongly connected component transition.

Another benefit of this approach is that by delaying synchronous executions to end in order with async executions means that the upfront execution of the graph will yield to the event loop sooner. As a result this execution change should lead to less jank and upfront CPU cost.

Doesn’t this mean that a polyfill in a could be defeated/prevented by (or almost as bad, not applied to) code in async? It seems very concerning to have async deps’ evaluation hoisted above the evaluation of sync deps that appear lexically before the async one.

Yes, async polyfills would need to be sync.
This should be weighed up against the fact that adding an async dependency will alter existing execution timing (an npm package dependency adding an async dependency in an upgrade path will change your own app executing timing, possibly introduing bugs), the cost of Rollup code behaving differently to unbundled code (the build will cause execution timing changes), , as well as the performance benefits (much less upfront CPU work since we are not firing off every execution we can, but instead yielding the event loop letting the async dependencies finish before hitting the dependent sync module CPU cost).
In addition the two bug scenarios above are not easily debuggable - and will appear to users as "random breaks". In my opinion this is the worst type of bug and far worse than a predictable bug where you can add a await 0 at the top of your polyfill to support async modules.

I think in your above example, preserving relative ordering among sync deps is much less important than ensuring that lexical evaluation order be retained - imo, nothing should be evaluated prior to a.js, in any of your examples, so “a” and “b” must always be logged before anything else.

Sure, I can appreciate that the relative ordering may seem surprising here despite the benefits.


API specification

This commit implements the API specification.
The specification can be read easily on
http://kairo.eti.br/ownca-rest-api/ or running the local HTTP
API locally, after merge it will be available on
http://github.io/ownca-rest-api/ as well.
pipenv shell
pipenv install
flask run

This commit has no functional implementation, and the idea is to
review the API specification until it gets ready and then create
the issues related to the implementation.
 I really enjoy seeing how you're constructing the flask API with flask-restx and the docs with swagger. I've never ran across those before, but it's a very cohesive setup for developers. Thanks for the learning opportunity :)

You're a coding machine haha! I really enjoy seeing how you're constructing the flask API with flask-restx and the docs with swagger. I've never ran across those before, but it's a very cohesive setup for developers. Thanks for the learning opportunity :)

Hahaha. I would like to have more time. 😀
That is the idea here, have fun, learn and leave something to help others. 😁
Thank you.

Thanks @lawndoc for reviewing.


Update version to 1.2.87-1 on production

Update version to 1.2.87-1

Hey! I see that you made changes to our Form component. Make sure to update the docs in FORMS.md accordingly. Cheers!


docs: Add vision-camera-base64 plugin to list

What

Changes

Tested on

Related issues

Hey thanks - can you upgrade your plugin to VisionCamera V3?


feat: Support relations on module models.

Changes
Not yet completed as we have to decide how we want to deal with 1:n relations to modules.

Adds loading and parsing of module model files and includes them in the validation of project models.
Allows to reference module tables by name
Allows to reference module datatypes and add a relation
Allows to reference module datatypes as a list (this will break at runtime)

This PR will enable us to implement type-checking of fields in the yaml files.
Example via model datatype:
class: ObjectUser
table: object_user
fields:
  name: String?
  userInfo: module:auth:UserInfo?, relation

Example via parent keyword:
class: ParentUser
table: parent_user
fields:
  name: String?
  userInfoId: int?, relation(parent=serverpod_user_info)

Opted to go for option 1, restrict relations with Lists solve this by documenting how to build your own "bridge" table to solve this use case.
Discussion point here on parent reference.
I have opted to just use the table name as is, since that required 0 additional logic to be added. There could be an argument made for having to reference the table by module:auth:table_name as well. Today we will have to disallow the same names for tables in your project and the module project regardless though. But in the future if we would move the tables in their own name space in the database we could maybe drop that requirement.
Current problem:
In a 1:n relation, the foreign key has to live in the module table. Modifying a table that is not owned by the user seems like a bad idea in general. But even if we allowed this we still run into problems as the generated code comes pre-shipped in the modules and are not modified by the user. Hence the code is not updated with the new requirements.
We can deal with this in a few ways:

Just not allow 1:n relations to modules

Cons, feels strange and might block real usecases.
Pro, easy

Create a binding table behind the scenes that the user owns but would be opaque when making queries.

Cons, complicated and adds additional logic requirements with migrations and queries.
Pros, natural interface for the user, we have to add this anyway if we want to support implicit n:m relations.

Create a virtual relation and make an additional query when joining instead of having a real relation

Cons, cannot have real relational restrictions in the database, potential extra overhead
Pros, natural interface for the user

Other solutions ?

Closes: #984, #1232
Pre-launch Checklist

[x] I read the Contribute page and followed the process outlined there for submitting PRs.
[x] This update contains only one single feature or bug fix and nothing else. (If you are submitting multiple fixes, please make multiple PRs.)
[x] I read and followed the Dart Style Guide and formatted the code with dart format.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///), and made sure that the documentation follows the same style as other Serverpod documentation. I checked spelling and grammar.
[x] I added new tests to check the change I am making.
[x] All existing and new tests are passing.
[x] Any breaking changes are documented below.

Starting review here - Could you please update the PR description so that it matches the implementation we have now chosen? For future reference.


Making realworld-postgres example compile

If you guys have pg_tmp on CI, I can fix this example so it compiles. I've been spamming @mehcode with my build.rs file for this on discord...

There may be something fun we can do with pg_tmp after we get migrations to enable super fast integration testing. Definitely worth exploring in the near future.

@abonander I have to admit, I'm a n00b at rust macros. Can you give an example of the sort of change you are talking about? After that I can make a PR with the change.

No worries. I'm going to merge this so the example compiles for now. Thanks for the fix. I'll leave it to @abonander to fix the macro itself ( I opened #105 ).


Suggest a new feature: Sidebars RTL Support

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal
Hello Bootstrab,
It would be great if you added Sidebars RTL Support to Examples on your page.
I will wait for the results.
Thanks
Motivation and context
I it common for us to use both LTR and RTL in our websites and it will be helpful if you added this feature ready to be used directly.

Hello,
Thank you for your comment. RTL Support is indeed an important feature. However, it seems like Bootstrap already has RTL support. I hadn't realized that until I tried to open a pull request and it was closed.
Please check this link to Bootstrap documentation about RTL support :
https://getbootstrap.com/docs/5.0/getting-started/rtl/


Crash when trying to profile the app

I am using an Android 8.1 device (arm-v7), Couchbase Lite 2.5, AGP and AS 3.4.1. When I try to profile the app, the app crashes within the cb-lite JNI lib.
The crash occurs when I write something to the DB.

    --------- beginning of crash
2019-05-21 14:13:53.508 com.xxx.yyy A/libc: Fatal signal 11 (SIGSEGV), code 1, fault addr 0x1f in tid 9209 (pool-8-thread-1), pid 9160 (xxx.yyy)
2019-05-21 14:13:53.564 ? W/crash_dump32: type=1400 audit(0.0:76): avc: denied { search } for name="com.xxx.yyy" dev="dm-2" ino=527669 scontext=u:r:crash_dump:s0:c512,c768 tcontext=u:object_r:app_data_file:s0:c512,c768 tclass=dir permissive=0
2019-05-21 14:13:53.592 ? I/crash_dump32: obtaining output fd from tombstoned, type: kDebuggerdTombstone
2019-05-21 14:13:53.593 ? I//system/bin/tombstoned: received crash request for pid 9160
2019-05-21 14:13:53.594 ? I/crash_dump32: performing dump of process 9160 (target tid = 9209)
2019-05-21 14:13:53.595 ? A/DEBUG: *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
2019-05-21 14:13:53.595 ? A/DEBUG: Build fingerprint: 'Cat/CatS61/CatS61:8.1.0/OPM1.171019.011/LTE_D0201121.0_S61_0.039.00:user/release-keys'
2019-05-21 14:13:53.595 ? A/DEBUG: Revision: '0'
2019-05-21 14:13:53.595 ? A/DEBUG: ABI: 'arm'
2019-05-21 14:13:53.595 ? A/DEBUG: pid: 9160, tid: 9209, name: pool-8-thread-1  >>> com.xxx.yyy <<<
2019-05-21 14:13:53.595 ? A/DEBUG: signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x1f
2019-05-21 14:13:53.595 ? A/DEBUG: Cause: null pointer dereference
2019-05-21 14:13:53.595 ? A/DEBUG:     r0 c7cba500  r1 ffffffff  r2 00000000  r3 cb184df4
2019-05-21 14:13:53.595 ? A/DEBUG:     r4 c7cba4a8  r5 c7cba500  r6 00000000  r7 c7cba450
2019-05-21 14:13:53.595 ? A/DEBUG:     r8 00000000  r9 cb19a9cf  sl 00000001  fp 00000000
2019-05-21 14:13:53.595 ? A/DEBUG:     ip cb3a417c  sp c7cba450  lr cb31ad17  pc cb31b230  cpsr 80070030
2019-05-21 14:13:53.594 ? W/crash_dump32: type=1400 audit(0.0:77): avc: denied { search } for name="com.xxx.yyy" dev="dm-2" ino=527669 scontext=u:r:crash_dump:s0:c512,c768 tcontext=u:object_r:app_data_file:s0:c512,c768 tclass=dir permissive=0
2019-05-21 14:13:53.597 ? A/DEBUG: backtrace:
2019-05-21 14:13:53.597 ? A/DEBUG:     #00 pc 00270230  /data/app/com.xxx.yyy-R_DyW_-TQaE8szARnAFdSw==/lib/arm/libLiteCoreJNI.so
2019-05-21 14:13:53.597 ? A/DEBUG:     #01 pc 0026fd13  /data/app/com.xxx.yyy-R_DyW_-TQaE8szARnAFdSw==/lib/arm/libLiteCoreJNI.so
2019-05-21 14:13:53.597 ? A/DEBUG:     #02 pc 002660bb  /data/app/com.xxx.yyy-R_DyW_-TQaE8szARnAFdSw==/lib/arm/libLiteCoreJNI.so (__gxx_personality_v0+330)
2019-05-21 14:13:53.597 ? A/DEBUG:     #03 pc 0018bee4  /data/data/com.xxx.yyy/libperfa_arm.so
2019-05-21 14:13:53.597 ? A/DEBUG:     #04 pc 0018c808  /data/data/com.xxx.yyy/libperfa_arm.so

As it does only occur if the AS profiler is running, I did also report it to google:
https://issuetracker.google.com/issues/133166400

I can reproduce it with the sample from couchbase.
I had to change two things:

Change the ARM version to v7, with ARMv8 it did not crash.
add a write operation to the sample.
My Fork: https://github.com/RocketRider/mobile-travel-sample

Now I can reproduce the issue with the following steps:

Start the App
Add Profiler
Click on "Proceed as guest"

Tested with following devices:
CAT S61 - Android 8.1: Crash
Samsung S9 - Android 9.0: Crash
Samsung Tab - Android 5.0: No Crash

Thanks Michael for all these detail steps to reproduce. I will look into it and get back to you as soon as I have an answer.
-Sandy

The documentation here:
https://docs.couchbase.com/tutorials/mobile-travel-sample/csharp/installation/travel-mobile-app.html
Explains how to avoid this problem.

Michael, can we get about double that much of the logcat, on either side of the failure?  We don't have any of the specific devices on which you are causing the crash.

Thank you for looking into it.
I guess it will crash on any Android 8+ device with an arm cpu.
Here is the full log from starting the app until the crash:
profiler-crash.txt

@RocketRider I am moving this issue to the couchbase-lite-android-ce repo.  Please follow it there


Handle 504 errors got error 504 from the server

Getting "got error 504 from the server" errors and noticed that the script just keeps going with new requests rather than retrying failed ones.
Adding handling for this could be helpful to prevent partial results

Sorry for the (long) delay. Working on it right now. I'm thinking up to 5 retries if it doesn't receive a fatal error, like 502 or 504. Is that enough?

Answering to myself: did a few tests after increasing the concurrent threads to 60 in order to force a gateway timeout situation (hope nobody gets too mad at me!), and it seems to be working fine now:
thread 1225 got non-fatal reply (502) from server, trying again in 2 seconds
thread 1227 got non-fatal reply (502) from server, trying again in 2 seconds
thread 1296 got non-fatal reply (502) from server, trying again in 2 seconds
thread 1300 got non-fatal reply (502) from server, trying again in 2 seconds

and after sometime, while the main thread is gathering results:
thread 1186 got a good reply after 1 tries
thread 1213 got a good reply after 2 tries
thread 1241 got a good reply after 1 tries
thread 1279 got a good reply after 1 tries
thread 1159 got a good reply after 2 tries

Not a single one went past 2 tries, so I guess 5 should be okay.
(no, the thread IDs don't match, I just picked some random lines -- I managed to get over 30 timeouts in one run while testing :-) )


tune download: error: An error happened while trying to locate the file on the Hub and we cannot find the requested files in the local cache. Please check your connection and try again or make sure your Internet connection is on.

Command:
tune download meta-llama/Meta-Llama-3-8B-Instruct --output-dir ./tmp/llama3/
Error:
Ignoring files matching the following patterns: *.safetensors
Fetching 13 files:   0%|                                                                                | 0/13 [00:00<?, ?it/s]
LICENSE: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 7.63k/7.63k [00:00<00:00, 22.9MB/s]
README.md: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 40.9k/40.9k [00:00<00:00, 575kB/s]
usage: tune download <repo-id> [OPTIONS]
tune download: error: An error happened while trying to locate the file on the Hub and we cannot find the requested files in the local cache. Please check your connection and try again or make sure your Internet connection is on.

Looks like the repo is up and you passed in the correct name 👍
You might want to make sure you have a stable internet connection and try again in a little bit?

Looks like the repo is up and you passed in the correct name 👍
You might want to make sure you have a stable internet connection and try again in a little bit?

Thanks for your reply. I tried it on my personal device, cloud server, and Google Colab. This error pops up on all the devices so I think it might not be the internet issue.

Works OK for me - have you been granted access to the model in HuggingFace, and set your HF tokens up correctly? You can either do huggingface-cli login or pass --hf-token <HF_TOKEN> to tune download.

Thanks a lot for all the replies. The issue is due to the huggingface token permission.


chore: Add CI tests

Resolves: #6
Using janus-idp/redhat-backstage-build as deployment image, since I don't know about any other vanilla image out there... I think we want to change this to an upstream image if possible.

@Rugvip CI image changed via https://github.com/backstage/charts/pull/15

Please let @vinzscam be involved in the review process while we're still in the setup and unification phase of these charts, especially for larger changes like this.

@Rugvip sorry, I was not expecting it being merged that fast either. @vinzscam please don't feel excluded.


Added distinct to MongoAdapter in experimental code

This includes the implementation of nested search() queries before a distinct() request

This does separate queries for getting the list of distinct values and their counts. Given the experimental and temporary status of this code, that is good enough for now. For the record---in case we want it later---I will link this Stack Overflow post describing how to obtain both in a single query: https://stackoverflow.com/a/33418582/1221924

With the previous commit (https://github.com/bluesky/databroker/pull/763/commits/c345fc5b47b41387bb61d67a6579c6536f3d01ad), I improved the way specs is requested with distinct in the mongo database and it returns a similar behavior that we can see in the base code of tiled and that is also referenced in https://github.com/bluesky/tiled/pull/436

Power-cycled to re-run tests. This was failing due to an upstream issue in vcrpy that seems to have been resolved by their latest release.


[testx] Enhanced test result collection and benchmark dashboard

Test result

colorize test output, when you have verbose mode enabled, it's hard to tell what is actually error

compile error #xxx
test failed
assert error

a more advanced UI for test coverage, built in html is a bit hard to use ... need to use monaco and vue (might need to put all UI related code under one folder)

Lint result

go lint, go vet (I think they have json output format)

Benchmark

benchboard was removed in https://github.com/dyweb/gommon/pull/40 references are in https://github.com/dyweb/gommon/issues/27
generate graph based on benchmark output
generate series based on benchmark across time

Ref

https://github.com/360EntSecGroup-Skylar/goreporter

This is kind of duplicated w/ benchhub's gotest and gobench framework, though it might be better to have benchhub wraps a grpc interface around gommon's functionalities

See https://github.com/benchhub/benchhub/issues/54


entc/gen: fix loss of int64 precision during JSON type decoding

Hello @eiixy, can you please add an example or testcase for this change?

Hello @eiixy, can you please add an example or testcase for this change?

Sure, I have added to example


is repairwallet crediting when it should be debiting?

I think repairwallet is crediting my balance when it should be debiting my balance. I have an incrementally growing balance on my wallet (that is not recognized by the ledger, and not spendable). It's a cosmetic problem, primarily, not a security problem. But for my local QT wallet (Mac) it means I can't tell which addresses actually have how many coins, which transactions are valid.
Most of the internal wallet transactions when I send coins to myself don't make it to the blockchain. They appear to accrue coinday but never stake because they have zero confirmations.

I encountered the same issue in my Windows wallet, seemingly related to the fork. My only solution was to dump all my keys, delete my wallet and import the keys again to force the wallet to properly rescan each address.

On Nov 28, 2015, at 08:16, mdpfeiffer<EMAIL_ADDRESS>wrote:
I think repairwallet is crediting my balance when it should be debiting my balance. I have an incrementally growing balance on my wallet (that is not recognized by the ledger, and not spendable). It's a cosmetic problem, primarily, not a security problem. But for my local QT wallet (Mac) it means I can't tell which addresses actually have how many coins, which transactions are valid.
Most of the internal wallet transactions when I send coins to myself don't make it to the blockchain. They appear to accrue coinday but never stake because they have zero confirmations.
—
Reply to this email directly or view it on GitHub.

This is interesting. I had issues with disappearing balance due to the fork, however sending coins to myself always results in a correctly broadcast transaction which gets transactions. Both on the paycoind and also QT. I've not had an issue with this bizarrely.


Check and document litellm support

https://github.com/BerriAI/litellm

Dropping this for simplicity.


Trouble getting definitions

I'm using 1.4.2
It used to be fine with 1.3.0, but it seems with 1.4.2 I have troubles with getting the definition of (and jumping to) functions/records/etc, and the references. I'd say it happens when the function/record/etc is within a module which is outside the current module, but within the repo. If I hover the thing and press option I can't jump to its location, etc.
Fun facts

I still get the type! (ex in a lens when I hover the function/record/etc)
Dev dependencies are fine. (ex I can jump to the definition of React or useEffect in React.useEffect(() => {…})

Can you set up a minimal reproduction somehow?

Yep. Say I've got
bsconfig.json
node_modules/
src/

(etc)
src/Bar.Res
let add = (a, b) => a + b

src/Foo.res
Bar.sub(4, 5)->ignore

in Foo.res, I've got the lens telling me sub is (int, int) => int, but I can't jump to its definition (in Bar.res)

Im having this issue as well

Please use the configuration option to set the path to the binary, and see if that helps.
Depending on the answer, we'l learn something new.

Yep. Say I've got
bsconfig.json
node_modules/
src/

(etc)
src/Bar.Res
let sub = (a, b) => a - b

src/Foo.res
Bar.sub(4, 5)->ignore

in Foo.res, I've got the lens telling me sub is (int, int) => int, but I can't jump to its definition (in Bar.res)

We have an entire set of tests that correspond to that abstract description.
Would you try the project in analysis/tests see if you have this issue?

Please use the configuration option to set the path to the binary, and see if that helps.
I've set the path properly (I know since the previous tries were met with a "can't find…"), but it hasn't improved the situation.

Would you try the project in analysis/tests see if you have this issue?

Good so next step is figure out what's different in your project.
Then report back.

On the off chance, I upgraded my hello world test repo to 10.0.0-beta.3, but nope.

Hello, I found one way to reproduce the bug,
When namespace is set to true in the bsconfig, not sure if it's the same for you @remitbri

Indeed! Good catch, @WhyThat . The analysis/tests project doesn't have the flag, my "hello world" repo has one - if I remove it everything's ok.

There used to be some namespace tests. So this might be relatively recent.

I don't think the compiler has changed anything about namespaces. Or at least I don't remember.

I just used the workspace-project and tried to execute the analysis binary on it.
With the current master, I get this:
➜  workspace-project git:(master) ✗ ../../_build/install/default/bin/rescript-editor-analysis definition app/src/App.res 0 22
null

With version 1.3.0, I get this:
➜  workspace-project git:(master) ✗ ../../_build/install/default/bin/rescript-editor-analysis_1.3.0.exe definition app/src/App.res 0 22
{"uri": "file://./node_modules/myplugin/src/Promise.res", "range": {"start": {"line": 0, "character": 0}, "end": {"line": 0, "character": 0}}}

I also downgraded the project to 9.1.4 and did a fresh build, same outcome.
So the regression has to be in the analysis binary, not in the compiler.

I'm not at the computer for a while, but maybe someone could do a git bisect and find what commit caused it? Should be fairly fast, it's not that many commits between 1.3.0 and 1.4.0.

Yes, I fixed it.
Tested it again after my changes, and now it gives me
➜  rescript-vscode git:(master) analysis/_build/install/default/bin/rescript-editor-analysis definition  app/src/App.res 0 22
{"uri": "file://./node_modules/myplugin/src/Promise.res", "range": {"start": {"line": 0, "character": 0}, "end": {"line": 0, "character": 0}}}

@remitbri @igna92ts @WhyThat
Feel free to check if it works for you with the VSIX of the PR build.

It works properly for the jump to definition but not for references nor rename (but I think it never work, but thinking about it I can't remember if these features has ever worked for project with namespace)

Yes, I guess this one has never been fixed: https://github.com/rescript-lang/rescript-vscode/issues/273


Issue 926 - Fix for Safari Browser

PR's to master will be closed. Please submit PR's to the development branch

@Eonasdan man, you are pretty good in keep people away from your projects
No problem I don't need it anymore

@rodfersou try reading the constrib doc it clearly states that PR's to master will be closed and that they should be submitted to the dev branch


[BUG] [SM] Defining a new policy in OU causes SM failure

Required Basic Info

Accelerator Version:  v1.2.5
Install Type: ANY
Which State did the Main State Machine Fail in: (e.g. Phase 0)

Describe the bug

it appears we only read the attached s3 policy files if defined at least once in a mandatory account
if defining a new policy in an ou, state machine will fail.
workaround, also define the same policy in any mandatory account

fix already in progress

This will ship in v1.2.6


[BUG] type issues with handleHooks and handleServerLoad

Bug report
Report for:  lucia-sveltekit 0.8.5
I'm following the docs on migrating, but having TypeScript troubles.
There are three problems I'm having

The auth.handleHooks function complains that setHeaders is the incorrect type.
It also expects _lucia: Session to exist on ServerLoadEvent, which it doesn't at that point
using the auth.handleServerLoad function as a wrapper like the docs show
a) doesn't keep the types of any locals set in hooks
b) breaks all downstream load function types

Everything is a pretty local TS problem except for 3b
To fix that, I'm just passing in the ServerLoadEvent to handleServerLoad separately, extracting _lucia and returning it with other data.
Doing it this way has the same missing _lucia: Session and setHeaders type issues as the auth.handleHooks function
Current behavior
This works: export const handle = auth.handleHooks()
This does not work: export const handle = sequence(auth.handleHooks(), ...handles)
export const load = auth.handleServerLoad(handleSession(), async (e) => { breaks all load types downstream
Expected behavior
Load the warning/route handling hooks and pass in _lucia to the main server load without breaking SK type safety. Declaring _lucia on App.Locals shouldn't be necessary because it doesn't actually exist there
Possible cause
The setHeaders issue seems to be because SK has it as Record<string, string> and in Lucia it's Record<string, string | number | string[] | null>
Changing this fixes the hooks issue, but the wrapper is still breaking things for me
Version
lucia-sveltekit 0.8.5 and kit next.484
Reproduction
repro

Problems 1 and 2 (and likely 3a) is a pretty simple fix but 3b is a hard one. Not sure if it's possible in TS but will try it

I'll just say that this might take a while

Should be fixed with #85 !

It's working now, thank you – that was fast!


build pimusicbox using linuxkit

I'm a big fan of debian and would love to have pimusicbox running on a debian where I can SSH in and add some stuff and upgrade some packages... but... I've come across linuxkit, which seems interesting for pimusicbox
https://github.com/linuxkit/linuxkit  A toolkit for building secure, portable and lean operating systems for containers
Although this goes in the other direction of having a "hackable" pimusicbox based on a pure debian, I think it could still be an interesting path to build the distro / images for pimusicbox.
I was inspired by a talk (in french) about linuxkit https://www.hooktube.com/watch?v=CWTJdve0g6U which explains quite well how the "embedded, task specific, linux distros" could benefit from this. I'm sure someone can find a decent intro on youtube if the project page is not explicit enough https://hooktube.com/results?search_query=linuxkit
Just putting this idea out there, in case someone else is tempted by this path and wants to share ideas and trying it out.

Thanks for the suggestion, closing in favor of #6 which already tracks a similar idea.

Or, would close if I could :-)


projector screen switch

Hello,
I have a projector screen that needs to be stopped 20 seconds after ON hex is sent.  I guess I need to send a stop command after 20 sec the On hex data is sent but I'm a newbie and I cannot figure out how :) any suggestions please? Thanks a lot.
{
"name": "Screen Down",
"type": "switch",
"sendCount": 1,
"interval": 20,
"enableAutoOff": true,
"data": "b2001202a7"
}

@pratkai why did you close this issue ? Have you find an answer ? if yes, why not share it ?

Sure, I could figure out. here it is:
{
"name": "Movie Screen",
"type": "switch",
"data": {
"on": [
{
"data": "b2001202a7...",
"pause": 23
},
{
"data": "b2001202a",
"pause": 0.3
}
],
"off": [
{
"data": "b2001202a9",
"pause": 24
},
{
"data": "b2001202a",
"pause": 0.3
}
]
        }
    },

Sorry. Thank you. Really, I don't need that, but just thinking about frustration of people who will google your exact same issue, see that you seem finded a solution, and keep it just for you…
We can ask..., but we must share... Thanks for them.

You're totally right, I should have done that.

👍


Bug fixes for Find Parasite Action

About the PR

Selecting a parasite source makes the ghost follow the source.
Dead ParasiteThrowers no longer show up as a possible parasite source.
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl:

fix: Fixed Find Parasite Action

Why's Verm review being requested? Pretty sure I made all the code being changed here.

Prob cause I went over the carrier code to fix bugs and such, Smug asked me to.

Ok then

Why's Verm review being requested? Pretty sure I made all the code being changed here.

Verm's more familiar with this code than me


refactor: move the setup-token to a delegator folder

Signed-off-by: laurentsimon<EMAIL_ADDRESS>This is in anticipation for https://github.com/slsa-framework/slsa-github-generator/issues/1576. We can store other public actions under delegator

Could you please change
https://github.com/slsa-framework/slsa-github-generator/blob/9c2f51074861f726667a943eb2223b90f3e46023/.github/workflows/e2e.verify-token.schedule.yml#L24
too?

Done.


Components directory not found

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
My issue was a warning in the build process, that the provided components directory was not found
 WARN  Components directory not found: /Users/**/storyblok

The module only resolves the storyblok components folder inside root source folder. I created a config entry in the nuxt config to set a custom components dir. The Patch is seen below and I added this to my nuxt.config
  storyblok: {
    dir: "~/vue/components",
  },

Here is the diff that solved my problem:
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 5a0e667..7e3ebd8 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -25,7 +25,8 @@ export default defineNuxtModule({

     // Enable dirs
     // nuxt.options.components.dirs = ["~/components/storyblok"];
-    addComponentsDir({ path: "~/storyblok", global: true, pathPrefix: false });
+    nuxt.options.storyblok.dir = nuxt.options.storyblok.dir || "~/storyblok";
+    addComponentsDir({ path: nuxt.options.storyblok.dir, global: true, pathPrefix: true });

     const runtimeDir = fileURLToPath(new URL("./runtime", import.meta.url));
     nuxt.options.build.transpile.push(runtimeDir);

This issue body was partially generated by patch-package.

Hey there! 🙂
Thank you very much for your feedback and the suggestion regarding the components directory. Alex and I are certainly considering adding this option in the future. However, for the time being, the current way (using a storyblok folder at the root level of your project) is set up on purpose because Nuxt 3 might silently fail otherwise.
That's why right now we recommend to add all of your Storyblok components (or blocks) to the designated storyblok folder and all other components to the components folder.

@manuelschroederdev
I also wanted to start migrating everything to nuxt3 and came across this issue.
Might it still fail in nuxt3 also with the stable version?
If you have a lot components, it does not feel right to have to move some storyblok related components to another directory...

Hello, I'm from the future and Nuxt 3 with Storyblok is now running fine from the components folder only. The storyblok folder is creating an unnecessary extra step and it would be nice to disable it or at least disable the warnings.
Thank you!

following up on this
Would it be possible to have Blok that are taken regardless of their folder?
Aka, I have a call to action button saved in /components/Shared/Btn/CTA.vue
Nuxt automatically pick it up in the code when I have a component <SharedBtnCTA />
Being able in Storyblok to say a blok technical name is SharedBtnCTA and have it use the same component would be amazing, especially in larger projects with tons of components

following up on this
Would it be possible to have Blok that are taken regardless of their folder?
Aka, I have a call to action button saved in /components/Shared/Btn/CTA.vue Nuxt automatically pick it up in the code when I have a component <SharedBtnCTA />
Being able in Storyblok to say a blok technical name is SharedBtnCTA and have it use the same component would be amazing, especially in larger projects with tons of components

This is how I use it, get the component by path and component name. If you named it like that in Storyblok, Nuxt3 automatically finds the right component.

I'll have a try whenever I have the time. The last time I tried, it didn't work though. Is there any specific configuration to do? (or maybe I was just too impatient to wait for an update on the bloks from Storyblok)

Thanks!
I'll have a try whenever I have the time. The last time I tried, it didn't work though. Is there any specific configuration to do? (or maybe I was just too impatient to wait for an update on the bloks from Storyblok)

Get the latest Nuxt 3 and @storyblok/nuxt version. (Also check the npm lock files, could be locked on a older version.)
Set your components settings in your config: export default defineNuxtConfig({ … components: {  global: true,  dirs: ["~/components"], }, … });

Should also work with the "~/storyblok" folder if defined, I guess. But I like to keep my components in the "~/components" folder.


[AIRFLOW-2344] Fix connections -l to work with pipe/redirect

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2344
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a JIRA issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
airflow connections -l uses 'tabulate' package with
fancy_grid format, which outputs box drawing characters.
It can occur UnicodeEncodeError with pipe or redirect,
since the default encoding for Python 2.x is ascii.
This PR fixes it and contains some flask8 related fixes.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Added tests.core:CliTests.test_cli_connections_list_redirect.
Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3244 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3244      +/-   ##
==========================================
+ Coverage   75.25%   75.28%   +0.03%
==========================================
  Files         196      196
  Lines       14608    14611       +3
==========================================
+ Hits        10993    11000       +7
+ Misses       3615     3611       -4

Impacted Files
Coverage Δ

airflow/bin/cli.py
61.85% <100%> (+0.16%)
:arrow_up:

airflow/models.py
87.08% <0%> (+0.04%)
:arrow_up:

airflow/jobs.py
82.89% <0%> (+0.26%)
: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 4c02ad7...005a045. Read the comment docs.

Exactly. Revised PR :)


support for nested resources

Add support for resources in nested objects (list/dict) (#177)

Hi! Already added https://github.com/CERT-Polska/karton/pull/186 but see good ideas in your PR (e.g. use of json.loads(..., object_hook= which is analogous to json.dumps). I will consider to merge some of it into #186.


Deleting large files.

Description
When deleting a file that's too large to go into the Recycle Bin, Files will not actually delete the file. I have to switch back to the regular file explorere to do so.
Steps To Reproduce

Find a very large file that the recycle bin can't hold.
Attempt to delete the file. Prompt will ask to permanently delete the file.
Files fails to delete the file (no error or anything, mouse icon shows its "working," but the file will remain.

Expected behavior
Well... said file being deleted.
Files Version
<IP_ADDRESS>
Windows Version
Windows 11 21H2 22000.194
Relevant Assets
No response

Thanks for the report

Could you show the prompt(s) that are showed to you when trying to delete such files?
Does it work if you tick the "permanently delete" option in Files confirmation dialog?

@Syltti can we get an update on this?


Upgrade to Scala 2.10.7

I have already published Scala.js 0.6.19, 0.6.20, 0.6.21 and 1.0.0-M1 for Scala 2.10.7. We should now add it to our CI for future versions.

Fixed in d0e56ed65d9d762c04f32175e0860d8fc024d1c3.


[DO NOT MERGE] run tests with latest pekko snapshots

@inline def asJava: CompletionStage[T] = javaapi.FutureConverters.asJava(f)

We should remove the inline def in pekko.


⚠ Mobitopup has degraded performance

In 1003c54, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 4527 ms

Resolved: Mobitopup performance has improved in 3282e6a.


Added zk prove/verify

To construct the zk proof, I'm reusing helper functions: https://github.com/edgeware-builders/anon/blob/web-client-rust-functions/pallets/merkle/src/merkle/helper.rs
We should modify those and turn them into gadgets, namely, we should be able to pass generators and prover/verifier instances, and rename them maybe

Looks good!


Force composition instead of inheritance.

LGTM.
If FileObjectQueue didn't expose file - we could maybe hide the classes too.

Yeah. It's worth discussing.


[Text Analytics] Add samples for API version 2022-10-01-preview features

Resolves: #31882

Added new samples, but note that the sample 11 corresponding to dynamic classification will be added in a separate PR.
Polished docs.

API change check
API changes are not detected in this pull request.


[9.x] Str::after() minor tweaks

In the original version of the after() method, $subject is split into an array then reversed and the first element of the resulting array is returned.
return $search === '' ? $subject : array_reverse(explode($search, $subject, 2))[0];

So, a temporary array is created every time.
In the modified version we use the same logic used in afterLast(), which means, we only use string search and manipulation to return the substring.
Also, the behavior of after() has been modified to allow it to return the remainder of a string after the first or last occurrence (controlled by the new $last parameter).
Stopping using array manipulation was the main goal of this change. The overall behavior of both functions wasn't changed.

I'm not sure I really see this as a big improvement? It's more code?

Yeah, this change is not intended to be a major improvement.
Both versions have the same amount of code. The differences are:
Current version:
public static function after($subject, $search)
{
    return $search === '' ? $subject : array_reverse(explode($search, $subject, 2))[0];
}

public static function afterLast($subject, $search)
{
    if ($search === '') {
        return $subject;
    }

    $position = strrpos($subject, (string) $search);

    if ($position === false) {
        return $subject;
    }

    return substr($subject, $position + strlen($search));
}

afterLast() and after can share almost the same algorithm, but they do the task differently. The after()  explodes as an array, reverses it, etc.
On the other side, the afterLast() uses another way of doing the same thing, but without creating unnecessary arrays.
The proposed change makes the two functions more consistent with each other (in my point of view).
Proposed version:
public static function after($subject, $search, $last = false)
{
    if ($search === '') {
        return $subject;
    }

    $position = $last ? mb_strrpos($subject, (string) $search) : mb_strpos($subject, (string) $search);

    if ($position === false) {
        return $subject;
    }

    return static::substr($subject, $position + static::length($search));
}

public static function afterLast($subject, $search)
{
    return static::after($subject, $search, true);
}


Update INSTALLATION_DOCKER.md

After testing the software on a machine with the M1 chip, I can confirm it does not work there. While this problem is resolved (if it is) I suggest updating the documentation to warn possible users.

Hi @JGNieto,
thank you for testing this. We now have one user with an M1 Mac in our group, we'll see what we can do.
On the plus side, that same user is running tierpsy (installed from source) on his M1 so that path should work :)


Change travis ci

Remove using docker compose configuration and files linked with her
Add configuration for travis with custom mongo daemon
Change composer.json

I can use services in travis ci, but there is old version (3.4.10) for service mongodb and itsn't possible to change version. See https://github.com/travis-ci/travis-ci/issues/7614. @jenssegers, If you don't agree with using non-docker configuration for tests, i can revert configuration with docker. Configuration with running tests without docker services faster because build custom container take more time (If use ready docker images jobs can faster). We can cache vendor files and composer cache with this configuration

@jenssegers, can you review please?

These are a lot of changes in 1 PR. From what I hear, your main issue is that the mongodb installation on Travis is an old version? I'll look into fixing one problem at a time :)

I can new PR with last changes in one commit.

@jenssegers, i created new PR with this changes


bugfix workflowstate encoding, missing generation field

Description

This PR updates the functions EncodeWorkflowState and DecodeWorkflowState so they dont use an embedded struct definition and instead use a well defined struct, additionally this PR adds the field Generation to the encoded data and a test to verify encoding and decoding works as expected.
Issue reference

Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Linter @famarting

tests need retrigger, these failures seem unrelated to my changes

ping, we need this 😄


NEXT-10928 - Optimise requests in LocaleToLanguageService

1. What does this change do, exactly?
The LocaleToLanguageService uses two separate requests in its localeToLanguage function which is called at least twice upon opening the administration. Since these entities are associated with one another, this function can be optimised by using a single request with the respective association. This PR achieves this by improving the (single) request.
Before (notice the language requests):

After (two requests less):

2. Describe each step to reproduce the issue or behaviour.
Open the administration. Look into the browser-dev-tools network tab.
3. Checklist

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

Hmm the CI failed in the install Shopware step. I guess that is not the fault of this PR 🤔


CHOWTapeModel: 2.10.0 -> 2.11.1

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jatinchowdhury18/AnalogTapeModel/releases.
meta.description for CHOWTapeModel is: "Physical modelling signal processing for analog tape recording. LV2, VST3 and standalone"
meta.homepage for CHOWTapeModel is: "https://github.com/jatinchowdhury18/AnalogTapeModel"
meta.changelog for CHOWTapeModel is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.11.0 with grep in /nix/store/3b9lsh15l3h3vbdb4fc4bgdf04lilwj4-CHOWTapeModel-2.11.0
found 2.11.0 in filename of file in /nix/store/3b9lsh15l3h3vbdb4fc4bgdf04lilwj4-CHOWTapeModel-2.11.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
CHOWTapeModel

Instructions to test this update (click to expand)

Build yourself:
nix-build -A CHOWTapeModel https://github.com/r-ryantm/nixpkgs/archive/273a9096df36cc3df9b74dc2dc2ea3db6a0ef95d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3b9lsh15l3h3vbdb4fc4bgdf04lilwj4-CHOWTapeModel-2.11.0
ls -la /nix/store/3b9lsh15l3h3vbdb4fc4bgdf04lilwj4-CHOWTapeModel-2.11.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

CHOWTapeModel

Maintainer pings
cc @magnetophon for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jatinchowdhury18/AnalogTapeModel/releases.
meta.description for CHOWTapeModel is: Physical modelling signal processing for analog tape recording. LV2, VST3 and standalone
meta.homepage for CHOWTapeModel is: https://github.com/jatinchowdhury18/AnalogTapeModel
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.11.1 with grep in /nix/store/037qjhxa7msixfl2z3qwccwan1blxa30-CHOWTapeModel-2.11.1
found 2.11.1 in filename of file in /nix/store/037qjhxa7msixfl2z3qwccwan1blxa30-CHOWTapeModel-2.11.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
CHOWTapeModel

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/037qjhxa7msixfl2z3qwccwan1blxa30-CHOWTapeModel-2.11.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A CHOWTapeModel https://github.com/r-ryantm/nixpkgs/archive/6a72c306738fd77e23d80b51352ec0864aee670e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/037qjhxa7msixfl2z3qwccwan1blxa30-CHOWTapeModel-2.11.1
ls -la /nix/store/037qjhxa7msixfl2z3qwccwan1blxa30-CHOWTapeModel-2.11.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

CHOWTapeModel

Maintainer pings
cc @magnetophon for testing.

Already on master.


Init

Setup (Creates) a project setup to work with cargo leet

Abandoning this PR as this isn't a priority.


Error with service worker when depoloying to firebase

I'm seeing a bunch of erros (below) in the browser console after deploying a newly created app to firebase. Looks like the sw is referencing my local files instead of the ones in the server.
I'm on Windows 10 and I had a similar issue with create-react-app that they fixed with this commit: https://github.com/facebookincubator/create-react-app/commit/22ca584b2b8db5cb6dfa601412a72e6947054cf9
 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/2.chunk.dee19c0c2b89ad8a3492.js?_sw-precache=cecc045fb475b3d5f4f96d2140782cf1. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/assets/favicon.ico?_sw-precache=53ac170e970ad034a55ee15ce198708c. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/assets/icons/android-chrome-192x192.png?_sw-precache=1ea58b225a597f78adfd987ed164e573. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/assets/icons/android-chrome-512x512.png?_sw-precache=b65626103d4cd30abdb0423e6ab15179. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/assets/icons/apple-touch-icon.png?_sw-precache=4e32b9c4edcb5babb823e4824f90bb8d. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/assets/icons/favicon-16x16.png?_sw-precache=2cbedfd68f59e42b0d03ba3946863ee7. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/assets/icons/favicon-32x32.png?_sw-precache=b9108564a20f06675f60b29337c71b3e. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/assets/icons/mstile-150x150.png?_sw-precache=3eb725b0686148a67f2feb546670b313. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/bundle.js?_sw-precache=d5bb8d2c92704cc0f102c0896195782e. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/favicon.ico?_sw-precache=53ac170e970ad034a55ee15ce198708c. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/index.html?_sw-precache=f1b6dd4db31797ed11807b114bcbdae2. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/manifest.json?_sw-precache=409a6ab1239000b8c4b64cf7c808d616. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/polyfills.chunk.906967bd865a58e79868.js?_sw-precache=fc493e4fc1f1a71239b36865099aaf69. URL scheme must be "http" or "https" for CORS request. (anonymous) @ sw.js:1 (anonymous) @ sw.js:1 sw.js:1 Fetch API cannot load file:///D:/projects/pwa/preact/one/build/style.css?_sw-precache=3ce120ab4c8b971cba535fc84d789361. URL scheme must be "http" or "https" for CORS request.

Same with ZEIT NOW
`><EMAIL_ADDRESS>start /home/nowuser/src

NODE_ENV=production npm run -s serve
05/28 12:46 AM (21s)
Error: EACCES: permission denied, open '/home/nowuser/src/build/polyfills.chunk.906967bd865a58e79868.js'
05/28 12:46 AM (21s)

npm ERR!  Exit status 1

I see that some effort regarding this is already thr
https://github.com/developit/preact-cli/blob/master/src/lib/webpack-config.js#L431
I cant check on a windows machine, can anyone pick this and try the fix suggested in the issue description?

@prateekbh Maybe we could do something like this so that the service worker takes in only HTTP requests???

@andrantis could you please describe exactly how you deployed to firebase?

Possibly due to #50?  I can take a look on windows.

Hmm.. How good it would be if there would be a plugin without such issues 😱

Related to #7

Time to add --sw offline-plugin.

off note: I'll raise a PR to migrate to workbox by default by next monday

@prateekbh what does that solve exactly? :-)

AFAIK it takes care of this problem, and it will bring much more flexible approach
Adding @addyosmani for more info :p

@developit should I close this as #7 has been resolved and absolute paths are no longer there, or do we want to track --no-service-worker here?

I think we're good to close since the original issue is fixed.


Engine crashes when using @export_category with a non-string argument

Godot version
4.0 beta 16 and 17
System information
Windows 11
Issue description
Before beta 16 you could write @export_category(name) and it would make a category with "name" as the name.
Now you must use a string when defining the category name. (@export_category("name"))
The issue is that the moment you type something that doesn't start with " the engine crashes.
Steps to reproduce

Create a script.
Write one of the following:
a) @export_category()
b) @export_group()
c) @export_subgroup()
In the parentheses write any character other than ".
Watch it crash.

If you managed to save the file the project will crash when opened.
Minimal reproduction project
Minimal.zip

CC @dalexeev


[$1000] On task view page clicking three dot menu hides the pin icon and mark as done button as well on header

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Login to ND
Click on FAB button -> Click Assign task
Enter title -> Click Next -> Select Share somewhere -> Click confirm task
This will navigate to created task -> Click three dot on right side -> Notice the position of menu hides all three option

Expected Result:
Menu should not hide the three option on top left same on other pages
Actual Result:
Opening the menu hides the 3 option on header
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.38-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @dhairyasenjaliya
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688826550912059
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018cd8bba8d8d66974
Upwork Job ID:<PHONE_NUMBER>067138560
Last Price Increase: 2023-07-31

Proposal
Please re-state the problem that we are trying to solve in this issue.
On task view page clicking three dot menu hides the pin icon and mark as done button as well on header
What is the root cause of that problem?
The root cause is we have not added anchorAlignment prop to <ThreeDotsMenu> on HeaderView.js that basically responsible to set position horizontal/vertical for menu here we are missing that’s the causes of this issue
What changes do you think we should make in order to solve the problem?
We just need to anchorAlignment prop to <ThreeDotsMenu> such a way that I doesn’t hide header option
We need to define default props of anchorAlignment to HeaderView.js and pass on <ThreeDotsMenu>
We need to set value like below as this can be found to other places that we have used <ThreeDotsMenu> which is consistent with other page
Optionally we can add custom values as desired
threeDotsAnchorAlignment:{
	horizontal: CONST.MODAL.ANCHOR_ORIGIN_HORIZONTAL.LEFT,
	vertical: CONST.MODAL.ANCHOR_ORIGIN_VERTICAL.TOP,
}

What alternative solutions did you explore? (Optional)
N/A

Yup, not a bug, closing. Thanks for clarifying @dhairyasenjaliya

This bug still exists

Okay this has been reopened, applying External and will have C+ review proposals

This has the same root cause as https://github.com/Expensify/App/issues/23140. Just happening in a different place.
@Julesssss should we cover this fix as well in https://github.com/Expensify/App/issues/23140?
cc: @Nikhil-Vats
I see that both issues are reported by same contributor - @dhairyasenjaliya and I think we can combine these into one.

@Julesssss @Ollyws can you confirm the above?

Yeah agree. Let's close this issue as a dupe

Got it. I will port this discussion over to the other issue.


Mathematics Details

Description
Maybe it's better if we have a section in README.md which we demonstrate the mathematics behind the Samila arts.
If it's OK I can handle it.

Good idea 👍


How to tell which version of vyper to use?

Environment information

ape and plugin versions:
vyper       0.2.0
ape 0.2.7
vyper 0.3.3+commit.48e326f

Python Version: 3.9.5

OS: osx

What went wrong?
Please include information like:
I have a contract that looks like this:
a: uint80

When I run ape compile I get:
INFO: Compiling 'MyContract.vy'.
ERROR: (VyperCompileError) vyper.exceptions.UnknownType: No builtin or user-defined type named 'uint80'
  contract "/DIR/vyper-wcbpnrwq.vy", line 26:3
       25 #     return price
  ---> 26 a: uint80
  -----------^
       27

[19956] Failed to execute script vyper_compile

How can I tell it to use 0.3.3 of vyper?

Running vyper and the contract name compiles fine

Got it. Went through the code and found it grabs it from the comments:
# @version ^0.3.3

It looks like we should create docs for each plugin

We can also support config driven versioning


feat(sdk/elixir): improve acronym word conversion

Use regular expression instead of dictionary of words to enable any module that use acronym word to generate correct name in Elixir.
Updates #6310

This approach is still not good through. It can solve on experimentalWithAllGPUs case, but not for VCSIgnoredPath


Viewing a file item's json causes the file to be downloaded

As an example, view id:4497b7bb42e543b691027840d1b9092a, a tile package.  Ideally, detect the type of the item before attempting to load it's data.

Definitely a painful experience! This was brought up in #72 so let's continue the conversation there. My thought is to set a timeout on the /data request and provide a button to continue loading for those times when you really do want to see the data.


NIFI-11845 Add support for migrating H2 2.1 and 1.4 to 2.2

Summary
NIFI-11845 Adds support for migrating from either H2 1.4 or 2.1 to the current version 2.2. This migration support enables seamless upgrades from existing NiFi deployments, including version 1.15.0 and earlier with H2 1.4, and deployments of 1.16.0 through 1.22.0 with H2 2.1.
H2 2.1 introduced changes to the internal database format, including references to org.h2 package names for certain value types. H2 2.2 introduced additional format changes, incrementing the format version from 2 to 3.
Previous migration support from 1.4 to 2.1 used a shaded version of H2 1.4 for reading existing databases. The updated migration approach uses the same shaded library strategy, introducing v14 and v21 modules for the shaded H2 version.
Improvements include a new DatabaseVersionReader which attempts to determine the database version based on the format: field in the H2 binary file header. The Reader maps the format version to the driver version for subsequent processing.
Additional changes include the DatabaseRepackager, which supports reading H2 2.1 files and replacing org.h2 package name references with the shaded package names. This approach allow runtime reflection operations to work when reading and exporting H2 files from NiFi 1.16.0 through NiFi 1.22.0.
H2 migration supports both NiFi and NiFi Registry databases.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

+1 LGTM, reproduced the issue and verified the upgrade works with the fix. Thanks for this solution! Merging to support/nifi-1.x


Artifact download and upload path syntax differs from unix tools like cp and mv

Double check the behaviour of the artifact downloading gear:

And make sure it's consistent with the rest of the unix world with how tools like cp work

Also it may be worth adding options like -R for recursive copies

Y'know, on looking at this, I actually don't think I agree:
It makes sense to me that buildkite-agent artifact upload foo/bar/bang.txt and then buildkite-agent artifact download y/ would result in y/foo/bar/bang.txt.
I'd be more inclined to look to rsync (http://man7.org/linux/man-pages/man1/rsync.1.html) for conventions than I would cp and mv.

I’ve been running in to a version of this in my pipelines 😅
Right now there’s a 1:1 correlation between the file system at upload time / the glob given to the upload command and the "artifact key space". I often rearrange the file system (or cd) so that I can buildkite-agent artifact upload without a prefix, but that doesn’t work when using the artifact_paths step property which I’d prefer to use over adding an entry to my commands array. Personally I like to drop the path prefix to my artifacts between steps to decouple the artifact name from how the repository is laid out today, so I can change the layout without having to think about my pipelines, this might just be me though 😄
What would you think about a destination syntax or a new option for where in the artifact key space things should be placed when uploading?


User's trying to delete user's  inactive ID gives 500 error

For example if I (user loarie) try to delete either inactive IDs on this observaton http://www.inaturalist.org/observations/915639
I get a 500 error on the identification, e.g.
http://www.inaturalist.org/identifications/1635471

seems to be fixed

seem fixed


handle server response on category duplicate

Steps:

Create category with name "aaa" and other category

Click on other category
Click change name category

Input category name - "aaa"

the server response should be processed, a category duplication error message should be shown

@Andrew-Sem


Suggestions disappear upon program switch

... and fails to reappear on program regaining focus.
E.g.

Switch to Notepad:

Suggestion gone.
Switch back:

Suggestions still missing.
Workaround for second case: Press Ctrl+Space

I believe this is intended behavior of the panel but you can always use the CTRL + SPACE combination at the end of the word to open the suggestion panel again.

I believe this is intended behavior of the panel

I can't imagine why it would be. It does nothing but disadvantage the user.

but you can always use the CTRL + SPACE combination

As I already said. And for the second case only.


Fixing squid:S00108 - Nested blocks of code should not be left empty

This pull request is focused on resolving occurrences of Sonar rule "squid:S00108 - Nested blocks of code should not be left empty". You can find more information about the issue here: https://dev.eclipse.org/sonar/rules/show/squid:S00108
Please let me know if you have any questions.
Sameer Misger

Hey
Sorry for late response, but could you your PRs point to dev branch instead of master?
Thanks!


arm64 - convert URL to PDF: convert to PDF: chromium PDF: chrome failed to start

Discussed in https://github.com/gotenberg/gotenberg/discussions/693

Originally posted by keyurmistry21 September 25, 2023
Hi,
I just Implemented gotenberg.
docker-compose.yml
gotenburg:
   image: gotenberg/gotenberg:7
   ports:
     - "3009:3000"

I ran the command
curl \
--request POST 'http://localhost:3009/forms/chromium/convert/url' \
--form 'url="https://sparksuite.github.io/simple-html-invoice-template/"' \
-o my.pdf

I got an error like this:
{"level":"error","ts":1695674237.8608663,"logger":"api","msg":"convert URL to PDF: convert to PDF: chromium PDF: chrome failed to start:\nqemu: uncaught target signal 5 (Trace/breakpoint trap) - core dumped\nqemu: uncaught target signal 5 (Trace/breakpoint trap) - core dumped\n[0925/203717.599775:ERROR:bus.cc(399)] Failed to connect to the bus: Failed to connect to socket /run/dbus/system_bus_socket: No such file or directory\n[0925/203717.805805:ERROR:scoped_ptrace_attach.cc(27)] ptrace: Function not implemented (38)\nAssertion failed: p_rcu_reader->depth != 0 (/qemu/include/qemu/rcu.h: rcu_read_unlock: 101)\n","trace":"895bbd5e-f793-4ab1-adad-4d353daddaac","remote_ip":"<IP_ADDRESS>","host":"localhost:3009","uri":"/forms/chromium/convert/url","method":"POST","path":"/forms/chromium/convert/url","referer":"","user_agent":"curl/8.1.2","status":500,"latency":2065853126,"latency_human":"2.065853126s","bytes_in":160,"bytes_out":21}
Does anyone have any solution ? I'm using Mac M1 pro.

Hello @keyurmistry21,
Looks like an issue from the arm64 platform Docker image variant. Which version are you actually using? 7.9.2? Could you try to downgrade the Gotenberg version until it works?

Oh, that works. But I'm curious why it hasn't been integrated with the latest version?

Did you find a working version of Gotenberg? Which one? 😊
My guess is that newer versions of Chromium might be the culprits.

Closing this for now. Feel free to reopen if it is happening with latest version.


[fix] directly define __int32 and __int64 for Coverity scans using icpx

Description
Coverity logs show that while building some of oneDAL is possible, some of the macros associated with the __INTEL_LLVM_COMPILER are not being properly handled and may be causing failures.  This directly defined __int32 and __int64 if not previously defined and solves the most occurring error. This is a more direct approach instead of querying for the compiler.

Checklist to comply with before moving PR from draft:
PR completeness and readability

[ ] I have reviewed my changes thoroughly before submitting this pull request.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation to reflect the changes or created a separate PR with update and provided its number in the description, if necessary.
[ ] Git commit message contains an appropriate signed-off-by string (see CONTRIBUTING.md for details).
[ ] I have added a respective label(s) to PR if I have a permission for that.
[ ] I have resolved any merge conflicts that might occur with the base branch.

Testing

[ ] I have run it locally and tested the changes extensively.
[ ] All CI jobs are green or I have provided justification why they aren't.
[ ] I have extended testing suite if new functionality was introduced in this PR.

Performance

[ ] I have measured performance for affected algorithms using scikit-learn_bench and provided at least summary table with measured data, if performance change is expected.
[ ] I have provided justification why performance has changed or why changes are not expected.
[ ] I have provided justification why quality metrics have changed or why changes are not expected.
[ ] I have extended benchmarking suite and provided corresponding scikit-learn_bench PR if new measurable functionality was introduced in this PR.

/intelci: run


Your new template is pending approval

When running python demo.py
Why do I get mail saying:
"Thank you for creating a new Awaaz.De template. Please allow up to 12 hours for it to be reviewed and activated by AD Staff."?
And even after 12 hours I haven't received any notification.

@ShivangiM There should be someone from my team reached out to you. If not please reopen the issue and comment here.


Flows stopped due to missing node types. Check logs for details. node-red

when i close the node-red and return back some node is gone !!

I have the same issue. Log indicates that its "waiting for missing types to be registered" of "mssql-cn" and "mssql". I am not sure but I have mssql-port node installed and it may be conflicting?

One time this happened to me and it was a bad config node.  In the editor you can click on the hamburger menu and select Configuration Nodes to see if any of them are causing an issue.


accesskey attribute on non-focusable elements (should cause a click element to fire

accesskey attribute on non-focusable elements (should cause a click element to fire: http://www.whatwg.org/specs/web-apps/current-work/#using-the-accesskey-attribute-to-define-a-command-on-other-elements).

This issue was imported from Google Moderator
Moderator votes: +4
Issue added by Hixie on 2012-04-15
To vote this issue up or down, simply include +1 or -1 in your comment.

+1

:+1:

+1

+1


License?

This repo currently lacks a license, can one be added? Thanks

Thanks for pointing this out. Done.


Add support for CommonJS style usage.

#789
Then the Browserify and Webpack will work well on bootstrap-select.
Please review it. Thanks.

Could you rebase on top of master? I updated the dist files

@t0xicCode I have rebased it.

Could this be done by using something like efacilitation/grunt-wrap-commonjs?

@t0xicCode Yes. I used grunt-umd to achieve it.

@t0xicCode Is there anything I should do?

@silviomoreto


CDN is not updated

The latest release is not available in CDN.
Because of that, the README.MD contains false data:
<script src="//cdnjs.cloudflare.com/ajax/libs/emojify.js/1.0.1/emojify.min.js"></script>

Bump

+1

+1

+1

+1


chore: expose store last authenticated user

Related Issues
https://github.com/descope/etc/issues/5329
Related PRs

branch
PR

react
https://github.com/descope/react-sdk/pull/465/files

Description
expose a way not to store last authenticated user
still need to figure out some types and test

@asafshen is this still relevant?

@asafshen is this still relevant?

Yes, I'll fix it soon


[Request] More Template import options

Hello, I wanted to ask if there could be in the future an option added, which makes it possible to use other template repositories than https://github.com/PufferPanel/templates. So that I could just insert the URL of the Template Repository, and then I could import all templates of another template repository.
And it would be great if it would be possible to add templates by uploading a JSON file and with this the panel creates the template.
Thanks

Adding other reporsitories besides this one would be kinda counterproductive because there is no guarantee for the templates to work not if they are really that what they claim to be, as for the upload you can select json in the upper right corner and copy the json in.

For the JSON, I know, but the option to direct upload templates to the panel would be much faster.
And with the repository I meant that the user has the option to fully manual add them, so they should not be included with the panel.

You can use the template editor in the UI to "upload" templates

You can use the template editor in the UI to "upload" templates

I know, but opening the file, copying the content, creating a template, opening the JSON tab from the template, removing the default content, pasting the new content, setting a name of the template which will be anyway get override by the template and saving, takes a bit more time than if it would be possible to just drag and drop the JSON over the templates tab from the panel, and it is imported. I know these steps are very fast done, but I would like it if this would be possible.


failed to set up socratex

I tried to deploy socratex following the instructions but got errors as below:
node:internal/process/esm_loader:100
internalBinding('errors').triggerUncaughtException(
^
AssertionError [ERR_ASSERTION]: Width must be between $ and 119.
at globalThis.assert (file:///home/dana_cheng/socratex/node_modules/utilitas/lib/horizon.mjs:15:22)
at Module.renderBox (file:///home/dana_cheng/socratex/node_modules/utilitas/lib/utilitas.mjs:534:5)
at initPrimary (file:///home/dana_cheng/socratex/main.mjs:146:30)
at runFunc (file:///home/dana_cheng/socratex/node_modules/utilitas/lib/callosum.mjs:27:52)
at Module.init (file:///home/dana_cheng/socratex/node_modules/utilitas/lib/callosum.mjs:121:15)
at async file:///home/dana_cheng/socratex/main.mjs:118:1 {
generatedMessage: false,
code: 'ERR_ASSERTION',
actual: false,
expected: true,
operator: '==',
status: undefined
}
Node.js v19.9.0
I tried both modern method and classic method. Same errors.
Also I tried to install it with Docker. I got nothing but this:
09350b3ddcbbd137b50681fe7119cb3198005627f8aabf90520b4fe62318374b
I tried to set it up on two Ubuntu VMs. I am stuck with the same error. Not sure what's missing?

Sorry, I will fix this issue tomorrow. 🤣

Fixed, thank you for submitting this issue. 🙏

Thank you so much Leask.
Now I got some progress. But I still didn't get what I expect. The
following is the output when I installed Socratex in Classic mode:
----------------------start of output-------------------------
╭ socratex v2.0.15
────────────────────────────────────────────────────────────────────────────────────────────────────╮
│
│
│ * Token authentication:
│
│   - PAC  :
https://www6.buntzen.xyz/proxy.pac?token=7168b6e6-5b56-3e7d-53e2-f7da931f3930
│
│   - WPAD :
https://www6.buntzen.xyz/wpad.dat?token=7168b6e6-5b56-3e7d-53e2-f7da931f3930
│
│   - Log  :
https://www6.buntzen.xyz/console?token=7168b6e6-5b56-3e7d-53e2-f7da931f3930
│
│
│
│ * Get help:
│
│   - GitHub : https://github.com/Leask/socratex
│
│   - Email  : Leask Wong @.>
│
│
│
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
[EVENT 2023-04-23T01:19:18.739Z] Initialized.
[EVENT > CALLOSUM 2023-04-23T01:19:18.740Z] Scheduled.
@. Secure Web Proxy started at https://www6.buntzen.xyz:443 (IPv6
::).
@.*** Secure Web Proxy started at https://www6.buntzen.xyz:443 (IPv6
::).
[CALLOSUM] Successfully launched within 5.44 seconds.
[EVENT > SSL 2023-04-23T01:19:22.708Z] Scheduled.
[EVENT > WEB 2023-04-23T01:19:22.709Z] Scheduled.
[SSL] Challenge HTTP Server started:
{"address":"::","family":"IPv6","port":80}
[WEB 2023-04-23T01:19:22.711Z] Restored 0 session(s), 0 bypass-item(s).
[WEB 2023-04-23T01:19:22.748Z] Saved 0 session(s), 0 bypass-item(s).
[WEB 2023-04-23T01:20:22.810Z] Saved 0 session(s), 0 bypass-item(s).
[WEB 2023-04-23T01:21:22.877Z] Saved 0 session(s), 0 bypass-item(s).
[WEB 2023-04-23T01:22:22.941Z] Saved 0 session(s), 0 bypass-item(s).
-------------------------- end of output---------------------------
I don't see the message about updating certificate. The output is not like
what you post in Github either.
I tried to set up proxy script using the PAC URL shown in the output. But
unfortunately the proxy doesn't work. I tested it in Firefox and got
timedout error like below.
Connection Timedout
Reference #0.3a65edea.8efb2b0e.1682213068
I don't know what I missed?
Thanks again.
Josh
On Tue, Apr 18, 2023 at 8:37 PM Leask @.***> wrote:

Fixed, thank you for submitting this issue. 🙏
—
Reply to this email directly, view it on GitHub
https://github.com/Leask/socratex/issues/9#issuecomment-1514086192, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A7IQFSZIAWQNWDOP736VVXLXB5MV7ANCNFSM6AAAAAAXCFPNNI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi, I met a similar problem and I think the reason is that ssl.init() does not create cert on the first time.

fixed https://github.com/Leask/socratex/issues/11


Requesting to be a Guest in the next Community Committee meeting.

Hi, I would like to be a Guest in the next Community Committee Meeting

@yourwpmadesimple always welcome.
You can find details about meeting dates and how to join here: https://github.com/nodejs/community-committee#meetings


The focus of mouse was not on the correct element

Repro steps:
• Navigate to https://makecode.microbit.org/beta on FireFox
• Drag block "show leds" and put it under the block "on button A pressed"
• Change the all element values to 1 in the forth line of the block "show leds"
• Move the mouse to the elements in the third line
Expected result:
The focus was on the elements in the third line
Actual result:
The focus was still on the elements in the forth line when the mouse was on the bottom of the element in the third line, and when clicked the mouse, the value of element in the forth line was changed.
Additional information:
The focus was on the element in the third line if the value of element in the forth line didn't change to 1

Dupe of #2721 ?

Still repro on firefox(58.0.2)