Can't create new table

Describe the bug
I can't create a new table. Return error.
To Reproduce
When you try to create a new table, server return this error:

Error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ') COMMENT='', COLLATE='utf8mb4_0900_ai_ci', ENGINE=InnoDB' at line 1

Version 0.4.0
OS: Windows 10 PRO
MYSQL Version: 8.0.17

Hello @pronexos,
i guess you can't reproduce this issue but if you remember approximately the table structure (fields, indexes, foreign keys) i can try to investigate in any case.

Hi Fabio! It was my fault. I didn't create any column when create the table, that's the problem.
An alert could be added to prevent a table from being created without adding a column.

I made this small change (e8af2d24a869f7667c069936648808952d2062ab) to avoid the error you faced.

Thanks! You are awesome!


Update: Removal of Hosting Price Page from Account and Pricing Category.

Pull Request
Removed Hosting price page from Account and pricing category
Fixes: #27

Pricing FAQ pages needs to be tweaked as well. I'll open a separate issue to track that effort. For now, I'll let this merge as the current scope is completed.


Don't allow "Pdf" in the substring of a new nusiance name

If the substring "Pdf" is in the new name of a nuisance edit rename then there seems to be errors in the Constraint Pdf production. For now, this will cause the python to exit if "Pdf" is in the name.

Closing this as it is apparently already fixed in a previous commit


Cannot run CLI with ts-node

Running CLI with ts-node results in an error when executing any AskScript code: Cannot find module './parser/askscript.grammar'
ts-node + CLI is useful for speeding up debugging and implementing new features & resources.
Lukaszs-MacBook-Pro:askql milimetr$ ts-node --files src/cli.ts
🦄 3
Uncaught Error: Cannot find module './parser/askscript.grammar'
Require stack:
- /Users/milimetr/Desktop/x/askql/src/askscript/index.ts
- /Users/milimetr/Desktop/x/askql/src/cli.ts
    at Generator.next (<anonymous>)
    at REPLServer.<anonymous> (/Users/milimetr/Desktop/x/askql/src/cli.ts:58:38)
    at Object.parse (/Users/milimetr/Desktop/x/askql/src/askscript/index.ts:60:27)
    at parseToAst (/Users/milimetr/Desktop/x/askql/src/askscript/index.ts:26:23)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Module.require (internal/modules/cjs/loader.js:1040:19)
    at Function.Module._load (internal/modules/cjs/loader.js:862:27)
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:980:15) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/Users/milimetr/Desktop/x/askql/src/askscript/index.ts',
    '/Users/milimetr/Desktop/x/askql/src/cli.ts'
  ]
}
🦄

That would be awesome! We would need ts-node to take care of compiling AskScript parser or just do this https://github.com/xFAANG/askql/issues/60

What's your plan on this @czerwinskilukasz1 ?

It seems as a workaround one can run:
npx pegjs src/askscript/parser/askscript.grammar.pegjs && ts-node --files src/cli.ts

Which is enough for now. Removing v1.2 tag.

Fixed with npm run cli:dev command.


What is the scale net?

I notice that there is a scale net in the code. However, I cannot find the corresponding descriptions in the paper. Could you please provide more explanations about this scale net?
Besides, I find that the scale net is not trained in the training code. I am also confused about this.
Thanks!

The scale net is pretrained to scale the shape of SMPL body mesh because we do not use the camera parameters predicted by HMR. During training, the net is fixed like other SMPL parameters. You can also use the camera parameters predicted by HMR as we do on DeepFashion, it will produce similar results.

The scale net is pretrained to scale the shape of SMPL body mesh because we do not use the camera parameters predicted by HMR. During training, the net is fixed like other SMPL parameters. You can also use the camera parameters predicted by HMR as we do on DeepFashion, it will produce similar results.

Thanks for the reply!
In "train_market.py", the scale net seems only randomly initialized, but no pre-trained weights are loaded. Did I miss something here?
By the way, could you please let me know why you do not use the camera parameters of HMR for market dataset?

Thanks for the reply!
In "train_market.py", the scale net seems only randomly initialized, but no pre-trained weights are loaded. Did I miss something here?
By the way, could you please let me know why you do not use the camera parameters of HMR for market dataset?

The pretrained model of the scale net now can be loaded correctly by "--scale_model". You can find it in the models for Demo. Because the quality of images on Market-1501 is not good as training images used by HMR, we expect that the scale net trained dedicatedly for Market-1501 can work better than the HMR model. Actually, the result is slightly better (probably lifting 0.001 for SSIM).

The pretrained model of the scale net now can be loaded correctly by "--scale_model". You can find it in the models for Demo. Because the quality of images on Market-1501 is not good as training images used by HMR, we expect that the scale net trained dedicatedly for Market-1501 can work better than the HMR model. Actually, the result is slightly better (probably lifting 0.001 for SSIM).


[Question] Custom error message for invalid type

Suppose this code:
export const userSchema = z.object({   id: z.string().nonempty("Missing Parameter"),   password: z.string().nonempty("Missing Parameter") });
If I want to have a custom invalid type error message, such as "Field id does not have the correct type", what can be done?
Neither the string or number functions have overloads for messages, meaning they don't accept a custom error message. Is there anything I can do to add this invalid type error message?
i.e.
z.string("Invalid type for id").nonempty("Missing Parameter")

Just started using the library and had same issue. So I'm not sure if there not any better solution to this.
I checked where the default messages come from. The defaultErrorMap
And found one location where you can override it: ParseParams in .parse & .parseSafe
import * as z from 'zod';
import {defaultErrorMap} from 'zod/lib/src/defaultErrorMap';

// schema.parse works the same
schema.safeParse(data, {
  errorMap: (error, _ctx) => {
    if (error.code === z.ZodErrorCode.invalid_type) {
      if (error.expected === 'string') {
        return {message: 'custom message'};
      }
    }
    return defaultErrorMap(error, _ctx);
  }
});

This will fallback to the default, except your overrides...
It does not look very nice in my opinion. Not sure if that is the intended way to do it...

Related issue
https://github.com/colinhacks/zod/issues/281

Duplicate of #97, this is on the roadmap.
@akomm If you upgrade to Zod 3 you can just call z.setErrorMap anywhere and it will globally update the error map for all calls to .parse/.safeParse - no need to pass in the custom error map every time.


Refacto the tester's code

The go code in it was a quick copy paste of another tool that was not very clean in the first place, and it was my first use of Golang ever, which is one of the reasons why it's so unclean at the moment. It would very much need to be done in a more elegant way.
I will do a few changes right now but I don't have the time to change all of it for now.

[ ] Total refacto
[x] Separate service and service config
[x] Remove duplicate database configuration
[x] Add newlines in some part of the code to make it clearer
[x] Remove error checks on the same lines as error definitions
[x] Fix comments
[x] Add comments to struct variables where it's needed

The total refacto will not be done for the 1.1.4.
The current state of the tester is fine enough for its purpose.


coverage loss and train loss error

I have read all the issues and cant find any solution.
My train_loss is good at the beginning of the train process ,about 9.0~8.0.
But after some train steps,the loss decreased quickly ,about 0.000x(0.0001,0.0007)
And the coverage loss is always 0 during the train process.
The model wont be good and I need some help and advice!
Thanks!!

By the way, I used Chinese language data.

problem solved after change the vocab size.


Fix short name for employ eu citizens

https://github.com/alphagov/govuk-app-deployment-secrets/blob/master/shared_config/find-eu-exit-guidance-business.yml#L194

The facet short_name is attribute used as a display value for one particular facet on the business readiness finder. This custom value provides a better grouping header for results. The facet value on the relevant content item needs to be consistent with what's currently in the finder facets definition.
This change will need deploying and the facet group importing and publishing as per the rake tasks documented here : https://docs.publishing.service.gov.uk/manual/publishing-a-facet-group.html

@steventux Does the facet schema need to be updated too?

@leenagupte no https://github.com/alphagov/govuk-content-schemas/blob/master/formats/facet.jsonnet#L52


'T

Could the 'TRY' button be disabled when the user has an empty path parameter? The field is required and therefore the user can still click 'TRY'.
Having the following request:

myapp/books   - it calls internally a service for retrieving all books
myapp/books/{book-uid}   - passing now an empty book-uid, it calls internally myapp/books, instead of myapp/books/

So I would like to disable 'TRY' button, and therefore the user can not make a request with an empty parameter.
Where it will be the line code to make this update?
Thank you!

Well, RapiDoc do not validate any of the inputs on purpose,
this allows people to test their APIs when they provide incorrect values. for instance you can check what response your API returns by sending a string where the field is defined as number. Same goes to check the API response when a parameter is not provided.
We wont be supporting this functionality as it will stop users from testing their APIs with negative cases


Probably wrong docker-compose file syntax

When I am trying to un-comment the lines for doing a team edition deployment as specified in the Production Docker install documentation as well as in the README.md
I get the following error
$ git clone https://github.com/mattermost/mattermost-docker.git
Cloning into 'mattermost-docker'...
remote: Enumerating objects: 20, done.
remote: Counting objects: 100% (20/20), done.
remote: Compressing objects: 100% (20/20), done.
remote: Total 1643 (delta 7), reused 10 (delta 0), pack-reused 1623
Receiving objects: 100% (1643/1643), 335.02 KiB | 1.09 MiB/s, done.
Resolving deltas: 100% (806/806), done.
$ cd mattermost-docker/
$ emacs docker-compose.yml
$ git diff
diff --git a/docker-compose.yml b/docker-compose.yml
index e045a30..027d9d2 100755
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -22,8 +22,8 @@ services:
   app:
     build: app
       # uncomment following lines for team edition or change UID/GID
-      # args:
-      #   - edition=team
+      args:
+        - edition=team
       #   - PUID=1000
       #   - PGID=1000
     restart: unless-stopped
$ docker-compose build
ERROR: yaml.parser.ParserError: while parsing a block mapping
  in "./docker-compose.yml", line 23, column 5
expected <block end>, but found '<block mapping start>'
  in "./docker-compose.yml", line 25, column 7

After looking further with previous issues, I came across this Merge Request  #146
According to it there should be a context: block in the docker-compose.yml
So I tried adding context and check if the docker-compose build works. Apparently it did not.
$ emacs docker-compose.yml
$ git diff
diff --git a/docker-compose.yml b/docker-compose.yml
index e045a30..b0b2487 100755
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -21,9 +21,9 @@ services:

   app:
     build: app
-      # uncomment following lines for team edition or change UID/GID
-      # args:
-      #   - edition=team
+      context: app
+        args:
+         - edition=team
       #   - PUID=1000
       #   - PGID=1000
     restart: unless-stopped
$ docker-compose build
ERROR: yaml.scanner.ScannerError: mapping values are not allowed here
  in "./docker-compose.yml", line 24, column 14
$

I also checked if there is an error with  my YAML, although online linters says that it  is valid, the docker-compose still fails.

Am I missing something or is there a genuine mistake in the  docker-compose file for the team edition deployment scenario?

I have the same problem upgrading an existing installation. The args: ... block throws a syntax error.

The syntax should be:
app:
    build:
      context: app
      # uncomment following lines for team edition or change UID/GID
      args:
        - edition=team
      #   - PUID=1000
      #   - PGID=1000

Thanks @VenturaFranklin, that worked for me!

I am trying to docker-compose  build using the below yml file.  but i am getting the error . Can some help please?
ERROR: yaml.parser.ParserError: while parsing a block mapping
in "./docker-compose.yml", line 2, column 2
expected , but found ''
in "./docker-compose.yml", line 3, column 3
version: '3'
services:
jenkins:
container_name: jenkins
image: jenkins/jenkins
ports:
- "8080:8080"
volumes:
- "$PWD/jenkins_home:/var/jenkins_home"
networks:
- net
remote_host:
container_name: remote-host
image: remote-host
build:
context: centos7
networks:
- net
networks:
net:

@umeshkirron Your comment seems irrelevant to the issue being discussed here.

@rajudev , Just pasted here ,  to  see if one can find any syntax  errors in the  docker composer yaml file . But  you can ignore it.
After many tries , finally my code worked.


Unrecognized field at: logging.type

logging:
  level: INFO
  loggers:
    "io.dropwizard": INFO
    "org.hibernate.SQL":
      level: DEBUG
      additive: false
      appenders:
        - type: file
          currentLogFilename: /var/log/myapplication-sql.log
          archivedLogFilenamePattern: /var/log/myapplication-sql-%d.log.gz
          archivedFileCount: 5
  appenders:
    - type: console

When i have added the above configuration in yaml file i am getting the below error.
* Unrecognized field at: logging.type
    Did you mean?:
      - level
      - loggers
      - appenders

@Madhukar07565 Which version of Dropwizard are you using?

0.8.1

I beleive Dropwizard 0.8.1 doesn't support complex configuration for individual loggers. This feature was added only in Dropwizard 0.9. Please, see the docs for the 0.8.1 version.


app is not defined

I was using flatpickr fine, but since I moved to a modular structure things stopped working.
My setup was:
<div>
<input type="text" placeholder="Select Date.." id=new-date>
</div>

And:
const fp = flatpickr("#new-date", {

This still works as expected also on jsfiddle, however on my modular setup it gives me the following error:
TypeError: Cannot set property 'D' of undefined plus a bunch of errors with lodash, this is definitly not the way to go, I read the issues and I figured I need to create a new instance. So I moved to:
const Flatpickr = require("flatpickr");
const fp = new Flatpickr("#new-date", {

And now I get a single Cannot read property 'add' of undefined which in other words flatpickr doesn't find my element. I tried changing it to a class and use .new-date but really no luck so far. I've been reading and googling as much as possible but I really can't figure what I'm missing. What I'm doing wrong?

I've change my setup, but I still can't seem to get it to work, I'm using Vue.
My component looks like this:
<template>
<input type="text" id="flatpickr" placeholder="Select a range" />
</template>

<script>
const Flatpickr = require("flatpickr");

var defaultStart = new Date(new Date().setDate(new Date().getDate() - 10)).toISOString().slice(0, 10)
var defaultEnd = new Date().toISOString().slice(0, 10)

export default {
  name: 'DatePicker',
  props:['startDate', 'endDate'], // I don't really need this but I should pass data to all Children
  mounted() {
    new Flatpickr('#flatpickr', {
      dateFormat: "Y-m-d",
      mode: 'range',
      altInput: true, // Human Readable
      minDate: new Date().fp_incr(-60), // 60 days from today
      maxDate: defaultEnd,
      // defaultDate: [defaultStart, defaultEnd],
      // minDate: '2017-01-01', // 60 days from today
      // maxDate: '2017-05-05',
      // defaultDate: ["2017-02-02", "2017-04-04"],
      locale: { firstDayOfWeek: 1},
      onClose: function(selectedDates, dateStr, instance) {
        let startDate = selectedDates[0].toISOString().slice(0, 10);
        let endDate = selectedDates[1].toISOString().slice(0, 10);
        this.$emit('change', { startDate, endDate });  // emit to eventHub
      }
    })
  }
}
</script>

I keep getting Cannot read property 'add' of undefined ? what am I missing??


HDDS-10678. [hsync] Merge recent commits from master #4

What changes were proposed in this pull request?
HDDS-10678. [hsync] Merge recent commits from master #4
Please describe your PR in detail:
Merge commits (38 in total) from master in the recent 2 weeks.
Conflicts are trivial.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10678
How was this patch tested?
Existing tests.

wrong base


Pointer type parameters

I have a go generate command in some code of the form:
//go:generate peds -maps entryHandlesMap<Entry,handles> -sets handles<*EntryHandle> -pkg=conntrack -file=peds.go

The *EntryHandle type parameter seems to be the cause of the issue: I get this error, running go generate on the containing package:
$ go generate
Error: Invalid set specification: handles<*EntryHandle>

Generate statically type safe code for persistent data structures.

USAGE
peds

FLAGS        EXAMPLE
  -file      path/to/file.go
  -imports   import1;import2
  -maps      Map1<int,string>;Map2<float,int>
  -pkg       package_name
  -sets      Set1<int>
  -vectors   Vec1<int>

instance.go:14: running "peds": exit status 1

Thanks for reporting, I suspect you are right. Will look into it (if you want to issue a PR with a fix that's also OK! :-)).

Just saw #3 that solved it the same way I did. Could you merge that?

Yes, done!


Support preparsed HTTP requests for WebSocket connections

We have a slight hack here: https://github.com/crossbario/autobahn-python/blob/master/autobahn/twisted/resource.py#L142
A better solution would be to reuse the already parsed (by twisted) HTTP requests, and not serialize to bytes and then parse again ..

needs to touch: autobahn.websocket.protocol.WebSocketServerProtocol.processHandshake


flatten the Travis pipeline => more parallelism => faster failure => …

…happier developers

Codecov Report

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

@@            Coverage Diff             @@
##           master     #215      +/-   ##
==========================================
- Coverage   65.31%   65.15%   -0.16%
==========================================
  Files          27       27
  Lines        1260     1260
  Branches      158      157       -1
==========================================
- Hits          823      821       -2
- Misses        437      439       +2

Impacted Files
Coverage Δ

core/shared/src/main/scala/scalaz/zio/Queue.scala
94.28% <0%> (-5.72%)
:arrow_down:

core/jvm/src/main/scala/scalaz/zio/RTS.scala
91.44% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

On Thu, 23 Aug 2018, 16:55 Tim Steinbach<EMAIL_ADDRESS>wrote:

@NeQuissimus requested changes on this pull request.
In .travis.yml
https://github.com/scalaz/scalaz-zio/pull/215#discussion_r212341258:

@@ -48,6 +50,8 @@ jobs:
scala: 2.11.12
- <<: *test
scala: 2.11.12

<<: *microsite

We won't want a microsite for 2.11. It will look identical to 2.12 and
overwrite the 2.12 one.

oh, the builds are not isolated. in that case i guess we can do without
verifying microsite with scala 2.11

In .travis.yml
https://github.com/scalaz/scalaz-zio/pull/215#discussion_r212341477:

   script: sbt ++$TRAVIS_SCALA_VERSION coverage test coverageReport

   after_success: bash <(curl -s https://codecov.io/bash)
 - &microsite

 stage: microsite

 stage: test

I think the microsite should stay its own stage.

why? does it depend on the other models being built beforehand?
—

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/scalaz/scalaz-zio/pull/215#pullrequestreview-148951496,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AImgXRnzjEWqvBC9f6-v9CDdf61o_pRYks5uTsJLgaJpZM4WJp0K
.

@sideeffffect We could rename the microsite stage to "documentation" and if we ever do something else, we can add it there. I would only parallelize the parts that are cross-built.

do we even need lint for both 2.11 and 2.12? what difference would it make?

Probably not, as long as test and publish are cross-built.

@NeQuissimus are you happy with this now?

btw, since this as good a place as any: I did a complete write-up of a build

Scala

multi-module
cross-built

Travis
Codecov
Sonatype

https://github.com/NeQuissimus/test-and-stuff
I will likely add a microsite that gets auto-deployed over the coming days/weeks.
Can you please check if we have other discrepancies?

@NeQuissimus sorry for replying so late, I've been a bi busy...
that testing project of yours looks great!!! besides microsite, the only other difference I can see is the [ -z \"${encrypted_316f730cabee_iv}\" ] || openssl ... business you've add recently


Nested  yaml/yml/json files don't show in document list

Describe the issue that you're seeing. Any Loom videos or screenshots usually help a lot!
Nested files of non md/mdx filestypes don't show up in document list

It only seems to show the directories if a md/mdx file is nested, and used as the format
Reproduction
https://github.com/tinacms/tinacms/pull/4255
Steps to reproduce

Pull this branch https://github.com/tinacms/tinacms/pull/4255
Add "reference" content to the root of the basic-iframe example (Ask James for access)
Run the basic-iframe example
Go to http://localhost:3000/admin/index.html#/collections/yaml_test/~
See how document list is empty

System Info
System:
    OS: macOS 13.4.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 26.79 MB / 32.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 16.18.0 - ~/.nvm/versions/node/v16.18.0/bin/node
    Yarn: 1.22.19 - ~/Documents/forestry/unity/hexadocs/node_modules/.bin/yarn
    npm: 9.1.2 - ~/.nvm/versions/node/v16.18.0/bin/npm
    pnpm: 7.4.0 - ~/.nvm/versions/node/v16.18.0/bin/pnpm
  Browsers:
    Chrome: 117.0.5938.149
    Safari: 16.5.2
  npmPackages:
    @tinacms/cli: 0.0.0-20231011031444 => 0.0.0-20231011031444
    tinacms: 0.0.0-20231011031444 => 0.0.0-20231011031444

Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug.

This issue was due to some indexing issues around certain documents. This PR should have fixed the error-handling to surface the issues better: https://github.com/tinacms/tinacms/pull/4259


Add support for obsolete header field format

Putting whitespace between header field name and colon is valid in the obsolete syntax.
The check in Field.newRawField() was an attempt to prevent users from creating invalid Field instances. But right now we only use this method with data coming from the MIME4J parser. So even an updated check has the potential to do more harm than good.
Fixes #1959

Reproduction steps in #1952 imply this is widely deployed...

In this case it was my mistake. This was once valid for the internet message format. That's why it's still included in the RFC and needs to be supported by software reading messages. We are however not allowed to create such header lines anymore (and we don't).

@philipwhiuk: https://tools.ietf.org/html/draft-thomson-postel-was-wrong-00


Split into multiple projects

The project is currently small but support multiple Database integration. It help to have them as extra projects because some people probably only need the core part.

Yeah let's see how it goes.

First of all, thanks for the port of this project to .NET!
I am interested in the separation of the project also as way to handle version dependencies. The current dependency on a fix version can lead to problems when my application uses a difference library versions. In my case I cannot run the currently nuget package because the project I am working uses Polly 7+ (throws a runtime error). The same might apply to StackOverflow.Redis, MySql etc.
A possible way to deal with this problem is to depend on version ranges.

I would like to work on the split up. I have currently the same problems polly.


Support for critical alerts

I added support for critical alerts.
Also made some formatting changes.
Feel free to accept or deny the PR.
I know #646 is already 'approved' but not merged, but this edit also adds the interface in typescript.

Any news on this?


How to update chrono.js?

I'd like min/mins to be parsed as minutes.
Wondering if this script is still using an older chrono.js, where that file is, or how to update it.

Updated to 1.4.8 which should address this c65a95ae23184772d12cc29b3629a9d268abe675


Deployment failure - Source controls conflict

Same issue as has been reported previously, it's causing all deployments to multiple regions to fail completely when synchronising the app service.
{
"status": "Failed",
"error": {
"code": "ResourceDeploymentFailure",
"message": "The resource operation completed with terminal provisioning state 'Failed'."
}
}
Attempting to resync, or disconnect and reconnect then resync also fails.

Deployment fails with the same issue. Please find the error below.
{   "code": "DeploymentFailed",   "message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.",   "details": [     {       "code": "Conflict",       "message": "{\r\n  \"status\": \"Failed\",\r\n  \"error\": {\r\n    \"code\": \"ResourceDeploymentFailure\",\r\n    \"message\": \"The resource operation completed with terminal provisioning state 'Failed'.\"\r\n  }\r\n}"     }   ] }

I have the same error as indicated above during deployment.  Deployment fails.

Any update on this ? It is holding us back from deploying :(


Make tab environment variables configurable

Add configurable environment variables to workspace & repository tabs.

Implemented in #179, and documented in #182.

Closing, v0.4.0 is released!


Integrating StyleSystem Classes for Component Wrapper div tags in SPA Editor - React Impl

Description
Changes done to EditableComponent to support StyleSystem based on appliedCssClassNames property from Sling Model exporters. Please refer to related issue for more details.
Related Issue
Here is the related issue requesting this feature. #68
Motivation and Context
These changes extend the style system capability/feature to SPA Editor implementation for React apps.
How Has This Been Tested?
Changes done locally and deployed to AEM instance via a brand new project to confirm that stysystem works as expected. Related issue #68 has screenshots of these tests. Aditionally unit tests are also included to validate this behavior.
Screenshots (if appropriate):
Refer to related issue.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Additional Notes
If this PR is approved/merged, the page below requires updates against limitations section where we now can support StyleSystem. Additionally we need to include documentation for using style system with examples either in the next archetype or in WKND events sample project. Additionally we need to capture the need for registering the property appliedCssClassNames against any custom sling model that needs style system capability.

@vladbailescu , @sharanyavinod , @godanny86 - Requesting your review and feedback. Related issue has all details you may need for reviewing this PR. Thank you so much !!

LGTM. Actually really smart to make it work across all components in 1 swoop.
Just curious, is the style system supposed to have this css class applied on the parent and not the same component? I can't say I actually used the style system so can't tell.
Seems to work fine though.

LGTM. Actually really smart to make it work across all components in 1 swoop.
Just curious, is the style system supposed to have this css class applied on the parent and not the same component? I can't say I actually used the style system so can't tell.
Seems to work fine though.

Great question. A while back me and @vladbailescu had a conversation on the same.
When you adjust a layout of a component, the layout info is indeed stored at the component level. However this data (columnClassNames) is exported as part of the parent responsive grid and not the individual comp as the class info makes sense only in the context of responsive grid. And when rendering HTL, data-sly-resource implementation is done in such a way that both layout classes and applied at the parent level.
For actual components, the idea is the applied classes should be exported within the same json exporter and not the parent component, as these classes have no relation with layout classes. And the implementation is done accordingly in WCM Core Components git repo. And why is this important - coz the child component wil re-render with new styles only if the incoming props are updated. Including the style classes in parent component may force a reload on the parent container which perhaps is not the best experience.

Well I lack rights to merge / review, but you have my seal of approval :)

@habansal , @CezCz , @sharanyavinod - Please feel free to add your comments and advise on the next course of actions. Thank you !

@CezCz - Thank you so much for approving the PR. Can you please advise on the next steps? How we can get this merged and included into the next version, and additionally updates to SPA Editor documentation as well as CHANGELOG on support for this feature.

@friendlymahi thanks for the contribution. We have a semantic release bot running, which releases things automatically. I forgot to format your commit appropriately and just committed an empty one. It should trigger the release which and changelog and so on.
I'd be grateful if you created an issue for angular editable components.

@CezCz - As requested, created an issue # 60 for Angular SPAs to support StyleSystem. I never worked on Angular to contribute over there, probably need other volunteers/contributors.
@niekraaijmakers

@friendlymahi thanks for the contribution. We have a semantic release bot running, which releases things automatically. I forgot to format your commit appropriately and just committed an empty one. It should trigger the release which and changelog and so on.
I'd be grateful if you created an issue for angular editable components.

Thanks for your response. If there are no more checks, please close the issue # 68 - [feature] StyleSystem Impl for React SPAs that leverage SPA Editor


feat(filecoin): implement deal expiration

This PR implements Deal Expiration (the act of changing a deal's state from Active to Expired after the specified duration)

I got approval from @tcoulter on Slack to merge this without his review (I also got @davidmurdoch's approval to merge without Tim's approval).
Tests fail in this PR due to not having fixes from #877 yet, but I'm going to apply fixes for that in a new PR and merge this for now to be done with it


Flare info

@bmino do we have related prs for components & interface side

we have just re-deployed Flare contracts to include some more recent changes. I'll update shortly. we will need accompanying PRs which bump the sdk version to utilize this after release


[Ahuna] New way of getting agent configurations (re-revert w/fixes)

Prevent non-private agents from becoming private
wip
wip2
changing getAgents calls + api calls
fixes
cleaning
spolu coms 1
migrate types for route
no-revert-to-private check in business code rather than model code
prevent auth from non-workspace user
new view names + relationOverride in agconftype
refactor getAgentConfiguratiosn
rebase fix
fix
generation issue
path
cleaning
cleaning + super_user
cleaning
duplicate exports
rebase from aric's optim

Old query had outer joins, returned all mentions
SELECT "mention"."id", "mention"."agentConfigurationId" FROM "mentions" AS "mention" LEFT OUTER JOIN ( "messages" AS "message" INNER JOIN "conversations" AS "message->conversation" ON "message"."conversationId" = "message->conversation"."id" AND "message->conversation"."sId" = '9c81201e61' ) ON "mention"."messageId" = "message"."id" WHERE "mention"."agentConfigurationId" IS NOT NULL;

New query with inner joins return only correct mentions:
SELECT "mention"."id", "mention"."agentConfigurationId" FROM "mentions" AS "mention" INNER JOIN "messages" AS "message" ON "mention"."messageId" = "message"."id" INNER JOIN "conversations" AS "message->conversation" ON "message"."conversationId" = "message->conversation"."id" AND "message->conversation"."sId" = 'ec1e367093' WHERE "mention"."agentConfigurationId" IS NOT NULL;

Following up on this work:

given incident this am + new optims of getAgentconf which changed the main getAgentConfigurations with views a bit, it's worth a new ptal
would have liked to merge sooner so you can pick up this afternoon, I regret that it turned out to take that long
ready to merge tomorrow morning & check on prod health for a while (or later of course, when good for you); I will be OOO monday and don't want to remain a blocker here)

To simplify the review, only 3 functions to look at, all in lib/api/assistant/configuration.ts

getAgentsConfiguration (with aric's optim + //ized call to global agent)
getConversationMention & comment explaining it's tested well this time (in prod)
createAgentconfiguration with the reduction to 1 sql call

Cheers !


fix hidden path separators

Show hidden path separators.
It's fine for plain text, but it was hidden in Markdown.

Hi @MicrosoftHeidi, Thanks for reviewing my pull request.
Any issues with the changes? Is there anything else I need to do on my end?

Hi @MicrosoftHeidi, the public branch was updated while you were reviewing my pull request, so I updated my commits too. Could you please check again?


iOS App consistently blacks out and UI disappear when green checkmark double clicked in the edit mode.

-Login to iOS app and go to Containers or terminals.
-Click edit on any terminal.
-click into the name of the terminal.
-now double click the green arrow
Observed. Application goes into black mode and never returns.  Thanks


is docs.pgvecto.rs gone?

Links in the postgres log files and in README.md point to docs.pgvecto.rs but this site isn't found. I found the docs on docs.vectorchord.ai instead. Do these references need to be updated?

We accidentally missed the domain renewal and are currently trying to recover it. All documents have temporarily been moved to the vectorchord.ai domain. Sorry for the inconvenience.


Interpolation in attribute values

According to this issue https://github.com/pugjs/pug/issues/2302 the old way of concatenating strings and variables won't work anymore. I was hoping that you would be willing to update this. Thank you!

@ifasina I think its fixed now in 1.0.11 please check and let me know for any issues


Add support for non-standard file extensions

Hi, recently in coffeelint added support for non-standard file extensions.
Is it possible to add support for custom file extensions in the plugin?
https://github.com/clutchski/coffeelint/pull/480

:+1:

+1

Thank you, its work! :smiley:


Frenzy charges doesn't give traps throwing speed with Slavedriver's Hand

Haven't tried with Architect's Hand.

Already fixed in the next update.


Tags Page UI issues

Affected module
Does it impact the UI, backend or Ingestion Framework?
UI
Describe the bug
A clear and concise description of what the bug is.

Button overflow issue for an in-markdown box of description
Unwanted API call is triggered on the Add and Edit cases if we close the modal and data disappears in form while saving the changes
when hit the delete button in the table, the delete button shows loading.

To Reproduce
Screenshots or steps to reproduce

https://github.com/open-metadata/OpenMetadata/assets/66266464/9a82650c-ab9d-4e49-bd22-fb2e3fa9b4bc
https://github.com/open-metadata/OpenMetadata/assets/66266464/202ea455-2c3a-493c-9e52-afd91bee77c9
Expected behavior
A clear and concise description of what you expected to happen.
Version:

OS: [e.g. iOS]
Python version:
OpenMetadata version: [e.g. 0.8]
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

I will take this up


feat: added support for eth_getTransactionReceipt() to the WS server (#2274)

Description:

adds support and test coverage for eth_getTransactionReceipt(txHash) endpoint to the websocket server.
slightly modify utils functions in ws-server package to better suit the new API
removed socketId in logs as dauplicated with connectionID

Related issue(s):
Fixes #2274
Notes for reviewer:

Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

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

Project coverage is 75.15%.  Consider uploading reports for the commit bcbf4c6 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2297   +/-   ##
=======================================
  Coverage   75.15%   75.15%
=======================================
  Files          13       13
  Lines         644      644
  Branches      118      118
=======================================
  Hits          484      484
  Misses        115      115
  Partials       45       45

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


Cast parsing int types as uint

Component
Cast
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (fc534d9 2022-05-23T00:04:42.704454Z)
What command(s) is the bug in?
cast call
Operating System
macOS (Apple Silicon)
Describe the bug
cast appears to be parsing int types as uint types. A minimal example:
pragma solidity ^0.8.0;

contract MyInt {
    constructor() {}

    function getInt() public pure returns(int256) {
        return -1;
    }
}

❯ cast call 0x50D75C1BC6a1cE35002C9f92D0AF4B3684aa6B74 "getInt()(int256)"
115792089237316195423570985008687907853269984665640564039457584007913129639935

Going to add on that in general ints seem to be poorly supported in foundry and ethers-rs too. Related:

https://github.com/foundry-rs/foundry/issues/29#issuecomment-1015480601
https://github.com/gakonst/ethers-rs/issues/842
There's also some issues with them in forge, e.g. if you run RUST_LOG=foundry_evm=trace forge test, ints get logged with the unsigned value (same as shown here) instead of the two's complement representation

Related to https://github.com/rust-ethereum/ethabi/issues/249


GitHub bug: GitHub Actions logs add scrolling bar to each line separately

Description
This is only visible if your OS does not hide the scrollbars
Screenshot

Example URL
https://github.com/pixiebrix/pixiebrix-extension/runs/4421465201?check_suite_focus=true

This could probably be fixed by dropping the fixed width that each line might have and then adding overflow-x: scroll to the whole log section (step)

That's unfortunate, I thought GitHub would wrap in this case.

Reported to GitHub for now

Fixed via CSS hotfix just to try it, but it doesn't appear to be working https://github.com/refined-github/refined-github/issues/5165

Still no fix. I closed this issue on an excess of confidence that they would have fixed it quickly. Nope.

GitHub does wrap on that page. What's the difference?

We have a hotfix, disable RG

We have a hotfix, disable RG and try safari

That's without RG, but indeed not Safari (it's Chrome).
On Safari with overflow: visible things do go back to normal, except the page itself now has a horizontal scrollbar.

Still no fix. I closed this issue on an excess of confidence that they would have fixed it quickly. Nope.

Maybe they are mostly using Chrome or have scrollbar hidden 🤷


TSCBasic: remove close overload in extension

The FileHandle extension added an overloaded close that causes
ambiguity when using close on a FileHandle.  This manifested as a
build failure with apple/swift-package-manager#3888.  Remove the
function as swift-driver, swift-package-manager, and sourcekit-lsp are
able to build without this function (relying on Foundation to provide
it).

@swift-ci please test

CC: @abertelrud

I just realized what happened here.  These were old covers that are no longer needed now that Foundation has refined its FileHandle API in Swift 5.0.
This was https://github.com/apple/swift-corelibs-foundation/commit/55c404ba3b7edac0f9712d5556bcd8f1d404e935

That means that if this causes problems, I can also switch SwiftPM to use the deprecated closeFile() call.  But fixing forward is better.

@compnerd Looks ready to go at this point?


Error for slash and apostrophes in export: / ' '

If the title (or any other field as far as I can tell) contains the combination of characters "/ ' '" (without leading and trailing double-quotes, possibly with other characters interleaved) the export (BibTeX and BibLaTeX) fails with: "An error occurred while trying to export the selected file."
As a side note: I tried to "Send Better BibTeX Error" (now apparently called "Report Better BibTeX Error") but the interface froze for minutes after clicking "Next". Much later the message "TypeError: wizard.advance is not a function" popped up – I don't know if this is of any help/relevance.
I'm on Firefox 49.0.2, Better Bib(La)TeX 1.6.94

Can you export such a reference to "BetterBibTeX JSON format", rename the resulting file to have a .txt extension, and attach it here?

Wow, that's fast :) Here you go:
items.txt

OK, that reproduces the bug. Looking into it.

OK, it looks like the problem happens deep in citeproc, which is either a bug in citeproc (BBT has been know to exercise citeproc in novel ways) or an inappropriate way of calling citeproc. The discussion on that will be happening here. I have a proposed workaround in there which is implemented in BBT here -- it passes the one test case you gave me and all my existing tests, and I'm curious how it will do on the rest it failed on for you.

Thanks for looking into this! Actually, there was only one Item that caused problems (containing something like '98/'99 in the proceedings title) even though it took a while to spot it and the weird thing is that I'm pretty sure I didn't edit that Item for a long time. I think the problem occurred after a recent update (I have auto-updating enabled). I know I'm probably not beeing very helpful here but this is a good sign: I've been using better bibtex happily and free from worries – very helpful piece of software, indeed! Thanks a lot!

It's very helpful -- you found an edge case that I wouldn't have found myself. If I were a betting man I'd say it's an edge case in citeproc actually, but only Frank (citeproc's author) can confirm that. The place where the error happens is a dark mystery to me, but the workaround seems to do its thing safely, at least according to my regression tests. The recent update also picked up a new citeproc, which looks like introduced this problem. The pre-release I linked to should be safe to use (it's just the release + the workaround), and you'll be auto-updated to the proper release when it goes out automatically.
Please don't close this yet, I want to know what Frank thinks.

Too many people are going to be affected by this, so I'm going to schedule a release including this workaround. When Frank chimes in I may adjust the workaround, but we'll see about that when it happens.


Track escrow in recipient's owner directory (RIPD-1523):

Introduce "fix1523" which corrects a minor technical flaw with
the original implementation of the escrow feature.
When creating an escrow, the entry would only be tracked in the
owner directory of the sender; as a result, an escrow recipient
would not be able to detect incoming escrows without monitoring
the ledger in real-time for transactions of interest or without
the sender communicating this information out of band.
With the fix in place, escrows where the recipient differs from
the sender will be listed in the recipient's owner directory as
well.

Looks like one of the unit tests is failing.  But the unit test output is a bit too quiet, so I can't tell which of the tests is failing...

pretty sure that the failure is #65 failed: AmendmentTable_test.cpp(744)....which I think is failing because the sha512_half hash is wrong in the supportedAmendments (as @bachase pointed out)

Codecov Report

Merging #2212 into develop will increase coverage by 0.01%.
The diff coverage is 90.62%.

@@             Coverage Diff             @@
##           develop    #2212      +/-   ##
===========================================
+ Coverage    70.05%   70.07%   +0.01%
===========================================
  Files          689      689
  Lines        50724    50729       +5
===========================================
+ Hits         35533    35546      +13
+ Misses       15191    15183       -8

Impacted Files
Coverage Δ

src/ripple/ledger/Directory.h
100% <ø> (ø)
:arrow_up:

src/ripple/app/main/Amendments.cpp
100% <ø> (ø)
:arrow_up:

src/ripple/protocol/Feature.h
100% <ø> (ø)
:arrow_up:

src/ripple/protocol/SField.h
100% <ø> (ø)
:arrow_up:

src/ripple/ledger/impl/Directory.cpp
85.71% <ø> (+23.02%)
:arrow_up:

src/ripple/protocol/impl/SField.cpp
91.78% <100%> (+0.03%)
:arrow_up:

src/ripple/protocol/impl/LedgerFormats.cpp
100% <100%> (ø)
:arrow_up:

src/ripple/protocol/impl/Feature.cpp
94.54% <100%> (+0.1%)
:arrow_up:

src/ripple/app/tx/impl/Escrow.cpp
90.65% <85%> (-0.59%)
:arrow_down:

src/ripple/basics/DecayingSample.h
77.77% <0%> (-8.34%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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

Rebased and squashed. For inclusion into 0.80.0.

Merged as c7c1b3cc3b7547504ae4d91ff6c41af18c3f264b


Setup test environment

Description

Setup test environment using jest
Setup github workflow for test
Add github action for test

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


Upgrade to Angular14 and latest packages

Step by step upgrade to Angular 14. Then replaced deprecated ComponentFactoryResolver with new implementation.

You can merge when ready @michaelmarcuccio

I do not have write access to the repo.

@joel-wenzel the github CI workflow needs to be updated with a higher node.js version.

Node.js version v12.22.12 detected.
The Angular CLI requires a minimum Node.js version of either v14.15, or v16.10.


Test and Confirmed cases data missing from covid-19-uk-data 19 May

Yesterdays (19 May) Tests and confirmedCases data is missing from file covid-19-uk-data. Sorry if this is because the source data wasn't published by UK Govt.

Yes, this is because those numbers were not available yesterday. They have been published today (20 May).

Will we ever get the numbers for 19th do you think? (Your csv and raw data files are still showing a gap for the 19th.)

Historical testing data is not being published (yet), so there will be a gap until PHE/DHSC start publishing a complete dataset I'm afraid.

Hi Tom,
Considering data from May19 onwards, the variables totalpositive and dailypositive in the file 'covid-19-tests-uk.csv' are not consistent. In fact, the variable totalpositive is lower in May20 than May19.
When you have time, could you please look into it?
Many thanks!
Giulia

Hello Giulia,
The consistency problem between DailyPositive and the TotalPositive is long lasting.
As you can see in the picture here below, the sum of daily and yesterday's total isn't always equals to today's total (red on column I).
Column J is the cumulative sum of DailyPositive since 2020-04-08: it seems they tried to fix the difference and this causes the inconsistency you noticed.

Mny thanks for your help!
Giulia


Empty OnRequest() Passes No Requests

Given the following code:
package main

import (
	"fmt"
	"log"
	"net/http"

	"github.com/elazarl/goproxy"
)

func main() {
	proxy := goproxy.NewProxyHttpServer()

	proxy.OnRequest().DoFunc(func(r *http.Request, ctx *goproxy.ProxyCtx) (*http.Request, *http.Response) {
		fmt.Println("Incoming:", r.URL.Hostname())
		return r, nil
	})

	log.Fatal(http.ListenAndServe(":8080", proxy))
}

Using this as an HTTP proxy does nothing. In other words, nothing is ever logged. It seems that when OnRequest has no conditions then it skips the DoFunc.
If you add conditions to OnRequest then it works as expected.

The problem here was that I wasn't using HTTPS MITM, so it wasn't intercepting HTTPS requests.


can't compile tensorflow from source

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): linux ubuntu 20.04
TensorFlow installed from (source or binary): source
TensorFlow version: 1.14 and 1.13
Python version: 3.8
Installed using virtualenv? pip? conda?: pip
Bazel version (if compiling from source): 0.24.1
GCC/Compiler version (if compiling from source): gcc-4.8.5 for bazel and gcc-8 for cuda libary
CUDA/cuDNN version: 10.1 / 7
GPU model and memory: 1070 ti 8gb

Describe the problem
I can't compile from tensorflow 1.14-1.13 source code (haven't tried other 1 tensorflow branches) and 2.4 has been successfully compiled with and without GPU. Originally the error was with newer versions of basel and gcc-8, but after reading the https://www.tensorflow.org/install/source table, I installed gcc-4.8, but the error persisted.
I have read the solutions
https://github.com/tensorflow/serving/issues/928 but adding the --cxxopt = -std = c ++ 11 flag didn't work for me
I also read that installing libc-ares-dev library might help, but that didn't help either
I thought the problem might be related to the fact that I am using cuda-10.1 and not 10.0 as in the table. I tried to set up the file without cuda, but I got exactly the same error, so I think this is not the problem
Provide the exact sequence of commands / steps that you executed before running into the problem
writte config file
WARNING: --batch mode is deprecated. Please instead explicitly shut down your Bazel server using the command "bazel shutdown".
You have bazel 0.24.1 installed.
Please specify the location of python. [Default is /usr/bin/python3]:
Found possible Python library paths:
/usr/local/lib/python3.8/dist-packages
/usr/lib/python3/dist-packages
Please input the desired Python library path to use.  Default is [/usr/local/lib/python3.8/dist-packages]
Do you wish to build TensorFlow with XLA JIT support? [Y/n]: n
No XLA JIT support will be enabled for TensorFlow.
Do you wish to build TensorFlow with OpenCL SYCL support? [y/N]: n
No OpenCL SYCL support will be enabled for TensorFlow.
Do you wish to build TensorFlow with ROCm support? [y/N]: n
No ROCm support will be enabled for TensorFlow.
Do you wish to build TensorFlow with CUDA support? [y/N]: y
CUDA support will be enabled for TensorFlow.
Do you wish to build TensorFlow with TensorRT support? [y/N]: n
No TensorRT support will be enabled for TensorFlow.
Found CUDA 10.1 in:
/usr/local/cuda/lib64
/usr/local/cuda/include
Found cuDNN 7 in:
/usr/lib/x86_64-linux-gnu
/usr/include
Please specify a list of comma-separated CUDA compute capabilities you want to build with.
You can find the compute capability of your device at: https://developer.nvidia.com/cuda-gpus.
Please note that each additional compute capability significantly increases your build time and binary size, and that TensorFlow only supports compute capabilities >= 3.5 [Default is: 6.1]:
Do you want to use clang as CUDA compiler? [y/N]: n
nvcc will be used as CUDA compiler.
Please specify which gcc should be used by nvcc as the host compiler. [Default is /usr/bin/gcc]: /usr/bin/gcc-8
Do you wish to build TensorFlow with MPI support? [y/N]: n
No MPI support will be enabled for TensorFlow.
Please specify optimization flags to use during compilation when bazel option "--config=opt" is specified [Default is -march=native -Wno-sign-compare]: -march=native -mno-avx
Would you like to interactively configure ./WORKSPACE for Android builds? [y/N]: n
Not configuring the WORKSPACE for Android builds.
Preconfigured Bazel build configs. You can use any of the below by adding "--config=<>" to your build command. See .bazelrc for more details.
--config=mkl            # Build with MKL support.
--config=monolithic     # Config for mostly static monolithic build.
--config=gdr            # Build with GDR support.
--config=verbs          # Build with libverbs support.
--config=ngraph         # Build with Intel nGraph support.
--config=numa           # Build with NUMA support.
--config=dynamic_kernels        # (Experimental) Build kernels into separate shared objects.
Preconfigured Bazel build configs to DISABLE default on features:
--config=noaws          # Disable AWS S3 filesystem support.
--config=nogcp          # Disable GCP support.
--config=nohdfs         # Disable HDFS support.
--config=noignite       # Disable Apache Ignite support.
--config=nokafka        # Disable Apache Kafka support.
--config=nonccl         # Disable NVIDIA NCCL support.
Configuration finished
build command
bazel build --cxxopt=-std=c++11 --config=cuda  //tensorflow/tools/pip_package:build_pip_package
output error
INFO: From ProtoCompile external/com_github_googleapis_googleapis/google/api/http.pb.h:
bazel-out/k8-opt/genfiles/external/protobuf_archive/src: warning: directory does not exist.
INFO: From ProtoCompile external/com_github_googleapis_googleapis/google/rpc/status.pb.h:
bazel-out/k8-opt/genfiles/external/protobuf_archive/src: warning: directory does not exist.
INFO: From Compiling external/mkl_dnn/src/cpu/simple_concat.cpp:
external/mkl_dnn/src/cpu/simple_concat.cpp:98: warning: ignoring #pragma omp simd [-Wunknown-pragmas]
PRAGMA_OMP_SIMD()
INFO: From Compiling external/grpc/src/core/ext/transport/chttp2/transport/chttp2_transport.cc:
external/grpc/src/core/ext/transport/chttp2/transport/chttp2_transport.cc: In function 'grpc_error* try_http_parsing(grpc_chttp2_transport*)':
external/grpc/src/core/ext/transport/chttp2/transport/chttp2_transport.cc:2466:40: warning: 'void* memset(void*, int, size_t)' clearing an object of non-trivial type 'grpc_http_response' {aka 'struct grpc_http_response'}; use assignment or value-initialization instead [-Wclass-memaccess]
memset(&response, 0, sizeof(response));
^
In file included from external/grpc/src/core/ext/transport/chttp2/transport/chttp2_transport.cc:44:
external/grpc/src/core/lib/http/parser.h:71:16: note: 'grpc_http_response' {aka 'struct grpc_http_response'} declared here
typedef struct grpc_http_response {
^~~~~~~~~~~~~~~~~~
ERROR: /home/dmitry/.cache/bazel/_bazel_dmitry/1319b3f3ad6252a51422db144992f79d/external/grpc/BUILD:507:1: C++ compilation of rule '@grpc//:gpr_base' failed (Exit 1)
external/grpc/src/core/lib/gpr/log_linux.cc:43:13: error: ambiguating new declaration of 'long int gettid()'
static long gettid(void) { return syscall(__NR_gettid); }
^~~~~~
In file included from /usr/include/unistd.h:1170,
from external/grpc/src/core/lib/gpr/log_linux.cc:41:
/usr/include/x86_64-linux-gnu/bits/unistd_ext.h:34:16: note: old declaration '__pid_t gettid()'
extern __pid_t gettid (void) __THROW;
^~~~~~
external/grpc/src/core/lib/gpr/log_linux.cc:43:13: warning: 'long int gettid()' defined but not used [-Wunused-function]
static long gettid(void) { return syscall(__NR_gettid); }
^~~~~~
Target //tensorflow/tools/pip_package:build_pip_package failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 281.090s, Critical Path: 57.54s
INFO: 1539 processes: 1539 local.
FAILED: Build did NOT complete successfully
(there was even more INFO output, but I did not copy everything, there really is a lot)

@Int37
Can you please refer the tested build configurations from here and see if the problem still persists with tested build configurations.
Can you upgrade to TF 2.x more recent versions for better performance. Thanks!

@ ravikyram, thanks for your answer, i installed python 3.7 but the problem persists. I already have everything suitable for tested build configuration, except perhaps except for cuda and cudnn, but as I wrote above, the problem remains even if i try to compile without cuda. I also tried to do what is written here https://github.com/tensorflow/tensorflow/issues/33758#issuecomment-547143723, but this did not fix my error.
I already have tensorflow 2.4 compiled, but I would also like to compile version 1.14 in order to use projects that ask for tensorflow 1.14 or higher

@Int37
TF 1.x is not actively supported.In this case please post the issue in stack overflow. . Thanks!


[BUG]: Executing more than one LLMEngine causes Morpheus to hang

Version
23.11
Which installation method(s) does this occur on?
Source
Describe the bug.
Executing more than one LLMEngine causes morpheus to hang.
This was first noticed while executing unittests as multiple tests will execute an LLMEngine.
Minimum reproducible example
import logging

import cudf

from morpheus.config import Config
from morpheus.config import CppConfig
from morpheus.llm import LLMEngine
from morpheus.llm.nodes.extracter_node import ExtracterNode
from morpheus.llm.task_handlers.simple_task_handler import SimpleTaskHandler
from morpheus.messages import ControlMessage
from morpheus.pipeline.linear_pipeline import LinearPipeline
from morpheus.stages.input.in_memory_source_stage import InMemorySourceStage
from morpheus.stages.llm.llm_engine_stage import LLMEngineStage
from morpheus.stages.output.in_memory_sink_stage import InMemorySinkStage
from morpheus.stages.preprocess.deserialize_stage import DeserializeStage
from morpheus.utils.concat_df import concat_dataframes
from morpheus.utils.logger import configure_logging

def _build_engine() -> LLMEngine:
    engine = LLMEngine()
    engine.add_node("extracter", node=ExtracterNode())
    engine.add_task_handler(inputs=["/extracter"], handler=SimpleTaskHandler())

    return engine

def run_pipeline():
    config = Config()
    CppConfig.set_should_use_cpp(False)
    insects = ["ant", "bee", "butterfly", "mosquito", "grasshopper"]
    mammals = ["lion", "dolphin", "gorilla", "wolf", "tiger"]
    reptiles = ['lizards', 'snakes', 'turtles', 'frogs', 'toads']
    df = cudf.DataFrame({"insects": insects, "mammals": mammals, "reptiles": reptiles})

    task_payload = {"task_type": "llm_engine", "task_dict": {"input_keys": ['mammals']}}

    pipe = LinearPipeline(config)
    pipe.set_source(InMemorySourceStage(config, dataframes=[df]))
    pipe.add_stage(
        DeserializeStage(config, message_type=ControlMessage, task_type="llm_engine", task_payload=task_payload))
    pipe.add_stage(LLMEngineStage(config, engine=_build_engine()))
    sink = pipe.add_stage(InMemorySinkStage(config))

    pipe.run()

    messages = sink.get_messages()
    responses = concat_dataframes(messages)
    print(f"Pipeline complete. Received {len(messages)} responses\n{responses['response']}")

if __name__ == "__main__":
    configure_logging(logging.DEBUG)

    iteration_num = 0
    while True:
        iteration_num += 1
        print(f"\n***********\nStarting iteration: {iteration_num}\n***********\n")
        run_pipeline()

Relevant log output
Click here to see error details
[Paste the error here, it will be hidden by default]

Full env printout
Click here to see environment details
[Paste the results of print_env.sh here, it will be hidden by default]

Other/Misc.
No response
Code of Conduct

[X] I agree to follow Morpheus' Code of Conduct
[X] I have searched the open bugs and have found no duplicates for this bug report

Executing the engine directly outside of a pipeline via asyncio.run does not hang.


Microsoft Word Add-in v3 Release, do RPA using Microsoft Word as IDE - done

My team-mate @ruthtxh has created an add-in that lets anyone do RPA with Microsoft Word.
Try it out and let us know! - https://github.com/kelaberetiv/TagUI/tree/master/src/office

v3 ideas to review for implementation

[x] text area in task pane for run output
[x] browse for datatable file & sheet
[x] take image snapshot as identifiers
[x] consider adding snapshot button
[x] discuss how to handle subflows?
[x] rename setup.exe --> Setup.exe
[x] toolbar for steps and functions
[x] popup tooltip to show purpose
[x] placeholder title to elaborate
[x] metadata to store run options

Adding sub-flows support for discussion before I forget

Consider snippets - feedback from Kevin
https://code.visualstudio.com/docs/editor/userdefinedsnippets

This is v3.0.4, Ruth will be doing the finishing touches before we publish later this week - TagUIWordAddInSetupV3.04.zip

Implemented and available from https://github.com/kelaberetiv/TagUI/blob/master/src/office/README.md

Improvements for next release:

[] improve snapshot tool for common select tag
[] auto check for indentation and convert to tabs

Source: telegram chat
Datatable selection for .xlsx sheet not saved in taskpane upon closing/reopening of word document. To check metadata.

Hi @ruthtxh I'm closing some issues on features / bugs which have already made their way into the packaged release.
For this issue I didn't close because I'm not sure if you still have outstanding action to complete before closing.

Closing but please ping back if this is still relevant or an issue.


Clean irrelevant puzzle

Clean irrelevant puzzle for #281

@olenagerasimova/z this pull request is too small, just 9 lines changed (less than 10), there will be no formal code review, see §53 and §28; in the future, try to make sure your pull requests are not too small; @g4s8/z please review this and merge or reject

Job gh:artipie/rpm-adapter#288 is not assigned, can't get performer


Use ctrlruntimeclient for migrations

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
none

/retest

/retest

/assign @mrIncompetent

/lgtm
/approve


add moviesnation for free american and korean shows

This site helps you to download the latest American and Korean movies and shows

why did you add  .idea in .gitignore file

it is because I use JetBrains Webstorm for development and webstorm is used to make this (.idea) file automatically if i push without adding it to the .gitignore file then the main branch has this file which is of no use
so to ignore this file I put it on the .gitignore file


Update QUICHE to 1080468

Risk Level: Low
Testing: Unit Tests
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A
Signed-off-by: Yan Avlasov<EMAIL_ADDRESS>

/lgtm deps

deprecated by https://github.com/envoyproxy/envoy/pull/25394 ?


Convert NBT tags to similar types

e.g.

Int -> Byte
List -> IntArray

The conversion between numbers have been finished. Conversion between collections won't be implemented.


[AutoRelease] t2-notificationhubs-2022-11-11-97598(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3402
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1017

issue link:https://github.com/Azure/sdk-release-request/issues/3402


Revert "Splunk: Optionally emit a counter, histogram, or nothing with get_json"

Reverts stripe/datadog-checks#85
More work needed: the check never emits error metrics because it fails at the auth stage (which is not actually a step that's necessarily needed!), and this stage can't be distinguished at a per-intent level. I have a solution, but uncertain if it's the one we want.

I guess this doesn't hurt anything -- it doesn't exactly fail, just doesn't do what we want

👍


stable toolchain works

Hm, is it not better to just remove the file?

While there's no specific reason we can't use stable, the compiler uses the beta channel, and this is why we also have been using that. I believe it is because using that same version to avoid possible problems when updating rustc to newer polonius revisions.
That being said, maybe I'm misremembering, and Niko may remember reasons to keep using beta, or the opposite, approve of switching to stable.

What compiler is meant in "the compiler uses the beta channel"? Rust compiler? I also do not understand "using that same version could avoid possible problems when updating rustc to newer polonius revisions".

Ah exactly yes, the Rust compiler: this project is made specifically to be used used in rustc, in the librustc and librustc_mir sub-crates. (Mainly the latter)

Ok, what do you mean by "the compiler uses the beta channel"?

A nightly version of rust compiler is compiled using the latest beta version at that time. A beta version is compiled using the latest stable. A stable version is compiled using the previous stable version.

rustc is built using bootstrapping: newer versions are built using an older version of itself, and the version it uses is the beta toolchain, the same as we use here today. There are more details in the rustc guide here about the different stages.

interesting to know, thanks


[Enhancement] Deleting old review and confirm code

[Enhancement] Deleting old review and confirm code

Se elimina código que no se usa más de Revisa y Confirma y Reviewables.
Se hizo un Code Cleanup automático con el plugin de Code Style de Meli.

Codecov Report

Merging #921 into feature/review_and_confirm will increase coverage by 0.71%.
The diff coverage is 31.86%.

@@                      Coverage Diff                       @@
##           feature/review_and_confirm     #921      +/-   ##
==============================================================
+ Coverage                       19.54%   20.25%   +0.71%
==============================================================
  Files                             368      358      -10
  Lines                           18028    17350     -678
  Branches                         1613     1574      -39
==============================================================
- Hits                             3524     3515       -9
+ Misses                          14026    13361     -665
+ Partials                          478      474       -4

Impacted Files
Coverage Δ

...o/review_and_confirm/ReviewAndConfirmActivity.java
0% <ø> (ø)
:arrow_up:

sdk/src/main/java/com/mercadopago/model/Token.java
16.66% <ø> (ø)
:arrow_up:

...cadopago/providers/PaymentMethodsProviderImpl.java
0% <ø> (ø)
:arrow_up:

.../java/com/mercadopago/model/SavedESCCardToken.java
57.14% <ø> (ø)
:arrow_up:

...ava/com/mercadopago/adapters/BankDealsAdapter.java
0% <ø> (ø)
:arrow_up:

...go/plugins/components/BusinessPaymentRenderer.java
0% <ø> (ø)
:arrow_up:

...a/com/mercadopago/components/ComponentManager.java
0% <ø> (ø)
:arrow_up:

...in/java/com/mercadopago/model/PaymentSettings.java
0% <ø> (ø)
:arrow_up:

...k/src/main/java/com/mercadopago/util/JsonUtil.java
100% <ø> (ø)
:arrow_up:

...d_confirm/components/ReviewAndConfirmRenderer.java
0% <ø> (ø)
:arrow_up:

... and 184 more

Continue to review full report at Codecov.

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

@zehemz dale perfecto


IHubClient : using [HubMethodName] #SignalR  #Feature

Add using of [HubMethodName] for client proxy
//TypeClientBuilder
//  private static void BuildMethod(TypeBuilder type, MethodInfo interfaceMethodInfo, FieldInfo proxyField)
// The first argument to IClientProxy.SendCoreAsync is this method's name
          generator.Emit(OpCodes.Ldstr, interfaceMethodInfo.Name);

Thank you.

I'm not sure why we would want to do this. The [HubMethodName] attribute is for the name of hub methods that the client invokes.
interfaceMethodInfo.Name is the name of methods you call on the client.

You can use the new attribute with a suitable name.

This isn't something we intend to do for 3.0.

How to make SignalR ignore specific method in a Hub (not expose it to JS)? Is there any attribute supported for that purpose?

I don't believe there is an attribute for that at this point, you can use a private or internal method though, which won't be exposed. Is there a scenario you have in which you want to have a public Hub method that is not exposed to the client?

Is there a scenario you have in which you want to have a public Hub method that is not exposed to the client?

@anurse It looks I'm not, but though my scenario is simple it involves clunky code. I want to broadcast the data on timely manner (every minute), but for clients who had subscribed for these broadcasts, I want to call their callback method the moment they subscribed with the cached data from previous broadcast. Here's the implementation
using Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.SignalR;
using System;
using System.Collections.Concurrent;
using System.Collections.Generic;
using System.ComponentModel.DataAnnotations;
using System.Threading.Tasks;

namespace CoreApp.Services.Hubs {

    // Methods to be called on a client from server side
    public interface IInfoHubClient {
        Task DataBroadcast(object data);
    }

    [Authorize]
    public class InfoHub : Hub<IInfoHubClient> {

        const string _DataGroupName = "Data";
        static readonly ConcurrentDictionary<string, bool> _DataSubscribers = new ConcurrentDictionary<string, bool>();
        static public bool DataSubscribed { get {
                var empty = _DataSubscribers.IsEmpty;
                if (empty)
                    _DataBroadcast = _defaultDataBroadcast;
                return !empty;
            } }
        static readonly object _defaultDataBroadcast = new { };
        static object _DataBroadcast = _defaultDataBroadcast;

        static public Task BroadcastData(object newBroadcast, IHubContext<InfoHub, IInfoHubClient> hub) {
            _DataBroadcast = newBroadcast ?? _defaultDataBroadcast;
            return hub.Clients.Group(_DataGroupName).DataBroadcast(newBroadcast);
        }

        public override Task OnDisconnectedAsync(Exception exception) {
            _DataSubscribers.TryRemove(Context.ConnectionId, out bool temp);
            return base.OnDisconnectedAsync(exception);
        }

        public async Task DataSubscription(bool enable) {
            if (enable) {
                await Clients.Caller.DataBroadcast(_DataBroadcast);
                _DataSubscribers.TryAdd(Context.ConnectionId, true);
                await Groups.AddToGroupAsync(Context.ConnectionId, _DataGroupName);
            }
            else {
                await Groups.RemoveFromGroupAsync(Context.ConnectionId, _DataGroupName);
                _DataSubscribers.TryRemove(Context.ConnectionId, out bool temp);
            }
        }

    }

    /* in broadcaster function:
            readonly IHubContext<InfoHub, IInfoHubClient> _hub; // from dependency injection

            object data = null;
            if (InfoHub.DataSubscribed) {
                /* very heavy data computation */
            }
            await InfoHub.BroadcastData(data, _hub);
    */

}

@slavanap
I will close this issue.
As for your problem, it seems to me that it does not exist. In any case, you can open a new issue or describe it on stackoverflow.


Add a terminal-side shortcut for clearing the screen and/or scrollback

It would be great to have the equivalent of Command+K on the Mac terminal.  This clears the terminal completely, not the equivalent of typing "clear", which only scrolls the cursor to the top of the terminal.  My use case is the following:

Open an Ubuntu terminal
ssh into a server
run a command which may produce large amounts of content and exceptions within the content
fix the exceptions by making a code change
CLEAR the terminal completely (not just a "clear" command)
re-run the command
now that the previous command output was cleared, we can search for new exceptions without getting the previous exceptions
NOTE: this also requires a "ctrl+F" type feature to exist in the terminal, which currently does not

Why just clear screen?  Why not a toolbar where common commands can be added as quick shortcuts?

But, this isn't a bash command, as I think that's what you are asking.  Command+K on a mac doesn't run any commands in the terminal such as "clear"... it clears the entire terminal buffer.  Commands such as "clear" don't do the equivalent.  I personally don't need a toolbar - I'm fine with adding them to the "Settings.json" file like all the other keyboard shortcuts.  Its just that "Clear terminal" is not one of the available options to create a shortcut for.  If you use VSCode, the exact equivalent command is "Terminal:Clear".  I map that to Alt+K to get the same experience.

Are we not talking about the same thing as issue # 1189 ?

@fzed51 Nope, this is unrelated to that one.
To clarify for future me - cmd+k is not something that's being handled by bash or the shell. This is the Terminal itself handling the keybinding, and clearing both the buffer and the scrollback.
You've described a user story that makes sense to me, so I'm gonna put this on the backlog.

We'd probably want #1142 to be able to support adding keybindings for Clear Screen, Clear Scrollback, or Clear Both all on the same "clear" action.
We'll need #1193 for clearing the conpty buffer. W/o this, we'll just get the screen repainted immediately.

I would like to help with that if possible, shouldn't be so difficult right? I mean can be just reset the buffer and initialize again the buffer

You should definitely read the comments here. Because of the windows console infrastructure we’re built on it’s not quite that easy.

Ok, I don't have any idea about the internals of the terminal, however, if in order to clear a buffer after press a key combination is so complex probably it will be impossible to scale the product when adding more complex stuff than this

It's just this issue in particular that's thornier than most. For compatibility with Windows console applications that do not expect a terminal to be connected, we need to maintain a text buffer for them to read. That's all. Because we don't have a way to clear that text buffer (from the terminal side), and we just haven't written one yet, it's only more complex than clearing the local screen. It's not a complexity class issue with all of Terminal.

Gotcha, thanks for the clarification, definitely isn't a beginners task

Bump

Don’t do that. It emails every subscriber to this issue or to this repository. We know that you want this fixed, somebody just needs to do the work.

oh sorry had no idea- thanks for letting me know!
sorry for the inconvenience!
On Wed, May 27, 2020 at 2:55 AM Dustin L. Howett<EMAIL_ADDRESS>wrote:

Don’t do that. It emails every subscriber to this issue or to this
repository. We know that you want this fixed, somebody just needs to do the
work.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/1882#issuecomment-634466731,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2CBH2IZDYFXMW232XFE23RTS2NJANCNFSM4H7ADSLQ
.

@prumbles
If you are running WSL2, you can run:
tput clear
This does reset the terminal buffer fully without scrollback.

@prumbles
If you are running WSL2, you can run:
tput clear
This does reset the terminal buffer fully without scrollback.

both tput clear and celar can't clear while the process is outputing texts to the screen
I want a hotkey to clear the buffer. like ctrl+K in the macOS terminal

So find seems be there with ctrl+shift+F, hopefully we will get the ctrl+shift+K soon

I upvote the need
This is really a needed a feature while I am running logcat on an android device and want to start seeing the log when I start my app
@microsoftopensource please prioritize this.

Adding my support for this feature!

All related issues are being closed by admins by mentioning this one, but there is no activity on this issue,
@microsoftopensource @zadjii-msft How to prioritize this development?

If you want to help prioritize this feature, you can hit that 👍 button on the OP, which will avoid sending an unnecessary ping to everyone that's subscribed to this thread 😉

On my terminal, it worked by input tput reset
But tput clear did not work.

I have created a workaround which worked on WSL/bash, SSH/bash (Linux tested) and PowerShell 7 for me with CTRL+ALT+K:
{ "command": { "action": "sendInput", "input": "\u001B[H#\rtput reset || printf '\e[1J\e[3J\e[r'|| Clear-Host\r\u001B[A\u001B[A\u001B[H\u001B[3~\u001B[F" }, "name":"Clear screen and Scrollback", "keys": "ctrl+alt+k" },
These steps will done with sendInput action:

send home key (if you already type something into shell)
send # (comment out what you have already typed)
send return (save commented out strings to history   note: \n : only work on linux so I used \r)
send tput reset or printf with CSI codes (if tput command fails) or Clear-Host (if both commands failed: eg in PowerShell)
send two times up key for history (needs to have history in shells)
send home key
send delete key to remove first character "#" from the beginning (written by step 2.)
send end key to put cursor back to the end (you can continue typing of the command)

For me the history and scrollback are cleared

open a second tab?

open a second tab?

I have a running process that is logging to the console, but since it is logging too much things and I just want to concentrate in newer lines I just want to clear the screen  so new lines don't get confused with older ones.
A new tab doesn't work for this case

Second tab doesn't work for many cases. SSH sessions, tail -f, long CWD you don't want to type again and again – to start with.

Any updates on this? Still annoying that there is no clear scrollback option. That's the only thing stopping me from uninstalling MobaXTerm

Others have noted that the command cls is not a perfect solution but for me it has reduced the issue to background noise and I am happy to have rediscovered cls (after approximately 30+ years -- I think I used to use it in DOS if I remember correctly).

There are other details in #1193 for those interested in the technical bits.
For everyone else who's just here for results:

There are other details in #1193 for those interested in the technical bits.
For everyone else who's just here for results:

Nice work! This works while running a "ping google.com -t"?
Thanks

I'm sort of confused, I'm hitting ctrl+shift+p to pull up the command menu, but I can't seem to find Clear buffer?

1.12 should be the first version that has this feature.

1.12 should be the first version that has this feature.


Dashes

What does the pull request do?

DashStyle.Dashes is marked as nullable.
What is the current behavior?

Following the changes made in https://github.com/AvaloniaUI/Avalonia/pull/8895, DashStyle.Dashes is null by default.
What is the updated/expected behavior with this PR?

To Reproduce：
var dashStyle = new DashStyle();
dashStyle.Dashes.Add(0.5); // NullReferenceException
How was the solution implemented (if it's not obvious)?

DashStyle.Dashes is marked as nullable.
Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

Fixes #9595

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: linxiua sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

You can test this PR using the following package version. 11.0.999-cibuild0027588-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

Thanks @linxiua !


Add ressources to training

https://enccs.github.io/instructor-training/
PARCE => PRACE
https://www.ebi.ac.uk/training/

https://programminghistorian.org/en/


Import iCub v2.5 head model from icub-models repo

Only the URDF file has been imported. All the meshes and configuration files have been not copied, and the model is still referring to the meshes and configuration files installed by icub-models, that so is effectively a dependency for this repo.
If you want to modify a mesh or a configuration file, for import it in this repo, and then modify the URDF file to refer to it specifying its path w.r.t. indicating that it belongs to the icub_head_epr directory.
Commit reference: https://github.com/robotology/icub-models/tree/5bc520ea2bd8368422818badc6893c4c81201ea4/iCub_manual/robots/iCubGazeboV2_5_visuomanip
The model has been modified by:

Removing the legs
Removing the fingers
Fixing the joints of the torso and the arms, and removing all the related controlboards

Example output of gazebo --verbose web-contest-sandbox.sdf :

This is the first step, then the meshes of the torso and arms will be substitute with appropriate simplified shapes.

Nice! Thanks @traversaro 👍🏻
I think that the robot looks already good. It appears quite evident that the torso and the arms cannot be used. Hence, do we really need cuboids?
What's your opinion?

Ok, then let's leave it like this!


fix memory leak

This PR fixes a trivial memory leak.
Valgrind Before:
==12974== LEAK SUMMARY:
==12974==    definitely lost: 0 bytes in 0 blocks
==12974==    indirectly lost: 0 bytes in 0 blocks
==12974==      possibly lost: 864 bytes in 2 blocks
==12974==    still reachable: 108,923 bytes in 1,082 blocks
==12974==         suppressed: 0 bytes in 0 blocks
==12974==
==12974== For counts of detected and suppressed errors, rerun with: -v
==12974== ERROR SUMMARY: 2 errors from 2 contexts (suppressed: 0 from 0)

Valgrind After:
==13204== LEAK SUMMARY:
==13204==    definitely lost: 0 bytes in 0 blocks
==13204==    indirectly lost: 0 bytes in 0 blocks
==13204==      possibly lost: 0 bytes in 0 blocks
==13204==    still reachable: 58,262 bytes in 841 blocks
==13204==         suppressed: 0 bytes in 0 blocks
==13204==
==13204== For counts of detected and suppressed errors, rerun with: -v
==13204== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

are you telling me I should stop using raw pointers?
:sob:

Either use raw pointers but remember to delete afterwards, or use unique_ptr and forget about that. The second option is recommended nowadays.


Array declaration Issue

Type of issue
Code doesn't work
Description
Some code examples are not running because the array declaration is like this:
int[] numbers = [4, 7, 10];
And I think that they must be something like this:
int[] numbers = {4, 7, 10};
You can try running some of the examples and the issue will occur
Page URL
https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/operators/lambda-operator#code-try-1
Content source URL
https://github.com/dotnet/docs/blob/main/docs/csharp/language-reference/operators/lambda-operator.md
Document Version Independent Id
e8eab8da-72ba-991a-c011-a41d2ed9939d
Article author
@BillWagner
Metadata

ID: a695fde2-da48-835f-423d-d34fb048d6ab
Product: dotnet-csharp
Technology: csharp-language-reference

Thanks for pointing this out @alejandronpereira
Those samples use collection expressions, a new C# 12 feature. Collection expressions aren't supported in the currently deployed Try.NET instance.
I'll revert these samples until we get the updated Try.NET installed on the Learn platform.


switch: add option to only include sessions from search paths

Not sure if anyone other than me would find this useful, but I sometimes I open unrelated tmux sessions for stuff in the background that pollute the switcher. To get to those there's always the option to use tmux's built-in switcher.
Alternatively, tms could be made to support command line flags for its options that take precedence over the config file, so that it's possible to use separate hotkeys for both of versions of the switcher. Let me know if that's preferable.

Can you add the config option to the config subcommand and update the README with the updated help output
Other than that this looks good to me

Looks great!


Nerves project generator should include nerves_init_gadget by default

The nerves project uses nerves_init_gadget by default now and the development ergonomics of it are an incredible relief for the workflow. So I think we should expand the nerves project generator a bit to include this.

FYI - This was fixed in #40.


CEF parsing fails in IPV4 encoded IPv6 addresses in IPv6 CEF fields

The java InetAddress Library does not return an Inet6Address type for IPv4 encoded addresses in IPv6 fields (c6a1, c6a2, c6a3, c6a4).  In this situation, the parser throws an exception.
This condition happens with some devices where an IPv4 address is communicating to an IPv6 device.

Not sure I understand the issue?

When an ipv6 key is set to a mapped ipv4 address, such as c6a3=0:0:0:0:0:ffff:a8e:6cc3
an IllegalArgumentException is thrown:
java.lang.IllegalArgumentException: Can not set java.net.Inet6Address field com.fluenda.parcefone.event.CefRev23.c6a3 to java.net.Inet4Address
...


Can't get cmd to recognise the spotdl command.

What is the purpose of your issue?

[x] Bug
[ ] Feature Request
[ ] Question
[ ] Other

Description

Tried installing numerous ways and it won't recognise the spotdl command.
Log

Q:\jack\Desktop\spotify-downloader-master>spotdl --song https://open.spotify.com/track/2DGa7iaidT5s0qnINlwMjJ --log-level=DEBUG
'spotdl' is not recognized as an internal or external command,
operable program or batch file.

You probably do not have python's bin directory available in your PATH environment variable. 2 possible ways to fix this:

run the official python installer and tick the checkbox mentioning to add python to PATH.
manually add python's bin directory (should be something like C:\Python3\bin) to your system's PATH variable. See this for more info.

I just needed to upgrade my pip and it worked.

See this for more info.

I'm sorry I put a wrong link here. Corrected now.
Either way, good to know it works.

Fixed by using py -3 -m spotdl


Better support for TF 0.12 for loops variables

Complex lookups inside templates show as unresolved references. The new template system allows for some very dynamic code which is likely impossible for the plugin to determine, very often though they are highlighted red due to being unresolved.

Originally posted by @benosman in https://github.com/VladRassokhin/intellij-hcl/issues/155#issuecomment-497503146

Yeah I also noticed that input variables declared as custom types with different attributes cannot be resolved object({<ATTR NAME> = <TYPE>, ... })

Hi,
Just a heads up on for_each loop construct within the dynamic block also has a similar issue. The plugin does not seem to interpret it as being component of the iterator object.
Ref: https://www.terraform.io/docs/configuration/expressions.html#dynamic-blocks
Thank you once again! Let me know if you want me to create a separate issue.

I've fixed all errors on initial screenshot in 0.7.1. Please try it out

Hi,
Plugin version - 0.7.1
Issue:
unresolved references in dynamic block

@apenner-epam Fixed problems shown on your screenshot, will be available in 0.7.2
@jerrygb Problem with 'key' and 'value' is fixed in 0.7.2 (it's the same as above)

Also, noticed that the repo here does not have v0.7.0 tag. Is that something you plan to checkin?

There's no tag because that 0.7+ is closed-sorced, unless some decisions is made.

Hi,
It resolves value & key attributes, but doesn't resolve object attributes.
Plugin version == 0.7.2
IntelliJ Idea == 2019.1.2

Example:
variable "s3_notification_configuration" {
  description = "s3 notification configuration"
  type = list(object({
    id            = string
    events        = list(string)
    filter_suffix = string
    filter_prefix = string
  }))
  default = []
}

// SQS
resource "aws_s3_bucket_notification" "this" {
  count  = length(var.s3_notification_configuration)
  bucket = aws_s3_bucket.this.id

  dynamic "queue" {
    for_each = var.s3_notification_configuration
    content {
      id            = queue.value.id
      events        = queue.value.events
      filter_suffix = queue.value.filter_suffix == "" ? null : queue.value.filter_suffix
      filter_prefix = queue.value.filter_prefix == "" ? null : queue.value.filter_prefix
      queue_arn = format(
        "arn:aws:sqs:%s:%s:%s",
        data.aws_region.this.name,
        data.aws_caller_identity.this.account_id,
        join("-", [var.name, queue.value.id, var.environment])
      )
    }
  }
}

Not sure if it is related, but I have a similar issue accessing complex variables with an index:

@d-helios, @jstewmon Fixed in 0.7.3, will be released shortly

@VladRassokhin
Thank you once again for 0.7.3 fixing all the registered 0.12 issues.
I would like to report one very minor formatting issue, with trailing commas for HCL1 interpolation syntax/HCL2 functions.  See screenshot below.

Functions that can take variable arguments can end with trailing commas according to HCL grammar, I believe. But the plug-in complains the behaviour as an error. See references below.
It should either be a warning formatting or proposed hint to remove the trailing comma, I think. Please correct me if I am wrong.
Ref:
https://www.terraform.io/docs/configuration-0-11/interpolation.html#merge-map1-map2-
https://www.terraform.io/docs/configuration/functions.html

@VladRassokhin I no longer see errors for valid keys, but there's also no error for invalid keys, and there's no completion help. Is that expected?

Also, I'm seeing errors referring to another resource from within a content block:

@jstewmon

but there's also no error for invalid keys, and there's no completion help. Is that expected?

Yes, for now.

errors referring to another resource from within a content block

Will create separate issue for that.

@jerrygb
Thanks, I've created separate issue.


refactor: duplication

@mkobayashime Could you review this PR?

@yudukikun5120
Could you run Prettier and squash commits into one?

@mkobayashime May I squash the commits when merged?

@yudukikun5120
Sorry for the confusion. I'd like you to squash these 5 (previously 4) commits into 1 commit, not the merge with squash.

@mkobayashime BTW, what do we have to squash the commits before the PRs merge for? What do squashing before the merges and "squash and merge" differ?

@yudukikun5120
Sorry, I actually confused Squash and merge with Rebase and merge. I'll welcome Squash and merge from next time, as long as:

commits can be bundled into 1 commit semantically (don't have multiple intention)
commit message is properly set for the new (squashed) commit


Describe UIDispatcher's usage in the Using Asynchronous Windows APIs document

This pull request fixes #427
It describes the usage of UIDispatcher.Post() method in the basic scenarios for both C# and C++.

The description contains the brief introduction explaining why is it necessary to use the UIDispatcher and points to the decs about IReactDispatcher and FileOpenPicker used in these examples (to avoid confusions and to present the control that's related to the UI thread).
There are both C# and C++ examples, both having the same descriptions and steps in them.
Note that the code for launching and handling the FileOpenPicker is as basic as it can be. It is not the part of this guide to explain the file handling in details. For more curious readers I've linked the official docs about it.
Microsoft Reviewers: Open in CodeFlow

@BartoszKlonowski thanks for this PR, we definitely needed it!

@BartoszKlonowski Let us know if you're able to finish this by Thursday. If not, we'll make the necessary changes and merge. It's a fairly important addition to make to the docs. :)

@rectified95 Due to national holiday this has been delayed, but the changes will be delivered today.


Add note recommending automated code formatting

Piranha is better when paired with automated code formatting, and
this note makes it clear that we don't want to commit to produce
pre-formatted code in all cases, much less to provide coding style related
configuration options.

cc: @msridhar


遇到了这样的问题：NameError: name 'deform_conv_ext' is not defined

Hello, the problem is that you didn't build the CUDA extensions successfully.  You could try line 82-83 in the README.md.


Add more descriptive error objects

Reftrofit errors returned from the Web API are sometimes
too generic. This change adds SpotifyError that contains
status code and message returned.
RetrofitError: 403 Forbidden
SpotifyError: 403 Insufficient client scope
RetrofitError: 400 Bad Request
SpotifyError: 400 No search query

@JMPerez @thelinmichael have a look when you have a moment

It looks great! +1

:rocket: Good stuff!


Google sheets script still sometimes does not work

Could just be us doing too much testing at the moment. Can we pay to increase our API call limit per day?

Could just be us doing too much testing at the moment. Can we pay to increase our API call limit per day?

That is almost definitely the reason. They seem to be working now. Whenever we make a new sheet, it makes a copy of the custom scripts (see https://script.google.com/). It's not clear to me if/how we can get all the sheets to share one script instead of making copies. I also could not find anything about paying to increase the quota, the only page I found was https://developers.google.com/apps-script/guides/services/quotas.
Anyways, I don't think this is a critical issue that we have to fix before this weekend, since our sheet creation won't be concentrated on one day during the hunt and it's only an annoyance if the functions stop working.

That is almost definitely the reason. They seem to be working now. Whenever we make a new sheet, it makes a copy of the custom scripts (see https://script.google.com/). It's not clear to me if/how we can get all the sheets to share one script instead of making copies. I also could not find anything about paying to increase the quota, the only page I found was https://developers.google.com/apps-script/guides/services/quotas.
Anyways, I don't think this is a critical issue that we have to fix before this weekend, since our sheet creation won't be concentrated on one day during the hunt and it's only an annoyance if the functions stop working.

Oh, gotcha, this is only affecting the handy spreadsheet functions that are in our template, but doesn't affect the actual sheets copying and updating done by Small Board. Yeah, it's not a huge issue if the functions stop working during the hunt (though hopefully they don't).

Oh, gotcha, this is only affecting the handy spreadsheet functions that are in our template, but doesn't affect the actual sheets copying and updating done by Small Board. Yeah, it's not a huge issue if the functions stop working during the hunt (though hopefully they don't).

How do I actually access the custom script? I opened one of our sheets with the cardinalitypuzzles@ account and clicked "Script editor" but it gives me some generic error.

How do I actually access the custom script? I opened one of our sheets with the cardinalitypuzzles@ account and clicked "Script editor" but it gives me some generic error.

It only works for me if cardinalitypuzzles@ is the main or only account I'm logged into. Once I'm on that page, if I add the smallboard GCP project ID under "Project Setting," then I'm able to edit the script using my normal account (probably because I'm added to the GCP project? but I think only RG and I are, so maybe we can test this). This project ID is not carried over when the sheet is copied.

It only works for me if cardinalitypuzzles@ is the main or only account I'm logged into. Once I'm on that page, if I add the smallboard GCP project ID under "Project Setting," then I'm able to edit the script using my normal account (probably because I'm added to the GCP project? but I think only RG and I are, so maybe we can test this). This project ID is not carried over when the sheet is copied.

One way we could deal with this is by creating a ton of sheets in advance of the hunt and then just renaming them as needed during the hunt, heh.

One way we could deal with this is by creating a ton of sheets in advance of the hunt and then just renaming them as needed during the hunt, heh.

We could consider just attaching a "master copy" of the custom function script to new blank spreadsheets, rather than making a copy of the script with every spreadsheet. There are ways to do this:

Via a library: https://developers.google.com/apps-script/guides/libraries#writingLibrary
Via a webapp: https://developers.google.com/apps-script/guides/content
https://stackoverflow.com/questions/28390885/how-to-run-google-app-script-function-from-another-project

However, either of these will make the custom functions noticeably slower, and we might hit other quotas if we're using the same script for everything. (Also, they'll be a nontrivial amount of work to implement.)

Handy to find google sheet id

I believe this issue is fixed with the replacement of scripts with named functions.


Proposal to somehow handle this error

Good day.
Is it possible to somehow handle this error and maybe convert it to some reasonable exceptino message (maybe log to to symfony logger), because it is very hard to find problem if something is wrong with the key, or password etc/
https://github.com/lexik/LexikJWTAuthenticationBundle/blob/78e8ddda0a57de4f2eb40ab247922e33e0772bc2/Services/JWSProvider/LcobucciJWSProvider.php#L130

agree this is poor error handling, I will look at it.
Thanks for raising!