Resurrect mnist examples and enabling unit tests

Trying to resolve previously observed segfaults by importing pyarrow before torch as explained here: https://github.com/selitvin/petastorm/blob/resurrect_mnist_examples/docs/troubleshoot.rst

Codecov Report

Merging #198 into master will increase coverage by 4.31%.
The diff coverage is 98.14%.

@@            Coverage Diff             @@
##           master     #198      +/-   ##
==========================================
+ Coverage   81.85%   86.17%   +4.31%
==========================================
  Files          65       68       +3
  Lines        3467     3544      +77
  Branches      528      532       +4
==========================================
+ Hits         2838     3054     +216
+ Misses        534      391     -143
- Partials       95       99       +4

Impacted Files
Coverage Δ

petastorm/pytorch.py
100% <100%> (+72.72%)
:arrow_up:

examples/mnist/tests/test_tf_mnist.py
100% <100%> (ø)

examples/mnist/tests/conftest.py
100% <100%> (ø)

examples/mnist/tests/test_pytorch_mnist.py
100% <100%> (ø)

examples/mnist/generate_petastorm_mnist.py
76.47% <100%> (+76.47%)
:arrow_up:

examples/mnist/pytorch_example.py
67.05% <83.33%> (+67.05%)
:arrow_up:

petastorm/workers_pool/process_pool.py
90.85% <0%> (-0.61%)
:arrow_down:

... and 6 more

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 3f996f0...d4cd605. Read the comment docs.


prioritised_trajectory_buffer: allow adding max_length_time_axis

Would love for this to be checked in. Actually ran into this bug myself.

Hi @garymm, I don't have privileges to merge this in anymore. @sash-a you know who can help verify this? Only comment I have on the PR is we should probably check if the sampling of the data is fine if the entire buffer is added. I remember when making it there was some funky issue with adding the entire buffer but it's possible we fixed that issue. I might be getting my wires crossed as well as it could've been regarding sampling the entire buffer.

I'm happy to review, thanks for this @garymm :smile:
Agree with @EdanToledo, can you add a call to sample in your test and then I think it's good to go

@sash-a done


alignmentManagerObject has not been assigned

Hello,
I launched my project in Play Mode with Holographic Remoting, my Hololens joined the match and is connected to the server but I have this error :

UnassignedReferenceException: The variable alignmentManagerObject of PlayerController has not been assigned.
You probably need to assign the alignmentManagerObject variable of the PlayerController script in the inspector.

No need to assign in the inspector because alignmentManagerObject has the attribute [HideInInspector] and normally alignmentManagerObject is set by HoloViveNetworkManager.cs with the SpawnPlayer() method.
What is my mistake?

Which Unity Version do you use?

Unity 5.6
I modified Holotoolkit and SteamVr and I haven't any problem for compile

I had issues with 5.6.
Using currently Unity 5.5.2f1 and for the Hololens build I used UWP SDK 10.0.1493.0.
See my issue (https://github.com/dag10/HoloViveObserver/issues/9) for more infos.

Not all my problems are fixed but it help to fix the detection controller problem in Unity 5.6:  https://github.com/ValveSoftware/steamvr_unity_plugin/pull/14/files

Did you fix your UnassignedReferenceException problem?
Have you tried to use the Hololens without holographic remoting?
I've got it running directly on the device and I have voice output on Hololens but no visuals.

Yes I fix my UnassignedReferenceException problem I can see the Vive controllers in the Vive application
But I still have this problems:
-Only when I deploy on Hololens (without holographic remoting) I have a skybox in the Hololens camera
(Is my Vive camera active? And why?)
-The Alignment system doesn't work, I don't see the pink ViveController in Hololens (May be obstructed by my hologram in the center of my scene)

May I ask which UWP SDK you use for the build of Hololens.

For the build in Hololens in Unity 5.6.1f1, UWP SDK is set on Latest Installed which is 10.0.14393.0

Oh I didn't solve my UnassignedReferenceException problem ! My Alignment Manager was unactive that's why.
My Bad.
Yes I know for the skybox that's the problem. My Hololens camera is OK.
Maybe it appeared because my app makes an error.

You can see the debug output of an app if you deploy with VS to Hololens with debugging enabled (F5).
I've created an empty scene in the Project and run Holographic Remoting and also run the app with Debug enabled over VS.
With both I get a similar error output.
Holographic Remoting: "D3D11: Failed to create render texture primary RTV (error 0x80070057)"
VS Debug: "D3D11: Failed to create render texture SRV (error 0x80070057)"

Ok my error was not to read the solved issues https://github.com/dag10/HoloViveObserver/issues/4
This fixed my prob.

-I still have a problem with the Hololens camera.
As soon as Hololens connect to the server, it changes the camera and I have a skybox like the Vive camera
-I can align the environment but I can't see the pink controller
In your VRPlayer prefab: Are your NetworkTransform script and your AlignmentClient script uncheck?

Here are some Screens which might help to find your prob:
Player Prefabs: http://images.devs-on.net/Image/A0B5gDjyPAARQECd-Bereich.png
Player Prefabs ingame after Sync: http://images.devs-on.net/Image/DtVsXTpbt6ibq4JZ-Bereich.png
NetworkManager: http://images.devs-on.net/Image/vwRxcwrz0sET2rYj-Bereich.png
VR Camera Rig and Hololens Camera: http://images.devs-on.net/Image/xYe3LCjdBX2tepMr-Bereich.png
Running Hololens in Holographic Remoting: http://images.devs-on.net/Image/gMwZ8eak6BSQl0vR-Bereich.png
As you can see from the last screen Holographic Remoting is partially working now.
The important piece of code is at Utils.cs there its gettinge determied which camera will be used.
http://images.devs-on.net/Image/FtD1gzEjEdTSKHtu-Bereich.png
I hope this helps I would be happy to see a working Unity 5.6 version.
I've you are not able to get it working you can also take my Unity 5.5.2 version https://github.com/mrwellmann/HoloViveObserver.


Write Log File after Tests are completed

It would be nice, if the log files were written after the tests are completed.
The log files shouldnt be deleted at the beginning of the tests.
An analysis of the errors etc. is so longer possible.

You mean the text/json logs? These are overwritten at the end of each Humbug run. The old logs won't tell you anything about a new Humbug cycle. In fact, the text log is written to incrementally as Humbug progresses because it makes a bit of debugging possible when something causes an unexpected failure.
You may be suggesting something like timestamped logs which would be a naming issue?


LineChartView - use of drawFilledEnabled with drawCubicEnabled and LineChartDataSet  draw anormal chart

With this data you can reproduce the bug :
	var xVals:[String] = ["0", "1", "2", "3", "4", "5", "6"]
	var yVals:[ChartDataEntry] = [
		ChartDataEntry(value: Double(1), xIndex: 1),
		ChartDataEntry(value: Double(5), xIndex: 3),
		ChartDataEntry(value: Double(3), xIndex: 5)
	]

	var dataSet:LineChartDataSet = LineChartDataSet(yVals: yVals)
	dataSet.drawCirclesEnabled = false
	dataSet.drawFilledEnabled = true
	dataSet.drawCubicEnabled = true

	var data:LineChartData = LineChartData(xVals: xVals, dataSet: dataSet)

It looks like there's not enough data to complete a cubic curve. We can of course simulate the last coordinate, and we've done that in the past, so this is a bug introduced lately I guess.
Can you confirm that it works when adding another point?

Yes it works when adding all points

I've looked into this - and the problem is the missing y values, which the renderer is no aware of.
@PhilJay do you think that just looping back on the prev point and looping forward on the next point until we have enough points - is enough to solve this? I know that there are some issues on Android with path sizes and a point could be found futher away on the chart and resulting in a bigger path...

Yes that could be a solution.
The issue concerning the path only arises if they are significantly larger (in terms of covered pixels) than the screen size.
I think that there still might be some issues in our implementation that cause the "path too big" problem.

This was solved in a recent commit, last week.


Use the validated bind_memory functions in Buffer and Image

Changelog:
### Bugs fixed
- Added missing validation for `BufferCreateInfo::external_memory_handle_types` in `Buffer::new`.

The Windows CI seems to be failing (for real this time).

I'm not sure I understand what's going wrong, can you explain?


Limit Python conversion on attribute access.

In particular, this limits errors caused by mutating implicit temporary
objects, e.g.
cdef vector[int] v = ...
v.pop()
See also github issue #1521.

Good intention, but there seem to be legitimate use cases for this as well that the change would break. I'm not entirely convinced that this should be done, especially without deprecation or preceding warning.

This seems similar in spirit to our disallowing of conversion of likely temporaries to char*.
Good point about a deprecation period. Do you have in mind legitimate use cases for this (that's common enough that an explicit cast to object would not be OK)? For stl containers, there are (for example) containment methods, but one would probably want to use the C++ variants rather than accidentally cause a Python conversion of the entire container.

An explicit cast to  would obviously fix the code by making it explicit, but it also means that we break existing user code, even if it's not as efficient as it could be. Our intuition seems to say that the current state does the wrong thing in most cases. The good thing about Python's containers is that mutation methods have no return values, which reduces the chance of false positives. Let's see what we change here.

Methods and attributes of integers and floats are extremely rarely used and thus do not count. Plus, these objects are immutable and calling the methods always returns a result, so they are never useless or dangerous.
Mutation methods of containers (clear(), append(), extend(), reverse(), sort(), ...) are always inefficient and often dangerous. Same for in-place operators like "+=" and "*=" (not sure if those even work currently).
Sequence methods like index(), count() or find() are most likely too inefficient for reasonable use, as is copy() which would just copy a freshly created container.
String mutation methods can be inefficient sometimes but their usage can also be perfectly ok. Strings are immutable and thus methods like encode(), lower(), format(), join() etc. always return a new and modified string anyway. The up-front conversion is often a reasonable price to pay for convenience.
For sequences, there are more traps than attribute access, e.g. slicing, in-tests and generally all usages of Python operators, but also of builtins like len() or max(). It's difficult to distinguish between legitimate and purely inefficient conversions here. Concatenation of converted lists might be ok, for example, if the result is then kept in a Python list. Same for string operations and many set operations.
Iteration over container method results such as dict.items() is usually optimised and, while more inefficient than direct C++ iteration, might be a legitimate convenience if the overhead is acceptable for the user. My guess is that this would be impacted by this change as well.

So, ISTM that the result is mixed. The majority of cases seem inefficient but there are legitimate use cases, even amongst the inefficient ones. Especially conversions to immutable Python types (definitely strings) seem quite reasonable in many cases.
What do you think about limiting the hard errors to lists and dicts as Python target types? We could still issue an inefficiency warning about others in some cases, but I wouldn't even do that generally.

I think I'm going to go ahead and merge this, the hard errors are, as you also suggest, limited to (methods, not operators, of) container types.

Your call.


Add OpenShift policies

Related Issue(s)

Description

Checklist

[X] I have read the policy contribution guidelines.
[X] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.

Sure, will make the changes.
It would be great if these recommendations for annotations are reported by the kyverno test utility.


Using Tabler Component dynamically?

Not really an issue, but more if a question. I've used the Vue import to bring in all icons, but is there anyway to generically instantiate the icon without the custom component? I'm trying something like this, but it seems... inefficient:
        <template v-if="iconName=='OneTwoThreeIcon'"><OneTwoThreeIcon :size="size" :stroke-width="strokewidth" /></template>
        <template v-else-if="iconName=='TwentyFourHoursIcon'"><TwentyFourHoursIcon :size="size" :stroke-width="strokewidth" /></template>
        <template v-else-if="iconName=='TwoFactorAuthIcon'"><TwoFactorAuthIcon :size="size" :stroke-width="strokewidth" /></template>
        <template v-else-if="iconName=='Deg360Icon'"><Deg360Icon :size="size" :stroke-width="strokewidth" /></template>
        <template v-else-if="iconName=='Deg360ViewIcon'"><Deg360ViewIcon :size="size" :stroke-width="strokewidth" /></template>

What I'm trying to do is something more like:
<TablerIcon :iconName="sMyObject.icon" :size="28" />
Cheers!

Try  https://vuejs.org/api/built-in-special-elements.html#component
<component :is="Deg360Icon"></component>

But I don't recommend this. You will bundle all icons, which is ~3.5mb in total.

Thanks Alex! I'll see if I can get my team to agree to a smaller list instead of all of them, either way there's a lot of overhead for a ton of icons I can pretty much guarantee won't get used... :-)
Cheers!
Bryce


After handleAction retrieve Promise to know when action has been performed

Summary
When handleAction or createFromAction is called a reference to the component is returned.
Now it is possible to call getStatus on this component instance which will return a Promise that will resolve when the action, or some event that is part of this action, has been performed.
In this example PR we resolve this Promise when the QR code image has loaded.
In theory any component can use this functionality to inform the merchant when an action has been implemented.
For example, I've also tested it with the card component - using it to know when the 3DS2 component has been rendered.
Tested scenarios
Tested with both handleAction and createFromAction against both Dropin and a standalone component
To test

either run the Dropin and select the first WeChatPay pm and then press "continue..."
or, go to the QRCodes playground page
then look at the console to see how the promise is resolved when the QR code loads

Relates to issue:  #941

Abandoned in favour of the onActionHandled callback (#1975)


[stable/minio]#1899 add nodeSelector for minio

[stable/minio] Added nodeSelector for minio.
related to #1899

/ok-to-test


Add SITE_EXTENSION_SDK_VERSION override

To be able to rebuild site extension against latest SDK if required

@glennc is doing something like this, but with UI

No, this is for CI, so we can override the version of SDK we build from on TeamCity


Update output-management.md

Fix CleanWebpackPlugin usage error.
replace
const CleanWebpackPlugin = require('clean-webpack-plugin');

to
const { CleanWebpackPlugin } = require('clean-webpack-plugin');

[x] Read and sign the CLA. PRs that haven't signed it won't be accepted.
[x] Make sure your PR complies with the writer's guide.
[x] Review the diff carefully as sometimes this can reveal issues.
Remove these instructions from your PR as they are for your eyes only.

已修复


[BUG] MidiSrv enumerates redundant/wrong ports if the device consists of multiple KS filter.

Describe the bug
As described in the following feedback, third party's custom driver may instantiate multiple KS filter for MIDI 1.0 device.
https://github.com/microsoft/MIDI/issues/305
With such devices, MidiSrv may enumerate redundant or wrong legacy ports.
To Reproduce

To enable MidiSrv enumerated ports, make sure the wdmaud2.drv is registered.
Download & install the Roland A-Series Keyboard driver:
https://www.roland.com/global/support/by_product/a-49/updates_drivers/4506be14-ffe5-4beb-8064-48ddf78fe56a/
Attach the Roland A-49 keyboard to PC and make sure the device is working with the installed custom driver.
Run 'midi.exe enum legacy'.

"A-Series Keyboard O-1" and "A-Series Keyboard I-1" are duplicated.
The output port "A-Series Keyboard MIDI OUT" is also enumerated as the input port with the name of "A-Series Keyboard MIDI OUT I-1".

Expected behavior
There are no redundant ports and no ports with the wrong direction.
Screenshots

Installer Name or Version
Windows.MIDI.Services.In-Box.Service.-.1.0.1-preview.7.24305.1438-x64.exe
Desktop (please complete the following information):

OS: Windows 11 24H2 build 26120.2215 (Insider Dev channel)

Device information, if this is with an external MIDI device:

Device: Roland A-49 keyboard
Driver: Roland A-Series Keyboard driver

Application Information

midi.exe
WinMM apps

This should be fixed in DP8. I spent the time today to redo the aggregate (KSA) enumeration so it goes through devices rather than filters. It makes it slightly less efficient, but the end result is more what folks will expect to see, and it also eliminates the issues created by having a KS filter-per-port.
Here are the new generated Group Terminal Blocks for a vendor-driver device, the MOTU Express 128. It reports each port # as its own KS filter.

There are still some bugs to work out, but the enumeration piece is largely complete other than fixing GTB numbers, and dealing with the auto-generated pin names our USB/KS stack creates, like these:

This change also makes it so the actual endpoint name is the name of the device itself, if the user doesn't override it.

I have an A-88 here, but it's in MIDI 2.0 mode at the moment. I'll test it later, as I assume it will respond in the same way the A-49 does.
The naming for WinMM ports is still being worked on as it is far more restrictive, but we will use these generated GTB names as the starting point, adding the direction and group #, and truncating to 32 characters as required.

In DP8


CLI support for next.js

getstorybook on an empty (pages/index.js) nextjs project yields :
• Detecting project type. ✖
Cannot read property 'react-native-scripts' of undefined

I think this is just a bug. Have issued a PR (#1195) to address the problem. I believe getstorybook will do the right thing in the fallback state and no special support is needed for next.js.

@rpersaud oops, closed this prematurely. i released the fix for this yesterday. would you mind upgrading @storybook/cli and trying again to verify the fix? it would help me a lot! 🙇

@shilman Yup, just upgraded and tried it - getstorybook now works! Good job!


browserHistory not working with Nginx rewrite rules

I write Nginx configuration according to this page.
        location /build/ {
            alias   /var/web/MFWebsite2/build/;
        }
        location /preview/ {
            rewrite ^([^\.]+)$ /build/index.html last;
            rewrite ^/preview/(.+)$ /build/$1 last;
        }

But <Router ref="router" history={browserHistory}> still doen't work.

Thanks for your question!
We want to make sure that the GitHub issue tracker remains the best place to track bug reports and feature requests that affect the development of React Router.
Questions like yours deserve a purpose-built Q&A forum. Could you post this question to Stack Overflow with the tag #react-router? https://stackoverflow.com/questions/ask?tags=react-router
We also have an active and helpful React Router community on Reactiflux, which is a great place to get fast help with React Router and with the rest of the React ecosystem. You can join at https://discord.gg/0ZcbPKXt5bYaNQ46.

@taion I think this is a feature request for browserHistory, not just a question.

It's misconfiguration on your end – please seek support through an appropriate medium.

Thanks, I'll seek for the right nginx configuration.


Update sql-database-read-scale-out.md

@CarlRabeler : Thanks for your contribution! The author, @anosov1960, has been notified to review your proposed change.


model: make forward compatible to sqlalchemy >= 2

seems not necessary anymore to migrate to flask>=3.0. the problem was in pytest-invenio. the change to not use the flask-sqlalchemy session was the problem. and the change has been done, because of the change in flask-sqlalchemy to not use bind anymore


chore: fix .travis.yml

I used the travis CLI to add the encrypted password and it made some changes that seemed to be wrong.

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


Update plugins, drop incubating, sort imports

Update plugin versions
Drop incubating status
Add plugin to sort imports in autoformat profile

This is an update to the parent pom

Obviously, some changes to repo locations, etc. won't apply until after infra graduation tasks, but this is preparatory.
I did see a bug where there is a missing pom file for the latest version of the formatter. I'll file an issue for that upstream.


bower -> libman

PR Fixes #28, Visual Studio 15.8 introduced Libman(Library Manager) as an alternative to bower/yarn/npm.
More info about Libman.

I moved wrong branch :-)

Thanks! I don't think the template was itself using Bower but this should help those who use it as a starting point.


Images should not be indexed on alt, but rather unique id

Hi,
I believe that storing images based on the alt is not an good idea and instead unique id should be used. The reason is that, uploading two images with the same filename (thus the same alts), replaces the previous image.
Instead of
https://github.com/ccrsxx/twitter-clone/blob/c6bf6fba409cbff2ae57bc8069f1e00a202853a0/src/lib/firebase/utils.ts#L139
I would rather use
const storageRef = ref(storage, `images/${userId}/${id}`);

Yeah, this should be fixed. In my testing it doesn't replace any images but just discards the new one and places the original in the tweet instead. I'll fix the issues you found in my PR as well, thank you. I'll contact ccrsxx about the username bug as well.

@Ketchupchh thank you!

Yup, but that's by design.
I made it so that when we upload the same image with the same alt, it will return the old image instead of creating the new image. It won't replace the old image with the new image tho.
This is the code:
https://github.com/ccrsxx/twitter-clone/blob/c6bf6fba409cbff2ae57bc8069f1e00a202853a0/src/lib/firebase/utils.ts#L141-L146
I did it to reduce the bandwidth usage on the storage.
But yeah, that's bad if you upload a different picture with the same alt, it'll return the old one.
I see that the bandwidth usage isn't that much, so I think replacing the alt to id is good idea now.

@ccrsxx Well, if you want to do this kind of optimisation I think that the better option would be to calculate a hash (e.g. sha256) of a file's content, and use that instead of a filename. That way, the same images with different names, would be stored only once.

Yup, that could be the solution. But for now, I''ll be using the randomly generated id.

Fixed in #40


Add graph pass and backward option

Transfer for PR #20837

Thank you @RickyLLu!

@mxnet-bot run ci [all]


Timed out trying to read data from the socket stream

I am getting this exception when trying to upload some files. Small files (2k) are okay, but this one (5k) always fails. Filezilla works fine.
Trace:
# Connect()
Status:   Connecting to X.X.X.X:21
Response: 220 (vsFTPd 3.0.3)
Command:  USER ***
Response: 331 Please specify the password.
Command:  PASS ***
Response: 230 Login successful.
Command:  FEAT
Response: 211 End
Response: 211-Features:
Response: UTF8
Response: EPRT
Response: EPSV
Response: MDTM
Response: PASV
Response: REST STREAM
Response: SIZE
Response: TVFS
Status:   Text encoding: System.Text.UTF8Encoding
Command:  OPTS UTF8 ON
Response: 200 Always in UTF8 mode.
Command:  SYST
Response: 215 UNIX Type: L8
Status:   Auto-detected UNIX listing parser
Uploading 170315_s3_170804_105536_768.xml...
# Upload("/results/170315_s3_170804_105536_768.xml", Overwrite, False)

# FileExists("/results/170315_s3_170804_105536_768.xml")
Command:  SIZE /results/170315_s3_170804_105536_768.xml
Response: 213 4296

# DeleteFile("/results/170315_s3_170804_105536_768.xml")
Command:  DELE /results/170315_s3_170804_105536_768.xml
Response: 250 Delete operation successful.

# OpenWrite("/results/170315_s3_170804_105536_768.xml", Binary)
Command:  TYPE I
Response: 200 Switching to Binary mode.

# GetFileSize("/results/170315_s3_170804_105536_768.xml")
Command:  SIZE /results/170315_s3_170804_105536_768.xml
Response: 550 Could not get file size.

# OpenPassiveDataStream(AutoPassive, "STOR /results/170315_s3_170804_105536_768.xml", 0)
Command:  EPSV
Response: 229 Entering Extended Passive Mode (|||37199|)
Status:   Connecting to <IP_ADDRESS>:37199
Command:  STOR /results/170315_s3_170804_105536_768.xml
Response: 150 Ok to send data.
Exception: Timed out trying to read data from the socket stream!

Even if I delete the file first with Filezilla the same thing happens. The actual file appears on the server OK.

Ignore, it turned out to be the MTU. Microsoft Azure set its Ubuntu server's MTU to 1500, which doesn't work.


Task - "Assignee" field is grayed out in the offline mode

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.12-0
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Open the testing app.
Go to any Task report
Disable the internet connection
Update task title/description/assignee
Verify that the updated fields are greyed out
Enable the internet connection on the device
Verify that fields are not greyed out
Disable the internet connection again
Verify that fields are not greyed out

Expected Result:
The fields in the Task Report are not greyed out after going offline second time (after update and being online).
Actual Result:
The "Assignee" field gets greyed out when going offline second time.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/037275e9-e93c-446d-ab0b-b158f193cfa3

View all open jobs on GitHub

Production
https://github.com/user-attachments/assets/f1b254ed-c292-40c9-8339-f4e4746a73ff

Almost sure this is not a backend problem, removing DB label

I just tested this both on dev and staging and I can't reproduce it. Can you retest please?

Tester can reproduce this issue
https://github.com/user-attachments/assets/28c6c3b7-eb04-4202-995a-bd50285f3620

For me it's happening only in group chat and only with assignee. Check my video please
https://github.com/user-attachments/assets/4c176255-b400-4f09-aa3f-3236d4773122

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Context https://expensify.slack.com/archives/C9YU7BX5M/p1722275824988609?thread_ts=1722260075.337219&cid=C9YU7BX5M
Removed blocker because we can't reproduce and it seems @izarutskaya's test went offline before the request to edit the task finished.

asking for fresh repro steps
https://expensify.slack.com/archives/C9YU7BX5M/p1722552223955659?thread_ts=1722260075.337219&cid=C9YU7BX5M

I cannot repro this issue anymore
Build v9.0.16-0
https://github.com/user-attachments/assets/3e6e3355-c780-42c3-92bb-e4308d685178


test: Add input byte size tests using C APIs

What does the PR do?
Add tests using C APIs to ensure input byte size mismatches are correctly handled.
Checklist

[x] PR title reflects the change and is of format <commit_type>: <Title>
[x] Changes are described in the pull request.
[x] Related issues are referenced.
[ ] Populated github labels field
[x] Added test plan and verified test passes.
[ ] Verified that the PR passes existing CI.
[x] Verified copyright is correct on all changed files.
[ ] Added succinct git squash message before merging ref.
[ ] All template sections are filled out.
[ ] Optional: Additional screenshots for behavior/output changes with before/after.

Commit Type:

[x] test

Related PRs:
https://github.com/triton-inference-server/server/pull/7372
Where should the reviewer start?
N/A
Test plan:

CI Pipeline ID:
16168987

Caveats:
N/A
Background

Adding server level input byte size checks (TBD) will invalidate current tests in L0_input_validation since the error returned from server will be different from core.
Some sophisticated logics were added in infer_request.cc that requires granular C API tests. See https://github.com/triton-inference-server/core/pull/364

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)
N/A

Updated CI Pipeline ID @GuanLuo


Redirecting to success page

Hello, i'm sorry i had to comment on this issue. I still have a problem redirecting to success page while testing. After a payment, i see A transaction successful pop up and it disappears but the page page remains the same. It is not redirected to success page.
Magento ver. 2.26
Paystack: Magento2.1 plugin (installed using composer)
PHP Version: 7.1
Server: Linux on Google Compute engine
Originally posted by @abbeyseto in https://github.com/PaystackHQ/paystack-magento2/issue_comments#issuecomment-429990057

Duplicate of #6

Hello,
I'm also experiencing the same issue: 400 (bad request). Can someone share a working fix here? I've looked through the thread and haven't found one.


az containerapp up error : 'NoneType' object has no attribute 'get'

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command

az containerapp up --name nscontainerapp-pamjobsservice --resource-group nsrg_pam --image nsresgistry.azurecr.io/pam.jobsservice:<IP_ADDRESS>-01 --debug
Describe the bug

cli.azure.cli.core.azclierror: The command failed with an unexpected error. Here is the traceback:
az_command_data_logger: The command failed with an unexpected error. Here is the traceback:
cli.azure.cli.core.azclierror: 'NoneType' object has no attribute 'get'
Traceback (most recent call last):
File "/usr/lib64/az/lib/python3.9/site-packages/knack/cli.py", line 233, in invoke
cmd_result = self.invocation.execute(args)
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/init.py", line 663, in execute
raise ex
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/init.py", line 726, in _run_jobs_serially
results.append(self._run_job(expanded_arg, cmd_copy))
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/init.py", line 718, in _run_job
return cmd_copy.exception_handler(ex)
File "/home/artur/.azure/cliextensions/containerapp/azext_containerapp/_client_factory.py", line 28, in _polish_bad_errors
raise ex
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/init.py", line 697, in _run_job
result = cmd_copy(params)
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/init.py", line 333, in call
return self.handler(*args, **kwargs)
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
return op(**command_args)
File "/home/artur/.azure/cliextensions/containerapp/azext_containerapp/custom.py", line 2726, in containerapp_up
up_output(app, no_dockerfile=(source and not _has_dockerfile(source, dockerfile)))
File "/home/artur/.azure/cliextensions/containerapp/azext_containerapp/_up_utils.py", line 864, in up_output
url = safe_get(
File "/home/artur/.azure/cliextensions/containerapp/azext_containerapp/_utils.py", line 1063, in safe_get
return model.get(keys[-1], default)
AttributeError: 'NoneType' object has no attribute 'get'
az_command_data_logger: 'NoneType' object has no attribute 'get'
Traceback (most recent call last):
File "/usr/lib64/az/lib/python3.9/site-packages/knack/cli.py", line 233, in invoke
cmd_result = self.invocation.execute(args)
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/init.py", line 663, in execute
raise ex
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/init.py", line 726, in _run_jobs_serially
results.append(self._run_job(expanded_arg, cmd_copy))
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/init.py", line 718, in _run_job
return cmd_copy.exception_handler(ex)
File "/home/artur/.azure/cliextensions/containerapp/azext_containerapp/_client_factory.py", line 28, in _polish_bad_errors
raise ex
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/init.py", line 697, in _run_job
result = cmd_copy(params)
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/init.py", line 333, in call
return self.handler(*args, **kwargs)
File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
return op(**command_args)
File "/home/artur/.azure/cliextensions/containerapp/azext_containerapp/custom.py", line 2726, in containerapp_up
up_output(app, no_dockerfile=(source and not _has_dockerfile(source, dockerfile)))
File "/home/artur/.azure/cliextensions/containerapp/azext_containerapp/_up_utils.py", line 864, in up_output
url = safe_get(
File "/home/artur/.azure/cliextensions/containerapp/azext_containerapp/_utils.py", line 1063, in safe_get
return model.get(keys[-1], default)
AttributeError: 'NoneType' object has no attribute 'get'
To Reproduce

Expected behavior

Environment summary

azure-cli                         2.43.0
core                              2.43.0
telemetry                          1.0.8
Extensions:
containerapp                      0.3.16
ai-examples                        0.2.5
ml                                2.11.0
ssh                                1.1.3
Dependencies:
msal                              1.20.0
azure-mgmt-resource             21.1.0b1
Additional context

@kanuw  I am not able to reproduce this issue.

Can you please try updating your extension to version 0.3.17.

Upgraded to version 0.3.18 and now it's working


enumeration in different environments

This depends on #60 . If the defineProperty is not supported properly, then probably the enumerable option is not supported as well. E.g. old msie iterates over non enumerable properties. So for example {} contains toString as enumerable and [] will contain slice, length, etc... as enumerable properties.
I have to do some research where this is supported, and if possible avoid to write a polyfill about that. I'd rather throw an incompatibility error by the environments which does not support proper enumeration. I don't want to write an iterator class if not necessary. I have to hide properties like id and _observer, but those can be added as exceptions to the iterator... If I have to write one, because some env is important, then I'll do something like this:
for (var i = iterate(o); i.hasNext; i.next()) {
    var value = i.value;
    console.log(value);
}

or
var i = iterate(o);
var value = i.value;
while(i.hasNext){
    console.log(value);
    value = i.next();
}

or (by non falsy values)
var i = iterate(o).rewind(-1);
while(var value = i.next())
    console.log(value);

it is a bit ugly and slow compared to
for (var key in o) {
    var value = o[key];
    console.log(value);
}

but it is still not as ugly and slow as
Object.each(o, function (value){
    console.log(value);
});

because you cannot use break & continue (only with try-catch hax) and you have to give the context if you want to use this

Moved to o3 https://github.com/inf3rno/o3/issues/58


Unable to convert open API 3 to 2

Tried to convert https://int-hub.totango.com/integration-hub-api_openApi.json and failed

Can you share more details? This command works fine for me:
api-spec-converter --from openapi_3 --to swagger_2  https://int-hub.totango.com/integration-hub-api_openApi.json

nevermind. Managed without this


[web] Canvas cannot be exported to bitmap

The problem was already mentioned in https://github.com/flutter/flutter/issues/57659 but I'm afraid it wouldn't get the required attention there. That bug was originally about resizing and the second bug is even much more serious.
Steps to Reproduce
  final pictureRecorder = ui.PictureRecorder();
  final canvas = Canvas(pictureRecorder);
  canvas.draw... // any drawing operation
  final picture = pictureRecorder.endRecording();
  final exported = await picture.toImage(width, height); //!!! not working

results in the error below. Basically, the underlying HTML5 canvas variable seems to be null when the call is made.
resize_repro.zip
Expected results:
No error.
Actual results:
TypeError: Cannot read property 'Symbol(dartx.toDataUrl)' of null
    at _engine._CanvasPool.new.toDataUrl (http://localhost:11103/dart_sdk.js:129211:28)
    at _engine.BitmapCanvas.new.toDataUrl (http://localhost:11103/dart_sdk.js:122112:32)
    at _engine.EnginePicture.new.toImage (http://localhost:11103/dart_sdk.js:136983:35)

Doc
[√] Flutter (Channel beta, 1.18.0-11.1.pre, on Microsoft Windows [Version 10.0.18363.836], locale en-US)
    • Flutter version 1.18.0-11.1.pre at E:\Android\flutter-beta
    • Framework revision 2738a1148b (10 days ago), 2020-05-13 15:24:36 -0700
    • Engine revision ef9215ceb2
    • Dart version 2.9.0 (build 2.9.0-8.2.beta)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
    • Android SDK at e:\Android\Sdk
    • Platform android-29, build-tools 29.0.1
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • CHROME_EXECUTABLE = e:\Android\chrome.bat

[√] Android Studio (version 3.6)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.8052
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[√] Connected device (2 available)
    • Web Server • web-server • web-javascript • Flutter Tools
    • Chrome     • chrome     • web-javascript • Google Chrome 81.0.4044.138

• No issues found!

HI @deakjahn
I tried to refactor your code to get a better understanding,
but I feel I didn't go anywhere
(here's the code anyway
Tested with the latest master
I'm getting an undesired behaviour for the reduced
and no data for resized
Thank you

The repro was originally made for the other reported bug. There was a problem with resizing, too. When I made a secondary bug report with the bigger problem, I didn't create a new repro. So, what pertains to this is the third item.
return await _pictureRecorder.endRecording().toImage(_width(), _height());

This gives an error for all my attempts, and by checking the source, this leads back to the wrapped _canvas of CanvasPool (if my memory serves right) being null when toImage() is called.

There is no ui.Image.toByteData(), either in DomCanvas or CanvasKit. Both return unimplemented. Any image manipulation beyond simply displaying whatever you already have is currently impossible.

There is no Picture.toImage(), so canvases cannot be converted into bitmaps. This means that there is no workaround for the above by drawing your image on a canvas and exporting that. Besides, drawing to the canvas would require somehow getting hold of the byte data of the image, and there is no toByteData(), as mentioned above.

We track this issue in the following two issues:
https://github.com/flutter/flutter/issues/42767 (CanvasKit)
https://github.com/flutter/flutter/issues/49857 (HTML)


Send error payload with MQTT, especially when NTP/RTC is out of sync

THIS IS A DRAFT
We are lacking an error reporting funnel.
When we have a Big Problem™, we should record it somewhere, or send it somewhere, if possible. Flashing a LED just doesn't cut it. Here is a proposed solution:
What to to when we detect an error:

obviously flash the LED and log an error message (already working somehow)
prepare an MQTT message with an error payload and possibly error properties
if we can send it, send it
if we can't send it, record it (and then replay it)

How to format the error message:
we SHALL have a global errors object, with one entry per error type.

the entry key MUST be the error code, so that the latest error of that type remains in the shadow
the entry value CAN be an object with nested properties. Properties differ with respect to the error type.

Example
"state" : {
  "reported" : {
    "errors" : {
      "ntp": {
        "lastNtpSync": "date_or_null",
      },
      "spiffs": null,
      "rtc" : {
        "stopped": {
          "lastSyncOffset": "diff_between_rtc_time_and_last_ntp_sync"
        }
      }
    }
  }
}

can be right to get also a timestamp ?


Share windows enviroment variables with WSL

I think could be nice to access Windows environment variables from wsl.
Could be variables starting with prefix WIN_, like $WIN_PATH, $WIN_USERPROFILE, etc...

You can access them with cmd.exe:
$ /mnt/c/Windows/System32/cmd.exe /c echo %PATH%
C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;...

Yes, I know that but, I hope some solution that reformats output to WSL Path instead Win Path, I mean rewrite \ to / and change drive C: to /mnt/c

win_path_to_wsl_path() {
    driver=$(echo ${1:0:1} | tr 'A-Z' 'a-z')
    other=${1:2}
    echo /mnt/${driver}${other//\\/\/}
}

$ win_path_to_wsl_path "C:\WINDOWS\system32"
/mnt/c/WINDOWS/system32


Update Drupal to 9.2.10

@bseeger I needed to specify symfony/string version ^5.4 in the composer.json file because the
upgrade command was trying to bring in version 6.x of that module, which requires PHP 8.


cc_banner-wrapper height on the basis of device.

we are using the script for desktop as well as mobile sites.
height of cc_banner-wrapper(relative block) is 66px . so for devices like ipad on which the inner content in 'cc_banner cc_container cc_container--open container'(fixed block) comes in multiple lines it's height increases and it hides the content below it. is there any portable solution for all mobile devices(so that height of cc_banner-wrapper is set to height of cc_banner)? do we have nay callback function?

Hi, I'm not sure of a good solution though, if the height of the banner didn't increase the text would get clipped!

I don't want text to be clipped so i can't keep my height fixed. i want its height to change dynamically and set accordingly so that content below it also doesn't hide.

If you don't want content to be hidden behind the banner, how about using the version that pushes the content down, instead of covering it. It's the 5th option from the left on this page https://silktide.com/tools/cookie-consent/download/

Tried to change visibility options but in mobile devices notification covers all the screen... can you help?


fix(naming): rename template styles.scss into componentName.scss

styles.scss doesn't exist anymore in the generated component.
use the componentName.scss file.

Deploy preview for pf-next ready!
Built with commit a64d1d3e20eab72a9dc82ea2f402261bb52a8f7d
https://deploy-preview-861--pf-next.netlify.com


connect doesn't apply ref to stateless component

Took a stab at #141
Probably would be better to create the render function beforehand, instead of hitting that conditional every render. Just getting this out there for discussion.
Worth noting that the detection isn't perfect, but solves the case mentioned here, and keeps the current test for wrappedInstance ref passing. We'd have to instantiate the component to know for sure, but this might be good enough.
Pretty much stole the check from ReactCompositeComponent

Wouldn't this give false positives for module pattern of non-Component-inheriting classes which are still valid in 0.14?

Yes, indeed. I'm not sure at this point how you'd figure that out before mounting the component. I'll keep hacking on it.

Thanks for the PR.
In the end I decided on a simpler approach: refs are now opt-in at the connect() call site.
Closed via https://github.com/rackt/react-redux/commit/2d3d0beade55477b3af65534ceb793db18b25705.


Introduce biome families

Group existing, detailed biomes in groups, so that less sophisticate user can use them

Actually, the ancient maps already kind of do most of that.
I did find a website a while that reorganized the 39 biomes into 13 groups. The breakdown is as follows:
Polar Ice, Polar Desert = Ice
Subpolar Moist Tundra, Subpolar Wet Tundra, Subpolar Rain Tundra = Tundra
Subpolar Dry Tundra, Boreal Desert, Boreal Dry Scrub = Cold Parklands
Boreal Moist Forest, Boreal Wet Forest, Boreal Rain Forest = Coniferous Forests
Cool Temperate Moist Forest, Cool Temperate Wet Forest, Cool Temperate Rain Forest = Mixed Forests
Cool Temperate Steppe = Steppes
Cool Temperate Desert, Cool Temperate Desert Scrub = Cool Deserts
Warm Temperate Moist Forest, Warm Temperate Wet Forest, Warm Temperate Rain Forest = Deciduous Forests
Warm Temperate Thorn Scrub, Warm Temperate Dry Forsest = Chapparal
Warm Temperate Desert, Warm Temperate Desert Scrub, Subtropical Desert, Subtropical Desert Scrub, Tropical Desert, Tropical Desert Scrub = Hot Deserts
Tropical Thorn Woodland, Subtropical Thorn Woodland, Tropical Very Dry Forest = Savannah
Subtropical Dry Forest, Tropical Dry Forest = Tropical Dry Forests
Subtropical Moist Forest = Tropical Seasonal Forests
Subtropical Wet Forest, Subtropical Rain Forest, Tropical Moist Forest, Tropical Wet Forest, Tropical Rain Forest = Tropical Rain Forests.
I also created a script for GIMP that would allow you to open up the Biome PNG, run the script, and that would organize things up for you. I'll go ahead and include that when I do a Pull request.


SSL/TLS invalid certificate

After entering the page https://gitea.io/:
gitea.io uses an invalid security certificate. The certificate expired on 28 August 2018, 18:33:27 GMT+2. The current time is 28 August 2018, 19:00. Error code: SEC_ERROR_EXPIRED_CERTIFICATE

The certificate is expired, if you really want to see the website and are using chrome, type in "thisisunsafe" to bypass the error.

ping @tboerger

We had initial reached the limit of new Let's Encrypt certificates after moving to Netlify hosting. After that I simply forgot to enable Let's Encrypt again ;)


Conditional Access: Admin Portals Needed for 'Install Apps' Functionality

Greetings,
I work in education, and it is often necessary to block our student accounts from various resources.  They are frequently compromised, and are sometimes created with fraudulent applications.  To that end, we recently blocked student access to 'Microsoft Admin Portals.'  Sadly, our legitimate students are now unable to install 365 apps on their devices, and we have no idea which admin portal is needed for this functionality.  Not only that, but the 'Microsoft Admin Portals' "app" is all inclusive, so there's no way we can grant access to just the one, we have to now grant access to all.  It would be wonderful if the 'Install Apps' feature wasn't part of an admin portal.  It would also be nice to be able to block access to these portals individually, instead of the current all-or-nothing approach.
With Gratitude!!

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

ID: 1c6d2ea9-04c9-d4e5-36b5-2ce7e17192e1
Version Independent ID: eff8d31c-d3ce-f72c-22ab-53b415b221b0
Content: Cloud apps, actions, and authentication context in Conditional Access policy - Microsoft Entra ID
Content Source: docs/identity/conditional-access/concept-conditional-access-cloud-apps.md
Service: entra-id
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@BikerAdmin
Thanks for your feedback! We will investigate and update as appropriate.

Hi @BikerAdmin
Thanks for your contribution. Please add your ideas in below link, so our production team can review it and update the same.
Ideas · Community (azure.com)


[4.3 Beta 3] Strange editor brightness and colors

Tested versions
Tested with v4.3.beta3.official [82cedc83c], seems to happen as far back as v4.3.beta1.official [a4f2ea91a]. If I use v4.3.dev6.official [89850d553] or older, the issue is no longer present.
System information
Godot v4.3.beta3 - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 (NVIDIA; <IP_ADDRESS>12) - 11th Gen Intel(R) Core(TM) i5-11600K @ 3.90GHz (12 Threads)
Issue description
In Godot 4.3 Beta 3, the editor is quite a bit brighter (4.2.2 Stable on the left, 4.3 Beta 3 on the right)

It seems sometimes clicking some things will change the brightness or color of the editor very subtly as well, as shown here:
https://github.com/user-attachments/assets/50c26e81-0a20-48d5-abc4-542329d42888
Steps to reproduce

Open Godot 4.3 beta (1-3)
Open a project (can be a new / empty one)
The editor will be bright as shown in the picture

Minimal reproduction project (MRP)
N/A

Do you have HDR enabled on the desktop? Do you have RTX HDR enabled?
RTX HDR will affect Godot, but once it becomes idle for a few seconds, it will no longer affect Godot unless Update Continuously is enabled in the Editor Settings.
If there's a way to disable RTX HDR on a specific using an application profile, it might be worth making a PR similar to https://github.com/godotengine/godot/pull/93737 so that it's never active on the Godot editor. It should remain usable on exported projects though (and perhaps projects being run from the editor too).

It seems that it was caused by RTX Dynamic Vibrance.


Localize views

Finishes #7

Payeasy:

Bank Transfer:

@camelmasa do you want to check this one too? :smile:

@shioyama I'll do that :fireworks:

@shioyama Thank you for created that :dancers: LGTM :+1:

@camelmasa thanks for reviewing!


Final install.sh

Thank you @nathom for suggesting the IF statement applications very much! This helps massively in lowering download/update speeds, I've added package checks as well and many more!
This is the final version of install.sh, (please tell what further can we do)
I've also updated my version to work with the DMG installer.
Please suggest what can I do further

Cyan is not sus. Cyan was doing wires.

Nooo but I saw him kill red and vent

We can also add
xattr command, it gets rid of apple quarantine?

Will see


Live mode does object repository makes sense and implementable? - KIV for now

As above.
Also, in live mode, below returns ERROR - unexpected EOF because there isn't a parse_backtick() to do validation of having an even number of backtick ` characters.
echo `variable

KIV for now, and error is an edge case without impact


Match non-terminal glob values

This change enables non-terminal *s in glob values , so long as they are separated by / . Previously (currently) parseGlob could handle matching strings like first/*/third, but the the condition for matching with parseGlob could never trigger since the * occurs before the end of the string, and isGlobValue only checks end of string to determine if parseGlob will be called .

Excusing myself from review: @alexboring if you need something specific call it out, else I defer to @mikeybtn as owner here.

Excusing myself from review: @alexboring if you need something specific call it out, else I defer to @mikeybtn as owner here.

It was @mikeybtn 's ask.

@danielmcgrath for old time's sake (because it's open source) :)

Looks neat! Though I wonder if it would be more useful to just allow the use of a regex? e.g.:
{
  "operation": {
    "path": "/foo/bar",       <- Simple string key like now, with simple glob support
    "method": {
      "match": "/put|post/i"  <- Complex matcher
    }
  }
}

i'll take it from here - alex, please add test coverage. need to see that this doesn't break the rule reachability validation in addRule().

i'll take it from here - alex, please add test coverage. need to see that this doesn't break the rule reachability validation in addRule().

@danielmcgrath thanks for popping by, we didn't mean to delete your comment!
on the comment you left - we had an internal discussion about adding that kind of expressivity here thru regex, and ended up lukewarm at best on it - it adds more complexity to configs which at least in our cases, could be just as well handled by the sender/client (eg client can send key="/foo/:bar" rather than building a harder-to-scan config like key=/\foo\/(.+)) ..
glob is a small exception to strict k/v, but like to keep from growing until we have a much stronger case.

@mikeybtn Let me know if you think I should add comments to the test. But I think that covers what we're trying to guarantee.

LGTM. Please update CHANGELOG.md and package.json with a new release number; see db8c56ace4a27e0fc4932e072b2254a26e748fdb for the last example.

@mikeybtn Look good to you?


Log Module

Its impossible to get log module to work and its bad written as no one understands how it actully works.
Insted of do "SetRules whatever" in one line why don´t have it as perform and let you add and delete stuff one by one?
And insted of have it to log everything by defualt why don´t let you don´t log anything by default or ignore everything by default depending on how mutch you like to log
Like me i just want to log one channel but it seems impossible to make it do that as it log everything whatever i do, the documentation for it don´t tell you mutch about how the rules for it work ether.
I have try whit "SetRules #znc !*" as it in my eyes match #znc first and that pass while the other rule ignore everything else but this don´t seems to be the case

The documentation is straight forward and easy to understand. Ask in #znc if you don't understand something.

And insted of have it to log everything by defualt why don´t let you don´t log anything by default or ignore everything by default depending on how mutch you like to log

That's not the purpose of log. It works as intended and logs everything by default instead of having to add every single rule that you want logged, just like any other similar piece of code.
If you want to log one particular channel without logging everything else, use the proper blacklisting syntax with wildcards.

I have actully asked there 2 days now, and no one seems to understand how the module is working

I run 1.6.3 and have this rules now
[2017-05-08] [22:35:56]  »» [Whiskey] setrules #php.se !#* !* [2017-05-08] [22:35:56]  »» [*log] +=========+=================+ [2017-05-08] [22:35:56]  »» [*log] | Rule | Logging enabled | [2017-05-08] [22:35:56]  »» [*log] +=========+=================+ [2017-05-08] [22:35:56]  »» [*log] | #php.se | true | [2017-05-08] [22:35:56]  »» [*log] +---------+-----------------+ [2017-05-08] [22:35:56]  »» [*log] | #* | false | [2017-05-08] [22:35:56]  »» [*log] +---------+-----------------+ [2017-05-08] [22:35:56]  »» [*log] | * | false | [2017-05-08] [22:35:56]  »» [*log] +=========+=================+
It still seems to log everything

its bad written as no one understands how it actully works

That's not a good way to ask for help, you know.
How many log modules do you have loaded? Show /znc listmods

Look like i have one DarthGandalf

[2017-05-09] [01:27:05]  »» [*status] Global modules:
[2017-05-09] [01:27:05]  »» [*status] +===========+===========+
[2017-05-09] [01:27:05]  »» [*status] | Name | Arguments |
[2017-05-09] [01:27:05]  »» [*status] +===========+===========+
[2017-05-09] [01:27:05]  »» [*status] | webadmin | |
[2017-05-09] [01:27:05]  »» [*status] +-----------+-----------+
[2017-05-09] [01:27:05]  »» [*status] | modperl | |
[2017-05-09] [01:27:05]  »» [*status] +-----------+-----------+
[2017-05-09] [01:27:05]  »» [*status] | modpython | |
[2017-05-09] [01:27:05]  »» [*status] +===========+===========+
[2017-05-09] [01:27:05]  »» [*status] User modules:
[2017-05-09] [01:27:05]  »» [*status] +============+===========+
[2017-05-09] [01:27:05]  »» [*status] | Name | Arguments |
[2017-05-09] [01:27:05]  »» [*status] +============+===========+
[2017-05-09] [01:27:05]  »» [*status] | disconkick | |
[2017-05-09] [01:27:05]  »» [*status] +------------+-----------+
[2017-05-09] [01:27:05]  »» [*status] | chansaver | |
[2017-05-09] [01:27:05]  »» [*status] +------------+-----------+
[2017-05-09] [01:27:05]  »» [*status] | push | |
[2017-05-09] [01:27:05]  »» [*status] +------------+-----------+
[2017-05-09] [01:27:05]  »» [*status] | fix_lagchk | |
[2017-05-09] [01:27:05]  »» [*status] +============+===========+
[2017-05-09] [01:27:05]  »» [*status] Network modules:
[2017-05-09] [01:27:05]  »» [*status] +==========+===========+
[2017-05-09] [01:27:05]  »» [*status] | Name | Arguments |
[2017-05-09] [01:27:05]  »» [*status] +==========+===========+
[2017-05-09] [01:27:05]  »» [*status] | fish | |
[2017-05-09] [01:27:05]  »» [*status] +----------+-----------+
[2017-05-09] [01:27:05]  »» [*status] | perform | |
[2017-05-09] [01:27:05]  »» [*status] +----------+-----------+
[2017-05-09] [01:27:05]  »» [*status] | nickserv | |
[2017-05-09] [01:27:05]  »» [*status] +----------+-----------+
[2017-05-09] [01:27:05]  »» [*status] | keepnick | |
[2017-05-09] [01:27:05]  »» [*status] +----------+-----------+
[2017-05-09] [01:27:05]  »» [*status] | crypt | |
[2017-05-09] [01:27:05]  »» [*status] +----------+-----------+
[2017-05-09] [01:27:05]  »» [*status] | log | |
[2017-05-09] [01:27:05]  »» [*status] +==========+===========+

As verified on IRC, it works.


Compiling zxing with msvc

I compiled zxing-cpp successfully with the msvc compiler but after including it in my own program it does not compile.
The standard library bit was not included in the Bithacks.h file.

After some searching I found out that the msvc compiler sets __cplusplus to 199711L per default.
The solution is to set the compiler option /Zc:__cplusplus (see here).
Maybe this could be mentioned in the documentation or checked differently?
Additional Information:

MSVC Version 19.40.33816
C++ Standard 20

First of all, it should compile without the <bit> header. Can you post the error message (preferably as text) that you get without setting /Zc:__cplusplus?
Second: can you check if replacing the #if __has_include(<bit>) ... line with the following line fixes your compile issue?
#if __has_include(<bit>) &&  (__cplusplus > 201703L || _MSVC_LANG > 201703L)

If I compile it without the /Zc:__cplusplus I get no member errors (in BitHacks.h):
error C2039: "countl_zero" is no member of "std". (line 42)
error C2039: "countr_zero" is no member of  "std". (line 86)
error C2039: "popcount" is no member of  "std". (line 142)
error C3861: "popcount": identifier not found. (line 142)

The __cpp_lib_bitops is defined but the __cplusplus variable is wrong as described before.
Your solution works perfectly (predefined macros)! Therefore /Zc:__cplusplus is not needed anymore.
#if __has_include(<bit>) &&  (__cplusplus > 201703L || _MSVC_LANG > 201703L)

Could you do the me the favor and check if the following code also works for you without /Zc:__cplusplus?
#if __has_include(<bit>) && __cplusplus > 201703L
#include <bit>
#if __cplusplus > 201703L && defined(__ANDROID__) // NDK 25.1.8937393 has the implementation but fails to advertise it
#define __cpp_lib_bitops 201907L
#endif
#else
#undef __cpp_lib_bitops
#endif


drop run!

We dropped all references to run! in 46c175f ( https://github.com/ManageIQ/floe/pull/149 )
Followup dropping it from from abstract interface: Floe::Runner#run!
Added required Runner#status! method to parent interface. It is called from Task.
Added yarddoc for the interface

update:

added method status! to the interface.
added yarddoc

@agrare @Fryguy Curious if you both want status! as part of the interface or want to make running? a method that can change the runner_context.
Almost feel like it could be renamed to check_status!
And calling it from Task#running?

update:

renamed run_async!(image, ...) to `run_async!(resource, ...) - no real change


feat(android): implemented getSupportedMediaTypeList method

Description

Added getSupportedMediaTypeList() that allows to find supported Codecs for device
Compatibility

OS
Implemented

iOS
❌

Android
✅

Windows
❌

Checklist

[X] I have tested this on a device/simulator for each compatible OS
[X] I added the documentation in README.md
[X] I updated the typings files (privateTypes.ts, types.ts)
[X] I added a sample use of the API (example/App.js)

@mikehardy as all checks has passed can you please merge this pr

Sure, note that maintainers sleep and have patience please. I checked it alike 9pm my time and it's only 6am now. I have to get my kid to school. Should I drop what I'm doing it should you be okay with waiting a bit...


doc: cmake: Unify getting started guides

Unify the format and mechanisms used in the different Getting Started
guides so that they are consistent in:

The way Kconfig is built
Avoiding using -B and -H CMake options
-DBOARD instead of export

Signed-off-by: Carles Cufi<EMAIL_ADDRESS>

I'm fine with this as is for the record. If we have to change this again, I think it'd be nice to use zephyr-app-commands just to keep things synchronized.

I will do that, I just needed a quick fix for today. Also the stuff in samples and other places where we give build instructions.

Yup, I'm going to change as much of that as I have time for. I'll take boards/ at least


fatal error: concurrent map read and map write

What's going wrong?
Client receives the following error. Pilosa cluster falls apart.
com.pilosa.client.exceptions.PilosaException: Server error (500): {"success":false,"error":{"message":"sending CreateField message: executing request: getting response: Post http://<IP_ADDRESS>:10101/internal/cluster/message: EOF"}}
What was expected?
The same test works on single instance Pilosa server. Expected it to work with clustered setup.
Steps to reproduce the behavior
Create 10 000 integer fields with range 0-100 in an index.
Information about your environment (OS/architecture, CPU, RAM, cluster/solo, configuration, etc.)
Google Kubernetes cluster with container optimized OS. Tried with 1-32 cores, 1-30 GB RAM. Solo works. Cluster with 3 nodes fails.
Pilosa 1.4.0, Java client 1.3.5.
Attaching log extract with concurrent map read and map write [pilosa__logs__2019-09-20T01-44.txt]

pilosa__logs__2019-09-20T01-44.txt
the logs

Thank you for reporting this, I see the problem.
We have an unlocked access to Index.createField at line 604 of server.go, should be a straightforward fix.
As a side note, would love to hear more about what you're doing with Pilosa. Currently I don't expect Pilosa to behave very well with 10s of thousands of fields, though this is a use case which has popped up more than once and we have some possible solutions. Would be great to get your input if you don't mind getting in touch, either on gopher slack, or via email jaffee at pilosa.com

Thank you for the fix. It doesn't fail now.


abort() works, but still get "Warning: Can't perform a React state update on an unmounted component". CodeSandbox provided

When returning abort as the useEffect cleanup function, the request aborts and the promise resolves as expected, but I still see the React warning: Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in a useEffect cleanup function.     in Test (at App.js:15)
Question: How do I cancel a fetch on unmount (and not receive the React warning)?
In case this Codesandbox link expires, I'll paste the code below. I tried to keep it simple -- one component unmounts another component before its fetch finishes
App.js - unmount <Test /> after 100ms (to interrupt the API call made by <Test />)
import Test from "./Test";
import "./styles.css";

export default function App() {
  const [bool, setBool] = useState(true);

  // 100ms after mounting, set bool to false
  useEffect(() => {
    setTimeout(() => {
      setBool(false);
    }, 100);
  }, []);

  return
{bool && <Test />}
;
}

Test.js - Make an API call using useFetch, and return abort from useEffect
import useFetch from "use-http";

export default function Test() {
  const { get, abort } = useFetch(
    "https://jsonplaceholder.typicode.com/posts/1"
  );

  const loadIt = async () => await get();

  useEffect(() => {
    loadIt();

    return abort;
  }, []);

  return
I am here
;
}

It should abort already when unmounting. See this line. I will take a look at this asap

I see, cool. abort does cancel the request, and any statements following the call to req.get() will run immediately after abort is called. Which kinda makes the React warning seem unwarranted ... or I don't have enough knowledge about unmounting/cleanup

I think I have a fix for this issue in this PR. I block all state from updating if the component is not mounted, so if it still happens I will be very confused.

Awesome. I will check it out.

Awesome. I will check it out.

This should be fixed with v0.4.0. Let me know if the issue persists and I'll reopen this.


Complex logic before computed call to C++ method

Heya,
Sorry if my question is a bit vague... I don't know how to word this properly, as I'm not too well versed in communicating this stuff, but...
I'm trying to reverse-engineer a MIPS binary compiled from C++.
The way this program/architecture handles references to class fields/methods seems strange, and it doesn't decompile very nicely.
Here's what I mean:

Instead of it showing something like this->myMethod(this);, it's all messed up and complex looking.
My question is:
Is there a way to "suppress" this logic, so that it's easier to see what's going on, but without breaking the decompilation?
This happens for all class fields, and it makes figuring out what's going on extremely difficult (or basically impossible).
Thanks in advance :D
UPDATE:
I managed to clean it up somewhat - by creating a __thiscall function definition and splitting the pointer variable into 4 * func *8, instead of the full 32-bit address, but it's still basically unreadable/unusable.

Isn't there some way to make Ghidra treat the data in reverse endianness?
EDIT: Another thing to note is that this binary was apparently compiled using Green Hills toolsets, with some compiler optimization going on.

Gonna close this topic now, as I might have found the answer.
While a C++ compiler was used, as far as I can see, this program was written in C, but with a custom scripting language that calls functions based on arguments on a virtual stack, hence the complex looking call.
I might open a new thread with further questions about simplifying decompilation.

Gonna close this topic now, as I might have found the answer.
While a C++ compiler was used, as far as I can see, this program was written in C, but with a custom scripting language that calls functions based on arguments on a virtual stack, hence the complex looking call.
I might open a new thread with further questions about simplifying decompilation.


IncreasingPriceCrowdsale throws 'rate' rpc-json error when interacting with contract

-getCurrentRate' works as intended but 'rate' is still available to be viewed in deployed contract via Crowdsale.sol
Therefore, 'rate' throws rpc-json error when called.
'rate' should not be available to be viewed when using IncreasingPriceCrowdsale.

Sadly, the Solidity language does not let us remove a function from a contract's interface, so the best we can do is revert when rate is called on that particular crowdsale. The alternative would be to manually expose rate in all child crowdsales but that one, but I fear that would be too cumbersome and not worth the effort.


Unit test density matrix operations

regular gates (very tedious though - particular attention to measurements)
calcFidelity
calcPurity
calcTotalProbability
oneQubitDephase
twoQubitDephase
oneQubitDepolarise
twoQubitDepolarise
combineDensityMatrices
getDensityAmplitude

added calcFidelity unit test, but needs combineDensityMatrices to be completed

added unit test for

calcTotalProbability

noise functions unit tested by comparison to Monte Carlo sampling


Integration with Travis CI

This patch contains 3 main changes:

A test.sh file that runs basic check_pgactivity lines and stops if there's an error. The file is not complete yet, this is just a PoC. The script takes the major version as parameter

A dockertest.sh that launches the test.sh inside any official docker postgres image (see https://hub.docker.com/_/postgres/ ).  The script takes the major version as parameter

A Travis CI config file that launches dockertest.sh on every major version from 9.2 to 9.6. (the postgres 10 image is on its way)

As a result, any developper can create a Travis CI account, activate integration for the check_pgactivity repository and then every new commit will be checked like this :
https://travis-ci.org/daamien/check_pgactivity/builds/241807277

Hello,
Thank you for this (old) PR and sorry for the lack of feedback. I needed some time and more experience in testing env before being able to review your PR.
In the process of next release, I started working on this subjecta and had a look at your PR few weeks ago. There's some pros and some cons. In short:

we might want to use Travis in short/mid term
Travis allows to test on various OS, including ubuntu
we can test check_pga against many different packaged version of PostgreSQL on Ubuntu
no need or very little need of scripting with Travis/Ubuntu to setup the test environment
Docker has fewer PostgreSQL available and requires (at first glance) more scripting to setup the test environment
your testing script doesn't seem to be the best fit as we have a LOT to test in check_pga, not just the success return code. We need to tests error code as well, plus stdout, stderr, perdata and so on.

I started a branch on my repo to create tests based on the standard TAP tests protocol using the Perl modules from PostgreSQL project. See folder t/ in https://github.com/ioguix/check_pgactivity/tree/tests . Perl modules from PostgreSQL are really powerful and I'm quite glad to be able to deeply tests check_pga with so few lines of code thanks to them. As far as I understand it, this should perfectly fit in Travis as well.
I'll keep feeding my branch and will create a WIP PR soon to prepare for a merge in next release 2.6. You'll be welcome to join the effort in regard with Travis there if you agree and have time.
Thank you!


redundant namespace in debug logs

I was debugging my project and was trying to determine whether to pass socket.id or socket.conn.id in order to join the default room that each socket is placed in. I modified the debug statements in the broadcast function so I could better understand what was going on. I made the following changes (added one debug statement, modified another debug statement):
  Redis.prototype.broadcast = function(packet, opts, remote){
    packet.nsp = this.nsp.name;
    if (!(remote || (opts && opts.flags && opts.flags.local))) {
      var msg = msgpack.encode([uid, packet, opts]);
      var channel = this.channel;
      debug('[ broadcast ] - opts: %O',opts)
      if (opts.rooms && opts.rooms.length === 1) {
        channel += opts.rooms[0] + '#';
      }
      debug('publishing message to channel %s %O %O', channel, packet, opts);
      pub.publish(channel, msg);
    }
    Adapter.prototype.broadcast.call(this, packet, opts);
  };

I learned that socket.id is the correct room to join (default room) however when I do this the second debug statement displays the namespace in the channel twice.
That is when using namespaces, socket.id is /namespace#some-socket-id and the channel will be /namespace#/namespace#some-socket-id
Despite the namespace redundancy in the channel variable everything seems to work fine.
It makes sense that the namespace will be duplicated because

opts.rooms[0] will be /namespace#some-socket-id
nsp.name will be /namespace
and channel is set in the constructor:  this.channel = prefix + '#' + nsp.name + '#';

Again as far as I can tell this isn't causing an issue but I thought I would point this out in case there is something I have overlooked (and there really is an issue)

Closed due to inactivity, please reopen if needed.


Derived Variables

Allows storing new derived variable configs.
Allows listing stored derived variable configs by user.
Allows looking up target derived variable config.
Allows patching analyses to include/remove stored derived variables.

@Foxcapades Per conversation about copies and provenance, see the following pseudocode to implement during analysis copy/import:

Better devine provenance type; instead of any, use:
{
  copyDate: string (ISO timestamp)
  copiedFrom: string (derived variable ID)
}

On analysis import/copy:

analysisCopy = make a copy of the analysis and assign new analysis ID

if (different user) {
  // assign new owner's ID to copy
}

if (no derived vars in copy) {
  // save off analysisCopy; done!
  return
}

// get all newOwner's existing DVs from derived_variable table

var newDerivedVarRefs = new ArrayList // DV refs to assign to analysisCopy

for (derivedVarRef in analysisCopy) {

  if (any existing DV has provenance that matches derivedVarRef) {
    // don't need a second copy; use the ID of the existing copy
    // note: displayName and description will NOT update; don't want to overwrite any changes the newOwner made
    newDerivedVarRefs.add(existing DV's ID)
  }
  else {
    // need to make a copy of this DV
    oldDv = lookUpDvToBeCopied(derivedVarRef)
    newDv = copyWithNewID(oldDv)
    newDerivedVarRefs.add(newDv.id)

    // handle provenance chain
    if (oldDv.hasProvenance) {
      newDv.provenance = {
        copyDate: new Date().isoFormat,
        copiedFrom: oldDv.provenance.copiedFrom // reference the original DV
      }
    }
    else {
      newDv.provenance = {
        copyDate: new Date().isoFormat,
        copiedFrom: oldDv.id
      }
    }
  }
}

analysisCopy.derivedVars = newDerivedVarRefs
// save off analysisCopy


unnecessary methods removal

Methods that do not make sense in a model should be removed and added to the appropriate class/helper.

closing as done


Unicode characters arent rendered right

Please use the following bug reporting template to help produce actionable and reproducible issues. Please try to ensure that the reproduction is minimal so that the team can go through more bugs!

A brief description
using zsh and the bureau theme the white hexagon unicode character (⬡)
Expected results
the icon is used to prefix the node version on the right side of the prompt, it should show up

Actual results (with terminal output if applicable)
shows up as  [] or a square with a question mark

Your Windows build number
Windows 10 Enterprise x64 1607 build 14393.0
Steps / All commands required to reproduce the error from a brand new installation
Strace of the failing command
Required packages and commands to install

See our contributing instructions for assistance.

This is a problem with your font.  Install a font with the required glyphs (must be .ttf not .otf).

im using monaco, same font on os x and it works fine.

@mikedevita OSX's terminal falls back more gracefully when glyphs are missing.

looks like picking a different font fixes it.

@mikedevita By the way, if you are using oh-my-zsh, you need to delete compfix.zsh and completion.zsh or it will deadlock atm.  It's a known bug #1085 and fixinbound.

@fpqc it works fine for me on windows, sometimes i do have to press ctrl+c a few times.. ill check that bug out.

@mikedevita It will happen inside of tmux mainly.

okay thanks, i dont use tmux.. if anything i use screen..

I think it would happen also in screen, if screen uses ptys.

@mikedevita which font you ended up with?
Also, can't see Monaco font as a choice in the console properties.

I'm having a similar problem. No matter what font I use, in or out of tmux, unicode gylphs can't appear.


DM - 5.2 - edit profiles

Associated epic : #9
as an Admin, I want to be able to edit profiles so that I can update information for a user
Acceptance Criteria:

get a user from database
update information for a user
new information is stored and saved in database

Tasks:

Create a method to access users from database
Create methods to change user info
Save the new info in the database
Test methods with dummy users

Tasks

create method to edit profiles and push back into database
tests are created to make sure method works as intended
code pushed to github


Algolia placeholder does not work docusaurus.config.js file.

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
According to https://docusaurus.io/docs/search#customizing-the-algolia-search-behavior
We can use
placeholder
type: string | default: "Search docs" | optional
The placeholder of the input of the DocSearch pop-up modal.
... this does not work and we get a TS error
theme-search-algolia.d.ts(12, 5): The expected type comes from property 'algolia' which is declared here on type 'ThemeConfig'
Reproducible demo
No response
Steps to reproduce
Add placeholder to Algolia field in Docusaurus config
Expected behavior
Search box modal would use my value instead of the default "Search docs"
Actual behavior
TS error
theme-search-algolia.d.ts(12, 5): The expected type comes from property 'algolia' which is declared here on type 'ThemeConfig'
Your environment

Public source code:
Public site URL:
Docusaurus version used:
Environment name and version (e.g. Chrome 89, Node.js 16.4):
Operating system and version (e.g. Ubuntu 20.04.2 LTS):

Self-service

[X] I'd be willing to fix this bug myself.

Note that you can already provide a custom label by using our i18n support: we consider those labels are translatable i18n english labels, so translating english labels will permit to customize the labels. You can also swizzle @theme/SearchTranslations.
Will be fixed in https://github.com/facebook/docusaurus/pull/10799 since I agree using our i18n system for customizing such label is a bit awkward and unintuitive if your site is not localized.


How to rerun jobs if the system crashes?

Hi,
Right now we are facing a problem. The job is in progress state. While processing the job, If the system crashes. If the system restarts, it is not picking the old jobs which are in the in-progress state.

Can you give me some more context? What queue type are you using, etc?


Create automation to update airtable data in c3 repo

Create a script that archives Airtable data using pyairtable.
The structure of the GCS bucket:
dgm-archive/
- gridstatus/
- ballotready/
- airtable/
   - {base_name}/
       - schema.json
       - {table_name_1}.json
       - {table_name_2}.json
   - ...
- ...

We'll use GCS object versioning for now and attach some metadata to objects. We'll save the schema generation number as metadata for the table objects so we know which version of the schema file they refer to.
Success criteria: create a github action that archives the data, runs the ETL and publishes it to GCS and BQ
TODO

[x] Create abstract archiver
[x] create Airtable arcivher
[x] create a script that collects all the archives and archives the data!
[ ] create a github action that archives the data, runs the ETL and publishes it to GCS and BQ

Nice to haves

[ ] The airtable API does not return a column if the value is empty for a record. This is a problem if the column of a table is completely null. However, the columns are all listed in the schema.json. We could figure out how to incorporate the schema.json into the extraction process so we can construct a dataframe that includes empty columns.
[ ] Incorporate schema.json into the extraction process so we can infer datatypes. The API allows us to pull all record values in as string and not have to deal with linking records across tables or maintain the datatypes but have to look up table values. Right now we are just pulling everything as a string because it's simpler and we already have a column name to datatype mapping set up.

@bendnorman thanks for adding notes here!
re: success criteria, I'm assuming this is done when:

[ ] we have a script that you can run manually, with some sort of auth env var set
[ ] the script grabs data from all airtable tables and puts it into GCS according to the structure above

What goes into schema.json? Will that be manually populated or does the script need to generate that too?

I expanded the scope of this issue to be the whole airtable automation project. Phase I in the design doc. I also added success criteria.
I'm grabbing the schema.json file from the Airtable API.


Fix license in README.md

According to the LICENSE file the code is licensed under Apache-2.0. However, in the file README.md it says the code was licensed under CC-BY-SA. Please fix to avoid confusion.

Done, thanks for the report!


Install talib and forex.analytics fails (solved)

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

Linux Ubuntu 18.04

Zenbot version (commit ref, or version):
4.1.0

Describe the problem
I have seen many post from people struggling to get Zenbot installed. It took me also some time and I have some experience with Linux.
The installation kept on crashing. I am using a complete untouched, fresh installed Ubuntu 18.04 machine.
Before you run:
npm install
you have to make some changes.
1.
Change the package.json file (use VI or VIM or any other editor)
Search for the line with:
"forex.analytics": "github:mkmarek/forex.analytics#7bc278987700d4204e959af17de61495941d1a14",
change it into:
"forex.analytics": "^0.0.15",
save the file.
2.
The Talib package fails to compile because a missing compiler that is part of Python. run the command:
apt install python-dev
3.
Now run
npm install
I was still missing an other package:
npm install tulind
Additional:
If you already ran the npm install script and it crashed (or did not finish) you also have to remove the forex.analytics directory manualy before running it again:
rm -R node_modules/forex.analytics

To fix it also for windows:
replace it instead with:
"forex.analytics": "github:trivalik/forex.analytics#1d3b9fe24f7b4571269c3656925d56347439ef6c",
@DeviaVir: best would be that you fork that repository
and remove talib from package.json, since it is part of forex.analytics.

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

Linux Ubuntu 18.04

Zenbot version (commit ref, or version):
4.1.0

Describe the problem
I have seen many post from people struggling to get Zenbot installed. It took me also some time and I have some experience with Linux.
The installation kept on crashing. I am using a complete untouched, fresh installed Ubuntu 18.04 machine.
Before you run:
npm install
you have to make some changes.
1.
Change the package.json file (use VI or VIM or any other editor)
Search for the line with:
"forex.analytics": "github:mkmarek/forex.analytics#7bc278987700d4204e959af17de61495941d1a14",
change it into:
"forex.analytics": "^0.0.15",
save the file.
2.
The Talib package fails to compile because a missing compiler that is part of Python. run the command:
apt install python-dev
3.
Now run
npm install
I was still missing an other package:
npm install tulind
Additional:
If you already ran the npm install script and it crashed (or did not finish) you also have to remove the forex.analytics directory manualy before running it again:
rm -R node_modules/forex.analytics

It works for me.
@conradhagemans Thank you so much!

Unable to build also on macOS where this workaround doesn't work.
The issue is probably the same. Just scrypt compliation is failing too.

Building on macOS 10.14.6, with node 10.16.3, indeed, fails on the following error:
[...]
building talib functions...
><EMAIL_ADDRESS>postinstall ./node_modules/forex.analytics
> node-gyp configure && node-gyp build
[...]
  CXX(target) Release/obj.target/analytics/src/functions/getIndicatorData.o
  CXX(target) Release/obj.target/analytics/src/functions/getMarketStatus.o
  CXX(target) Release/obj.target/analytics/src/functions/getTrades.o
  SOLINK_MODULE(target) Release/analytics.node
clang: error: no such file or directory: '../lib/ta-lib/lib/libta_abstract_csr.a'
clang: error: no such file or directory: '../lib/ta-lib/lib/libta_func_csr.a'
clang: error: no such file or directory: '../lib/ta-lib/lib/libta_libc_csr.a'
make: *** [Release/analytics.node] Error 1

A brew install ta-lib, as suggested here, didn't help.

@jorisw check this #1934

I have just pushed 1937
as one possible solution.
Builds well.
@jorisw Check this out if you don't mind about forex_analytics.

Thanks, removing forex and commenting its use out fo train.js is what I ended up doing. Runs on mac now.

Going to look if I can fix forex later.


Release Google.Cloud.DataCatalog.V1 version 1.5.0

Changes in this release:

Commit 4d51dc2: build: change links to make them absolute

Created release for Google.Cloud.DataCatalog.V1-1.5.0


[zh]sync removed file for content/en/docs/reference

Fix #25247  [zh]sync removed file for content/en/docs/reference

/language zh

/assign @tengqm

Please change the "Fix #25247" text in your commit message into "Related #25247".
This PR is not a complete fix 25247. Putting "Fix #25247" there will get the issue auto-closed.

/lgtm

Please change the "Fix #25247" text in your commit message into "Related #25247".
This PR is not a complete fix 25247. Putting "Fix #25247" there will get the issue auto-closed.

ok

/approve


Inspirational Quote - RahulSoni0

Fullfilled all the criteria

@RahulSoni0 please give your own image as Photo

@RahulSoni0 please give your own image as Photo

Done . Please check


fordpass instance crashes when started

Describe the bug
Adapter is installed. Instance with correct username, password and VIN added. Every time the instance is started, it crashes.
These are the messages from the logfile

host.banditpi | 2021-12-04 15:23:02.889 | info | Do not restart adapter system.adapter.fordpass.0 because disabled or deleted
host.banditpi | 2021-12-04 15:23:02.888 | error | instance system.adapter.fordpass.0 terminated with code 3 (NO_ADAPTER_CONFIG_FOUND)
fordpass.0 | 2021-12-04 15:23:02.247 | warn | Terminated (NO_ADAPTER_CONFIG_FOUND): Without reason
fordpass.0 | 2021-12-04 15:23:02.187 | error | adapter disabled
host.banditpi | 2021-12-04 15:22:33.975 | info | "system.adapter.fordpass.0" disabled
host.banditpi | 2021-12-04 15:22:30.182 | info | Restart adapter system.adapter.fordpass.0 because enabled
host.banditpi | 2021-12-04 15:22:30.181 | error | instance system.adapter.fordpass.0 terminated with code 6 (UNCAUGHT_EXCEPTION)
fordpass.0 | 2021-12-04 15:22:29.589 | warn | Read-only state "fordpass.0.Odometer" has been written without ack-flag with value "1527"
fordpass.0 | 2021-12-04 15:22:29.587 | warn | Read-only state "fordpass.0.Lockstatus" has been written without ack-flag with value "LOCKED"
fordpass.0 | 2021-12-04 15:22:29.586 | warn | Read-only state "fordpass.0.VIN" has been written without ack-flag with value "WF0TK3SU5MMA11423"
fordpass.0 | 2021-12-04 15:22:29.583 | warn | Terminated (UNCAUGHT_EXCEPTION): Without reason
fordpass.0 | 2021-12-04 15:22:29.581 | info | terminating
fordpass.0 | 2021-12-04 15:22:29.547 | error | Cannot read property 'fuelLevel' of null
fordpass.0 | 2021-12-04 15:22:29.547 | error | TypeError: Cannot read property 'fuelLevel' of null at Fordpass.onReady (/opt/iobroker/node_modules/iobroker.fordpass/main.js:197:51) at processTicksAndRejections (internal/process/task_queues.js:95:5)
fordpass.0 | 2021-12-04 15:22:29.545 | error | unhandled promise rejection: Cannot read property 'fuelLevel' of null
fordpass.0 | 2021-12-04 15:22:29.544 | error | Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch().
fordpass.0 | 2021-12-04 15:22:28.478 | info | starting. Version 0.0.2 in /opt/iobroker/node_modules/iobroker.fordpass, node: v14.18.0, js-controller: 3.3.18
host.banditpi | 2021-12-04 15:22:26.009 | info | instance system.adapter.fordpass.0 started with pid 16192
host.banditpi | 2021-12-04 15:22:25.805 | info | "system.adapter.fordpass.0" enabled

This Software is no longer maintained.
But someone wrote a new Adapter. You can search for it in the ioBroker Forum.


Add E2E: CustomFee

No Tests exist in Java/JS (doesn't make sense for an E2E test, this isn't a transaction nor a query, nor a mirror query)


feat: jwt_backends - create backend mechanism and add authlib support

Hello @k4black,
I've implemented the changes discussed in issue #40. The idea is to add a JWTBackend concept to fastapi-jwt for better security and customization. This allows using other authentication libraries besides python-jose, addressing the security concerns mentioned in #40.
Here's what's new in this PR:

A jwt_backends folder with PythonJoseJWTBackend and AuthlibJWTBackend.
A dynamic import system in jwt_backends/__init__.py.
A define_default_jwt_backend function, so users can pick their preferred backend. I think most will either stick with the default or set it once.
I've kept python-jose support for now, given its popularity, but we might want to consider a deprecation warning.

I think this approach could also help with:

#7: making it easier to create custom backends.
#26: allowing decode_kwargs customization and the "subject" to "sub" swap. Though, standardizing on "sub" might be simpler, that's a side point.

I hope this PR helps.
Have a great week!
Kind regards,

I'm looking at moving to JWT's for a few FastAPI projects and was considering building my own framework.  Looking through fastapi-jwt, I love the design, however also feel that it's not in our best interest to rely on python-jose. Has there been any consideration of merging this into the project?  If so, is there any additional work needed I could assist with?

@rjjanuary, it seems that the maintainer of this project is not very active.
Regarding this PR, I don't think it will be merged, and tbh I would be in favor of a full deprecation of python-jose entirely.
I'm now using fastapi-users which provide an auth framework in which you can select what you want and what you want to change. The project is actively maintained too. They had a similar issue with passlib being unmaintained, and the maintainer of fastapi-users handled the situation very well. See: https://github.com/fastapi-users/fastapi-users/issues/1325

Thank you @hasB4K,  I appreciate the quick response.  I'll look deeper into fastapi-users, however at first glance it may be a little overkill for current projects.

@hasB4K Thank you for the great contribution!
I'll revise design of backends a bit to use less =None hacks (Keep backends through error if lib not installed and make jwt.py itself select backend) and push to this branch today!

@hasB4K Thank you for the great contribution!
Sorry for not being really active, if you eager to use and update this project i can grant you rights. Please write me email at kdchernyshev at gmail dot com and we can have a short call.
Regarding your contribution:
I'll update backends structure a bit to make jwt.py make a backend choice. For now i'll keep backends feature, but in general I agree with deprecation and removal of jose.


Add confirm text

Added the option to add a custom text to a "Confirm" button. Currently it showed the rather unfriendly "Submit Query" text on my setup (might be OS/browser dependent?). It now shows the text "Confirm" by default, but the user can specify a different text by use of an optional (non-keyword) argument.
I hope you like it, but am happy to make any changes as you see fit.

The current behaviour is to let the user agent choose a text, according to localization settings. We want to maintain this behaviour, at least as the default one. Could you adjust the default to fall back to this behaviour and only override if you explicitly pass some text?

Thanks for the inspiration for #294! Since #294 is now merged, this one should probably close.

Wow I really dropped the ball there, thanks for picking it up!


proxies

the server I'm connecting on requires a proxy connection to the another server and I can only access it via an authenticated SOCKSv5 proxy, I've checked the source code and seems like graphql-request doesn't have proxy support,

By default graphql-request is limited by what the Fetch API can provide. If you need to do some kind work (such as a proxy) on the request it is possible to override this implementation using a custom fetch function.


[5.8] Added passes method declaration

Added passes method declaration to ValidatorContract
There is method called like that in the Illuminate\Validation\Validator, but there are no such declaration in interface
Closing this https://github.com/laravel/ideas/issues/1397
Adding this will make IDE highlighting while referencing the interface.
It will also introduce BC to those who implement this interface so it will go to 5.8

We already have the fails method on the interface which is the opposite of passes. IMO we don't need both.

Yeah, it feels weird to have a convenience method in an interface as a required method definition.


fix: hash approval errors + return txHash

What it solves
approveHash error notifications, as well as returning txHash.
How this PR fixes it
Relative to the legacy processTransaction error handling, contract errors occuring from approveHash are now displayed in a notification.
A few type changes were made here in order to simplify the error handling as well.
The txHash is also returned here as owner addition/editing checks that the txHash exists in order to update the address book.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 17 (47.06%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 34.641%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/safe/components/Settings/ManageOwners/AddOwnerModal/index.tsx
0
1
0.0%

src/routes/safe/components/Settings/ManageOwners/ReplaceOwnerModal/index.tsx
0
1
0.0%

src/logic/safe/store/actions/createTransaction.ts
7
14
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
3326

Relevant Lines:
8622

💛  - Coveralls


KAFKA- 12347: expose offsets to streams client (WIP)

collect the offsets after they are committed
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Is moved to https://github.com/apache/kafka/pull/10211


Crash when searching for artist "MASTER BOOT RECORD"

             (node:29790) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'replace' of null
    at TidalApi.TidalAPI.getArtURL (/usr/lib/node_modules/tidal-cli-client/node_modules/tidalapi/lib/client.js:342:53)
    at Promise (/usr/lib/node_modules/tidal-cli-client/app/Artist.js:20:36)
    at new Promise (<anonymous>)
    at Artist.updateArt (/usr/lib/node_modules/tidal-cli-client/app/Artist.js:18:16)
    at Object.communicationEvents.subscribe [as handler] (/usr/lib/node_modules/tidal-cli-client/app/UI/MainScreen.js:100:34)
    at Observable.fire (/usr/lib/node_modules/tidal-cli-client/app/Observable.js:45:37)
    at module.exports.on (/usr/lib/node_modules/tidal-cli-client/app/UI/ArtistsList.js:13:38)
    at module.exports.EventEmitter._emit (/usr/lib/node_modules/tidal-cli-client/node_modules/blessed/lib/events.js:98:20)
    at module.exports.EventEmitter.emit (/usr/lib/node_modules/tidal-cli-client/node_modules/blessed/lib/events.js:117:12)
    at module.exports.List.enterSelected (/usr/lib/node_modules/tidal-cli-client/node_modules/blessed/lib/widgets/list.js:586:8)
(node:29790) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)                       │
(node:29790) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

That's what it spews out after I hit return.
To replicate:

Run tidal-cli

Input :

Input artist

Input full or partial query "MASTER BOOT RECORD"

Press RETURN

That`s pretty much it.

Thank's for your feedback, I'll try to repair it as soon as possible.

It was an issue with that app couldn't find artist image, because MASTER BOOT RECORD doesn't have one in Tidal. Now it displays the default image from Tidal. Closing


fix(test): corrected commit from May 23 that added invaid parameter d…

…efinition

There were the following issues with your Pull Request

Commit: 40506605e57eb223cf045753133b104de44298df
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com

@ldimaggi snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1399-fabric8-ui.int.rdu2c.fabric8.io