Removing First Line, But It's Not Empty

I've been seeing empty transcription results, and did some digging to find that you're trying to remove the first line assuming that it is empty. In my testing, it's never empty. Perhaps you could create a more elegant solution that looks for empty array elements after it's parsed and then remove those?
https://github.com/ariym/whisper-node/blob/92e8b86200416a6af9e5698a02481a54811e1321/src/tsToArray.ts#L11

Same here. The first line is never empty for me, even if I add a long blank at the beginning of the audio. I removed the shift() function, and it seems to work fine. Also, I often see [BLANK_AUDIO] at the end of my transcript array, which can be easily filtered out.

Could it be that the first line is the translation?
I recorded in german but my first line was in english


Ability to pass client certificate to backend

What would you like to be added:
The ability to pass client certificates to the backend when using frontendValidation within the Gateway resource.
The path is: spec.listeners.tls.frontendValidation.
When this is enabled, the frontend prompts the client to provide their certificate, which should then be passed to the backend services in a header.
Why this is needed:
In a lot of scenarios, some applications needs a client certificate. This is a standard procedure, a lot of other ingress controllers has.

Are you talking about XFCC or something else?

Are you talking about XFCC or something else?

Exactly that, yes.
In Traefik its X-Forwarded-Tls-Client-Cert.
Apparently, Envoy is X-Forwarded-Client-Cert


[5.6] Operator optional!

Sending to branch master as recommended https://github.com/laravel/framework/pull/22361

Could you also add some tests to this functionality?

@sisve Done!

Should we change orWhereTime signature too?
From:
public function orWhereTime($column, $operator, $value)

To:
public function orWhereTime($column, $operator, $value = null)

@a-komarev
done!

@themsaid can you review this?


BatcherImplTest is flaky

BatcherImplTest sometimes fails with the following log:
com.google.api.gax.batching.BatcherImplTest > testThrottlingBlocking FAILED
    org.mockito.exceptions.base.MockitoException:
    No argument value was captured!
    You might have forgotten to use argument.capture() in verify()...
    ...or you used capture() in stubbing but stubbed method was not called.
    Be aware that it is recommended to use capture() only with verify()
    Examples of correct argument capturing:
        ArgumentCaptor<Person> argument = ArgumentCaptor.forClass(Person.class);
        verify(mock).doSomething(argument.capture());
        assertEquals("John", argument.getValue().getName());
        at com.google.api.gax.batching.BatcherImplTest.testThrottlingBlocking(BatcherImplTest.java:895)

Duplicate of #1615


Add functions : isDirectory, grep

For checking file is directory and filtering the result list, add two functions.

Thank you @jsharp83 !!


CI: add default drop all test

Steps:

Immediately start a container with cilium in default enforcement mode with a default deny policy (ingress, egress) enabled that selects the labels of the container
ping the container: no packets should get through ever to the endpoint.

I can take a look at this.


Field description is not visible

[Enter feedback here]
Field description is not visible when seeing the list of all fields in the organization from Org settings->Process->Field
Looks like a tool bug

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: aa7a0948-ee09-8734-ed6e-bc45bd78ce01
Version Independent ID: da9ac11c-958d-f5dd-efad-0c20c2d7cf2f
Content: Work item fields, attributes and how to list them - Azure Boards
Content Source: docs/boards/work-items/work-item-fields.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@angiridhar  - Thank you for the feedback. I will open a bug for the product team to review.


Compose dataframe inspecting

The idea is to keep the "shell" of the inspect representation
constant but delegate the remaining to a shared implementation.
df_as_str was removed because we are missing the polars flag
to make it work, so I thought we could skip it too.

If everyone is happy, I can try something similar for Series.

Any idea what I did wrong with the Rust code? :D

The Rust code is fine -- same problem I mentioned in #229. :)

Implementation for series is in. It also expects a backend but we don't use it, I will open up a separate issue for that (it is a large change which I can't work on because of my arm injury).

A rebase will get checks to pass after I merge #234. Then I'll fix #202 to use this and drop lazy/0 and we can go on from there :).

@cigrainger please go ahead and merge #202 and I will rebase this after (going to bed now) And pls don’t drop lazy/0, just don’t expose it :) I will work on exposing it tomorrow.


fix to issue #1

Removed jquery file and added CDN as per https://code.jquery.com/

Thank you!

No problem!


[Feature Request]  Spanish first names (male and female)

Added spanish firstnames (male and female)

https://github.com/chancejs/chancejs/pull/409

Is there any way to chance the language by default @utahraul ?


Adding Padding To "THE BEST TECH STARTUP SECTION" #301

Pull Request Template

Please provide all of the details below. Pull requests missing any of the information below will be closed!
Providing the issue number/url helps us identify the problem being solved and keep the issues tab clean and tidy

Summarise what this PR fixes
Adding Padding To "THE BEST TECH STARTUP SECTION"
Changing the H1 and P Font Size and alignment
What ISSUE NUMBER does this fix?
Fixes #301
Checklist:
(Add an x between the brackets to check the items)

[x] I have summarised what this PR fixes above
[x] I have provided the issue ID/URL above
[x] This PR fixes only the issue mentioned above (create separate PRs for separate issues)
[x] I was the first one to claim the issue or I created the issue

Looks better!


fix map bug

fix map bug

@Ye-Ting could you add a test case that verifies the presence of this issue?? It's good to have that for preventing any further regression issues.
And thx for contributing :smile: !

I add a test case .
Is the usage your first thought ?
or have other problem ?

@Ye-Ting I'll merge it in later today. Thx for your effort!

@Ye-Ting I'm going to fix another thing I'm about to resolve in another branch and then I'll release a new version with your fix. Thx again!


Space Travelers Hub Project add-display-function

In this branch following  changes done:
-Added display function in Missions.jsx file
-Checked linters

Hi @otmanTR ,
Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights
✅ Correct use of Github workflow
✅ No linter error
✅ Tasks are completed according to the requirements 👍


fixed broken favicon link, previous/next post links due to non null b…

Sorry these have come in chunks. I'm fixing them as I come across them not specifically looking for them.

No problem, thanks 😄


Rust compiler do not provide useful information when there are unsatisfied constraints inside CGP component wiring

Currently, when there is any unsatisfied constraint inside a context-generic provider implementation, it is not shown in the error message when trying to implement a consumer trait. Due to the indirection through blanket implementations, Rust only prints an unhelpful message that the blanket implementation is not implemented without explaining why. More details about this issue is explained in the debugging chapter of CGP book.
There is not much we can do within this project to fix the issue. Instead, this has been reported to the Rust project at https://github.com/rust-lang/rust/issues/134346, with a fix PR available at https://github.com/rust-lang/rust/pull/134348. This serves as a tracking issue for tracking the progress of fixing this at the Rust compiler.

It seems like the patch from https://github.com/rust-lang/rust/pull/134348 does not cover the case when type equality constraints are not satisfied. I will have to investigate other code paths in Rust's error reporting, and see whether it is possible to report constraint errors caused by type inequalities.

Thank you for implemening this crate.
I commented on the GH discussion in the book repo before I found this issue; here is the link as reference.
https://github.com/contextgeneric/cgp-patterns/discussions/1#discussioncomment-11683666
In a nutshell, I have a use case that very closely resembles the problem CGP solves. I also poked my buddies over at the Iggy project to take a closer look especially w.r.t. to their work on serialization.
For the error issue, I did read through the linked compiler issue and I sense that patching the Rust compiler feels a bit like the hard road thus the suggestion to at least consider the option of generating check traits with a macro.
For the most part, I only need to know which trait bound is not satisfied to find a solution and, from what I saw in the book, the check traut gives me exactly this. In case the default generated check trait doesn't do that trick, I can still remove the derive, implement the check manually and uncomment & debug line by line. Based ony experience, that might be the case in less than 10% to 15% of the issues arising from trait bound violations so having a sensible default for the remaining 80% is certainly more than good enough to get substantial work done.

@marvin-hansen thanks for your feedback! I have replied you on the other thread. In my experience of using CGP in large scale projects, debugging unsatisfied indirect dependencies are the biggest challenge for using CGP, and the check traits approach is not general enough to handle more advanced cases. I'm still experimenting on better ways to support debugging CGP programs, so hopefully we'll find better approach in the near future.
I'm happy to provide help to solve any challenge you have in the Iggy project. If you can point me to a specific problem, I can help you prototype some changes together and see if it solves the problem. It could also serve as a good blog post for CGP.

@soareschen
Thank you so much. Just to clarify, I'm not a contributor to the Iggy project myself, it just so happened that because I'm using the Iggy message streaming myself in a relatively large project (> 100 crates) that I came to know the team and their current challenges. I cannot vouch if they are going to use this crate, but they are looking into it.
That said, I'm working on a pluggable integration design pattern with static dispatch using the enum_dispatch crate predominantly for usage in a template service. It's a mind boggling challenge and once the design pattern stands, it will be used to integrate probably up to 50 data sources.
Until I found this crate via TWIR, I had exactly the problem of dispatching multiple default implementations...
Because my V0.1 with proc macros and a lot of trait hackery just worked out last week and nothing has been scaled up yet, I'm okay with rewriting everything using CGP. Another reason is maintainability, flexibility in changing providers later. Basically everything that GCP solves. Error handling really comes second because there is more than one way to solve this, but there are only very few ways to deal with loosely coupling and composing providers and consumers...
I had a chat with some other engineer and the general interest in a no compromise, relatively low code, and full static dispatch zero cost abstraction design pattern for data integration is indeed there.
This may take bit of time, but when it's all set and done, writing a blog post about pluggable integration with CGP is already on my list.

I have moved the discussion for this topic to https://github.com/orgs/contextgeneric/discussions/3.


Make reindexing work with child documents

The scanner now requests the _parent field (on top of the default _source field).
When the reindexer finds that field in the hit, it sets parent on the IndexRequest.
In index_test: Added a new 'comment' type that is a child of 'tweet' together with its mapping.
Added a comment instance to the test dataset.

Wonderful. Thank you. Give me some time to review.
Why did you disable the reindexer tests?

Ah. Forget it. Tests are okay. I'm probably in weekend mode...

When adding this I looked at the source of stream2es, a tool that is able to copy child documents across indexes. They include _source and _routing in the scan request: https://github.com/elastic/stream2es/blob/86bcf684034d912e25cce754e8ea77b6fcb5ff9d/src/stream2es/es.clj#L96 and the reuse that data when indexing. I thought that adding _parent always made sense, but arbitrarily decided not to include _routing. After reading the ES documentation more attentively I now think that _routing should be added as well: https://www.elastic.co/blog/customizing-your-document-routing
I agree with you that the ScanService should take a Fields method, and the Reindexer should be the one telling the ScanService to include those fields in all reindexing operations.
Please tell me if you agree with the approach and I will add a commit to this PR.

Thanks for clarifying.
I agree with your approach. Let me add two more thoughts.
The official Python client also has helpers for search & scan and bulk indexing. It's a very good source, and it adds other fields as well in certain cases (just skimmed over it right now). Maybe we should have a look at what they're doing.
There is a generate-api branch which consumes the REST API definition from Elasticsearch and uses go:generate to generate service templates. It's not quite complete, could get some more love, but it's a good approximation to the final service. Plus: It adds a REST API compliant service. I just added it a few weeks ago, so services like SearchService were created before having the generator. Why am I saying this? Because you might use it to see what's also still missing in the ScanService besides the Fields setter.

Still in the process of writing tests for this PR, as part of the test I am using the GetService and found a bug, the fix is in PR: https://github.com/olivere/elastic/pull/69
If you are able to merge it I would then like to rebase all the commits in this PR (68) on top of the new master but am not sure github will be able to continue associating the rebased commits to this PR?

Already merged. Looks good. :-) If they only had a JSON spec for the response types.

So I rebundled the initial commits into one that includes the Fields method on ScanService.
I looked at https://github.com/olivere/elastic/tree/generate-api and saw search.gen.go with 40 attributes for SearchService, it didn't help me deciding which methods to add to the ScanService.
I looked at the python client and almost none of the fields they inject into the index-request can be obtained from a search or scan query https://github.com/elastic/elasticsearch-py/blob/master/elasticsearch/helpers/init.py#L30-L31 . This is not a bug, but is misleading.
I am currently finalizing a second commit for this PR, that allows the user to optionally take control of the mapping from SearchHit to BulkIndexRequest. This is similar to the expand_action_callback of the python client: https://github.com/elastic/elasticsearch-py/blob/master/elasticsearch/helpers/init.py#L43

Ha. You probably had to add them all to be compliant ;-)
Re the PR: No worries. Take your time. We don't need to rush anything. Your commits are high-quality. Thanks for your work until now.

Re the expand_action: Isn't it for passing these parameters to the Bulk API? They don't have to be part of the response; they could've been added by the function passed into expand_action_callback.

You are right: I hadn't thought of the use of expand_action outside the scope of a reindexing.

OK so the PR is ready for review:

The reindexer by default honors the _routing and _parent fields, I believes this is a sane default.
If a user wants to do something different he can specify the fields the Scanner should return and/or specify the HitToIndexRequest function. There is a test-case that demonstrates copying the _ttl of each document from source to target. The HitToIndexRequest function can also be used to perform random transformation of data while reindexing, like for instance modifying felds for data migrations.

The HitToIndexRequest function, enabling transformations, is really helpful in my use cases. Will review and merge tomorrow morning if it's looking good at second sight.

I think you are right. It's called a Reindexer after all. So we go for option c) and leave your PR as is. If we want a transformation utility, we can write a `Transformer.
If you agree with that, I'll merge now.

I'd like to explore idea d) today because it may not just bring us extra flexibility, but also a much clearer separation of labor between the reindexer (provides the machinery) and the handler (do the actual work).
I'd do this in another branch. At the end of the day you would merge either what we have now or design d).

Okay. I like idea d). Much better than what I've had in mind. It's just that I don't want Reindexer to do too much as it might get too complex. Time for exploration, I guess :-)

Alernative d) is in https://github.com/olivere/elastic/pull/71

Yes. I've seen that. Thank you. I'd prefer PR #71. What about you?  (Also see a last question regarding naming in #71).

I also prefer #71. I linked the two PRs together so that github will remember the discussions here brought us to #71


[Apple Reminders] No way to add back the reminder permission after removing it

Extension
https://www.raycast.com/raycast/apple-reminders
Description
I needed to remove the reminder permission from an app, so I used tccutils reset Reminders, and now raycast won't ask me aboiut the reminder permission again and I can't add it back :(
Steps To Reproduce

use Apple Reminders and accept reminder permission
run tccutils reset Reminders
Can't add reminder permission back again :(

Current Behaviour
Can't add reminder permission back again :(
Expected Behaviour
Can add reminder permission back again :)

Don't you have a disabled toggle over here?

Don't you have a disabled toggle over here?

I do not because of the command I ran. I think I found a temporary fix; only one of the buttons in the extension setting would ask for that permission, and it's not every time.

I think an easier fix would be to have an ask permission toggle/button in raycast's ui itself, and not in the extension.

only one of the buttons in the extension setting would ask for that permission, and it's not every time.

Which one?

Sorry for the late reply. The button that worked was the activate button in menu bar reminders.

Although now the permission is gone again and doesn't work anymore

OK I actually found the way to to add it back correctly.
https://github.com/DocSystem/tccutil/
using that tool, i run sudo python3 ~/tccutil.py --reminders -e and drag the raycast app back. This worked. Maybe you can add the repo url to the (that place on the left)


netstat change uint size from 32 to 64 bits

Signed-off-by: binjip978<EMAIL_ADDRESS>

fixes https://github.com/prometheus/procfs/issues/418
also minor change to struct layout to use 16 bytes instead of 24 according to fieldalignment analyser.


when a typedef is inside a module, return types stop linking to the type definition

In the following example, when jsdoc is generated (with the default template and docstrap) there is no link from the return type to MyType.
If I remove the module definition at the start it fixes it.
/** @module my/module */

/** @typedef {Object} MyType
    @property {string} prop
*/

/**
  * My class
  * @class
  */
function Func() {
}

/**
   @returns {MyType} the return type
  */
Func.prototype.myfunc = function() {
};

I've tried prefixing MyType with the module and it doesn't help.

This is not exactly obvious, but a typedef inside a module has inner scope by default. So this will work to link to MyType:
/**
 * @returns {module:my/module~MyType} the return type
 */
Func.prototype.myfunc = function() {};

If you prefer, you can add a @static tag to the typedef, and you can then link to it as module:my/module.MyType.

Hey guys it would be nice to have if you parse the @returns or @param {module:my/module~MyType}
so when you show the link it only says MyType and not module:my/module~MyType

Alternatively you can just mark the inner typedef as @global and just refer to it as {MyType} anywhere you like, if you don't mind littering your global namespace with module-specific types.
This is irritating though - if inner typedefs are listed in the JSDoc documentation page for a module, what reasonable rationale is there for not linking to that typedef definition when it's used in JSDoc comments in that same module?


Update API -> complete becomes fit_transform

Not too much going on here, but I did add some commentary to README.md.
I also deleted test_knn_timings.py. It was timing things that don't exist anymore.
@iskandr Do you have time to make KNN inductive? Should be pretty easy?

Pull Request Test Coverage Report for Build 146

4 of 6 (66.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 67.144%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fancyimpute/solver.py
3
5
60.0%

Totals

Change from base Build 144:
-0.1%

Covered Lines:
797

Relevant Lines:
1187

💛  - Coveralls

Pull Request Test Coverage Report for Build 146

4 of 6 (66.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 67.144%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fancyimpute/solver.py
3
5
60.0%

Totals

Change from base Build 144:
-0.1%

Covered Lines:
797

Relevant Lines:
1187

💛  - Coveralls

@iskandr I don't really feel like I need much of a review here. I'll wait a week and then merge.

@iskandr I asked Ilya to join our party and review stuff, so gonna merge.


Captcha Not Shwoing

Captcha Not Shwoing

Hi @mrityunjaybhagat this project is archived please find new project here https://github.com/AjinkyaBijwe/WebRTCVideoCall


Village claim erase + someone claim over it

IGN: lovetodead
My town, Village, had a base before the error (the error occur between the server restart and when i make the ticket , i don't know why my base was unclaim itself ). I had 2 town members and we are active.  But now someone just stole our claim base and i can prove it , last day my girlfriend work on the front window so i have some screen , anyways , so the guy stole our stuff and claim over our base and i don't know how to get back all my stuff and my base . The name of who claim over it and actually stole our base and stuff is TinyBananas ( btw he use our ressouces and pixelmon item for only god and moderator know )
Ps: TinyBananas is the name of the town ,i think
Coords are -61 63 -7675
i give you some screen of my base .

ps: one more proof the sign are ritten in french and we are french ( by the way sorry if i make something strange with the grammar )
PS: i just see the mayor change ( not a big threat but well ^^' )

their claim has been deleted, issue resolved.


Anomalous non-use of optAnnotations in grammar

In P4_16 Language Specification 2017-05-22, Section E contains the production:
optAnnotations
    : /* empty */
    | annotations
    ;

which makes it convenient to emplace optional annotations into all the different places in the grammar in which they can appear.  However, there are a few places in the grammar that anomalously decline to use this convenient non-terminal, namely:
instantiation
    : typeRef '(' argumentList ')' name ';'
    : annotations typeRef '(' argumentList ')' name ';'
    ;

typedefDeclaration
    : annotations TYPEDEF typeRef name ';'
    | TYPEDEF typeRef name ';'
    | annotations TYPEDEF derivedTypeDeclaration name ';'
    | TYPEDEF derivedTypeDeclarationame ';'
    ;

variableDeclaration
    : annotations typeRef name optInitializer ';'
    | typeRef name optInitializer ';'
    ;

Now, the fourth alternative for typedefDeclaration is clearly a typo, since obviously it should read
    | TYPEDEF derivedTypeDeclaration name ';'

Assuming that this typo is corrected, then the reader is left to wonder what in the world the grammar is trying to imply in these productions by using extra verbiage rather than the convenient non-terminal optAnnotations.
I understand that the grammar has been transcribed from the parser definition and these anomalies are probably the result of historical accident.  Nonetheless, they should be cleaned up.

We have fixed the typos in the master document, but we haven't yet released the new pdf/html. I think that we should rebuild the pdf/html for such small bug-fixes.
Unfortunately simplifying some of these productions with optAnnotations makes the grammar ambiguous. The grammar could be much nicer without bison limitations. It looks like we can simplify the typedef production, though.

Good point. We can post an official live "master" version on
p4lang.github.io. I'll take care of this.
-N
On Fri, Jun 23, 2017 at 3:01 PM, Mihai Budiu<EMAIL_ADDRESS>wrote:

We have fixed the typos in the master document, but we haven't yet
released the new pdf/html. I think that we should rebuild the pdf/html for
such small bug-fixes.
Unfortunately simplifying some of these productions with optAnnotations
makes the grammar ambiguous. The grammar could be much nicer without bison
limitations. It looks like we can simplify the typedef production, though.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/p4lang/p4-spec/issues/295#issuecomment-310782601, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABwi0m7L1ppzTRZ55hdxscyE9zltbyM9ks5sHDWogaJpZM4OEEyL
.

I will submit a PR to change the grammar for typedef.
I have tried the other two changes on the bison file (frontends/parsers/p4/p4parser.ypp) and they both produce lots of shift/reduce conflicts. We have chosen to use the real bison grammar in the spec (minus some very important missing pieces, like priorities and the lexer) so that we are sure that it is implementable using bison.


add currentScript to Document

Ups.. This should be in the pull request https://github.com/Microsoft/TSJS-lib-generator/pull/12. How I do it?

Hi @dashaus, thanks for contributing! To update your pull request you can just push new commit to the same branch (which is patch-1)


Feature request: edit title and authors from web UI

Might be nice since otherwise we don't have a way to specify those information for PDFs that don't have IDs.

I agree. But this need quite a lot of implementation including DB schema change. Please use tag for now.

https://github.com/akawashiro/jendeley/pull/245 resolves this.

Great, thanks!


code assistant integration

https://st.yandex-team.ru/INFRAML-1023

first need to do this https://github.com/ydb-platform/ydb-embedded-ui/issues/1716


Cache problem

System Information :

OS: Windows 11
Snap Camera Version: 1.21
Signature Patch applied: yes

I can't get the cache files I keep getting Network Error error

Log.txt

Is this URL reachable in your browser?
https://studio-app.snapchat.com/vc/v1/import/cache

No :(

Is your Docker Server running and working?

yes running

Did you edit any config files?
Cache import must be enabled in the config.yml (it's enabled by default)
Also is https://studio-app.snapchat.com reachable at all?
And is your snap camera app working?

not edited
no I can't access
filters in the application do not come


⚠ Samsung has degraded performance

In a675db7, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 6842 ms

Resolved: Samsung performance has improved in c30324d.


Create separate folders for each Lab

I think we should have Lab 9 and Lab 10 as seperate folders, containing the python files and for lab 9 the python and matlab files. Also the report should be one full html doc.

full report from both labs is report.ipynb


Update the readme/Makefile

So i update the Readme to let the user know that the Makefle is broken and i also made the Readme.md easy to understand a bit more .
Thanks For Reading
TigerClips1

Thanks, I did some edits as well, debugging in PS4 seems to be possible only in the Rider, I wasn't able to enable the VS remote debugger with the PS4, so I edit the info a bit.


Do we really need ceph-salt grains?

(Relying on the pillar is more secure because only the master node can set it)
For targetting, we can use pillar:
salt -I ceph-salt:minions state.apply <state>

instead of grain:
salt -G ceph-salt:member state.apply <state>

So, apart from ceph-salt:execution grains that are set during execution for orchestration, do we really need any other ceph-salt grains?
(AFAIK, Suse Manager doesn't support grains)

I think we still need grains when we want to target minion by role, for instance:
salt -G ceph-salt:roles:admin test.ping


Had this bug on the newest version(Beta19) 1.16.5 forge 36.1.31

// I bet Cylons wouldn't have this problem.


Fix enums for consumers using isolated modules.

When importing @lastolivegames/becsy in a composite project or a project using the isolatedModules flag, the following error is encountered:
node_modules/@lastolivegames/becsy/index.umd.d.ts:104:37 - error TS2748: Cannot access ambient const enums when the '--isolatedModules' flag is provided.

104 type QueryFlavorName = keyof typeof QueryFlavor;
                                        ~~~~~~~~~~~

This PR moves all const enum to enum to avoid the above interoperability issue.
Let me know what you think!

I definitely want to support isolated modules mode, which is why I have "isolatedModules": true in tsconfig.json.  I don't understand why that doesn't error on the export const enum.  Conversely, I don't understand why you get an error for module-local const enum declarations, as these are indeed isolated to the module.
While I'd like to understand why the TS compiler isn't working as expected that can often be a foolish quest.  I'm OK with this simple change as long as it doesn't affect performance -- did you try yarn perf to see?  If it does, I'd like to find another approach.

This TS issue looks potentially relevant, but it's not (yet) resolved.

did you try yarn perf to see? If it does, I'd like to find another approach.

Sure thing, details below. If I interpret this correctly my machine is ~20% slower at executing JavaScript than what you used to create the baseline. 😉 Can't see a regression going from 3df15d4 to 364fb1.

 `3df15d4` → `364fb1` benchmarks
Before 3df15d4
                        dev                   perf
ecs-benchmark
  packed1             4,775 op/s ( -18%)    73,403 op/s ( -30%)
  packed5             4,969 op/s (  -9%)    69,870 op/s ( -31%)
  simpleIter          1,227 op/s ( +15%)    51,221 op/s ( -12%)
  fragIter            6,235 op/s ( -44%)   147,718 op/s ( -23%)
  entityCycle           379 op/s ( -30%)     1,936 op/s
  addRemove           2,738 op/s (+103%)     8,072 op/s ( -13%)

ecs-packed
  packed1             4,965 op/s ( -11%)    55,412 op/s ( -18%)
  packed5             4,819 op/s ( -10%)    51,138 op/s ( -22%)
  simpleIter            959 op/s (  -5%)    27,797 op/s ( -29%)
  fragIter            9,019 op/s ( -12%)    83,361 op/s ( -35%)
  entityCycle           363 op/s ( -31%)     1,869 op/s
  addRemove           2,591 op/s (+103%)     7,443 op/s ( -14%)

ecs-packed-elastic
  packed1             4,484 op/s ( -19%)    34,239 op/s ( -23%)
  packed5             4,434 op/s ( -18%)    35,093 op/s ( -20%)
  simpleIter          1,159 op/s ( +21%)    19,884 op/s ( -10%)
  fragIter            9,307 op/s (  -9%)    65,050 op/s ( -23%)
  entityCycle           375 op/s ( -27%)     1,698 op/s
  addRemove           2,684 op/s (+113%)     7,521 op/s ( -12%)

After 364fb1
                       dev                   perf
ecs-benchmark
  packed1             5,179 op/s ( -17%)    77,992 op/s ( -31%)
  packed5             5,140 op/s ( -13%)    71,627 op/s ( -34%)
  simpleIter          1,354 op/s ( +18%)    55,067 op/s ( -12%)
  fragIter           10,028 op/s ( -16%)   149,103 op/s ( -28%)
  entityCycle           394 op/s ( -33%)     2,010 op/s
  addRemove           2,620 op/s ( +80%)     8,173 op/s ( -18%)

ecs-packed
  packed1             4,906 op/s ( -18%)    56,530 op/s ( -22%)
  packed5             4,893 op/s ( -15%)    52,032 op/s ( -27%)
  simpleIter          1,207 op/s ( +11%)    28,590 op/s ( -33%)
  fragIter            9,700 op/s ( -12%)   125,642 op/s (  -8%)
  entityCycle           376 op/s ( -33%)     1,921 op/s
  addRemove           2,703 op/s ( +97%)     7,949 op/s ( -14%)

ecs-packed-elastic
  packed1             4,766 op/s ( -20%)    33,801 op/s ( -30%)
  packed5             4,743 op/s ( -19%)    34,845 op/s ( -26%)
  simpleIter            900 op/s ( -12%)    13,317 op/s ( -44%)
  fragIter            6,932 op/s ( -37%)    61,322 op/s ( -32%)
  entityCycle           383 op/s ( -31%)     1,788 op/s (  -7%)
  addRemove           2,663 op/s ( +96%)     7,280 op/s ( -21%)

I don't understand why that doesn't error on the export const enum. Conversely, I don't understand why you get an error for module-local const enum declarations, as these are indeed isolated to the module.

This is the best explanation of the issue https://ncjamieson.com/dont-export-const-enums/#isolated-modules I've read to date. I suspect we end up implicitly exporting QueryFlavor via QueryBox['results'] → QueryFlavorName → QueryFlavor, thus the type is in fact accessed which triggers the error reported.

Thanks!  I agree that performance seems to hold up, though it looks like my efforts to adjust benchmark results according to a dynamic empty loop baseline are completely ineffective.  I'll merge and rerun the benchmarks on my machine to double check.
And yeah, I found that blog post too, but I still can't make sense of what we're seeing.  I do have isolatedModules set yet tsc isn't warning me about the issues.  I could work around the QueryFlavor export by manually constructing the type instead, but I don't want to waste time fighting the language so let's just go with non-const enums throughout as the most straightforward solution.

Released in 0.8.3 -- thanks again!


Create snv_vcf_exporter.py

Exports common variant SNV VCFs from a VDS.
Simpler version of get_genotype_vcf.py without rare variant and VRE estimation.

will clean up linting errors later - not related to this pr

As a note, I am happy for this to get through as it will unblock Hope but this will create duplicated files  / duplicated runs for the common variants and I still need the rare variant files and the VRE for my pipeline so need this error to be resolved rather than bypassed?


Is the "Send" operation defined by lwm2m1.1 supported now?

If not, any suggestion for me to achieve it?

I have it available in https://github.com/sbertin-telular/wakaama/tree/lwm2m_1.1 and hope to get it merged at some point.

Hi, how is this proceeding? Any expectations for PR?
For me the SEND is the only feature really missing to support my current use cases, so it would be nice to get this officially supported soon!

This is still stalled waiting for someone to merge PRs. I fully intend to merge the work I've done once that happens. The Eclipse IoT PMC has started to make moves to get this project active again, but I don't have any insight into when it will actually happen.

This should come soon hopefully. (see #488)


[Bug] Generated sidebar code uses single ' instead of double "

Bug report
What I did
php artisan backpack:crud page
What I expected to happen
generate

 Pages

in sidebar_content.blade.php
What happened
generated

 Pages

Note the nested single quotes.
This is not compatible with my pretty printer, it breaks the code by inserting a space
href='{{ backpack_url(' page') }}'
This is also not consistent with the other sidebar_content.blade.php code provided by backpack.
Is it a bug in the latest version of Backpack?
After I run composer update backpack/crud the bug... is it still there?
Yes
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 8.0.2 (cli) (built: Feb  3 2021 18:36:40) ( ZTS Visual C++ 2019 x64 )
Copyright (c) The PHP Group
Zend Engine v4.0.2, Copyright (c) Zend Technologies
with Zend OPcache v8.0.2, Copyright (c), by Zend Technologies
LARAVEL VERSION:
v8.77.1@994dbac5c6da856c77c81a411cff5b7d31519ca8
BACKPACK VERSION:
4.1.62@273c277e826c157ef0913962769a7ec9a44b6233

Hey @Polfo
Thanks for the report.I know it works, but it was weird indeed to have single quotes on the html elements.
I've just submitted PR #126 that should address this issue.
I am going to close this, if you happen to give that PR a try let us know if it fixed for you.
Merry Christmas,
Pedro

Heads-up @Polfo , we're currently very very deep into finishing the 4.2 launch. So you might get late answers from us on Github, when issues are minor. Let me be clear - minor issues like these are still important. They add up, and having them fixed makes up for a MUCH better developer experience. We really appreciate you submitting them, and want you to keep doing that.
But in the next few weeks, we might not answer to minor issues at all (or answer late). Please know that we saw them... we'll take care of them... but we won't spend time on them right away. Not until we're done launching 4.2.
That is the priority numero uno for us - the 4.2 launch. Afterwards, we can take care of these "quality of life" improvements.
Thanks again for helping out.
Cheers!


Managed vs unmanaged storage allocation

The section "Use TRIM with standard storage" appears to imply that trimming a disk without resizing will save money even when using managed disk. This doesn't agree with other sources, e.g.,  https://docs.microsoft.com/en-us/azure/virtual-machines/linux/standard-storage#pricing-and-billing.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: aa688416-61ed-5a02-eab0-4a39b6cdc533
Version Independent ID: c4aded8c-ae13-f863-362d-cc77f72fa023
Content: Attach a managed data disk to a Windows VM - Azure
Content Source: articles/virtual-machines/windows/attach-managed-disk-portal.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the feedback! We are currently investigating and will update you shortly.

@swilder0123 I do find this confusing as well. Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@cynthn Cost-effective Standard Storage and unmanaged and managed Azure VM disks doc says, billing is based on disk size whereas Attach a managed data disk to a Windows VM - Azure doc says you can save on billing by trimming(no mention of resizing here).
Can you please take a look and determine which doc needs an update?

+1 Managed disks are charge by allocated space not consumed so the section on 'TRIM' is misleading in an article specifically about 'managed disks'. Please remove it.

I guess the first sentence does imply that this doc is about managed disks even though nothing in the ToC suggests that. In any case, the TRIM recommendation specifically applies to unmanaged when it shows up over here:
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/about-disks-and-vhds#one-last-recommendation-use-trim-with-unmanaged-standard-disks

@cynthn It looks like the information on TRIM needs to be removed from this doc. Would you be able to confirm and update as appropriate. Need your expert touch on this. Thanks :)

@swilder0123 Thanks for raising this as an issue! I have created a pull request to remove the TRIM section. Thanks again!
#please-close


Tests for links as hashes

I created a new block called across linked objects,
inside this block there is another block where links are multihashes,
since in my view links could also be paths e.g.{name: {@link: /hash/person/name}} and I expect where links are paths to happen soon (I can just remove it for now)
The tests are not passing:

[ ] I am not sure how I can point directly to the hash of a string (see line 130 and 153)
[ ] if {about: {'@link': 'hash'}} and I look for /about, the link should be resolved, instead of returning {'@link': 'hash'}
[ ] same for friends/0/bob

(my reference spec is both the ipld one and my attempt for ip-paths)

re 1: I am not sure that is possible with IPLD/cbor as I don't know if they actually allow anything else than a map at the top level

@dignifiedquire: I think that should be just an IPFS object (not an ipld one)

You can just run ipld.multihash on any buffer to get a usable hash to simulate sth like that

After rereading the spec @link only allows for merkle links in the /ipfs object space. So sth like QmSjjs../prop is a merkle path which is not allowed as a link.

@dignifiedquire thanks for this! My tests did not test this yet and I will remove the block then(I will address the need of this feature in a different place).
We haven't yet solved 2 (correct?)

I used your previous suggestion on just using ipld.multihash
        const alice = {
          name: {
            '@link': ipld.multihash(aliceName)
          },

I got:
1) node test blocks resolve across linked objects where links are multihashes resolves link to string:
     Uncaught AssertionError: expected { Object (@link) } to deeply equal 'Alice'
      at test/ipld-tests.js:162:31
      at access (src/resolve.js:19:14)
      at access (src/resolve.js:43:7)
      at src/resolve.js:30:9
      at src/ipld-service.js:65:14
      at BufferList._callback (node_modules/ipfs-blocks/src/block-service.js:41:9)
      at BufferList.end (node_modules/bl/bl.js:98:10)
      at ReadStream.onend (_stream_readable.js:498:10)
      at endReadableNT (_stream_readable.js:913:12)

Just got word from @jbenet and merkle paths are allowed to, need to fix the resolver for that

Merged and fixed all cases except for the string one, which will not work in this way and I don't think will be needed at this level as all ipfs objects at the end of the day will become IPLD objects anyway.


Add ?Sized bound to SourceCode to allow casting &str` into it.

I'm trying to cast a borrowed &str to a dyn SourceCode but I'm receiving the error:
error[E0277]: the size for values of type str cannot be known at compilation time
and
help: the trait `Sized` is not implemented for `str`
note: required for the cast from `&str` to `&dyn SourceCode`
Which leads me to Source Code's definition, where it is said to be implemented by str and &str (as expected), but it is not marked as ?Sized which I suspect is the reason why this cast can't be made.

I think this is happening because you’re trying to turn &str into &dyn SourceCode… that is, str (which is unsized, and does NOT implement SourceCode), into dyn SourceCode
try casting &&str to &dyn SourceCode

Well, it is [implemented for str].(https://docs.rs/miette/latest/miette/trait.SourceCode.html#impl-SourceCode-for-str)
I couldn't do it with &&str because I store an owned String and I'm trying to provide a SourceCode for Diagnostic::source_code.


Unserializable data breaks app

When using crayon router the app doesn't load anymore when using the devtools.
Error on firefox:
DataCloneError: The object could not be cloned.

Error on chrome (referenced symbol can also be something else):
Uncaught DOMException: Failed to execute 'postMessage' on 'Window': Symbol(Symbol.species) could not be cloned.

Example on codesandbox (try to load without and with devtools):
https://lk9m0.csb.app/
Source:
https://codesandbox.io/s/recursing-cori-lk9m0

Fixed in 81ae8d6caafd6cb8e8f464dac2625e10f443bd75


Add Lake County Illinois USA imagery

Description
Currently there is Cook County IL USA imagery,

But no Lake County IL USA imagery.
However it is available on
https://data-lakecountyil.opendata.arcgis.com/ .
So please somebody request it for me.
I can't. https://github.com/osmlab/editor-layer-index/issues/new says

Your ability to open an issue in this repository is currently blocked. If you feel this is in error, please contact the repository's owner.

(Yes, tried that.)
Screenshots

When creating an issue, it asks to fill out the following form:

When requesting the addition of an imagery layer, please provide the following information.
URL of the imagery
(required)
Two letter country code
(required)
Desired layer name
(required)
License
(required)
See FAQ at https://github.com/osmlab/editor-layer-index/blob/gh-pages/FAQ.md#what-imagery-licenses-are-compatible-with-this-index for information
about which licenses are compatible with this index.
Is the license compatible with OSM?
(required)
Attribution (text, URL, required?)
(optional)
Privacy policy (URL)
(required)
Icon
(optional)
Bounding polygon
(if possible, provide a bounding polygon where the imagery is valid, e.g. as a GeoJSON or OSM file.)
See FAQ at https://github.com/osmlab/editor-layer-index/blob/gh-pages/FAQ.md#how-can-i-draw-a-bounding-polygon) for information about how to draw a bounding polygon.

URL of the imagery
All I know is for Cook County's tiles, iD uses
https://gis.cookcountyil.gov/imagery/rest/services/CookOrtho2020/...
And
$ HEAD https://gis.cookcountyil.gov/
307 Moved Temporarily
https://hub-cookcountyil.opendata.arcgis.com/
whose URL looks like
https://data-lakecountyil.opendata.arcgis.com/search?q=aerial photography&sort=-modified&type=document link
OK, I'll make a note on
https://wiki.openstreetmap.org/wiki/United_States/Illinois/Lake_County
Somebody will figure it out.
Two letter country code
US
Desired layer name
Lake County Orthoimagery 202x
Etc.
This is all getting over my head. All I know how to do is draw footways
by clicking my mouse. I wish someone else would follow through with
this. Thanks.
Oh, bounding box. The land area of
https://www.openstreetmap.org/relation/536313
plus a kilometer or two of margin.

When creating an issue select "There is an issue with a background imagery source." and you would be sent to osmlab/editor-layer-index where is where imagery issue should be added. As it stands now this issue should be closed or transferred to the correct repo.

in such case moving this issue there should not be done, and as it is misplaced here it likely should be closed


unique-id is not unique

Hello
with the change from internal numbers to unique-ids there are ambiquities, since there are devices with one mac-address but several channels. For that reason, some objects are not longer constructed and therefore can not be accessed by data-points.
If there are only two channels, one actor (light) and one sensor there's no problem. But if there are several actors and/or sensors, only the first are considered during the construction-process of the corresponding object-tree .
Example:
Xiaomi offers a device with two switches and two sensors (Power and consumption).
In the deconz-api that looks like that:
`"lights": {
...
"19": {
"etag": "b0e30b1c47ee454d562ccce4e4e52bcf",
"hascolor": false,
"lastseen": "2020-12-02T12:38:44.397",
"manufacturername": "LUMI",
"modelid": "lumi.relay.c2acn01",
"name": "Licht Terrasse",
"state": {
"alert": "none",
"on": false,
"reachable": true
},
"swversion": "09-20-2018",
"type": "Dimmable light",
"uniqueid": "00:15:8d:00:04:2a:c0:1f-01"
},
...
"36": {
"etag": "90dc71465cc2d9453e789de22bcf08a4",
"hascolor": false,
"lastseen": "2020-12-02T12:38:14.305",
"manufacturername": "Unknown",
"modelid": "lumi.relay.c2acn01",
"name": "Kabel Terrasse",
"state": {
"alert": "none",
"on": true,
"reachable": true
},
"swversion": "09-20-2018",
"type": "Dimmable light",
"uniqueid": "00:15:8d:00:04:2a:c0:1f-02"
},
}
...

"sensors": {

    "77": {
        "config": {
            "battery": 10,
            "on": true,
            "reachable": true,
            "temperature": 1900
        },
        "ep": 1,
        "etag": "eb8c3b5fc60b9867cab8c618fab3a86e",
        "lastseen": "2020-12-02T12:38:44.397",
        "manufacturername": "LUMI",
        "modelid": "lumi.relay.c2acn01",
        "name": "Consumption 77",
        "state": {
            "consumption": 602,
            "lastupdated": "2020-12-02T12:35:04.910"
        },
        "swversion": "09-20-2018",
        "type": "ZHAConsumption",
        "uniqueid": "00:15:8d:00:04:2a:c0:1f-01-000c"
    },
    "79": {
        "config": {
            "battery": 10,
            "on": true,
            "reachable": true,
            "temperature": 1900
        },
        "ep": 1,
        "etag": "de7464043a986dfe6381708b5d9fa402",
        "lastseen": "2020-12-02T12:38:44.397",
        "manufacturername": "LUMI",
        "modelid": "lumi.relay.c2acn01",
        "name": "Power 79",
        "state": {
            "current": 0,
            "lastupdated": "2020-12-02T12:38:24.501",
            "power": -1,
            "voltage": 229
        },
        "swversion": "09-20-2018",
        "type": "ZHAPower",
        "uniqueid": "00:15:8d:00:04:2a:c0:1f-01-0b04"
    },
}
...

}
...
`
The problem is e.g. in the match methode, which only takes the mac-address but not the following expression: mac = mac.match(/..:..:..:..:..:..:..:../g);
I could think of two possible solutions:

Take the complete expression as uniqueid, which would break the current unique-ids, enforcing users to rework their skipts (once again)
Extend the current uniqueid with channels as sub-objects - only if there are more than one channel. That would not break the currect data-points (given that the first channel is represented as it is now)

@kneisel thak you for your explanation, i know this already. Solution 1 is easy but causes the same problem as in version 1, more than one entry in ioBroker for one device.
I have choosen a solution that is near your second one, but with a significant change to the whole device management in the adapter. In future versions the adapter has its own device definitions and will not longer make use of the definitions from deconz.

@Jey-Cee: thank you! Seems a reasonable approach.
... you've time till spring - I have four more Xiaomi multi-channel devices waiting to be built into the garden-illumination ;-)
... have a merry christmas time.

Development of verion 2 stopped, this is no longer relevant.
For more information (German) look into: https://forum.iobroker.net/topic/39627/deconz-aadapter-beta-ist-zustand-und-zukunft


hapi server example doesn't work for me

The hapi server example doesn't work...it says the web socket connection connection times out...
server code
 1 import { promisify } from 'bluebird'
  2 import Hapi from 'hapi'
  3 import inert from 'inert'
  4 import chokidar from 'chokidar-socket-emitter'
  5 import horizon from '@horizon/server'
  6
  7 import Html from './html'
  8
  9 const server = new Hapi.Server
 10 const start = promisify(server.start, { context: server })
 11
 12 const httpServers = server.connections.map(c => c.listener)
 13 horizon(httpServers, {
 14   project_name: 'hep',
 15   auth: {
 16     token_secret: 'ahqhrbqrq3498130y459h83rg1h3rg',
 17     success_redirect: '/'
 18   },
 19   auto_create_collection: true,
 20   auto_create_index: true,
 21 })
 22
 23 server.register(inert)
 24 server.connection({ port: 3000 })
 25
 26 server.route({
 27   method: 'GET',
 28   path: '/{any*}',
 29   handler: {
 30     directory: {
 31       path: './public'
 32     }
 33   }
 34 })
 35
 36 server.route({
 37   method: 'GET',
 38   path: '/',
 39   handler: (req, rep) => rep(Html)
 40 })
 41
 42 chokidar({ port: 9080 })
 43 start()
 44   .then(() => console.log(`Server running at ${server.info.uri}`))

client code
horizon.js
  1 import Horizon from '@horizon/client'
  2 export default new Horizon({ host: 'localhost:3000' })

Home.js
  1 import React, { Component } from 'react'
  2 import {
  3   observable,
  4   computed,
  5   action,
  6 } from 'mobx'
  7 import { observer } from 'mobx-react'
  8 import { Link } from 'react-router'
  9 import horizon from '../horizon'
 10
 11 @observer
 12 export default class Home extends Component {
 13   componentDidMount() {
 14     console.dir(horizon)
 15     const chat = horizon('messages')
 16     chat.store({
 17       text: 'wat a butiful horizon',
 18       datetime: new Date(),
 19       author: 'zaney mczaneface',
 20     })
 21     chat.fetch().subscribe(
 22       items => items.forEach(item => console.log(item)),
 23       console.error
 24     )
 25   }
 26   render() {
 27     return (
Home Page
)
 28   }
 29 }

error
WebSocket connection to 'ws://localhost:3000/horizon' failed: Connection closed before receiving a handshake response
Subscriber.js:171 Uncaught [object Event]SafeSubscriber.__tryOrUnsub @ Subscriber.js:171SafeSubscriber.error @ Subscriber.js:134Subscriber._error @ Subscriber.js:85Subscriber.error @ Subscriber.js:65Subject.error @ Subject.js:69socket.onerror @ WebSocketSubject.js:130

Server version: 2.0.0
Client version: 2.0.0

yeah.. .same here... It's mind boggling why it is so hard to simply require horizon/client (I use it on a backend)

Great support.

For anyone else confused by this, you have to specify that you are not authenticating (wish they told you that in the examples!!!)
Anyway, I made a barebones working express app here, if anyone else is interested.

Sorry about my non constructive rant... Realizing this might not be a bug but a learning curve my side, I picked the discussion up on Horizon  and forget to close it here.
Many thanks for the example, where you include the server and not the client   In my case the  solution was to add global.WebSocket = require('ws'); to get horizon/client working on a headless server.

Haha, this was my issue...so you might have one open somewhere else! And I understand the frustration haha... new technologies are hard. That's what I love about open source though, if something is going wrong, you can just fix it yourself. With firebase, you would have to open a support ticket, and wait for someone else to fix it, and you wouldn't learn anything...I actually learned a lot just by going through the source and figuring out what was wrong.


post-processing effect rendering pipeline

For #2962

Change List

new PostProcessEffect and ScreenPass classes
new PostProcessEffect rendering logic in DeckRenderer

Coverage decreased (-0.3%) to 61.737% when pulling fef9e3ffc5d3ade54895922e05ecc31920615455 on post-effects into e16d0c36a9885315b6b1c5d349e9bcbc06998728 on master.

What is your strategy to delete the models/buffers when they are no longer used?

What is your strategy to delete the models/buffers when they are no longer used?

the models and passes will be delete when user delete their references to PostProcessEffect objects and set deck.effects = [].
The two frame buffers are created when user adds any PostProcessEffect, and will live all the time after.

All resources must be explicitly deleted by calling .delete(). See past memory leak fixes

All resources must be explicitly deleted by calling .delete(). See past memory leak fixes

Done, thanks for reminding.


allow comma separated syslog targets on startup

This PR allows a user to pass a comma separated list of default syslog targets, rather than just one.

:+1: The Docker cluster I'm working on could really benefit from this.


Autostart in tray

How about autostart on launch setting? Want have an icon of this magnificent application after launch of my Windows and Archlinux. Thanks!

Autostart in tray

https://github.com/bastimeyer/livestreamer-twitch-gui/wiki/Launch-parameters

I'm requesting all of it in settings in application, in gui.

Well, IMO, this is totally unnecessary, since it would need to be implemented for Windows, OSX and various Linux desktop environments.
Just manually create a startup shortcut, which should only take about a couple of seconds, once...

Just found this issue with Google. Would it be possible to add a command line option (or maybe a setting) that will start the client minized/in tray? I like the notifications when someone goes live so I would like to have it sit in my system tray all the time.
I know how to add it to my autostart in Cinnamon but I've no clue how to start it in systray. I'm also not sure if that would even help all that much since the systray minimize features seems to be bound to the minized button, not the window state.

Would it be possible to add a command line option (or maybe a setting) that will start the client minized/in tray?

This already exists... See my comment above with the link to the wiki.
The --tray launch parameter works even if you've disabled tray mode in the settings.

Wow, I somehow missed that comment entirely. Thank you.


Format isn't applied if an image is saved without an extension

Expected
I can format and save an image without an extension, like so:
import Mogrify

open("image.jpg")
|> format("png")
|> save(path: "no_extension")

open("no_extension")
|> verbose # Has type of png, even though there's no extension

Actual
When performing the above actions, the newly saved file maintains its original type
Workaround
import Mogrify

image = open("image.jpg")
  |> format("png")
  |> save()

File.rename(image.path, "no_extension")

open("no_extension")
|> verbose # Has type of png, as expected

Thanks for the report. Reproducible with test:
  test ".format when output path has no extension" do
    orig = open(@fixture) |> verbose
    assert "jpeg" == orig.format

    path = Path.join(System.tmp_dir(), "mogrify_test_no_ext")

    orig
    |> format("png")
    |> save(path: path)

    image =
      open(path)
      |> verbose

    assert "png" == image.format
  end

Seems like convert is more suitable in general. https://www.imagemagick.org/Usage/basics/#mogrify_convert
Probably not a viable solution here, though, since it will change too much of how this library works. We might need to use your workaround involving a temp file and renaming the result to the desired filename.

Mogrify v0.9.0 (this library) uses convert instead of mogrify now (except when in_place: true is passed to save/2).
It still breaks in this case because the convert command will ignore the -format flag. The output file needs to explicitly contain the file extension. So we probably want to detect when the requested format does not match the requested path's file extension. In that case we should use the temp file workaround, and rename the result to the desired filename.


Added class name bindings for disabled and toggled

This PR is #11 with a cleaner commit history.
This PR adds more classNameBindings to the component:

.x-toggle-container-checked is added when toggled is set to true.
.x-toggle-container-disabled is added when disabled is set to true.

Published as 0.5.1

Great! Thanks again for the quick response.

No problem, glad you find this addon useful :smile:


Cmake core inclusion update

Description

If merged this pull request will Fix Issue #667 by reworking the options for ZeroMQ.  There is now a single CMAKE option for each of the core types ENABLE_XXX_CORE.  For ZeroMQ and MPI if the appropriate library is not found it is an error and CMAKE will halt.
The update also includes some updates to the inclusion for fmt and jsoncpp to slightly reduce configuration time by not searching for things we know to be there based on HELICS requirements.
Proposed changes

change all the core options to ENABLE_XXX_CORE
make not finding ZMQ or MPI an error if the corresponding core type was enabled
redo the C++ code that was using the previous option names
add some variables to reduce the configuration time for fmt and jsoncpp

This PR will be rebased once #664 is merged so it won't get merged until after that one


Feat: Allow spoofing host console settings

I am proposing a feature where the games being run on the emulator with imitate the current system configuration of the host 3DS.
This would be useful for games such as Tomodachi Life, which has an anticheat system built-in which disables all shop rotations for 24 hours after a change in console ID.
This obviously makes playing the game through Artic Base not particularly appealing, as each time you swap between the emulator and the real 3DS, the anticheat system will be activated, hypothetically leading to shops never changing rotation if you swap devices frequently.
This could either be the default behavior or an opt-in setting depending on your preference.
I am unsure if this is covered under this sentence in the readme:

Things that will never be implemented
...

Dump any kind of system data, such as keys, archives or console unique files. This includes caching data between play sessions.

Which console ID are you talking about?

I believe it checks this value. The in-game message in Tomodachi Life specifically mentions the game being played on a different console, and I'm not sure how else the game could be discerning that

Sorry, I was busier than expected today
I will come back to this tomorrow

how did you even manage to get tomodachi life to run in arctic, every time i try to load it on mine, it just throws an error and gets stuck at the 3 foods loading screen

I have confirmed that changing only the console ID and nothing else causes this message to appear when next loading the save, indicating that the anti-cheat has been tripped:

how did you even manage to get tomodachi life to run in arctic

Works absolutely fine for me ¯\_(ツ)_/¯

Odd, for me on default CITRA settings when it's loading the game it just throws a critical check the log error and gets stuck, and this error doesn't happen on any of the other games I've tried with it, I'm just lost

citra_log.txt.old.txt
yeah, heres a log of me attempting to boot it, and the error doesent happen on any other game that i tried (which were concentration training, pikmin movies, mario 3d land, and pokemon ultra moon)

TBH for the purposes of game compatibility you could probably just spoof the console ID and nothing else if that's more to your preference

@PabloMK7 Upon attempting to load into Tamadochi Life after installing the new version of Artic Base, I am now met with this message:

After saving in the emulator and loading up on my 3DS, the same message appears there, so it seems that some check that game is doing is somehow giving different results between the emulator and the 3DS.
I have ensured that this was not caused by accidentally going back in time by a few seconds by waiting at least 5 minutes between saving on one device and loading on another.

This makes sense, the RTC is most likely different between console and citra. Not sure if there is a way to fix this.


Jsont.zero doesn't encode

Jsont.zero is documented as encoding all values to null, but any encoding fails with "No encoder for ignore base map".
I suspect this is because the encoder goes through Repr.Null Base.ignore instead of an equivalent to Jsont.null, so it eventually reaches Error.no_encoder.

Thanks for the report!


Your app contains content that isn’t compliant with the Broken Functionality policy. App loads, but crashes

Describe the bug
When I try to upload my app update to Google Play I get this error :-
Your app contains content that isn’t compliant with the Broken Functionality policy.
App loads, but crashes
my code is :-
    isProd: true,
 ..... // closure
  final callback = ThreatCallback(
    onAppIntegrity: () => exit(0),
    onObfuscationIssues: () => exit(0),
    onDebug: () => exit(0),
    onDeviceBinding: () => exit(0),
    onHooks: () => exit(0),
    onPrivilegedAccess: () => exit(0),
    onSecureHardwareNotAvailable: () => exit(0),
    onSimulator: () => exit(0),
    onSystemVPN: () => exit(0),
    onUnofficialStore: () => exit(0),
  );

and I am using this command to build the app
flutter build appbundle--release --obfuscate --split-debug-info=build/app/outputs/symbols
I tried to build apk to test it on my phone and it worked fine without crashing after removing this part of code for sure :-
    onUnofficialStore: () => exit(0),
I used :-
flutter build apk --release --obfuscate --split-debug-info=build/app/outputs/symbols
Console of Google don't show additional information but the only thing I did is adding freeRasp to my code then uploading it as update so no another code errors and app works fine my local device after removign store part ..
I want refer to that "signingCertHashes" include the SHA of store too with release and debug SHA

Version of freeRASP: [e.g. 6.7.1 "latest" ]

any help on this ..

Hello,
does the APK of the application, which is resigned by Google work on your device as well? Does the beta or internal testing version work as well? Btw using exit(0) is not an appropriate killing of the application on Android devices, see https://stackoverflow.com/questions/45109557/flutter-how-to-programmatically-exit-the-app
Kind regards,
Talsec team


As part of this, review the content in https://hmlr-design-system.herokuapp.com/patterns/search-by-title-number/

This could be expanded to include more guidance on numbers in general such as using separators for thousands, currency formats or decimals?


Styling guidelines/parts and dogfooding

Using our custom element today in a site and it strikes me that while I had more lengthy documentation at first in the glitch we created this from, we skipped a lot of that and have since also done some tweaking and it really could use some documentation and 'tests'. I say 'tests' here because if we create a bunch of cases that need to be explained I feel like we'll inevitably really improve things too and find stuff we hadn't considered.
A really simple example of this is "I would like to style the twistys in collapses... how?"  including simply "They don't seem to be appearing" which is currently the case if you display on a back background, because they are black background svgs.  Currently you have to provide a new version of the background image which uses fill='white' instead of fill='black'.
:where(spicy-sections > [affordance*="collapse"])::before {
	background-image: url("data:image/svg+xml;charset=UTF-8,%3csvg xmlns='http://www.w3.org/2000/svg' xmlns:xlink='http://www.w3.org/1999/xlink' x='0px' y='0px' width='10px' height='10px' viewBox='0 0 270 240' enable-background='new 0 0 270 240' xml:space='preserve'%3e%3cpolygon fill='white' points='5,235 135,10 265,235 '/%3e%3c/svg%3e ");
}

Maybe this is fine because people will want to change all manner of thing about it, I'm sure - but for something so simple as "I have a dark background" this feels a little broken currently.  In the very least this should be well documented.
Are there any volunteers who would like to help me assemble such a list of "I would like to do X" pens and short documentation (and then file bugs if we cannot)?

I'd love to help with this!

(btw there's a typo in your post 'back background')

Great, feel free to just create some and link them here, or we can meet up and create a list to start - let me know what you want to do.  I'd love opinions on where to put this - the readme could get quite long if we put it all there - I have the previous docs I made that answered a bunch of things, tho it is incomplete and outdated, we could review it. (note, I deleted your comment about the typo because I fixed it in the post, thanks)

Stuff we need to document

[ ] Shadow parts
[ ] Spicy-H

Stuff we need to demo

[ ] Demo: Vertical tabs
[ ] Demo: Exclusive Collapse
[ ] Demo: Replicate popular design systems

Stuff we need to log/discuss

[ ] Find friction or pain points in styling API


Fixup pycandela initialization, validate pycandela docs

To test, cd candela/python/pycandela then run jupyter-notebook. Then create a new notebook and copy and paste in any of the pycandela examples in the sphinx docs (note the docs have been updated on this branch).
There is still an issue with geo, geodots, and lineup, since those examples do not have a good default height and do not allow setting of height well. That can be addressed in another PR.
With this PR tested and merged, it will be time to push a new pycandela version to pip.

The tests are failing due to something about too many regex groups, originating from the girder_client install. Not sure what to do about it.

Looks like we are hitting this error on Travis http://stackoverflow.com/questions/39829473/cryptography-assertionerror-sorry-but-this-version-only-supports-100-named-gro

Following to the GitHub issue from that link seems to suggest it's fixed but pip caches need to be removed. However it does not seem that we are caching python stuff - unless Travis is doing it automatically somehow.

I just created a virtualenv locally and repeated the failing pip command, and I get the same error :worried:
I don't know how pip caching works but I tried to defeat it with a virtualenv. Maybe @zachmullen has some insights?

Ah it's not a girder-client problem; it's in the "cryptography" module, which comes in (I think) via requests.

https://github.com/pyca/cryptography/issues/3187
This says the problem has been fixed but that we may need to delete any cached wheels.

See https://github.com/DigitalSlideArchive/large_image/pull/106/files/1e6333d75626f9bcffcb12ce6cad2900f1c94fb4..40d3f637ae1c6cbf154be2418a1b636fbac0d284

Thanks @zachmullen.
A more direct fix is to install pycparser==2.14 in the .travis.yml file before the other pip commands. I'm preparing a PR for Candela along those lines now.

This might have just been fixed:
https://github.com/eliben/pycparser/issues/151#issuecomment-254551279

Once #452 is vetted and merged, we can rebase this branch on master and let the tests pass before merging.

Heh yep, so can we just restart the Travis builds for this branch and see if it works? Or is something still pinned somewhere to a bad version?

I restarted the builds to see what happens.

Current coverage is 58.36% (diff: 100%)

No coverage report found for master at 1da25f4.

Powered by Codecov. Last update 1da25f4...bbd5c49

Woo hoo!


Abstract class inheritance convenience

Defining a logger as follows is of significant convenience.
protected val LOG = KotlinLogging.logger {}

However, when used on abstract classes, we end up with a logger for the abstract class. Whilst this outcome is technically correct, we lose a lot of the convenience that Kotlin offers. Instead we end up with a load of extra boilerplate code in implementations which Kotlin has been so powerful to help reduce.
I can do one of the following in my abstract class already.
protected val LOG = KotlinLogging.logger(this.javaClass.name)
protected val LOG = KotlinLogging.logger(this::class.jvmName)

However, it would feel sensible to be able to have KotlinLogging provide a convenience method for the following.
protected val LOG = KotlinLogging.logger(this)

This method would also have the benefit of being capable of being used in multi-platform common code.

I think this goes against the logging conventions. Usually logger is static and also point to the current file/class so its easier to find where the log message was created and not to inherited class.


Message delivered up the stack and throwing an exception: a warning with

the exception name rather than the entire stack trace is logged
(https://issues.jboss.org/browse/JGRP-1958)
This is a manual backport of
https://github.com/belaban/JGroups/commit/03d7c93145f6a9314c2380f5ad66bdc8ec4209f0
for EAP 6.4 https://bugzilla.redhat.com/show_bug.cgi?id=1399195
@belaban  could you please review if I backported correctly?

@johnou I like that, and will keep it in mind for changes in master. But this is only a backport to an obscure and old branch nobody uses... :-)


sdkmanager installed to cmdline-tools/bin

It looks like with the update to the SDK version, sdkmanager is now being extracted to cmdline-tools/bin instead of tools/bin.

FYI, this is fixed in https://github.com/android-actions/setup-android/pull/70.


Add HA functionality to WebHdfsClient

Motivation and Description
HDFS supports High Availability (HA) through the use of multiple namenodes. This is currently not supported by WebHdfsClient as it allows to specify only one namenode. Since version 2.1.0, the hdfs package allows to specify multiple namenodes, by passing an url containing multiple namenodes separated by ';'. This functionality is forwarded through to WebHdfsClient in this commit.
Have you tested this? If so, how?
I tested the HA functionality manually on a cluster with two namenodes. I did not add unit tests, as for this one would need to extend WebHdfsMiniCluster to support multiple namenodes, which is non-trivial and probably overkill.

@tarrasch As the main author of the webhdfs client, could you take a quick look at this patch?

Thanks @adaitche :)


Beginner help - how to send current line to R

Sorry for the dumb question. I'm a new IntelliJ user and your plugin looks awesome.  I've installed it ok and opened a .R file and the syntax colouring seems to work fine.   But what do I press to send the current line to the R session running in the Terminal.   I can obviously start a Terminal inside IntelliJ and start R in it by typing R at bash prompt,  but I don't see an R menu added to the IntelliJ menu or how you get to the screenshot on this project's README.md under "Integration with R session" section.
Armed with the answer I'm happy to submit pull request to README.md.
Thanks in advance.

Ok so right click!  Now I get the menu you show in README.md and "R Code Evaluation" and the "Evaluate" item.   I selected a line of R and clicked "Evaluate" but nothing appears to happen.  I started R in the Terminal and tried again but nothing happened.   Btw, I just want to send the current selection to the Terminal, whatever the Terminal might be running at the time (usually either bash or R).

HI Mattdowie,
r4intellij does not work with the builtin terminal of Intellij yet. On Macos in can send the current line or selection to the terminal, iterm2, or to R.app. Under Windows it should send it to the R application. It's not yet supported under linux, because there's not generic way to communicate between apps on linux.
The shortcuts to do it are visible right next the action items in the context menu.
I hope this answers your question.

Ok but I am on Macos.

On Macos in can send the current line or selection to the terminal, iterm2, or to R.app.

Great.  But exactly how.  I have clicked R Code Evaluate ->  Evaluate  and nothing happens.

Ah. Now I see that a new R app was started. That wasn't clear to me! It just appears in the dock and didn't come to the front of the screen. Ok so now I see that Ctrl+Option+Enter does send the line to R.
Still stuck on how to send the current line to the terminal/iterm2.   How to do that please?

You can change the evaluation target in the preferences.
There you can also change the focus behaviour. The default is here that the focus stays in intellij, but you can also change it so that it changes to the evaluation target.

Ok got it now. I changed evaluation target in preferences to Terminal and also changed Evaluate shortcut to a single F5 keypress.  This is exactly what I wanted.  Plus, evaluate sends to the last active Terminal,  which is exactly what I wanted too so I can have multiple Terminals (similar to NppToR on Windows). This is awesome!    Never found a way to do this on Linux/Ubuntu,  so very happy.
I'll submit a pull request to README.md ...

Actually it does not work on linux/ubuntu but just on macos. :-( All it's doing is to use applescript to remote-control the terminal/iterm/r.app. I'd love to add it for linux as well, but there does not seem any java-accessible framework to do it.

I never found a way in linux either and would like that as well.  This ability, rightly or wrongly, is the main reason I'm sticking with Mac for now.


Added docs/cloud toggle on the first few steps in build-your-first-mo…

Description & motivation
See this Slack convo for context.
TL;DR: I added Cloud/Core toggle functionality to the first few steps of the process explained on /tutorial/build-your-first-models where previously, it was formatted with just Markdown headers.
Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[X] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

@JKarlavige I just want to call out that when I ran npm install inside website, it updated the package-lock.json and it's going to be committed in this PR. I think it should be fine, but let me know if not!

I just want to call out that when I ran npm install inside website, it updated the package-lock.json and it's going to be committed in this PR. I think it should be fine, but let me know if not!

To follow up on this. The package-lock is restructured for me every time I run npm-install. It looks like it only restructures the file, while the dependencies still have the same version and the site behaves the same. So I don't see this as an issue.


As an Admin, I want to be able to see the status of the engagement on the Engagement page (tweak) *click edit to see text

The current status table shows Draft/Published (and Scheduled when implemented). We need to replace the status of Published with the status of the engagement as can be seen on the engagement page: Upcoming, Open, Closed. (These statuses indicate that the engagement has been published).
The possible options for this column will be:
Draft
Sent for Review (future implementation)
Scheduled (future implementation)
Published/Upcoming
Published/Open
Published/Closed
Archived (future implementation)

Hey team! Please add your planning poker estimate with Zenhub @carlygreenehill @djnunez-aot @marieanik @saravanpa-aot @tiagograf-AOT @VineetBala-AOT

Yes, please @VineetBala-AOT

@marieanik  --- i think this card is looking good!
thoughts: in Engagements tab, the Status column comes between "date created" and "date published"  -- that's where you wanted Status to appear, yes?
in Surveys tab, Status column appears farther right just before "Reporting". I find this view easier on the eyes for some reason!


Fix stats bugs

(closes #169)
(closes #198)
Fix bugs related to stats based on PED bugs reported.

Codecov Report
Base: 46.39% // Head: 46.33% // Decreases project coverage by -0.05% :warning:

Coverage data is based on head (44a8f78) compared to base (df89a63).
Patch coverage: 12.50% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #243      +/-   ##
============================================
- Coverage     46.39%   46.33%   -0.06%
  Complexity      685      685
============================================
  Files           143      143
  Lines          3479     3483       +4
  Branches        412      413       +1
============================================
  Hits           1614     1614
- Misses         1723     1727       +4
  Partials        142      142

Impacted Files
Coverage Δ

...va/seedu/clinkedin/logic/commands/FindCommand.java
100.00% <ø> (ø)

src/main/java/seedu/clinkedin/ui/MainWindow.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/clinkedin/ui/ResultDisplay.java
0.00% <0.00%> (ø)

.../main/java/seedu/clinkedin/model/ModelManager.java
80.19% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Looking at the screenshot in the PR, perhaps you can mention explicitly what 7/10 is or what the brackets mean, because currently it might be a bit confusing to people looking at the graph for the first time.

Looking at the screenshot in the PR, perhaps you can mention explicitly what 7/10 is or what the brackets mean, because currently it might be a bit confusing to people looking at the graph for the first time.

Tried this, but adding additional words seems to make the piechart weirdly small to fit the extra words!


Tooltip Mousehover issue on Vega "3.0.10" and Kibana 6.2.2

I tried to create circle packing example from Vega example page and noticed that tooltip is not functioning as intended. It only works for the first Mousehover event and shows the data but when I hover the mouse to another element, it doesn't work. I have to move mouse away and wait for 1-2 seconds then move it back. The example code I will also post below:
vega-circle-packing.txt

@emilmirzayev do you get the same results if you install the latest plugin?

I updated my Kibana to 6.2.4 today and this issue still persists

Hi again Yuri,
today I updated both Kibana and Vega to the newest version. 6.2.4 Kibana and 3.2.1 Vega.
Tried this code again and the results are still the same. Tooltip only works for the first hover, then I have to hover away, wait a bit and then hover back

Same results with 6.2.3 with not built in Vega plug in installed through kibana plugin install

Hi Emil, my apologies for being quiet. I suspect this is due to the https://github.com/vega/vega/issues/1259 issue in vega packaging - that's why the libraries were not actually updated. I have already fixed it, but I'm afraid it will only be shipped as 6.3 (already in code freeze, will ship shortly).  I also suspect that this plugin is not affected by that issue.  I will try to look at the issues soon!

Hi Yuri, thanks for response. I also saw your issue #1259 and was suspecting that could be the issue I was facing. Is there any dates for 6.3 announced already?

Not sure, but I'm hopping it will happen within the next week or so

Hi,
Just to add another experience.
I'm using Kibana 6.3.1 and I can confirm this issue seems to be present.
The first tooltip appears, then it's a bit random, sometimes the tooltip prints, sometimes it does not. And sometimes it works fine for a few points.
I'm using a map with points and shapes drawn, as I've not yet been successful to change the cursor from a hand to a classic cursor I'm not sure it's not also an issue of simply me being not exactly on the point.
Hope this helps :).

bug also present in kibana 7.6 with the airport vega example, if we specify a cursor in mark we can't have other cursor as hand.


fix-warning: return null when calling async function in a chain of pr…

…omises
This PR fixes #14
When using bluebird, warning are triggered because some promises are created in promises chains and not returned.
Returning null solve the problem.

Thank you so much for fixing and for the PR :) I really appreciate your efforts! Great work. I merged and I will release today @robinbiondi

@jjayasagar Thanks for the merge!


$uibModal bindToController not working when controller is class

When class is used as controller in a modal the scope properties are not bind to the controller.
The bug can be seen here: plunker http://plnkr.co/edit/5S4uv22eHYkYhfHA0LMx?p=preview

Closing this as this project is no longer being maintained, per the README and the ISSUE_TEMPLATE.


Is v3 produciton ready?

Hello, yesterday you published version 3.0.0 (BTW performance increase looks very good), but this release is marked as "Pre-release" on GitHub, but for composer (packagist) it is stable release. Also version number v3.0.0 looks like stable.
It this version ready for production use or not?

released v3.0.0


Hmx hm phy setup

How do i change modes? so one unit is master?

It is currently hardcoded into the script, first phy is master

Then i need to cross the cables to get it to work?

Then i need to cross the cables to get it to work?

Yes, or change line:
phy.master = (i == 0) # set master on 1st phy
into
phy.master = (i == 1) # set master on 2nd phy


docs(blog): june community update

Community update blog for the month of June 🎉

@derberg missed anything??


allow use of context in dataload in a backwards compatible way

@tonyghita this is my first pass at addressing your request. Would this solve your use case?
What I like about this implementation is that it introduces no breaking changes.
Usage
batchFn := func (ctx context.Context, keys []string) []*dataloader.Result {
  // use context for timeout/deadlines/ect...
}

loader := NewBatchedLoaderContext(batchFn)

loader.LoadContext(key)

The only potential problem I see here is that in order to use the context, you'll have to make sure that you always use "LoadContext", otherwise you'll get an empty context in your BatchFunc. Furthermore, the context that will be used in the BatchFunc will be the context of the first item in the batch (ie, the item that started the batch) which could be based on a race condition. So, I fear that users could have batches that sometimes fail because they forgot to use LoadContext in just one place. That could be a hard issue to track down?

This change is

The only potential problem I see here is that in order to use the context, you'll have to make sure that you always use "LoadContext", otherwise you'll get an empty context in your BatchFunc.

Yes that could be a gotcha... I think this is unavoidable without breaking the interface. I think that's okay failure behavior though.

Furthermore, the context that will be used in the BatchFunc will be the context of the first item in the batch (ie, the item that started the batch) which could be based on a race condition.

This is fine and works the same as with closures... the context is scoped to the request. I would expect batches across the same request to share the same context.
I don't think it's a race condition since the context is copied wherever it's used.

I have decided to introduce these changes as a v2 version of this library. See the v2 branch.


[FIX] renamed odoo to openerp for access early versions of posBox

Let's wait for instruction on posbox configuration


Add a11y support for screen readers

#203 #192

[x] Using WAI-ARIA specification
[x] Trap focus on modal
[x] Checked a11y with ChromeVox

Since we've merged https://github.com/biig-io/ngx-smart-modal/pull/215 into master, your PR has conflicts with the new code. I let you understand the new way services are handled by modal component and do the appropriate changes. I apologize for the inconvenience and I'm at your disposal for further help if needed.
Regards! 🖖

New PR is here #225 with new version


Why one container for webserver and scheduler with LocalExecutor

First of all, thank you so much for creating this image. It's working great.
I have only recently started using docker and I am still learning the best practice, so please forgive me newbie question.
I am wondering, why did you decide to let the webserver and scheduler share on container when using the LocalExecutor? I though it would be best to always use separate containers for all processes?
Thanks, Kim

LocalExecutor uses the multiprocessing Python library to create processes run locally, and for this reason, it is a single-host deployment and it is not recommended for production systems.

I see, thank you. But doesn't this only mean that the scheduler executes all threads on the same machine / docker container?
My understanding is that the scheduler and webserver are not connected and don't need to be on the same machine / docker container.
So I am wondering why the choice was made to run them in the same container.
Once again, I am just a beginner and I am trying to understand the design of this docker setup to learn how to create my own in the future. I'm not trying to criticize or change anything :)

It's probably due to this

I'm using LocalScheduler with a separate container running the webserver and had no issues so far. I havent checked if there's any issues with logs though.
I had to change the entrypoint.sh to be able to run the container without the scheduler.

@grahovam Can you post what changes have you made to entrypoint.sh to separate webserver container from scheduler in LocalExecutor? I'm in the same situation and this might be the only solution I've been searching for few hours.

@sys-temd Sorry, I don't remember :) It's been a year ago and I don't run airflow on Docker anymore. But as far as I remember, Airflow has published an official docker image. Probably you should check this out :)

Yeah the new official docker image doesn't combine scheduler and webserver. I have documented my process of migrating from puckel's here:
https://github.com/puckel/docker-airflow/issues/536#issuecomment-626333283


zstack监控功能

这个 PR 实现什么功能/修复什么问题:
monitor.go中GetMonitorData函数通过传入的参数拼装请求参数，然后调用zstack.go中的getMonitor()，通过GET请求，获取相应的指标监控数据
是否需要 backport 到之前的 release 分支:

release/2.13
release/3.0
release/3.1

/area region

Codecov Report

Merging #5132 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #5132      +/-   ##
=========================================
+ Coverage    6.73%   6.75%   +0.01%
=========================================
  Files         546     546
  Lines       95258   95258
=========================================
+ Hits         6420    6433      +13
+ Misses      88300   88287      -13
  Partials      538     538

Flag
Coverage Δ

#aFlag
6.75% <ø> (+0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/appsrv/response.go
69.23% <0%> (+3.07%)
:arrow_up:

pkg/cloudcommon/cronman/cronman.go
54.85% <0%> (+5.33%)
: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 746c43e...4787829. Read the comment docs.

/cc @swordqiu
/cc @yousong

/lgtm

/lgtm

/lgtm
/approve


Bug in level 5 in the Endless mode

Hey, I love you game, but there is a weird bug in level 5 Endless mode!
Screenshot_20240228-013909_ChipDefense.png
There is no defence there and they are always attacking.

I want to add that this is not specific to level 5 (they were generated). It happened to me in level 33 of the endless mode just now as you can see above the CPU on the left. It is impossible to play this level since there are no chip slots between the CPU and the the spawn:

I suggest to check in the generator that there is at least one chip slot between the spawn and the CPU. I have already won rounds were I only had one slot, so this is possible. I would recommend at least two slots, though.

Yes, i hope they fix it soon.

Ok, I got you, it's all generated randomly.
Thank you.

Please do not close this issue. It is still an issue that should be tracked and resolved at some point. Just because it is randomly generated doesn't mean that this should happen.
Also, could you rename this issue to something like "zero defense circuits in random levels in endless mode" or similar? It is not specific to level 5. Thank you :)

OK.

I admit that the random level generation still needs some work. And that situations like this should be avoided. Will try to come up with some kind of solution to this ...
In the meantime, if you are stuck, you can go to "Settings" and uncheck "Keep levels in endless mode". This will generate a new one each time you restart the level.

In the meantime, if you are stuck, you can go to "Settings" and uncheck "Keep levels in endless mode". This will generate a new one each time you restart the level.

Thanks for the tip. I re-played the last seven levels because I thought it may change things, but it did not, before I found that switch ;)

added some code in 1.30 to guarantee that there are at least 2 nodes on each track.


Automated cherry pick of #2772: fix: allow set object acl in batch

Cherry pick of #2772 on release/2.12.
#2772: fix: allow set object acl in batch

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.12@2f3c212). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.12   #2774   +/-   ##
==============================================
  Coverage                ?   7.22%
==============================================
  Files                   ?     472
  Lines                   ?   74560
  Branches                ?       0
==============================================
  Hits                    ?    5384
  Misses                  ?   68724
  Partials                ?     452

Flag
Coverage Δ

#aFlag
7.22% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/buckets.go
1.4% <0%> (ø)

pkg/cloudprovider/objectstore.go
3.18% <0%> (ø)

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 2f3c212...cb7988f. Read the comment docs.

/lgtm
/approve


Add manual region tags

Hey, are you planning to keep adding comments here? If so I can hold off reviewing until everything's there... (contents otherwise LGTM.

Hey, are you planning to keep adding comments here? If so I can hold off reviewing until everything's there... (contents otherwise LGTM).

I had to do a second pass but this should be ready for review/merge.


Add some nice links on the shader page for new shader l33t hackers

Would be nice to add some informational links / icons that contains some simple information on what a shader is and how to / tutorials on how to code the specific shader we use.
Throw some links down below on this issue.

Haven't checked it out but maybe https://www.youtube.com/playlist?list=PLGmrMu-IwbguU_nY2egTFmlg691DN7uE5


node-fetch unusable with esm package

Given an index.js file with import fetch from "node-fetch"
When I run cli command node -r esm index.js
Then I should see the script import fetch and run.
Version 3 gives this error.
 at Object.Module._extensions..js (internal/modules/cjs/loader.js:1080:13) {
 code: 'ERR_REQUIRE_ESM'
}

Now, I've read the similar threads on this.  One thing that hasn't been mentioned, that the maintainers overlook, is there are still MAJOR projects out there that can't be used with ESM directly.  Cucumber.js is one of them.  The top level project must be CJS for cucumber to work, but you can convince it to work seemlessly with the ESM package.

As popular as this package is, did any of the maintainers bother to test whether these changes would work with it?

We can't possible test each and every "thing" in the wild. I haven't even heard of Cucumber upon till this point.
It's unreasonable to think that we should test Cucumber, WebPack, rollup, TypeScript, babel, gulp, jest, ts-jest, jest-fetch-mock and everything that is out there. If we make a Major change, then it's up to others to make it work.

Hi.

The top level project must be CJS for cucumber to work

I don't know much about cucumber but they have announced that they have experimental support for esm https://github.com/cucumber/cucumber-js/blob/main/docs/esm.md
Could you take a look in https://github.com/node-fetch/node-fetch/issues/1279 if you find some helpful solution/workaround?

Seems unusable here as well 🤨

I'm trying to get up to speed to actually contribute code on the Cucumber project, and ESM support is one of the branches I'm following to get a feel for what needs to be done.  So I'm aware of the ticket and the problem that the Cucumber test library needs the project to be CJS to run correctly unless node is ran using the esm package.
I don't expect any maintainer to test everything in the wild, that's why I filed a bug. If you aren't interested in maintaining interoperability with other projects that's your prerogative.
Honestly, if you're going to do such a break it would be better to follow the path the moment.js team did - they created a new package named luxon.js and marketed moment as bug-fix only.  In a similar manner you could make a new package node-fetch-esm and declare the current branch as entering a bug fix only phase with no new features, also set an end of life for the project around 3 to 5 years out from now.

Would you be open to v2 having feature parity of v3 but compiled down to cjs?
Making the business case to migrate an entire app to ESM is difficult.
I'm OK with sticking the v2, but would love to get typescript typings backported to that version.
@types/node-fetch is just a stub at ^3.0.0 and at 2.5.12 it had bad typings for the AbortController.
I'm afraid if it's not usuable, it'll just divide the ecosystem, we'll have a handful of packages published like node-fetch-usable and one of them is bound to have a security vuln.

We can't possible bring everything from v3 to v2 as some stuff would be breaking.

We were also forced to downgrade to v2 due to this issue. Why you don't consider to build the library to a single file? This would increase the load performance, compatibility and decrease the size of the library.

Why you don't consider to build the library to a single file? This would increase the load performance, compatibility and decrease the size of the library.

It's not beneficial, read this thread #1227

This is extremely opinioned position, which shouldn't be the case for the lib, as both should be supported, while they are and will be both first-class module systems. A consumer should be able to use the library disregarding of which module system he/she uses. This is even disrespectful to those, who for any of the reasons use cjs.

We can't possible bring everything from v3 to v2 as some stuff would be breaking.

I could be wrong, I often am, but if the choice is between a breaking change in a minor version and having 2 unusable versions, i'd vote for breaking semver once.

In a similar manner you could make a new package node-fetch-esm and declare the current branch as entering a bug fix only phase with no new features, also set an end of life for the project around 3 to 5 years out from now.

That's why we have major versions. Can't create a new package every time something major changes.

there are still MAJOR projects out there that can't be used with ESM directly

Use node-fetch 2.x then
I just migrated a small project to ESM and happily using node-fetch 3.x. node-fetch is usable with esm. Not sure why this issue was reported / is still open.

we also have similar cjs/esm issues open that are basically a dupl of this, so closing this one...
There isn't any issue with node-fetch itself
This (now) ESM-only package works as intended. if you have a problem with using ESM-only packages or any other solution highlighted in #1227 then it's really up to you to fix it
There is  also a discussion about having  a  exposed cjs file that only export fetch (not headers, Response and Request)
that is related to this: #1327