bug: Android widget not working on Android 13. 桌面小工具無法在原生Android 13起作用

Prerequisites

[X] I have read the Contributing Guidelines.
[X] I agree to follow the Code of Conduct.
[X] I have searched for existing issues that already include this feature request, without success.

Countdate version
v1.5.2
Current Behavior
當我將小工具拖動到桌面時，發現它竟然會閃退？
系統版本：Android 13
安裝來源：Droid-ify
系統：LineageOS
架構：arm64-v8a
狀態：rooted，並且安裝MicroG。
Expected Behavior
不能新增小工具。
Steps to Reproduce
No response
Additional Information
No response

影片：https://ppt.cc/fP3M0x

我也遇到了与你相同的问题，我不知道这其中到底经历了什么。
你提供的影片，你在计算台海战争倒数？
好吧！可以承认，这会发生的，只是时间早晚问题，中国台湾嘛。

Please focusing on what is best not just for us as individuals, but for the overall community

@hakodaj106 I help you reproduce the current problem.
https://github.com/sk5s/countdateapp/assets/92437055/0bd999ae-7546-455a-8b18-e1287338d149

Related issue #18


How to change Item Icon size, Text size and Badge size, because on my device it's looks very small?

@adilkhan95 those sizes are taken from google material design specs (https://www.google.com/design/spec/components/bottom-navigation.html). in future we might provide a way to change them but for now we will stick with material design guide lines

You can always override those dimensions in your project, eg:
    <dimen name="badge_corner_radius">13dp</dimen><!-- 15 -->
    <dimen name="badge_width_padding">4dp</dimen><!-- 4 -->
    <dimen name="badge_height_padding">1dp</dimen><!-- 1 -->
    <dimen name="badge_top_margin">2dp</dimen><!-- 2 -->```

@rafakob thanks for your support

Can someone show me how to override these dimensions? I added these value in dimens.xml, but it seems like not working for me.

after adding these lines you need to modify the values those are the default values so they look same.
Change the values of these as you like and check.

Is that mean I need to edit the source file instead of just override it with my own dimens.xml file?

If you don't need animations you can use tab layout for that.

@rafakob @Ashok-Varma   Hi, I have an issue, if I set this dimensions values in dimens.xml file and modify the default values I can not see a change. Can you show me how to override these dimensions? Thanks in advance


Jsx ppx v4 part2

This moves https://github.com/rescript-lang/syntax/pull/586 to a branch on the repo.

OK so I've tested this with the genType example project for TypeScript.
With V3 no change at all. Also with "version": 3 in the config it's fine.
With V4 I get:
FAILED: src/ImportHookDefault.cmj

  We've found a bug for you!
  /Users/cristianocalcagno/GitHub/genType/examples/typescript-react-example/src/ImportHookDefault.res

  Multiple definition of the type name props.
  Names must be unique in a given structure or signature.

which is not surprising.

In terms of merging this, there's still the outstanding comment about forwardRef.
The rest seems fine, and could continue in another PR.
Let me know if you have questions about forwardRef.

With V4 I get:
FAILED: src/ImportHookDefault.cmj

  We've found a bug for you!
  /Users/cristianocalcagno/GitHub/genType/examples/typescript-react-example/src/ImportHookDefault.res

  Multiple definition of the type name props.
  Names must be unique in a given structure or signature.

which is not surprising.

While this is a breaking change, I'm wondering whether it could be turned into a net positive.
If it make sense to enforce: at most one component for each module.
mmm that might be getting in the way of libraries or component factories etc
worth thinking about anyway

One possible view is that for component definitions, what the ppx does is completely transparent and documented.
So one can just write a component definition directly without even using the ppx.
That is at least an escape hatch that one could use.
So this is about just accepting the current situation and putting it in context.
A separate thing to think about is to tweak the name of the props type being defined.

One possible view is that for component definitions, what the ppx does is completely transparent and documented. So one can just write a component definition directly without even using the ppx. That is at least an escape hatch that one could use.
So this is about just accepting the current situation and putting it in context.
A separate thing to think about is to tweak the name of the props type being defined.

I had noticed it with the test case in innerModule before, I think it is worth thinking about: one component per module. If we tweak the name of the props type, the error will be gone in the definition site. In the caller site, the uppercase, the application site, is parsed only to Foo.make anyway. So, IMHO, I agree with your opinion that accepting the current situation and putting it in the context as you said.

Not sure I understand which one you're proposing:
1 rename "prop" where necessary to avoid multiple definitions
2 enforce one component per module (so many inner modules are ok)
Which one are you thinking about?

2 enforce one component per module (so many inner modules are ok)

This is what I'm proposing.

2 enforce one component per module (so many inner modules are ok)

This is what I'm proposing.

I think this needs to be enforced directly in the ppx. Not by a compiler error.

So for each module in the ast, check that at most 1 component definition is there.

Can I ask what your proposal is?

I think this needs to be enforced directly in the ppx. Not by a compiler error.

I guess it is not a difficult job to achieve with ast_iterator.

Can I ask what your proposal is?

I think this needs to be enforced directly in the ppx. Not by a compiler error.

I guess it is not a difficult job to achieve with ast_iterator.
EDIT: not even ast_iterator is needed, I guess. We can identify how many react components are there at the module level.

Exactly. It fits naturally in the existing ppx mapper.

There are a lot of visual changes so don't know exactly everything that has changed. But notice that hasReactComponent could be changed in a sub-module, and should be restore to its previous value when processing the submodule is finished.

Most diff is cased by indent by if else expression added. Actually, hasReactComponent value is restored per modules because it is assigned inside the structure mapper.

Shouldn't config.hasReactComponent be restored?

I add to restore the old value of config.hasReactComponent

This is looking good for an initial merge as WIP. Seems low-risk as existing projects are using V3 should not be affected.


Display Unpublish button only when canDeleteFromLive is true

Changed display condition for Unpublish button from canPublish to canDeleteFromLive

Hey Sebastian, wanna write a quick addition to SiteTreePermissionsTest for this?
The permissions stuff is notoriously difficult (and easy to break), so we need full test coverage here.

Unfortunately, I have no experience with writing unit tests for SS. The fix does not affect the Unpublish action itself since that performs the right check, it's only the CMS that doesn't check for the right permission before displaying or hiding the button. I see that 3.0 already fixed this so maybe you might want to check the 3.0 branch. The only weird thing I see is that in 3.0 you don't check for canDeleteFromLive like I do, but for canPublish && canDeleteFromLive. Not sure if this makes sense.

Testing guide is here: http://doc.silverstripe.org/sapphire/en/topics/testing

Merging this patch matches the current state of master, so I'm inclined to allow it.


feat: shops

@correttojs I fixed the location problem the latitude and longitude are now fully visible.

@correttojs I fixed the location problem the latitude and longitude are now fully visible.

nice @mgnisia! I'm going to merge this branch
I checked the google geolocation api but it's hard to do the reverse geolocation, as we receive multiple values (and maybe wrong)

okay no problem! 😊 I already added a field for the bundesland and I think we can add a separate text field for address as well


[SQL Lab] Async query results serialization with MessagePack and PyArrow

CATEGORY
Choose one

[ ] Bug Fix
[x] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

Async query performance in SQL Lab, particularly with large result sets, is fairly poor due to how data is serialized and stored. This PR introduces a RESULTS_BACKEND_USE_MSGPACK config option to use PyArrow to serialize the pandas DataFrame directly, and MessagePack for serializing the results payload. Compared to the existing JSON serialization, Arrow and MessagePack provide improved performance and result in much smaller payloads sent to S3 or other cache backends.
Benchmarks with 100K rows of birth_names examples data (multiple runs)
JSON
avg serialization: 573ms
avg deserialization: 200ms
total: 773ms
compressed payload size: 816718
peak memory usage: 281.1 MiB
Arrow/msgpack
avg serialization: 70ms
avg deserialization: 40ms
total: 110ms
compressed payload size: 452634
peak memory usage: 266.2 MiB
Benchmarks were performed on a Macbook Pro 2.6 GHz i7, 32GB running macOS 10.14.5 and Python 3.6.8. memory-profiler was used for memory usage stats.
TEST PLAN

Testing thus far has been limited to Postgres, mainly on Superset examples data. For full compatibility testing:

Enable RESULTS_BACKEND_USE_MSGPACK = True in superset_config.py
Run queries in SQL Lab with various DB backends containing multiple data types
Ensure displayed results and CSV downloads contain correctly formatted data

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@mistercrunch

Codecov Report

Merging #8069 into master will increase coverage by 0.04%.
The diff coverage is 81.13%.

@@            Coverage Diff             @@
##           master    #8069      +/-   ##
==========================================
+ Coverage   65.57%   65.61%   +0.04%
==========================================
  Files         469      469
  Lines       22496    22536      +40
  Branches     2446     2446
==========================================
+ Hits        14752    14788      +36
- Misses       7624     7628       +4
  Partials      120      120

Impacted Files
Coverage Δ

superset/__init__.py
74.1% <100%> (+0.18%)
:arrow_up:

superset/utils/core.py
87.83% <100%> (ø)
:arrow_up:

superset/dataframe.py
94.82% <100%> (+0.18%)
:arrow_up:

superset/sql_lab.py
77.18% <76%> (+0.17%)
:arrow_up:

superset/views/core.py
71.55% <80.95%> (+0.33%)
:arrow_up:

Continue to review full report at Codecov.

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

cc @etr2460 on SQL Lab backend performance work

Closing & reopening to re-trigger Travis build

Rebasing might solve the JS build issue, been discussed on Superset Slack #airbnb-lyft-sprint channel


Allow adjustment of histogram bins

It would be great to allow:

Adjustment of histogram bin width / locations via the UI
Using integer bin limits for integer images (e.g. uint8, uint16)

For example, if you show the coins image in Napari (File > Open Sample > napari builtins > Coins) and view the histogram, it has an odd 'spiky' appearance where every other bin is higher or lower:

I think this is because the limits of the bins aren't set to integer values (even though this image is uint8 and therefore only contains positive integers). E.g. I think the bins are np.linspace(1, 252, 100) according to here, so for example bin 20 onwards is: 51.7, 54.2, 56.7, 59.3, 61.8, which means the bins accept pixel values of (52, 53, 54), (55, 56), (57, 58, 59), (60, 61)... This alternating of 2 and 3 pixel values in each bin gives this spiky appearance which is a bit misleading to the actual values. Would it be possible to set integer bin edges, or allow this to be set as an option? Thanks!

👍 to doing this, a pull request that implements both suggestions (integer bins for integer data, and a bit of GUI to customise the bins) would be welcome. I'd recommend going with the integer bins first as that will require much less design work and be easier to implement.


Recommendation for including barba.js with require.js module loader

Is there any recommendation for doing the above? Is it best to use the commonjs converter tool, or manually convert each module?

Hello @designingSparks ,
the library is exported with webpack UMD which should make the library compatible with require.js.
So there should be no need to convert.


Define trap (interrupt/exception) handler delegation

At this point, it is more a less coincidental which trap handlers an SBI may or may not delegate to an OS.
While OpenSBI handles e.g. unaligned access, this is not to be expected in general.
Now comes the hard part:
Should we define a set of what exactly is / should be delegated, or at least say that the OS should be prepared for anything to be delegated? It is crucial for the OS to know. Otherwise, there are dozens of possible combos of (non-)delegations.

Can we start by enumerating the full list and the default behavior seen in OpenSBI?

I'd create a trap-handlers.adoc file and a link from the sbi.adoc. WDYT?

Sure. Go with that. However, I'm not sure it's really related to sbi per-se. It's more of how to configure m-mode delegation to s-mode, if m-mode is implemented.

In what case does it matter whether the interrupt/exception is delegated or is handled M-first? Should not the OS expect to handle all defined S exceptions and interrupts - whether delegated or emulated?

In what case does it matter whether the interrupt/exception is delegated or is handled M-first? Should not the OS expect to handle all defined S exceptions and interrupts - whether delegated or emulated?

This is to make clear what to expect and what not, more elaborate:
https://github.com/riscv-non-isa/riscv-brs/pull/84#discussion_r1365900394

@ved-rivos @orangecms  can we revisit this and reach a conclusion?

@ved-rivos @orangecms  can we revisit this and reach a conclusion?

That'd be great. I've literally just had another discussion around this the other day. While Linux now has the patches for handling unaligned access itself, they are currently not effective e.g. on JH7110 boards, where vendor firmware does not delegate the respective exceptions. So that potentially skews performance results, depending on the software.

Also @atishp04
Do we want an SBI extension where an OS can negotiate delegations? I.e agree if a particular trap would go fw-only, fw-first or kernel-first.
That's probably more workable then attempting to pin down which traps can be delegated or not (or should be), as that's highly dependent on the system in question. It seems like it would also help solve the real issues you see with JH7110.

I would also expect to see all exceptions that can be handled by the OS to be delegated. I did some evaluations with preempt_rt and saw latencies of 300us on exceptions handled in machine mode. As this can be triggered by any user I fear that undelegated exceptions may be an issue.

I would also expect to see all exceptions that can be handled by the OS to be delegated. I did some evaluations with preempt_rt and saw latencies of 300us on exceptions handled in machine mode. As this can be triggered by any user I fear that undelegated exceptions may be an issue.

@TobiasSchaffner, could this be achieved by having an OS /negotiate/ with the M-mode firmware on delegations it would like to have (via an SBI extension) - instead of simply mandating a list of delegations in a spec?

We already have a SBI extension proposal close to the final stages. It started with misaligned delegation and expanded to other config behaviors for ISA (ADUE, CFI etc).
https://lists.riscv.org/g/tech-prs/topic/105041801#866
Please review and let us know if you feel some other exception should also be added to the fwft feature list.

@TobiasSchaffner, could this be achieved by having an OS /negotiate/ with the M-mode firmware on delegations it would like to have (via an SBI extension) - instead of simply mandating a list of delegations in a spec?

I think I am not the right person to answer this. At the moment I do not understand where this "firmware first" approach comes from and in which cases the OS would want to reject to handle exceptions that are handleable in S-Mode.

@TobiasSchaffner : Is this related to misaligned access exception or something else ?

@atishp04 The latencies can be triggered with any undelegated exception no matter if it is an unaligned memory access or e.g. and illegal instruction exception.
At the moment OpenSBI delegates:

Breakpoint
Environment Call
Fetch page fault
Load page fault
Store page fault
Misaligned instruction fetch

See: https://github.com/riscv-software-src/opensbi/blob/master/lib/sbi/sbi_hart.c#L195

@TobiasSchaffner, could this be achieved by having an OS /negotiate/ with the M-mode firmware on delegations it would like to have (via an SBI extension) - instead of simply mandating a list of delegations in a spec?

I think I am not the right person to answer this. At the moment I do not understand where this "firmware first" approach comes from and in which cases the OS would want to reject to handle exceptions that are handleable in S-Mode.

Consider that SBI may provide some unimpl instruction emulation. That's "firmware first". Presumably anything not handled would be injected for handling into the OS (I guess there's no point not to inject, but it looks like some implementations don't delegate for some traps period). A "kernel first" approach would be a contact that the OS takes full responsibility for all illegal insn handling and SBI impl does none of it.
I don't think an OS might want to reject handling exceptions. I think it's more about a kernel just might not care enough and be okay with whatever SBI impl does. Hence negotiating for each desired trap instead of mandating (in a spec) that everything is delegated.

We already have a SBI extension proposal close to the final stages. It started with misaligned delegation and expanded to other config behaviors for ISA (ADUE, CFI etc). https://lists.riscv.org/g/tech-prs/topic/105041801#866
Please review and let us know if you feel some other exception should also be added to the fwft feature list.

@atishp04
Oh neat. Can we add a dependency on FWFT now or should we wait?

An OS should have handling for all exceptions specified by the Priv. specification and can be delegated to S-mode. In some implementations M-mode may emulate  some some of the exceptions. In other cases there may be firmware first handling. In both cases this should not lead to a functional difference to the S-mode software. I think of an implementation that handles unaligned load/store through emulation in M-mode as not very different than an implementation that handles unaligned load/store poorly in hardware - these just lead to performance issues and not functional issues. The BRS specification would not want to place performance requirements. So whether an exception is delegated or not should not be material unless it affects the functioning of the S-mode.

Ved, we are aware of all that.
This is not about functional differences. It is about performance guarantees and expectations. I still do not understand why it is that unaligned access isn't delegated. You can just do it. All I'm asking for is clear statements. We are still at "be prepared for everything but do not expect anything". That is a non-binding contract.
My suggestion is to declare performance profiles, just like with the platform profiles. Call the one delegating everything possible the "high-perf" profile, and what OpenSBI will do the "negotiable" profile or something like that. Then it is clear what the OS/end user can expect. WDYT?

On

The BRS specification would not want to place performance requirements

Who decides that, why would that be? Would you want another working group to define performance profiles? If so, why?

Who decides that, why would that be? Would you want another working group to define performance profiles? If so, why?

How would one define performance profiles? A machine may implement unaligned access in HW and take say a 1000 cycles to do the access. Another machine may do unaligned accesses in 3 cycles. Is the first machine then incompatible with risc-v and why is 3 cycles sufficient and not 1 cycle.
Performance is something that should be selected by the application. A system integrator building a network router or a laptop would pick the components needed for that application to meet the performance objectives of that application. What may be adequate performance for a 1 gigabit router may not be adequate for a terabit router. But BRS is applicable to both applications.

Honestly I'd steer this in a different direction. I agree with Ved that the BRS doesn't concern itself with performance expectations, but in my opinion this isn't about performance, as we're not trying to mandate specific performance for anything in the spec. It's about functionality, in the sense that an OS may simply wish to provide its own functionality instead of relying on the firmware. It may choose to do so because it thinks it can do a better job, or as a possible workaround for bugs. In case of unaligned accesses, an OS may choose to disallow unaligned accesses entirely. In case of other delegations, avoid cycles spent in foreign non-interruptible code. Whatever the reason.
The FWFT allows an OS to potentially dial the level of low-level awareness it needs. Perhaps RAS is another such example, eventually, where an OS can choose kernel-first on a system where the default would be firmware-first.

Agreed that the FWFT extension allows an OS to pick between kernel and OS first approach.

@orangecms can you rework this PR to reference the still in progress FWFT extension as a required mechanism.

We already have a SBI extension proposal close to the final stages. It started with misaligned delegation and expanded to other config behaviors for ISA (ADUE, CFI etc). https://lists.riscv.org/g/tech-prs/topic/105041801#866
Please review and let us know if you feel some other exception should also be added to the fwft feature list.

@atishp04
Oh neat. Can we add a dependency on FWFT now or should we wait?

FWFT will be ratified as a part of SBI v3.0 towards end of this year. As BRS is already in ARC review, I am not sure what's the policy for pointing to unratified specs.

ok we can pend this on SBI 3.0


Release CosmosDB Track 2 Library Preview

Goal:
Release a CosmosDB Track 2 package

Deadline for UX study: 6/4

https://github.com/Azure/azure-sdk-for-python/pull/6134 has been merged with feedback from the UX study.


rust-musl-builder is currently unmaintained

Comment by the project author.
It seems important to at least add a note on the state of rust-musl-builder, or consider removing it (especially if an active fork is available)

Thanks, removed this reference in c5f90f3


Show Source of Conflict

Fixes #256

Closing this as stale but will leave issue #256 open as the requirement still stands.


readinessProbe for Envoy

Hi,
At first I thought this issue may be covered by https://github.com/istio/istio/issues/1425, however I now feel it is different.  If i'm wrong, please close this issue.
At a high level, I need to ensure that istio-proxy does not become Ready before it has successfully received configuration information from the control plane and has configured its listeners.
At the moment, an application pod could become Ready and istio-pilot pod would be immediately ready (due to the lack of readinessProbes on the istio-pilot container), yet the application would not be able to receive or send communications because istio-pilot has not actually initialised.
This results in a situation where we cannot do 0 downtime rollouts of an application with istio-proxy, as kubernetes will add it to the service before it's truly Ready to handle traffic.  This is particularly visible in our ingress-nginx controller, which (sometimes) starts receiving external mesh traffic before istio-proxy has configured, thus ingress-nginx is unable to send the traffic to the upstream components.
I'm wondering if there is some sort of readinessProbe that I could add to the sidecar, perhaps curl http://<IP_ADDRESS>:15000/stats | grep <something here> which would indicate that proxy has received its configuration from the control plane and the listeners are configured correctly.
Hope this makes sense, it's quite an urgent one for us to fix so any advice would be appreciated.
Cheers
Karl

This is about the actual readiness of istio-proxy/istio-pilot. I am not actively working on it.

This looks like its covered by #6082

@frankbu how would I utilise that issue to resolve this one?

For a proper solution, seems you need to wait for the solution to #882.
If you are looking for a short term workaround, I guess you could look at getting Envoy stats, something like what istioctl will be doing for #6082.

@Stono FYI readiness probe for istio-proxy was implemented by https://github.com/istio/istio/pull/7465

@costinm is it too late to get this into 1.0.2?


src/services/ng2-uploader.js references to angular2/core which breaks

Although src/services/ng2-uploader.ts correctly references to @angular/core but the transpiled verson src/services/ng2-uploader.js still references to angular2/core which I think is incorrect.
When I use ng2-uploader in my project, I got error complaining about angular2 not existed.
It's possible that the version uploaded to npmjs is bad.

I agree.
System.register(['angular2/core'],
Should be removed from src/services/ng2-uploader.js.

This is resolved with the new version.


updating routes for root path to be application#index

Coverage remained the same at 68.06% when pulling 491d159773438f12c96da96ca5c3f98a95c10a93 on updating_tests into 8747c57aa9f5684d2b13f52f769e8a30cbe11fd4 on development.


Update ibm_catalog.json

Description
I've completed a content review and made edits to the ibm_catalog.json file text as part of AHA Epic DAF-299.
This review is related to the following issues:
https://github.ibm.com/GoldenEye/issues/issues/7808
https://github.ibm.com/Bluemix/docs/issues/10671
Note: This PR is related to PR #64. There were conflicts in that PR so for simplicity I have created this new PR with the latest changes for the content. Refer to https://github.com/terraform-ibm-modules/terraform-ibm-observability-da/pull/64 for a history of the discussion and changes.
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

@acolleen can you resolve the conflicts?

/run pipeline

/run pipeline

/run pipeline

@iamar7 Can you do a final review on the changes so we can merge?

@maheshwarishikha Or Shika, can you review?

Thanks for the close review @iamar7. Made some assumptions for your suggestions that don't quite work as is. Let me know.

Thanks for the close review @iamar7. Made some assumptions for your suggestions that don't quite work as is. Let me know.

Thanks @SirSpidey. I've went through the changes which you made and it works perfectly fine. There is only one more comment which needs to get resolved after that this PR will be good to get merged.

@iamar7 I think we're all set now for a final review.... 🙏

:tada: This PR is included in version 1.8.2 :tada:
The release is available on:

GitHub release
v1.8.2

Your semantic-release bot :package::rocket:


Hang on close after closing a large binary

Version and Platform (required):

Binary Ninja Version: 2.4.3338-dev
OS: macOS
OS Version: 12.3.1

Bug Description:
On close of a large binary the actual destruction of the BinaryView is done on a worker thread to avoid blocking the UI for a long time. If you close the UI while this is happening, Binary Ninja will sometimes hang trying to exit (this doesn't happen very often).
Stack trace of problem worker thread:
  * frame #0: 0x00000001a59a8290 libsystem_kernel.dylib`__psynch_cvwait + 8
    frame #1: 0x00000001a59e283c libsystem_pthread.dylib`_pthread_cond_wait + 1236
    frame #2: 0x00000001a5931230 libc++.1.dylib`std::__1::condition_variable::wait(std::__1::unique_lock<std::__1::mutex>&) + 28
    frame #3: 0x000000010762b0fc libbinaryninjacore.1.dylib`BinaryNinjaCore::MainThreadAction::Wait(this=0x000060017f81c540) at mainthread.cpp:57:10 [opt]
    frame #4: 0x000000010762b458 libbinaryninjacore.1.dylib`BinaryNinjaCore::ExecuteOnMainThreadAndWait(action=0x00000002d6966df8)> const&) at mainthread.cpp:103:7 [opt]
    frame #5: 0x000000010732dee4 libbinaryninjacore.1.dylib`BinaryNinjaCore::FileMetadata::UnregisterViewOfType(this=<unavailable>, name=0x0000000136632778, view=<unavailable>) at filemetadata.cpp:969:2 [opt]
    frame #6: 0x00000001071d7254 libbinaryninjacore.1.dylib`BinaryNinjaCore::BinaryView::~BinaryView(this=0x0000000136632760) at binaryview.cpp:3196:11 [opt]
    frame #7: 0x00000001071a0df4 libbinaryninjacore.1.dylib`BinaryNinjaCore::BinaryData::~BinaryData() [inlined] BinaryNinjaCore::BinaryData::~BinaryData(this=0x0000000136632760) at binarydata.h:10:8 [opt]
    frame #8: 0x00000001071a0dd0 libbinaryninjacore.1.dylib`BinaryNinjaCore::BinaryData::~BinaryData() [inlined] BinaryNinjaCore::BinaryData::~BinaryData(this=0x0000000136632760) at binarydata.h:10:8 [opt]
    frame #9: 0x00000001071a0dd0 libbinaryninjacore.1.dylib`BinaryNinjaCore::BinaryData::~BinaryData(this=0x0000000136632760) at binarydata.h:10:8 [opt]
    frame #10: 0x00000001071d7680 libbinaryninjacore.1.dylib`BinaryNinjaCore::BinaryView::~BinaryView() [inlined] BinaryNinjaCore::Ref<BinaryNinjaCore::BinaryView>::~Ref(this=<unavailable>) at refcountobject.h:107:12 [opt]
    frame #11: 0x00000001071d766c libbinaryninjacore.1.dylib`BinaryNinjaCore::BinaryView::~BinaryView() [inlined] BinaryNinjaCore::Ref<BinaryNinjaCore::BinaryView>::~Ref(this=<unavailable>) at refcountobject.h:104:3 [opt]
    frame #12: 0x00000001071d766c libbinaryninjacore.1.dylib`BinaryNinjaCore::BinaryView::~BinaryView(this=0x000000010d07c600) at binaryview.cpp:3199:1 [opt]
    frame #13: 0x000000010730619c libbinaryninjacore.1.dylib`BinaryNinjaCore::ElfView::~ElfView() [inlined] BinaryNinjaCore::ElfView::~ElfView(this=<unavailable>) at elfview.cpp:289:1 [opt]
    frame #14: 0x0000000107306198 libbinaryninjacore.1.dylib`BinaryNinjaCore::ElfView::~ElfView(this=<unavailable>) at elfview.cpp:289:1 [opt]
    frame #15: 0x000000010790f1dc libbinaryninjacore.1.dylib`BinaryNinjaCore::Worker::ProcessingThread() [inlined] std::__1::__function::__value_func<void ()>::operator(this=<unavailable>)() const at function.h:505:16 [opt]
    frame #16: 0x000000010790f1c8 libbinaryninjacore.1.dylib`BinaryNinjaCore::Worker::ProcessingThread() [inlined] std::__1::function<void ()>::operator(this=<unavailable>)() const at function.h:1182:12 [opt]
    frame #17: 0x000000010790f1c8 libbinaryninjacore.1.dylib`BinaryNinjaCore::Worker::ProcessingThread(this=0x0000600000061b60) at worker.cpp:125:4 [opt]
    frame #18: 0x000000010790ea78 libbinaryninjacore.1.dylib`BinaryNinjaCore::Worker::ProcessingThreadWrapper(arg=<unavailable>) at worker.cpp:73:18 [opt]
    frame #19: 0x00000001a59e226c libsystem_pthread.dylib`_pthread_start + 148

The main thread is busy joining the worker threads, so we have a deadlock.
This can happen in headless as well, as the background BinaryView destruction happens from the core.

The general structure of this is:

The main thread calls BNShutdown and starts shutting down the worker threads
The worker threads all start shutting down
The main thread attempts to join each worker thread in the process of shutting down
The worker threads destroy their lingering references to the open files and call ExecuteOnMainThread to finalize them
Now the worker threads are waiting on the main thread action, and the main thread is stuck trying to join the worker threads
Deadlock

Fix is in build 3.4.4194-dev. If issue persists please reopen.


BFormInput - @input event passes InputEvent as argument instead value of the input.

I use BFormInput as:

And I expect that handleInputValue function will receive a value of the input as string.
But instead, I got InputEvent:

Is it how it supposed to be? I can't get actual value of the input, because val.data contains only one character that was pressed in current event. Or null if I press backspace.
In bootstrap-vue @input was passing string value of the input field. Why has this changed?
Also, in this comment also said that val should be string:
https://github.com/bootstrap-vue-next/bootstrap-vue-next/issues/558#issuecomment-1215183620

OS: Windows 10
Package Version: ^0.19.0
Pnpm 9.1.0

I got it, I need to use @update:modelValue in vue 3 instead @input.

The vue 2 modelvalue in general was different. It was input event and value prop. With the vue 3 changes, it simply made more sense to pass the native events. I think it provides more details and expected functionality when messing with input components


Changes made in pm.py file are not reflected in console

Thanks for reporting. For me, it looks different. Did you check out latest main?

Here you go, I downloaded the latest main and still nothing happen when i modify the ui.py and pm.py :


Logging out of demo hub does not actually log you ou

Steps to reproduce

Log in to demo hub
Once server starts, log out
Click the login button again

Expected result
I'm taken to a screen that asks me to provide username / password to log in
Actual result
I'm logged in automatically!
How to fix?
Auth0 has documentation about the various levels of logging out.  We've logged out at the "Application Session Layer", but not at the "Auth0 Session Layer". We need to send users to the logout endpoint to do that.
This might need work in OAuthenticator.

This might need work in OAuthenticator.

Globus case seems relevant: https://github.com/jupyterhub/oauthenticator/blob/master/oauthenticator/globus.py#L21

Globus case seems relevant: https://github.com/jupyterhub/oauthenticator/blob/master/oauthenticator/globus.py#L21

I think so too! Maybe all we need is to write our own logout handler (another useful comment about this here). I will give this a try today and see how it goes.

Maybe all we need is to write our own logout handler (another useful comment about this here).

That sounds like the way to go!

I will give this a try today and see how it goes.

Super! Thx.


[Python] Recipe on how to find entries equal to a set of values.

>>> pc.drop_null(pc.index_in(pa.array([1, 2, 3]), value_set=pa.array([1, 3])))
<pyarrow.lib.Int32Array object at 0x11e2a63a0>
[
  0,
  1
]

What's the precise intention here? (the title of the issue doesn't seem to fully match what you did)

Mostly a note to write a recipe out the question that was discussed on the user mailing list ( https://mail-archives.apache.org/mod_mbox/arrow-user/202111.mbox/browser ). I felt it was a good topic for a recipe as it's a fairly common need.

A related JIRA: https://issues.apache.org/jira/browse/ARROW-13035. With something like that you could get the indices of the True values after checking for a condition (also answered about that on the mailing list)


Advertisement test operator priority issues

The shifting operations have lower priority than the arithmetic operations in Python and in the le_advertisement_test.py example there are many cases of a<<n + b (which are getting interpreted as (a+b)<<n) that I believe should be (a<<n) + b.

Thanks for your patches.
This module is no longer being actively developed, and focus has shifted to https://github.com/TheCellule/python-bleson


service: Add create_nidaqmx_task(s) API

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Add reservation.create_nidaqmx_task(s).
Why should this Pull Request be merged?
Part of new session management API.
What testing has been done?
Ran new unit tests.
Updated DAQmx example to use new API (not part of this PR).

Example:
diff --git a/examples/nidaqmx_analog_input/measurement.py b/examples/nidaqmx_analog_input/measurement.py
index c22569e..629f562 100644
--- a/examples/nidaqmx_analog_input/measurement.py
+++ b/examples/nidaqmx_analog_input/measurement.py
@@ -49,24 +49,19 @@ def measure(pin_name: str, sample_rate: float, number_of_samples: int) -> Tuple[
         sample_rate,
         number_of_samples,
     )
-    session_management_client = create_session_management_client(measurement_service)
-    with session_management_client.reserve_session(
-        context=measurement_service.context.pin_map_context, pin_or_relay_names=[pin_name]
-    ) as reservation:
-        task: Optional[nidaqmx.Task] = None
-
-        def cancel_callback() -> None:
-            logging.info("Canceling measurement")
-            task_to_abort = task
-            if task_to_abort is not None:
-                task_to_abort.control(TaskMode.TASK_ABORT)
-
-        measurement_service.context.add_cancel_callback(cancel_callback)
-
-        grpc_device_channel = get_grpc_device_channel(measurement_service, nidaqmx, service_options)
-        with create_task(reservation.session_info, grpc_device_channel) as task:
-            if not reservation.session_info.session_exists:
-                task.ai_channels.add_ai_voltage_chan(reservation.session_info.channel_list)
+    with measurement_service.context.reserve_session(pin_name) as reservation:
+        with reservation.create_nidaqmx_task() as session_info:
+            task = session_info.session
+            if not session_info.session_exists:
+                task.ai_channels.add_ai_voltage_chan(session_info.channel_list)
+
+            def cancel_callback() -> None:
+                logging.info("Canceling measurement")
+                task_to_abort = task
+                if task_to_abort is not None:
+                    task_to_abort.control(TaskMode.TASK_ABORT)
+
+            measurement_service.context.add_cancel_callback(cancel_callback)

             task.timing.cfg_samp_clk_timing(
                 rate=sample_rate,
diff --git a/examples/nidaqmx_analog_input/pyproject.toml b/examples/nidaqmx_analog_input/pyproject.toml
index 71b60fb..05a898c 100644
--- a/examples/nidaqmx_analog_input/pyproject.toml
+++ b/examples/nidaqmx_analog_input/pyproject.toml
@@ -7,7 +7,7 @@ authors = ["National Instruments"]
 [tool.poetry.dependencies]
 python = "^3.8"
 nidaqmx = { version = ">=0.8.0", extras = ["grpc"] }
-ni-measurementlink-service = {version = "^1.1.0"}
+ni-measurementlink-service = {version = "^1.1.0", extras = ["nidaqmx"]}
 click = ">=7.1.2, !=8.1.4" # mypy fails with click 8.1.4: https://github.com/pallets/click/issues/2558

 [tool.poetry.group.dev.dependencies]
@@ -15,7 +15,7 @@ mypy = ">=1.0"
 grpc-stubs = "^1.53"
 # Uncomment to use prerelease dependencies.
 # nidaqmx = { git = "https://github.com/ni/nidaqmx-python.git", branch = "master", extras = ["grpc"]}
-# ni-measurementlink-service = {path = "../..", develop = true}
+ni-measurementlink-service = {path = "../..", develop = true, extras = ["nidaqmx"]}

 [build-system]
 requires = ["poetry-core>=1.2.0"]


Fatal error if token is missing

Laravel Turnstile Version: 1.0.0
Laravel Version: 9
PHP Version: 8.0

Description:
NjoguAmos\Turnstile\Http\Middleware\TurnstileMiddleware::validateTurnstile(): Argument #1 ($token) must be of type string, null given, called in /vendor/njoguamos/laravel-turnstile/src/Http/Middleware/TurnstileMiddleware.php on line 22
When spammers are submitting the form directly without any token, the app throws a fatal error.
The desired functionality would be treating a missing token as an invalid token without the fatal error.
This was data sent
--data<EMAIL_ADDRESS>https://冠天下娛樂城.com/&submit=&input_name=ShawnMef&_method=POST'

Thank you for raising this issue. Let me have a look.

Upgrade to v1.0.1


Fix validation in AttributeSerializer

Motivation and context

The current implementation of to_internal_value doesn't call the base class method, which is supposed to carry out generic validation of serializer fields. This means that we essentially don't validate attribute fields beyond what is done incidentally by database contraints.
Fix it by removing the overridden method entirely, and adding a special field class for the values field.
How has this been tested?

Manual testing.
Checklist

[x] I submit my changes into the develop branch
[x] I have added a description of my changes into the CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

/check

@zhiltsov-max Could you review this, please?


Way to teleport to registered landmarks via Python script call

Right now, it is possible to register landmarks in the ARENA builder and teleport to them by clicking on them in the Desktop ARENA viewer in the bottom left. Would it be possible to trigger this same teleportation to a landmark via Python script call?
(Requested by ARENA-ETC team.)

Resolved with b1b3a2b (incorrectly though the issue was in arena-core)


Rename Mt. Coronet areas

Mt. Coronet's floors were misnumbered: a small room on 4F was labelled as 5F, causing the subsequent floors to all be off-by-one. This pull request corrects that numbering, aligning the floor numbering with the official guidebook.
Additionally, a number of the areas within Mt. Coronet had vague or otherwise unclear names (especially "cave"). This pull request renames those areas for clarity. It also ensures that all areas on 1F include "1F" in their name, for consistency.

SGTM, thanks.


fix: Use PHP 8.2 as craftcms PHP version default

[!NOTE]
Logging this as a Draft so it's ready to officially submit as soon as we’ve cut the release; at the moment, 5.0.0-beta.1 hasn't actually been tagged! 😅

The Issue
We have just tagged Craft 5.0.0-beta.1, and with that, are requiring PHP 8.2. The craftcms project type currently uses PHP 8.1, which will not satisfy the requirements.
Our upgrade guide covers switching PHP versions using the config command (ddev config --php-version=8.2), but we would like for all projects moving forward to use PHP 8.2—with which Craft 4 is also fully compatible.
The current database version (MySQL 8.0) is still our recommended engine, so no changes have been made to that configuration.
How This PR Solves The Issue
The craftCmsConfigOverrideAction() function now sets app.PHPVersion = nodeps.PHP82.
Manual Testing Instructions
Create a new DDEV project:
mkdir craft-five
cd craft-five
ddev config --project-type=craftcms --docroot=web --create-docroot
ddev composer create -y --no-scripts<EMAIL_ADDRESS>ddev craft install

If Craft doesn't complain about the PHP version (or emit a lower-level language error), you’re all set!
Automated Testing Overview
No automated tests have been added or updated.
Related Issue Link(s)
Release/Deployment Notes

Craft CMS project types now default to PHP 8.2.

Hmm, that line doesn't probably need to be there in the first place, since 8.1 is default.
Also, 8.2 will be the default in DDEV v1.23, coming as next release probably in a couple of months.

https://github.com/ddev/ddev/issues/5767

I'm still OK with this, just things to consider. It would also be OK to update the quickstart and add it there, which would make it effective immediately in the "latest" docs.

And of course, thanks so much for watching these things and keeping up the maintenance!

Noted! I'll talk with the crew and revise or close. We are currently planning on a 6–8 week beta period, so the docs route may be preferable, in the event our timelines don't quite line up. ✌

Still, there's nothing wrong with your change here, does no harm, just requires future maintenance as time moves along and PHP versions move along.  And you do a great job keeping track, so should be fine anyway.

PHP 8.2 default:

https://github.com/ddev/ddev/pull/5835

But I note that Craft has a bunch of overrides that set it to 8.1, so we'll definitely want to update things here. The two ways of doing it would be to let it track the defaults (not do overrides) or just update the overrides.
Do we also want to force composer_version: 2.7 ?

@AugustMiller this needs a restart now, since PHP 8.2 is now default, etc. Do you want to continue with this one or start over again?

Closing for now, happy to reopen, happy to continue the conversation here or elsewhere. Definitely want this to be up-to-date for Craft 5 and DDEV v1.23.0

We should finish this! The new release is almost ready. Reopening, we'll have to get it done.

As we're out of beta now, I've removed the version override so that we can take advantage of DDEV's default, moving forward. This way, when DDEV is ready to move to 8.3 as the default, Craft projects will automatically get the latest version (rather than needlessly locking users to an older release).
Thanks for reviving, Randy! Holler if you need anything else from us. 💞


command not found when executing

Okay so im new to this whole running script thing, ive installed wget, coreutils in ubuntu and updated my binaries, but i cant seem to download all it says is command not found

Please run the following commands in console and post the full output:
mkdir test
cd test
git clone https://github.com/Shnatsel/furaffinity-dl.git
furaffinity-dl/furaffinity-dl scraps/---

omg thanks so much, btw how do you use cookies, i downloaded the extension but it shows cannot open cookies file no such file or directory, you have provided cookies file but it does not contain valid cookies

dundundunnn@DESKTOP-H0BMTU7:~/test$ furaffinity-dl/furaffinity-dl gallery/chisasa cookies.txt
Cannot open cookies file ‘cookies.txt’: No such file or directory
--2019-03-23 18:25:49--  https://www.furaffinity.net/gallery/chisasa
Resolving www.furaffinity.net (www.furaffinity.net)... <IP_ADDRESS>, <IP_ADDRESS>, 2606:4700:10::6814:463b, ...
Connecting to www.furaffinity.net (www.furaffinity.net)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [text/html]
Saving to: ‘/home/dundundunnn/.cache/furaffinity-dl/fa-dl.zfq2GwhAps’
/home/dundundunnn/.cache/fura     [ <=>                                             ]  43.25K  --.-KB/s    in 0.07s
2019-03-23 18:25:50 (594 KB/s) - ‘/home/dundundunnn/.cache/furaffinity-dl/fa-dl.zfq2GwhAps’ saved [44293]

ERROR: You have provided a cookies file, but it does not contain valid cookies.
If this file used to work, this means that the cookies have expired;
you will have to log in to FurAffinity from your web browser and export the cookies again.
If this is the first time you're trying to use cookies, make sure you have exported them
in Netscape format (this is normally done through "cookie export" browser extensions)
and supplied the correct path to the cookies.txt file to this script.
If that doesn't resolve the issue, please report the problem at
https://github.com/Shnatsel/furaffinity-dl/issues

Looks like you need a primer on how paths in Linux work.
You have a home directory where you normally have a bunch of folders such as "Music", "Downloads", etc. When you want to refer to to your home directory from the command line, you can do so by typing ~. Directories are separated with slashes, so if you want to refer to the "Downloads" folder in your home directory, you'd type ~/Downloads
There is also a concept of current directory in the shell. You can type ls to list all the files in the current directory; most programs operate on it by default, as does furaffinity-dl. It defaults to your home directory, but you can navigate to other places by using cd command.
So what you probably want to run to use furaffinity-dl something like:
mkdir chisasa
cd chisasa
~/furaffinity-dl/furaffinity-dl gallery/chisasa ~/Downloads/cookies.txt

sorry im new to this, It returned as,
-bash: /home/dundundunnn/furaffinity-dl/furaffinity-dl: No such file or directory

Put whatever path you've used to download furaffinity-dl in there. Or just git clone into the home directory all over again.

Oh I get what you mean now, when i renamed the folder from the file explorer ubuntu didnt update it so thats why, instead i restarted ubuntu, also i get permission denied when using furaffinity-dl from the home folder, i used from the test folder instead. thanks for everything though

It seems it's working for you now, so I'm closing the issue.


Inner text of clear button is not customizable

Hi there,
I am using the componenet for an application used in different countries.
Could we make that happen?

@blaur we can add the custom text option for clear button in next release.


feat: Add encode_max for EthMessage

closes #6083

lgtm. this is half of what's needed to close https://github.com/paradigmxyz/reth/issues/6083. the next half is plugging it into https://github.com/paradigmxyz/reth/blob/720815a83465a4c79c0066e466ad53d005cc32c1/crates/net/network/src/eth_requests.rs#L181-L181, then we're done!

The goal is to remove the below check and instead use encode_max right?
https://github.com/paradigmxyz/reth/blob/720815a83465a4c79c0066e466ad53d005cc32c1/crates/net/network/src/eth_requests.rs#L181-L185

yeah that's right, but it would be good to do it before we fetch the block from store

lgtm, make sure to update docs too which is using that old code https://github.com/paradigmxyz/reth/blob/main/docs/crates/network.md

waiting for emhane's response, then fix the nits

@emhane I'm not quite sure if we should stick with the approximation based approach or just do encoding while we're handling the response to get the accurate length
wdyt

let's keep the encoding trait and get the actual length. let's use the approximation to allocate capacity for the buffer.

friendly bump here @supernovahs @emhane

Let me know if this makes sense

Let me know if this looks good

Let me know what do you think of this impl

looks like ci issue for failing tests.working on my local


Multiple parallel uploads are failing

I receive the following error when trying to upload files concurrently.
bad_request: more than one upload using auth token 3_20160731105204_3cffcbaf031eeb76c0d6fdf4_4d28746dd5ee28ffcb887cd4f9f3e250b018531a_001_upld
This is due to the fact that the same upload url is used underneath which is fetched on bucket.internalGetUploadURL() and stored as a property on the bucket itself. Thus, this limits the bucket on using the UploadFile() once at a time.
I think the easy fix in this case would be to not store the auth property anymore on the bucket, and fetch the uploadURL everytime when uploading a file or when calling GetUploadURL method (which also the correct use-case, there's no need for storing the uploadURL ahead of time - bc it can expire).

Ah, yes, I forgot that B2 have stopped parallel uploads using the same token (this did work during the beta). Thanks for this and the other PR, I will look at both this afternoon!

@pfzero - This is working with the tool in /b2, parallel uploads complete as expected.
Regarding storing the upload URL, I have switched to using a pool to store recent upload URLs. It may be that some expire, but this should be treated as a non-fatal error by any code using the library (it will need to handle other non-fatal errors such as server errors, so this should not add any overhead)


How to start?

Hi All,
I have just discovered this library and think it might be useful for face tracking application I am working on. I already have a working detector algorithm. I want to understand how do I integrate that with norfair to enable tracking.
I saw an example code here: https://tryolabs.github.io/norfair/2.2/reference/
I want to understand what do my detector needs to return for this piece of code: detections = detector(frame)
When I read the definition of Detection class here: https://github.com/tryolabs/norfair/blob/009a1b171ab14336d79d7b7b02dfa5f45066c79e/norfair/tracker.py#L748, it is expecting points as a (num_points, num_dimensions) shaped array...where num_dimensions is 2 or 3. So, it's not the bounding box coordinates that I should pass for num_dimensions?
Please advise. Appreciate your help!

Hello! You need to convert the detections of your model to a list of Detection instances. So in your case, since you are working with bounding boxes, you can consider it as 2 points (the corners of the bounding box).
For example, if one of your detections has coordinates bbox = np.array([[x0, y0], [x1, y1]]), you can define a norfair detection as detection = Detection(bbox).
As you can see in the definition you linked, you can also provide the scores (the scores argument), and also class identifiers (the label argument) which might be useful if you have more than one type of class in your use case.
To put a super explicit example, if you have a bounding box with coordinates [[125, 312], [214, 489]], with score 0.75, and class id is '3', a norfair detection like the following would correspond to an object like this
score = np.array([0.75, 0.75])
bbox = np.array([[125, 312], [214, 489]])
label = 3

detection = Detection(bbox, scores = score, label = label)

You will need to make a list of Detection instances in the current frame, and pass it to the Tracker.update method in order to do the tracking.
Maybe looking at the code in some of our demos with particular detectors can help, like the demo with YoloV5 which tracks bounding boxes (as in your case) whenever the argument --track-points is set to 'bbox'.
Don't hesitate to ask if you need further guidance.

Thanks for the detailed response, @aguscas ! I really appreciate it! That makes sense.

@aguscas , just a small clarification! do I need to provide score for every point rather than the whole bounding box? I see that in your example we have just one bounding box but two scores. Thanks!

@utility-aagrawal , yep, for the moment that's the way norfair works. The reason behind that is because norfair can also work for keypoint detection, where each point might have a different score.
Maybe that is something that we should change in the future, so that one doesn't need to do that whenever you only have a single score for the whole object, it might be a little annoying to have to define the array of scores.

Makes sense, @aguscas ! Thanks so much for the quick turnaround!

I closed this issue, but feel free to open another one if you need more help


Create issue templates

This looks like a good first set of issue templates to me. I guess we might want to update / add more details later on, but this is a good start already. :+1:


Update rancher-compose-executor version v0.14.14

please update the commit with the version number.

failures seems unrelated

LGTM


Ability to zoom out

Wish I could zoom in or out of the browser to test certain things about the webUI im working on. But only way to zoom the webrowser is to zoom in or out of VS code which I dont want.

Can you describe it more clearly? By using images or using gif (video)?
Because I don't understand it much.
Thank you!

Understood. I will try to do this.
Thanks for your suggestion!


add interpolated strings delimited by double quotes.

interpolated strings should accept all escapes that single quoted strings accept, plus string interpolation using a # character.

added in v0.2.1


Fix so that openidc does not run when only federation is enabled

The openidc tasks were ran anyways if any type of federation was enabled.
This fixes that problem by requiring openidc to be enabled.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test


Internet Adapter properties change every time tool is ran

Describe the bug
Every time I run the tool and check my Internet Adapter settings, They are gone and need to be re-added else I can't access any website/
To Reproduce
Steps to reproduce the behavior:

Run the tool
Go to 'tweaks'
Select 'Run O and O Shutup'
Run tweaks
Restart
Try and access any website
See error

I'm unable to replicate this error.


Cleanup Rust fallback functions

For each of the routines that is implemented in platform-specific assembly, there are Rust fallback routines (originally C fallback routines) that are used when rav1d is built with asm disabled. A large part of this code is still raw c2rust output because cleanup of these routines wasn't necessary during the initial phases of development (initial development specifically focused on asm support, so the fallback routines weren't necessary).
Cleanup of these routines is also more difficult than other transpiled code because these routines need maintain an unsafe extern "C" interface since they replace unsafe asm calls. This often means that arguments can't be made fully safe, limiting how safe the body of the function can be. In general we try to split these functions into two parts:

foo_c_erased - An unsafe extern "C" function that acts as the fallback an asm routine.
foo_rust - A (hopefullly) safe implementation of the fallback routine.

The _c_erased variant takes the raw arguments that are passed to the asm routine and converts them to safe arguments to then pass to the _rust variant. This isn't always possible, though, and we may need to use unsafe arguments/operations in the _rust version.
The following files still need to be cleaned up:

[x] #814
[x] #815
[x] #816
[x] #817
[x] #818
[x] #819
[x] #820

The work covered by this task is now being tracked by the more general https://github.com/memorysafety/rav1d/issues/839 and https://github.com/memorysafety/rav1d/issues/835 issues.


Circle defined by center and diameter

Circle::new() creates a circle by center and radius. Is it useful to be able to create one by giving a diameter instead? The diameter can then be an odd number, however won't be symmetrical around the circle's center point. Does this matter?

I think it would be easier to use the top left point instead of the center as a parameter. This would for example make it easier to draw a circle inside a rectangle.
Or if we decide to add a Rect (see #141) type this could also be used instead.

I've just stumbled on this issue.
Current Circle defined by center and radius only allows building circles with an odd diameter (2 * radius + 1). It is impossible to build circles with an even diameter. We can only build half of the possible circles. This is an issue when you want to draw a circle centered on a display with an even width and height (pretty much any display in existence ?).
This is related to #182. With a coordinate system based on the center of pixels and a circle defined by its center, there is no way to build a circle with an even diameter. One solution would be to change the coordinate system to lie on the grid between pixels, but then we couldn't build circles with an odd diameter.
I think the best way to solve this is, as suggested by @rfuest, to define a circle by its top left point and its diameter. This allows building both odd and even diameter circles.
I'm willing to implement this, but I'm wondering what's the best way to do it:

keep the current Circle and add a new CircleDiameter (or a better name) ? (I don't really like this)
change the implementation of Circle breaking API compatibility (in 0.7.0 I guess) ?
something else ?

I'm leaning toward the second solution. What do you think ?

I would also prefer the second solution. Having two types to represent circles would be confusing and unnecessary because all circles can be represented using the new type.
Changing the implementation is a breaking change, but the new type can provide additional constructors and getters to make porting old code easier. Some getters like center would only be able to output rounded values, but I guess that can be expected when not using floating point coordinates.
One focus of 0.7 will be geometry (like #182 you mentioned), so this will be a good time to implement this change.

I'm on the fence about Ellipse being a separate shape or not. Here's my thinking on an API that supports ellipses and circles at the same time:
let center = Point::new(20, 20);

// Circle
let circle = Ellipse::new(center, Size::new(10, 10));

// Ellipse
let ellipse = Ellipse::new(center, Size::new(25, 10));

Thoughts?

I'm not sure if they should be separate shape or not, but if we add support for Ellipse, it should be defined by top-left and bottom-right rather than center and two "radius", for the same reason discussed just above.

Oh doy yeah, should've read the issue a bit slower...


Initial list of features is broken

When you first load the site listing all the features, it incorrectly shows the unofficial features as well. If you check then uncheck the unofficial check it works as expected
Steps:

Click settings to see list of features

Expected:
List of features and percentage scoring to respect the default settings with unofficial unchecked
Actual:
Unofficial is unchecked but they are still listed. Check and uncheck unofficial and it fixes itself.
This has just started happening recently.

@dstorey Good find, thanks for pointing that out. Should be fixed now, can you confirm?

@fyrd Confirmed. Thanks!


Topics for future Code Corners

A list of ideas/suggestions for future Code Corners, roughly grouped by themes. The ideas that are here and unchecked haven't been scheduled yet. Once they're checked, they should be scheduled on the Code Corner Google calendar.
More than one idea can be scheduled for a single Code Corner if there is enough time.
Add a comment to this issue to express your interest in one of these topics or to suggest new ones.
Code quality:

[ ] Version control with git/github. Best practices.
[ ] Testing (py.test, nose, etc.) and Test-driven development
[ ] Debugging interactively
[ ] Profiling and optimisation: how to make code faster
[ ] Parallelisation: multiprocessing, mpi4py
[ ] Graphical User Interfaces (GUI): qt/tkinter

Python tools/packages/techniques:

[ ]  Package Management and Installation (anaconda, pip, etc.)
[ ] Working with astronomical data/quantities: introduction to astropy (units, coordinates,...)
[ ] Introduction to Numpy
[ ] Working with tables: astropy, pandas
[ ] Wrapping Python with Fortran & C code
[ ] Data visualisation: Making nice plots with matplotlib
[ ] Data visualisation: beyond matplotlib (bokeh, seaborn, ...)
[ ] Exploring data interactively
[ ] ...

Other tools:

[ ] Mounting remote drives
[ ] Making life easier with sshr: using ssh config file, ssh keys to avoid typing the password

I'd be very interested to hear about the following ones:

Version control best practices
Testing
profiling and optimisation
parallelisation
wrapping Fortran & C
visualisation beyond matplotlib

I'm happy to lead version control and testing, and may have some input in the profiling one but hasn't done it for a while now so would be more interested in looking at other people's solutions.

Great suggestions!
We already have in the plans a discussion on version control, will come back with further ideas.
Brigitta: would be great if you want to talk about testing. What do you have in mind for that?

On 8 Nov 2017, at 16:54, Brigitta Sipocz<EMAIL_ADDRESS>wrote:
I'm happy to lead version control and testing, and may have some input in the profiling one but hasn't done it for a while now so would be more interested in looking at other people's solutions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/IoA-coding/codecorners/issues/1#issuecomment-342860953, or mute the thread https://github.com/notifications/unsubscribe-auth/AFvcgrJMopwjcuZbIpIQhJLy3MiBNuJ6ks5s0c64gaJpZM4PzzZ1.

The concept of unit tests via the example of using pytest, and mention of CI services.
It either can be a notebook based simple example (which format usually ends up being not much more useful than the "hello world" intros, but that's what you can fit in 20 minutes), or could just dive into astropy's or some other package's test suite to show how it's done there.

As a shameless plug, I've recently open-sourced a system I developed at the IoA for turning command-line applications (e.g. python scripts) into web-based job systems.
Screencast (source code):

It was developed to make it easier to share (via URLs) and track (timestamps, users, outputs, etc.) command-line applications. I'm using using it in production to completely automate data requests.
I don't think the project is astronomy-related enough to merit a full codecorner tallk, but I'm mentioning it here because if anyone in the IoA likes the look of it and thinks it will save them some time or make sharing their code (e.g. at conferences, collaborations) easier then I'd be keen to do one-to-one sessions explaining how to set it up and work with it. I'm personally interested in seeing how people use it, what needs to change, how to make it easier, etc.

@adamkewley wow fantastic! I would personally be very interested in learning about Jobson. I was trying to achieve something similar with Django+celery but never got it to work. I think several people have repetitive task to be applied on data, and keeping track of them as they get executed is really useful. I quickly watched the video but I'd be happy to meet to understand better how it works and the typical use-case. I think it would be suitable also for a code corner, but I'd like to have a better look at it.

Hey @mtazzari , I'm glad you like the look of it. I'm free most of the time during the work week so I'm flexible to what you want. I'll PM you my IoA email but just set a time and I can come to your office or something.

Hi @mtazzari, can I request a code corner on github and generally version control?


Is it possible to scan more than one barcode at a time?

Basically, I was to check if it possible to obtain the data on two barcodes in the camera field of view?

make it true in your component.
 reactivate={true}


can i create application/ld+json in reactjs JSX  For seo

can i create application/id+json in reatjs jsx google seo

Yes! And you don't need react-helmet to do it. react-schemaorg is probably what you're looking for.


Node.ovc service fails silently if port forwarding on cloudspace already exists

If port forwarding exists on a cloudspace to a certain vm and I try to create new vm with the same port forwarding it does not fail and still marks the execution as succeeded. The result is the new virtual machine is created successfully but the port forwards are not there.

validated on 8.2.0 it gives an error
RUN:0bf37c7950a13628ccaabbc75469831d
step:1 (ok)
- sshkey                    09bcfa8d2a                     | install         (ok)
- vdcfarm                   auto_71                        | install         (ok)
- g8client                  islam                          | install         (ok)
step:2 (ok)
- account                   ashraf                         | install         (ok)
step:3 (ok)
- vdc                       testdocker                     | install         (ok)
step:4 (error)
- node.ovc                  testforward                    | install         (error)
step:5 (new)
- os.ssh.ubuntu             testforward                    | install         (new)

@ashraffouda could you please add your blueprint?

sshkey__09bcfa8d2a:
g8client__islam:
url: 'be-g8-4.demo.greenitglobe.com'
login: 'ashraf'
password: 'ashrafashraf'
account__ashraf:
accountusers:
- 'ashraf'
vdc__testdocker:
description: '8b12ace3b2'
g8client: 'islam'
account: 'ashraf'
location: 'be-g8-4'
uservdc:
- 'ashraf'
node.ovc__testforward:
description: 'fa645a4342'
os.image: 'Ubuntu 16.04 x64'
bootdisk.size: 10
sizeID: 2
vdc: 'testdocker'
ports:
- '2200:22'
actions:
- action: 'install'


Animated Diagram

@objectadjective @morganmfritz446 @therenee Can we get a review on this, pleashe?

@therenee

This week we can make the animated diagram follow the colors of the current website. Later we'll update for the new visual design.

Changes to be done:

Match colors to style guide
add descriptive text


How to ban a user

Banning of a user
I don't know if I am just dumb or if there is no feature for it but it seems to me that there is no method to ban a user from a discord server. I see one for kick but no ban.
If anyone would point to me what class it would be in (ie. SocketUser, SocketGuildUser, RestUser). I found a class called RestBan but I couldn't create a constructor because of the security

Banning a user would help me a lot in my application and I would seriously like to be pointed to, or request this feature.

~bwookzy

SocketGuild.AddBanAsync() ?

God damnit I didn't think that it would be in the server...
I'm stupid and thank you @Mezo

Close the issue? @bwookzy

Also, next time ask minor questions like these on Discord API#dotnet_discord-net and leave the issues to actual bugs in the lib.


ingressgateway 503 when create a AuthorizationPolicy

Bug Description
When I create a AuthorizationPolicy,
apiVersion: security.istio.io/v1beta1
kind: AuthorizationPolicy
metadata:
  creationTimestamp: "2022-08-22T07:14:34Z"
  generation: 2
  name: ts-776dd07e
  namespace: dev
  resourceVersion: "155397728"
  uid: 8c46e0ad-3222-4c1c-9701-282f1e8c996f
spec:
  action: DENY
  rules:
  - to:
    - operation:
        methods:
        - GET
        - POST
        paths:
        - /hello
        - /hello2
  selector:
    matchLabels:
      serviceName: ts

and visit th service ts from ingressgateway, the gateway logs is
[2022-08-22T10:26:04.466Z] "GET /hello HTTP/1.1" 503 UC upstream_reset_before_response_started{connection_termination} - "-" 0 95 1 - "<IP_ADDRESS>" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36" "defeddf2-694f-9da9-94d2-20a941170f10" "ts.dev.ingw.cloud" "<IP_ADDRESS>:12417" outbound|12417||ts.dev.svc.cluster.local <IP_ADDRESS>:57086 <IP_ADDRESS>:8080 <IP_ADDRESS>:29691 - abc

and the ts log is
[2022-08-22T10:34:12.821Z] "- - -" 0 - - rbac_access_denied_matched_policy[ns[dev]-policy[ts-776dd07e]-rule[0]] "-" 2879 943 1 - "-" "-" "-" "-" "<IP_ADDRESS>:12417" inbound|12417|| <IP_ADDRESS>:56622 <IP_ADDRESS>:12417 <IP_ADDRESS>:33482 outbound_.12417_._.ts.dev.svc.cluster.local -

Maybe because of the mtls?  looks no path and no method from the ts log.
When I set mtls mode DISABLED, it's work fine...
Version
➜  ~ istioctl version
client version: 1.13.5
control plane version: 1.13.3
data plane version: 1.13.3 (17 proxies)
➜  ~ kubectl version --short
Client Version: v1.21.11
Server Version: v1.20.8

Additional Information
No response
Affected product area

[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[X] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Control Plane Revisions

Is this the right place to submit this?

[X] This is not a security vulnerability
[X] This is not a question about how to use Istio

Looks like the service is configured as TCP but you are doing HTTP maybe?

Yeah, I don't set the protocol. Thanks


re-add the interconnection report as one of the rotating homepage elements

I think we removed it because we didn't want the observatory to be front and center, but the report still should be I think.
cc @collina @sloane101 @critzo

Resolved in https://github.com/m-lab/m-lab.github.io/commit/a3babb8ef40924c1cf8e383283e34022bbcf6234


Create MoveIt! specific launchfiles

This PR adds launchfiles that sets up the planning contexts for MoveIt! and opens the MoveIt! RViz interface.

Would it make sense to put this somewhere in blue_moveit rather than blue_core?
We've generally been trying to keep blue_core as barebones/minimal as possible.

I put this in bringup because it looked like the package used to starting the arm, so likewise this is for starting the arm with a working MoveIt interface.
I'll defer to the team's decision though (:

That makes sense, hm...
Perhaps we could make a blue_moveit_bringup package within the blue_moveit metapackage? It's nice to compartmentalize these things.
@wuphilipp can make the final call ☎
(we should also make sure to document all of this)

I agree. I've gone ahead and moved these files in this pull request https://github.com/berkeleyopenarms/blue_moveit/pull/7. We should add documentation to run this.


wrong color space and range convert when filter graph use "format=rgba"

input yuv444p bt709 tv range，convert to rgba format，output rgba color looks fading
ffmpeg complain when calling av_buffersrc_add_frame_flags :
[src_0 @ 0000026ab60be580] Changing video frame properties on the fly is not supported by all filters.
[src_0 @ 0000026ab60be580] filter context - w: 1080 h: 1080 fmt: 0 csp: unknown range: unknown, incoming frame - w: 1080 h: 1080 fmt: 0 csp: bt709 range: tv pts_time: 0.000989583

maybe input context unspecific color space and range

Thanks for your feedback, we will check and fix it as soon as possible

`    nlohmann::json graph_para = {
{"dump_graph", 0}
};
auto graph = bmf::builder::Graph(bmf::builder::NormalMode,
bmf_sdk::JsonParam(graph_para));
nlohmann::json decode_para = {
    {"input_path", "D:/work/bmf_example_files/big_bunny_10s_30fps.mp4"}
};
nlohmann::json enc_para = {
    {"output_path", "D:/Documents/Downloads/result.mp4"}
};
auto decoder = graph.Decode(bmf_sdk::JsonParam(decode_para));
auto rgba = graph.FFMpegFilter({ decoder[0] }, "format", "rgba");
graph.Encode(rgba[0], decoder["audio"], bmf_sdk::JsonParam(enc_para));
graph.Run();`

before encoder, rgba Tensor is the same as raw. There is no problem when using ffmpeg cmd: ffmpeg -i raw.mp4 -filter_complex "[0:v]format=rgba[rgba];[rgba]format=yuv420p[yuv420p]" -map "[yuv420p]" output.mp4

So, do you mean it's still fading after BMF encoding? What's the csc range of the encoded video?

before encoder, rgba Tensor is the same as raw. There is no problem when using ffmpeg cmd: ffmpeg -i raw.mp4 -filter_complex "[0:v]format=rgba[rgba];[rgba]format=yuv420p[yuv420p]" -map "[yuv420p]" output.mp4

So, do you mean it's still fading after BMF encoding? What's the csc range of the encoded video?

yes, it still fading after BMF encoding. before encoding csc = CS_RGB, cr= CR_JPEG

update debug result:
before encoder avcodec_send_frame，dump AVFrame to yuv binary，result shows Cr data offset is Wrong

Could you please try bellow PR to verify whether the issue was resolved?
https://github.com/BabitMF/bmf/pull/162

After some research, if you need to convert from RGB to YUV (BT.709), you need to explicitly specify the color space. You can use either the "format" filter of FFmpeg 7.1 or the "scale" filter with some parameters to do this.


Just a question ...

Hi Maxime,
did you had the time, if your package is compatible to Julia v1.9.0?
I sthe following relevant for the package?
https://github.com/JuliaLang/julia/blob/v1.9.0-beta4/NEWS.md#delimitedfiles
Regards,
Stefan

Hi Stefan,
Thanks for bringing this to my attention.
I'll have to make an adjustment for v1.9.0 compatibility. The parameteroptimisation function (among others) has a method where measurements are loaded directly from the CSV files. One of my priorities for this package will be to find a replacement (while avoiding an additional dependency) for the DelimitedFiles module, which will no longer be in Julia Base.
Regards,
Maxime


Update all remaining legacy SQL queries to use the new syntax

User Story - Business Need
SQLAlchemy v1.4 supporters two syntaxes for making database queries, but SQLAlchemy v2 only supports the newer syntax.  Before we can update to v2, we need to convert all instances of legacy syntax to the newer syntax.
This work began with #1538.  This ticket should finish the job.
User Story(ies)
As a VA Notify developer
I want to update legacy database query syntax to the new version
So that we can upgrade to SQLAlchemy v2.

[x] discuss w/ QA

Additional Info and Resources

SQLAlchemy v2.x changelog
Migration Guide

app queries

Engineering Checklist

[ ] Review relevant upgrade/migration docs before getting started

IMPORTANT NOTE: Legacy queries generally start with <model class>.query.  Depending on context, updated syntax might use a new style query (i.e. stmt = select(Notification).where(...) followed by an execution statement) or an updated ORM call (i.e. replace Notification.query.count() == 1 with notify_db_session.session.get(Notification, notification_id) is not None, or write a statement to select the count).

[x] Refactor all query statements the match these regular expressions, which overlap significantly:

[x] grep -rn -E "\.query(\.|\()"
[x] grep -rn filter_by (not all matches are part of queries)

[ ] Update the following ./app/ queries:

[x] app/dao/complaint_dao.py
[x] app/dao/services_dao.py
[ ] The remaining ./app/ queries need a TODO: comment and to be deleted, ensuring the methods return appropriate values.

[ ] TODO comments have tickets listed. These should be cleanup tickets. We do not use fido2, organisations, nor letters.

[ ] Collect unit tests that need updating and/or investigation into a new ticket. No changes to tests are required for this task!

We sometimes see unit tests that query SMS and email but only add notifications for one or the other. Or there are multiple services queried but the test only does one. Add a TODO comment and add these to a ticket, so we can investigate them.

Acceptance Criteria

[ ] No legacy queries remain
[ ] All unit tests pass

QA Considerations

[ ] QA regression passes
[ ] Pay extra care to JSON responses
[ ] Performance Testing to ensure there are no significant slowdowns to investigate

Potential Dependencies
Leave blank if n/a
Out of Scope

Do not upgrade any dependency.  Separate tickets exist for those upgrades.
Do not update unit tests as part of this

Hey team! Please add your planning poker estimate with Zenhub @cris-oddball @EvanParish @k-macmillan @ldraney

I upgrade all remaining legacy queries in app/ as part of the first task (the grep statements).  Ergo, the TODO task became a non-issue.

Needing @kalbfled to check the AC boxes off in this ticket. Will look at performance on Monday.


Integrated terminal wrapping issue

VSCode Version: insiders 6/30
OS Version: MacOs

Everything still OK:

Now enter c and the prompt string is wrapped:

@kieferrm do you have any terminal or font family/size/lineheight settings in settings.json? Can you also copy and paste the output of echo $PS1

No special settings. Here is the output: \s-\v\$

Duplicate of https://github.com/Microsoft/vscode/issues/9902


Upgrade various jar, gem versions for maintenance

looks good to me, will merge it.
QQ : How did you decide on those specific version changes in GemFile ?

bundler auto manages that, when I bumped up the one dependency (nokogiri).
I think we can merge the other PRs, cut a release and then land this.?  (to guard against any issues from these jar version upgrades)

We plan to merge this after a release is made.


ci/cd: Add a pre-deploy to use release reqs file

When we cut a release for Tern, we gather the direct and transient
dependencies via pip-compile. We copy the output of this command
to docs/releases/vx_x_x-requirements.txt in order to maintain
reproducibility for the future. However, when we build and release the
Tern PyPI package there is a chance that the requirements may change
between when we cut the release and when we build and release the
package to PyPI since the official requirements.txt does not pin to
exact versions and only lists the direct dependencies.
This commit adds a step to the deploy process that copies the
docs/releases/vx_x_x-requirements.txt file to requirements.txt so that
pbr and PyPI will package Tern using the same requirements recorded
when the release was cut.
Resolves #389
Signed-off-by: Rose Judge<EMAIL_ADDRESS>

Looks good to me. Merging.


dmg2img gchdm_203_970.dmg is corrupted

Hello, I'm trying to follow this guide https://obsproject.com/forum/resources/streaming-on-linux-using-elgato-game-capture-hd-and-obs-studio.294/ to install this driver so I can use my Elgato Game Capture HD game capture device in my Linux Mint OS, but, after doing "wget http://files.elgato.com/gamecapture/gchdm_203_970.dmg" then "dmg2img gchdm_203_970.dmg -o gchdm_203_970.dmg.img" I get a message saying "ERROR: dmg image is corrupted".
Also, this link https://www.elgato.com/en/game-capture-hd-support on this page https://github.com/tolga9009/elgato-gchd/wiki/Firmware just leads to the Elgato site's home page.
So what do I do? How do I get my Elgato Game Capture HD working on Linux? The resources and guides I'm seeing are not working properly in 2024.

Hi there,
the Wiki page here and the guide you were following were updated 2016, the links are dead. The oldest driver version I could find for Mac was 2.0.5, seems like they've pulled 2.0.3 from their website. You can still try to extract firmware files from the Windows driver. According to our Wiki, you should get version 3.2 here:
https://edge.elgato.com/egc/windows/egcw/3.20/final/GameCaptureSetup_<IP_ADDRESS>3_x64.msi
I have found the download link here, if you need to try another version: https://help.elgato.com/hc/de/articles/360027964072-Elgato-Game-Capture-HD-Software-Release-Notes-Windows
If it doesn't work anymore, I can't fix it or provide further support unfortunately. I'm very sorry, but it's been a decade since I worked on this software. I have marked the repository as discontinued a long time ago. Also please keep in mind, before you put too much effort in this: this is proof-of-concept at best and is overall a frustrating experience.
Cheers,
Tolga


More Colors, Bug Fixes and Transitions

More Colors (Buttons, Inputs)
Transitions (Button hover, Input Focus)
Button disabled Bug fix
Textarea only vertical resize (Why is more than 100% width possible?)

This looks cool. Give a some time to pull it down and mess with it. I'll circle back again on Monday.

@enlore Thanks. I will add some more functions to use colors soon. Like Text Colors, Row Colors, etc.

Maybe alerts should be placed into bones...

@gld1982ltd I will keep this in the Pull Request. In bones we can add a Navbar, Tab View and Checkboxes as well. All pure CSS and as less as possible lines.

@enlore I'm waiting on your results. I'll check back in later.

Most of this looks good. I like the colors, the transitions on the form elements, and the texarea fix. The alert classes are nice, too, for creating content that really pops out. Good stuff.
 I'm looking for them to scroll horizontally at small screen sizes. Currently the cells stack:

Could you remove the display: block from td and th elements in the _tables.css module?
For extra credit, you could make a u-table-responsive class, too. We could use this class to wrap tables in a div to accomplish horizontal scrolling tables, a la responsive tables in Bootstrap.  If not, I'll put that together after I merge in your work. Either way, here's the intended result:

Also, we have utility classes that do text-align: right and text-align: center already. Having a text-left class isn't a bad idea though, so could you stick with our established pattern and just make a u-text-left class in core/_utils.css?
Lastly, drop the vendor prefixes and let Autoprefixer do its thing. I think adding a feature to the build process to target specific browsers wouldn't be a bad idea, but that's out of scope for this commit.
Thanks!

@enlore I will remove the text-center etc. today. And it would be helpful if you edit the table.css aftee the merge. Also we can work on an new work Process. Maybe with a Gulpfile.
It works pretty good and is used also in Other Projects. Easy to edit. Easy to build. Easy extandable.
Hopfully we move to Gulp.

@oltmannsdaniel We were using gulp, but for this project it is a little overboard. PostCSS is perectly capable of doing this without gulp. As I stated above, any build/development issues can be resolved in a different commit.

@enlore Can you go ahead and merge this with the table fixes you mentioned? I'll start fixing up the build system later today, and should be done with it by tonight to submit a PR.

Will do.
On Wed, May 11, 2016, 7:41 AM Gary DeLaney<EMAIL_ADDRESS>wrote:

@enlore https://github.com/enlore Can you go ahead and merge this with
the table fixes you mentioned? I'll start fixing up the build system later
today, and should be done with it by tonight to submit a PR.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/skeleton-framework/skeleton-framework/pull/46#issuecomment-218447341

--
Nick Lorenson
Dev @ HCA
m |<PHONE_NUMBER>
<EMAIL_ADDRESS><EMAIL_ADDRESS>Legal stuff: for our mutual protection, anything we send back and forth via
email is to be considered confidential unless stated otherwise.


Fix: Update size attribute for post-card-image to match actual image size in template

Fixes freeCodeCamp/freeCodeCamp#39043
Problem:
Images on news page has sizes attribute set in a way that it always serve image that is 1000px wide for every device resolution.
How it was solved:
size attibute is set according to how the image is actually displayed on page. Now it takes into account that for device resolution > 768px the image is only 285px wide.

Thanks @darek-grund, I've checked this out locally and it's working perfectly. @ahmadabdolsaheb could you double-check?
When I reviewed this, I noticed that unsplash images do not get scaled.  Your PR is fine, but I'd like to keep the issue open until we've decided what to do about that, so I will edit out the 'closes' bit.


TypeScript

It would be awesome if this lib were typed 😎

@nandorojo hey, just saw this — i'll see what i can do haha
(i'll update you once I've finished cleaning the code + TS rewrite)


Command Not Found

On any command I get this as a return
go-checksum: command not found
I have installed the go-checksum using the commands within the readme.

Is $GOPATH/bin  in your $PATH?
------------------ Original ------------------
From: CircuitBreaker<EMAIL_ADDRESS>Date: Fri,Mar 5,2021 1:11 AM
To: vikyd/go-checksum<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [vikyd/go-checksum] Command Not Found (#3)

I have export PATH=$GOPATH/bin:$GOROOT/bin:$PATH   and export GOROOT=/usr/local/go

Check $GOPATH/bin/go-checksum, if existing, try to use absolute path.
------------------ Original ------------------
From: CircuitBreaker<EMAIL_ADDRESS>Date: Fri,Mar 5,2021 2:11 AM
To: vikyd/go-checksum<EMAIL_ADDRESS>Cc: VikydZhang<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [vikyd/go-checksum] Command Not Found (#3)
I have export PATH=$GOPATH/bin:$GOROOT/bin:$PATH   and export GOROOT=/usr/local/go
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


Build tflite_runtime with custom ops

Click to expand!
Issue Type
Build/Install
Have you reproduced the bug with TF nightly?
No
Source
source
Tensorflow Version
tf 2.7.2
Custom Code
No
OS Platform and Distribution
No response
Mobile device
No response
Python version
3.7
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Hi,
Has anybody been able to build(cross-compile) tflite_runtime for aarch64 with custom ops?
I tried running for TF 2.7.2. Ref (https://github.com/tensorflow/tensorflow/tree/r2.7/tensorflow/lite/tools/pip_package)

Standalone code to reproduce the issue
CI_DOCKER_EXTRA_PARAMS="-e CUSTOM_BAZEL_FLAGS=--define=tflite_pip_with_flex=true"   tensorflow/tools/ci_build/ci_build.sh PI-PYTHON37   tensorflow/lite/tools/pip_package/build_pip_package_with_bazel.sh aarch64

Relevant log output
2023-01-06 09:48:04 (74.8 MB/s) - 'get-pip.py' saved [2569500/2569500]

/install/install_pip_packages_by_version.sh: line 24: /usr/local/bin/python3.7: No such file or directory
ln: failed to create symbolic link '/usr/include/python3.7/numpy': No such file or directory
The command '/bin/sh -c /install/install_pi_python3x_toolchain.sh "3.7"' returned a non-zero code: 1
ERROR: docker build failed. Dockerfile is at /home/ubuntu/tensorflow/tensorflow/tools/ci_build/Dockerfile.pi-python37

Hi @ankitmaurya001 !
Thanks for reporting this issue. I could replicate this issue in my local machine.
@arfaian!
Could you look at this bug.
Thank you!

Hi @mohantym @arfaian thanks for considering this issue. If I could get some guidance, I would really appreciate the opportunity to contribute to fixing this bug. Is that something that could be arranged? Just let me know and I'll do my best to help out. Thank you!

Hi @ankitmaurya001
As per the documentation, currently custom ops and delegates cannot be registered.

Caveats
You cannot use TensorFlow Select ops, only TensorFlow Lite builtins.
Currently custom ops and delegates cannot be registered.

Thanks.

Hi @pjpratik but one of the examples shows how to enable TF OP support.  Is it not supported for aarch64?

Hi @ankitmaurya001
The use of Flex OPs is given in the example. It is supported for aarch64 as well.
https://github.com/tensorflow/tensorflow/blob/056987333d10837ab3f087e86c50ba95d85fd77c/tensorflow/lite/tools/pip_package/build_pip_package_with_bazel.sh#L176
Can you try in latest version and let us know if the issue still persists?
Thanks.


Import gMock alongside gTest and give shorter aliases to the libraries. [phab:D1201]

Created by shahms at 2016-12-29 19:17:28:
Import gMock alongside gTest and give shorter aliases to the libraries.

schroederc accepted this revision at 2016-12-29 19:20:54

shahms closed via commit a599d27b8393bcbeb027a430296414767cd46bbf at 2016-12-29 19:21:49


Please contribute into XCTest framework

Hi guys!
You have wonderful output that looks like a good contribution to XCTest framework.
Why don't you create pull request into https://github.com/apple/swift-corelibs-xctest ?
I think apple could use your implementation of https://github.com/pointfreeco/swift-custom-dump dump and it could nicely fit into https://github.com/apple/swift-corelibs-xctest/blob/main/Sources/XCTest/Public/XCTAssert.swift#L173 to get more useful output.

Hi @AgranatMarkit, I'm going to convert this to a discussion since we use issues for bug tracking.
The short of it is I don't think Apple takes contributions like this into frameworks that it also maintains internally. If you hear differently, though (by opening an issue or discussion with Apple), let us know!


Fail to load theme if change to Modern

If you change the theme to modern and force update css, there is a library error.
How to reproduce:

force update of CSS file

app.use(sass({
  src: path.join(__dirname, 'public'),
  dest: path.join(__dirname, 'public'),
  force: true,
  debug: true
}));

change theme to modern

@import "themes/modern/variables";
...
@import "themes/modern/modern";

Error Output:
[sass]  error: File to import not found or unreadable: ../../lib/bootstrap-social.

The reference is to the first line of _modern.scss
@import "../../lib/bootstrap-social";

Other Information -
The bootstrap_social reference in default (which does not give an error) is:
@import<EMAIL_ADDRESS>
Which is also imported in main.scss
If you change the theme in main.scss and do not force update, it will not update the main.css and you will never see an error or change in styling.

Changing the modern import to
@import<EMAIL_ADDRESS>

which I thought would be the solution
leads to a further error
[sass]  error: Incompatible units: 'rem' and 'px'.

in /Users/jess/myproject/node_modules/bootstrap/scss/_variables.scss:445:41
Error: Incompatible units: 'rem' and 'px'.
    at options.error (/Users/jess/myproject/node_modules/node-sass/lib/index.js:291:26)

The full solution is to change the import for bootstrap social in modern/modern
and to change the $font-size-base to rem in modern/_variables


feat: support array of path with name (sync all method)

Can anybody pls merge this PR?

@alexmingoia @jbielick @wachunei

I have no merging permissions :(

@ZijianHe

Fine, just close it in favor of https://github.com/koajs/koa-router .


Import scanned score sheet filled out by human by image processing

The score sheet could be generated in a way that it would be easy to scan them into the system after creating them with #72 . This way we have the automation of the competition and the fail-safety of using paper score sheets as backups without having the organizational effort to manually type in score sheets to the system.

Regarding (refer issue 72)

Scan manually typed information? Handwriting recognition is hard :/
I was more thinking about boxes to be checked. The score sheets would need to be designed in a way that everything can be marked by checkboxes. Things like comments don't need to be digitalized, it's sufficient if the comment and signing area is just cropped and stored without actually OCR-ing it.
Most of the scoring sheets can be digitalized that way. The last 10 or something need to be checked by a human.
I think the biggest barrier between the scoring system and persons using it it the fact that mobile devices are necessary. Not everybody has the capabilities to borrow or buy like 10 devices. Utilizing paper would be the first step for everybody to accept the system. If they see that it generally works well they will consider using mobile devices.

Work in progress:
https://github.com/janblumenkamp/rcj-rescue-scoring/tree/scoresheet

This feature is included in the following version.
"2019-from-Japanse-version" branch.


SAP Screen Personas script open other tcode for get data

Hi Conrad,
In tcode VL02N (change delivery order), I can add other sales order through sub menu 'Outbound Delivery-Deliver Sales Order'.
In that sub menu, help for Order field is not suitable because I can't find order item and copy order item to 'Only from item' field.
So, I must open another tcode, i. e.: VL10C for get sales order number and order item. After that, I copy-paste sales order number and order item manually into VL02N.
Is it possible for create script button in tcode VL02N for open/running tcode VL10C? After user select/check/tick sales order- order item and press back button, then script will back to tcode VL02N with data of selected sales order-order item in VL10C and pass that data into submenu 'Outbound Delivery-Deliver Sales Order'.
Thank you for your help.
Best regards,
Andy Pataselano

Hello @cjbernal. This seems to be for you. Can you please have a look and change the assignee if necessary? Thank you.

Hi Michael,
This doesn’t exactly pertain to the tutorial, but I can answer the question. No worries. Thank you for forwarding it.
Kind regards,
Conrad
From: Michael Czadankiewicz<EMAIL_ADDRESS>Sent: Monday, February 3, 2020 2:16 AM
To: SAPDocuments/Tutorials<EMAIL_ADDRESS>Cc: Bernal, Conrad<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [SAPDocuments/Tutorials] SAP Screen Personas script open other tcode for get data (#4566)
Hello @cjbernalhttps://github.com/cjbernal. This seems to be for you. Can you please have a look and change the assignee if necessary? Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/SAPDocuments/Tutorials/issues/4566?email_source=notifications&email_token=AKJ6TIEDZTMBF3QKCACOJOLRA7VHBA5CNFSM4KPBJAW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKTIE4I#issuecomment-581337713, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKJ6TIGB5DICEVWIUIJH3J3RA7VHBANCNFSM4KPBJAWQ.

Hi Andy,
Thanks for checking in. Even though this is a little outside the tutorial, this is something we can definitely help you with.
Short answer: Yes, you should be able to do exactly what you suggested. You would simply use the script recorder to record those actions and attach that script to the button in VL02N.
If you have any difficulties with that, please let me know!
Kind regards,
Conrad

Hi Conrad,
Thank you for answering my question even though my question is not related to your tutorial.
I just upgrade SAP Netweaver related to SAP Support backbone change. During upgrade process, I must freeze server from coding and configuration until 2 weeks. I also must down production server almost 20 hours.
On my server, the version of Personas is 3 SP02. If I upgrade to SP10, do I have to follow the same procedure as for updating Netweaver?
In Personas wiki, I only found blog about SAP Screen Personas 3.0 Installation Guide, but I can't found blog about how to upgrade SP for SAP Screen Personas.
Best regards,
Andy Pataselano

Hi Andy,
Technically speaking, upgrading Personas is independent of the NetWeaver upgrade. It doesn’t require downtime, per se; but people using flavors while the upgrade is running may experience problems or even dumps due to syntax errors. This is because the upgrade is only partially done. The problems will be certainly gone once the upgrade finished. For this reason, it is advisable to perform the upgrade together with the NW upgrade, in the same downtime period.
The SAP Screen Personas upgrade process is much faster than the NW upgrade, although going from SP02 to SP10 will take a little longer than the usual one SP upgrade, since the upgrade queue is larger and pretty much all objects will be touched.
When upgrading from SP03 or below, existing flavors must be upgraded manually, as described in note 2465796.
Kind regards,
Conrad

Hi Conrad,
Thanks for the explanation. It's better if you add this problem to the SAP Screen Personas wiki.
Best regards,
Andy Pataselano