[WIP] Add ML option for coarse solver

First draft. Have not tested yet, and need to figure out how to transform input parameters to AdditionalData.

It's running ML now, I can see the output. Next steps:

[ ] propagate parameters from boost::property_tree to Teuchos::ParameterList
Control the coarse levels from input deck. Hopefully not too hard, as ML parameters are flat.
[ ] Fix operator and grid complexities
If one uses AMG for coarse levels too, it needs to combine its complexity with amge
[ ] Double check whether we solve the right thing
Should ML be used as a preconditioner, or as a standalone solver? Do we pass residual or rhs?
[ ] Study the convergence

Try something like that
boost::property_tree::ptree ptree; // assume it contains the data structure
std::stringstream ss;
boost::property_tree::xml_parser::write_xml( ss, ptree );
auto parameter_list =
// NOTE I couldn't find a static function to parse the stream in Teuchos :/
Teuchos::ParameterList dummy;
auto plist = dummy.getParametersFromXmlString( ss.str() );

I think this is ready for review. It runs and converges, but I have not designed the test yet as I need to study the convergence.

Can you add a test that uses ML.

@Rombur Of course, but I need to check few things first.

@dalg24 Thank you for the comments, that worked!

@Rombur Added ML test. Ready for final review.


Remove ON CONFLICT constraint checking known limitation, when ready

Related to https://github.com/cockroachdb/cockroach/issues/23699
Currently documented as a 2.0 Known Limitation via this PR.
Once the linked issue is addressed, the known limitation should be removed.

see https://github.com/cockroachdb/cockroach/pull/26642


Install Expo in genymotion emulator

i was try use expo and install expo in android studio emulator , it was success
but when i install expo to genymotion emulator , it show me error
i try fix thi error haft of day but no use
can anyone help me with this ??
i'm newbie , and sorry about bad english but please help
Thanks you for everything

hi! looks like you posted this to the forum, where @ccheever gave you an answer


wasm: update V8 to v9.0.257.17.

Signed-off-by: Piotr Sikora<EMAIL_ADDRESS>

@Shikugawa are you also mainataining Stable Release in 2021Q2, or do we need to find a new maintainer? In any case, it looks that an update to RELEASES.md is in order.

@Shikugawa are you also mainataining Stable Release in 2021Q2, or do we need to find a new maintainer? In any case, it looks that an update to RELEASES.md is in order.

@PiotrSikora I'm not a stable maintainer for 2021/Q2. So we should find a new maintainer. Could we do that? cc @lizan

/retest

/lgtm deps

@Shikugawa are you also mainataining Stable Release in 2021Q2, or do we need to find a new maintainer? In any case, it looks that an update to RELEASES.md is in order.

@PiotrSikora I'm not a stable maintainer for 2021/Q2. So we should find a new maintainer. Could we do that? cc @lizan

@mattklein123 @htuch @duderino @lizan we don't have active maintainer for Stable Releases. Could you find any volunteers?

@mattklein123 @htuch @duderino @lizan we don't have active maintainer for Stable Releases. Could you find any volunteers?

As #16027, @dmitri-d has elected as new stable maintainer.

As #16027, @dmitri-d has elected as new stable maintainer.

Thanks! I've updated RELEASE.md in https://github.com/envoyproxy/envoy/pull/16055.


Suggestions to improve the documentation

Thanks a lot for this tool and its detailed documentation which make it easy to get started.
A few things I noticed while following the provided instructions:

you should add libXpm and gtk2 to the list of dependencies on https://intel.github.io/tsffs/setup/linux.html#install-local-dependencies
https://intel.github.io/tsffs/fuzzing/compatibility.html uses test.efi to test micro-checkpoints, but the sample output ("Working...") corresponds to HelloWorld.efi from the tutorial section. It is a good thing to prefer test.efi here (since it doesn't require docker to compile), you should just update the text to mention the expected output is "4141414141414141" rather than "Working..."
in the same section, you should use ./build.sh rather than ninja to build, otherwise the tsffs header is missing and linking fails.

Thanks for the notes, I'll incorporate these suggestions in the next docs update.


Add option to throw error on adding foreign key without index

Hello,
I have met the problem about foreign key with no index, and proposes changing behaviour about it.
In #290, in the case of MySQL (InnoDB), trying to add_foreign_key without index displays warning.
I think the warning is not enough and throwing error is more better because it's not idempotent.
(I met the problem in my system that executes ridgepole applymultiple times in a time (I know it's strange), and found error in its second execution, I couldn't found its mistake in CI)
Also, I have confused about its error message please add an index explicitly for your future operations.
I thought its future operation is far from now, but it is just next ridgepole apply.
I agree that the future includes next time, but I suspect wording like please add an index before your next operation or like it is preferable.

I agree that the future includes next time, but I suspect wording like please add an index before your next operation or like it is preferable.

I've changed the warning message in https://github.com/winebarrel/ridgepole/pull/307.
By the way, I think always raising an error is better than adding an option.
@winebarrel What do you think about that? If you agree with me, I'll change the behavior.

Since it is a specification only for MySQL, I think it is not good to always raise an error.
How about raising an error only for MySQL?

How about raising an error only for MySQL?

check_foreign_key_without_index was already a MySQL-only check...

By the way, I think always raising an error is better than adding an option.

I agree with your idea 👍

I agree with your idea 👍

Thank you for your reply! I created the pull request https://github.com/winebarrel/ridgepole/pull/310.

I released the new version 0.8.10, which raises an error if an InnoDB column has a foreign key but no index, so I'll close this issue.


Deleting items does not remove them from the parent

When I delete something from my store using the RemoveData-action the item gets deleted from the store, but the reference from the parent does not get removed.
Examples
{     "id": "page1",     "sections": [{         "id": "section1"     }] }
normalizr turns this into:
{   normalized: {     result: [       'page1'     ],     entities: {       sections: {         sektion1: {           id: 'sektion1'         }       },       pages: {         seite1: {           id: 'seite1',           sections: ['sektion1']        }       }     }  }
Now I run this.store.dispatch(new RemoveData({ id: section.id, schema: sectionSchema })); which leads to the following state:
{   normalized: {     result: [       'page1'     ],     entities: {       sections: {       },       pages: {         seite1: {           id: 'seite1',           sections: ['sektion1']        }       }     }  }
Which obviously breaks the code then ... any ideas why this is happening?

Please see #20
RemoveChildData support is comming in v2.2.0

@michaelkrone working fine for me as well! Thanks for the fast fixes!

Hi,
The issue persist, When using RemoveChildData it's deleted well the child but the reference inside the parent still there.
Before Remove child Data

After Remove child data

As we can see the group schema inside Policy still have the id.
Thank you.


Desktop Notifications not working

My Setup

Operating System: macOS Mojave 10.14.6
App Version: 2.17.9
Installation type: AppStore

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

Description
I'm not getting desktop notifications either here or in PC with the other users.

The sound does work, it sounds for every message, but no pop up notification.
Current Behavior
Push notification does work on my phone, but desktop notifications doesnt.
Expected Behavior
It should notify as Slack or any other app does.

Little Update, they started working a few times, but now instead of showing a notification, the bell rings and the entire screen of the app shows up, but if I type or something, it still interacts with the program if was using before the app popping up.

Do you have the same debug console errors as mine?  #1618

Do you have the same debug console errors as mine? #1618

Excuse me being too new on this, how do I see the debug console?

View > Open Dev Tools

Seems to be the same of the one you redirected your post.

See #1579

And #1509

Can you guys please try version 2.17.11 which was just released?
The PR #1662 should fix this problem. 😊

I have tested with 2.17.11 and it's working now! Thanks!

Please test if the latest release did solve this. If not please reopen the issue or create a new one and tell us. I will be closing this one for now.
Thank you!


LLVM changes for LLDB

This PR contains a number of commits from the swift-llvm/stable-next branch that are needed for an LLDB merge into swift-lldb/master.  I will be including this PR in some cross-repository testing while I get the merge ready.

Hmm, something about this doesn't look right.  Hang on...

(Closing until I address the confusion.  It is a small handful of commits, but something is looking wrong above).

I think I see what happened here.  The change itself, if you look at the files changed above, is a small set of changes (12 files and small number of lines changed, mostly additions).  It looks like somewhere along the line somebody pushed a rebase that altered history (either repo or my end) for this branch.  That makes all the commits show up as they did above, even though the net change is much much smaller.
I'm redoing the change against a totally clean swift-3.1-branch branch on a new repo.


The <sys/vmmeter.h> header is missing in iOS SDK

FYI I've just filed a bug report with Apple about this, and referenced lmbmdbx as an example of a 3rd party library whose build breaks as a result.

Also FYI, it's not even necessary to include this header on Apple platforms. It's only needed for the struct vmtotal info declaration at core.c:9857, which is not compiled on Apple because the earlier #elif defined(__MACH__) section is used instead.
So you could fix the problem by wrapping the #include like so:
#ifndef __MACH__
#include <sys/vmmeter.h>
#endif

@snej, thanks.
As far as I remember, there were some necessary definitions.
But I have redone it several times (in particular this code).
Let me check.

FYI, Apple just notified me that they've fixed the bug I reported. <sys/vmmeter.h> is present in the iOS 14 SDK (part of Xcode 12.)


Input Array

I thought is was possible to have an input array, but is cannot figure it out..
What is need is the following
@reduxForm({
      form: formName,
      fields: ['box']
    })

<input type="checkbox" name="box[1]" {...this.props.fields['box']} />
<input type="checkbox" name="box[2]" {...this.props.fields['box']} />
<input type="checkbox" name="box[3]" {...this.props.fields['box']} />

I know that this will not work, i would just like to know if there is a possibility.
The closest i got was
@reduxForm({
      form: formName,
      fields: ['box[1]','box[2]','box[3]',]
    })

<input type="checkbox" name="box[1]" {...this.props.fields['box[1]']} />
<input type="checkbox" name="box[2]" {...this.props.fields['box[2]']} />
<input type="checkbox" name="box[3]" {...this.props.fields['box[3]']} />

But then this will not be submitted as an array.
Kind Regards,
Raymond

right now i am busy using the change() function and that looks promission, so for now this is not an issue anymore.

No, this is not currently supported. What I've done in my project is create a faux-input RadioList component that will either generate "select one" radio buttons or "select multiple" checkboxes (both are really fake inputs, styled with check icons on divs), and then RadioList provides support for value and onChange, which is all that redux-form really needs for an input component to work.

thx for your response.
I have done this with real checkboxes and the change action.
I have implemented it in redux-form-generator package.
Take a look at the file /src/types/checkboxList.js on how i implemented this.
I believe this is a good way to go, but any input would be welcome if you have some :)

I'm confused, was this issue closed before the introduction of these docs -> http://erikras.github.io/redux-form/#/examples/complex?_k=8ayeza ?

I thinks so, yes.

Yep, never mind got it working.
Loving redux!

@thomasdavis could you point to the specific part of the docs that satisfy this use case? Having trouble implementing it.

I am struggling to find a documented example of how to handle multiple checkboxes. I would really appreciate a hint! :weary:

@markau +1


feat: ability to select local files only for android

@zedelashvililevani please review and test. Don't forget to add localFilesOnly when opening a picker


kivy.factory.FactoryException: Unknown class

Getting  kivy.factory.FactoryException: Unknown class <NavigationDrawerIconButton> on latest kivymd. Please check the issue

Yes I also take this error.

I think this is why you are having this issue https://github.com/kivymd/KivyMD/issues/176 This repository needs update.


Updating sampler failed An error occurred while sending the request.

Requirement - what kind of business use case are you trying to solve?
I'm trying to deploy Jaeger as a DaemonSet by following the Production setup instructions (https://github.com/jaegertracing/jaeger-kubernetes#production-setup) with Cassandra as the backend.
In the application, I have the following:
` var tracer = Configuration.FromEnv(sp.GetRequiredService())
.GetTracer();
                return tracer; '

This works totally fine in local with a Docker image. I can see traces on test runs. I'm passing the service name using the environment variable JAEGER_SERVICE_NAME.
Problem - what in Jaeger blocks you from solving the requirement?
This doesn't seem to be working on Kubernetes (running on Azure). Logs show the following error (Warn) but is not totally useful:
Updating sampler failed An error occurred while sending the request.
Could you please help debug this?

It looks like the client can't reach the agent. How are you starting up the agent and what are the configurations you are passing into your tracer? You might have to update some host:ports so that the client knows where the agent is.

Setting the ENVIRONMENT variable JAEGER_SAMPLER_MANAGER_HOST_PORT fixed the problem. But I'm still not able to see the service in the UI.


Allow multiple certificates to be added to the XML metadata

We should be able to add multiple certificates to the DigiD and eHerkenning config, so that they are all used in the metadata.
This way, when a certificates expires, we can upload the new certificate and generate metadata containing both the old (currently valid) and the future valid certificate. This enables a smooth transition.

Published django-simple-certmanager 2.2.0 for this, work in django-digid-eherkenning is ongoing.


Optionally remove use of BIF timers from get and put FSMs.

The BIF timer lock is quite expensive to take, and as the number of
cores in your system increases, so does contention on it for commonly
spawned processes.
This patch has very little performance impact on systems below 20 cores,
but is necessary to achieve higher performance down the road.
It transfers responsibility for killing the FSMs to the client (although we
retain the older code path in case the new one is a problem), backstopping
the client by using cheaper state transition timers.  Logic is added to
reconnect the client with remotely spawned put coordinators.
Also addresses #683

Can one of the admins verify this patch?


Implement pyupgrade

[x] Set literals
[x] Dictionary comprehensions
[x] use datetime.UTC alias
[x] Format Specifiers
[ ] printf-style string formatting
[x] Unicode literals
[x] Invalid escape sequences
[x] is / is not comparison to constant literals
[x] .encode() to bytes literals
[ ] extraneous parens in print(...)
[x] unittest deprecated aliases
[x] super() calls
[x] "new style" classes

[x] rewrites class declaration
[x] removes __metaclass__ = type declaration

[x] forced str("native") literals
[x] .encode("utf-8")
[x] # coding: ... comment
[x] __future__ import removal
[x] Remove unnecessary py3-compat imports
[ ] import replacements
[x] rewrite mock imports
[x] yield => yield from
[ ] Python2 and old Python3.x blocks
[x] remove six compatibility code
[x] open alias
[x] redundant open modes
[x] OSError aliases
[x] typing.Text str alias
[x] Unpacking list comprehensions
[x] Rewrite xml.etree.cElementTree to xml.etree.ElementTree
[x] Rewrite type of primitive
[x] typing.NamedTuple / typing.TypedDict py36+ syntax

[x] typing.NamedTuple
[x] typing.TypedDict

[x] f-strings
[x] subprocess.run: replace universal_newlines with text
[x] subprocess.run: replace stdout=subprocess.PIPE, stderr=subprocess.PIPE with capture_output=True
[x] remove parentheses from @functools.lru_cache()
[x] replace @functools.lru_cache(maxsize=None) with shorthand
[x] pep 585 typing rewrites
[x] pep 604 typing rewrites
[ ] remove quoted annotations

Please add the tests from pyupgrade to assure equivalent behavior.

I guess I can just pick some rules in here for implementation?

I guess I can just pick some rules in here for implementation?

Yes, can you comment on the one you choose so there is no conflict?

I'll look at yield -> yield from.

I can do Remove unnecessary py3-compat imports and import replacements.

I can dig into typing.Text str alias.

I am going to take these two next:

subprocess.run: replace universal_newlines with text
subprocess.run: replace stdout=subprocess.PIPE, stderr=subprocess.PIPE with capture_output=True

Today I will work on:

Rewrite xml.etree.cElementTree to xml.etree.ElementTree (UP023)
OSError aliases (UP024)
Unicode literals (UP025)

Going to start on rewrite mock imports (UP026)

I will take .encode() to bytes literals (UP027)

I will take: Unpacking list comprehensions (UP027)

I will take: yield => yield from (UP028)

I will take: Format Specifiers (UP029) next

I will take:  printf-style string formatting (UP031) next. I will have this done by Sunday.

I will take: f-strings (UP032)

I will take: extraneous parens in print(...) (UP033)

I will take:  import replacements (UP035)

Import replacements might requiring auditing some of what we already do. Like, we do some of the replacements as part of the other Pyupgrade rules. I'm not totally sure what we're missing. (I'm also worried that it'll be a really tricky one to implement :joy:)

I am probably going to regret saying this, but this seems like its not too challenging. The process I was thinking is:

Search this dict for matches (converted to a rust hashmap)
Replace any specific items that match. If its a muli-line then we can move the replaced items to a new line. I would use libcst imports here
Return the new strings

I will take: Python2 and old Python3.x blocks (UP037)

I will take:  remove quoted annotations (UP038) AKA the LAST one 🎉🎉🎉.

@colin99d - Awesome. Like pyupgrade (IIUC), let's only enforce that when from __future__ import annotations is present.

I'm not sure where is a good place to drop this, but it would be great to have pydowngrade along with pyupgrade.
This will allow to develop a project with newest set of features, and maintain compatibility with previous python version by downgrading certain things when building a wheel for specific version.
Hope it's not completely out of place 😄

Hello Bobronium,
I love the idea! As a developer on a python library myself, it is annoying not being able to add the newest features in my code, because some people are still on python 3.7. However, I see two issues with this idea:

This would take a while to code (probably just as long as coding all the pyupgrade features), but with a much smaller potential use case.
Some of the lints would not be able to be reverted, for example, if you ran the lint that removes "u" before a string, then ruff would not know which strings to add it back to.

I think a much better implementation for this would be to have your CI that uploads pip take the version that works for your oldest members, and then upgrade it to the correct version for each user group downloading it. While this doesn't help the developers, it would mean all users of the code get the newest features.

In testing this out using Ruff 0.0.251 and enabling "UP" rules, the first pyupgrade feature mentioned in the original checklist ("Set literals", https://github.com/asottile/pyupgrade#set-literals) does not appear to be handled and is not listed under the rules here https://beta.ruff.rs/docs/rules/#pyupgrade-up, despite being checked off above. Perhaps it was actually never implemented? Should I file a new issue for this?

@sjdemartini - I think those rules are handled instead by flake8-comprehensions rules, like C405.

Aha, indeed, thank you Charlie!

No prob! Same is probably true of dictionary comprehensions.

First of all, thanks for the excellent tool! Loving it 😍
After reading those last comments here, I got curious on exactly which extra rules besides UP I should use to get the full pyupgrade functionality in ruff.
I haven't found that information in the docs, so I did some testing and the closer I got was:
ruff --target-version py311 --select UP,C401,C402,C403,C404,C405,F632,W605 --fix foo.py

Description of those rules:
# Activate all the rules that are pyupgrade-related
select = [
    "UP",    # pyupgrade
    "C401",  # flake8-comprehensions: unnecessary-generator-set
    "C402",  # flake8-comprehensions: unnecessary-generator-dict
    "C403",  # flake8-comprehensions: unnecessary-list-comprehension-set
    "C404",  # flake8-comprehensions: unnecessary-list-comprehension-dict
    "C405",  # flake8-comprehensions: unnecessary-literal-set
    "F632",  # pyflakes: is-literal
    "W605",  # pycodestyle: invalid-escape-sequence
]

The only pyupgrade feature not supported in ruff is remove six compatibility code.
The only ruff UP rule not supported in pyupgrade is UP038: non-pep604-isinstance: Use X | Y in {} call instead of (X, Y).
On a side note, since this PR is highly ranked in the "ruff pyupgrade" search, maybe we could add the ruff rules IDs to the items in this PR description, exactly as being done in "Implement Pylint" (#970)? This way people can better correlate the pyupgrade feature name with the ruff rule. This may help.

Thanks, this is a great comment! I'd love to add the relevant annotations to the issue. If anyone is interested in writing a comment that does that, I'm happy to copy it into the description.

Thanks @charliermarsh, I'm happy to provide it:
- [X] Set literals / `C401` `C403` `C405`
- [X] Dictionary comprehensions / `C402` `C404`
- [X] Format Specifiers / `UP030`
- [X] printf-style string formatting / `UP031`
- [X] Unicode literals / `UP025`
- [X] Invalid escape sequences / `W605`
- [X] `is` / `is not` comparison to constant literals / `F632`
- [X] `.encode()` to bytes literals / `UP012`
- [X] extraneous parens in `print(...)` / `UP034`
- [X] unittest deprecated aliases / `UP005`
- [X] `super()` calls / `UP008`
- [X] "new style" classes
    - [X] rewrites class declaration / `UP004`
    - [X] removes `__metaclass__ = type` declaration / `UP001`
- [X] forced `str("native")` literals / `UP018`
- [X] `.encode("utf-8")` / `UP012`
- [X] `# coding: ...` comment / `UP009`
- [X] `__future__` import removal / `UP010`
- [X] Remove unnecessary py3-compat imports / `UP029`
- [X] import replacements / `UP035`
- [X] rewrite `mock` imports / `UP026`
- [X] `yield` => `yield from` / `UP028`
- [X] Python2 and old Python3.x blocks / `UP036`
- [X] remove `six` compatibility code / not ported to ruff
- [X] `open` alias / `UP020`
- [X] redundant `open` modes / `UP015`
- [X] `OSError` aliases / `UP024`
- [X] `typing.Text` str alias / `UP019`
- [X] Unpacking list comprehensions / `UP027`
- [X] Rewrite `xml.etree.cElementTree` to `xml.etree.ElementTree` / `UP023`
- [X] Rewrite `type` of primitive / `UP003`
- [X] `typing.NamedTuple` / `typing.TypedDict` py36+ syntax
    - [X] `typing.NamedTuple`  / `UP014`
    - [X] `typing.TypedDict` / `UP013`
- [X] f-strings / `UP032`
- [X] `subprocess.run`: replace `universal_newlines` with `text` / `UP021`
- [X] `subprocess.run`: replace `stdout=subprocess.PIPE, stderr=subprocess.PIPE` with `capture_output=True` / `UP022`
- [X] remove parentheses from `@functools.lru_cache()` / `UP011`
- [X] replace `@functools.lru_cache(maxsize=None)` with shorthand / `UP033`
- [X] pep 585 typing rewrites / `UP006`
- [X] pep 604 typing rewrites / `UP007`
- [X] remove quoted annotations / `UP037`
- [X] use `datetime.UTC` alias / `UP017`

Please add the tests from pyupgrade to assure equivalent behavior.

BTW, the only change I did was moving the "use datetime.UTC alias" item to the end, because it was the only one not following the order from the pyupgrade README.


Error when using scalar as nexus method in interface definition

When using the extended method inside interfaces definition nexus throws a TypeError when making the schema.
This is the scalar definition:
scalarType({
    name: 'Instant',
    asNexusMethod: 'instant',
    description: 'Instant in milliseconds since Unix epoch',
    parseValue: (value: any) => (typeof value === 'number' ? Instant.fromEpochMillis(value) : null),
    serialize: value => (value instanceof Instant ? value.toMillis() : null),
});

This is the interface definition:
interfaceType({
    name: 'Notification',
    definition: t => {
        console.log('id: ', t.id);
        t.id('id');
        console.log('instant: ', t.instant);
        t.instant('timestamp'); // Breaks in this line
    },
});

Log and error:
id:  [Function: id]
instant:  [Function (anonymous)]
id:  [Function: id]
instant:  undefined
TypeError: t.instant is not a function
    at Object.definition (/home/lotus/IsoWorkspaces/Croct/admin/backend/src/infrastructure/graphql/schema/workspace/schema.ts:44:11)
    at walkType (/home/lotus/IsoWorkspaces/Croct/admin/backend/node_modules/@nexus/schema/src/builder.ts:737:11)
    at /home/lotus/IsoWorkspaces/Croct/admin/backend/node_modules/@nexus/schema/src/builder.ts:741:7
    at Array.forEach (<anonymous>)
    at SchemaBuilder.checkForInterfaceCircularDependencies (/home/lotus/IsoWorkspaces/Croct/admin/backend/node_modules/@nexus/schema/src/builder.ts:740:29)
    at SchemaBuilder.getFinalTypeMap (/home/lotus/IsoWorkspaces/Croct/admin/backend/node_modules/@nexus/schema/src/builder.ts:802:10)
    at makeSchemaInternal (/home/lotus/IsoWorkspaces/Croct/admin/backend/node_modules/@nexus/schema/src/builder.ts:1546:92)
    at Object.makeSchema (/home/lotus/IsoWorkspaces/Croct/admin/backend/node_modules/@nexus/schema/src/builder.ts:1585:49)
    at Object.<anonymous> (/home/lotus/IsoWorkspaces/Croct/admin/backend/src/infrastructure/graphql/definition.ts:7:30)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)

The interface definition is being called twice and on the second run the extended method is undefined.

Using t.instant?.('timestamp') to skip when the method is not defined works. Without the error the definition function is actually called 3 times and the third time the function is defined. The last one is the one used on the schema, resolvers, etc.


WEB3-136: make more Steel types public

This PR makes the following types public:

BeaconInput
BlockInput
StateDb

It also generalizes the BeaconInput concept of having a BlockInput and a header-based commitment.

Make it a draft for now to be sure that we don't need to expose even more


Implement protocol level ping/pong

Hey,
websocket pings are part of the protocol for a long time. They are nothing special, just protocol frames with opcode 0x9.
This implementation lacks it, so everyone has to implement their own, non-protocol-level solution.
Is there a chance you will implement this in the near future?

See as well https://www.w3.org/TR/websockets/#ping-and-pong-frames .

If there is demand of the feature due to the indication of a few people, I'll consider implementing it.

I had a 2 day long investigation, the result is you'll be unable to implement it.
Here's why: https://github.com/crossbario/autobahn-js/pull/396#issuecomment-433444831
I guess as the current state of affairs shows, everyone has to implement their proprietary client side hearbeat.

You can construct arbitrary WebSocket binary payload, but not arbitrary WebSocket frames in JavaScript in the browser run-time context. You can construct arbitrary frames in JS on NodeJS of course - and hence AutobahnJS supports automatic WebSocket client-initiated heartbeating there, but not for browsers.


Change colors according to design

This PR out of date and should be closed.
If we still need it - please open new PR


Can I create scenes/routes during runtime using RNRF?

I am loading a tree of items from a JSON file. Each tree item should be a Screen in the app.
The user should be able to navigate through this tree of screens like in the iOS settings dialog:

I was hoping to load the JSON when the app starts, and to dynamically generate screens and nested navigators based on the data.
This is not possible in React Native Navigation, though: screens need to be known statically.
Is there a way to do this in RNRF, or is it a React Native specific limitation?
I'm still new to RN, sorry if this is a dumb question.

Well, RNRF is built on top of react-native-navigation. So it is the same, you cannot generate screens dynamically.

@pekka I would have all of the screens load, but some sort abstraction that will show what routes are available to use versus what aren't. If you are using a store like Redux or Mobx then that would be a good place to store that.

It is not supported.


Update README.md

Hi @Code-Hex,
I'm Charly for The Guild, thank you again for your awesome plugin! ✨
I made the following change proposal in the attempt to fix the documentation issues on our side:
https://github.com/dotansimha/graphql-code-generator/pull/7438
In short, our plugin hub system is parsing each plugin README.md as MDX.
The documentation generation (with Vercel/Next.js) fails with your plugin because of the following sentence in your README.md:

import types from generated typescript [...]

is identified by MDX as an invalid JS import expression...
I'm aware that the issue should be fixed on our side but it would be way faster to fix it this way 🏎
Let me know what you think.

@charlypoly Ah, I see. Thank you very much!

@Code-Hex Thank you! ⚡

@Code-Hex we will need you to publish a new version of the package 🙏🏼 (or update the existing one)

@charlypoly I just published now. (0.1.1)


migrator: teach the tool to handle qualified replacement. rdar://32466196

@swift-ci please smoke test and merge


Release of version <IP_ADDRESS> of xSqlPs

Releasing version <IP_ADDRESS>

Hi @kwirkykat, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Katie Keim). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Use as a standalone proxy server?

Hello,
How we could use it as a standalone HTTP proxy server with https support as well ?

Thanks for your interest! :+1:  This project currently (and in the future) only implements the client side.
If you're looking for a server side HTTP proxy implementation in PHP, make sure you're following my profile as this is something that I currently have in the pipeline :shipit:

I have a question for the Shipit Squirrel... How is this idea coming along? I have spent quite some time looking for a good quality PHP-based proxy library that supports TLS. I've found a few things that will do HTTP (e.g. jenssegers/php-proxy) but my understanding of the HTTPS conversation is probably not good enough to make a reliable solution for myself :smile_cat:

I'm actively working on this at this very moment, so stay tuned :-)
I have a working prototype locally and I'll make sure to publish this ASAP, but expected this to take ~4 weeks for the first public beta :shipit:

Splendid, many thanks @clue, I will keep a look-out. I have found this script that looks like a good stop-gap in the meantime.

Just for interest, here is my current work - it's a play/record HTTP proxy with support for PSR-6 (for storage) and Guzzle middleware (to capture HTTPS sites). It'll be interesting to see if the listener part of my project can be easily replaced by your forthcoming package.


Adding a section on the Store Referer header

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

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps-chromium/microsoft-store.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 5008a46:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps-chromium/microsoft-store.md
:white_check_mark:Succeeded
View

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

LGTM @mikehoffms

Docs Build status updates of commit 69d8f55:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps-chromium/microsoft-store.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 6ed8b0c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps-chromium/microsoft-store.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 2c1681b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps-chromium/microsoft-store.md
:white_check_mark:Succeeded
View

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


LOG4J2-3473 garbage-free default disruptor WaitStrategy

This PR implements the change discussed in LOG4J2-3473:
make the default disruptor WaitStrategy used in Log4j garbage-free.

I realize this PR is a bit difficult to review.
This is the commit in the LDAP disruptor repo that has the change we are trying to introduce here:
https://github.com/LMAX-Exchange/disruptor/commit/a18f8198ab94590f0a5ec7c3378ba87f997ea670


Android xhr.upload.onprogress and addEventListener('progress') not supported?

Description
We are doing XMLHttpRequest PUT request (sending base64 encoded image data, setting content type header, etc).
iOS sends progress updates, Android does not.
We tried setting progress listener the following ways:
xhr.addEventListener('progress')
xhr.upload.addEventListener('progress')
xhr.upload.onprogress = () => {}
xhr.onprogress = () => {}
None of these handlers end up being called on Android, however on iOS they work fine.
The request completes successfully with a 200.
Reproduction
Solution
Additional Information

React Native version: 0.39.2
Platform: both Android and iOS tested (issue only on Android)
Operating System: MacOS

Ended up moving to a native (Obj-c and Java) solution for now which does provide upload progress events on Android.
Regarding XmlHttpRequest, not sure if the Android interface is different from iOS or not but no luck with XmlHttpRequest updateProgress on Android.

@dguillamot Looks like there is some sort of API for progress on android.
https://github.com/facebook/react-native/blob/e87e181998bf4f8efba2c7e04bec0929344600b9/ReactAndroid/src/main/java/com/facebook/react/modules/network/NetworkingModule.java#L190

@christopherdro Yes, I did see that last week but xhr.upload.addEventListener('progress')  did not result in any updates on Android - only iOS. For now, we have solved it with a Native solution, thanks though.

Hey @dguillamot, would you mind sharing your solution? I am facing the same problem, and I currently don't have the knowledge to implement it natively (learning java but that will take me some time...).

I'm on RN=0.42 and using XHR with a Content-Length set before uploading begins. I'm seeing the progress events, but no bytesSent or length or anything else that I can use to compute a progress%. Anybody else have luck with this?

I'm on RN=0.42 and using XHR with a Content-Length set before uploading begins. I'm seeing the progress events, but no bytesSent or length or anything else that I can use to compute a progress%. Anybody else have luck with this?

Any ideas how to get send progress events on Android with bytesSent?

@christopherdro Why is this issue closed? This problem persists.

i trying to upload video with youtube v3 api, and have some dilemma. When i use FormData , xhr.upload.onprogress is working. But then youtube api return processing error, because FormData add some data to file object.  So, i use something like xhr.send({uri: url}); and it works, video is uploading, BUT xhr.upload.onprogress stopped working. I still don't know why...
P.S. sorry for my ugly english


Semantic errors ignored in tests with cargo check --tests and cargo build --tests

Cargo version: cargo 0.19.0-nightly (fa7584c14 2017-04-26)
cargo check --tests and cargo build --tests seem to ignore semantic errors in tests.
I don't know if that's intentional, but it's frustrating to be unable to cargo check code in my tests properly.
Examples:
Common Cargo.toml
[package]
name = "check_test"
version = "0.1.0"
authors = ["me"]

[dependencies]

src/lib.rs example 1

#[cfg(test)]
mod tests {
    #[test]
    fn it_works() {
        asset_eq!(1, 1); // intentional typo (semantic error)
    }
}

What happens: running cargo check --lib --tests completes with no errors (code 0)
What should happen: running cargo check --lib --tests should indicate:
cannot find macro `asset_eq!` in this scope
  help: did you mean `assert_eq!`?

src/lib.rs example 2

fn not_a_test() {
    asset_eq!(1, 1); // intentional typo (semantic error)
}

#[cfg(test)]
mod tests {
    #[test]
    fn it_works() {

    }
}

What happens AND should happen: running cargo check --lib --tests indicates:
cannot find macro `asset_eq!` in this scope
  help: did you mean `assert_eq!`?

src/lib.rs example 3

#[cfg(test)]
mod tests {
    #[test]
    fn it_works() {
        i'm syntaxically incorrect code
    }
}

What happens AND should happen: running cargo check --lib --tests (actually cargo check gives the same output, so I think that syntax is checked regardless of cargo filters) indicates:
message: 'expected one of `!`, `.`, `::`, `;`, `?`, `{`, `}`, or an operator, found `'m`
  label: expected one of 8 possible tokens here'

Thanks for the report! This is actually expected, albeit somewhat odd, behavior. What's happening here is that we're actually never compiling the library in test mode! Specifically when you pass --tests that actually only includes integration tests right now (e.g. tests/*.rs) not the library tests as well. Additionally the --lib flag means "the compiled library", not "the tested library".
I think there's two possible fixes to this issue:

Like cargo rustc, we could add a --profile flag to check. This means that to check the library you'd execute cargo check --profile test --lib
We could also just add the library's tests to the --tests flag.

I'd personally be in favor of the latter, it seems to be the most reasonable! This is actually I think a relatively easy issue to tackle, would you be interested in sending a PR? I could help guide you through where the code is located and how to write a test!

@alexcrichton Thank you for your answer.
Actually, I would be thrilled to work on this issue, and I agree with you on the second option. Is there a way I can contact you so that we can talk about that?

Oh awesome! We can either coordinate here or I'm also on IRC in #cargo on irc.mozilla.org as acrichto, either way is fine by me!

Related PR: https://github.com/rust-lang/cargo/pull/4039

This is fixed by #4592.  --tests now does all tests, including unit tests.  Also, cargo check now has a --profile flag that can be used to target unit tests individually, such as cargo check --lib --profile test.


Improve handling of NuGet packages that require assembly binding redirects

The IOT hub client SDK version 1.0.22 requires an assembly binding redirect (this is added automatically when installing the SDK into a console app). However in functions that assembly binding redirect is not applied and it results in an error.
Steps to reproduce:

Create a function with this code and project.json:

using System;
using System.Text;
using System.Threading;
using System.Threading.Tasks;
using Microsoft.Azure.Devices.Client;

private const string DeviceConnectionString = "connection string”;

public static void Run(string input, TraceWriter log)
{
                DeviceClient deviceClient = DeviceClient.CreateFromConnectionString(DeviceConnectionString, TransportType.Http1);
                if (deviceClient == null)
                {
                    log.Info($"Failed to create DeviceClient!");
                }
                SendEvent(deviceClient,log).Wait();
                log.Info($"C# End");
}

static async Task SendEvent(DeviceClient deviceClient, TraceWriter log)
{
    string dataBuffer;

        dataBuffer = "Hello world" + Guid.NewGuid().ToString();
        Message eventMessage = new Message(Encoding.UTF8.GetBytes(dataBuffer));

        log.Info($"C# Send message to the device");
        await deviceClient.SendEventAsync(eventMessage);

}

{
"frameworks": {
    "net46":{
        "dependencies": {
            "Microsoft.Azure.Devices.Client": "1.0.22"
                    }
                }
        }
}

Paste in an appropriate IOT hub connection string and attempt to run the code

Expected result: no assembly loading errors occur
Actual result: Exception while executing function: Functions.DevicesTest. mscorlib: Exception has been thrown by the target of an invocation. mscorlib: One or more errors occurred. Microsoft.Azure.Devices.Client: Could not load file or assembly 'Validation, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=2fc06f0d701809a7' or one of its dependencies. The system cannot find the file specified.

The experience of debugging this sort of scenario would be improved by https://github.com/Azure/azure-webjobs-sdk-script/issues/1017

The experience of debugging this type of issue would also be improved by https://github.com/Azure/azure-webjobs-sdk-script/issues/1023

@fabiocav proposing that we relax some of the matching rules.


Use double qoutes around database name.

When using a database name that contains dashes or underscores, the
CREATE DATABASE statement fails with a syntax error. Use double quotes
around the database name to solve this.

:+1: from me, whatever that's worth.
I am surprised that this hasn't been caught much sooner since there are numerous places where double quotes should be used when handling database or role names. Even my very first PR (#32) was about that.

/me doesn't have see merge button :|
ping @DavidS

This scares me, as it unveils a dangerous lack of test coverage :-/

:|
@DavidS what's a good… or, general way to catch these?

clone https://github.com/puppetlabs/puppetlabs-postgresql/blob/master/spec/acceptance/db_spec.rb for postgresql::server::database ? That's also probably the case why it wasn't caught earlier, because everyone uses ::db and nobody uses ::database. Which opens another can of worms...

i really meant, in a general way.

Quickcheck for puppet? But what is your invariant? Applies-without-error, and applies-second-time-without-changes?
I admit, that would have caught this one.

well, in this particular case: all databases, tables, and so on, in strings, are quoted.

Test coverage, code reviews for test coverage, or quickcheck. I know no easy ways.

let's go with quickcheck, hypothetically.
what would a quickcheck for puppet need, to provide minimal functionality?

could we test with puppet apply --noop?
could we integrate with something like rspec-puppet?
would we have to integrate with something like beaker?
how important is the difference between apply and agent? important enough to test for it?


Prez curie implementation discussions

Hi @recalcitrantsupplant, a couple of initial comments based on the curie implementation mvp made in https://github.com/RDFLib/prez/commit/028c494dbf48e8dc5c4982ce7e369cbca4247d03.
Why is the default separator used for a curie a - and not :?
http://www.w3.org/2004/02/skos/core#prefLabel -> skos-prefLabel
versus
http://www.w3.org/2004/02/skos/core#prefLabel -> skos:prefLabel

A generic method of generating curies for unknown namespaces:

https://linked.data.gov.au/dataset/qldgeofeatures/BelyandoBasin -> ns0:BelyandoBasin
https://linked.data.gov.au/dataset/qldgeofeatures -> ns1:qldgeofeatures
https://linked.data.gov.au/def/qg-sites/400GeorgeSt -> ns2:400GeorgeSt

Basically the same behaviour as RDFLib.
Suggestion: Maybe Prez users can declare these unknown or uncommon namespaces using the properties from VANN.
Example:
@prefix skos: <http://www.w3.org/2004/02/skos/core#> .
@prefix vann: <http://purl.org/vocab/vann/> .

<https://linked.data.gov.au/def/qg-sites>
    a skos:ConceptScheme ;
    vann:preferredNamespacePrefix "qg-sites" ;
    vann:preferredNamespaceUri "https://linked.data.gov.au/def/qg-sites/" ;
.

If the preferred namespace prefix is not declared, then Prez will just generate its own prefix for the namespace like above (ns0, ns1, etc).
Usage of VANN may be proposed as an optional thing in VocPub and any other relevant profiles for Prez.

Thanks for the feedback,
Why is the default separator used for a curie a - and not :?
I researched whether ":" is legal in URLs outside of the scheme - there definitely isn't one consistent opinion. It's at least not clear that different servers would handle it safely. I left it as a parameter in the functions that create/expand the curies, so we could allow setting it via an environment variable as well.
Re. the unknown namespaces, have a look at what I've implemented in the curie_functions.py and reference_data/prefixes/standard.ttl
My thinking was to keep where the prefixes are defined separate from the instance data so as not to impose system requirements on the data.

I think : is fine in the path segment and query string of URLs.
I've used it successfully in the past and here's an example service that's using it online. https://prefix.zazuko.com/rdf:type.
The only consideration is with relative URLs where the : would be read as the end of the protocol scheme. But I think this situation doesn't apply here.

OK I'll make it configurable so that if there are issues we can opt to substitute it.
The "prez:link" internal links that get generated are relative. Could this cause the issue you're referring to? I guess it depends on how the FE uses the links...

I think it'll only cause a problem if it's a relative link that's a curie within an  element's href attribute.
Example:

OK I'll make it configurable so that if there are issues we can opt to substitute it.

Sounds good!

implemented


Change #000000 in flat mode

In flat mode event 'change' not work then pick color #000000
http://jsfiddle.net/dwf9bs4v/

Confirmed same issue in Full Mode. Change events do not fire if "#000000" is the picked colour


how create my adress?

not found in api doc.
how can create my wallet adress ?

you can use "getnewaddress" or "getnewaddress + wallet name of you"
ex: getnewaddress "mywallet" or only getnewaddress

Thanks @hungdung1326!


self.subprocess.stdout.readline() freezes

I'm trying to communicate with KataGo. GTP is ready on the engine side. Unfortunately after this line:
https://github.com/jtauber/gtp/blob/master/gnugo_vs_gnugo.py#L21
everything freezes, he program hangs.
Do you know the reason?
I'm using Pyhton 3.7 on MacOS.

self.subprocess.stdin.flush() is needed before self.subprocess.stdout.readline()


contact us page

(Thanks for reporting an issue! Please make sure you click the link above to view the issue guidelines, then fill out the blanks below.)
I want to design contact page.
@sifa123 can i work on this issue

Okay

I would like to work on this issue please assign me to improve the contact page @sifa123


[Feature Request]: RBAC in Autogen Studio

Is your feature request related to a problem? Please describe.
Hi, I just want to know if there is already any effort to develop RBAC feature in Autogen Studio.
If not, do you guys have any suggestions or guideline how the community can contribute to implement this feature.
Describe the solution you'd like
No response
Additional context
No response

@victordibia


Fix undefined bug for template strings.

Fixes #40
cc: @AlmeroSteyn is this the expectation?

Coverage remained the same at 98.795% when pulling d77e0ad41489aa6edbb40fe7fcce99065a29419d on fix-template-literal into c055fab1eee3b5180a905d20cf56f6062c1ba5b5 on master.

@evcohen Thank you!

template literals always toString directly, and null and undefined toString to "null" and "undefined" respectively.

The issue only pertained to undefined which now works perfectly as expected.
But if I apply the fix in my fork and run with null it still returns an empty string instead of "null".
 Is it possible the tests were incorrect?

Yes that looks to be the case. Fixing now


get vector variables out of a model properly

related to #822

@whoburg, ready for review

review complete. And I think this needs a unit test before merging.

@whoburg, responded to code review and added test

retest this please

retest this please

this looks like a good improvement, let's merge it and deal with any issues related to #844 when fixing that issue.


[no jira][risk=no] Fix npm audit results

Update libraries reported by npm audit

Thank you @MatthewBemis!


Added data assistant example

@micdavis checks failing on the PR

tensorflow<2.12 related to https://pypi.org/project/tensorflow-gpu/#description

on the notebook, I'd prefer that we clear all cells so the output is not saved in the file -- just code and markdown cells

Still error'ing but current commit looks like outputs are cleared.


Swagger model does not support an array of type

a model type or property type which is of type array is not supported:
type: [ "string", "null"]
For example, the following swagger is valid in editor.swagger.io but the properties name2 is not parsed.
---
swagger: '2.0'
info:
  version: 0.0.0
  title: Simple API
paths:
  /:
    get:
      responses:
        200:
          description: OK
          schema:
            $ref: "#/definitions/Person"

definitions:
  Person:
    description: A person
    type: "object"
    properties:
      name1:
        type: "string"
      name2:
        type: [ "string", "null"]

The spec doesn't support multi-types, and so Swagger Model does not as well.


Loop Habits import

Hi,
Just came across this project and seems very interesting. Would be interested to import from my existing setup from the (Loop) Habits Android Project (https://github.com/iSoron/uhabits).
A sync option with the app might be a later feature as it will require collaboration from both projects but an option to move my existing habits over might be a good starting point.

Hi @Gandalf-the-Blue
Thanks for reaching out.
Good news, currently i'm actively working on the import feature, and it will be available in the next few weeks.
Importing from uhabit setup is a genius idea, and I will update you once it is available.


Ventura Blocker?

Hi, I know this isn't really an issue, but I'm not sure where to ask this... Is there a chance you're going to do a Ventura Blocker this year? Monterey Blocker was incredibly helpful to us last year, when months after it was released VMware Workspace One still didn't have a way to delay macOS software updates.
 I would rather our users not miss out on critical security updates, and honestly 90 days after the release of Monterey was not enough time for all of our vendors to issue updates. I predict a similar outcome this year as well.
Thanks for all you do for us Mac Sysadmins!

https://github.com/dufto/venturablocker (?)


Highlight scalar assignments

In x="foo", highlight "foo", as we already do for typeset x="foo" and for x="foo (incomplete) in command position.
The RHS of an assignment is implicitly double quoted (like reserved word typeset, unlike builtin typeset).  I'm not sure whether this has implications for the highlighting.

The double quotes are eaten by https://github.com/zsh-users/zsh-syntax-highlighting/blob/4ce56a821e9988c1a24fa1b1d62ed57f72893217/highlighters/main/main-highlighter.zsh#L551-L552
So how about this?
diff --git a/highlighters/main/main-highlighter.zsh b/highlighters/main/main-highlighter.zsh
index 12e48fe..cd21d65 100644
--- a/highlighters/main/main-highlighter.zsh
+++ b/highlighters/main/main-highlighter.zsh
@@ -547,7 +547,7 @@ _zsh_highlight_main_highlighter_highlight_list()
         fi
         (( in_redirection++ )) # Stall this arg
         continue
-      else
+      elif ! _zsh_highlight_main_highlighter_check_assign; then
         _zsh_highlight_main_highlighter_expand_path $arg
         arg=$REPLY
         _zsh_highlight_main__type "$arg" 0

It also fixes #571.

I think it would be better to not change arg, so how about this?
diff --git a/highlighters/main/main-highlighter.zsh b/highlighters/main/main-highlighter.zsh
index eb49d8a..958c048 100644
--- a/highlighters/main/main-highlighter.zsh
+++ b/highlighters/main/main-highlighter.zsh
@@ -559,8 +559,7 @@ _zsh_highlight_main_highlighter_highlight_list()
         continue
       else
         _zsh_highlight_main_highlighter_expand_path $arg
-        arg=$REPLY
-        _zsh_highlight_main__type "$arg" 0
+        _zsh_highlight_main__type "$REPLY" 0
         res="$REPLY"
       fi
     fi

With the above change, the only place arg is overwritten is for parameter expansion (which makes the highlighting of
foobar='x=$(ls)'
$foobar

interesting).

Scalar assignments have been highlighted for a while.  (They went away and came back in #663.)  The concern in the previous comment is valid so I filed it as #670.


Looking forward to support for SAM

Hi, do you have plans to support SAM?

Hey @SHEN2BAIYI,
thank you for your request; you mean the Segment Anything Model (SAM) from Meta, right?
Currently, we don't have any concrete plans to support this model.
However, we always value community input on potential new features, so we'll add it to our list of future to-dos.
Also, we always encourage community contributions, so if you're particularly interested in SAM integration, you're very welcome to start implementing it as a community contribution and open a PR.
I will close this issue as complete for now; if, in the future, the community or our team decides to add support for SAM, we will link back to this issue.


Boilerplate checks on 2014, but commit included a 2015 (breaks travis)

Revert boilerplate to 2014 until hack/verify-boilerplate.sh is fixed

@thockin @brendandburns can I get a merge on this so travis starts passing again?

cc: @zmerlynn

Sorry, I still need to add the precommit hooks.

I should have these in my repo now. Egg, face, apologies. I'll fix hack/verify-boilerplate.sh now.

My pr #3225 was out with one proposed fix for boilerplate issue of "Copyright 2015".

Missed that. New fix should last another year. I was about to add a loop
but this is easy enough.
On Mon, Jan 5, 2015 at 2:13 PM, Dawn Chen<EMAIL_ADDRESS>wrote:

My pr #3225 https://github.com/GoogleCloudPlatform/kubernetes/pull/3225
was out with one proposed fix for boilerplate issue of "Copyright 2015".
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/3226#issuecomment-68791510
.


Initial support for Tang nano 9k board

Pending PR https://github.com/amaranth-lang/amaranth/pull/734 for platform support.

The CI is stuck on "amaranth.vendor.gowin" missing.
I guess all it needs is to wait that https://github.com/amaranth-lang/amaranth/pull/734 gets merged...

Yes, that's right @josuah, that's up to @whitequark. I've ironed out a few more things in the last weeks while actually working with the board.

I now have a Tang Nano 9K and can test this PR.

See https://github.com/amaranth-lang/amaranth-boards/pull/233.

Obsoleted by #233. Thank you for this work!


test-branch/execute-job/wrong-commit-sha-on-pr/b777f17e-9cf2-4ace-ab42-5461964f9350

PR for testing 206b05248d1df151d88656b51eca16e8f5377bba

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


Add "Ball and Stick" Render option (new default)

The new default rendering style should be with balls and sticks:
http://3dmol.csb.pitt.edu/viewer.html?url=http://3dmol.csb.pitt.edu/tests/test_structs/benzene.sdf&type=sdf&style=stick:radius~0.15;sphere:radius~0.4
or..
http://3dmol.csb.pitt.edu/viewer.html?url=https://pqr.pitt.edu/static/data/mol2/WQ/WQLVFSAGQJTQCK-VKROHFNGSA-N.mol2&style=stick:radius~0.1;sphere:radius~0.35&surface=opacity:0.7;colorScheme:whiteCarbon

Saving this for later, because this makes everything ball and stick
pqr.threeDMole.all_viewers[0].setStyle({}, {sphere: {radius: 0.4}, stick: {radius: 0.15}});

See ad07351 for this WIP. @vonbearshark

I've changed my mind - I think the current look is different enough that people like it, so let's just add this as another option (i.e., not just sticks and spheres).


Admin Product listing does not update page on url

Description
Seems that the product listing in admin area does not update the current page in the browser url, so when I visit a product detail and go back, the list restart always from page 1

Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know.
Thank you again for your valuable feedback.


dataset.py requires missing npy

Hi folks,
I followed your README and ran the default ``python train.py --trainer mapfe4mp --device_gpu 0"
The first time, error reports not found preprocessed data.
Then I modified config_mapfe4mp.yml  to enable preprocess_data and save_data.
Then I run again  but encountered the issue that missing
arget_agent_orientation.npy
oracle_centerlines.npy
relevant_centerlines.npy
during preprocessing.
This breaks data_utils.py function load_processed_files_from_npy, as required_variables_name_list not all satisfied.
I wonder how to perform preprocessing properly?
thank you.

Sure. Best wish to your thesis defense.

Hi folks, I followed your README and ran the default ``python train.py --trainer mapfe4mp --device_gpu 0" The first time, error reports not found preprocessed data.
Then I modified config_mapfe4mp.yml to enable preprocess_data and save_data.
Then I run again but encountered the issue that missing arget_agent_orientation.npy oracle_centerlines.npy relevant_centerlines.npy during preprocessing.
This breaks data_utils.py function load_processed_files_from_npy, as required_variables_name_list not all satisfied. I wonder how to perform preprocessing properly? thank you.

Hi，Have you successfully reproduced it?

Hi, It's a bit of hard but possible to make it run. I will close this comment. Thanks.

Hi, i am working on It @librlike, It will be done the next week, as well as the repository of argo2.

That will be great !


Update Notebook Example

Dear Developers Team,
Problem
In the current master branch, the notebook is not executable as the NICE package (v0.2.3) has a different structure than intended by the notebook's usage of the package.
Changes
In the PR, I updated the notebook to use the notebook properly and to show the output of the individual cells.
Best, Roman

Dear Roman,
Thanks for resolving this issue. I merged your pull request with the master branch.
Best,
Dieter


 However the exit code is 0.
Example input index.js:
spaghetti

output of js-compute-runtime --skip-pkg bin/index.js bin/main.wasm:
Exception while evaluating JS: (new ReferenceError("spaghetti is not defined", "<stdin>", 1))
  @<stdin>:1:29
  @<stdin>:1:121

Error: failed to initialize JS

Caused by:
    0: the `wizer.initialize` function trapped
    1: Exited with i32 exit status 1
       wasm backtrace:
           0: 0x3d82a1 - <unknown>!<wasm function 8744>
           1: 0x3d9434 - <unknown>!<wasm function 8760>
           2: 0x1c23a - <unknown>!<wasm function 300>
           3: 0x1bfd5 - <unknown>!<wasm function 298>
           4: 0x1c554 - <unknown>!<wasm function 303>

Wizer failed with status: exit status: 1

This is the correct error but the status code is 0:
...
Wizer failed with status: exit status: 1

➜  echo $?
0

This makes it impossible for tooling to detect build errors.

@williamoverton - do you know what version of js-compute-runtime this error occurred on?
I tried to replicate this but unfortunately could not, the exit code I have is 255
❯ js-compute-runtime --skip-pkg src/index.js index.wasm
Exception while evaluating JS: (new ReferenceError("spaghetti is not defined", "<stdin>", 1))
  @<stdin>:1:1

Error: failed to initialize JS

Caused by:
    0: the `wizer.initialize` function trapped
    1: Exited with i32 exit status 1
       wasm backtrace:
           0: 0x3d61d5 - <unknown>!<wasm function 6078>
           1: 0x3d7d0c - <unknown>!<wasm function 6088>
           2: 0x402ec - <unknown>!<wasm function 263>
           3: 0x3ffb2 - <unknown>!<wasm function 261>
           4: 0x407a7 - <unknown>!<wasm function 266>

Wizer failed with status: exit status: 1
❯ echo $?
255

@williamoverton I've now confirmed this bug and found the fix for it 👍
The bug lies within the JavaScript package which unfortunately looks to not be open source. The package creates a child-process for js-compute-runtime but doesn't reflect the exit code of the child process onto the parent process.
The fix for this would be to capture the child-process result object on line 53 of @fastly/js-compute/js-compute-runtime-cli.js and apply the status as the process.exitCode, the diff below shows the change required.
-spawnSync(binaryPath, process.argv.slice(2), {
+const result = spawnSync(binaryPath, process.argv.slice(2), {
  stdio: "inherit",
});
-
+process.exitCode = result.status;
console.log(" ");

@torch2424 - it looks like the fix has not yet been published, do you know when that will be?

@JakeChampion Hello! 😄
Yes, so we're currently in the process of automating our deployments, and our plan was to use this next release to test our Github Action for this.
So, once we get that nailed down, it should be released! If you don't mind me asking, is this fix blocking you on your project?
Let me know, thanks! 😄

It's not blocking me, I already have the patch applied in my projects 😊


Streaming the camera to a media player uses internal url

Version of the custom_component

0.9.23
Configuration

service: camera.play_stream
data:
  media_player: media_player.bedroom_home_hub
target:
  entity_id:
    - camera.front_door_main

Describe the bug
The stream will fail to play and the logs show that an internal_url was used (http://ip_address/...) when I do not have that defined in my configuration. I do have external_url only, to avoid this issue with casting devices.
Debug log

Failed to cast media http://ip_address:8123/api/hls/dd56f293afd20bed8fad30c64b7bde11198bc2d66108f2edc7821a23b354d6b8/master_playlist.m3u8 from internal_url (http://ip_address:8123). Please make sure the URL is: Reachable from the cast device and either a publicly resolvable hostname or an IP address

Note that I replaced my real internal address with ip_address above.

Hmm, I'm not sure what to do here. I get what you are trying to do, and to cast some of these devices need a public facing URL. But I'm not sure how to make your external_url map to your internal IP address.  If you have multiple cameras, how would HA know which one to route to just from the external URL alone?

For multiple cameras, the internal_url is still the same. It's the Home Assistant http url not camera IPs. The path would be different per camera, essentially HA proxies the stream itself. At least that's my understanding of how HA handles cameras and streaming.
Example:
http://ip_address:8123/api/hls/foo
http://ip_address:8123/api/hls/bar

would become
https://my.external.domain/api/hls/foo
https://my.external.domain/api/hls/bar

The method that this component uses to report its camera's address for the play_stream service would need to reference the external_url for a cast-device to play it.


Support EXEC for evaluating commands without /bin/sh available yet

In keeping with Docker best practices, I want to avoid using ADD during my stage3 Gentoo install, using something like
COPY tar /tar
COPY stage3.tar.bz2 /stage3.tar.bz2
EXEC ["/tar", "-C", "/", "-xvf", "/stage3.tar.bz2"]
RUN ... rest && \
    of && \
    Gentoo && \
    install && \
    commands && \
    here ...

Could we get a feature like EXEC, so we can finally move away from ADD?

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@mcandre not following - why can't you just use RUN instead of EXEC at that spot?

Closing as it's unclear from issue what feature it should be. Feel free to open new issue.


Add Github action to automatically deploy to Heroku.

Automatic deployments using Heroku-Github Oauth integration
is currently broken.
Change-type: patch
Signed-off-by: Carlo Miguel F. Cruz<EMAIL_ADDRESS>

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@balena-ci I self-certify!


Minimalist example shows spinning window.

I've tried a few minimalist examples but the parent never renders the iframe.  In this example I see my "Parent Page" html rendered, I see the iframe box, though the pre-rendering spinner just spins and the page is never shown.  I can see the 200 response from /helloworld.  No console errors.  No network errors.  Full refresh. Not sure where to look.  Tried firefox, chrome, etc.  I couldn't even see a sample js fiddler to try as a base.
Maybe these cnds are old?
`

Parent Page

        Parent Page

`

It just seems to stick at zoid-invisible   If I put the iframe to zoid-visible manually I can see the page rendered.  I suppose there's a create parameter I'm not setting?

Of course, it was my fault because the parent and child component tags didn't match perfectly!

Parent

Parent Page

<script>
    // Render the Zoid component
    MyComponent().render('#my-container');
</script>

CHILD:

Child

Child Page


Multiple outputs do not work

Hi!
I'm trying to use logstash-logger to send my rails logs to remote Logstash server and to write them to file, as usual.
I've added to application.rb
    config.logstash = [
      {
        type: :file,
        path: 'log/production.log'
      },
      {
        type: :tcp,
        port: 5000,
        host: 'localhost'
      }
    ]

And receive
13:07:04 worker.1 | no implicit conversion of Symbol into Integer
13:07:04 worker.1 | /Users/smekhonoshin/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/logstash-logger-0.7.0/lib/logstash-logger/device.rb:31:in []' 13:07:04 worker.1 | /Users/smekhonoshin/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/logstash-logger-0.7.0/lib/logstash-logger/device.rb:31:in parse_uri_config'
13:07:04 worker.1 | /Users/smekhonoshin/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/logstash-logger-0.7.0/lib/logstash-logger/railtie.rb:9:in setup' 13:07:04 worker.1 | /Users/smekhonoshin/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/logstash-logger-0.7.0/lib/logstash-logger/railtie.rb:33:in block in class:Railtie'
Does anybody know how to solve this issue?

This feature has been merged but not released yet. If you want to try it out, you can link to the Github repo.

@dwbutler thanks, I'll check it out.
And another question, is it possible to write to file raw rails logs and send to logstash their json representation?
For development it is more pretty to see them in original format.

@dwbutler, I have the same problem on 0.7.0 gem from rubygems.org; I thinks that you need remove it from the oficial docs of 0.7.0 version.

I don't think it's possible to do that from a single Logger instance. The logger will first format the log message into a string using the formatter, and then send the formatted output to the device.
You should probably just not use LogStashLogger in development. =)

This feature has been released in version 0.8.0. Sorry again for the confusion.


refactor: UI design system refactor

Issue #, if available: GALI-1707
Description of changes: refactor to use the Cloudscape Design System instead of the AWS UI Design System. Cloudscape is the "evolved" AWS UI and we followed this migration documentation. See comment below for the comparisons of the UI screens.
Checklist:

[x] Have you successfully deployed to an AWS account with your changes?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS UI login screen:

Cloudscape login screen:

AWS UI workspaces screen:

Cloudscape workspaces screen:

Cloudscape create workspaces screen:

Cloudscape create researcher user

screen:


Crash when the installation is missing "account"

https://github.com/probot/pino/blob/0d0527c819c5287e95ca136209b4ce5611d4610d/index.js#L75 crashes.
The webhook probot gets contains:
},
  "installation": {
    "id": 12276402,
    "node_id": "MDIzOkludGVncmF0aW9uSW5zdGFsbGF0aW9uMTIyNzY0MDI="
  }

So it has no information about account or user. Honestly, I don't know why. But probot works, my integration worked since 2018. Would be great if error reporting would not crash.

can you share the full event payload? I'd like to understand what the event/action is where the account property is empty. For the time being, can you send a PR to address the case, to prevent the crashes?

Sure I can, but it is not going to be much helpful. None of my events have account property. The app is subscribed only to Pull Request events. So this happens with synchronize, labeled, unlabeled, opened, reopened...
Yep, will send a PR.

Maybe because the app is installed not to the whole organization, but only to some repositories?


Patch 4

What is it?

[x] Feature / enhancement
[ ] Bug
[ ] Docs / tests / types / typos
[ ] Infra

Description

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have ran pnpm change and documented my changes
[ ] I have made corresponding changes to the Qwik docs
[ ] Added new tests to cover the fix / functionality

I close this one, your website is already there.


Zip task should not require absolute working directory

Commit https://github.com/loresoft/msbuildtasks/commit/9ea598cdc230e6c1ccd76ea5574bdce01a8bfb9d started to expand the path of the Zip task's Files parameter.
Since the WorkingDirectory parameter is (only) used to strip a common prefix from all input paths, it stands to reason that the WorkingDirectory argument should also be expanded to restore the previous behavior.

Also, since all relative paths in an MSBuild file are relative to the file itself, the WorkingDirectory's default value (if empty, i.e. not specified) could be $(MSBuildProjectDirectory).
That way the user doesn't have to do it, like in this example that packages all source files into a ZIP file with relative file paths:
<Target Name="AfterBuild">
    <ItemGroup>
        <FilesToZip Include="**\*.cs" Exclude="obj\**\*.*"/>
        <FilesToZip Include="**\*.xaml"/>
        <FilesToZip Include="**\*.csproj"/>
        <FilesToZip Include="**\*.xml"/>
        <FilesToZip Include="**\*.md"/>
    </ItemGroup>
    <Zip ZipFileName="$(OutputPath)\$(AssemblyName)-demo.zip"
         Files="@(FilesToZip)"
         WorkingDirectory="$(MSBuildProjectDirectory)"/>
</Target>


screw_ie8 is not a supported option

On the current master branch, I end up with this error when I run npm run build.
ERROR in main-ee2fb0ea.js from UglifyJs
DefaultsError: `screw_ie8` is not a supported option

Fixed UglifyJS version in https://github.com/ModusCreateOrg/budgeting-sample-app-webpack2/pull/60
Thanks for reporting!


Run lint instead of test in CircleCI

Instead of running npm test that should fail, lets just print that we have no tests and then exit cleanly
Fixes failing build from here: https://github.com/ipfs/interface-ipfs-core/pull/36

Seems circle is not happy due to a missing dependency, I've give it a kick to restart with no cache, let's see if that was it.

@diasdavid passing now :+1:


Specify correct key_dst for fixtures

Fixtures do not use the default key_dst defined by KeyGen.

Discussed on the WG call on the 11th of Dec. Multiple approvals. Merging. Thank you!


Implementation of analytics

Implement Google Analytics and/or investigate a suitable alternative.
Currently access to Google Analytics is not giving updated/full information.
Some results are not currently appearing in Google Analytics and are being manually added to data studio.

Frequently visited sections
Types device
User time on site
Bounce rate

RapidPro is is using PowerBI.
There is a Google Analytics -> PBI plugin.
Poll and survey data analytics

Agreed with Laurie that we will stick with Google Analytics for v2 and investigate and look at introducing PowerBI from v2.1.

@lauriemarkle - The old IoGT sites use Google's original Universal Analytics (UA).  Are you happy for us to swtich over to the new version 4 of Google Analytics instead?  The latest version takes a modified approach, so the metrics are not identical - but now seems like a very good time to switch over.
Version 4 tracks all events for each user on the website instead of creating a summary for a user's session. It has fewer fixed/default reports, but has a more powerful way to perform analysis of the user's interactions with the sites.
Also - are you using Facebook tracking?

Hi Ian,
Yes, let's make the switch. I agree that now is the right time. The
challenge will be the loss of CO dashboards, but I think there's a remedy.
We get access to a Facebook Partners Portal for Free Basics. It incorrectly
tracks traffic, so I don't use it for this purpose. It only is used to map
IoGT CO URLs with Free Basics portals globally. This should not be
impacted.
On Wed, Jun 2, 2021, 3:47 PM Ian Edwards @.***> wrote:

@lauriemarkle https://github.com/lauriemarkle - The old IoGT sites use
Google's original Universal Analytics (UA). Are you happy for us to swtich
over to the new version 4 of Google Analytics instead? The latest version
takes a modified approach, so the metrics are not identical - but now seems
like a very good time to switch over.
Version 4 tracks all events for each user on the website instead of
creating a summary for a user's session. It has fewer fixed/default
reports, but has a more powerful way to perform analysis of the user's
interactions with the sites.
Also - are you using Facebook tracking?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unicef/iogt/issues/22#issuecomment-852997581, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ARYZHTWKJJH6AHOL4UZDYZTTQYR6JANCNFSM4YCP5X2A
.

GA settings implemented here


Add replicas_status request.

It's similar to ping! method
I haven't founded CONTRIBUTING file so I don't know that a rules to change version

@archan937 come please take a look


fix: change index.js to not have a default export

This allows TypeScript clients to import start like so.
import { start } from 'faas-js-runtime';

start(fn);

Otherwise, there was a problem with a default import of start in index.js
but a const type declaration of start in index.d.ts. This change fixes that
Signed-off-by: Lance Ball<EMAIL_ADDRESS>

should probably also update the example in the readme


sqlite-net ARM processor

I'm an inexperienced hobby programmer, so forgive my ignorance.
I'm writing a .net application that needs to run on Windows and Linux Mono (bot i386 and ARM processors).  I started using Syste.Data.SQLite and EF6 but that failed me on Mono, especially on the ARM platform.
Research suggested I use sqlite-net, and I love the simplicity and ease of use, but I still can't get it to work on the ARM Mono platform.
Besides sqlit-net, I'm not sure which other Nuget packages are necessary for this multi-platform (single executable) deployment.
The error I'm experiencing on the ARM Mono version is:
System.TypeInitializationException: An exception was thrown by the type initializer for SQLite.SQLiteConnection ---> System.DllNotFoundException: e_sqlite3
at (wrapper managed-to-native) SQLitePCL.SQLite3Provider_e_sqlite3/NativeMethods:sqlite3_libversion_number ()
at SQLitePCL.SQLite3Provider_e_sqlite3.SQLitePCL.ISQLite3Provider.sqlite3_libversion_number () [0x00000] in :0
at SQLitePCL.raw.SetProvider (ISQLite3Provider imp) [0x00000] in :0
at SQLitePCL.Batteries_V2.Init () [0x00000] in :0
at SQLite.SQLiteConnection..cctor () [0x00000] in :0
--- End of inner exception stack trace ---
It seems to be looking for "libe_sqlite3.so"
I'd welcome some advice please?

libe_sqlite3.so is pre-compiled for x64. This means it won't work on ARM out of the box.
The workaround would be compile sqlite3.c yourself for ARM, then replace libe_sqlite3.so for your compiled build.

Thanks lubos.  Once complied, would I place the library in the /arm subdirectory like the other /x86 /x64 versions at present?

No. /x86 and /x64 sub-folders are for Windows only. On Linux, you need to have libe_sqlite3.so in the base directory just like most of your .NET assemblies.

I was afraid you'd say that.  My issue is I have no idea what architecture my application is being deployed to, so don't know which library to deploy.  Ideally I need to deploy all of the various assemblies and have the application select the most appropriate for the underlying OS and processor (Windows vs Mono, i386 vs ARM, etc).  I'm afraid I don't know where to start in trying to achieve this.
Until now, I've been lucky that all of my applications complied against and x86 CPU have worked on Windows and Mono without issue.

I've discussed this with @ericsink who has made the underlying library SQLitePCL.raw and he has made serious effort into trying to make Linux load binary from sub-folders based on the platform but it became too complicated.
The only way would be to compile SQLite3.c on target machine during deployment. This means you wouldn't distribute any binary on Linux and your package could stay noarch. Or simply prepare multiple downloads - one for each platform.

FWIW, I do expect this is going to work with .NET Core (as opposed to Mono).  In fact, I've just published a prerelease testing build of SQLitePCLRaw.lib.e_sqlite3.linux with ARM support included:
https://www.nuget.org/packages/SQLitePCLRaw.lib.e_sqlite3.linux/1.1.8-pre20170717084758

I still need to test this on an RPi -- hope it works!

I have received feedback that my test package works (on .NET Core).  This will be part of the soon-to-be-released SQLitePCL.raw 1.1.8.


Rename the 'default' table slice type to 'caf'

The 'default' table slice type has not been the default for a while now when Apache Arrow is available. The name 'caf' more accurately describes what it does.

I have verified this to work locally using ./configure --without-arrow.


Enable dataPath in all errors, rename verboseErrors to reflectErrorsValue

Depends on #110.
Can be done now with a speedup of dataPath from #110 and this PR.

Codecov Report

Merging #111 into chalker/split-methods will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted Files
Coverage Δ

src/index.js
98.80% <100.00%> (ø)

src/scope-functions.js
100.00% <100.00%> (ø)

Needs rebase

Rebased.


Schedule jobs with ignored cache hints.

Armed with restartable jobs https://github.com/ocurrent/ocluster/issues/167, allow jobs to be started on an alternate worker from the cache hint (e.g. because the required worker is full or out of service) but potentially terminate and restart the job on the original worker if it becomes available

We often see the current scheduling to result in one worker getting overloaded with a massive queue due to cache hints while other workers are starved for work.
As I understand this issue, it entails an improved scheduling algorithm that will take cache hints into account, but also consider worker capacity and availability. There are times where this should be able to really improve our thru put.
I'm gonna rename the issue to make that intent a bit clearer.


Add support for other init systems beyond runit and Windows

Please add support for other init systems on supported platforms, such as sysvinit and systemd units.

We should do for this cookbook what I propose for the chef client cookbook here
https://github.com/opscode-cookbooks/chef-client/issues/282
See the services section.

:+1: to @jtimberman's idea.

:+1: to this

👍

We're planning to implement basically what @jtimberman referenced above with chef-client.  We'd replicate that same functionality with the push jobs provider.  Out of the box the provider would just use whatever the native init system was.  You could override that if you enjoyed runit for some reason, but out of the box you'd get sys-v, systemd, or upstart depending on platform and release.

That sounds delightful @tas50! Working with a team trying to implement this behind the firewall and the runit dependencies are a bit.... annoying. Seeing the cookbook move to this is ❤.

This was done


Use React.FC and remove PropsWithChildren

This PR
Closes https://github.com/LowLevelAcademy/LowLevelAcademy/issues/1 by

Changing all React.FunctionComponent to React.FC
Removing all unnecessary instances of PropsWithChildren
Removing some unneeded identifiers

I went with React.FC everywhere instead of FC everywhere because it seemed more consistent but would be happy to import FC from React in every file instead!

I just realized I never really tested this 🤦 I can't get the build up. When I run npm run dev I get
./node_modules/next/dist/client/dev/amp-dev.js
Error: Cannot find module '@babel/preset-typescript' from '/Users/marklodato/repos/LowLevelAcademy'

are we missing a dependency somewhere or something?

are we missing a dependency somewhere or something

Hmm, yeah, most likely I have forgotten to include some of the dependencies locally and add them to package.json.
Could you please give it a try after npm install --save-dev @babel/preset-typescript
Thank you!

Awesome, that worked. I'll test it and push that change up too 👍

Hmm, now it fails when I navigate to the encoding numbers module. The LessonHeader fails to compile and it seems like a babel error:
error - ./components/LessonHeader.tsx 16:18
Module parse failed: Unexpected token (16:18)
File was processed with these loaders:
 *<EMAIL_ADDRESS> * ./node_modules/next/dist/build/webpack/loaders/next-babel-loader.js
You may need an additional loader to handle the result of these loaders.
|   }, props.module), /*#__PURE__*/React.createElement("div", {
|     className: "badges"
>   }, props.badges?.map(badge => /*#__PURE__*/React.createElement("span", {
|     key: badge,
|     className: "module-badge"

I have no idea what's going on with this one 🤔

Oh, might need next/babel in the presets, lemme try that

Do I need to run the server to test this as well? When I try to click through the modules I get 404s when hitting /compile. I tried to start up the server in development mode but got:
~/r/server (master|✔) $ ROCKET_ENV=development RUST_BACKTRACE=1 cargo run --release
    Finished release [optimized] target(s) in 0.11s
     Running `target/release/server`
🔧 Configured for development.
    => address: <IP_ADDRESS>
    => port: 8000
    => log: normal
    => workers: 24
    => secret key: generated
    => limits: forms = 32KiB
    => keep-alive: 5s
    => tls: disabled
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: NotPresent', src/main.rs:58:58
stack backtrace:
   0: <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt
   1: core::fmt::write
   2: std::io::Write::write_fmt
   3: std::panicking::default_hook::{{closure}}
   4: std::panicking::default_hook
   5: std::panicking::rust_panic_with_hook
   6: rust_begin_unwind
   7: core::panicking::panic_fmt
   8: core::option::expect_none_failed
   9: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  10: tokio::runtime::enter::Enter::block_on
  11: tokio::runtime::thread_pool::ThreadPool::block_on
  12: tokio::runtime::context::enter
  13: tokio::runtime::handle::Handle::enter
  14: rocket::async_main
  15: server::main
  16: std::rt::lang_start::{{closure}}
  17: std::rt::lang_start_internal
  18: main

Oh, maybe I need that docker image mentioned in the README?

Do I need to run the server to test this as well?

Not necessarily. The server integration testing definitely could be improved and most likely we’ll just provide a complete Docker image for it. The Docker image mentioned in the Readme is only for the Rust compiler, for now.
In the meantime, though, I believe it doesn’t work with code from this repository out-of-the-box.
Well, anyway, this repo also could use some automatic unit/integration tests. :)
But if it works for you when you run the build command (npm run build), then you don’t need to do any integration testing for this particular pull request since it modifies types only.

Yeah I was just clicking through the app in case, when I was modifying a type, I accidentally deleted some code that mattered :-)
But npm run build works so this is otherwise ready to review!

No worries! Let me know if there are any other kind of pedantic things I can offload from you. I'm very excited about this site and I think it's an awesome idea :-)


Giving 777 permission to cache folder is unnecessary and dangerous

Giving a folder or file 777 permission is not recommended in any case.
A more suitable permission would setting permission to 744 and change the owner to the correct user the webserver runs as.
Verified that it is working:
https://mc.selea.se/lookup

Yeah, that isn't necessary.  The guide was written before I knew more about Linux permissions.
Directory permission of 744 or 774 suffices when the owner is webserver (www-data:www-data) or if the webserver is in the group.  In fact, my setup is exactly that.
If you'd like, you can PR updates on the README.md.  Else, I will update it as version 1.0.0 comes around completion.

Alright!
Looking forward to 1.0.0! Already had good use of this tool, even thou I did'nt manage to get setup.php to work (don't know what's wrong thou).

I removed the cache implementation.  I don't know if cache will be necessary at all, but if anyone thinks otherwise, I guess I can come back to it.  Thank you for the issue! (Commit at close: e5b36f36d79bfcea6891ca0b2f95266ead01b0fd)


Ignoble repost of this repo !

They took the main.js, suppressed comments and added THEIR names as author!
https://iohack.webnode.fr/s0urce-io/ and its doing the same thing as here !

Yup, sure looks that way. I appreciate the heads up.


Compare GNU Parallel to WireIt's main usage

Notice: this is not a feature request or an error respond on using wireit, just a question which I'm curious about.
If this issue violate any rule of this project, please directly close this issue.
GNU Parallel's wikipedia page:
https://zh.wikipedia.org/wiki/GNU_parallel
Since the main usage of wireit looks as same as this tool, I'm wondering if there is any difference between these two.
Or this may be my misunderstood of both wireit and GNU parallel's usage, in that case, may I ask for a explanation of these two tools?

Hi @danny-huang-openfind, thanks for the question.
Similarities:

Both execute shell commands
Both can run multiple shell commands in parallel

Differences:

Wireit is tied to the npm ecosystem. It has to be launched using an npm-script runner like npm, pnpm, or yarn. GNU parallel is a more general-purpose unix command that you invoke directly from the command-line.

Wireit scripts are configured in the scripts section of a package.json file, while GNU parallel scripts are provided directly as command-line arguments.

Wireit is significantly more powerful than GNU parallel, with the following additional features:

Wireit scripts are connected into a graph using dependencies. So if you say "A depends on B and C" then B and C will first run in parallel, followed by A running only when both B and C are done. GNU parallel runs all commands in parallel, since it has no notion of a dependency graph.
Wireit has a built-in watch mode, where scripts will automatically re-execute whenever their input files change.
Wireit won't run scripts that have recently run and don't need to run again.
Wireit can restore output from a cache (either from local disk, or on GitHub Actions).
Wireit has the concept of services, where a process like a server can be brought up and shut down automatically


Failing: Model compiling using npx whisper-node download

I'm getting this error in the compiling part of the npx whisper-node download command:
(ENTER for base.en): base
Downloading ggml model base...
Done! Model base saved in C:\Users\node_modules\whisper-node\lib\whisper.cpp\models\ggml-base.bin
You can now use it like this:
main.exe -m C:\Users\node_modules\whisper-node\lib\whisper.cpp\models\ggml-base.bin -f C:\Users\node_modules\whisper-node\lib\whisper.cpp\samples\jfk.wav
[whisper-node] Attempting to compile model...
process_begin: CreateProcess(NULL, uname -s, ...) failed.
Makefile:2: pipe: No error
process_begin: CreateProcess(NULL, uname -p, ...) failed.
Makefile:6: pipe: No error
process_begin: CreateProcess(NULL, uname -m, ...) failed.
Makefile:10: pipe: No error
'cc' is not recognized as an internal or external command,
operable program or batch file.
'head' is not recognized as an internal or external command,
operable program or batch file.
I whisper.cpp build info:
I UNAME_S:
I UNAME_P:
I UNAME_M:
I CFLAGS:   -I.              -O3 -std=c11   -fPIC -mfma -mf16c -mavx -mavx2
I CXXFLAGS: -I. -I./examples -O3 -std=c++11 -fPIC
I LDFLAGS:
I CC:
I CXX:

cc  -I.              -O3 -std=c11   -fPIC -mfma -mf16c -mavx -mavx2   -c ggml.c -o ggml.o
process_begin: CreateProcess(NULL, cc -I. -O3 -std=c11 -fPIC -mfma -mf16c -mavx -mavx2 -c ggml.c -o ggml.o, ...) failed.
make (e=2): The system cannot find the file specified.
make: *** [Makefile:176: ggml.o] Error 2

Did you installed make tool ? In windows

@ChetanXpro yes I have it installed but it could be possible I have not done it correctly. Could you link me to some correct instructions for installing? I believe I have it installed correctly though.


Make user enumeration by timing attacks more difficult

When authenticating a user, we skip any calls to BCrypt if we weren't able to find a user with the provided email address. This results in a faster response time to the client and less work on the server... but it also means one could possibly monitor the response times from the app to see if the provided user has an account.
When we detect that a user does not exist with the provided email, we should go ahead and perform a bcrypt check anyway, and always return false from that check. This will keep timings consistent with the response one gets when they use an email that exists but get the password wrong.
I took a quick look at how this could be done. I feel like the best way would be a new "dummy_authenticate" method on the password strategy that is called for this explict purpose. That's a breaking change to the contract of the password strategy though.
Another alternative would be to handle this in the user mixin itself. We'd want to make sure we produce similar timings in any event.

As I understand it, the strategy's #authenticated? method (e.g. https://github.com/thoughtbot/clearance/blob/7a63a2a9363729c743693d74f725337b860ee314/lib/clearance/password_strategies/bcrypt.rb#L12 ) needs to take a constant amount of time. So changing all the #authenticated? methods in lib/clearance/password_strategies should do it?
def authenticated?(password)
  result = ::BCrypt::Password.new(encrypted_password || "") == password
  encrypted_password.present? && result
end

User#authenticate would also need to be updated in order to make it to the #authenticated? method.

Great catch, and now I see why @derekprior thought it should be an API change.
I just tried sketching out an API that is resistant to timing attacks but I couldn't do it quickly.

@mike-burns you also can't do BCrypt::Password.new(encrypted_password || "") because "" is not a valid hash and it will throw BCrypt::Errors::InvalidHash. You could do a cheap hashing implementation BCrypt::Password.create("", cost: 1), but I don't know if a cost of 1 will create a recognizable difference for a timing attack (basically it would now take slightly longer for accounts that don't exist).

You could always just use some random valid bcrypt hash instead of "" there.

There is an option of just doing the dry run of encryption if the condition fails. Meaning that in terms of time comparison of encrypted string with Password.new("encrypted_string") == "password" is roughly the same as doing Password.create("random_stuff")
Here are some benchmarks to prove it:
n = 1000

Benchmark.bm do |x|
  x.report { n.times { BCrypt::Password.create("Hello") } }
  x.report { n.times { BCrypt::Password.new("$2a$10$fELdjGQGJU23naA9lon9FO8jPlrFiDwFcyU2BcnEtg9NfTzfEhmVW") == "Hello" } }
end

Result:
       user     system      total        real
  65.270000   0.150000  65.420000 ( 65.590176)
  65.800000   0.130000  65.930000 ( 66.031897)

So what I am getting at here is that it might be good to have authenticated method that looks like this:
def authenticated?(password)
  if encrypted_password.present?
    ::BCrypt::Password.new(encrypted_password) == password
  else
    ::BCrypt::Password.create("my_random_string")
    false
  end
end

Please note that I am not a security expert and my assumptions might not be correct. :)

👋 Hey! We've recently opened a bug bounty against this issue, so if you want to get rewarded 💰 for fixing this vulnerability 🕷, head over to https://huntr.dev!