refactor(execution): make the block state of TransactionExecutor an O…

…ption to allow moving it to chunks

This change is

Codecov Report
Attention: Patch coverage is 60.00000% with 16 lines in your changes missing coverage. Please review.

Project coverage is 77.99%. Comparing base (79b6ae8) to head (b8da0b4).

Files
Patch %
Lines

.../blockifier/src/blockifier/transaction_executor.rs
54.54%
13 Missing and 2 partials :warning:

...es/blockifier/src/blockifier/stateful_validator.rs
85.71%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1947      +/-   ##
==========================================
- Coverage   78.03%   77.99%   -0.05%
==========================================
  Files          61       61
  Lines        8760     8788      +28
  Branches     8760     8788      +28
==========================================
+ Hits         6836     6854      +18
- Misses       1486     1496      +10
  Partials      438      438

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

crates/blockifier/src/blockifier/transaction_executor.rs line 50 at r1 (raw file):

    // The transaction executor operates at the block level. It moves the block's state when
    // operating at the chunk level and gets it back after the chunk is committed, so it needs to
    // be wrapped with an Option<_>.

Suggestion:
    // The transaction executor operates at the block level. In concurrency mode, it moves the block state to the worker executor
    // - operating at the chunk level - and gets it back after committing the chunk. The block state is
    // wrapped with an Option<_> to allow setting it to `None` while it is moved to the worker executor.


An exception case during run Base32 gtest

Thanks to @kuro11pow2 , I find an bug during Base32 gtest!
TestParamType(std::vector<u_int8_t>{ 0x10, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0 }, "100000000")

It is a special case because it goes 1<PHONE_NUMBER>0000000000<PHONE_NUMBER>0000000000. And when<PHONE_NUMBER>0000000000 enters the bitsToBase32 function, nothing happens because of the condition of while loop while(value != 0).
The best way I think is to add exception code to catch<PHONE_NUMBER>0000000000input case.
if(!value)result = "0000";

Because it is an unusual case.
I'm really appreciate to 'kuro11pow2' for making a sharp test case.
Please give me some solution or I'll make PR with my solution.

It's my pleasure :smile:

Thanks for your notice! I'll update Base64 too. 😀


out of date reference

there is reference to an Ignite presentation from 2018 that has a broken link

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

ID: 3f47362e-5ac2-badf-9a90-f9f7831e5767
Version Independent ID: 2c5b4b0a-29ca-6d6c-121c-5d7a81d6ca7e
Content: Microsoft 365 NIST 800-53 action plan — Top priorities for your first 30 days, 90 days, and beyond
Content Source: microsoft-365/compliance/NIST-action-plan.md
Product: microsoft-365-enterprise
GitHub Login: @BrendaCarter
Microsoft Alias: bcarter

@DeanGross  - Thank you for submitting feedback.
I will get this issue over to the Microsoft 365 writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@DeanGross  Thank you for the feedback. I have removed outdated information from the doc via PR https://github.com/MicrosoftDocs/microsoft-365-docs/pull/682

@DeanGross - From our understanding, the issue has been resolved. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!


feat(renderer): definable default color for symbolic icons

Adds a new field to the renderer style to support defining a default icon color for symbolic icons. By default, this color will be set to the same value as the text color for maximum visibility. The color may be overridden by the button or container that it is placed in. Icons marked as symbolic will opt into this color if no color was defined.
Merge with https://github.com/pop-os/libcosmic/pull/150

Added fix for https://github.com/pop-os/libcosmic/issues/154: 1f8fe038


Error in running cosimulation

Hi,
I'm using Vivado 2018.2.
I have tried to run a cosimulation using the testbench provided by you.
At the start of the log file, there are the following two warnings:
INFO: [COSIM-47] Using XSIM for RTL simulation. INFO: [COSIM-14] Instrumenting C test bench ... WARNING: [COSIM-369] AXI_master port 'in.V' has a depth of '256'. Insufficient depth may result in simulation mismatch or freeze. WARNING: [COSIM-369] AXI_master port 'out.V' has a depth of '256'. Insufficient depth may result in simulation mismatch or freeze.
The simulation stops almost at the end with an error that I think to refer the depth of axi master interfaces.
How can i fix this?
I attach the log file, and I'll wait for your suggestion.
LOG FILE:
BlackBoxJam.log
Thanks,
Sara
PS: sometimes, relaunching the same cosimulation, it stops with the following error:
ERROR: System recieved a signal named SIGSEGV and the program has to stop immediately! This signal was generated when a program tries to read or write outside the memory that is allocated for it, or to write memory that can only be read. Possible cause of this problem may be: 1) the depth setting of pointer type argument is much larger than it needed; 2)insufficient depth of array argument; 3)null pointer etc. Current execution stopped during CodeState = ENTER_WRAPC_PC.

Hi,
can anyone help me?
Thanks.
Sara

Ok, thanks.
But why just 512? WhatsApp is depth related in this case?
Thanks for your response.
Sara


preset near-operation-file ignores typechecking

Describe the bug
When working with the preset near-operation-file there is no typechecking done for the validity of the queries.
Currently it will report errors if the query / mutation is not found, or if the fields selected are not found. It does not however typecheck for arguments or fragments and will happily generate wrong code with invalid fragments or arguments.
Your Example Website or App
https://github.com/klyve/graphql-codegen-preset-test
Steps to Reproduce the Bug or Issue
Given the following codegen yaml
generates:
  generated:
    documents: "*.graphql"
    preset: near-operation-file
    presetConfig:
      extension: .generated.ts
      baseTypesPath: ./types.ts
    plugins:
      - typescript-operations
      - typescript-react-apollo
    config:
      withHooks: true
    schema: ./schema.graphql

The schema
schema {
  mutation: Mutation
  query: Query
}

type User {
  id: ID!
  name: String!
}

input CreateUserInput {
  name: String!
}

type Query {
  user(id: ID!): User!
}

type Mutation {
  createUser(input: CreateUserInput!): User!
}

And the near-operation file:
mutation CreateUser($input: CreateUserInput!) {
  createUser(unknown: $unknown) {
    ...UNKNOWN_FRAGMENT
  }
}

Running this through codegen with the preset near-operation-file
results in no errors but the expected result is:

unknown is not an argument
UNKNOWN_FRAGMENT does not exist in the schema

As observed it skips the typechecking and generates code for the invalid file
Expected behavior
Expected behavior is for codegen to do typechecking for the near operation files and report errors if any of the queries have the wrong arguments or fields.
Screenshots or Videos
No response
Platform

OS: macOS
NodeJS: 16.16.0
@graphql-codegen/cli: 2.11.5
@graphql-codegen/introspection: 2.2.1
@graphql-codegen/near-operation-file-preset: 2.4.1
@graphql-codegen/typescript: 2.7.3
@graphql-codegen/typescript-operations: 2.5.3
@graphql-codegen/typescript-react-apollo: 3.3.3
graphql: 16.5.0

Codegen Config File
overwrite: true
generates:
tests/generated:
documents: "tests/*.graphql"
preset: near-operation-file
presetConfig:
extension: .schema.ts
baseTypesPath: ./graphqlTypes.ts
plugins:
- typescript-operations
- typescript-react-apollo
config:
withHooks: true
schema: ./schema.graphql
Additional context
No response

if someone wants to revive this one https://github.com/dotansimha/graphql-code-generator-community/pull/414


[full-ci] update reva to latest edge

Description
Update reva to get the latest changes from edge.
Related Issue

Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

@micbar I rebased to get a full-ci run again because #4532 was merged, and it would be nice to see that everything is fine for this PR on top of that one.


Fix null terminator in Demangle function.

If the demangled name is longer than out_size, the null terminator is missing from the output. This will cause a crash in the DemangleInplace() function (symbolize.cc) when calling strlen on the buffer.
Reproduction Steps:
Use a function with a demangled name longer than 256 characters.
Attempt to log a fatal error with a stack trace.
This will cause a crash.
Example input for DemangleInplace that makes it crash (on Ubuntu):
"ZN4kudu9FsManager9AddTenantERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEES8_St8optionalIS6_ESA_SA"

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 67.24%. Comparing base (2075ae8) to head (bdd0d1b).

Files with missing lines
Patch %
Lines

src/demangle.cc
0.00%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1137      +/-   ##
==========================================
+ Coverage   64.03%   67.24%   +3.21%
==========================================
  Files          20       14       -6
  Lines        2580     1914     -666
  Branches      888      688     -200
==========================================
- Hits         1652     1287     -365
+ Misses        663      448     -215
+ Partials      265      179      -86

Files with missing lines
Coverage Δ

src/demangle.cc
81.81% <0.00%> (+31.81%)
:arrow_up:

... and 15 files with indirect coverage changes

I added a second commit extending the SymbolizeWithDemangling (just demangle a longer function). It crashes without my fix.

I had another look at the existing unit tests and now I'm not convinced that trimming the output is a good idea. This will possibly result in some sort of ambiguity or even will confuse users.
The easiest way of fixing the issue is to simply return false whenever the buffer size is too small to be consistent with the behavior of the existing demangler. In the long term, however, we should switch to dynamic memory (re)allocation (e.g., by using std::string as the return type) to avoid insufficient buffer size situations.

@martonka I pushed some changes to your branch. Make sure to pull these before extending the PR.


parse_m3u这个方法在线解析参数写错了

这块应该是path

@biancangming path can be url or file path. path is checked if it is url or not else it will be file path. urpparse(path).scheme !="" or re.search(self.__url_regex, path) checks if it is a url else it will be a file path. Did you have any problem ??

你的源码将path写成url，导致此处报错，这是我修改后的结果 ^0^

@biancangming Thank you for pointing it out. I didn't get what you meant before and now another issue was raised and I understand what you were trying to say before... Thanks a lot.


Improve the usage of the addon by non-hassio consumers (docker and kubernetes)

Hi @thor0215, I'm planning to send you a PR to make the code a little easier to be used by external docker or kubernetes. This would translate into:

bypass the run.sh and instead jump right into the xfinity_usage_addon.py with the right environment variables. Alternative would be a new environment variable named MANUAL or BYPASS or something else that allows the run.sh to just bypass all HA/bashio code.
define an extra docker volume for the browser persistent shared storage (not sure if your plan is to support this or not so let me know)
allow oneshot/no pooling. Kubernetes already has a strong CronJob subsystem so it's better to just have it reschedule a new run if needed (backoffPolicy) but also when it runs. We could for instance say, run every 30m but only from hour x to y. Kubernetes also allows the definition of a total deadline.
potentially run the image without root.
precreate the /config/xfinity.log log file if it doesn't exist.

Example of a kubernetes cronjob:
apiVersion: batch/v1
kind: CronJob
metadata:
  name: xfinity-usage
  namespace: tools
spec:
  schedule: "0/30 * * * *"
  concurrencyPolicy: Forbid
  failedJobsHistoryLimit: 3
  successfulJobsHistoryLimit: 1
  startingDeadlineSeconds: 360 # 6 min
  jobTemplate:
    spec:
      backoffLimit: 0 # Don't retry.
      template:
        spec:
          initContainers:
            - name: create-log
              image: busybox
              command: ["touch", "/config/xfinity.log"] # Create the log file.
              volumeMounts:
                - mountPath: /config
                  name: config
          containers:
            - name: xfinity-usage
              image: ghcr.io/thor0215/hassio-xfinity-usage-aarch64:<IP_ADDRESS>.2.2@sha256:897360c2f9e8e85d040f6da18ae11c1b524f80dec5c40b703e717df69272bba9
              imagePullPolicy: IfNotPresent
              command: ["/bin/bash"]
              args: ["-c", "python3 -Wignore xfinity_usage_addon.py || true"] # Always return true while oneshot mode is not supported.
              resources:
                limits:
                  memory: 4Gi
                  cpu: 1000m
                requests:
                  memory: 2Gi
                  cpu: 200m
              env:
                - name: TZ
                  value: America/Los_Angeles
                - name: POLLING_RATE
                  value: "3600"
                - name: PAGE_TIMEOUT
                  value: "60"
                - name: LOGLEVEL
                  value: "debug_support" # Abuse debug_support to support oneshot mode.
                - name: MQTT_SERVICE
                  value: "true"
                - name: MQTT_USERNAME
                  valueFrom:
                    secretKeyRef:
                      key: MQTT_USERNAME
                      name: xfinity-usage-secret
                - name: MQTT_PASSWORD
                  valueFrom:
                    secretKeyRef:
                      key: MQTT_PASSWORD
                      name: xfinity-usage-secret
                - name: MQTT_HOST
                  value: "mqtt.example.com"
                - name: MQTT_PORT
                  value: "443" # Or 1883 for unencrypted mqtt.
                - name: XFINITY_USERNAME
                  valueFrom:
                    secretKeyRef:
                      key: XFINITY_USERNAME
                      name: xfinity-usage-secret
                - name: XFINITY_PASSWORD
                  valueFrom:
                    secretKeyRef:
                      key: XFINITY_PASSWORD
                      name: xfinity-usage-secret
              volumeMounts:
                - mountPath: /config
                  name: config
          volumes:
            - emptyDir:
                medium: Memory
                sizeLimit: 1Gi
              name: config
          restartPolicy: Never
          securityContext:
            runAsNonRoot: false

I am making some changes to that the HA bashio part of the run script runs if it detects bashio it will execute the bashio code.  I'm not sure if I fully understand the non-root request. Would you need the dockerfile to be something like this:
Do all the system stuff as root then add the non-root user and run the script as non-root?
apt-get blah blah
setup non-root user
USER non-root
RUN shell script
Take a look at my run.sh script I'm upload to the branch for this issue and let me know your thoughts on everything.

Let me know if you want to keep this open or if the last few releases address most of your issues.

@thor0215 Was there an env var created to address the first point?

Let me know if you want to keep this open or if the last few releases address most of your issues.

Sorry, I've been swamped. I'll test this during the weekend.

@thor0215 Was there an env var created to address the first point?

The Dockerfile still has CMD ["/run.sh" ]. I don't know how to make the Dockerfile only do that for HA Addon setups. If someone has some way to do that, let me know.

The easiest way it to put that logic inside the /run.sh then just do if hassio do X, if not do Y. There might be some sort of env variable that hint to being inside hassio before calling hassio shell scripts.

So right now run.sh should only do the HA stuff if it sees the /config/options.json file, but always run the python script.  Does that solve request number 1?

I think so, you can also complement with the env. There's should be something there to say I'm HA. Thanks for looking into this!


GroupManagerDao.mergeGroup MysqlDataTruncation

2015-01-31 22:34:48 [INFO] rourke750 issued server command: /nlmg rourke rourke750
2015-01-31 22:34:48 [WARN] com.mysql.jdbc.MysqlDataTruncation: Data truncation: Incorrect integer value: 'rourke' for column 'group_id' at row 1
2015-01-31 22:34:48 [WARN]      at com.mysql.jdbc.MysqlIO.checkErrorPacket(MysqlIO.java:3591)
2015-01-31 22:34:48 [WARN]      at com.mysql.jdbc.MysqlIO.checkErrorPacket(MysqlIO.java:3525)
2015-01-31 22:34:48 [WARN]      at com.mysql.jdbc.MysqlIO.sendCommand(MysqlIO.java:1986)
2015-01-31 22:34:48 [WARN]      at com.mysql.jdbc.MysqlIO.sqlQueryDirect(MysqlIO.java:2140)
2015-01-31 22:34:48 [WARN]      at com.mysql.jdbc.ConnectionImpl.execSQL(ConnectionImpl.java:2626)
2015-01-31 22:34:48 [WARN]      at com.mysql.jdbc.PreparedStatement.executeInternal(PreparedStatement.java:2111)
2015-01-31 22:34:48 [WARN]      at com.mysql.jdbc.PreparedStatement.execute(PreparedStatement.java:1362)
2015-01-31 22:34:48 [WARN]      at vg.civcraft.mc.namelayer.database.GroupManagerDao.mergeGroup(GroupManagerDao.java:616)
2015-01-31 22:34:48 [WARN]      at vg.civcraft.mc.namelayer.GroupManager.mergeGroup(GroupManager.java:91)
2015-01-31 22:34:48 [WARN]      at vg.civcraft.mc.namelayer.command.commands.MergeGroups$1.run(MergeGroups.java:83)
2015-01-31 22:34:48 [WARN]      at org.bukkit.craftbukkit.v1_7_R4.scheduler.CraftTask.run(CraftTask.java:71)
2015-01-31 22:34:48 [WARN]      at org.bukkit.craftbukkit.v1_7_R4.scheduler.CraftAsyncTask.run(CraftAsyncTask.java:53)
2015-01-31 22:34:48 [WARN]      at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
2015-01-31 22:34:48 [WARN]      at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
2015-01-31 22:34:48 [WARN]      at java.lang.Thread.run(Thread.java:745)

can you run select * from faction_id where group_name = 'rourke'

I see someone did this in the terminal did you get it rourke?
On Sun Feb 01 2015 at 10:01:46 AM rourke750<EMAIL_ADDRESS>wrote:

can you run select * from faction_id where group_name = 'rourke'
—
Reply to this email directly or view it on GitHub
https://github.com/Civcraft/NameLayer/issues/33#issuecomment-72370415.

yea I know whats' wrong.
On Sun, Feb 1, 2015 at 11:22 AM, ttk2<EMAIL_ADDRESS>wrote:

I see someone did this in the terminal did you get it rourke?
On Sun Feb 01 2015 at 10:01:46 AM rourke750<EMAIL_ADDRESS>wrote:

can you run select * from faction_id where group_name = 'rourke'
—
Reply to this email directly or view it on GitHub
https://github.com/Civcraft/NameLayer/issues/33#issuecomment-72370415.

—
Reply to this email directly or view it on GitHub
https://github.com/Civcraft/NameLayer/issues/33#issuecomment-72371259.

So, this issue is closed, TTK linked it to me, but ... I still can't access the blocks that used to be reinforced to the group that this botched merge deleted. So, I've got a bunch of factories and other infrastructure that is completely unusable. Any suggestions on restoration path?

Yes, good.  Message me on reddit /u/rourke750 with cords to factories, what group they should be on.  I'll try sort it out tonight.


Update OWNERS for contributor site to grow reviewers/approvers

Per this Slack discussion, I've added further reviewers/approvers to the contributor site based on prior staffing and current involvement in the Kubernetes Contributor Summit.
We hope that this will mean less of a bottleneck on website updates as we gear up to organize the EU 2024 edition of the Contributor Summit in Paris.

@dipesh-rawat thank you so much Dipesh! it seems i am so used to using the em dash that i didn't realise i had done so when adding in the extra reviewers – i very much appreciate you catching this!
@mrbobbytables @sftim this should be good to merge now – we want to make sure we have enough reviewers for the contributor site so that changes and approvals don't have to rely on a small group, thanks 🙏


Don't fail when no secrets are present

Currently, when loading secrets, if no secrets are found an exception is thrown:
if (secretsList.length === 0){
      throw new Error(`No matching secrets were returned for prefix "${prefix}".`);
}

There are valid use cases where secrets may not (yet/ever) be present, such as using this action in a reusable composite GitHub Action, where any given repository may access AWS accounts that may or may not have secrets in Secrets Manager (if they do, they will be exposed).
Proposing adding a flag such as fail-on-empty-secrets to facilitate this.

Thanks for the report, @cloudshiftchris. We have added this as an enhancement request.


board/sim: Enable hellocxx in loadable config

Summary
to test the support of C++ ELF binary
Impact
sim:loadable
Testing
Pass CI

ping @pkarashchenko


Support motion compensation

iRacing is adding OpenXR support - OpenXR motion compensation would be very helpful

Hi, I can sponsor the development of this feature, please reach me out<EMAIL_ADDRESS>

Having just finished building an SFX100 style motion rig and running Sim Racing Studio  and OpenXR-Toolkit with Opencomposite this would be the dream-team to add Motion Compensation to the toolkit !!
Racing sims are bearable but in FS2020 you eventually fetch up with your head outside the cockpit on roll / pitch
Fingers crossed -
PS I'd love to beta test developments with this

As a very enthusiastic VR sim racer with a motion platform this would be a very useful solution. Like Octo1234 I have my fingers crossed, happy to beta test with a professional background in testing software.
Thanks
Fred

Thank you all, but right now the issue is not lack of hardware or lack of beta testers, the issue is lack of our developers' time.

I organized the fork of https://github.com/openvrmc/OpenVR-MotionCompensation and I would support an openxr motion compensation project as well, by individual sponsorship, github effort, or even trying to help round up sponsorship.
What I can say is I think it should be an isolated project.  I do not think it should be combined with a larger feature set.  I say this because the larger the feature set, the more trouble for maintainers and the less likely we will be able to sustain for the long term.
Unfortunately, while knowledgeable in the use case and software development, I have no knowledge of the required toolchain and I am 20 years away from my C++ days.
Otherwise, please feel free to contact me if I can help organizing etc.

The OpenXR-Toolkit might be what what SteamVR is for OVRMC, the popular, underlying platform to support it. Though practically it should just plug into SimRacingStudio from and end user perspective (and potentially open for other motion platforms software to use it)

@karloy007 - I am just looking into this and unaware of the toolchain or dependencies.  With that said, I am very aware of maintaining many and very popular open source packages - my only caveat is to say that the project focusing on MC should be as simple as possible with no frills.  That is part of the reason why the original openvr-mc functionality went unmaintained, until I forked and started cutting, then thankfully @Dschadu stepped in and took over as I was out of my element.
My commercial thought is that we need a platform agnostic, hardware-based solution for virtual tracker.  Given all the motion platforms in the market, subsetted by those that want VR, I think a yearly subscription has enough justification to pay for dev.  SimRacingStudio has shown an appetite for this and I think it's a welcome place to do so.

@mbucchia "You need someone who understands or has time to learn the OVRMC ecosystem so they can integrate the solution to it."
by echo system do you mean the shared memory files that provides the inputs as the actual position of the motion platform?

I have no idea of the specifics. I'm told OVRMC supports several scenarios, some driven by motion controller or off the shelf IMUs, and some driven by the platform itself sending telemetry or motion data (that shared memory you are talking about).
The main issue here again is I don't have time to go and read about what OVRMC does. And even if I did, there seems to be too many different configurations to support.

Here is the OpenXR API layer sample I give to everyone:
https://github.com/mbucchia/XR_APILAYER_NOVENDOR_fov_modifier
This code shows you how to override the behavior of xrLocateViews(), which is typically what games use to locate the camera pose to render with.
Right now this code shows how to override the FOV values, but overriding the pose itself is done exactly the same way.
For the case of FlyPT etc (the use case of OVRMC that feeds from shared memory to do the pose compensation), that sample code gives you 90% of the code needed to hook up to OVRMC. The missing 10% is replicating this code to xrLocateSpace() when that function is used instead of xrLocateViews() (this is what OpenComposite does). You also need to hook into xrCreateReferenceSpace() in order to keep track of what XrSpace handles refer to the VIEW space. This is what I told XRNeckSafer to do (and you can go see their code as well).
This sample along with XRNeckSafer gives you 100% of the OpenXR specific code that needs to replace the OVRMC OpenVR hooks.

For the motion controller handling (so not the shared memory, but using the controller as IMU), the thing to do is a little more complicated.
You need to read the motion controller pose, which means you need to create an XrAction and then hook into xrSuggestInteractionProfileBindings() to add a mapping for this action to the desired tracker pose (the action path, so something like /user/hand/left/aim/pose). For Vive trackers you also need to force requesting the XR_HTCX_vive_tracker_interaction instance extension so you can use the Vive tracker action paths.
To create an XrAction you will also need an XrActionSet that you create yourself and you will need to hook into xrAttachSessionActionSets() in order to register it along with the apps other action sets.
Then you will need to query the controller pose like you would normally do with OpenXR (so create an action space and then use xrLocateSpace()).
All of this is demonstrated in existing OpenXR Toolkit code for the eye tracker. Forcing requesting the extension, registering the actionset, adding the suggested binding and querying the pose, all of it can be 100% copy/paste from OpenXR Toolkit.
All that's needed then is hook up to OVRMC.

Find a C/C++ developer who understands how OVRMC works, and they can use the content of the 2 messages above to add support for the shared memory and the motion controller use cases. Likely doable in 2-3 days of work. I'm available to answer questions too.

as reference, here is the latest OVRMC source code:  https://github.com/openvrmc/OpenVR-MotionCompensation/releases/tag/0.3.6. Core logic seems to be in vrmotioncompensation.cpp.
GNU GENERAL PUBLIC LICENSE - https://ovrmc.dschadu.de/en/license

Commenting to ask for compatibility with FlyPT Mover motion software using the named pipe/shared memory file 'motionRigPose' either in OXRTK and/or through OVRMC.  Or directly in the PimaxXR runtime!

At this point in time, I do not plan to include motion compensation in OpenXR Toolkit.
Instead, I am working with another developer to create their own OpenXR API layer, so it can be immune to incompatibilities due to shortcomings of our graphics code (Eg: what we are seeing today with iRacing).
Closing for now.

The spin-off project in question is becoming public now:
https://discord.gg/7Q3vZe58VH


Added Always First Functionality

This pull request resolves #164.

This enables the new file type F__*.sql which is processed before all other file types.
This option defaults to False but can be enabled with the configuration -af or --always-first. This ensures that these files can be selectively executed depending on the environment.

e.g. In the CI/CD pipeline, you enable this configuration option when you need to clone the production environment to create the QA environment. However, this option is turned off for the elevation from QA to Prod because no cloning operation is taking place.

Tests were added to ensure the functionality works as designed. These tests cove:

Returning the proper number of files executed when the option is turned on or off.
Returning expected errors based on naming conventions.
Validating the appropriate data for the Change History Table.

Added this new functionality to the README documentation.

Is there an update on when this will be reviewed?

@CaptCorpMURICA Will discuss this next week and get back to you. Thank you for your contribution and your patience while we catchup on open items and PRs.

Any discussion on when this might make it into a release?  Thank you

I just welcomed my first child, so I haven't had the ability to implement the requested changes. I'll try to get it wrapped up in the next month or so.

Any discussion on when this might make it into a release?  Thank you

I just welcomed my first child, so I haven't had the ability to implement the requested changes. I'll try to get it wrapped up in the next month or so.

Congratulations !!

Any update on this one?

It'd be great to add into the README file along with the other prefix file name section

Can we bump this? I would need this functionality in production already.


Allow specification of file system ids

At present all file systems in the region are monitored.
Allow users to specify the file systems by ID - if no identifier is given, then all should be checked.

Already implemented - closing this issue.


Newbie needing help with TFT colours and fonts

Hi all,
Firstly a big thanks to Ed for sharing all his hard work on this Internet radio. Amazing stuff.
I have got mine up and running and am using an ST7735 1.8" display, and am very happy to have got it working.
As there is plenty of room on the display, I would really like to make the font bigger and maybe add some background colours, but can't work out how to do this (not new to Arduino code but certainly no expert...). I've seen the;
"#define dsp_setTextSize(a)      tft->setTextSize(a)                // Set the text size"
line in the bluetft.h tab, but am not sure what to do with it.   I've read about setTextSize(1) and setTextSize(2), but no idea how the (a) in this example relates to that.
And as for background colours.... no idea where to start.
Any help much appreciated.
Thanks,
Ralph

dsp_setText is just a macro.  The goal is to make a function "dsp_setText()" that will work on all included display drivers.
So if you want to change the size, look for calls of "dsp_setText" in the main program.
For colours, look for "BLACK" or "RED" or "YELLOW" in the main program.

Hi Ed,
Many thanks for the reply, much appreciated.
My main interest was to make the text in the middle block that displays the station being played a bit bigger - helps with my ageing eysight.....
Thanks for the hints - I am gradually developing a better understanding of the sketch structure, so will work away on that.
BR,
Ralph

I have successfully replaced the original AdadruitGFX & ILI9163C library with TFT_eSPI library with minimal modification and eventually redesigned the GUI.
Using TFT_eSPI will allow you to use many types of LCD (e.g. I am currently using a 320x240 ILI9341 with touch screen control)

@Derek-K would you be able to share your code with TFT_eSPI? thanks

@h1aji As far as I remember, just load the TFT_eSPI library, and if there are any changes, it will be very minor.

@Derek-K Figured out, easy. Thanks


restrict the operator when installed via OLM to only see secrets to support default use-case

cc @leandroberetta
Proposal to restrict the operator secret permissions. This would go with https://github.com/kiali/kiali-operator/pull/378
The problem here is if any service mesh tenant has other named secrets that Kiali needs to see, the user has to manually update this role. This is actually not a horrible thing in my mind (it forces the user to think before they grant these privileges to the operator service account). But it will be an annoyance for users that need to do this.
I am trying to figure out the best way to tell a user how to update the operator's cluster role to add secrets to this list, but it is very difficult (if not impossible) to do it in a single kubectl command that patches the rule list item. It may be best just to tell the user how to add a new rule to the role permissions. I'm still looking at this to find the best way to do it. This is something we'll need to document in the FAQ if we do this.

This seems to be the best way to do it. If you want to add "get" access to new secrets, you add them to the list of rules in the existing operator clusterrole like this:
kubectl patch clusterrole kiali-operator --type "json" -p '[{"op":"add","path":"/rules/0","value":{"apiGroups":[""],"resources":["secrets"],"verbs":["get"],"resourceNames":["NEW_SECRET_NAME"]}}]'

Notice the above uses op add using the list index 0 - so this will add the new rule as the first item in the rules list.
Now if you want to modify that new list of secrets, you can use op replace with the same list index 0 - so this will modify the existing list you created via the above command. Use this if you forgot to add a secret name or you need to change the spelling of the new secret name.
kubectl patch clusterrole kiali-operator --type "json" -p '[{"op":"replace","path":"/rules/0","value":{"apiGroups":[""],"resources":["secrets"],"verbs":["get"],"resourceNames":["NEW_SECRET_NAME","ANOTHER_SECRET_NAME"]}}]'

It's possible the name of the clusterrole (as created by OLM) is not "kiali-operator" - I'll have to confirm that. If the name is different, we'll probably need to do something like:
kubectl patch clusterrole $(kubectl get clusterroles -l app.kubernetes.io/name=kiali-operator -o jsonpath='{.items[0].metadata.name}') ...and the rest...

Well, can't use the -l selector because when OLM installs it, the role doesn't have those labels. And the name of the role is not kiali-operator so we will need to do something to get the name. But it is just as easy to do this via:
add:
kubectl patch $(kubectl get clusterroles -o name | grep kiali-operator) --type "json" -p '[{"op":"add","path":"/rules/0","value":{"apiGroups":[""],"resources":["secrets"],"verbs":["get"],"resourceNames":["NEW_SECRET_NAME"]}}]'

replace:
kubectl patch $(kubectl get clusterroles -o name | grep kiali-operator) --type "json" -p '[{"op":"replace","path":"/rules/0","value":{"apiGroups":[""],"resources":["secrets"],"verbs":["get"],"resourceNames":["NEW_SECRET_NAME","ANOTHER_SECRET_NAME"]}}]'


Validate CSV - validate particular value under a given column

Validate CSV file - Need to validate particular value under a given column with respective given column value

Closing this request


Add wikidata asynchronously based on wikipedia

As a first step towards #2680, when the user enters a Wikipedia article title using the Wikipedia preset field, pair the resulting wikipedia tag with a wikidata tag. The query is asynchronous using JSONP, because neither wikidata.org nor wqt.wmflabs.org supports XmlHttpRequests from any hosted iD instance.
Also tracked in Phabricator as part of the Wikimanía 2015 Hackathon.
/cc @pigsonthewing @bhousel

Cool, looks like a good start... I'm mostly concerned about event.change(asyncTags); what will happen if that callback fires after the user has left the field and maybe edited other things.

Indeed, it is possible to cause wikidata to be added to the wrong item if the Wikidata API request takes long enough that you’ve already selected another feature. Mutating context.entity(entity.id).tags works, but a change event still needs to be posted to update the inspector if the original object is still selected.

@bhousel, f31f992a21c5ad4d2c30c9a30566201941ea5d3e looks a bit weird, but the idea is that we update the entity the normal way (same as the synchronous update) if the same entity was selected as before. That way the UI gets updated too. Otherwise, we only update the underlying model. I used Network Link Conditioner to verify that the race condition is gone.

@bhousel is correct, modifying tags in place won't work.
@paulmach, I seem to recall you faced a similar issue (how to make an edit in response to an asynchronous external request) for the slide tool. How did you end up solving it?

What's the status on this?

What's the status on this?

Nobody is working on this currently..

Asynchronous tag updating was really tricky to get right (and adding test coverage was even trickier) but I think this will work ok.  Code here for the curious.
merged as
db6234b 2016-05-26 Initiate async wikidata lookup onchange only (not onblur), add tests
0135fbe 2016-05-25 Update async wikidata tag if graph unchanged
669cad7 2016-05-23 Add wikidata asynchronously based on wikipedia <Minh Nguyễn>

Thank you so much, @bhousel!

From the OSM end, if I look at an object or the changeset that a wikidata tag was added in, how do I know whether the wikidata tag was added manually by the user (and presumably checked beforehand) or whether it was derived automatically?  Is something like "source:wikidata=automatically added by iD based on wikipedia" going to be added?
Is there going to be any kind of duplicate checking / warning?  We've had examples in the past where people add wikipedia links to "category of thing A" (e.g. "McDonalds") to an "example of thing A" (a specific fast food joint) in OSM.  Obviously there are many examples of valid duplicate links to wikidata (e.g. the various bits of the M42 motorway, which is taginfo's most common wikidata tag), so it'd be difficult to do.

From the OSM end, if I look at an object or the changeset that a wikidata tag was added in, how do I know whether the wikidata tag was added manually by the user (and presumably checked beforehand) or whether it was derived automatically?

It's probably safe to assume that if you see both wikipedia and wikidata tags modified in the same changeset, the user probably entered the wikipedia tag and iD fetched the wikidata tag for them.

Is something like "source:wikidata=automatically added by iD based on wikipedia" going to be added?

You can open an issue for discussion of this if you feel it's important.. Putting source:*=* on objects has kind of fallen out of favor, but I guess adding a tag like source:wikidata=wikipedia would be alright.

Is there going to be any kind of duplicate checking / warning? We've had examples in the past where people add wikipedia links to "category of thing A" (e.g. "McDonalds") to an "example of thing A" (a specific fast food joint) in OSM. Obviously there are many examples of valid duplicate links to wikidata (e.g. the various bits of the M42 motorway, which is taginfo's most common wikidata tag), so it'd be difficult to do.

I can't think of any way to prevent this.  People can always add wrong things in the wikipedia tag.  We can't duplicate check against all of osm, only the tiles that we've already downloaded, so that makes a check here not very useful - and like you said, sometimes it's an ok thing to do.


WebVR does not work on Chrome 56

After updating mobile Chrome on Android to version 56, none of the examples on the demo page nor the WebVR project we are developing (it was forked from the base VR project) work anymore.
We need to use the WebVR functionality with third party headsets that do not have a clicker button, so the expected behavior is that the screen splits into two halves without the need to use the supported headset. (e.g. Daydream or Vive)
The error we're getting is No VR display or headset is detected., although the same code worked on Chrome 55, just before the update.
I presume that the issue here is that WebVR is supported in Chrome 56, but since the headset is not actually connected, it shows the correct error.

This is an issue that I'm currently looking into as it is currently affecting all WebVR demos that I've tried (three.js and the examples on webvr.info) do not work unless polyfill is forced.
Yes, you are correct that since Chrome 56 now has implemented the WebVR API, polyfill is no longer needed. The issue is that only a small number of devices (I'm assuming the Daydream supported devices) are deemed to a be a compatible display for use with WebVR.

Just to clarify - this is due to polyfill assumed all mobile devices are VR capable. But WebVR has different assumptions about devices, so pretty much all non-daydream mobile phones didn't get VR love :)
Here is related issue on https://github.com/googlevr/webvr-polyfill/issues/177

@yaustar, @Maksims  thank you guys for clarifying this. Will follow the thread from now on.
Do you plan on adding any modifications to the WebVR polyfill included in Playcanvas before the official fix is out?

Since Playcanvas source is not available through online code editor, where could I force usage of the webvr polyfill?

You could override property on pc.VrManager which called isSupported to return false in cases where it is true but you still want to enforce polyfill to get loaded.

I'm a bit confused, not sure if this is a bug or what not.
If you open any WebGL demo, i.e. WebVR 360 Image by itself in Chrome 56, VR & Gyroscope do not work. All you can do is tap/click & drag the screen to look around.
But, if you open any of tutorials page, i.e. WebVR 360 Image Tutorial Page, VR button & Gyroscope are fully functional and work perfectly like they were used to.
I connected my android phone to mac, inspected the page to see if there are any error differences, but there were none. Same errors.
Am I'm missing something here?

It's a http/https issue. WebVR is only enabled on webpages that are on the Origin Trial and only on https.
The tutorials pages are on http so WebVR isn't natively enabled and the polyfill is used instead.
So if you load http://playcanv.as/p/v6qoi4Yt/ (note it's on http), it will 'work'.

Even as a web dev, I wouldn't have thought that https would be such an issue. Thanks for pointing that out @yaustar, have been trying to debug this all morning.

Here's another bug, not really sure it's a bug or not. So I forked the 360 VR image tutorial project, and was expecting same results as on the tutorial project, but it just shows brown/red background in VR instead. Link: https://playcanv.as/p/2EMTn1Dv/
Also, why the User Experience is different on iOS devices vs Android? On Android you have option to turn on the VR. But by default you can use gyroscope to move camera around. On iOS it forces you to use phone in cardboard mode, otherwise it will not work.

Hi together,
thanks for this thread!
I experienced the same problem when using the samples from webvr.info:
https://webvr.info/developers/
When I used the standard version of the actual Chrome 56, the demos did not start.
But then I did the following: downloaded
Chrome Beta (58.0.3029.82)
When I started the demos, I got a message that
google vr services
have to be installed. After I did this, all demos worked again :-)
In addition, I have of course the
Cardboard app (4.2)
installed.

Hi @renatoruk, can you confirm if this is still an issue for you? And if so, on which devices?
There are differences in the way Android behaves on different devices. In particular Daydream-ready devices (Google Pixel phones) can handle WebVR natively, but other devices do not.

Hello @daredevildave, yes, webVR is functional now in Chrome 57 on Android. I tested on Samsung Galaxy S7 so it probably ran the polyfill version.

Hola, tengo un problema al ver contenido webVr en chrome 58 (Android), la imagen sale reducida a un cuadro muy pequeño en la parte inferior izquierda.

@Dhellblazer does this happen to many WebVR demos as well as other engines/frameworks? If so, it is Chrome 58 regression bug, and should be reported here: https://bugs.chromium.org/p/chromium/issues/list

Hello everyone, as I mentioned earlier in Chrome 58 there is the problem that it does not load well the screen, because it comes reduced or only a part. With happiness I comment that the version 59 must leave the next week, at the moment you can download the Beta version, in this version (Chrome 59) the problem is solved. Thank you.


Add Support for Forms

This PR adds the BilinearForm and QuadraticForm types along with the AbstractForm
We define an AbstractForm to be of the structure <x|A|y> where x, y are Tensor Network States and A is a Tensor Network Operator. The AbstractForm contains the mapping from the vertices of the original states to their names in the Form, along with the mapping from the vector space to the dual space.
The BilinearForm is the generic case where x, y and A are all distinct and constructors are defined for this case.
The QuadraticForm is the case when x and y are identical and the constructor definitions reflect that. In this case functions like gradient(qf::QuadraticForm, state_vertices::Vector) are defined for getting the derivative of the qf with respect to the vertices of the original state x.
A short series of tests are included.

Codecov Report
Attention: 29 lines in your changes are missing coverage. Please review.

Comparison is base (da7636e) 73.24% compared to head (92e6a24) 73.29%.

Files
Patch %
Lines

src/Forms/bilinearform.jl
42.10%
11 Missing :warning:

src/Forms/abstractbilinearform.jl
0.00%
9 Missing :warning:

src/Forms/quadraticform.jl
70.37%
8 Missing :warning:

src/Forms/construct_form.jl
95.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #136      +/-   ##
==========================================
+ Coverage   73.24%   73.29%   +0.05%
==========================================
  Files          68       72       +4
  Lines        4033     4108      +75
==========================================
+ Hits         2954     3011      +57
- Misses       1079     1097      +18

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

@JoeyT1994 you marked a number of comments as resolved but then I don't see the corresponding changes made.

@mtfishman I think something has gone out of sync with Github because they are resolved on my local machine. I think it keeps using .jl files from FormNetworks before it was renamed to formnetworks. Let me try to resolve it

I don't think so, at least for now. Just trying to change the folder name, git is very strange about renaming case sensitive things so I think I need to use some extra commands

Okay renaming dealt with

Looks good. I think this already takes us pretty close to being able to use general tensor networks in alternating_update!


Tune Data Loading for IO and CPU utilization [06-04-2016 to 13-04-2016]

Review Owner: @gvramana

closing this as the requirement is handled


Use shim link button behaviour from FE toolkit

This functionality has been moved into the DM frontend toolkit
and so can be pulled in as a dependancy instead.
Related PR - https://github.com/alphagov/digitalmarketplace-frontend-toolkit/pull/394

I will be the winner of that race...


Bubble menu glitches when the editor changes positions

Hey, first of all thanks for the great work
I am trying to wrap my head around bubble menu glitch, that you can check out here https://svelte.dev/repl/89cf6d5d4017435b8b7474c088057ff4?version=3.46.4
If there are many instances of tiptap and they get remounted (like if the positions change) the bubble menu gets rendered with a style "visibility: visible;" (if you click on the tiptap editor after that it goes away)
I can't find a way to make it not do that :(

Hi @jesteracer. I can see the bug. Tried to locate it, but there seems to be nothing wrong here.
It could be with tiptap itself. Took a shortcut to verify this, I tried reproducing this with tiptap's react package. It happens there as well.

I kind of "fixed" it (updated the example code), by just wrapping the  with a div with dispay:none property
Since inside this lib there is also a div, that wrapps the actual component maybe you can add this to the library — it should not break things

Thanks. Nice workaround 👍 .

Since inside this lib there is also a div, that wrapps the actual component maybe you can add this to the library

Maybe If nothing works, First Let's see if this bug can be fixed.

I think you need to add type="button" to the buttons as well.

There are two options to solve this.

wrap a div around bubble menu with display none property, See https://github.com/sibiraj-s/svelte-tiptap/issues/5#issuecomment-1046469762

<div style="display:none;">
   <BubbleMenu editor={$editor}></BubbleMenu>
</div>

Move the BubbleMenu inside EditorContent (Requires v1.1.0)

<EditorContent editor={$editor}>
   <BubbleMenu editor={$editor}></BubbleMenu>
</EditorContent>

Since this is a bug in the bubble menu plugin itself. Closing this issue. There is an open issue to track this https://github.com/ueberdosis/tiptap/issues/2568


fix: load jsdom before SSR build

tested on https://github.com/userquin/vite-ssg-esm-cli-150 with mock and cjs and https://github.com/userquin/vite-ssg-build-v-client-master with mock
fix #230
fix #51
fix #254

maybe we can register global jsdom before loading the ssr entry instead before ssr build


How to increase sample frames number to more than 6?

Hi, thank you so much for the great works! I have questions about sampled frame number, in the paper mentioned

During inference, we uniformly sample 6 frames with center crop.

I am keen to know is possible for us to sample more than 6 frames during inferences? I got this error when I try to use more than 6 frames per video clip.
IndexError: index 6 is out of range
Look forward to your response. Thank you!

it should be able to handle more than 6 frames. can you post more call stacks when you hit this error?

please re-open it if you still hit the issue.

Hi @amsword ,
Thank you for follow up, I still hit the issue and here are the details:
I use the codes as given in here
import av
import numpy as np
from PIL import Image
from huggingface_hub import hf_hub_download
from transformers import AutoProcessor, AutoModelForCausalLM

processor = AutoProcessor.from_pretrained("microsoft/git-large-vatex")
model = AutoModelForCausalLM.from_pretrained("microsoft/git-large-vatex")

# set seed for reproducability
np.random.seed(45)

def read_video_pyav(container, indices):
    '''
    Decode the video with PyAV decoder.
    Args:
        container (`av.container.input.InputContainer`): PyAV container.
        indices (`List[int]`): List of frame indices to decode.
    Returns:
        result (np.ndarray): np array of decoded frames of shape (num_frames, height, width, 3).
    '''
    frames = []
    container.seek(0)
    start_index = indices[0]
    end_index = indices[-1]
    for i, frame in enumerate(container.decode(video=0)):
        if i > end_index:
            break
        if i >= start_index and i in indices:
            frames.append(frame)
    return np.stack([x.to_ndarray(format="rgb24") for x in frames])

def sample_frame_indices(clip_len, frame_sample_rate, seg_len):
    '''
    Sample a given number of frame indices from the video.
    Args:
        clip_len (`int`): Total number of frames to sample.
        frame_sample_rate (`int`): Sample every n-th frame.
        seg_len (`int`): Maximum allowed index of sample's last frame.
    Returns:
        indices (`List[int]`): List of sampled frame indices
    '''
    converted_len = int(clip_len * frame_sample_rate)
    end_idx = np.random.randint(converted_len, seg_len)
    start_idx = end_idx - converted_len
    indices = np.linspace(start_idx, end_idx, num=clip_len)
    indices = np.clip(indices, start_idx, end_idx - 1).astype(np.int64)
    return indices

# load video
file_path = '/path/to/video.mp4'
container = av.open(file_path)

# sample frames
# HERE is the variable I made a change so that num_frames is more than 6
num_frames = 16 #model.config.num_image_with_embedding

indices = sample_frame_indices(
    clip_len=num_frames, frame_sample_rate=4, seg_len=container.streams.video[0].frames
)
frames = read_video_pyav(container, indices)

pixel_values = processor(images=list(frames), return_tensors="pt").pixel_values

generated_ids = model.generate(pixel_values=pixel_values, max_length=50)

print("Generated caption:", processor.batch_decode(generated_ids, skip_special_tokens=True))

and it return this error
│                                                                                │
│   587 │   │   """Get the absolute index for the list of modules"""                 │
│   588 │   │   idx = operator.index(idx)                                            │
│   589 │   │   if not (-len(self) <= idx < len(self)):                              │
│ ❱ 590 │   │   │   raise IndexError('index {} is out of range'.format(idx))         │
│   591 │   │   if idx < 0:                                                          │
│   592 │   │   │   idx += len(self)                                                 │
│   593 │   │   return str(idx)                                                      │
╰────────────────────────────────────────────────────────────────────────────────────╯
IndexError: index 6 is out of range
Need your help to advise how to sample more than 6 frames for one video. Thank you.

@ee2110 Were you able to increase the frame limit?

@ee2110 Were you able to increase the frame limit?

Hi @abisekrk , regrettably no, unable to sample more than 6 frames for each video. Therefore, I try other videoQA or captioning models. I was still keen to know if we can increase the frame limit.


Is it possible to get all validation errors?

I'm working on a some basic crud stuff and noticed that flywheel appears to raise an exception on the first validation failure and that's it.
Is it possible to get a list of all the fields with failed validations?

Just a quick example of what I'm thinking of.
    def pre_save_(self, engine):
        """ Called before saving items """
        self.__engine__ = engine
        v_err = []
        for field in six.itervalues(self.meta_.fields):
            try:
              field.validate(self)
            except ValueError, e:
              v_err.append(e.message)
        if len(v_err) > 0:
          raise ValueError("{}".format(v_err))

Not sure how best to integrate this as an optional feature, possibly an option you can set when defining the model?
Custom error messages would be great too but I digress.

I've made a Base class/mixin to subclass along with Model to add this feature if anyone is interested in it:
class Base(object):
  def validate(self):
    self._fields_with_errors = []
    for field in self.meta_.fields.itervalues():
      try:
        field.validate(self)
      except ValueError, e:
        res = re.search('Validation check on field (.*) failed for value (.*)', e.message)
        self._fields_with_errors.append(res.group(1))
    return len(self._fields_with_errors) == 0

This seems like a pretty good idea. I'll put it on my list of things to do for the next version.


app freeze when working

sorry for annoying u , when i try to retweet tweets app is freeze even if used thread to excute retweet query , how i can solve this problem

Without any code I cannot really help unfortunately.

Closing as no answer.


[TypeScript] Union type with leading symbol not highlighted correctly

type RuleNode =
  | TSESTree.ClassBody
  | TSESTree.Program
  | TSESTree.TSModuleBlock
  | TSESTree.TSTypeLiteral
  | TSESTree.TSInterfaceBody;

Nice, another 'feature' that doesn't get mentioned at all in the "spec". Seems to work with & too.


ci: comment some codes to find the error

No need to merge it since bug related with deployment has been resolved


[Feat] Add Absolute Import Rule

There are many places where imports are implemented as:
import { IPoint } from '../../models/Point';
We should implement a rule that would implement all imports as for example:
import { IPoint } from 'models/Point';

If added as ESLint rule it might impact all the existing imports so be careful, and apply changes to existing imports if needed.

@yuvalbl @SejoB there is already a rule added via /anyway-newsflash-infographics/tsconfig.json from global imports inside 'src' so all the new global imports should be working from under 'src', but I didn't find a way to change the relative imports retroactively except manually. Are you familiar with something automatic?
Related docs https://create-react-app.dev/docs/importing-a-component/#absolute-imports

solved by #831


Return null when elementName is falsey

First off, great lib - thanks for making it!
I think this lib could be better if it simply ommitted inputs when the elementName is null or empty. As it is today, it will raise an exception and kill the entire form scraping process.
Some possible solutions: getNamedFormElementData() could be made to return null instead of throwing an error when elementName is falsey, OR getFormData() could skip the call to getNamedFormElementData() whenever the current iteration's elementName is falsey.

Thanks for the report - v1.2.2 skip falsy values in getFormData() and doesn't throw if an empty string is passed to getNamedFormElementData()


Quiz Exercises: Reduce useless loading of quiz batches

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Motivation and Context

Quiz batches were loaded a lot when they were not needed.
Description

Dashboard used to load the batches for every synchronized quiz exercise, even tho it can only be useful when the quiz is currently active.
Beyond that there were a lot of other places were the batches were loaded with some other request even tho they were not required.
Steps for Testing
Noting in particular should have changed, other than that in the query log there should be a lot less queries involving quiz batches.
Just checking over quiz things in general is a good idea tho.
Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Tested on ts2, works fine. It is not possible to import a quiz, is this intended?

Importing full quiz exercises in another PR, importing is only possible one the question level here.


add a view mode to the cli to show either all text or raw

In order to help debug a juttle program in the CLI that contains
visual sinks, add a --view-mode option to the cli which can be either "text" in
which case all views are rendered as text), or "raw" in which case it just
prints out the raw underling data that would go to the views over the
websocket.
@go-oleg @mstemm

I'd suggest a simpler raw mode that just uses the text view for all sinks instead of this raw viewer. In some cases users might want the exact sequence of points, marks, ticks, eof, etc. but I think in some cases you just want the output.

@mstemm Good idea -- I updated the PR to now support both raw and text output.

lgtm. Can you also update docs/reference/cli.md with the new option?


[Resources]: Support more SQL libraries

Support more SQL libraries as output types from the database macros, such as (async) diesel or seaORM.

This issue is a part of the Product Feature Poll (all issues). React with :+1: if you want this feature. Comment if you have suggestions related to this feature.

Update: async diesel support was added in https://github.com/shuttle-hq/shuttle/releases/tag/v0.42.0

Closing since async-diesel is supported, and SeaORM can be used with SQLx. Comment or open new issues for further suggestions.


Mobile Responsive Design:

Issues: The website may not adapt well to different screen sizes (mobile, tablet, desktop), leading to poor user experience on smaller devices.
Enhancements:
Ensure the website is fully responsive by using media queries and mobile-first design principles. Im

Sorry I cant do this


jemallocator & pyo3 ?

Hello,
I'm trying to use jemallocator in my pyo3 extension (20% speed up in my cargo test, so very interesting !), but I couldn't get it to work:
https://github.com/gnzlbg/jemallocator/issues/142
My lib is based on https://github.com/PyO3/rust-numpy/tree/master/examples/simple-extension,
and I followed instruction from https://crates.io/crates/jemallocator
Reproduction: https://github.com/NotSqrt/rust-numpy/commit/408a5469f5697f4dd0dd6ab57e0f3c38acca36a9
python3 setup.py develop
python3 -c "import rust_ext"

fails with:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/user/rust-numpy/examples/simple-extension/rust_ext/__init__.py", line 1, in <module>
    from .rust_ext import *  # noqa
ImportError: /home/user/rust-numpy/examples/simple-extension/rust_ext/rust_ext.cpython-36m-x86_64-linux-gnu.so: cannot allocate memory in static TLS block

Any tips ? Any idea whether it's even possible to change the allocator in a cdylib ?
Thanks !

https://github.com/jemalloc/jemalloc/issues/937 seems related

It seems jemallocator = {version = "0.3", features = ["disable_initial_exec_tls"]} will be enough !

Thank you for sharing the workaround.


Oauth2 support

The only sample you appear to have works with service accounts.   Does this library also support normal Oauth2 authentication or just service account authentication?
I tried setting the GOOGLE_APPLICATION_CREDENTIALS to a normal Oauth2 cred file and i get the following error message.
(node:14561) UnhandledPromiseRejectionWarning: Error: The incoming JSON object does not contain a client_email field
    at JWT.fromJSON (/home/linda/WebstormProjects/google_nodejs_samples/node_modules/google-gax/node_modules/google-auth-library/build/src/auth/jwtclient.js:195:19)
    at GoogleAuth._cacheClientFromJSON (/home/linda/WebstormProjects/google_nodejs_samples/node_modules/google-gax/node_modules/google-auth-library/build/src/auth/googleauth.js:301:16)
    at ReadStream.<anonymous> (/home/linda/WebstormProjects/google_nodejs_samples/node_modules/google-gax/node_modules/google-auth-library/build/src/auth/googleauth.js:335:36)
    at ReadStream.emit (events.js:228:7)
    at endReadableNT (_stream_readable.js:1185:12)
    at processTicksAndRejections (internal/process/task_queues.js:81:21)

After a while managed made it work using a "hybrid" approach to authentication with OAuth2.

Client-side : prompt user to authenticate and authorize the app with given scopes
After being authorized, client obtains a code that it sends to the server
Server (using google.auth.OAuth2) exchanges the code received for an access_token + refresh_token
Server saves these tokens in DB for later re-use
Server can then use the tokens as credentials to instantiate BetaAnalyticsDataClient and make calls

Server-side code to exchange code for tokens (step 3 & 4) :
     const { data } = await axios.post(
        'https://accounts.google.com/o/oauth2/token',
        qs.stringify({
          code,
          client_id: appClientId,
          client_secret: appClientSecret,
          redirect_uri: 'http://..............', // from my experience, this should be an authorized callback in Google API Console, but not used here so it doesn't matter which one
          grant_type: 'authorization_code',
        }),
        { headers: { 'Content-Type': 'application/x-www-form-urlencoded', }, }
      );

      // SAVE CREDENTIALS DATA TO DB

And to make an API call (step 5) :
      // RETRIEVE CREDENTIALS DATA FROM DB

      const oauth2Client = new google.auth.OAuth2(
        this.appClientId,
        this.appClientSecret,
        'http://localhost:3000/admin/connecteurs/oauth2callback'
      );
      oauth2Client.setCredentials(CREDENTIALS DATA);

      // Set OAuth2 client as credentials for Analytics
      const sslCreds = grpc.credentials.createSsl();
      const credentials = grpc.credentials.combineChannelCredentials(
        sslCreds,
        grpc.credentials.createFromGoogleCredential(oauth2Client)
      );
      this.analyticsDataClient = new BetaAnalyticsDataClient({
        sslCreds: credentials,
      });

It would definitely be much nicer to have something a bit simple server-side, or better documented (it took me a few days to figure all this out based on the various APIs references.

Hi there! We are migrating this repo to googleapis/google-cloud-node. If this issue still persists, please reopen over there. Thank you!


Clarification Needed on TensorFlow Lite Version and GPU Acceleration Configuration

Thank you for developing this excellent library. I've been using the native TensorFlow Lite library directly with Swift/Kotlin and am now migrating my code to use react-native-fast-tflite. Having experience configuring TensorFlow Lite, I have a couple of questions regarding the Android configuration:

Version Selection: Why does this library use an older version of TensorFlow Lite (2.12.0) instead of the latest version (2.16.1)? Is there a specific reason for pinning to this older version?

GPU Acceleration Configuration: I noticed that 'uses-native-library' declarations are required to enable GPU acceleration. While I understand their necessity (as GPU loading fails without them), I'm curious about why they're needed here when they weren't required in my previous direct TensorFlow Lite implementations.

For context, when I previously installed TensorFlow Lite directly, the following Gradle dependencies were sufficient for GPU acceleration:
implementation 'org.tensorflow:tensorflow-lite:2.15.0'
implementation 'org.tensorflow:tensorflow-lite-gpu:2.15.0'
implementation 'org.tensorflow:tensorflow-lite-gpu-api:2.15.0'

Is the need for these additional configurations related to the older TensorFlow Lite version (2.12.0) used in this library? Or am I missing something?

Hey @bglgwyng - thank you for the kind words.
Well, to be honest I don't have an answer to your questions. I built this library a while ago fairly quickly, and I didn't investigate GPU delegates any further. It worked, but I didn't use this library for any of my RN apps.
I know a lot of people use react-native-fast-tflite (and even a lot of my clients use it), but I haven't done any significant changes here since it's release.
I'd appreciate it if you find answers to those questions after just trying stuff out, maybe we can bump the version to latest, and maybe we can simplify the GPU delegate. PRs of course welcome!
Thanks!

iirc I tried to integrate TFLite by source, to avoid having to depend on a pod or gradle input. By source was a bit too complicated because I couldn't get it to build. That would've also worked with Simulators, and is a shared C++ codebase. I haven't pursued this further, but would be cool if you could get that working

@TkTioNG Hello! I found that you are the author of this PR
Here I asked some questions related to the PR, could you please provide answers for them?

@bglgwyng Hi!
OpenCL is not necessary compulsory to enable GPU acceleration. May be you can try to compile your app without using it?
I believe that without using OpenCL, the device will try to use the default graphics api instead (I cannot guarantee you with that XD)
Just that some of the tflite model, and some configurations (like serialization) will need OpenCL, where the default graphics api cannot provide and will failed to be initialized.

@TkTioNG Thanks for the brief reply! What about the version? Do you think it is ok to bump it up?

https://github.com/mrousavy/react-native-fast-tflite/pull/85

@bglgwyng did you find any solution?  I also cannot load any model with GPU enable flag on iOS on models trained after 9/1/00 (approx date).

@lucksp #85  did you try the latest version? I'm using an old model and my problem may not be the same with yours.


EmptyDataError: No columns to parse from file

Hey!
I am trying to run this in google colab and and the 2.1 code block gave me this issue
ValueError                                Traceback (most recent call last)
 in <cell line: 3>()
1 phage_genomes_path = general_path+'/phages_genomes'
2 phanotate_path = '/usr/local/lib/python3.10/dist-packages/phanotate.py'
----> 3 phlp.phanotate_processing(general_path, phage_genomes_path, phanotate_path, data_suffix=data_suffix)
/content/drive/MyDrive/PhageHostLearn/code/phagehostlearn_processing.py in phanotate_processing(general_path, phage_genomes_path, phanotate_path, data_suffix, add, test)
229     """
230     phage_files = listdir(phage_genomes_path)
--> 231     phage_files.remove('.DS_Store')
232     if add == True:
233         RBPbase = pd.read_csv(general_path+'/RBPbase'+data_suffix+'.csv')
ValueError: list.remove(x): x not in list
So i modified the phagehostlearn_processing.py code a little bit to this
if '.DS_Store' in phage_files:
phage_files.remove('.DS_Store')
But now I am receiving this error,
EmptyDataError                            Traceback (most recent call last)
 in <cell line: 3>()
1 phage_genomes_path = general_path+'/phages_genomes'
2 phanotate_path = '/usr/local/lib/python3.10/dist-packages/phanotate.py'
----> 3 phlp.phanotate_processing(general_path, phage_genomes_path, phanotate_path, data_suffix=data_suffix)
8 frames
/content/drive/MyDrive/PhageHostLearn/code/phagehostlearn_processing.py in phanotate_processing(general_path, phage_genomes_path, phanotate_path, data_suffix, add, test)
253             temp_tab.write(split + b'\n')
254         temp_tab.close()
--> 255         results_orfs = pd.read_csv(general_path+'/phage_results.tsv', sep='\t', lineterminator='\n', index_col=False)
256
257         # fill up lists accordingly
/usr/local/lib/python3.10/dist-packages/pandas/util/_decorators.py in wrapper(*args, **kwargs)
209                 else:
210                     kwargs[new_arg_name] = new_arg_value
--> 211             return func(*args, **kwargs)
212
213         return cast(F, wrapper)
/usr/local/lib/python3.10/dist-packages/pandas/util/_decorators.py in wrapper(*args, **kwargs)
329                     stacklevel=find_stack_level(),
330                 )
--> 331             return func(*args, **kwargs)
332
333         # error: "Callable[[VarArg(Any), KwArg(Any)], Any]" has no
/usr/local/lib/python3.10/dist-packages/pandas/io/parsers/readers.py in read_csv(filepath_or_buffer, sep, delimiter, header, names, index_col, usecols, squeeze, prefix, mangle_dupe_cols, dtype, engine, converters, true_values, false_values, skipinitialspace, skiprows, skipfooter, nrows, na_values, keep_default_na, na_filter, verbose, skip_blank_lines, parse_dates, infer_datetime_format, keep_date_col, date_parser, dayfirst, cache_dates, iterator, chunksize, compression, thousands, decimal, lineterminator, quotechar, quoting, doublequote, escapechar, comment, encoding, encoding_errors, dialect, error_bad_lines, warn_bad_lines, on_bad_lines, delim_whitespace, low_memory, memory_map, float_precision, storage_options)
948     kwds.update(kwds_defaults)
949
--> 950     return _read(filepath_or_buffer, kwds)
951
952
/usr/local/lib/python3.10/dist-packages/pandas/io/parsers/readers.py in _read(filepath_or_buffer, kwds)
603
604     # Create the parser.
--> 605     parser = TextFileReader(filepath_or_buffer, **kwds)
606
607     if chunksize or iterator:
/usr/local/lib/python3.10/dist-packages/pandas/io/parsers/readers.py in init(self, f, engine, **kwds)
1440
1441         self.handles: IOHandles | None = None
-> 1442         self._engine = self._make_engine(f, self.engine)
1443
1444     def close(self) -> None:
/usr/local/lib/python3.10/dist-packages/pandas/io/parsers/readers.py in _make_engine(self, f, engine)
1751
1752         try:
-> 1753             return mapping[engine](f, **self.options)
1754         except Exception:
1755             if self.handles is not None:
/usr/local/lib/python3.10/dist-packages/pandas/io/parsers/c_parser_wrapper.py in init(self, src, **kwds)
77
78         kwds["dtype"] = ensure_dtype_objs(kwds.get("dtype", None))
---> 79         self._reader = parsers.TextReader(src, **kwds)
80
81         self.unnamed_cols = self._reader.unnamed_cols
/usr/local/lib/python3.10/dist-packages/pandas/_libs/parsers.pyx in pandas._libs.parsers.TextReader.cinit()
EmptyDataError: No columns to parse from file
I don't think this is because of the change in code I made but I would like some help in resolving this.
Any help would be highly appreciated! @dimiboeckaerts
Thank you!

Hi again Jemina, thank you for pointing out the small bug in the code, I will adjust it accordingly! The remaining problem you still have here actually relates to the comment I made on the other GitHub repo: here you are in fact using PHANOTATE to process a phage genome into its CDSses. My best guess is that running PHANOTATE on Google Colab does not simply work like that. Although it probably should be able to run in a Colab theoretically, perhaps not the way I coded it (I haven't checked this out). You can run PHANOTATE easily on your local computer, it's quire fast and does not take a lot of memory. Same for Kaptive, you can actually just run it locally on your computer. As you can see, running the entire pipeline is not very straightforward at this point. I aim to further simplify it and make it more user-friendly once it is published!

After a short break, I came back to this project again and found that phanotate has updated to produce faa file. This made my work easy and now I have processed 248 genomes of mine and 248 faa files are at my hand, Can i please get some help in proceeding further with these to produce embeddings file and detect RBPs? @dimiboeckaerts
Also, I was learning about the tools you have used, and came across Kaptive, which is trained Klebsiella genomes. So if I have to use this module for other bacterial species then Kaptive will not work right? That's what my understanding is. So, is there a more generalized tool available so that its easier to adapt it for other bacterial species or do i have to modify the bacterial genome processing for my bacteria separately? @dimiboeckaerts

After a short break, I returned to this project again and found that the phanotate had been updated to produce faa file. This made my work easy and now I have processed 248 genomes of mine and 248 faa files are in my hand, Can I please get some help in proceeding further with these to produce embedding files and detect RBPs? @dimiboeckaerts
Also, I was learning about the tools you have used and came across Kaptive, trained for Klebsiella genomes. So if I have to use this module for other bacterial species then Kaptive will not work, right? That's what my understanding is. So, is there a more generalized tool available so that it's easier to adapt it for other bacterial species or do I have to modify the bacterial genome processing for my bacteria separately? @dimiboeckaerts


Introduce setup.py

cc: @tbetbetbe

[Current coverage][cc-pull] is 68.60%

Merging [#18][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #18   diff @@
==========================================
  Files            25         25
  Lines           742        742
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            509        509
  Misses          233        233
  Partials          0          0

Powered by Codecov. Last updated by [450fe27...98b1f33][cc-compare]
[cc-base-branch]: https://codecov.io/gh/googleapis/artman/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/googleapis/artman/compare/450fe27fae45b318a35a2fd095b2f07a8f3c3402...98b1f33cac0cb1f5c762112c8969e392dc8fd7ca
[cc-pull]: https://codecov.io/gh/googleapis/artman/pull/18?src=pr

LGTM


How to use our MediaPipe calculators and TFLite models in Unity

Plugin Version or Commit ID
v0.8.2
Unity Version
2020.3.30f1
Your Host OS
macOS M1
Target Platform
UnityEditor
Description
I've built the MediaPipeUnityPlugin and run succesfully in Unity.
Now I want to build the MediaPipeUnityPlugin with my calculators.
I have calculators and tflite models different from the original MediaPipe repo.
My question is how to use our calculators (i.e. our MediaPipe repositories) while building the plugin.
Code to Reproduce the issue
No response
Additional Context
No response

Please read https://github.com/homuler/MediaPipeUnityPlugin/issues/242#issuecomment-900249401 and https://github.com/homuler/MediaPipeUnityPlugin/issues/277#issuecomment-921433238.

There are 2 ways to achieve this.

Fork mediapipe and pull it in place of the original repository.
Create files under mediapipe_api and apply patch files if necessary (just like this plugin does).

In the former case, you need to switch mediapipe repository's URL to yours.
https://github.com/homuler/MediaPipeUnityPlugin/blob/42106737e3d4b57efddd2e4d8e646b7b8e283fba/WORKSPACE#L76
However, I recommend the latter method if possible, since you may have to modify the patch files by yourself when they conflict with your changes.
In any way, please read the above comments and if you have any questions, please ask.

I'm trying to switch mediapipe repository's URL to mine. I created a new release including my changes (https://github.com/ahmethaydarornek/mediapipe/archive/v<IP_ADDRESS>.tar.gz).

The problem is that Prefix "mediapipe-<IP_ADDRESS>" could not be found in the archive. Error message:

You must change sha256, too.
Otherwise, the cached file will be reused.

Which directory should I use to create sha256? I'm very new about bazel workspace.
I've tried the following command but it gives the same sha256 ("54ce6da9f167d34fe53f928c804b3bc1fd1dd8fe2b32ca4bf0b63325d34680ac")
shasum -a 256 /private/var/tmp/_bazel_ahmethaydarornek/db66cf427f2cddd2341b41874f0fe701/external/com_google_mediapipe/temp3958638767779553797/v<IP_ADDRESS>.tar.gz

I'm very new about bazel workspace.

Well, I think you should read the official document first.
If you really don't care, remove the sha256 attribute and try again. Bazel will download the target file and tell you the sha256 value (if the content has not been tampered with).

Thanks for your helps. The following command also gives the sha256:
curl -L https://github.com/ahmethaydarornek/mediapipe/archive/v<IP_ADDRESS>.tar.gz | sha256sum


Implement the search for international results

Reason (Why?)
The usefulness of einander-helfen.org depends on the versatility of the results. We want to be able to offer international, as well as national, results.
Solution (What?)
The user should be able to switch between national and international search. They should be able to search for countries and/or continents, as well as do a full-text search of the international posts, either within a selected country/continent or without selecting one to show all international results relevant to their search term.
Out of scope
The search for the national results stays as is. The international search is merely an extension to that.
Layout/UI will be done in another (frontend) issue.
Relation to other Issues
#224
#244 has to be done before implementing this issue.
Acceptance criteria
When international search ("Weltweit" in the dropdown next to the location search bar) is selected and the user starts typing in the location search bar, autocomplete should offer countries or continents instead of postal code and place. Any given search results in "international mode" should be international posts only, located anywhere but in Germany.
Searching for a country or continent returns posts located in that country/continent. Full-text search without selecting a country/continent returns all fitting international results. Full-text search with a country/continent selected returns all fitting results in the respective country/continent.

Correct select size in tablet view
Edit smartphone header
Don't use radius in interntaional search

Correct select size in tablet view
Edit smartphone header
Don't use radius in interntaional search


JOSS-required community guidelines?

Where are these? They are not clear in the home directory. They should be there or in the README.md

Thanks! I forgot to write them. Contribution instructions are now in CONTRIBUTING.md


Citation

Hi kite authors,
Thanks for developing a useful tool.
Do you all have a preferred citation for kite?
I've already cited kallisto and the bustools papers, but for kite I'm using the Github repo as a software citation for now.
Thanks!
Jacob

Same question here. I'm using the GitHub repo as well.

Bump (@jasegehring) -- I've got a paper going out soon and just wanted to double check this citation in case you have a preference!


moving pod install command to npm run install:pod

Simplifying the usage by moving all the commands into npm/npx

Please fix the json conflict


Android M java.net.PlainSocketImpl$PlainSocketInputStream vs Android N java.net.SocketInputStream?

Hi Folks,
In Android M java.net.Socket.getInputStream() gets the java.net.PlainSocketImpl$PlainSocketInputStream
whereas android N java.net.Socket.getInputStream() gets the  java.net.SocketInputStream
Whats is the implementation changes or behavioral changes of these 2 API's.
Problem:Reading from socket is stuck indefinitly in android N device.
App Design(https://github.com/square/okhttp/issues/2641):
1)Okhtp http library with connectionpool(TCP socket reuse) is used.
2)SSL is handled by WolfSSL by setting the FD.
https://www.wolfssl.com/documentation/wolfssl-jni-javadocs/com/wolfssl/WolfSSLSession.html#setFd-java.net.Socket-
3)Read is stuck in Android N when same socket is used to read data.
https://www.wolfssl.com/documentation/wolfssl-jni-javadocs/com/wolfssl/WolfSSLSession.html#read-byte:A-int-
4)By default, wolfSSL simply gets the internal file descriptor of the socket and does a native recv() call on that descriptor.Guess is that the behavior of PlainSocketInputStream and SocketInputStream differ, where SocketInputStream blocks.
https://github.com/wolfSSL/wolfssl/blob/master/src/io.c#L290
@swankjesse your thoughts on this?
PS: Above works in android < N.
Thanks
Nithin

Sounds like a bug in WolfSSL. Nothing we can fix at our layer.


Sort map fields for TextFormat.

Since Bool isn't Comparable, add a sorting function to MapKeyType and use that
in TextFormatEncodingVisitor.

I'm not sure this is the best way to do this, but this is what occurred to me last night.
I'm not thrilled with it having to make something public, but I didn't see a better way to do this that let us restrict the needed definitions to something within the library since the Visitor api itself is public.
Fixes #1075

Talking with @allevato in person, we realized we might be able to do this like the Custom JSON support with a protocol internally to the library.  I'll take a shot at that before moving ahead with this since it would avoid adding to the public interfaces.

And that isn't gonna work because I'd need an associated type, so I guess we're sorta stuck exposing something else at the moment. :(

Landing since no one has come up with a better idea.  We can always revisit if something better comes along (especially if we haven't cut a release), but hopefully the underscore symbols gives up a little wiggle room.


test package name change

What issue does this pull request resolve?
What changes did you make?
Is there anything that requires more attention while reviewing?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.263%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4110

Relevant Lines:
4206

💛  - Coveralls


SQL: Support more result formats, add columns header.

Add result formats for line-based JSON and CSV.
Add X-Druid-Sql-Columns header with a list of all columns that
the response will contain.
Add more comprehensive documentation on what callers should expect
when making Druid SQL queries.

I realized that a single blank line is a better trailer (it still lets you detect truncation, and is easier to ignore). I'll change this.


Update deploy.sh

Thanks Subhadeep, but I prefer it the way it is

Its my first PR. I'm more than happy that I'm able to contribute whether or
not it's useful. Thank you so much for the acknowledgement.
On Fri, 11 Oct, 2019, 10:41 PM Gregor Martynus<EMAIL_ADDRESS>wrote:

Thanks Subhadeep, but I prefer it the way it is
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hoodiehq/first-timers-bot/pull/230?email_source=notifications&email_token=AIW2HKO66X5NJIP5IYV33UDQOCXSJA5CNFSM4I7XGVUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBAUF3A#issuecomment-541147884,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIW2HKKRZKK7VQRJGDDCV3TQOCXSJANCNFSM4I7XGVUA
.

Congratulations on your first pull request :) It was perfect 👌 Just not desired. Often times it's a good idea to create an issue first to ask the maintainers if the change is something they would be interested in, then offer to send a pull request. Good luck!

I actually didn't know any of the process. Thanks a lot for the
information. I'll keep these suggestions in mind the next time.
So according to you I should click that raise issue button to ask for
permission and further insight right?
On Fri, 11 Oct, 2019, 10:51 PM Gregor Martynus<EMAIL_ADDRESS>wrote:

Congratulations on your first pull request :) It was perfect 👌 Just not
desired. Often times it's a good idea to create an issue first to ask the
maintainers if the change is something they would be interested in, then
offer to send a pull request. Good luck!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hoodiehq/first-timers-bot/pull/230?email_source=notifications&email_token=AIW2HKNDZ2RPJ7HUAGWGM4TQOCY2DA5CNFSM4I7XGVUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBAVB3A#issuecomment-541151468,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIW2HKJJTUO5JTPZKMCX3TDQOCY2DANCNFSM4I7XGVUA
.

Yes. If you find something that looks like you could improve, create a new issue about it and offer to send a pull request. For very simple things like a typos you can send a pull request directly

Thanks for your reply. I'll keep these in mind.
On Fri, 11 Oct, 2019, 11:01 PM Gregor Martynus<EMAIL_ADDRESS>wrote:

Yes. If you find something that looks like you could improve, create a new
issue about it and offer to send a pull request. For very simple things
like a typos you can send a pull request directly
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hoodiehq/first-timers-bot/pull/230?email_source=notifications&email_token=AIW2HKOZO4CWXKLWRDZOLI3QOC2A3A5CNFSM4I7XGVUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBAV4YY#issuecomment-541154915,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIW2HKOW23PZBBZSPSPOTUTQOC2A3ANCNFSM4I7XGVUA
.


Use R.swift

See: https://github.com/mac-cain13/R.swift
You can get rid of segue and other constants :)

Neat, I'll take a look!

This looks really cool, but our existing work flow works okay and I don't foresee us having time to move over to R, even if I'd prefer to use it. Thanks for opening the issue and bringing this to our attention.


Implement U013: Unnecessary TypedDict syntactic form

https://github.com/charliermarsh/ruff/issues/351

@charliermarsh I think I'm almost done here, might be interesting to have your feedback.
I also have to implement: Do not convert when keys are not valid identifiers
// MyType9 = TypedDict('MyType9', {'in': int, 'x-y': int})

(We have IDENTIFIER_REGEX in src/flake8_bugbear/constants.rs that I think can help with that.)

Two question:

Where should I locate IDENTIFIER_REGEX? Seems wrong to import from flake8
When there is one non valid identifier, we must not return any check, because it is a valid reason to use the functional form. Do you have an idea on how to structure that? It seems to me complicated to juggle between fix errors (meaning we failed to fix something fixable) and check errors (meaning that there is no check to raise). Of course I can duplicate the parsing in each phase, but it seems like a waste

Let's put IDENTIFIER_REGEX in... src/python/identifiers.rs for now.
I think we should just do that validation when we run the check, and in the fix, assume that each string is a valid identifier. That's generally the pattern we've followed elsewhere: the check is responsible for all validation, and then the fix can make assumptions about the input. Does that make sense?

I think this one is OK now. I guess I'll be quicker to do NamedTuple with this experience


Allow client to initialize some input ports when debugging

This small change gives the client the ability to manually initialize some input ports before firing up debugFullscreen similar to how fullscreen works on regular elm modules. I hope this will contribute a small step towards closing #36. It is only intended for initialization of ports. Never-the-less this makes it possible to debug many (most?) embedded widgets which are impossible to debug right now.

This should get added in a safer way when we add things back after Elm 0.17


Add answer column in the hunt's main page (closes #25)

As described in #25, it would be useful to have the answer to solved puzzles visible in the hunt's main page. This PR adds the "Answer" column in the puzzle list which reveals answers as they are solved. Example of how this looks like:

Tried running it locally. Seems to be working fine.


tensorflow c++ api session->Run segv for batch size>1

Hi,
i build tensorflow 1.4.0 to .so and call c++ session api Run to predict, when the input batch size is 1, the  Run api works well, but  batch_size    > 1, the Run will segv as follows:
#12 0x00002b12e0122c72 in tensorflow::LaunchMatMulBase<Eigen::ThreadPoolDevice, float>::launch(tensorflow::OpKernelContext*, tensorflow::Tensor const&, tensorflow::Tensor const&, Eigen::array<Eigen::IndexPair, 1ul> const&, std::vector<long long, std::allocator >, bool, tensorflow::Tensor) () from lib/64bit/libtensorflow_cc.so
#13 0x00002b12e0123231 in tensorflow::MatMulOp<Eigen::ThreadPoolDevice, float, false>::Compute(tensorflow::OpKernelContext*) () from lib/64bit/libtensorflow_cc.so
#14 0x00002b12e20e6a4c in tensorflow::ThreadPoolDevice::Compute(tensorflow::OpKernel*, tensorflow::OpKernelContext*) () from lib/64bit/libtensorflow_framework.so
#15 0x00002b12e20b7669 in tensorflow::(anonymous namespace)::ExecutorState::Process(tensorflow::(anonymous namespace)::ExecutorState::TaggedNode, long long) () from lib/64bit/libtensorflow_framework.so
#16 0x00002b12e20a6290 in std::_Function_handler<void ()(), std::_Bind<std::_Mem_fn<void (tensorflow::(anonymous namespace)::ExecutorState::)(tensorflow::(anonymous namespace)::ExecutorState::TaggedNode, long long)> ()(tensorflow::(anonymous namespace)::ExecutorState, tensorflow::(anonymous namespace)::ExecutorState::TaggedNode, long long)> >::_M_invoke(std::_Any_data const&) () from lib/64bit/libtensorflow_framework.so
#17 0x00002b12e1d576b7 in Eigen::NonBlockingThreadPoolTempltensorflow::thread::EigenEnvironment::WorkerLoop(int) () from lib/64bit/libtensorflow_framework.so
#18 0x00002b12e1d565a2 in std::_Function_handler<void ()(), tensorflow::thread::EigenEnvironment::CreateThread(std::function<void ()()>)::{lambda()#1}>::_M_invoke(std::_Any_data const&) () from lib/64bit/libtensorflow_framework.so
#19 0x00002b12e7726f50 in execute_native_thread_routine_compat () from /icd/dlsh_t2b/luther/tf.integ2.dev/tools.lnx86/lib/64bit/libstdc++.so.6
#20 0x0000000004385269 in create_head(void*) ()
#21 0x00002b12e71309d1 in start_thread () from /lib64/libpthread.so.0
*** Stack trace in log file.
free model
terminate called after throwing an instance of 'std::system_error'
what():  Resource deadlock avoided
the predict code :
std::vectortensorflow::Tensor outputs;
tensorflow::Status run_status = session_->Run({input},{model_outputs_name_}, {}, &outputs);
After i upgrade tensorflow to 1.8.0, crash too, please help to review this issue.
chao

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

Can you please test it against the latest version of TensorFlow and post your findings?
Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!