[WIP] Introduce notion of an asynchronous session commit

This is very much still a work in progress. Much more testing must be done before creating a full Pull Request, and there are a couple of corner cases that still will be problematic (such as MergeContent/MergeRecord with a Correlation Attribute set). Perhaps will have to introduce something into the Processor API that will indicate whether or not the Processor can be used in a given runtime engine (NIFI, MINIFI, STATELESS) with a given configuration, etc.
But making available for review/testing by others who may be interested
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Neat concept, it's easy to see the pattern of putting "clean-up" code that used to follow the commit() call inside the new commitAsync consumer.

Thanks @gresockj. Closed pull request in favor of https://github.com/apache/nifi/pull/5042. This PR has the contents of this branch plus additional commits, and is pushed against a different branch that represents the Jira number.


Minor: Use Expr::apply() instead of inspect_expr_pre()

Currently inspect_expr_pre() is a specaial wrapper around Expr::apply(), but all callsites of inspect_expr_pre() could use Expr::apply() directly. This PR (toghether with https://github.com/apache/arrow-datafusion/pull/9913) refactors all callsites of inspect_expr_pre() so technically it could be removed too. But because inspect_expr_pre() is a public function removing it would be a breaking change...

But because inspect_expr_pre() is a public function removing it would be a breaking change..

I think what we should do is deprecate it -- I can handle doing so in a follow on PR

Thanks @alamb for the review!


x-editable compatibility

When using both tablesaw swipe with x-editable to edit the fields of the table, it resets to show only the first n columns if i try to edit a field from a cell which is not originally visible, but gets visible after swipe. So that makes it impossible to edit a field in a column not in the first n visible ones.

I do believe he meant x-editable as in (http://vitalets.github.io/x-editable/index.html). Is it possible to use both of these "plugins" on the same table? I'm hoping to combine tablesaw, x-editable, and Datatables (http://www.datatables.net/) to combine the functionalities.

You are correct, I do mean this plugin. In my experience,  I could not get tablesaw and x editable to work without minor bugs/issues. I have been using Datatables with x editable for a while and it seems to be fine. Although I have also seen some minor issues when trying to make specific columns sticky (not scrollable). I would advise using only one of these with x-editable, preferably Datatables.

Thanks. Appreciate it.

Thanks @bilias23!
Closing as wontfix for now until more people drop in requesting this.


[sema] Fix error message for enums with an invalid raw type

Resolves https://bugs.swift.org/browse/SR-9019

@swift-ci Please smoke test

Hi @adamshin, thanks for the patch.
You should run the tests and make sure to update them with respect to the changes you’ve made to the diagnostic text.
Also note that CI requires privileges for security purposes.

Ah, thanks for the info. This is my first time contributing so I'm still learning the ropes. 🙂
I didn't see any tests that seemed to be affected by this change. Would I need to add a new one?

No problem.  I wrote a short explainer of some Swift-specific testing concepts that help here
A quick git grep reveals you should update the following files
test/Generics/inheritance.swift
test/IDE/print_ast_tc_decls_errors.swift
test/Parse/enum.swift
test/Sema/accessibility_private.swift
test/Sema/enum_raw_representable.swift

@swift-ci please smoke test


Parse FilterMode in gltf

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
support feature in #879

Codecov Report
Base: 44.06% // Head: 42.05% // Decreases project coverage by -2.00% :warning:

Coverage data is based on head (9094fb6) compared to base (6f113e9).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           dev/0.9    #1110      +/-   ##
===========================================
- Coverage    44.06%   42.05%   -2.01%
===========================================
  Files          343      343
  Lines        17954    17959       +5
  Branches      2493     2495       +2
===========================================
- Hits          7911     7553     -358
- Misses        9394     9803     +409
+ Partials       649      603      -46

Impacted Files
Coverage Δ

packages/loader/src/gltf/parser/TextureParser.ts
6.38% <0.00%> (-0.76%)
:arrow_down:

...ckages/core/src/shadow/CascadedShadowCasterPass.ts
18.55% <0.00%> (-29.39%)
:arrow_down:

packages/core/src/RenderPipeline/RenderContext.ts
71.42% <0.00%> (-28.58%)
:arrow_down:

...ges/core/src/RenderPipeline/BasicRenderPipeline.ts
23.61% <0.00%> (-26.39%)
:arrow_down:

packages/core/src/Camera.ts
34.01% <0.00%> (-21.83%)
:arrow_down:

packages/rhi-webgl/src/WebGLRenderer.ts
46.42% <0.00%> (-20.72%)
:arrow_down:

packages/core/src/SceneManager.ts
41.37% <0.00%> (-17.25%)
:arrow_down:

...ackages/core/src/input/keyboard/KeyboardManager.ts
34.72% <0.00%> (-13.89%)
:arrow_down:

packages/core/src/Engine.ts
68.80% <0.00%> (-13.77%)
:arrow_down:

packages/core/src/RenderPipeline/SpriteBatcher.ts
13.58% <0.00%> (-13.59%)
:arrow_down:

... and 30 more

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

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


Using official Universal Robot Client Library

This should solve #65

Any reason why not use the urscript in the client library?

There is still a problem with the client library and ROS2 which i couldn't solve yet. The lib works and compiles but since it is a cmake package it is not added to the ament_index. As a result we are not able to find the client library in any launch files where the urscript is specified. I found this pose( https://answers.ros.org/question/341870/ros2-pkg-list-show-cmake-packages/) which is roughly related but was not yet able to get the client library correctly registered on the ament_index. Therefore as a workaround for the moment I just updated the urscript to the most recent version in the driver itself where we can find it from launch files.


Add option to use a different KMS key when sharing

This project gave me a great starting point, so foremost thank you for that!
Just as reported in the sister project (eg: https://github.com/awslabs/rds-snapshot-tool/issues/60), my use case was to share a snapshot with a different account. Since some of the clusters are using the default KMS key this doesn't really work, as the destination account can never access the needed KMS key and therefore can't make a local copy.
To fix this I implemented an extra copy step in the source account (after the take snapshot) to bring all the snapshots to use the same KMS key and then share them. It's potentially less efficient as it generates an extra snapshot copy, but it makes the process generic after that point.
Maybe it's something that can be added to the project, but it would make the generic solution more complex.

I created a PR to add this functionality. We're using it in production at my company. Feedback welcome!
And like fschroder, I appreciate this project, which got my company about 80% of the way to where we needed to be with regards to a snapshot-copying solution.


[BUG] Pytorch frontend error when the value of prim::Constant is a tensor in cuda

Script to reproduce the issue:
import torch
from tvm import relay

class Foo(torch.nn.Module):
    def __init__(self, bar):
        super(Foo, self).__init__()
        self.bar = bar

    def forward(self, x):
        return self.bar + x

foo = Foo(torch.cuda.FloatTensor([[1.0, 2.0], [1.0, 2.0]]))
x = torch.cuda.FloatTensor([[1.0, 2.0], [1.0, 2.0]])
traced_foo = torch.jit.trace_module(foo, {'forward': x})
shape_list = [('input0', (2, 2))]
module, params = relay.frontend.from_pytorch(traced_foo, shape_list)

Fix is quite straightforward. Just need to convert the tensor to cpu when the tensor is in cuda at line https://github.com/jxx123/incubator-tvm/blob/master/python/tvm/relay/frontend/pytorch.py#L2174.
Will submit a PR soon.

PR link: https://github.com/apache/incubator-tvm/pull/6051


must-gather: Fix oc get command

remove quotes as shellcheck error
is ignored
Signed-off-by: crombus<EMAIL_ADDRESS>

/retest

/cherry-pick release-4.6


interface can't have more than 1 extends; class can't have both extends & implements!

Civet fails to transpile interface A < B, C to TS.
Civet transpiles class Q < W implements A, B as class Q extends W(implements(A, B)).
BtW, it'd be nice for Civet to have a shorthand for keyword implements as maybe <<:
class Q < W << A, B
https://Civet-Web.Vercel.app/#aW50ZXJmYWNlIEEgPCBCCiAgICBmb286IHN0cmluZwoKLy9pbnRlcmZhY2UgQSA8IEIsIEMgLy8gZmFpbHMKLy8gICAgZm9vOiBzdHJpbmcKCmNsYXNzIFEgPCBXCiAgICBmb286IHN0cmluZwoKY2xhc3MgUSBpbXBsZW1lbnRzIEEsIEIKICAgIGZvbzogc3RyaW5nCgpjbGFzcyBRIDwgVyBpbXBsZW1lbnRzIEEsIEIgLy8gVyhpbXBsZW1lbnRzKEEsIEIpKQogICAgZm9vOiBzdHJpbmc=

Adding implements to ForbiddenImplicitCalls should fix it being parsed as function application.
Changing InterfaceExtendsClause? to InterfaceExtendsClause* may fix the multiple extends for interfaces.


Manhunt USD capture freezes the game

When trying capture a Manhunt scene to USD via RTX Remix the game freezes.

Which Remix version - 0.1.1
Which GPU - 4090
Which GPU driver version - 531.61
Which game build - Steam build with crosire d3d8to9

Attach logs!
NvRemixBridge.log
d3d9.log
manhunt_d3d9.log
To Reproduce
Steps to reproduce the behavior:

Start the game (To access the full RTX Remix menu 640x480 is recommended)
Load any of the maps, sometimes this only works via the scene select menu
Try the USD capture
The game will freeze
The process has to be manually stopped

Expected behavior
The game will freeze when trying to capture the game. There will still be data written to the capture folder but it seems incomplete at random. (For example lights don't always show up) The game remains frozen even after a 20 min wait.
Screenshots

This has since been fixed and capturing works now


Update auto-generated documentation

Repository: openmicroscopy/ome-documentation
Already up-to-date.
Generated by build OMERO-DEV-latest-docs-autogen#1215.

--no-rebase

Merge build docs look fine, merging for the release prep


Extend spo list contenttype with option listUrl, closes #3810

Closes #3810

Superb PR @MathijsVerbeeck! We'll review it asap.

Merged manually!


Improve DNSRecord reconciliation

How to categorize this issue?

/area scalability
/kind enhancement
What would you like to be added:
We should improve the DNSRecord reconciliation such that it is only performed when really needed (similar to what we do for the Infrastructure reconciliation):

during shoot creation
in the maintenance time window
during shoot deletion if the record is not "healthy"

Why is this needed:

DNS APIs typically have quite low API limits, and especially on large landscapes where https://github.com/gardener/gardener-extension-shoot-dns-service is installed they might be hit quite fast when many shoots are reconciling.
Despite the fact that some of the DNSRecord extensions use custom rate limiter, this approach doesn't scale well.
Also, the DNS records for a shoot cluster are static during its lifetime, so there isn't much benefit of reconciling them again and again (just produces unnecessary API calls in 99,9% of the cases).

/assign


NASA fork sync.

Github has been adamant that the NASA forks is 66 commits ahead. It is not - as the 0 files changed here demonstrated.
This PR is an attempt to make Github stop complaining.

Look north son, you hear this howling that the winds carries? This comes from the land of Vermont, where souls are being tortured

We ran into this with a branch in another repo where it thinks the branch is ahead of main, even though it isn't (like your 0 files different indicates).  Turns out it sees every pull from main as a new commit and that cascades down if each of the commits you bring in has a pull from main...

Is there something we can do to keep that from happening in the future?

Well what I'll do going forward is to go NOAA/main ---(with website sync)---> NASA/main -----> NASA/geos/main hopefully using their own website interface will help.


sendPhoto method

Hello,
thank you for your library, I need help about the use of the method sendPhoto because I see that there's only one parameter, but, how can I use photo option?
Thank you.

I'm arrived here:
bot.sendPhoto({chat_id: 'id-here', files: {photo: {filename: 'filename-here.ext', stream: undefined}}}, function (err){
 if(err) console.log(err);
 console.log(" --> Image sent");
});

but i don't know what have to be placed in files.photo.stream

It doesn't work for me, I already have a png image in local but that written method example doesn't send it.

Works, thx.


build errors in resume function on line 2183

dd("coctx:%p", coctx) on line 2183 will cause build failed.

Don't know which file, so close the issue.


User limit

hi
I just installed haraka outbound and tries to set user limit   I used rate_limit plugins and since it require redis server so I commented the redis server section in  rate_limit.js file  and run the service  bt when I tries to send mail it shows
[CRIT] [C1D6142D-9594-49E7-BB86-5A0E9180DEC3] [core] Plugin rate_limit timed out on hook disconnect - make sure it calls the callback
plz guide me on this  attaching screenshot of files and errors

Actually I tried with redis server also bt it shows error thats why I  commented in rate_limit.js file
and guide me also how to set rate_limit.ini file to set user msg limit on haraka-outbound server

You need a working installation of redis. You can't just comment out code
willy-nilly and expect it to still work.
On Tue, Jul 28, 2015 at 7:42 AM, vivek779<EMAIL_ADDRESS>wrote:

Actually I tried with redis server also bt it shows error thats why I
commented in rate_limit.js file
and guide me also how to set rate_limit.ini file to set user msg limit on
haraka-outbound server
—
Reply to this email directly or view it on GitHub
https://github.com/baudehlo/Haraka/issues/1102#issuecomment-125570239.

ok I figure it out how to run rate_limit plugin with redis server and it runs
but whenever I tries to send mail to it it shows error
plugin_rate limit time out
plz guide me what mistake i am doing  attaching screenshot


Tag ShioRamen release commits

[x] choose labeling/versioning scheme:

example: shio-v0.1.0 where
the first 0 is the release version
the 1 is the feature? version
the second 0 is the maintenance version (for bug fixes)

[x] choose commit to label as "first" release:

just choose tip of main whenever you do it; we can fix bugs

[ ] communicate tag's presence to Fusion team

This process doesn't have a pull request associated with it, but the documentation for the full process is here.
I used commit ba4c8a6465a5bc909f09198f4c9447f0164c448b because GitHub showed it as the most recent commit on main to pass the automated tests. Tag info below:
git tag -a shio-v0.1.0 ba4c8a6465a5bc909f09198f4c9447f0164c448b -m "initial release commit"
git show shio-v0.1.0
git push origin shio-v0.1.0

Tag shio-v0.1.0 shows up in the GitHub remote now.

@tjanssen3 will you please communicate this tag's presence to the fusion team?

Left a message in the fusion-metrodr channel about this here


Agent running in a pod crashes from time to time

It runs fine most of the time but it will usually crash once a day and the pod never restarts, looking at the log I see the following:
datadog-skkcv — raise CheckException(
datadog-skkcv — datadog_checks.base.errors.CheckException: The agent could not connect to any of the following URLs: ['https://<IP_ADDRESS>:2379/metrics', 'http://<IP_ADDRESS>:2379/metrics'].
datadog-skkcv — Deprecated constructor API returned:
datadog-skkcv — __init__() got an unexpected keyword argument 'agentConfig'
datadog-skkcv — 2024-02-09 07:16:00 UTC | CORE | ERROR | (pkg/collector/scheduler.go:208 in getChecks) | Unable to load a check from instance of config 'etcd': JMX Check Loader: check is not a jmx check, or unable to determine if it's so; Python Check Loader: could not configure check instance for python check etcd: could not invoke 'etcd' python check constructor. New constructor API returned:
datadog-skkcv — Traceback (most recent call last):
datadog-skkcv — File "/opt/datadog-agent/embedded/lib/python3.9/site-packages/datadog_checks/etcd/etcd.py", line 74, in __init__
datadog-skkcv — super(Etcd, self).__init__(
datadog-skkcv — File "/opt/datadog-agent/embedded/lib/python3.9/site-packages/datadog_checks/base/checks/openmetrics/base_check.py", line 126, in __init__
datadog-skkcv — raise CheckException(
datadog-skkcv — datadog_checks.base.errors.CheckException: The agent could not connect to any of the following URLs: ['https://<IP_ADDRESS>:2379/metrics', 'http://<IP_ADDRESS>:2379/metrics'].
datadog-skkcv — Deprecated constructor API returned:
datadog-skkcv — __init__() got an unexpected keyword argument 'agentConfig'; Core Check Loader: Check etcd not found in Catalog

The problem gets solved by redeploying the pod, but this is very inconvenient.


Alternative notebook to convert WFM data to wavelength

Supersedes #53.
This is a different approach to computing wavelengths for WFM data.
Instead of runnig the simulation with one opening at a time, finding frame edges (removing outliers), then applying a time offset to each frame, we define a function that connects tof to wavelength.
This is done by plotting wavelength as a function of tof for the neutrons that make it to the detector

(this is the same figure as those made by scippneutron's chopper cascade module).
Then, we take the mean wavelength inside each tof bin, and use that as a 1d linear interpolator that computes wavelengths given an array of tofs

This is 100 times simpler than the old approach and yields very good results:

I think we should consider renaming the tof coordinate in this library to toa

I think I agree with this but this would break all the existing notebooks that users have. We should probably keep the old tof coordinate as well for some time period, to avoid too many issues, with a deprecation period?
Shame that the library is called tof... :grimacing:

In your calculation of err_wfm, I think some of the wavelengths are NaN (in between frames). Is this properly handled in what you did? (I don't know how cumsum handles NaNs)

I think we should consider renaming the tof coordinate in this library to toa

I think I agree with this but this would break all the existing notebooks that users have. We should probably keep the old tof coordinate as well for some time period, to avoid too many issues, with a deprecation period?

Yes that's a problem. But maybe a good opportunity for us to practice doing a proper deprecation cycle.

Shame that the library is called tof... 😬

No I think that's still a fine name!

In your calculation of err_wfm, I think some of the wavelengths are NaN (in between frames). Is this properly handled in what you did? (I don't know how cumsum handles NaNs)

That's handled in .hist. Note that we are histogramming a sc.Variable there, so if the variable is nan it doesn't fit in any bin and is dropped.

Even if you look at https://github.com/scipp/scipp/issues/3556 ?

If I histogram with enough bins, I can see the nans in there (see left edge in plot). They just don't show up in the other plots because they don't contribute much. This would be fixed by fixing scipp.

In your calculation of err_wfm, I think some of the wavelengths are NaN (in between frames). Is this properly handled in what you did? (I don't know how cumsum handles NaNs)

That's handled in .hist. Note that we are histogramming a sc.Variable there, so if the variable is nan it doesn't fit in any bin and is dropped.

Even if you look at scipp/scipp#3556 ?

Good to know! I assumed they were dropped


smaller font sizes in data view

System information:
DBeaver version 6.0.3
java.runtime.name=Java(TM) SE Runtime Environment
java.runtime.version=1.8.0_202-b08
os.name=Mac OS X
os.version=10.14.4
osgi.arch=x86_64
Connection specification:

PostgreSQL 9.3.24
PostgreSQL JDBC Driver 42.2.5
OpenVPN

Describe the problem you're observing:
Smaller font size at random cells
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

@biglaci is it still reproducible for you?

@uslss  sorry no, i didn't see this bug long time..

so ticket will be closed.
if it will reproduce again - feel free to reopen the ticket


Run rename-unsafe-lifecycles Codemod

This runs the rename-unsafe-lifecycles codemod from https://github.com/reactjs/react-codemod
I am planning to run each codemod in preparation for the React 16 console fix.

looks like I may have to bump to the latest version of 15 for this one to work


add build documents from source code

add build documents from source code to help users quickly use

Do you mean how to invoke the build? If so, you can run this to build and run tests:
./gradlew build

If you want to skip tests, run:
./gradlew build -x test

Would you like to open a PR to add those to the README?

Just started to get into iceberg. I figured to do a PR for this: https://github.com/apache/incubator-iceberg/pull/304

#304 is merged in.


Configure desktop view

Configured the page for desktop view:

Added desktop navigation
Updated styles for all sections
Added desktop-only elements

@Olamarx Hi. I don't understand what you mean by 'has a link in the live demo'. Could you explain further please? Thank you!


Fixing duration text for videos

The duration info in the list of videos is wrong when a video has a duration like 90 seconds it's displayed as 1 minute where it should be 1 minute and 30 seconds

Can I book a session please? For tomorrow?
Igor,

From: Andrey Pozolotin @.>
Sent: 06 October 2023 22:06
To: kubestaff/golearn @.>
Cc: Igor Morais @.>; Assign @.>
Subject: Re: [kubestaff/golearn] Fixing duration text vor videos (Issue #35)
Assigned #35https://github.com/kubestaff/golearn/issues/35 to @IgorM1996https://github.com/IgorM1996.
—
Reply to this email directly, view it on GitHubhttps://github.com/kubestaff/golearn/issues/35#event-10579735635, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCBVWBIILT47M2LAMKBO5RTX6BXEPAVCNFSM6AAAAAA5WLGH2KVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQGU3TSNZTGU3DGNI.
You are receiving this because you were assigned.Message ID: @.***>

sure plz contact me over chat

My apologies, but I don't have the chat due to extenuating circumstances. Once everything is sorted I will be on it.

From: Andrey Pozolotin @.>
Sent: 07 October 2023 16:17
To: kubestaff/golearn @.>
Cc: Igor Morais @.>; Mention @.>
Subject: Re: [kubestaff/golearn] Fixing duration text for videos (Issue #35)
sure plz contact me over chat
—
Reply to this email directly, view it on GitHubhttps://github.com/kubestaff/golearn/issues/35#issuecomment-1751736456, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCBVWBPPHD6SXCKFSUGQH73X6FXAJAVCNFSM6AAAAAA5WLGH2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJRG4ZTMNBVGY.
You are receiving this because you were mentioned.


[2.0.1] Split available_roles and roles_mapping into separate yaml documents …

…(#3097)

available_roles splitted into roles-mapping and roles-enabled
documents

roles-mapping added to the Init by default

No changelog and documentation updates needed? There is a bunch of references in code and documentation referring to configuration/feature-mapping:

After this merge, apply will be broken.

@sbbroot Could we pls make a feature branch for this for now and not merge it straight into develop?


Update org.semanticweb.hermit to <IP_ADDRESS>7

Updates net.sourceforge.owlapi:org.semanticweb.hermit from <IP_ADDRESS>2 to <IP_ADDRESS>7.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Need to stick with OWL API v4 versions of HermiT.


Java/Kotlin - Fix Revoke OAuth Token

Fixed the call to Revoke OAuth Token
https://nylas.atlassian.net/jira/software/c/projects/TDT/boards/57?selectedIssue=TW-3288

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

Project coverage is 80.13%. Comparing base (3ca6a84) to head (12624b7).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #242   +/-   ##
=========================================
  Coverage     80.13%   80.13%
  Complexity      287      287
=========================================
  Files            31       31
  Lines           861      861
  Branches         87       87
=========================================
  Hits            690      690
  Misses          121      121
  Partials         50       50

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

@mrashed-dev Done 😁


Fix #741, Document CFE_TBL_Unregister use-case

Describe the contribution
Fix #741 - Documents CFE_TBL_Unregister on shared table use case vs use by owning task.
Testing performed
CI - Documentation only
Expected behavior changes
None
System(s) tested on
CI
Additional context
#1509 documents potential races, global table handles often aren't protected so care needs to be taken when using shared tables.
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

CCB:2021-06-09 APPROVED

users should register at initialization


Support for sybase data types

Hello,
I'm trying to adapt turbodbc to sybase's (SAP) Adaptive Server Enterprise driver - while I have no problem using pyodbc, I really wanted to leverage pyarrow/numpy/asyncio functionalities that turbodbc offers.
It appears that for most data types everything comes out alright, however, I've found cases where Binary types are not well supported.
Errors found so far:
RuntimeError: Error! Unsupported type identifier for column timestamp @ NULLABLE SQL_VARBINARY (precision 8, scale 0))
Error! Unsupported type identifier for column claim_id @ SQL_BINARY (precision 16, scale 0))
Driver:
Adaptive Server Enterprise
p.s I really love this library, so thanks a lot for putting it together! On the flip side however, what are the chances of turbodbc joining efforts with the pyodbc team? (had to ask)

Hi! Thanks for reporting your findings! I will, however, close this issue as a duplicate of #56. The binary type is pretty much up the list of features I would like to add next, but time is an issue right now.
Concerning your meta-question, chances of the pyodbc team (essentially Michael Kleehammer) and the turbodbc team (essentially myself) joining forces are rather slim. The code bases are very different, here are just a few differences:

Modern C++ vs. C-style programming
Pybind11 vs. Python C-API usage
Batched data transfer vs. single-row statements
Well-developed build infrastructure vs. ad-hoc development/testing
Modern columnar API vs. standard row-based API

That's not saying that pyodbc is bad or anything; it is battle-tested, highly compatible and is used by many production systems. Turbodbc still has to play catching up in a few regards. It is, however, not easy to port over turbodbc achievements into the pyodbc codebase or vice versa.

@nrhy The Python C-API is relatively far away from object-orientation, let alone C++11. It also requires developers to handle reference counting, which is a real pain to get right.
Pybind11 allowed me to expose my C++11 code base to Python with little hassle and with the same code for Python 2 and 3. Previously I used Boost Python, which meant more hassle and did not easily allow me to test my code with both Python 2 and 3 on my Mac. I could have used other ways to write the driver, e.g., with cffi, ctypes, cython, but in the end I chose the tool that allowed me to write C++11 code with the least issues. Packaging sucked, though ;-).

i can imagine, thank you! :)


Create auto

share

share

This is not a valid Pull request. Please do not use this repo for your practice.


Border/drop shadow on Container element

Suggestion:
Perhaps we should try having the Container element (which we use quite a bit) give the enclosed elements a border and a drop shadow to make them stand out more.  The result I'm envisioning would look kind of  like Material Cards:
http://www.material-ui.com/#/components/card
@blukai

Tested but don't think this looks better.


[Candidate 2v2 Morning Dew ZH v7] Soldiers inside bunkers can damage themselves when firing

Observed behavior
Soldiers inside bunkers can damage themselves when firing.
Expected behavior
Soldiers inside bunkers do not damage themselves.
Videos
https://github.com/TheSuperHackers/GeneralsRankedMaps/assets/4720891/8b859d5f-834e-4be8-a2f6-bfed37873059
https://github.com/TheSuperHackers/GeneralsRankedMaps/assets/4720891/14266eec-ba1f-4f24-9aac-c3f3d1344ae0

those bunkers made it a bit too bunkery in my opinion, so I removed them altogether


Arithmetic operations are not supported in YAML, but appear in several tests.

Summary Title:
We have these in several tests. How are these not failing?
Description:

[ ] src/app/tests/suites/certification/Test_TC_CADMIN_1_3.yaml:                value: PIXIT.CADMIN.CwDuration * 1000
[ ] src/app/tests/suites/certification/Test_TC_CADMIN_1_4.yaml:                value: PIXIT.CADMIN.CwDuration * 1000
[ ] src/app/tests/suites/certification/Test_TC_CC_6_3.yaml:              minValue: ColorTempPhysicalMaxMiredsValue - 1
[ ] src/app/tests/suites/certification/Test_TC_FLW_2_1.yaml:              minValue: MinMeasuredVariable + 1
[ ] src/app/tests/suites/certification/Test_TC_REFALM_2_2.yaml:                value: PIXIT.REFALM.AlarmThreshold * 1000
[ ] src/app/tests/suites/certification/Test_TC_REFALM_2_2.yaml:                value: PIXIT.REFALM.AlarmThreshold * 1000
[ ] src/app/tests/suites/certification/Test_TC_REFALM_2_3.yaml:                value: PIXIT.REFALM.AlarmThreshold * 1000
[ ] src/app/tests/suites/certification/Test_TC_RH_2_1.yaml:              minValue: CurrentMinMeasured + 1
[ ] src/app/tests/suites/certification/Test_TC_S_2_2.yaml:                value: maxScenes - 1
[ ] src/app/tests/suites/certification/Test_TC_S_2_2.yaml:                value: maxScenes - 1
[ ] src/app/tests/suites/certification/Test_TC_S_2_2.yaml:                value: maxScenesMinusOne / 2
[ ] src/app/tests/suites/certification/Test_TC_S_2_2.yaml:                value: maxScenesMinusOne / 2
[ ] src/app/tests/suites/certification/Test_TC_S_2_2.yaml:                value: fabricCapacity - 1
[ ] src/app/tests/suites/certification/Test_TC_S_2_2.yaml:                value: fabricCapacity - 2
[ ] src/app/tests/suites/certification/Test_TC_S_2_2.yaml:                value: fabricCapacity - 2
[ ] src/app/tests/suites/certification/Test_TC_S_2_2.yaml:                value: fabricCapacity - 3
[ ] src/app/tests/suites/certification/Test_TC_S_2_3.yaml:                value: maxScenes - 1
[ ] src/app/tests/suites/certification/Test_TC_S_2_3.yaml:                value: maxScenes - 1
[ ] src/app/tests/suites/certification/Test_TC_S_2_3.yaml:                value: maxScenesMinusOne / 2
[ ] src/app/tests/suites/certification/Test_TC_S_2_3.yaml:                value: maxScenesMinusOne / 2
[ ] src/app/tests/suites/certification/Test_TC_S_2_3.yaml:                value: fabricCapacity - 1
[ ] src/app/tests/suites/certification/Test_TC_S_2_3.yaml:                value: fabricCapacity - 2
[ ] src/app/tests/suites/certification/Test_TC_SWTCH_2_1.yaml:              maxValue: NumberOfPositionsValue - 1
[ ] src/app/tests/suites/certification/Test_TC_TCTL_2_1.yaml:              maxValue: MaxTemperatureValue - MinTemperatureValue
[ ] src/app/tests/suites/certification/Test_TC_TMP_2_1.yaml:              minValue: CurrentMinMeasured+1
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_1.yaml:              maxValue: AbsMinCoolSetpointLimitStep5 - MinSetpointDeadBandValue
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_1.yaml:              maxValue: OccupiedCoolingSetpointValue - MinSetpointDeadBandValue
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_1.yaml:              maxValue: MinCoolSetpointLimitValue - MinSetpointDeadBandValue
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_1.yaml:              maxValue: MaxCoolSetpointLimitValue - MinSetpointDeadBandValue
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_2.yaml:          value: MaxCoolSetpointLimitValue + 1000
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_2.yaml:          value: MaxHeatSetpointLimitValue + 1000
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_2.yaml:          value: MaxCoolSetpointLimitValue + 1000
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_2.yaml:          value: MaxHeatSetpointLimitValue + 1000
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_2.yaml:          value: MaxHeatSetpointLimitValue + 1000
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_2.yaml:          value: AbsMaxHeatSetpointLimitValue + 1000
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_2.yaml:          value: MaxCoolSetpointLimitValue + 1000
[ ] src/app/tests/suites/certification/Test_TC_TSTAT_2_2.yaml:          value: AbsMaxCoolSetpointLimitValue + 1000
[ ] src/app/tests/suites/certification/Test_TC_WNCV_4_3.yaml:          value: attrCurrentPositionLiftPercent100ths / 100
[ ] src/app/tests/suites/certification/Test_TC_WNCV_4_4.yaml:          value: attrCurrentPositionTiltPercent100ths / 100

Steps to reproduce:
grep -E '(minValue|maxValue|value):\s*[a-zA-Z0-9.]+\s*[-+/]+\s[a-zA-Z0-9.]' src/app/tests/suites/certification/*.yaml

Answer - they're not failing because arithmetic operations ARE supported, but only if you put a space between the operation and the variable. And we just didn't write that down anywhere, so they mysteriously fail if you don't do it right, and it therefore looks like it's totally unsupported.

Closing this, will open issue to track the problem with spaces.


fix: 避免azure卸载磁盘失败

这个 PR 实现什么功能/修复什么问题:
避免azure卸载磁盘失败
是否需要 backport 到之前的 release 分支:

release/2.13

/area region
/cc @zexi

Codecov Report

Merging #6619 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6619      +/-   ##
==========================================
- Coverage    7.16%    7.16%   -0.01%
==========================================
  Files         647      647
  Lines      109335   109335
==========================================
- Hits         7831     7829       -2
- Misses     100904   100905       +1
- Partials      600      601       +1

Flag
Coverage Δ

#aFlag
7.16% <ø> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/appsrv/response.go
66.17% <0.00%> (-2.95%)
:arrow_down:

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 d236823...d4b1666. Read the comment docs.


install ruff in dev environment

installs ruff inside the poetry environment

pins the ruff linter/formatter version (prevents unrelated changes in PRs due to changes in lint rules or formatting)
consistent linting/formatting for all contributors
the ruff vscode extension prefers ruff version installed in environment to bundled version
allow contributors to call ruff manually (poetry run ruff ...)

what needs to be updated to get the flake.nix jobs working? these are a complete mystery to me

This project uses Nix for setting up the dev environment including tools like pre-commit and Ruff. Hence, Ruff shouldn't be installed via Poetry. I believe that if you set up your dev environment via Nix and start VS Code from an active Nix environment, then VS Code's Ruff extension should pick up the ruff binary installed by Nix.
/cc @yajo

ah i see, i was using the Poetry environment

I had a deeper dive into the way the nix environment is set up in this repo. It's... complex. There's an awful lot of indirection there.
Nevertheless, I think it renders this PR redundant, no?

I had a deeper dive into the way the nix environment is set up in this repo. It's... complex. There's an awful lot of indirection there.

Interest in joining https://github.com/copier-org/copier/discussions/1468#discussioncomment-8026103?

Nevertheless, I think it renders this PR redundant, no?

Yes


ENABLE_LEGACY mismatch using cmake

When I run cmake on commit 36b08db without any options it says
-- The following features have been disabled:
ENABLE_LEGACY, Decode legacy (< 4.0) WavPack files.
Adding -DENABLE_LEGACY=1 shows
CMake Warning:
Manually-specified variables were not used by the project:
ENABLE_LEGACY
The proper way appears to be adding -DWAVPACK_ENABLE_LEGACY=1 , so it should be reflected in the first message

Hi. I am unfortunately not very familiar with cmake, but I found the command cmake -LH and it displayed this, which seems to be the right thing:
// Decode legacy (< 4.0) WavPack files
WAVPACK_ENABLE_LEGACY:BOOL=OFF

I looked at the CMakeLists.txt file and it used both WAVPACK_ENABLE_LEGACY and ENABLE_LEGACY as different things so I don't really know how I would fix this.
Any comments @evpobr ?
Thanks!

@gitterdude , this happened due to different feature names and options. Generally speaking, this is not an error, they do not have to match and the message does not say to use the ENABLE_LEGACY option anywhere.
@dbry , in general, there is no problem to make the names of features and options match, why not. I will make PR.

Ready, #129 .


Change spelling and grammar

If you wanted to change the API from Subscribtion to Subscription and a few other small documentation changes
See Issue https://github.com/Funkia/hareactive/issues/14

Current coverage is 89.64% (diff: 100%)

Merging #15 into master will not change coverage

@@             master        #15   diff @@
==========================================
  Files             5          5
  Lines           483        483
  Methods         166        166
  Messages          0          0
  Branches         23         23
==========================================
  Hits            433        433
  Misses           50         50
  Partials          0          0

Powered by Codecov. Last update 7a3e836...c17aa9d

Nice. Thank you!


Remove global box-sizing setting in diete.css

This requires to go through all the components and set the box-sizing to one of the values (and not with * selector which affects snippets including children).
We need to also test (ideally for new components as well) that they don't break if user sets globally box-sizing to either border-box, content-box or inherit.
Related to #296

We need to also test (ideally for new components as well) that they don't break if user sets globally box-sizing to either border-box, content-box or inherit.

Maybe that's too big of an effort to test all this. Instead we can keep the box-sizing: border-box globally in the app.pcss, test with that and document that it works and is tested with that and other settings may cause some components to be broken.
This is still better than the alternative of including just one component and that breaking something unrelated in your existing app.


[WIP] Reduce/Eliminate Noisy Test Output

🤔 Should all runs of create_job_result_and_run_job perform the logging error disable? Should that be an optional argument to the function?

I dont think all runs of create_job_result_and_run_job should have logging error disabled. Just the ones that are intended to fail and throw an exception. Also, create_job_result_and_run_job runs the job and job_result.log() here is the reason there are errors logged in the unittest. If we set the argument of log() logger = None here, i think the noise during testing will stop as well. But I am not sure how to do that in the unittest. So I just disabled logging altogether.


Fix flaky integration test for ProvisioningRequest

What type of PR is this?
/kind bug

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #2483
Special notes for your reviewer:
There was a race condition and the AdmissionCheck didn't get Ready state before finishing a Workload
Does this PR introduce a user-facing change?

NONE

/assign @alculquicondor

/hold cancel


Unitialized Variable in pdfForGeneralExponent() function from StableRand.cpp

On line 533 of StableRand.cpp the variable theta0 is uninitialized when used 2 lines later.

Fixed in https://github.com/astralord/RandLib/commit/3c2698f61e7f0e55003d542b684bbc5c696d4910


React Node Support in getString()

In the values object you pass to getString() you can now use React nodes, in addition to simple strings, enabling the embedding of HTML tags, images, etc.
Activated unit tests for vulcan:lib/intl/formatLabel
Added unit tests for vulcan:lib/intl/getString

@eric-burel I have updated the pull request and added unit tests.

Awesome, it's good to see more unit tests, thanks for taking time to add them :) feel free to merge of course

@eric-burel Thank you for the gentle encouragement!


refactor(BaseService): improve prepare_request() docstring

This commit simply modifies the docstring comments related to the headers and params arguments to the BaseService.prepare_request() function in order to clarify that any entries in either of the two dictionary arguments will simply be ignored if they have a value of None.

:tada: This PR is included in version 3.20.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Exception in update_ram()

Issue summary
import retro

games = [
    'StarFox-Snes',
    'JikkyouOshaberiParodius-Snes',
    'StarFox-Snes',
]

for game in games:
    print(game)
    env = retro.make(game=game)
    env.reset()
    env.step(env.action_space.sample())
    env.close()

Running this script on my machine will often work but sometimes segfault or print this exception:
StarFox-Snes
Sound buffer size: 128160 (32040 samples)
JikkyouOshaberiParodius-Snes
Sound buffer size: 128160 (32040 samples)
StarFox-Snes
Sound buffer size: 128160 (32040 samples)
Traceback (most recent call last):
  File "/Users/csh/agi/sandbox/oneoff/retro-state-fail.py", line 12, in <module>
    env.reset()
  File "/Users/csh/agi/retro/retro/retro_env.py", line 227, in reset
    self.data.update_ram()
IndexError: No known mapping

python ~/agi/sandbox/oneoff/retro-state-fail.py
StarFox-Snes
Sound buffer size: 128160 (32040 samples)
JikkyouOshaberiParodius-Snes
Sound buffer size: 128160 (32040 samples)
[1]    69484 segmentation fault  python ~/agi/sandbox/oneoff/retro-state-fail.py

System information

macOS 10.13.6
Python 3.6.6 :: Anaconda, Inc.
0.6.1.dev36+g3305752

The StarFox-Snes issues were caused by two bugs in the snes emulator:
diff --git a/libretro/libretro.cpp b/libretro/libretro.cpp
index f66920f..7898521 100644
--- a/libretro/libretro.cpp
+++ b/libretro/libretro.cpp
@@ -481,7 +481,7 @@ static bool merge_mapping()
 	}
 	uint32 len=a->len;
 	if (!len) len=(0x1000000 - a->select);
-	if (len && ((len-1) & (len | a->disconnect))==0 && ((char*)a->ptr)+a->offset+len == ((char*)b->ptr)+b->offset)
+	if (len && ((len-1) & (len | a->disconnect))==0 && a->start + len == b->start && ((char*)a->ptr)+a->offset+len == ((char*)b->ptr)+b->offset)
 	{
 //printf("merge/consec\n");
 		a->select &=~ len;
diff --git a/memmap.cpp b/memmap.cpp
index edb8b64..ae238bb 100644
--- a/memmap.cpp
+++ b/memmap.cpp
@@ -3228,10 +3228,10 @@ void CMemory::Map_SufamiTurboPseudoLoROMMap (void)
 	map_lorom_offset(0xc0, 0xdf, 0x8000, 0xffff, 0x100000, 0x200000);

 	// I don't care :P
-	map_space(0x60, 0x63, 0x8000, 0xffff, SRAM - 0x8000);
-	map_space(0xe0, 0xe3, 0x8000, 0xffff, SRAM - 0x8000);
-	map_space(0x70, 0x73, 0x8000, 0xffff, SRAM + 0x4000 - 0x8000, false);//these two seem to duplicate the above ones in
-	map_space(0xf0, 0xf3, 0x8000, 0xffff, SRAM + 0x4000 - 0x8000, false);//way that makes the duplicates hard to find.
+	map_space(0x60, 0x63, 0x8000, 0xffff, SRAM);
+	map_space(0xe0, 0xe3, 0x8000, 0xffff, SRAM);
+	map_space(0x70, 0x73, 0x8000, 0xffff, SRAM + 0x4000, false);//these two seem to duplicate the above ones in
+	map_space(0xf0, 0xf3, 0x8000, 0xffff, SRAM + 0x4000, false);//way that makes the duplicates hard to find.

 	map_WRAM();

@@ -3257,8 +3257,8 @@ void CMemory::Map_SuperFXLoROMMap (void)
 	map_hirom_offset(0x40, 0x7f, 0x0000, 0xffff, CalculatedSize, 0);
 	map_hirom_offset(0xc0, 0xff, 0x0000, 0xffff, CalculatedSize, 0);

-	map_space(0x00, 0x3f, 0x6000, 0x7fff, SRAM - 0x6000);
-	map_space(0x80, 0xbf, 0x6000, 0x7fff, SRAM - 0x6000);
+	map_space(0x00, 0x3f, 0x6000, 0x7fff, SRAM);
+	map_space(0x80, 0xbf, 0x6000, 0x7fff, SRAM);
 	map_space(0x70, 0x70, 0x0000, 0xffff, SRAM);
 	map_space(0x71, 0x71, 0x0000, 0xffff, SRAM + 0x10000);

The JikkyouOshaberiParodius-Snes issue could be also a memory map issue, but I didn't look at it.  It's not clear to me if the map_space() fix is correct.

Currently running into this problem with pokemon blue. I believe my data.json file to be accurate, and I can get it to run when it the scenario file makes no calls to the data.json file ie its this:
{
}
Wondering how to progress forward

That sounds more like you don't have a valid memory address.

My numbers are in the same format as other gameboy data.json files in the retro build. So i don't think I do and they show up in the integration UI but again are at max value due to a bug in the integration UI

Did you try the solution here? https://github.com/openai/retro/issues/147

Working on implementing now

I'm encountering the same error trying to do a custom integration of the original SimCity on SNES. I built my data.json file using the integration tool to find the addresses of variables.
Issue summary
import retro
import os

SCRIPT_DIR = os.path.dirname(os.path.abspath(__file__))

retro.data.Integrations.add_custom_path(os.path.join(SCRIPT_DIR, "custom_integrations"))
env = retro.make("SimCity-Snes", inttype=retro.data.Integrations.ALL)
env.reset()

When I run the above I get the following error
Traceback (most recent call last):
    File "testrun.py", line 8,, in <module>
        env.reset()
    File "home/louis/wm/gym-retro/retro/retro_env.py", line 206, in reset
        self.data.update_ram()
IndexError: No known mapping

I also get the same issue running other methods of the GameData class which interact with game memory, like data.set_value() for example.
data.json:
{
  "info": {
      "buildtype": {
      "address": 8258061,
      "type": "|u1"
    },
    "money": {
      "address": 8260509,
      "type": "<i4"
    },
    "population": {
      "address": 8260517,
      "type": "<u2"
    }
  }
}

scenario.json:
{
 "done": {
    "variables": {
      "population": {
        "op": "greater-than",
        "reference": 1000
      }
    }
  },
  "reward": {
    "variables": {
      "population": {
        "reward": 1.0
      }
    }
  }
}

System Info
0.8.0.dev5+g6886936
Python 3.8.6
Arch linux

I'm encountering the same error trying to do a custom integration of the original SimCity on SNES. I built my data.json file using the integration tool to find the addresses of variables.
Issue summary
import retro
import os

SCRIPT_DIR = os.path.dirname(os.path.abspath(__file__))

retro.data.Integrations.add_custom_path(os.path.join(SCRIPT_DIR, "custom_integrations"))
env = retro.make("SimCity-Snes", inttype=retro.data.Integrations.ALL)
env.reset()

When I run the above I get the following error
Traceback (most recent call last):
    File "testrun.py", line 8,, in <module>
        env.reset()
    File "home/louis/wm/gym-retro/retro/retro_env.py", line 206, in reset
        self.data.update_ram()
IndexError: No known mapping

I also get the same issue running other methods of the GameData class which interact with game memory, like data.set_value() for example.
data.json:
{
  "info": {
      "buildtype": {
      "address": 8258061,
      "type": "|u1"
    },
    "money": {
      "address": 8260509,
      "type": "<i4"
    },
    "population": {
      "address": 8260517,
      "type": "<u2"
    }
  }
}

scenario.json:
{
 "done": {
    "variables": {
      "population": {
        "op": "greater-than",
        "reference": 1000
      }
    }
  },
  "reward": {
    "variables": {
      "population": {
        "reward": 1.0
      }
    }
  }
}

System Info
0.8.0.dev5+g6886936
Python 3.8.6
Arch linux

I don't see anything obviously wrong with your setup, if you try with different variables removed, can you narrow it down to one particular variable you are defining?

I don't see anything obviously wrong with your setup, if you try with different variables removed, can you narrow it down to one particular variable you are defining?

The issue seems to occur with any variable (I tried a few others as well). As I mentioned it can also be replicated by calling any of the methods which interact with game memory. An empty scenario.json avoids the error, but only because it means that env.reset() does not involve game memory.
I tried the included Airstriker-Genesis rom and I'm able to access the memory with no issues. I'll try replicate it on a supported SNES rom once I work out where to locate the actual rom. I wonder if it could be a SNES specific issue potentially?

The issue seems to occur with any variable (I tried a few others as well). As I mentioned it can also be replicated by calling any of the methods which interact with game memory. An empty scenario.json avoids the error, but only because it means that env.reset() does not involve game memory.
I tried the included Airstriker-Genesis rom and I'm able to access the memory with no issues. I'll try replicate it on a supported SNES rom once I work out where to locate the actual rom. I wonder if it could be a SNES specific issue potentially?

Very possible

Very possible

Running BattletoadsDoubleDragon-Snes I'm able to access game memory without any issues.
I thought it could be an issue with sourcing the rom so I grabbed the USA SimCity rom from the No-Intro Collection which is linked in the documentation.

Running BattletoadsDoubleDragon-Snes I'm able to access game memory without any issues.
I thought it could be an issue with sourcing the rom so I grabbed the USA SimCity rom from the No-Intro Collection which is linked in the documentation.


Add logrus hook to show all timestamps and their diffs

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
We need to investigate what is taking so much time during the promotion. The hook is a start and we should add more verbose logging once we identify huge diffs without any obvious fix.
Builds on top of https://github.com/kubernetes-sigs/promo-tools/pull/638
The patch adds a logrus hook to the image promoter to display the timestamps including milliseconds as well their diff to the previous log message.
For example this:
logrus.Infof("First")
time.Sleep(100 * time.Millisecond)
logrus.Infof("Second")
time.Sleep(420 * time.Millisecond)
logrus.Infof("Third")

Will result in:
INFO[10:45:25.528] First                                         diff=0s
INFO[10:45:25.628] Second                                        diff=100ms
INFO[10:45:26.048] Third                                         diff=421ms

Which issue(s) this PR fixes:

Refers to https://github.com/kubernetes-sigs/promo-tools/issues/637
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

Display log timestamps and their diffs to the previous message during the promotion.

/retest

/lgtm


Bump version to 2021.11.002

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

I know there are newer versions of ELPA. I nevertheless start from 2021.11.002, because some users might like to test multiple ELPA versions via conda-forge. (So far, the following newer ELPA versions are not in conda-forge.)

2023.05.001
2022.11.001
2022.05.001
2021.11.002

@conda-forge-admin, please rerender

@conda-forge-admin, please restart ci

Thank you so much @awvwgk for your kind review! I think the errors in the previous travis CI is just due to time out, so I restart CI again now.


Performance enhancement : add an autoQueueManagement option on LazyReact

LazyFutureStreams could automatically use better performing Agrona based Queues for all operators except for flatMap (which may need to use an unbounded Queue).

This is would add more complexity than value


NewFormFieldSelect cannot be retrieved by request.Payload.StringSlice in octant 0.21.0

What steps did you take and what happened:
I want to upgrade antrea-octant-plugin by using the latest 0.21.0 octant plugin API. However, I met an issue in the latest 0.21.0 octant but the same code can work in octant 0.16.1.
I used component.NewFormFieldSelect for desTypeCol which is defined here https://github.com/antrea-io/antrea/blob/main/plugins/octant/cmd/antrea-octant-plugin/traceflow.go#L635. Now when I use
request.Payload.StringSlice(dstTypeCol), it returns the error that shows payload does not contain this field.
I did not meet this issue in previous 0.16.1 release, so I would like to know if there is any change in recent releases for this part which may cause this issue? Is there anything that my plugin needs to update?
Thanks a lot for your information!
What did you expect to happen:
request.Payload.StringSlice() can get the value defined in component.NewFormFieldSelect()
Anything else you would like to add:
Found some warning logs like below:
W0707 09:51:31.637535   18983 reflector.go:424] /__w/octant/octant/internal/objectstore/dynamic_cache.go:389: watch of *unstructured.Unstructured ended with: an error on the server ("unable to decode an event from the watch stream: unable to decode to metav1.Event") has prevented the request from succeeding
Environment:

Octant version (use octant version): 0.21.0
Kubernetes version (use kubectl version):1.19
OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc):Ubuntu 18.04.3

Sorry you ran in to this, we will take a look.

This is fixed and will be part of the next nightly build and our 0.23 release which is coming out 03-Aug-2021


[bug] ExportOptionsPlist error for key 'method': expected one of {}, but found app-store

Hi,
I have this problem when try deploy my app now:
[15:46:02]: $ /usr/bin/xcrun /opt/homebrew/Cellar/fastlane/2.184.0/libexec/gems/fastlane-2.184.0/gym/lib/assets/wrap_xcodebuild/xcbuild-safe.sh -exportArchive -exportOptionsPlist '/var/folders/4y/0lvdrkv10fq_4dk8n4nxtmmr0000gn/T/gym_config20210526-75711-lh1eui.plist' -archivePath /Users/ubookmobile/Library/Developer/Xcode/Archives/2021-05-26/App\ 2021-05-26\ 15.45.58.xcarchive -exportPath '/var/folders/4y/0lvdrkv10fq_4dk8n4nxtmmr0000gn/T/gym_output20210526-75711-ee1w8o'
+ xcodebuild -exportArchive -exportOptionsPlist /var/folders/4y/0lvdrkv10fq_4dk8n4nxtmmr0000gn/T/gym_config20210526-75711-lh1eui.plist -archivePath '/Users/ubookmobile/Library/Developer/Xcode/Archives/2021-05-26/App 2021-05-26 15.45.58.xcarchive' -exportPath /var/folders/4y/0lvdrkv10fq_4dk8n4nxtmmr0000gn/T/gym_output20210526-75711-ee1w8o
objc[75743]: Class AMSupportURLConnectionDelegate is implemented in both /usr/lib/libauthinstall.dylib (0x1f673d160) and /System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDevice (0x1187002b8). One of the two will be used. Which one is undefined.
objc[75743]: Class AMSupportURLSession is implemented in both /usr/lib/libauthinstall.dylib (0x1f673d1b0) and /System/Library/PrivateFrameworks/MobileDevice.framework/Versions/A/MobileDevice (0x118700308). One of the two will be used. Which one is undefined.
2021-05-26 15:46:03.178 xcodebuild[75743:5514047] [MT] IDEDistribution: -[IDEDistributionLogging _createLoggingBundleAtPath:]: Created bundle at path '/var/folders/4y/0lvdrkv10fq_4dk8n4nxtmmr0000gn/T/ubook_2021-05-26_15-46-03.178.xcdistributionlogs'.
2021-05-26 15:46:03.192 xcodebuild[75743:5514047] [MT] IDEDistributionMethodManager: -[IDEDistributionMethodManager orderedDistributionMethodsForTask:archive:]: Error = Error Domain=IDEDistributionMethodManagerErrorDomain Code=2 "Unknown Distribution Error" UserInfo={NSLocalizedDescription=Unknown Distribution Error}
error: exportArchive: exportOptionsPlist error for key 'method': expected one of {}, but found app-store

Error Domain=IDEFoundationErrorDomain Code=1 "exportOptionsPlist error for key 'method': expected one of {}, but found app-store" UserInfo={NSLocalizedDescription=exportOptionsPlist error for key 'method': expected one of {}, but found app-store}

** EXPORT FAILED **
[15:46:03]: Exit status: 70

+---------------+-------------------------+
|            Build environment            |
+---------------+-------------------------+
| xcode_path    | /Applications/Xcode.app |
| gym_version   | 2.184.0                 |
| export_method | app-store               |
| sdk           | iPhoneOS14.5.sdk        |
+---------------+-------------------------+

[15:46:03]: ▸ sent 134416834 bytes  received 6272 bytes  89615404.00 bytes/sec
[15:46:03]: ▸ total size is 134378069  speedup is 1.00
[15:46:03]: ▸ Copied /Users/ubookmobile/Developer/workspaces/ios/ubook-ios/Pods/ubook/Release/ubook.xcframework/ios-arm64_arm64e_armv7_armv7s to /Users/ubookmobile/Library/Developer/Xcode/DerivedData/Ubook-eydstraulkwyohdjzkkywtxgowfk/Build/Intermediates.noindex/ArchiveIntermediates/ubook/BuildProductsPath/Release-iphoneos/XCFrameworkIntermediates/ubook
[15:46:03]: ▸ ** ARCHIVE SUCCEEDED **
[15:46:03]:
[15:46:03]: ⬆  Check out the few lines of raw `xcodebuild` output above for potential hints on how to solve this error
[15:46:03]: 📋  For the complete and more detailed error log, check the full log at:
[15:46:03]: 📋  /Users/ubookmobile/Library/Logs/gym/App-ubook.log
[15:46:03]:
[15:46:03]: Looks like fastlane ran into a build/archive error with your project
[15:46:03]: It's hard to tell what's causing the error, so we wrote some guides on how
[15:46:03]: to troubleshoot build and signing issues: https://docs.fastlane.tools/codesigning/getting-started/
[15:46:03]: Before submitting an issue on GitHub, please follow the guide above and make
[15:46:03]: sure your project is set up correctly.
[15:46:03]: fastlane uses `xcodebuild` commands to generate your binary, you can see the
[15:46:03]: the full commands printed out in yellow in the above log.
[15:46:03]: Make sure to inspect the output above, as usually you'll find more error information there
[15:46:03]:
+---------------+------------------+
|           Lane Context           |
+---------------+------------------+
| PLATFORM_NAME |                  |
| LANE_NAME     | testflight_ubook |
+---------------+------------------+
[15:46:03]: Error packaging up the application

+------+-----------------------------------+-------------+
|                    fastlane summary                    |
+------+-----------------------------------+-------------+
| Step | Action                            | Time (in s) |
+------+-----------------------------------+-------------+
| 1    | Verifying fastlane version        | 0           |
| 2    | Switch to send_to_testflight lane | 0           |
| 💥   | gym                               | 7           |
+------+-----------------------------------+-------------+

[15:46:03]: fastlane finished with errors

[!] Error packaging up the application
make: *** [deploy-ubook] Error 1

MacOS Big Sur
Fastlane 2.184.0

Can anyone help me?
Thanks guys.

Hi, sorry.
Here is:
<details><summary>✅ fastlane environment ✅</summary>

### Stack

| Key                         | Value                                                      |
| --------------------------- | ---------------------------------------------------------- |
| OS                          | 11.3.1                                                     |
| Ruby                        | 2.7.3                                                      |
| Bundler?                    | false                                                      |
| Git                         | git version 2.30.1 (Apple Git-130)                         |
| Installation Source         | /opt/homebrew/Cellar/fastlane/2.184.0/libexec/bin/fastlane |
| Host                        | macOS 11.3.1 (20E241)                                      |
| Ruby Lib Dir                |<EMAIL_ADDRESS>                  |
| OpenSSL Version             | OpenSSL 1.1.1k  25 Mar 2021                                |
| Is contained                | false                                                      |
| Is homebrew                 | true                                                       |
| Is installed via Fabric.app | false                                                      |
| Xcode Path                  | /Applications/Xcode.app/Contents/Developer/                |
| Xcode Version               | 12.5                                                       |

### System Locale

| Variable | Value       |   |
| -------- | ----------- | - |
| LANG     | en_US.UTF-8 | ✅ |
| LC_ALL   | en_US.UTF-8 | ✅ |
| LANGUAGE |             |   |

### fastlane files:

<details><summary>`./fastlane/Fastfile`</summary>

```ruby
fastlane_version "2.170.0"

Excon.defaults[:ssl_verify_peer] = false

before_all do
	# before all
end

##############################################################################################
# TESTFLIGHT
##############################################################################################

desc "Prepare evertything to publish to Testflight"
lane :testflight_prepare do
	clean_build_artifacts
	increment_build_number(xcodeproj: "./Ubook.xcodeproj")
end

desc "Download debug symbols from testflight and upload to Firebase Crashlytics"
lane :update_debug_symbols do |options|
	download_dsyms(app_identifier: options[:app_identifier], version: "latest", wait_for_dsym_processing: true)
	upload_symbols_to_crashlytics(gsp_path: "./Ubook/Targets/" + options[:target] + "/GoogleService-Info.plist")
	clean_build_artifacts
end

desc "Execute evertything to an app be published to Testflight"
lane :send_to_testflight do |options|
	gym(
		scheme: options[:scheme],
		clean: true,
		export_options: {
			method: "app-store"
		}
	)

	#ENV["DELIVER_ITMSTRANSPORTER_ADDITIONAL_UPLOAD_PARAMETERS"] = "-t DAV"
	ENV["FASTLANE_ITC_TEAM_ID"] = "24214800"
	ENV["FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD"] = ENV["UBOOK_APPLE_SPECIFIC_PASSWORD"]

	pilot(skip_submission: options[:skip_submission], skip_waiting_for_build_processing: false)
	update_debug_symbols(
		app_identifier: options[:app_identifier],
		target: options[:target]
	)
end

desc "Ubook"
lane :testflight_ubook do
	send_to_testflight(
		app_identifier: "br.com.ubook.ubook-app",
		scheme: "ubook",
		skip_submission: true,
		target: "Ubook"
	)
end

desc "Ureader"
lane :testflight_ureader do
	send_to_testflight(
		app_identifier: "br.com.ubook.ureader",
		scheme: "ureader",
		skip_submission: true,
		target: "Ureader"
	)
end

desc "Saraiva"
lane :testflight_saraiva do
	send_to_testflight(
		app_identifier: "br.com.ubook.ubook-saraiva",
		scheme: "saraiva",
		skip_submission: true,
		target: "Saraiva"
	)
end

desc "Nextel"
lane :testflight_nextel do
	send_to_testflight(
		app_identifier: "br.com.ubook.ubook-nextel",
		scheme: "nextel",
		skip_submission: true,
		target: "PlayLivros"
	)
end

desc "Livros Narrados"
lane :testflight_livros_narrados do
	send_to_testflight(
		app_identifier: "br.com.ubook.ubook-livros-narrados",
		scheme: "livros-narrados",
		skip_submission: true,
		target: "LivrosNarrados"
	)
end

desc "Augusto Cury Audiolivros"
lane :testflight_augusto_cury_audiolivros do
	send_to_testflight(
		app_identifier: "br.com.ubook.ubook-augusto-cury-audiolivros",
		scheme: "augusto-cury",
		skip_submission: true,
		target: "AugustoCury"
	)
end

desc "Planeta de Libros"
lane :testflight_planeta_de_libros do
	send_to_testflight(
		app_identifier: "com.ubook.reader.planetadelibros",
		scheme: "planeta-de-libros",
		skip_submission: true,
		target: "PlanetaDeLibros"
	)
end

desc "Send all apps to Testflight"
lane :testflight_all do
	testflight_ubook
	testflight_ureader
	testflight_saraiva
	testflight_nextel
	testflight_livros_narrados
	testflight_augusto_cury_audiolivros
	testflight_planeta_de_libros
end

##############################################################################################
# GENERAL
##############################################################################################

lane :refresh_dsyms do
	download_dsyms
	upload_symbols_to_crashlytics
	clean_build_artifacts
end

`./fastlane/Appfile`
apple_dev_portal_id ENV["UBOOK_APPLE_DEV_PORTAL_ID"]  # Apple Developer Account
itunes_connect_id ENV["UBOOK_ITUNES_CONNECT_ID"]     # iTunes Connect Account
team_name "Ubook Editora Ltda"

fastlane gems

Gem
Version
Update-Status

fastlane
2.184.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.4.0

atomos
0.1.3

CFPropertyList
3.0.3

claide
1.0.3

colored2
3.1.2

nanaimo
0.3.0

xcodeproj
1.19.0

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.7.0

terminal-table
1.8.0

plist
3.6.0

public_suffix
4.0.6

addressable
2.7.0

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

artifactory
3.0.15

babosa
1.0.4

colored
1.2

highline
2.0.3

commander
4.6.0

excon
0.81.0

faraday-excon
1.1.0

faraday-net_http
1.0.1

faraday-net_http_persistent
1.1.0

ruby2_keywords
0.0.4

faraday
1.4.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.3

faraday-cookie_jar
0.0.7

faraday_middleware
1.0.0

fastimage
2.2.3

gh_inspector
1.1.3

json
2.3.0

mini_magick
4.11.0

naturally
2.2.1

rubyzip
2.3.0

security
0.1.3

xcpretty-travis-formatter
1.0.1

dotenv
2.7.6

bundler
2.2.15

simctl
1.6.8

jwt
2.2.3

uber
0.1.0

declarative
0.0.20

trailblazer-option
0.1.1

representable
3.1.1

retriable
3.1.2

mini_mime
1.1.0

multi_json
1.15.0

signet
0.15.0

memoist
0.16.2

os
1.1.1

googleauth
0.16.2

httpclient
2.8.3

rexml
<IP_ADDRESS>

webrick
1.6.1

google-apis-core
0.3.0

google-apis-playcustomapp_v1
0.2.0

google-apis-androidpublisher_v3
0.3.0

google-cloud-env
1.5.0

google-cloud-errors
1.1.0

google-cloud-core
1.6.0

google-apis-iamcredentials_v1
0.3.0

google-apis-storage_v1
0.3.0

rake
13.0.3

digest-crc
0.6.3

google-cloud-storage
1.31.1

emoji_regex
3.2.2

jmespath
1.4.0

aws-partitions
1.461.0

aws-eventstream
1.1.1

aws-sigv4
1.2.3

aws-sdk-core
3.114.0

aws-sdk-kms
1.43.0

aws-sdk-s3
1.95.0

uri
0.10.0

forwardable
1.3.1

logger
1.4.2

cgi
0.1.0

date
3.0.0

timeout
0.1.0

stringio
0.1.0

zlib
1.1.0

ipaddr
1.2.2

openssl
2.1.2

ostruct
0.2.0

strscan
1.0.3

io-console
0.5.6

delegate
0.1.0

fileutils
1.4.1

singleton
0.1.0

open3
0.1.0

yaml
0.1.0

psych
3.1.0

mutex_m
0.1.0

generated on: 2021-05-27

```

Hi, @paulo-coutinho did you figured out what's the problem and maybe you know how to solve it?

In my case i need add/remove some problematic frameworks and solved


fix: manually rate limit fetching opted out list in opt out worker

Let me know if there are any concerns here around using Process.sleep that I'm not aware of :)

Looks like it ran to completion on dev-green 👍

Just checked back on this this morning. We're still getting occasional failures, so I'm backing the timeout off again

Looking at the dev-green logs now, it looks like it mostly succeeds, but still fails due to "throttling" about a quarter of the time, despite the other instance logging skipped indicating it is respecting the lock.
It occurs to me that the 10 requests/sec limit is likely at the level of the AWS account, meaning it's actually shared among all three alerts_concierge environments (prod, dev, dev-green). Since the check is not clock-scheduled but instead occurs 1 hour after the completion of the last check, the check times for each environment will drift around and sometimes overlap.
I think this is probably okay for a short-term fix, since it should still succeed every few hours, and there is no downside (other than the expense) to sending messages to an opted-out number for a while — they are blocked at the carrier level immediately once the stop keyword is sent.
If we wanted to really make sure we stop sending ASAP, we could set the delay to >300ms, which would allow the checks in all three environments to be running concurrently; with the current list size, each check would "only" take ~45s. Not a huge fan of this, considering it's tying an aspect of the code to the number of environments it happens to be deployed to, but again, might be acceptable if we plan to delete this soon anyway. Your call!

Hah that all makes sense :) I think for now sticking with a 200ms timeout should be a slightly better balance between success and failures. I'm leaning towards axing this code anyways once we start receiving the SMS delivery failures

Also, we should expect to still see some failures on dev-green even if the delay was ideal, because this same code is not (yet) deployed to dev or prod, and so those environments are going to be less respectful of the rate limit.


Electric custom types (eg, Postgres date type) not generating correct SQL for query.

All ElectricTypes custom types currently implement DialectAwareSqlType via CustomElectricTypeGeneric. Which is great, but there is currently a bug in drift which, when creating variables via query builders, it does not correctly pass through the electric type as the helper is checking against CustomSqlType and not UserDefinedSqlType.
See: https://github.com/SkillDevs/electric_dart/blob/master/packages/electricsql/lib/src/client/conversions/custom_types.dart
Helper:
/// Utilities to derive other expressions with a type compatible to `this`
/// expression.
extension WithTypes<T extends Object> on Expression<T> {
  /// Creates a variable with a matching [driftSqlType].
  Variable<T> variable(T? value) {
    return switch (driftSqlType) {
      CustomSqlType<T> custom => Variable(value, custom),
      _ => Variable(value),
    };
  }
}

I've currently opened a PR to type-check against UserDefinedSqlType instead of CustomSqlType as that is what the Variable constructor is expecting: https://github.com/simolus3/drift/pull/2909
This is more of an FYI issue as queries on for example the Postgres date type are currently not working in Electric Dart.

Good catch! Thanks for the heads up. We'll keep this open until drift releases a hotfix.

Thanks @davidmartos96, doesn't look like the fix is far off :)

Fixed in version 2.17.0 of drift. Thank you!


fix(AbstractProtectedRoutable): use session user object

The values are in the user property instead of the session object itself.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 5 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.197%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Auth/AbstractProtectedRoutable.php
0
5
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
137

Relevant Lines:
147

💛  - Coveralls


Add PHPStorm EAP build 212.4638.20

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@OfficialEsco Please help me with this PR. I am not getting what is wrong in my manifest. It is successfully running on my machine.

@OfficialEsco Please approve this PR also. It has also taken 6 days to pass.


Support alternative YouTube livestream URL format

Please add support for livestream URL format like this (supported by youtube-dl and mpv):
https://www.youtube.com/c/FRANCE24English/live
https://www.youtube.com/c/FRANCE24/live
France24 YouTube livestream often changes its URL, making previous ones unavailable:
https://www.youtube.com/watch?v=qKEk1YCOz8Q
https://www.youtube.com/watch?v=gq11un3xqsA
https://www.youtube.com/watch?v=NkruFxgEisY
https://www.youtube.com/watch?v=dHI7hP90ze0
I managed to get a main URL (the one that I shared and is not supported by livestreamer) that always refers to the current livestream URL. I took it from the source code of France24 tweets:
https://twitter.com/FRANCE24/status/691656070327070721
https://twitter.com/France24_en/status/691656277202706433
https://twitter.com/France24_fr/status/633928942227968000

Currently I can use a hack for livestreamer to get the referred YouTube video id: livestreamer curl https://www.youtube.com/c/FRANCE24English/live | grep -o https://www.youtube.com/v/.*? | tr -d ? best
By the way, I have seen that the main URL does not play very well with youtube-dl and mpv, it is cut sometimes, I don't know why.

+1 for Support of these formats in Livestreamer!


Offline Messages

If My client (Paho) Has cleanSession=False Then Where Moquette stores the offline Message , Whether It stores in Memory Or MapDB Database that used by moquette,
Sorry for my bad english
Thanks
Poovarasan Vasudevan

By default moquette uses in memory DB, but in moquette.conf you could select between H2 and MapDB
storage_class io.moquette.persistence.h2.H2PersistentStore
or
storage_class io.moquette.persistence.mapdb.MapDBPersistentStore


chore(git hook): add precommit-hook to handle validation checks

This change adds validation check for linting, formatting & test concurrently on commit

Fixes #880

Hi @EspiraMarvin does this mean that the lint/test operations will occur whenever a developer pushes local commits to GitHub. I.e. these operations will occur regardless of whether or not there is an associated PR?
Thanks
Ronny

@ronzulu Yes, according to the PR, the operations will run regardless of whether or not there is an associated PR.

Thanks. I think this is really helpful.

@EspiraMarvin 👀 ...

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.96%. Comparing base (f2c5739) to head (3051169).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #903      +/-   ##
==========================================
- Coverage   93.16%   92.96%   -0.20%
==========================================
  Files          42       42
  Lines        4537     4537
  Branches      644      371     -273
==========================================
- Hits         4227     4218       -9
- Misses        308      317       +9
  Partials        2        2

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

@EspiraMarvin 👀 ...

@st3v3nmw  this is ready


Indentation handled by number of characters, not indentation width

The following GDScript code is valid and accepted by the parser (emphasis is on the indentation):
										extends Tabs

func _ready():
	var x = 0
	if 1 == 1:
	 x = 10
		x = 20
  x = 30
 x = 40
	x = 50

This is impossible to understand without revealing the whitespace characters used:
[-TAB-][-TAB-][-TAB-][-TAB-][-TAB-][-TAB-][-TAB-][-TAB-][-TAB-][-TAB-]extends Tabs

func _ready():
[-TAB-]var x = 0
[-TAB-]if 1 == 1:
[-TAB-][SPACE]x = 10
[-TAB-][-TAB-]x = 20
[SPACE][SPACE]x = 30
[SPACE]x = 40
[-TAB-]x = 50

Obviously, the GDScript parser only handles indentation by counting the characters used to indent, not the visual width of them like Python does.
Then again it seems to ignore whitespace at the start of a file.
I think it would be better to handle it like Python, defaulting tabs to a specific number of equivalent spaces, and then using the resulting indentation width rather than character count. S. http://www.secnetix.de/olli/Python/block_indentation.hawk section "How does the compiler parse the indentation?" about the Python indentation implementation which I would recommend for GDScript aswell.
On a side note, multiple instances of this parser error popped up when opening such an indented file (though this was in a 2.2 build):
ERROR: GDParser::_reduce_expression: Method/Function Failed, returning: op
   At: modules\gdscript\gd_parser.cpp:1540

This is a bit of a bummer. Not that I like using spaces, but I started using them to make code look better on Github (because it, for some unfathomable reason, uses 8-wide tabs, as we all know).
I noticed this while trying to line up these lines:
#..|...|...|...|...|...|x     <-- 'x' marks the spot
func pause():            paused = true
func unpause():            paused = false
func toggle_pause():    paused = !paused   # this one just so happens to be there

Not that I like using spaces, but I started using them to make code look better on Github (because it, for some unfathomable reason, uses 8-wide tabs, as we all know).

You can use the Refined GitHub add-on to decrease the tab width. Alternatively, you can use one of many userstyles, or even write your own:
* {
  tab-size: 4;
}

I think it would be better to handle it like Python, defaulting tabs to a specific number of equivalent spaces, and then using the resulting indentation width rather than character count.

Note that AFAIK, while Python 2 worked this way, Python 3 just yells at you if you mix tabs and spaces. I think we should do the same.
Mixing indentation styles and relying on an arbitrary visual width for tabs is really asking for trouble.

@akien-mga I agree with your conclusion on that. But we're still left with the issue of, when using (exclusively) spaces, the tab key always adds 4 spaces instead of lining things up (in 3.1.1 stable).
I was about to open an issue about that, but then i didn't because I interpreted this one as being about that.

That's a different issue, this one was about the GDScript language parser, what you ask for is an IDE feature.

Should I open one then?

If there isn't an existing issue about it, sure.

Well, I found #28811 from #28724, so I guess it's even already fixed!


Cannot resolve module 'fs'

Hey there, thanks for the loader,
I get the following warning when running Webpack with this loader.
WARNING in ./~/jade/lib/runtime.js
Module not found: Error: Cannot resolve module 'fs' in /Users/Username/Dev/www/project/node_modules/jade/lib
 @ ./~/jade/lib/runtime.js 180:18-31

I think found a possible solution which I think you actually gave @sokra. https://gitter.im/webpack/webpack/archives/2014/08/26 looks like you need to add
"browser": { "fs": false }
https://github.com/kangax/fabric.js/blob/master/package.json#L17-L21
Could be alos that I screwed something up, thanks in advance for any help

+1

Can someone please explain what does the above config do?

@omerts I can try but the question seems a little broad, is there something specific about it you wish to understand?

@brenwell how does it solve the problem, does it cause webpack to mock the fs module during compilation time?

Not sure about any docs, but yes I believe this creates an empty module for fs.

@brenwell ok thank you :)

Worked! Thanks alot @zalad

I don't understand why I'm getting this error, because I'm only using the path module in my webpack.config.js.  Shouldn't it only complain if my browser code accidentally requires path?

@jedwards1211 its just a console warning (just red color), not throw error

Docs for Webpack config: https://github.com/webpack/docs/wiki/ (was hard to find)

@zalad, Where to add that code and how to add that code ?

@afoysal in case anyone is still confused. it should be added like:
var config = {
    node: {
        fs: "empty"
    }
}

I have already added that code but unfortunately I am still having the same error. Below is my code
module.exports = {
entry:  __dirname + "/app/assets/scripts/app.js",
output: {
    path: __dirname + "/app/temp/scripts",
    filename: "app.js"
},
module: {
    loaders: [
        {
            test: /\.js$/,
            exclude: 'node_modules',
            loader: 'babel-loader',
            query: {
                presets: ['es2015']
            },

        }
    ]
},
 node: {
    fs: 'empty',
    // tls: 'empty'
}

}

@omerts (just my guess) setting 'fs' to 'empty' means there is NO implementation of the 'fs' module. Take a look at this module 'node-lib-browser'. By default, webpack's target is 'web' and there is no implementation for some of the node core libraries. That is where 'node-lib-browser' comes into the picture. For 'fs', there is NO implementation, and thus should be set as 'empty'.

add this to your config:
node: {
fs: "empty"
}

This can also be fixed on library level. The library using fs can add this to the package.json:
"browser": {
  "fs": false
}

to say: "For browser build I don't need the fs package. Give me an empty one.".

add this to your config:
node: {
  fs: "empty"
}

When I do this and console.log(fs); it just returns
▼ {}
  ⯆ __proto__:
    ⯈ constructor: ƒ Object()
    ⯈ __defineGetter__: ƒ __defineGetter__()
    ⯈ __defineSetter__: ƒ __defineSetter__()
    ⯈ hasOwnProperty: ƒ hasOwnProperty()
    ⯈ __lookupGetter__: ƒ __lookupGetter__()
    ⯈ __lookupSetter__: ƒ __lookupSetter__()
    ⯈ isPrototypeOf: ƒ isPrototypeOf()
    ⯈ propertyIsEnumerable: ƒ propertyIsEnumerable()
    ⯈ toString: ƒ toString()
    ⯈ valueOf: ƒ valueOf()
    ⯈ toLocaleString: ƒ toLocaleString()
    ⯈ get __proto__: ƒ __proto__()
    ⯈ set __proto__: ƒ __proto__()

Is this caused by the 'empty' keyword? For reference, I'm using an electron-aurelia-webpack ES6 set-up.


install cli scripts into the virtual environment

Platform and UV information: uv 0.3.1 (Homebrew 2024-08-21) running on MacOS Sonoma 14.1.2
I am working to migrate a number of packages from Poetry to uv.  Perhaps I'm just not understanding the documentation, but I can't figure out how to install CLIs into the venv created by uv.
I manage multiple CLIs out of single packages. During development, I need to run the CLIs within the virtual environment. How would this be accomplished using uv?
Minimal package layout:
├── pyproject.toml
├── src/
    └── package_name/
        ├── __init__.py
        ├── cli1.py
        ├── cli2.py
        ├── models/
        ├── utils/
        ├── ...

This was easily accomplished in Poetry using tool.poetry.scripts in the pyproject.toml file. This would allow invoking any of the cli scripts within the virtual environment rather than the running the globally installed version
[tool.poetry.scripts]
    cli1 = "package_name.cli1:app"
    cli2 = "package_name.cli2:app"

is similar functionality available in uv?  or a workaround that I'm not seeing?

Hey! We support the standard [project.scripts] table instead of declaring some tool-specific section :)


Add hollow triangle patten in java

Added a hollow triangle pattern program in java.

I'm assuming that you already check and test your code. Nice work, thanks


Why would people want to sign up?

I cannot tell why one would want to sign up. What does signing up give you? It seems like anyone can provide data right now, and there is nothing explaining why one might want to sign up.

This was a good point. Going to close this since it is being resolved by the design in #294 that is going to be implemented.


fix: convert afterCompile to emit

✅ Closes: #351

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Adopt setuptools packaging

Moves package metadata to setup.cfg as setup.py was deprecated long time ago.
This enables git tag based versioning, making possible to make a new (pre)-release but just a tag creation, prefereably using https://github.com/ansible-community/tox-ansible/releases page.

Ah, the proliferation of all the install methods. setup.py, setup.cfg, and pyproject.toml. Some day maybe we'll be able to go back to only one method!
One question: how do we set version and release, now? Do we just tag and push? Or do we tag, push, and create release in GitHub manually?

Ah, the proliferation of all the install methods. setup.py, setup.cfg, and pyproject.toml. Some day maybe we'll be able to go back to only one method!
One question: how do we set version and release, now? Do we just tag and push? Or do we tag, push, and create release in GitHub manually?

While pushing a new tag would also create a release, I do not recommend this approach and usually I block pushed.
The better way of making a release is to go to https://github.com/ansible-community/tox-ansible/releases -- edit the current Draft and put the desired version in the "tag version" field, which is empty for drafts. At the same time you have the opportunity to edit the release name and its release notes, which should already contain all changes merged since last release (release-drafter takes care of this.). At this moment the notes are empty because we did not merge anything that did not had "skip-releasenotes" label (we do not want CI changes in release notes).


Update development.md

OPS Build status updates of commit a902cf4:
:clock10: Preparing: average preparing time is 50 sec(s)

OPS Build status updates of commit a902cf4:
:clock10: Incremental building: average incremental building time is 26 sec(s)

OPS Build status updates of commit a902cf4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/development.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Also expose headers in CORS when they are allowed.

This is just a draft, I'm not sure why this isn't the default so I'm worried it might be for security reasons.

When allowing headers, also expose the headers for remote scripts to use.
Description
Allowed headers in CORS are also allowed to be used by remote scripts.
Related Issue
Can't find any
Motivation and Context
https://emscripten.org/docs/api_reference/Filesystem-API.html#FS.createLazyFile depends on being able to see the values of the range and content type of the remote URL.
I've been trying to get datasette-lite to load a 32GB SQLite FTS5 database. The CORS headers do not go far enough to satisfy createLazyFile.
https://github.com/simonw/datasette-lite/pull/49
How Has This Been Tested?
I served a directory with:
cargo run -- -c \* -p 3001 -z -g debug -j "origin, content-type, content-encoding, accept-ranges" -d ~/db_dir/all_records
I was able to call it with my build from
https://github.com/simonw/datasette-lite/pull/49
and a URL not unlike:
http://localhost:8009/?url=http%3A%2F%2Flocalhost%3A3001%2Fdb.sqlite#/db?sql=SELECT+*+FROM+records+WHERE+records+MATCH+"hacienda+heights"
It seemed to satisfy Enscriptem's createLazyFile from looking in the console and not throw errors complaining about not being able to access the accept-ranges and content-type headers from the code.
Screenshots (if appropriate):

This is just a draft, I'm not sure why this isn't the default so I'm worried it might be for security reasons.

I would suggest that instead of forcing this to be  something by default. Why not just let users decide to expose headers when they want via a flag.

OK, I'll see about adding this as an option.

Properized it I guess?

Whoops need to fix tests.

Released on v2.13.0


ResourcesFragment: "tags off" gets stucked

How to reproduce:

open a folder
click "tags off" button (picture of crossed-out cloud)
undo by clicking "tags on"
open any resource
go back (using system navigation button)

Expected:

all resources in the folder displayed

Actual:

only resources without tags displayed (the same as after step 2)

Fixed in #37


[Feat]: Implement Codebase Search Feature with Embedding Matching

Is your feature request related to a problem?
Developers need a way to quickly search across the codebase to find specific functions or references, such as where a push notification listen method is implemented.
Describe the solution that you'd like
The solution involves creating embeddings for every file in the codebase. When a user queries something specific, like "Where is the push notification listen method," the system will match this query against all file embeddings. The top matching files will then be provided as context to the language model along with the user's query.
Describe alternatives that you have considered
An alternative could be to use traditional text search algorithms like regex or full-text search databases, but these may not offer the same level of semantic understanding as embedding matching.
Additional Information
Research articles on embedding matching and experiment with a subset of files to understand how effectively they match against a set of queries.
Acceptance Criteria

Embeddings are created for every file in the codebase.
A search feature is implemented that matches user queries against these embeddings.
The top 5 matching files are used as context in the language model prompt.
The feature is tested to ensure it accurately finds relevant files based on various queries.

Issue resolved by #163


docs: addition of param and result sections

Improves documentation. (addresses #40)

Signed-off-by: Leonardo Rossi<EMAIL_ADDRESS>

@hachreak LGTM 👍  please rebase. Thanks


Do you L2 Normalize the CLIP Features?

Hi @antoyang,
After extracting the features from CLIP-L/14, do you L2 normalize the features before passing it to the subsequent pipeline? Thanks

Hi, the features are not normalized.


Trigger upgrade when kustomize patches are modified

Add logic to get shaSum of all computed post render hook patches and trigger helm upgrade when there is a change.
Resolves #11

WIP - Would like some quick feedback before I add tests.
The design called for adding UID + revision of each patch, but all patches are collected anyways and it seems easier and less error prone to just get a hash of the set of patches.
The remote observed release isn't actually a CRD as I understand it so it seems like patching the local release object with the sha of the local patches after and update/create has happened.

The design called for adding UID + revision of each patch, but all patches are collected anyways and it seems easier and less error prone to just get a hash of the set of patches.

Yes, this sounds like a better solution.

The remote observed release isn't actually a CRD as I understand it so it seems like patching the local release object with the sha of the local patches after and update/create has happened.

My only concern would be possible edge cases like, e.g. upgrade succeeded but annotating release object failed. By putting annotation on k8s Release object, we are kind of storing actual state on CR. This is why I was trying to store patch information of remote observed helm release and best place I found was Chart.Metadata.Annotations which is also not great.
Stating this, I am also fine with storing sha on Release CR as well as long as we consider/handle possible edge cases.

@kferrone - When this PR lands we should have support for basic patching and detection of updates for helm releases.


feat: add trivy and custom security checks

Implement Trivy security scans and npm outdated check in CI/CD workflow
Issue Number: Fixes #123
Description of Changes
This pull request implements the following improvements to our CI/CD workflow:

Added Trivy vulnerability scan for the filesystem
Implemented Trivy secret scan for the filesystem
Added npm outdated package check
Generated and uploaded a combined security report

Example report
trivy-V-and-S-Scan-report.md
These changes enhance our security posture by automatically scanning for vulnerabilities and secrets in our codebase, as well as identifying outdated npm packages that may need updating.
Additional Notes

The Trivy scans are configured to run on every pull request (opened, reopened, or synchronized).
The combined report is uploaded as an artifact for easy review.
No new dependencies were added to the project for these scans.

🛠 Type of Change

[ ] 🐛 Bug fix
[x] ✨ New feature
[ ] 🚨 Breaking change
[ ] 📚 Documentation update

✅ Checklist
To ensure a smooth review process, please check off each item as you complete it:

[x] Code Style: My code adheres to the project’s style guidelines.
[x] Self-Review: I have reviewed my own code and made improvements.
[x] Comments: I’ve added comments to explain complex or non-obvious parts of the code.
[x] Documentation: I’ve updated the documentation to reflect my changes.
[x] Warnings: My changes introduce no new warnings.
[x] Tests: I’ve added tests to verify that my changes work as expected.
[x] Passes Locally: All new and existing unit tests pass on my local machine.

Hi @cb7chaitanya,
Let me know if you need more details on what I’ve done so far. For additional context, here’s an article I wrote on Medium that explains why Trivy is a smart choice for security scanning.
link: https://medium.com/@vt3630/trivy-the-smart-choice-for-security-scanning-when-and-why-b985a1302720

Super cool @vishutyagi0210


How to penalize the loss of one class more than the other in tensorflow for a multi class problem?

Let's say my model has two classes Class 1 and Class 2. Both Class 1 and Class 2 has a equal amount of training and testing data. But I want to penalize the loss of the Class 1 more than Class 2, so that one class has a fewer number of False Positives than the other (I want the model to perform better for one class than the other).

This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.


Store credentials by something else than the username

Hi,
I really like your module, but there is one thing that is missing for me.
~Is there a way to store (and get) credentials by title/website?~ I found the parameter Target in Set-Credential. How do I get it (based on the target)?
I use my username on multiple services, but with different password.
I could maybe use DOMAIN for the website/service, but it's not that beautiful. I'd have to remove the domain before using the credential.
Thank you a lot

Well. Solved
Set-Credential -Target "ServiceName" -Credential (Get-Credential)
Find-Credential -Filter "ServiceName"