Inter-window messages sent to a view via ViewId.update_state() are broken

This one I'm not sure how to solve:  If I have a view in a popup window, and it sends a message to the view that created it in the main application window, the message is delivered (and the message may close the popup!) - but the view does not process the message until some OS-level event like waving the mouse over the main window causes it to wake up and process messages in its queue.
Here is a gist with a complete, runnable example demonstrating the problem - I'd recommend just copy/pasting it and running it (instructions on exactly what to do will be in the main window) - it's big as bug demos go, but I figured it would help to have something that shows the utility of this sort of thing.
I get the optimization I think is being attempted here - if the window is not active, have it be completely quiet.  In general, that's a good thing - but this is a little too aggressive.  If it's intended to remain that way, then there needs to be some way for a caller to flag a message as urgent or at least a way to bang on its event loop and force some messages through.

Some notes from digging into this a little bit, to see what possible solutions could be:

The entry path to process any messages at all for a view is ApplicationHandle.handle_window_event(WindowEvent::RedrawRequested) -> WindowHandle.render_frame() -> WindowHandle.process_update_no_paint -> WindowHandle.process_update_messages()
So, there is literally *no path to reach process_update_messages() currently except via an OS-level paint event

I suspect creating any path to call that method that does not do it by generating such an event could easily violate assumptions floem makes internally about threading, or platform requirements that paints happen on a specific thread
In a sense, the root problem here is that ViewId.request_repaint() does nothing if the view is in an inactive window - it won't be processed until something else - native input event processing from winit triggers a repaint for its own reasons.

That, I think, is the bug here: a repaint request should not be deferred until ... something else requests a repaint.  In a don't-call-us-we'll-call-you framework like this, where all work is driven by repaint requests, repaint requests themselves have to be special and privileged.

Now, how to fix it minimally?
It seems like forcing a repaint event through winit - i.e. tell the OS the window needs a repaint - is the only safe and sane way to do it - it ensures platform restrictions about when and how repainting code runs is honored.  So, something like

Repaint requested on id
Find the root view id for the window
Find the window id from the root
Is it the active window?

Yes? Put the message in the queue and done, as it happens now
No? Do whatever dance is necessary with winit to send an OS-level repaint event to the parent window (I haven't looked at how to do that yet - going to do that now)

I see a few other things that seem questionable in the core update-messages plumbing:  The use of thread-locals for the message queues is probably because you needed something mutable not because you needed something local to a specific thread - thought experiement:  If I spawn a thread and push 100k state update messages to a view, then park it unitil the end of time, will those messages ever be processed?  No.  Will they ever be dropped?  No.  I think the data structure you want is a treiber stack (doesn't have to be my implementation that I just linked, but it's the right algorithm - a single-ended stack using atomics - UPDATE_MESSAGES is single-ended stack, and you just need a way to push and pull in a thread-safe way that doesn't run into mutability / borrow issues - there might even be benefits to the queue not needing a heap-allocated Vec per id).  Anyway, that's for later examination - I'm mentioning it here because it's pretty intimately tied to code this bug touches (and so I or someone remembers to look at it).

More notes:  Thus far, digging through the code, I have not found any way for a View to get a reference to the WindowId that contains it (or for how to get a reference to the ApplicationHandle that owns the Window and can see windows and views).  The relations (not surprising in Rust) are entirely one-way.
At the application-level, you get a window id when you create a window, so the user application knows the window ID being used - and the view can find its root view in the window inside the floem crate - it can associate a view with a window id, and, if there was a way to call a force_native_paint(WindowId) function, trigger that paint - but that would be a dreadful API, since the application author would have to pass window ids down through every view that might possibly need to force a repaint, know they need to force one, and imperatively do so when needed - and the whole point here is that request_repaint() should work as advertised and simply cause a repaint no matter what.
Altering ViewId lets me detect when the repaint request is happening on a view in an inactive window accurately - that solves half the problem:
    fn is_in_active_window(&self) -> bool {
        if let Some(root) = self.root() {
            return crate::window_handle::get_current_view() == root;
        }
        false
    }

    fn schedule_native_paint_event(&self) {
        println!("Need native paint for {:?} root {:?}", self, self.root());
    }

    pub fn request_paint(&self) {
        let active = self.is_in_active_window();
        self.add_update_message(UpdateMessage::RequestPaint);
        if !active {
            self.schedule_native_paint_event();
        }
    }

but what's needed to implement schedule_native_paint_event() to actually do something requires:

A way to get the WindowId for a view that is not in the active window
A way to get the WindowHandle for that id
Some way that does not go through the message queues which will not be processing messages to tell the window handle to take the native winit window handle and post whatever message the OS requires to send a native repaint event to that window

It's lovely how de-coupled Views are from windows - until you have a situation like this where there is no alternative to calls flowing up, not down the tree of lovely decoupled things.

Best I can think of is some similar registry of root-view:window-id and window-id:app-handle-id (app-handle-id is not a thing at present) and a background thread that sits quietly waiting on a channel for messages saying force a repaint to this native window id and dispatches them - which could probably do some sort of OnceCell lazy init thing so applications that don't ever post to that queue don't incur any overhead?
Am I missing something here?

One other note: There is also the issue of knowing the window's location on screen when instantiating other windows - which also is likely to need to be addressed by a C view accessing its containing window (though I'll note on Mac OS you can only query window size on the main thread, so some caching is needed) - I'm not sure the plumbing has this information initially either.

I've created a PR with a fix, and solving some general issues around window ids - it allows views to get their window id if they are realized on screen, extends WindowId with methods (in a separate trait) that let things like the window's screen bounds and monitor bounds for the monitor it is on be queried - following the same pattern used for ViewId already.
I am wondering a bit if there shouldn't be a sort of ScreenCoordinateSpace type you can get for a View, which has the necessary information to, say, compute window locations.  For example, updating the demo code I linked in the first comment above, computing a proper location for a popup window relative to a View is more complex than would be really delightful - we have to reach into the WindowId once to get the bounds-on-screen, and again to get the monitor bounds (in order to reposition the target point so the popup cannot extend beyond the monitor bounds):
    fn compute_screen_location(&self, local_point: Option<Point>) -> Option<Point> {
        if let Some(bds) = self
            .id()
            .window_id()
            .and_then(|wid| wid.bounds_of_content_on_screen())
        {
            let mut start = Point::new(bds.x0, bds.y0);

            start.x += self.window_origin.x;
            start.y += self.window_origin.y;

            if let Some(lp) = local_point {
                start.x += lp.x;
                start.y += lp.y;
            }

            if let Some(wid) = self.id().window_id() {
                if let Some(mbds) = wid.monitor_bounds() {
                    let expected_size = square_size();
                    if mbds.x1 < start.x + expected_size {
                        start.x -= (start.x + expected_size) - mbds.x1;
                    }
                    if mbds.y1 < start.y + expected_size {
                        start.y -= (start.y + expected_size) - mbds.y1;
                    }
                }
            }

            Some(start)
        } else {
            local_point
        }
    }

Thoughts?

I'm thinking of create_window took Into it would make it easy to provide convenience types to compute a screen position relative to a view (or whatever), and at long as they implement that too, it would be transparent and source-compatible for old code.

Updated the patch to add a ScreenLayout struct, obtainable from a ViewId, which contains a snapshot of the window bounds and other information necessary to convert from view locations to screen locations and back (it takes several calls to Window that return Result to obtain all of the necessary information to do these conversions, so it makes more sense to me to take care of that and offer the user something which is "batteries included").

I'm thinking maybe we should move process_update() to ApplicationHandle. What happened here is a event in one Window triggered updates in both windows. So the ApplicationHandle should process all update messages in CENTRAL_UPDATE_MESSAGES after dispatching the event.

Makes sense.
Seems like we might want to do something about AppState - it contains some things that are logically global and some things that are local to a window.

@dzhou121 - Take a look at this pr - it fixes the problem with very minimal changes.
I think in the bigger picture, you are right about process_update() belonging to AppHandle not WindowHandle, but looking at the code I asked myself if the code moves, does it change what windows get to process events? and the answer was that no matter what, we need to iterate all the windows and give them a chance to process updates.
I do think the code in ApplicationHandle and WindowHandle needs some untangling of what really belongs to windows and what belongs to the application, but we may want to solve this one thing now and boil the ocean later :-)
I would keep the other PR for this issue open and use it as a base for some improved window-management features, since it implements a few already.

Fixed in 6fe51db.


Use the version number present in the global.json file when generating documentation

This pull request addresses an issue whereby the documentation can be generated for a release with the incorrect version number. The version number is now calculated from the global.json file, thereby limiting the chances of the incorrect version number being used on a release.
Regular expressions and basic character manipulation has been used to avoid the need for JSON parsing and Semver libraries.
Backport to 6.x on merge.

ported to 6.x: 3b9cde872900d1ab76015fa92066f70fe3f2d171
ported to 5.x: 4e080471b2aeb923874371e8098c5faef6a2391a


cross platform: Runtime/types Part 2/2

https://github.com/Microsoft/ChakraCore/pull/726/commits/68482b2b4c51f25073ce5cc7e392be553726c049 also fixes https://github.com/Microsoft/ChakraCore/issues/758

we are using SEH for asm.js to avoid the outofbound check. I am not sure if it's easy to get rid of that SEH usage.

@obastemur please separate out the commits for #758 into its own PR to keep this review focused on Runtime/Types.

we are using SEH for asm.js to avoid the outofbound check. I am not sure if it's easy to get rid of that SEH usage. I don't know what does FF do for their 64bit asm.js support, should be something similar.

In that case; how about keeping as is and putting this info as a TODO comment there? Looks like getting rid of SEH is curicial for asm.js support on linux.

Actually, @obastemur, Curtis has a better suggestion. Use a destructor to execute the code in the finally part and get rid of the catch block. We still need the __try/__finally for Windows because of the behavior Yong talks about, but we could just have the finally block call the same method as the destructor (and use a flag for the destructor to know whether an exception was thrown or not)

Ok, talked to @jianchun and @Yongqu. My mistake, Yong was talking about other places where we use __try/__finally, not in this particular case. Here, we care only about C++ exceptions, the SEH usage for asm.js is not relevant here. In this case, please convert to using the TryFinally pattern that @jianchun introduced. You can replace the AbnormalTermination check with checking the hasException flag.

the SEH usage for asm.js is not relevant here.

please convert to using the TryFinally pattern

Could you please elaborate why? In that case I don't see a reason to maintain __try , __finally there. IMO, the try/catch(...) option implemented already takes care of the task cross platform.
BTW; @digitalinfinity thanks for the review!

You're right, I think try/catch should be fine. Although, instead of catch (...), for this case, better to scope to catch (Js::ExceptionBase) (with the rethrow). And you can remove __try/__finally on windows too 😄

@digitalinfinity thanks again. PR is updated.


The Files crash on startup after updating to 0.22.5

Describe the bug
The app is crashing on startup
Desktop (please complete the following information):

OS Version: [e.g. Windows 10 2004 20277 fe_release]
App version: [v0.22.5]

Additional context
Reinstall and reset settings doesn't work
Log file
debug.txt
debug_fulltrust.txt

me too

fyi @gave92

A bug that was causing crash at startup was fixed after 0.22.5 in #2648 but this stacktrace looks different
@mdtauk could you share your debug.txt as well?

A bug that was causing crash at startup was fixed after 0.22.5 in #2648 but this stacktrace looks different
@mdtauk could you share your debug.txt as well?
@tsvietOK is it working for you before and/or after #2648?

debug.txt
debug_fulltrust.txt

Aaand we have a third stacktrace different from the other two, great :/

Sorry @gave92

@gave92 Version 0.22.5
debug.txt

ok @tsvietOK your stacktrace is the one that (should) have been fixed in #2648

@gave92 Just tested in debug mode on master branch, seems like issue fixed.

Not sure what you did, but 0.23.0 is not crashing for me

@mdtauk glad to hear that!
@Tester09 could you test if the issue is solved for you in version 0.23?

@gave92  Unfortunately the app still crashing, even after re-install
debug.txt

@Tester09 This issue should be resolved wtih #2663.


Fail to restore cache

Sometimes it fails and leads to error in next stage (npm ci in my case).
2020-04-23T20:28:44.4658044Z Cache Size: ~15 MB (15859712 B)
2020-04-23T20:28:44.4688571Z [command]/bin/tar -xz -f /home/runner/work/_temp/024f94ae-a4be-402d-9086-1da569018142/cache.tgz -C /home/runner/.npm
2020-04-23T20:28:44.5725649Z
2020-04-23T20:28:44.5726691Z gzip: stdin: unexpected end of file
2020-04-23T20:28:44.5731247Z /bin/tar: Unexpected EOF in archive
2020-04-23T20:28:44.5731460Z /bin/tar: Unexpected EOF in archive
2020-04-23T20:28:44.5731750Z /bin/tar: Error is not recoverable: exiting now

Related build: https://github.com/mrdoob/three.js/runs/613235373?check_suite_focus=true
Similar issue that fixed in another action: actions/setup-node/issues/116
Similar issues here: #257 #267

Closing in favor of #267

Progress! Now we have a warning.
Cache Size: ~10 MB (10387456 B)
/bin/tar -xz -f /home/runner/work/_temp/ff46900f-412d-436a-8112-ed27d4437ae4/cache.tgz -C /home/runner/.npm

gzip: stdin: unexpected end of file
/bin/tar: Unexpected EOF in archive
/bin/tar: Unexpected EOF in archive
/bin/tar: Error is not recoverable: exiting now
[warning]Tar failed with error: The process '/bin/tar' failed with exit code 2


Remove icon from addAction

Description
In my case, when I'm trying to send a notification with android_actions payload, the app crashes because it tries to find the icon.
couldn't inflate view for notification com.myapp/0x2e37b245
                                            android.content.res.Resources$NotFoundException: Resource ID #0x1
                                                at android.content.res.Resources.getValue(Resources.java:2452)
                                                at android.content.res.Resources.getDrawable(Resources.java:1947)
                                                at android.content.Context.getDrawable(Context.java:409)

And then:
FATAL EXCEPTION: main
                                                                        Process: com.myapp, PID: 18384
                                                                        android.app.RemoteServiceException: Bad notification
posted from package com.myapp: Couldn't expand RemoteViews for:
StatusBarNotification(pkg=com.myapp user=UserHandle{0} id=775402053 tag=null score=20
key=0|com.myapp|775402053|null|10278: Notification(pri=2 contentView=com.myapp/0x109009e
vibrate=[0,300] sound=null defaults=0x0 flags=0x10 color=0xff00acd4 category=call actions=2
originalPackageName=N originalUserId=0 vis=PRIVATE))

As we have a line: notification.addAction(1, actionTitle, pendingActionIntent);
The 1 in this case points an unexisted path to icon (Resource ID #0x1).
Solution

Changed the icon argument to 0.

I was thinking to support icon here but google removed icon support for Android 7.
looks good


[bazel] Use ROM in SRAM program test instead of Test ROM

Prior to this commit, we relied on the PMP-configuration code of the Test ROM that enables writing and execution for the entire SRAM region. Switching to the ROM means that we cannot reuse that code, so we needed to figure out how to set up the PMP using OpenOCD commands. This proved very tricky due to implementation details of OpenTitan's debug module^0.
This commit also introduces a sequence of GDB commands that disable the watchdog timer to prevent the device from shutting down while we are debugging.
Fixes #13968

@dmcardle, can you also add Fixes #14486 to your commit message? Thanks!

This should be good to go now, PTAL @alphan!

@dmcardle can you bazel run //:buildifier_fix and push again? Thanks!

@dmcardle can you bazel run //:buildifier_fix and push again? Thanks!

Done!


renderInput missed in TypeScript definitions

no method renderInput in interface DatetimepickerProps

Same problem!
Can i make a pr, renderInput: (props: any, openCalendar: () => void) => JSX.Element; ?

@dominikeinkemmer You need to fork this repo, create a branch, push to that branch and then make a PR from that branch -> this repo. You get unauthorized error because you are trying to push to this branch, which you are not authorized to do. I hope this makes sense.

@simeg ah I see, thank you :)

@simeg @dominikeinkemmer  Is this fixed yet?

I needed this for my project, so I have submitted a pull request with exactly that.

I can't find type definition for this package. How do you install types for this ?

You don't need to install.  https://github.com/YouCanBookMe/react-datetime/blob/master/DateTime.d.ts

This is fixed in version 3. Sorry for the delay in fixing it!


Question about backend regex engine used in CredSweeper

Hi,
I have a question regarding the backend regex engine used in CredSweeper.
My understanding is that at the backend, re is used. But is this true?
If I run the following rule,
- name: some name
  severity: high
  type: pattern
  values:
    - (.*+)
  filter_type: GeneralPattern
  use_ml: false
  validations: []
  min_line_len: 1
  message: "some message..."

I get
2023-10-04 14:39:16,557 | INFO | __main__:276 | Run analyzer on path: ['secret.txt']
2023-10-04 14:39:16,623 | INFO | app:169 | Start Scanner
2023-10-04 14:39:16,623 | DEBUG | app:241 | Start scan file: secret.txt
2023-10-04 14:39:16,623 | DEBUG | scan_type:95 | Valid line for pattern: regex.Regex('(.*+)', flags=regex.V0) in file: secret.txt:1 in line: .
2023-10-04 14:39:16,623 | DEBUG | scan_type:61 | Filtered line with filter: ValuePatternCheck in file: secret.txt:1  in line: .
20

From the logs it seems like the regex module is used instead of re.

Hello @numairmansur ,
please, specify your version of CredSweeper. --version
It looks like obsolete code.

https://github.com/Samsung/CredSweeper/blob/d3bdb380334bfdbe87dbf4e6d86f012b0be97c9f/credsweeper/scanner/scan_type/scan_type.py#L95 - the line has no logger

Hello.
Actually we decided to use rather re  than regex on June. #350

I am using version 1.4.10
python -m credsweeper --version
CredSweeper 1.4.10

I see so on 1.4.10 we were still using regex and now we have switched to re?

Got it! Thanks so much!
Just out of curiosity: Why did you guys decide to switch to re? Overall it seems like regex offers more features compared to re.

Main disadvantage was sometime vulnerabilities in regex that requires urgent fix and so on.
Performance still the same and all necessary features were migrated to standard re (python3.8 - the minimal supported version)
So, the necessary of using regex has vanished, that why we have migrated to standard re.

Got it.
But I think there are still some features not supported in re. For example, variable-length lookbehind assertions.
>>> import re
>>> pattern = re.compile(r'(?<=a{2,4})b')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File<EMAIL_ADDRESS>line 227, in compile
    return _compile(pattern, flags)
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 294, in _compile
    p = _compiler.compile(pattern, flags)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 747, in compile
    code = _code(p, flags)
           ^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 580, in _code
    _compile(code, p.data, flags)
  File<EMAIL_ADDRESS>line 153, in _compile
    raise error("look-behind requires fixed-width pattern")
re.error: look-behind requires fixed-width pattern
>>>
>>>
>>>
>>> import regex
>>> pattern = regex.compile(r'(?<=a{2,4})b')
>>> match = pattern.search('aaab')
>>> bool(match)
True

@numairmansur
Could you share a rule in which you apply the feature of regex and does not supported with current version of CredSweeper?
Finds the rule a real credential or it is just for academic investigation?
You can even do a pull request. Let it fails in regex compilation - we will try to fix it together.

@babenek Yup this is only for academic investigation. I will circle back with a CredSweeper rule that maybe uses variable-length lookbehind assertions to find a credential! Thank you. I can close this issue for now and will re-open it when I have it.


Fixed broken internal links in docs for ash_phoenix

Contributor checklist

[ ] Bug fixes include regression tests
[ ] Features include unit/acceptance tests

Only changes to docs so I assume above checklist is not relevant.

So we actually have a system for docs that is admittedly strange but allows us to have intelligible docs on both github & hex & ash-hq.org. I'll make some PR suggestions to show what I mean.

Closing as this has been open for a while and is a bit out of date.


Add a quiet configuration option

Just to keep the bot a little more quiet in certain environments, such as IRC.

Definitely a useful option, I'll merge it if you're using this repo, but the official hubot-sentimental node module now points to this repo instead: https://github.com/hubot-scripts/hubot-sentimental
Would you mind sending a merge request into that repo?


create reviewer assignment algorithm

@ElizabethN do you think this issue is still valid to solve? we are looking at the bot deciding who to assign a review to depending on whether they are available and have no issue assigned to them at the moment.
my argument is that there could be applications that need particular people to work on them so may need manual assignment because the algorithm won't know how critical a particular review issue is. do you think we can neglect that and go on with the automation?


Bug in PHP syntax highlighting with comments

VSCode Version: Code 1.15.0 (8b95971d8cccd3afd86b35d4a0e098c189294ff2, 2017-08-09T20:09:15.006Z)
OS Version: Linux x64 4.8.0-58-generic
Extensions:

Extension
Author (truncated)
Version

xml
Dot
1.9.2

vscode-docker
Pet
0.0.16

gitlens
eam
4.3.3

php-debug
fel
1.11.1

vscode-auto-open-markdown-preview
hnw
0.0.4

mdhelper
jos
0.0.11

php-symbols
lin
2.1.0

Go
luk
0.6.63

dotenv
mik
1.0.0

prettify-json
moh
0.0.3

php-docblocker
nei
1.2.0

markdown-preview-enhanced
shd
0.2.5

With the latest update to VS Code we are running into issues with the code highlighting with comments.
Steps to Reproduce:

Insert the following code and set the syntax as PHP and you will see that code highlighting is not working:

/**
 * echos test
 * @return void(nothing to see here)
 */
function test() {
    echo 'test';
}

Now if you put as space between void and (nothing to see here), the code highlighting works.

I understand that this isn't really the correct way to do comments in docblocks, but this is a legacy codebase and it was working in the prior version. I don't think that these comments should stop it from highlighting the code anyways.

Reproduces without extensions: Yes

This should actually be fixed in Insiders

@roblourens I am not familiar with Insiders? Can you elaborate on what that means?

The Insiders version is updated nightly, and you can get it here: https://code.visualstudio.com/insiders
Fixes like this one are available there, and will be released in the next stable update early next month , 1.16.0.

Thanks @roblourens


Implement ability to fall back to a secondary Eth1 nodes if the primary node is unavailable .

Description
As a staker, I want to be able to fail-over automatically to a secondary/tertiary Eth1 node so that if the primary Eth1 node is unavailable or incorrect I am able to continue to propose blocks correctly.
Acceptance Criteria

Configuration option to list a sequence of Eth1 nodes
Nodes are to be tried in the specified order
If the node I am connected to is unresponsive or incorrect, a different node should be selected. Order of retries is TBD (Start from the top again? Try round-robin?)
Helpful log warnings are emitted to alert the operator to what is happening

Related: https://github.com/emeraldpay/dshackle

I don't seem to be able to assign this to him, but https://github.com/tbenr is looking into this.


Add StatefulSet to persist mongo data

Closes #18

I REALLY need to stop using github as the wrong user.

I REALLY need to stop using github as the wrong user.


API routes under api/websites/ returning 401.

Hi! First of all, thanks for the wonderful umami software!
Setup
Umami v1.39.5 running inside docker-compose, behind Nginx, with Postgres
I'm facing some issues with the dashboard. It is just showing me a blank page.

I've inspected the Chrome console and found that the API routes under the api/websites are returning 401 Unauthorized. Anything that I am missing?

The rest of the API routes are working fine. Real-Time is working fine as well.
There aren't any events. Just page views. Not a lot of data as well.
There are some errors when I do the docker logs <container name>. I've pasted them at https://pastebin.com/dxiSDGAb
Thanks!

Actually, it's now working again (and not sure why 🤔 ) But I am gonna close the issue!

I just experienced this too; in my case (at least) it was a result of refreshing an open page after upgrading the Umami version.  Logging out and in again solved it.


Locked node icon position is wrong when node is collapsed

Node isn't collapsed:

Node collapsed:

Fixed!


Deprecation warning when trying to make a contribution

Prerequisites:
Build identifier: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:43.0) Gecko/20100101 Firefox/43.0
Steps to reproduce:
1.Open an addon with contributions
2.Try to contribute
3.Observe the console
Expected results:
No warnings in console
Actual results:
"Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help http://xhr.spec.whatwg.org/" deprecation warning in console
Please see screencast for this bug: http://screencast.com/t/3PJBbrsH9x

Its not reducing the experience in this case because we are opening a window on the click event. Given that this will be part of the re-write, I think we shouldn't do anything with this.


Give Flow a helping hand

Flow didn't know which type to pick for addEventListener.

TBR... Letting Jenkins do its thing before landing.


missing Hardware Flow Control for UART

1. SDK version（SDK 版本）
{
V2.112
}

2. In what kind of operation problems appear, and how to reproduce the problem ?（什么样的操作步骤问题会出现，是否是稳定复现，如何复现问题？）
{
@Neutree There is no option for configuring Hardware Flow controls in UART_Config_t
suggestion :
typedef struct{
    UART_Baud_Rate_t      baudRate;
    UART_Data_Bits_t      dataBits;
    UART_Stop_Bits_t      stopBits;
    UART_Parity_t         parity;
    UART_Callback_t       rxCallback;
    UART_Error_Callback_t errorCallback;

    bool                  hwfc_enable;

    bool                  useEvent;
}UART_Config_t;

}

Ping @Neutree. It's been 65 days with no replies...

reopened to get attention!


Change how set_type is set in ChipDataset()

Previously it was set with an integer from the SetType enum. It is now
set with the enum itself, or a string describing the set ("train",
"test", etc.). None is now the default value in the constructor, which
defaults to SetType.ALL. All checks have been changed to assume that
an acutal SetType object is stored in the class.
This still needs some tests in insure we don't break it again...

Codecov Report

Merging #53 into master will decrease coverage by 0.25%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
- Coverage   93.03%   92.77%   -0.26%
==========================================
  Files          19       19
  Lines        1191     1204      +13
  Branches      172      178       +6
==========================================
+ Hits         1108     1117       +9
- Misses         67       69       +2
- Partials       16       18       +2

Impacted Files
Coverage Δ

pelops/datasets/dgcars.py
100% <100%> (ø)
:white_check_mark:

pelops/datasets/veri.py
88.23% <100%> (ø)
:white_check_mark:

pelops/datasets/compcar.py
100% <100%> (ø)
:white_check_mark:

pelops/utils.py
53.24% <100%> (ø)
:white_check_mark:

pelops/datasets/str.py
100% <100%> (ø)
:white_check_mark:

pelops/datasets/chip.py
80.88% <75%> (-2.76%)
:x:

Continue to review full report at Codecov.

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

@agude As soon as the circleci tests finish I think I'm good with merging


Optimize size of processed ruleset in common scenarios.

Optimize size of processed ruleset in common scenarios.
StyleSetEvalator pre-filters and cached styleSet for each (layer, geometryType)
tuple, so in usual scenarios we search very short rulesets (1-2 for buildings, 2-4 for pois).
Extra, Profile helper added

@heremaps-bot Decoders test please

Decoders tests please

Decoders tests please

Decoders tests please

Decoders tests please

Will be delivered as part of #853


chore(travis): cleanup the tests on Travis.

Remove OSX tests for Node 4, remove tests for Node 5. Other cleanup tasks.

@jkuri I also added Node 7. It was easy enough.

LGTM

Thanks @jkuri


Is there a way to run the complete project with single infer command

I want to run a scan on all the java files in my root directory. Is there a way to run it in single command something like infer -- javac

It will run for entire project only. try with your build variant.
infer --reactive -- ./gradlew

for a java project. Not for the android project

What is the current command you are using?

infer -- javac /file.java
I want to know a command like infer -- javac

Add this in your command.
-pr "specific module/package/project path for infer to run".
-pr is project root. But your terminal path should be in android root only. Because all supporting files(python files) which helps to run infer will be available in android root. But you can specify your module for infer to run using -pr

thank you


README.md seems "ubuntu-centric"...  does grpc install on Centos?

Please answer these questions before submitting your issue.
Should this be an issue in the gRPC issue tracker?
Create new issues for bugs and feature requests. An issue needs to be actionable. General gRPC discussions and usage questions belong to:

grpc.io mailing list
StackOverflow, with grpc tag

Please don't double post your questions in more locations; we are monitoring both channels, and the time spent de-duplicating questions is better spent answering more user questions.
What version of gRPC and what language are you using?
I'm happy to use any stable version of gRPC, and I will be using C++.
What operating system (Linux, Windows, …) and version?
I'm happy to use any stable version of Centos Linux.
What runtime / compiler are you using (e.g. python version or version of gcc)
I can install whichever gcc you recommend.
What did you do?
I installed grpc on Ubuntu, and it runs great.
But we also need it to run on Centos, and your README.md doesn't seem to have any section on how to install on Centos.
That is, I see no "yum install" but rather "apt-get install".
Before I dive into this, I just would like some reassurance that it "should work."?

My bad, I meant INSTALL.md, not README.md....


[hotfix][docs] Move out the time zone page from streaming concepts section

What is the purpose of the change

This pull request move out the time zone page from streaming concepts section

Brief change log

move out the timezone page as it is not streaming specific

Verifying this change
This change is a doc hotfix without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

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

Appreciate if you can help merge @sjwiesman , I don't have merge authority yet.


feat: exam token UI

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

blocked on: #55662

@Sembauke Things to do here:

[x] Only generate tokens

Make clear to Campers doing so invalidates old tokens

[ ] Put this behind a feature flag

We do this in some other places in the codebase using growthbook, and have some of it documented in the staff wiki

In a following PR:

Handle errors
Improve UI with things like a modal showing the token, allowing easy copy, and explaining the token will not be shown again

Put this behind a feature flag

Do we really need a feature flag? I removed it from this pr.

Do we really need a feature flag?

Yes, the endpoints are not live yet. So, we cannot have the client show the UI.

I thought about deleting setGeneratedExamToken as well. I don't know why I didn't


Updates PF to fix a regression

Minor optimization on ConditionField

Codecov Report

Merging #218 (654803b) into master (0fa89bb) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #218   +/-   ##
=======================================
  Coverage   91.03%   91.04%
=======================================
  Files         116      116
  Lines        2511     2512    +1
  Branches      470      470
=======================================
+ Hits         2286     2287    +1
  Misses        223      223
  Partials        2        2

Impacted Files
Coverage Δ

src/components/Condition/ConditionField.tsx
90.90% <100.00%> (+0.16%)
:arrow_up:


Typography Control doesn't trigger Save button for the second time

Hi @aristath,
Issue description:

The issues is the typography field works for the first time. After selecting font and save for the first time then again changing fonts for the second time doesn't change "Save" button to "Save & Publish". If the we have to change the different for the second time we have to refresh customizer to save it. Can you please look at this.

Version used:

2.2.4

Using theme_mods or options?

theme_mods

Code to reproduce the issue (config + field(s))

Kirki::add_field( 'my_config', array(
'type'        => 'typography',
'settings'    => 'h1_font',
'label'       => __( 'H1 Font' ),
'section'     => 'typography_section',
'default'     => array(
'font-family'    => 'Source Sans Pro',
'variant'        => 'regular',
),
) );

Thanks

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/32088937-typography-control-doesn-t-trigger-save-button-for-the-second-time?utm_campaign=plugin&utm_content=tracker%2F2030945&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2030945&utm_medium=issues&utm_source=github).

Hey there!
Yes, that's a known bug. I've tried to resolve it but so far I've been unable to find a solution.
If you want to contribute I've posted a bounty on https://www.bountysource.com/issues/31575403-typography-control-does-not-apply-changes-after-save?utm_campaign=plugin&utm_content=tracker%2F2030945&utm_medium=issues&utm_source=github
If you add anything to that bounty hopefully someone will be able help us fix it.
Closing as a duplicate of https://github.com/aristath/kirki/issues/717


Adding kosher token

Add Kosher token

I can help stop wasting your time refer this zakimzf/profile#1

close for logo url fix


updated gnome-extension target -y flags, caffeine-plus to readme

Addresses https://github.com/iancleary/install-scripts/issues/34. I didn't go makefile route as the relative path is better in bash from a homedir.
This probably could be solved with yadm...


Setup basic GitHub Actions CI

Currently failing due to https://github.com/microsoft/sarif-js-sdk/issues/5 (moving to draft while we fix that).

What the consideration between using macos vs ubuntu as the representative for UNIX?

FWIW - CI is failing until we fix #5 (which @scalvert has in flight over in #6)

What is the consideration between using macos vs ubuntu as the representative for UNIX?
I presume we have more macos usage than other UNIX?

Yes, you are totally correct - I would absolutely expect us to have more macOS users than Ubuntu users. Unfortunately, the macOS CI jobs are a bit finicky (slow to start, less overall capacity, transient failures, etc) in other repos whereas the Ubuntu ones seem to be pretty rock solid.
Additionally (I think I mentioned this inline in another comment), we don't really know if we will have platform specific code ATM. Even if we eventually do have a number of platform specific things, it is very likely to be the same between macOS and Ubuntu (essentially Windows is the one where most platform specific difference are super obvious).

finicky

Thanks! That's valuable insight. I've been using macos on very small scales so never noticed.  :-)

@jeffersonking agree with what @rwjblue is saying, and that's been my experience too. The macos jobs would quite frequently fail for random reasons, which of course can be quite annoying in terms of test stability. Having to repeatedly re-run the jobs not only ended up being inconvenient, but it also degraded the confidence level of the stability of the test suites.

@rwjblue needs a rebase with main.

Probably good to mark this ready for review too, after the rebase.


Ruby enumerable methods

Greetings, everyone! This project is about rebuilding the Ruby enumerable methods, for this goal I used other methods, mostly iterators and conditionals like if or unless and some methods that were declared before in this same file. Through the code lines, you will find comments that I hope give a comprehensive description of it's use.
I Can not wait to read and follow your observations, which I really believe will allow a great improved final version.
Excellent day to you!

I saw that the rubocop linter asked me to make changes, but when doing them, the same rubicop marked those changes as an as errors, for this reason, I disabled some cops like Style/SymbolProc
I'm at your disposal to clarify the details of this action.


Why source and dest must be int?

Hello,
I was wondering why do we need to consider source and dest are int and not strings. Indeed, it would make more sense (to me) because usually, source and dest are IP addresses.
Thanks

Hi,
You are absolute right. Source and destination are usually IP addresses. We just map those strings to integer values to process them and have a fair comparison with the baseline. Our method can be extended to take input as strings as well.
Thanks.


Add reverse scalar string function.

Problem Statement
As per https://www.postgresql.org/docs/current/functions-string.html
reverse(text) -> text
Example:
reverse("abcde") -> "edcba"
reverse("rotator") -> "rotator"
This improves PostgreSQL compatibility and in my use case, Tableau.
Possible Solutions
No response
Considered Alternatives
No response

Workaround:
CREATE FUNCTION reverse(str TEXT)
RETURNS TEXT
LANGUAGE JAVASCRIPT
AS 'function reverse(str) {return str.split("").reverse().join("");}';


Fix BitmapCollection::orCardinality API when combining a non-inverted bitmap with an inverted one

Fixes https://github.com/apache/pinot/issues/14486.
The existing API incorrectly calculates n(b1 ∪ b2') as numDocs - n(b2) - n(b1 ∩ b2). It should instead be numDocs - n(b2) + n(b1 ∩ b2). The following Venn diagram clearly demonstrates the issue:

There was a unit test in BitmapCollectionTest that covered this case but the unit test itself was also buggy unfortunately. It combines a non-inverted bitmap [0, 5] with an inverted bitmap [0, 4] (i.e., bitmap [1, 2, 3, 5, 6, 7, 8, 9]) which should result in bitmap [0, 1, 2, 3, 5, 6, 7, 8, 9] - i.e., cardinality of 9 (but the test's expectation is 7).
The existing logic for n(b1' ∪ b2) is correct however, and the existing unit test combining the inverted bitmap [0, 5] with a non-inverted bitmap [0, 4] does have an expected cardinality of 9.
This patch also adds a small integration test with the original generated query that caught this issue in a CI run.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (59551e4) to head (c16d5c5).
Report is 1351 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (c16d5c5). Click for more details.
HEAD has 53 uploads less than BASE

Flag
BASE (59551e4)
HEAD (c16d5c5)

integration
7
1

integration2
3
1

temurin
12
1

java-21
7
1

skip-bytebuffers-true
3
1

skip-bytebuffers-false
7
0

unittests
5
0

unittests1
2
0

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #14487       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436        3     -2433
  Lines        133233        6   -133227
  Branches      20636        0    -20636
=============================================
- Hits          82274        0    -82274
+ Misses        44911        6    -44905
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
0.00% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
?

skip-bytebuffers-true
0.00% <ø> (-27.73%)
:arrow_down:

temurin
0.00% <ø> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles


Record syntax error

Description:
For some reason, the following doesn't work:
{x: {}}

Steps to reproduce:
https://www.squiggle-language.com/playground#code=eNqrVirOyC8PLs3NTSyqVLIqKSpN1QELuaZkluQXwUQy8zJLMhNzggtLM9PTc1KDS4oy89KVrJSqK6wUqmtrlWoB2%2FAayg%3D%3D
Expected behavior:
This should create a record with an x key empty record inside of it
What I got instead:

Duplicate of #1059


Redundant closing braces after expansion of a snippet

When I was trying to use the snippets in CPP, it will insert redundant closing braces for some snippets. Like main fori, after the expansion, the result will go like below
int main(#cursor#)int argc, char *argv[])
{

    return 0;
}

Thouth it works fine for for.

This seems to be a very basic function, shouldn' there be anyone else who has noticed that as well, or just me???

What you mean redundant? Seems this is a problem of yasnippet, not oh-my-emacs

You see there is one extra closing bracket inputted after the cursor, and lead to [yas] Snippet 1 exited..
I think that the closing bracket is generated by some others. But being a new Emacs user, I'm not sure.

I tried but this bug didn't reproduced on my ome, maybe you can turn off smartparens-mode and try again.

It worked, thanks!

@Plypy consider closing this issue?


Even in eager mode, Keras is passing custom models non-eager tensors in 'fit'

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): 10.14
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): '1.14.1-dev20190301'
Python version: 3.7

Describe the current behavior
When calling keras.model.fit on a custom model, it seems the model is passed a graph-mode tensor instead of an eager tenser, even when in eager mode.
Describe the expected behavior
If in eager mode, the tensors passed to the call method of a custom model should be eager tensors. Otherwise, the advantages of eager mode, like the ability to use native control flow, are lost.
Code to reproduce the issue
import tensorflow as tf
tf.enable_v2_behavior()
from tensorflow import keras

class MyModel(keras.Model):
    def call(self, x):
        if x > 0:
            return x + 1
        else:
            return x - 1

m = MyModel()
m(tf.constant(0))  # This works, returns -1 as expected
m.compile(loss='mse', optimizer='sgd')
m.fit(tf.constant(0), tf.constant(1)) # This fails

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-55-2c7beacc3d4f> in <module>
----> 1 m.fit(tf.constant(0), tf.constant(1))

/anaconda3/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, validation_freq, max_queue_size, workers, use_multiprocessing, **kwargs)
    804         steps=steps_per_epoch,
    805         validation_split=validation_split,
--> 806         shuffle=shuffle)
    807
    808     # Prepare validation data.

/anaconda3/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py in _standardize_user_data(self, x, y, sample_weight, class_weight, batch_size, check_steps, steps_name, steps, validation_split, shuffle, extract_tensors_from_dataset)
   2501       else:
   2502         cast_inputs = x_input
-> 2503       self._set_inputs(cast_inputs)
   2504     else:
   2505       y_input = y

/anaconda3/lib/python3.7/site-packages/tensorflow/python/training/tracking/base.py in _method_wrapper(self, *args, **kwargs)
    454     self._setattr_tracking = False  # pylint: disable=protected-access
    455     try:
--> 456       result = method(self, *args, **kwargs)
    457     finally:
    458       self._setattr_tracking = previous_value  # pylint: disable=protected-access

/anaconda3/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py in _set_inputs(self, inputs, outputs, training)
   2773             outputs = self.call(inputs, training=training)
   2774           else:
-> 2775             outputs = self.call(inputs)
   2776           # Reset to the previously saved value. If `call()` had `add_metric`
   2777           # or `add_loss`, then `_contains_symbolic_tensors` will have been set

<ipython-input-52-94b7a4f52815> in call(self, x)
      1 class MyModel(keras.Model):
      2     def call(self, x):
----> 3         if x > 0:
      4             return x+1
      5         else:

/anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/ops.py in __bool__(self)
    658       `TypeError`.
    659     """
--> 660     raise TypeError("Using a `tf.Tensor` as a Python `bool` is not allowed. "
    661                     "Use `if t is not None:` instead of `if t:` to test if a "
    662                     "tensor is defined, and use TensorFlow ops such as "

TypeError: Using a `tf.Tensor` as a Python `bool` is not allowed. Use `if t is not None:` instead of `if t:` to test if a tensor is defined, and use TensorFlow ops such as tf.cond to execute subgraphs conditioned on the value of a tensor.

Note that even adding run_eagerly=True to the compile call doesn't resolve this.

I believe you need to pass dynamic=True to the model constructor:
m = MyModel(dynamic=True)

Otherwise Keras assumes that your model is able to take symbolic tensors (and uses this for shape inference) even if you've asked it to compile in eager mode. This was added in https://github.com/tensorflow/tensorflow/commit/d0373dcae4810422f7575cb56a91d6695d2c4ec6, which has a little more context.

There is a bug here, though, in that Keras should probably tell you why this is failing.

Thanks, that does the trick!
There also might be a bit a documentation issue here - I don't recall this being prominently highlighted in the new tutorials and overviews of eager mode/keras.

Thank you @jekbradbury for the quick response on this! We will definitely need to throw an error explaining the issue here. I will look into that.
@malmaud I agree that this is not documented well. This concept is very new and we are working to see if we can remove the need for this hence the lack of documentation.

How can I add dynamic=True or compile in eager mode keras.Sequential?

Can we get some more explanation for how to add dynamic=True and what that does?

I found a solution that works for me and posted it in this StackOverflow answer.
tl;dr set the run_eagerly attribute manually after compiling your model:
model.compile()
model.run_eagerly = True

@brett-daley It works! Thank you.

I am having the exact same problem with keras version 2.3.0-tf and tensorflow version 2.2.0. Adding dynamic=True gives me this error for a functional model:
TypeError: ('Functional models may only specify 'name' and 'trainable' keyword arguments during initialization. Got an unexpected argument:', 'dynamic')

I was able to fix my issue by subclassing tf.Keras.Model. Also, there was no need to pass dynamic=True to the model constructor; run_eagerly=True when compiling works.

@surGeonGG Can you be more specific please? I assume you did more than just simply subclassing because propagating the init via a super-call shouldn't have any effect.

I am having the same issue when running this on 2.3.0. Doing this also doesnt resolve the issue
model.compile()
model.run_eagerly = True

Same with PrattJena, running on 2.3.0 and have same issue even with run_eagerly = True

I also have this same issue. Any pointers towards a solution?

I also have this same issue. Any pointers towards a solution?

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.

Hi Alfred,
This issue has been resolved. Thanks so much for checking in though.
On Fri, Feb 19, 2021 at 7:10 PM Alfred Sorten Wolf<EMAIL_ADDRESS>wrote:

Hi There,
We are checking to see if you still need help on this, as you are using an
older version of tensorflow which is officially considered end of life . We
recommend that you upgrade to the latest 2.x version and let us know if the
issue still persists in newer versions. Please open a new issue for any
help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need
help with this issue, please provide us with more information.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/26268#issuecomment-782282428,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKEAU4WCOELDST4GRUNK6FDS72ZTZANCNFSM4G3H55ZQ
.

Closing this issue since it is resolved. Feel free to reopen if necessary. Thanks!

This isn't solved.  I was able to reproduce the error even after passing the suggested line after the compile() function is called.


Add zooming and panning to preview 2d

Hmm looks like this one still needs a lot of work:

the behaviour should be similar to the Reference panel (zoom around cursor, pan with middle mouse button as if you just grab and move the image)
didn't test, but looking at the code, this breaks export and parameter controls
there are other related requests, maybe this PR is a good excuse for them

Hmm I look into that. Good to know that reference panel has similar functionality. I don't use it since loading image is broken (at least on linux) maybe I should also look at this :) I exported it to "unreal" and looked fine so I am not sure about second point.  To be honest I was just fed up with lack of zoom on 2d nodes, so I implemented it for myself. It's really basic implementation and can be improved.

The export function that this patch breaks is the one in the 2D preview's context menu.

I made few changes, it should zoom to cursor right now and pan with middle button. (I think there is a bug in reference panel - "multiplier" variable is calculated incorrectly and it doesn't work on stretched panels) I also fixed export in 2d preview's context menu.

Hmm more info about the multiplier variable bug? I don't understand.

Ok so here is a problem on the video: https://streamable.com/5031j1 I think that the variable "multiplier" is incorrect but It's only a guess. I think there is a problem with conversion from "Panel space" to Shader UV space. I didn't look further into it.

All this should be solved in master now (fixed it on another branch while investigating a bug).
@wojtekpil , thanks for helping, and please don't hesitate to tell me if master behaves how you expected.

All this should be solved in master now (fixed it on another branch while investigating a bug).
@wojtekpil , thanks for helping, and please don't hesitate to tell me if master behaves how you expected.

Hi, yes it seems to be working correctly now. It seems that, this merge request is no longer required.

Hi, yes it seems to be working correctly now. It seems that, this merge request is no longer required.


Array type is incompatible with string when using rest with union type

I have a function that can take an array or strings as arguments:
type Args = Array<string | Array<string>>;

function myFn( ...args: Args ) {
  // use Array.isArray to check first member of arg etc.
}

myFn( [ 'foo', 'bar' ] );
myFn( 'foo', 'bar' );

This fails with:
  7: type Args = string | Array<string>;
                          ^^^^^^^^^^^^^ array type. This type is incompatible with
  7: type Args = string | Array<string>;
                                ^^^^^^ string

Am I annotating this incorrectly? (Flow 0.24)

flow version 0.24.1 will warn you that:

Correct code:
function myFn( arg: string | Array<string>, ...args: Array<string> ) {
}

This will allow myFn(['str'], 'str') though

Array<[string, mixed]>;
static freeze(o: T): T;
static getOwnPropertyDescriptor(o: any, p: any): any;
static getOwnPropertyNames(o: any): Array;
static getOwnPropertySymbols(o: any): Symbol[];
static getPrototypeOf: Object$GetPrototypeOf;
static is(a: any, b: any): boolean;
static isExtensible(o: any): boolean;
static isFrozen(o: any): boolean;
static isSealed(o: any): boolean;
static keys(o: any): Array;
static preventExtensions(o: any): any;
static seal(o: any): any;
static setPrototypeOf(o: any, proto: ?Objectyne

Couldn't repro with 0.26.0.
function myFn( ...args: Array<string | Array<string>> ) {
  // use Array.isArray to check first member of arg etc.
}

myFn( [ 'foo', 'bar' ] );
myFn( 'foo', 'bar' );

gives Found 0 errors.

Sorry, I left out the full example, which points to Array.isArray and .reduce as culprits:
function myFn( ...args: Array<Object | Array<Object>> ) {
  const stores = Array.isArray( args[0] ) ? args[0] : args;
  const foo = stores.reduce( ( memo, store ) => {
    memo[ store.key ] = store;
    return memo;
  }, {} );
  console.log( foo );
}

myFn( [ { key: 'foo' }, { key: 'bar' } ] );
myFn( { key: 'foo' }, { key: 'bar' } );

Fails with:
  4: function myFn( ...args: Array<Object | Array<Object>> ) {
                                            ^^^^^^^^^^^^^ array type. This type is incompatible with
  4: function myFn( ...args: Array<Object | Array<Object>> ) {
                                                  ^^^^^^ object type

Shouldn't flow recognize store as Array<Object>? It fails even if I type stores as such.

cc @avikchaudhuri

This is not a refinement that will be supported by Flow.


"mount_base_64_uploader" method not recognized from rails console

Hello!
I'm running into a strange thing currently, which I believe is in someway inherent to this gem, though it it possible that something is afoot in my rails environment...
The gem works flawlessly in tests, and I am able to access models with the mount_base_64_uploader on it when program execution is paused in debug statements.
However, when enter the rails console and I try to access an activerecord model which contains that uploader, it throws the following error. (the model in my case is ClientPhoto).

Any insights as to why this may be happening?

Hello!
Could you try to load ActiveRecord first? (just call ActiveRecord) in the console before calling ClientPhoto.
I suppose it has something to do with the Railtie - https://github.com/lebedev-yury/carrierwave-base64/blob/master/lib/carrierwave/base64/railtie.rb#L4

Just tried it out, got the same error message : /
Looks like a good guess as to where it might be occurring, possibly I will snoop around this a bit if I get some free time.
Thanks

@JackCurrie any updates on the issue? I will close it, if you still have the problem, feel free to reopen it


The output is centered at (0, 0, 0)

The input character moves to the left, and the model outputs the action to lock the hips, but does not move to the left. How to solve it?

Hi @liuhaorandezhanghao,
As many works, TCMR produces a mesh centered at the root joint location per frame.
If you want to translate the mesh, you should obtain the absolute 3D coordinate of the root joint.
It is a little bit different research area, and you can checkout this repo: https://github.com/mks0601/3DMPPE_ROOTNET_RELEASE


fix: implement processMap function to MAP structured data

Hi @jcustenborder,
Maybe you saw If you are using HeaderToField as transforms and your data struct is Map, the plugin is throwing error like "MAP is unsupported ..." I implemented processMap function and I tried to use it and It works well. Maybe you want to merge it as contribution

@jcustenborder Actually no but I can add some unit tests for the function. I'll update the PR quickly

@jcustenborder I added test for processMap Is that okay for that?

@jcustenborder Hey can you check the PR?

@peacecwz
Could you fix same issue on the ChangeCase SMT..?
(If not.. I will try to implement processMap method on the ChangeCase, by referring to your commit.)

@peacecwz see https://github.com/peacecwz/kafka-connect-transform-common/pull/1


[3.0] opcode,ast: fix invalid restore of '4 DIV 2' (became '4DIV2') (#792)

What problem does this PR solve?
Cherry-pick #792 to release-3.0
What is changed and how it works?
Check List
Tests

Unit test

Code changes

Has exported variable/fields change

Side effects
Related changes

That's weird.
----------------------------------------------------------------------
FAIL: builtin_time_test.go:1143: testEvaluatorSuite.TestFromUnixTime

builtin_time_test.go:1190:
    c.Assert(ans.String(), Equals, t.expect, Commentf("%+v", t))
... obtained string = "2016-01-01 00:00:00"
... expected string = "2016-01-01 08:00:00"
... {isDecimal:false integralPart:1451606400 fractionalPart:0 decimal:0 format: expect:2016-01-01 08:00:00}

----------------------------------------------------------------------

Is our CI broken on the release-3.0 branch?

i think so. probably some timezone being ignored


Settle for an acronym to use in the codebase (PN, NM, ...)

Hi @timonmerk, I hope this discussion does not sound to dumb but in the recent refactoring I renamed the class Feature to NMFeature to make it less generic, NM standing for NeuroModulation.
But now I realized that the base class for the Stream class is called PNStream, PN for PyNeuromodulation.
If we're going to keep on going with this naming practice, we should choose an acronym to use in all cases, whether it's NM, PN, PNM or PyNM (this one is too long maybe).
Or maybe we ditch the acronym altogether and just go for BaseFeature, or FeatureProtocol, BaseStream or sth like that.

Thanks a lot for raising this point! We should definitely keep the naming consistent throughout module names but also example imports, e.g. import py_neuromodulation as nm
We should also keep the acronym nm, that's across the features and preprocessing points established, so it would be good to keep it throughout the package.

I hope it's ok that I assigned you here @toni-neurosc, it could be part of #305 or even separate PR

Sure, np, I checked and there wasn't much to change.

I change PNStream to NMStream
I change import py_neuromodulation as pn to nm in "test_multiprocessing.py"

In PR #305 all imports were changed to refer to the specific modules, and I think internally we should keep it this way so it's at all times clear at first glance what the inter-dependencies between modules are.
But when writing tests or demos/examples, then it should always read
import py_neuromodulation as nm

So that would be about it for now, I guess we're all on the same page about this now?
I'll maybe wait for  @SamedVossberg to confirm and then close it afterwards.

Alright, I will stick with the nm notation in the tests and examples as well.
I did get notifications. Don’t know why I don’t show up in your suggestions @toni-neurosc though🤔


Added newsAPI dataset with sqlite

Description
Added the documentation that was needed for assignment 2 including the teams feedback, interview questions, and peer evaluations, of course done individually. Included are also the slides for assignment 2. And Nami's code
Type of change

[X] New Feature
[X] Documentation

How has this been tested?
Documentation was looked over by the team and given the OK. Nami's code i have no idea what it does ahaha
Checklist

[X] My code follows the style and contributing guidelines of this repo
[X] My code has been tested and will not break anything that is existing

I will say she does have an API key in that notebook so just fyi on that


Add testing NS for pipeline-service

The testing NS and its related resources are used to run pipeline-service smoke tests.

/retest

/lgtm

/retest


Using rez env interactively in windows

What is the preferred way of creating interactive rez environments?
I have several rez packages built and rez env is resolving them, but it doesn't seem to be applying them to the current shell (windows cmd).
For example, I'm using rez env to build a context with the requests python library
rez env requests

Rez appears to find and resolve the package, but it's not updating the PYTHONPATH in my shell.
If I run the following command, it will launch a child shell and the packages are correctly applied.  Is this how it is supposed to work -- that you have to launch a child process to apply the package environment?
rez env requests -c CMD

Hey,
rez-env requests is supposed to put you into an interactive subshell,
configured for that environment. When you're in the new shell, the prompt
is updated so that something like ']$' becomes '>]$' (note prepended
chevron).
There must be some problem in your scenario - some issue with the shell
startup. I'm not familiar with how this should work on Windows so I'm
hoping others will chime in.
Thx
A
On Thu, Jan 12, 2017 at 2:15 PM, Brendan Abel<EMAIL_ADDRESS>wrote:

What is the preferred way of creating interactive rez environments?
I have several rez packages built and rez env is resolving them, but it
doesn't seem to be applying them to the current shell (windows cmd).
For example, I'm using rez env to build a context with the requests
python library
rez env requests
Rez appears to find and resolve the package, but it's not updating the
PYTHONPATH in my shell.
If I run the following command, it will launch a child shell and the
packages are correctly applied. Is this how it is supposed to work -- that
you have to launch a child process to apply the package environment?
rez env requests -c CMD
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/375, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABjqSsrCkl6O3xgozNAchl6X2ODQxQvZks5rRZrRgaJpZM4LhU8G
.

Can you share some more information like the code of the package.py file to investigate?
Cheers,
Thorsten

I took another stab at getting rez to work on windows today.  I have this same issue with the hello_world example.  In windows cmd.exe and in git-bash rez-env does not put the environment in a sub-shell, and therefore it does not affect $PATH.

Out of curiosity, what happens if you do "rez-env PACKAGES -- cmd.exe"?
On Wed, Feb 8, 2017 at 3:40 PM Jordan Hueckstaedt<EMAIL_ADDRESS>wrote:

I took another stab at getting rez to work on windows today. I have this
same issue with the hello_world example. In windows cmd.exe and in git-bash
rez-env does not put the environment in a sub-shell, and therefore it
does not affect $PATH.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/375#issuecomment-278499482, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEYePcXq0fbSYIleR4ysfWm1BsVdNu_ks5ralKHgaJpZM4LhU8G
.

@elrond79 Yes, that will drop you into a shell, but that is different from the behavior in other shells.  In other shells, not specifying a command will give you an interactive shell.  The current behavior (without the PR I submitted) when using rez-env in a windows CMD shell is to spawn a shell with the correct rez enviroment, but then immediately exit.

There are different PRs doing similar things or tackling it slightly different. We definitely need to clean up the windows side of rez and rework the cmd plugin a bit.

Fixed with #377


Suspicious flash setup on teensy40/teensy41 boards

Describe the bug
Hi, had a chance of checking out the teensy40/teensy41 flash configuration recently due to a build breakage (https://github.com/zephyrproject-rtos/zephyr/pull/55341). That was an obvious misconfiguration and has been fixed, but there's still a big question mark on the flash node configuration:
https://github.com/zephyrproject-rtos/zephyr/blob/29cc07efbd71b779d584a30fb495b4a3de6e0e91/boards/arm/teensy4/teensy40.dts#L38-L46
https://github.com/zephyrproject-rtos/zephyr/blob/29cc07efbd71b779d584a30fb495b4a3de6e0e91/boards/arm/teensy4/teensy41.dts#L11-L19
Specifically, compared to other boards based on nxp/nxp_rt1060.dtsi (https://github.com/zephyrproject-rtos/zephyr/blob/main/boards/arm/mimxrt1060_evk/mimxrt1060_evk.dts#L113), it seems like the flash size should match the size of the second reg of the parent node (0x200000 and 0x800000, think that's used by the build system), and the flash node should be using nxp,imx-flexspi-nor to enable the actual flash driver allowing the flash to be read or written in runtime.
The size numbers looks incorrect anyway, size for flash devices is defined as:
  size:
    type: int
    description: flash capacity in bits

so 8388607 does now quite compute.
I don't own one of this board, @dleach02 do you happen to have one for testing?
To Reproduce
west build -p -b teensy41 samples/drivers/flash_shell, try to use the flash
Expected behavior
Flash device should work in runtime.
Impact
Right now I think this breaks any application trying to use the flash in runtime on that board (mcuboot, settings...)
Logs and console output
n/a
Environment (please complete the following information):
n/a

Fixed by https://github.com/zephyrproject-rtos/zephyr/pull/56616#event-8952713427


Make owntracks.py pylint clean

Description:

Make owntracks.py pylint clean
refactor validation logic into validate_payload function
add debugging messages to validation

Checklist:
If code communicates with devices:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Looks good! :dolphin:


Fix GitHub Actions check query files

The latest Ubuntu doesn't come with libfuse2, so the appimage wouldn't run without this dependency as shown below.
This pull request fixes this issue and re-enables checking query files Action.
Problem:

Fix:

Thank you! Just wanted to do the same!


"Server cannot be reached" state causes high CPU load

When the server can't be reached the Playground causes very high CPU load – is this caused by the (SVG) animation? If so, how could this be implemented more efficiently?

Yeah, looks like graphql-playground is just endlessly re-rendering itself even when there's no interaction. Here's a Chrome performance report for my graphql-yoga UI instance:

I have same problem

Thanks for reporting!
Because of this we recently disabled the animation for this screen.
This should fix the performance issue for now.

@timsuchanek I'm using the latest 1.6.2 and it's still a problem. Or is this fix not included in 1.6.2?

Running 1.6.2 and it stills an issue. It makes my MacBook Pro (2015) really really slow! I don't know why this issue was closed. @timkendall any update?


Londonbusstop cannot find stops that are already listed in the app

@dinosaursrarr Trying to use the app, and put my location in settings. the app then finds 4 bus stops for me to select. Regardless of which one I select, I have error Unknown Stop
log: [app-id] TFL StopPoint search result does not contain stop code.

Thanks for flagging this, I wasn't aware.
I suspect something has changed with the API format. It's possibly related to the recent TfL cybersecurity incident, but probably not.
I'll look into it this week.


Update GTAVWebhookScript.cs

How is this  Failed to execute command spawn_attackers_and_shoot. Error: The type initializer for 'SHVDN.NativeMemory' threw an exception.


Render inlineCode and code

is there an option to separate code and inlineCode in the components?
Here's my code:
const components = {   h1: ({ children }) => {     return <Heading as="h1">{children}</Heading>;   }   code: ({ children }) => {     return
{children}
;   },   inlineCode: ({ children }) => {     return
{children}
;   } };
  const content = useHydrate(doc, {     components: components,   });

This doesn't work.. it only render the code for both: example and example

Hmm I can't reproduce this. See https://6ezyv.sse.codesandbox.io/blog/hello-world
Can you take a look at this codesandbox: https://codesandbox.io/s/immutable-cookies-6ezyv?file=/pages/blog/[...slug].js


[2011] Remove rubocop govuk

Context
rubocop-govuk has been updated to v4 and required a lot of fixes. We can't pin to the current version as there is a dependency that is no longer available. This made us think that relying on another external gem for our rubocop config is probably unnecessary.
We've also somehow managed to commit the gem update into master so our linting is broken (as is our CI seemingly but we'll look at that separately).
So... we decided to remove it. Turns out this requires a lot of fixes too but it's a noble cause and we now have control of our own destiny as far as rubocop is concerned.
Changes proposed in this pull request

remove rubocop-govuk
add rubocop-rails
add rubocop-rspec
refactor the config (a la https://github.com/DFE-Digital/apply-for-teacher-training/pull/4934 thanks @despo 👏 )
generate (a quite frankly massive) .rubocop_todo.yml
remove things from said rubocop_todo.yml and fix stuff until I lost the will to live. I'll continue in some subsequent PRs when I get the feeling back in my fingers.

Guidance to review
I've only addressed pretty non-controversial auto-correctable things thus far but best reviewed commit by commit as I've tried to do one rule per commit to make it at least a bit reviewable.
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Remove rubocop-govuk


Error Message when parsing trip from input-field

If a trip is added via input field, nothing happens if there is any not supported field.
E.g. not yet supportet tag escooter_rental (amenity)!
Parser for Trips not working Screenshot 2022-06-07 104414
There should be a general error, E.g. some fields could not be parsed, or even better the text wich was not poassible to translate into a trip information.

Doesn't happen any more.
Therefore I close it again.


Remove source files and the build script from the npmignore

...so that npm can build locally when using a git dependency. You know, like when you have a commit point in a custom fork you want to use:
{
  "dependencies": {
    "react-resizable": "github:dnissley-al/react-resizable#c3cd595"
  }
}

If you put the build script and source files in .npmignore file, the prepublish task seems to silently fail when it doesn't find the build script. This leaves the installed package in a strange state with no sources, no build script, and no build either. That's strange and bad, in my opinion -- github dependencies should probably just not ignore any files. But hey... that's where we are.

Actually, this is not such an easy problem to solve right here, as it's a problem I've had with other packages as well. Maybe I should file an issue with npm. Github dependencies should probably not pay attention to npmignore/gitignore and should also install devDependencies.

Thanks. Looking forward to seeing how/if this is fixed on the npm side.
On Sep 5, 2016 4:58 PM, "Dylan Nissley"<EMAIL_ADDRESS>wrote:

Closed #41 https://github.com/STRML/react-resizable/pull/41.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/STRML/react-resizable/pull/41#event-778670831, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABJFPy4Gxp9hJgD1ZWcvafqAttdn1RuXks5qnJCHgaJpZM4J0sL7
.


Wrote a blog post about Better TypeScript Lib

Here's the blog post. We can share it on social media to popularize this library.
<EMAIL_ADDRESS>On a side note, could you enable GitHub Discussions for this repository so that we can have longer discussions on topics that aren't necessarily issues?

Wow, I'm so grateful for this 🥰 just shared on my Twitter.
Also Discussions is now enabled 🙂


feat: Add Sync + Send to Catalog trait

fix: https://github.com/apache/iceberg-rust/issues/166

Thanks for the PR @ZhengLin-Li, and @liurenjie1024 & @Xuanwo for the review 🙌


Hamersley Leaflet notebook

--- I got the following to work thusly, giving an error with the old version.
vtk = True
if vtk:
vtkexp = gp._plot.export_to_vtk(geo_model, path=vtk_path, name=test_data_name + '.vtk', voxels=False, block=None,
surfaces=True)

Miguel says it is a fix that he needs to make, but that will get it to work for the current version.

sure, as with LoopStructural, its better if the modelling engine takes over the export task (making it an import) since the goalposts keep moving! I'll put this in the notebooks.


Code improvements

added constants to the backend
implemented poetry in the backend
added linter to the frontend
dockerized

Please exclude linter changes into a different PR


Else if are not supported

I write this issue very quick cause I'm not done for my work today but I found a bug !
This tree sitter plugin support the if and else pattern in templ but the else if of Go is not.
In my treesitter tree I can see some ERROR node on when should be component_if_statement.
I will look later this day for making a PR to address this issue since I think it should be relativity easy.
But for now I made this Issue as a reminder for myself ^^

I'm closing this since you fixed it in #12


DD-1183 Service Document problems

Fixes DD-
Description of changes
Service document is adapted SWORD2 required behaviour
How to test

Start service dd-sword2
run curl command: curl -v -u user001:user001 http://localhost:20320/servicedocument

Related PRs
(Add links)
*
Notify
@DANS-KNAW/dataversedans

@aliassheikh, from the JIRA issue:

Leave out maxUploadSize, as we don’t enforce an limit. OK
Add app:accept elements to the collections, one with application/zip and one with application/octet-stream which are the two media types we use. OK
Same for app:accept@alternate=”multiparet-related” (not sure what this element means; some more investigation would be useful) -> MISSING
Add sword:mediation and set it to false.  -> MISSING

OK, lets leave the accept@alternate out then. Looks OK now.


ocamlPackages.bigstring: init at 0.2

Motivation for this change
Dependency of tezos
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

This package is already in nixpkgs at this time, so closing.

Sorry, I misread and confused bitstring vs bigstring

I rebased this onto master, will merge after ofborg finishes.


License

Hey, welcome to the chess engine world!
Could we have a license added to the repository?
Thanks!

Hey, thanks for pointing that out. I just added the license! 9d5232e


New InstantiateFromURL activation in lectures

We will need to have some sort of a comment to tell people how to instantiate if required.
using InstantiateFromURL
github_project("QuantEcon/quantecon-notebooks-jl", tag = "v0.9.6")
# ] instantiate  # uncomment if required

Done. 736f61861e1b26c917dc1042f55df5fb01686777


TypeScript implementation

TypeScript initial implementation very similar to the ones in the other languages.

Thanks @webartoli!


stopped working

since yesterday integation stopped working
Logs:
2022-04-13 21:31:30 WARNING (Thread-344) [custom_components.aquaariston.aristonaqua] <custom_components.aquaariston.aristonaqua.AquaAristonHandler object at 0x7f0b4833b460> Retrying in 72.0 seconds
2022-04-13 21:31:32 WARNING (Thread-347) [custom_components.aquaariston.aristonaqua] <custom_components.aquaariston.aristonaqua.AquaAristonHandler object at 0x7f0b4833b460> Authentication login error
2022-04-13 21:31:32 WARNING (Thread-347) [custom_components.aquaariston.aristonaqua] Connection errors: 23
2022-04-13 21:31:32 WARNING (Thread-347) [custom_components.aquaariston.aristonaqua] ariston action nok for _get_main: Login parsing of URL failed
Any idea what is wrong?

I have updated HA to 2022.4.4 - integration is OK :)

Same issue since same time but don't resolve with last HA update  2022.4.7
`Logger: custom_components.aquaariston.aristonaqua
Source: custom_components/aquaariston/aristonaqua.py:1677
Integration: AquaAriston (documentation, issues)
First occurred: 03:54:25 (343 occurrences)
Last logged: 10:39:17
ariston action nok for _get_main: Login parsing of URL failed
ariston action nok for _get_main: Login request exception`

reinstall custom-component solve it :)


ponyc failing to link via ld.gold on Alpine

Hello there,
I'm currently in the process of updating ponyc to 0.28.1 on Alpine Linux Edge. Building anything (e.g. pony-stable) fails though:
...
Optimising
Writing build/release/stable.o
Linking build/release/stable
/usr/lib/gcc/x86_64-alpine-linux-musl/8.3.0/../../../../x86_64-alpine-linux-musl/bin/ld.gold: error: cannot find -lexecinfo
/usr/lib/gcc/x86_64-alpine-linux-musl/8.3.0/../../../../x86_64-alpine-linux-musl/bin/ld.gold: fatal error: build/release/stable: No error information
collect2: error: ld returned 1 exit status
Warning: environment variable $CC undefined, using gcc as the linker
Error:
unable to link: gcc -o build/release/stable -O3 -march=native -mcx16 -latomic  -fuse-ld=gold build/release/stable.o -L"/usr/lib/pony/0.28.1/bin/" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/" -L"/usr/lib/pony/0.28.1/bin/../lib/native" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../lib/native" -L"/usr/lib/pony/0.28.1/bin/../packages" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../packages" -L"/usr/local/lib" -Wl,-rpath,"/usr/local/lib" -Wl,--start-group -l"rt" -Wl,--end-group  -lpthread  -lponyrt-pic -ldl -lm -lexecinfo

ponyc -v:
ponyc -v
0.28.1 [release]
compiled with: llvm 6.0.1 -- gcc (Alpine 8.3.0) 8.3.0
Defaults: pic=true ssl=openssl_0.9.0

See https://github.com/alpinelinux/aports/pull/8626/files for the APKBUILD I use for building ponyc.
The crash seems to be related to https://sourceware.org/bugzilla/show_bug.cgi?id=23856.

It would appear that the execinfo library isn't installed and is required.
see: error: cannot find -lexecinfo

Ah, sorry, pasted the wrong log (from my native host and not in docker) - this is the right one:
Optimising
Writing build/release/stable.o
Linking build/release/stable
/usr/lib/gcc/x86_64-alpine-linux-musl/8.3.0/../../../../x86_64-alpine-linux-musl/bin/ld.gold: fatal error: build/release/stable: No error information
collect2: error: ld returned 1 exit status
Error:
unable to link: gcc -o build/release/stable -O3 -march=native -mcx16 -latomic  -fuse-ld=gold build/release/stable.o -L"/usr/lib/pony/0.28.1/bin/" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/" -L"/usr/lib/pony/0.28.1/bin/../lib/native" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../lib/native" -L"/usr/lib/pony/0.28.1/bin/../packages" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../packages" -L"/usr/local/lib" -Wl,-rpath,"/usr/local/lib" -Wl,--start-group -l"rt" -Wl,--end-group  -lpthread  -lponyrt-pic -ldl -lm -lexecinfo
make: *** [Makefile:43: build/release/stable] Error 255

The docker container using Alpine builds fine (and plenty of folks use for building programs after that). First step would be to investigate what is different about your environment:
https://github.com/ponylang/ponyc/blob/master/.dockerhub/alpine/Dockerfile
I'm imagining that a lot has changed since Alpine 3.9 with the Alpine you are using?

This is on Alpine Linux Edge, which will be released as 3.10 soon (it's currently equivalent to 3.10-rc2)

What happens if you run:
gcc -o build/release/stable -O3 -march=native -mcx16 -latomic  -fuse-ld=gold build/release/stable.o -L"/usr/lib/pony/0.28.1/bin/" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/" -L"/usr/lib/pony/0.28.1/bin/../lib/native" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../lib/native" -L"/usr/lib/pony/0.28.1/bin/../packages" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../packages" -L"/usr/local/lib" -Wl,-rpath,"/usr/local/lib" -Wl,--start-group -l"rt" -Wl,--end-group  -lpthread  -lponyrt-pic -ldl -lm -lexecinfo
directly?

The exact same thing:
gcc -o build/release/stable -O3 -march=native -mcx16 -latomic -fuse-ld=gold build/release/stable.o -L"/usr/lib/pony/0.28.1/bin/" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/" -L"/usr/lib/pony/0.28.1/bin/../lib/native" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../lib/native" -L"/usr/lib/pony/0.28.1/bin/../packages" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../packages" -L"/usr/local/lib" -Wl,-rpath,"/usr/local/lib" -Wl,--start-group -l"rt" -Wl,--end-group -lpthread -lponyrt-pic -ldl -lm -lexecinfo
/usr/lib/gcc/x86_64-alpine-linux-musl/8.3.0/../../../../x86_64-alpine-linux-musl/bin/ld.gold: fatal error: build/release/stable: No error information
collect2: error: ld returned 1 exit status

Can you try with -fPIE added to that invocation, as mentioned in this comment in bugzilla (https://sourceware.org/bugzilla/show_bug.cgi?id=23856#c2)?

Same thing:
gcc -o build/release/stable -O3 -march=native -mcx16 -latomic -fuse-ld=gold build/release/stable.o -L"/usr/lib/pony/0.28.1/bin/" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/" -L"/usr/lib/pony/0.28.1/bin/../lib/native" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../lib/native" -L"/usr/lib/pony/0.28.1/bin/../packages" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../packages" -L"/usr/local/lib" -Wl,-rpath,"/usr/local/lib" -Wl,--start-group -l"rt" -Wl,--end-group -lpthread -lponyrt-pic -ldl -lm -lexecinfo -fPIE
/usr/lib/gcc/x86_64-alpine-linux-musl/8.3.0/../../../../x86_64-alpine-linux-musl/bin/ld.gold: fatal error: build/release/stable: No error information
collect2: error: ld returned 1 exit status

I'm unable to reproduce this on Alpine 3.10 via docker. I ran everything in the following docker container:
sudo docker run -it --rm alpine

The commands I ran were the following:
apk add --update \
  alpine-sdk \
  libressl-dev \
  binutils-gold \
  pcre2-dev \
  libexecinfo-dev \
  coreutils \
  linux-headers \
  cmake \
  python

I installed llvm 7 instead of llvm 6 because alpine 3.10 doesn't have pre-built packages for llvm 6.
apk add --update llvm7-dev

git clone https://github.com/ponylang/ponyc

I tried both master and tag 0.28.1 without running into issues.
make default_pic=true LLVM_CONFIG=llvm7-config

Output showing OS version, ponyc version, and compiling pony-stable successfully:
/ponyc # cat /etc/os-release
NAME="Alpine Linux"
ID=alpine
VERSION_ID=3.10.0
PRETTY_NAME="Alpine Linux v3.10"
HOME_URL="https://alpinelinux.org/"
BUG_REPORT_URL="https://bugs.alpinelinux.org/"
/ponyc # cd pony-stable/
/ponyc/pony-stable # make
../build/release/ponyc  stable -o build/release
Building builtin -> /ponyc/packages/builtin
Building stable -> /ponyc/pony-stable/stable
Building files -> /ponyc/packages/files
Building time -> /ponyc/packages/time
Building collections -> /ponyc/packages/collections
Building ponytest -> /ponyc/packages/ponytest
Building random -> /ponyc/packages/random
Building buffered -> /ponyc/packages/buffered
Building term -> /ponyc/packages/term
Building promises -> /ponyc/packages/promises
Building strings -> /ponyc/packages/strings
Building signals -> /ponyc/packages/signals
Building capsicum -> /ponyc/packages/capsicum
Building json -> /ponyc/packages/json
Building format -> /ponyc/packages/format
Building options -> /ponyc/packages/options
Generating
 Reachability
 Selector painting
 Data prototypes
 Data types
 Function prototypes
 Functions
 Descriptors
Optimising
Writing build/release/stable.o
Linking build/release/stable
/ponyc/pony-stable # ../build/release/ponyc --version
0.28.1-9213b98b [release]
compiled with: llvm 7.1.0 -- cc (Alpine 8.3.0) 8.3.0
Defaults: pic=true ssl=openssl_0.9.0

I installed llvm 7 instead of llvm 6 because alpine 3.10 doesn't have pre-built packages for llvm 6.

Yup, llvm6 is in the testing repo for now (like ponyc). I guess I'll try with LLVM7 now that 7.1.0 works better with musl.
I tried the same steps, but it still errors out for me :thinking:
 ~/D/ponyc-0.28.1> sudo env LLVM_CONFIG=/usr/bin/llvm7-config default_pic=true make install

ponyc -v
0.28.1 [release]
compiled with: llvm 7.1.0 -- cc (Alpine 8.3.0) 8.3.0
Defaults: pic=true ssl=openssl_0.9.0

~/D/pony-stable-0.2.0> make
sed s/%%VERSION%%/"0.2.0 [release]"/ stable/version.pony.in > stable/version.pony
mkdir -p build/release
ponyc  stable -o build/release
Building builtin -> /usr/lib/pony/0.28.1/packages/builtin
Building stable -> /home/rasmus/Downloads/pony-stable-0.2.0/stable
Building files -> /usr/lib/pony/0.28.1/packages/files
Building time -> /usr/lib/pony/0.28.1/packages/time
Building collections -> /usr/lib/pony/0.28.1/packages/collections
Building ponytest -> /usr/lib/pony/0.28.1/packages/ponytest
Building random -> /usr/lib/pony/0.28.1/packages/random
Building buffered -> /usr/lib/pony/0.28.1/packages/buffered
Building term -> /usr/lib/pony/0.28.1/packages/term
Building promises -> /usr/lib/pony/0.28.1/packages/promises
Building strings -> /usr/lib/pony/0.28.1/packages/strings
Building signals -> /usr/lib/pony/0.28.1/packages/signals
Building capsicum -> /usr/lib/pony/0.28.1/packages/capsicum
Building json -> /usr/lib/pony/0.28.1/packages/json
Building format -> /usr/lib/pony/0.28.1/packages/format
Building options -> /usr/lib/pony/0.28.1/packages/options
Generating
 Reachability
 Selector painting
 Data prototypes
 Data types
 Function prototypes
 Functions
 Descriptors
Optimising
Writing build/release/stable.o
Linking build/release/stable
/usr/lib/gcc/x86_64-alpine-linux-musl/8.3.0/../../../../x86_64-alpine-linux-musl/bin/ld.gold: fatal error: build/release/stable: No error information
collect2: error: ld returned 1 exit status
Warning: environment variable $CC undefined, using gcc as the linker
Error:
unable to link: gcc -o build/release/stable -O3 -march=native -mcx16 -latomic  -fuse-ld=gold build/release/stable.o -L"/usr/lib/pony/0.28.1/bin/" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/" -L"/usr/lib/pony/0.28.1/bin/../lib/native" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../lib/native" -L"/usr/lib/pony/0.28.1/bin/../packages" -Wl,-rpath,"/usr/lib/pony/0.28.1/bin/../packages" -L"/usr/local/lib" -Wl,-rpath,"/usr/local/lib" -Wl,--start-group -l"rt" -Wl,--end-group  -lpthread  -lponyrt-pic -ldl -lm -lexecinfo
make: *** [Makefile:43: build/release/stable] Error 255

@Cogitri to make sure I understand, did you try in a clean docker container like I did or in your existing environment or some other configuration/environment?
It would be extremely helpful if you're able to provide instructions on how to reproduce the issue in either a docker container or a VM starting from a clean/base Alpine 3.10 install.

I tested this both in a clean environment via Docker and in my own system. I built ponyc manually on my system and via abuild in Docker (which is Alpine's package format). To build it with abuild:
git clone https://github.com/alpinelinux/docker-abuildt
git clone https://github.com/Cogitri/aports -b ponyc
cd docker-abuild && make
cd ../aports && cd testing/pony-stable && ../../../docker-abuild/dabuild -r
Docker-abuild should automatically pull a clean alpine container from dockerhub, containing only the alpine base and a few build tools. This should build pony-stable with the ponyc currently in our repos, you can do dabuild -r in aports/testing/ponyc to build ponyc.

@Cogitri I ran the commands you provided and did not encounter any errors. In case it matters, I ran the commands in a Virtualbox VM of Ubuntu Bionic on Windows 10 on x86_64.
vagrant@ubuntu-bionic:~/aports/testing/pony-stable$ sudo ../../../docker-abuild/dabuild -r
Unable to find image 'alpinelinux/docker-abuild:edge-x86_64' locally
edge-x86_64: Pulling from alpinelinux/docker-abuild
2004ee2154b4: Pull complete
394498201e0f: Pull complete
78aa2f98ce2b: Pull complete
ab5b5d86f594: Pull complete
17eb0a500025: Pull complete
b196ba2d604c: Pull complete
Digest: sha256:3bf92939a037bd7b822e72c6cf6a5b32bb08bd2ce5b54ebb09f2d98811e4c73e
Status: Downloaded newer image for alpinelinux/docker-abuild:edge-x86_64
Generating RSA private key, 2048 bit long modulus (2 primes)
....................................+++++
...............+++++
e is 65537 (0x010001)
writing RSA key
>>>
>>> You'll need to install /home/builder/.abuild/-5d1a7ec6.rsa.pub into
>>> /etc/apk/keys to be able to install packages and repositories signed with
>>> /home/builder/.abuild/-5d1a7ec6.rsa
>>>
>>> Please remember to make a safe backup of your private key:
>>> /home/builder/.abuild/-5d1a7ec6.rsa
>>>
'/home/builder/.abuild/-5d1a7ec6.rsa.pub' -> '/etc/apk/keys/-5d1a7ec6.rsa.pub'
fetch http://dl-cdn.alpinelinux.org/alpine/edge/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/edge/community/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/edge/testing/x86_64/APKINDEX.tar.gz
WARNING: Ignoring /home/builder/packages/main/x86_64/APKINDEX.tar.gz: No such file or directory
WARNING: Ignoring /home/builder/packages/community/x86_64/APKINDEX.tar.gz: No such file or directory
WARNING: Ignoring /home/builder/packages/testing/x86_64/APKINDEX.tar.gz: UNTRUSTED signature
(1/4) Upgrading scanelf (1.2.3-r0 -> 1.2.4-r0)
(2/4) Upgrading pax-utils (1.2.3-r0 -> 1.2.4-r0)
(3/4) Upgrading nghttp2-libs (1.38.0-r0 -> 1.39.1-r0)
(4/4) Upgrading expat (2.2.6-r0 -> 2.2.7-r0)
Executing busybox-1.30.1-r2.trigger
OK: 185 MiB in 54 packages
>>> pony-stable: Building testing/pony-stable 0.2.0-r0 (using abuild 3.4.0-r0) started Mon, 01 Jul 2019 21:44:39 +0000
>>> pony-stable: Checking sanity of /home/builder/aports/testing/pony-stable/APKBUILD...
>>> pony-stable: Analyzing dependencies...
>>> pony-stable: Installing for build: build-base ponyc libexecinfo-dev
WARNING: Ignoring /home/builder/packages//testing/x86_64/APKINDEX.tar.gz: UNTRUSTED signature
WARNING: Ignoring /home/builder/packages/main/x86_64/APKINDEX.tar.gz: No such file or directory
WARNING: Ignoring /home/builder/packages/community/x86_64/APKINDEX.tar.gz: No such file or directory
WARNING: Ignoring /home/builder/packages/testing/x86_64/APKINDEX.tar.gz: UNTRUSTED signature
(1/7) Installing binutils-gold (2.32-r0)
(2/7) Installing libffi (3.2.1-r6)
(3/7) Installing llvm6-libs (6.0.1-r3)
(4/7) Installing libexecinfo (1.1-r1)
(5/7) Installing ponyc (0.28.1-r0)
(6/7) Installing libexecinfo-dev (1.1-r1)
(7/7) Installing .makedepends-pony-stable (20190701.214440)
Executing busybox-1.30.1-r2.trigger
OK: 234 MiB in 61 packages
>>> pony-stable: Cleaning up srcdir
>>> pony-stable: Cleaning up pkgdir
>>> pony-stable: Fetching pony-stable-0.2.0.tar.gz::https://github.com/ponylang/pony-stable/archive/0.2.0.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   127    0   127    0     0    679      0 --:--:-- --:--:-- --:--:--   675
100 51726    0 51726    0     0   119k      0 --:--:-- --:--:-- --:--:--  119k
>>> pony-stable: Fetching pony-stable-0.2.0.tar.gz::https://github.com/ponylang/pony-stable/archive/0.2.0.tar.gz
>>> pony-stable: Checking sha512sums...
pony-stable-0.2.0.tar.gz: OK
>>> pony-stable: Unpacking /var/cache/distfiles/pony-stable-0.2.0.tar.gz...
sed s/%%VERSION%%/"0.2.0 [release]"/ stable/version.pony.in > stable/version.pony
mkdir -p build/release
ponyc  stable -o build/release
Building builtin -> /usr/lib/pony/0.28.1/packages/builtin
Building stable -> /home/builder/aports/testing/pony-stable/src/pony-stable-0.2.0/stable
Building files -> /usr/lib/pony/0.28.1/packages/files
Building time -> /usr/lib/pony/0.28.1/packages/time
Building collections -> /usr/lib/pony/0.28.1/packages/collections
Building ponytest -> /usr/lib/pony/0.28.1/packages/ponytest
Building random -> /usr/lib/pony/0.28.1/packages/random
Building buffered -> /usr/lib/pony/0.28.1/packages/buffered
Building term -> /usr/lib/pony/0.28.1/packages/term
Building promises -> /usr/lib/pony/0.28.1/packages/promises
Building strings -> /usr/lib/pony/0.28.1/packages/strings
Building signals -> /usr/lib/pony/0.28.1/packages/signals
Building capsicum -> /usr/lib/pony/0.28.1/packages/capsicum
Building json -> /usr/lib/pony/0.28.1/packages/json
Building format -> /usr/lib/pony/0.28.1/packages/format
Building options -> /usr/lib/pony/0.28.1/packages/options
Generating
 Reachability
 Selector painting
 Data prototypes
 Data types
 Function prototypes
 Functions
 Descriptors
Optimising
Writing build/release/stable.o
Linking build/release/stable
make: *** No rule to make target 'check'.  Stop.
>>> ERROR: pony-stable: check failed
>>> pony-stable: Uninstalling dependencies...
WARNING: Ignoring /home/builder/packages/main/x86_64/APKINDEX.tar.gz: No such file or directory
WARNING: Ignoring /home/builder/packages/community/x86_64/APKINDEX.tar.gz: No such file or directory
WARNING: Ignoring /home/builder/packages/testing/x86_64/APKINDEX.tar.gz: UNTRUSTED signature
(1/7) Purging .makedepends-pony-stable (20190701.214440)
(2/7) Purging ponyc (0.28.1-r0)
(3/7) Purging binutils-gold (2.32-r0)
(4/7) Purging libexecinfo-dev (1.1-r1)
(5/7) Purging libexecinfo (1.1-r1)
(6/7) Purging llvm6-libs (6.0.1-r3)
(7/7) Purging libffi (3.2.1-r6)
Executing busybox-1.30.1-r2.trigger
OK: 185 MiB in 54 packages
vagrant@ubuntu-bionic:~/aports/testing/pony-stable$

Hm, no, that shouldn't matter, but that's super weird :o
I guess I'll closes this for now then and see if I can get this working somehow