Vows v1.0.0 - Refactoring

Code style/structure
I think that some bits of the source should be re-written. The core functionality is great, but I think there is a reliance on too many implicit actions that make the code difficult to maintain.
ex: so much this!
Many uses of this, especially where they can mean different things need to go.
this.run = function (options, callback) {
            var that = this;
            this.done = function(){
                that.runWithin(options, callback);
            }
            // If there is a beforeSuite defined, the tests will run only after this.done() was called.
            if(typeof(this.beforeSuiteFunc) === 'function') {
                this.beforeSuiteFunc.call(this);
                return;
            }
            this.done();
        }

My purpose being the rewrite is to decrease complexity and add more documentation so that it is easier for others to contribute.
Suite.js is 480 lines long. That's just unacceptable from a comprehension perspective, so I'll be massaging the code base into a more modular and functional style.
What are the core features of vows?
Breaking changes / new direction
Right now, vows uses a lot of special syntax, (batch, suite, etc) which is helpful for describing the program, but it is not necessary or particularly helpful.
Right now a basic test looks like this.
vows.describe("Vows test file with camel case").addBatch({

  "The test file": {
    topic: function () {
        return { flag: true };
    },
    "is run": function (topic) {
        assert.isTrue(topic.flag);
    }
  }
}).export(module);

I think it should look something like this.
vows('Vows test file with camel case', () => {
  return {
    "The test file": {
      topic: () => {
          return { flag: true };
      },
      "is run": (topic) => {
          assert.isTrue(topic.flag);
      }
    }
  }
})

One of the most powerful aspects of vows is that it takes advantage of plain js objects to describe tests. I think it is simply taking this core feature to it's final purpose by making a 'batch' return a plain js object in a function body.
This testing syntax models the simplicity of the testing library tape, but the own internal working of the way vows runs the tests will stay the same.
Test Execution

Synchronous
The base case here is that the topic needs to evaluate sync functions and regular data types.

   topic: () => {
       return { flag: true };
   }

  topic: 10

  topic: { flag: true }

Asynchronous
Handles promises!

  topic: () => {
     return new Promise((resolve) => {
       setTimeout(() => {
         resolve(42)
      }, 1000)
    })
  }

Legacy
Callbacks. :)

Event

Parallel

Context
Test Hooks

Needs a teardown hook
it's reasonable to use a bit of state in your tests. While state can be 'setup' in an outer context, I think keeping the teardown hook is simply a good idea for a concrete place to wipe away the test state.

@mmhansen love the enthusiasm! Having been deep in the internals of suite.js I feel your pain about complexity. For better or for worse that complexity is derived from having sync and async topics live side by side. e.g. reimagining the example you have as async:
const fs = require('fs');
const path = require('path');
vows('Vows test file with camel case', () => {
  return {
    "The test file": {
      topic: (done) => {
          fs.readFile(path.join(__dirname, 'test-file.txt'), 'utf8', done);
          return { flag: true };
      },
      "is run": (topic, err, contents) => {
          assert.isTrue(topic.flag);
          assert.equal(err, null);
          assert.equal(typeof contents, 'string');
      }
    }
  }
})

The scenario above makes choices on questions (that you could go either way on):

How do you differentiate sync from async? The above does it the mocha way by checking the arguments.length to the topic`. This will become more difficult with nested topics if you choose to keep those
Do you provide err to test functions for async topics? The above does, but today they are assumed to be broken tests unless you set allowErrors: true on the suite ... ?? not sure if that exists actually ??
Do you provide the literal topic to the results of an async topic? The above does, but that could obviously get tricky.

@indexzero hm, that's an interesting example case. Are you saying that they would try to return both an object and an async callback?

@indexzero hm, that's an interesting example case. Are you saying that they would try to return both an object and an async callback?
I think the best behavior in that situation would be to take whatever is returned first. So in this case, I would get the { flag: true } as the topic, and ignore the done that is called from readfile.
Do you think that there is a use case for that example, or is it just what a crazy user could do?
I guess things get pretty tricky when you bring in the callback pattern. If the only way to handle asyncs was to wrap them in promises then I wouldnt have much trouble differentiating those from a primitive value.
I think we can still handle callbacks with this.callback, because topics have to be passed down to children.
'A banana': {
        topic: new(Banana),

        'when peeled *synchronously*': {
            topic: function (banana) {
                return banana.peelSync();
            },

....

So your example could be
vows('Vows test file with camel case', () => {
  return {
    "The test file": {
      topic: () => {
        fs.readFile(path.join(__dirname, 'test-file.txt'), 'utf8', this.callback);
      },
      "is run": (err, contents) => {
        assert.equal(err, null);
        assert.equal(typeof contents, 'string');
      }
    }
  }
})

To be honest, I don't like using this too much, but it would keep everything working the same as it is now.


[bug] RebuildAccel doesn't handle NULL entities or those without a valid model

Describe the bug
A NULL entity will currently throw a generic GLua error (this may be adequate, however untested), and an entity with either no model or a model not present on client hasn't been tested (depending on how util.GetModelMeshes works, this either works fine, by just not adding any meshes, or if it returns nil then it will throw an error).

Fixed in 000e43157255521096167fcb7c9081a9dc1a6d37


Add helper methods for creating SshAgent with KeyPair

I was writing a few tests which utilized remote-exec for newly generated keys not in my local SSH agent, and I found myself copying the same bits of code found in the test/terraform_ssh_example_test.go example.
Here I propose adding two new methods to the ssh package, SshAgentWithKeyPair and SshAgentWithKeyPairs. With these users will only need to add something such as:
sshAgent := ssh.SshAgentWithKeyPair(t, keyPair)
defer sshAgent.Stop()

I'll let the tests run, and once they pass, I'll create a new release and paste the link here.

https://github.com/gruntwork-io/terratest/releases/tag/v0.10.1

Awesome! ... However, I just thought of a possible problem, one that was implicitly working before because of the environment variable swap magic.
Should terraform.InitAndApply() (namely the apply step) also be driven by the in-process ssh agent?
The use case I'm considering is if the terraform plan uses a remote-exec that relies on the SSH agent for connection.
Is this a use case you want to cover with terratest?
Here's a snippet that might illustrate it better:
keyPair := aws.CreateAndImportEC2KeyPair(t, awsRegion, uniqueName)

terraformOptions := &terraform.Options{
	TerraformDir: terraformDirectory,
	Vars: map[string]interface{}{
		"aws_region":        awsRegion,
		"instance_name":     uniqueName,
		"aws_key_pair_name": keyPair.Name, // this is attached to the instance,
                                                   // so that remote-exec works
	},
}

sshAgent := ssh.SshAgentWithKeyPair(t, ec2KeyPair.KeyPair)
defer sshAgent.Stop()

// In the previous iteration `SSH_AUTH_SOCK` env var was changed to the
//  in-process agent's socket file at this point, which then was then
// passed along to terraform:
terraform.InitAndApply(t, terraformOptions)

How are you thinking of forcing Terraform to use the in-process SSH agent?

Forcing terraform isn't a problem, all you need to do is pass the ssh agent into the terraform.Options struct, and then set the environment before launching terraform:
os.Setenv("SSH_AUTH_SOCK", SshAgent.socketFile)
exec.Command("terraform", ... )

.. I've tested that and it seems to work.
But then you run into the problem with t.parallel() again.
The solution that comes to mind is to use a mutex to only allow one test to change that environment variable at a time. I'm not sure if that's considered hacky or not, I'm still kind of new to go. I also don't know off hand if that env var needs to be in place for the life of the terraform process, or just on initiation. The latter would obviously be more ideal, for concurrency's sake.
If all that seems too convoluted, I understand. I don't need this feature for anything important right now, and I've already hacked around it. I just figured I should point out that terraform modules with remote-exec (or any terraform ssh connection) are potentially untestable with terratest, unless the keys are already present in the local SSH Agent. (making the tests less portable)

Oh, you know what, using env vars, I think we can make this work even with t.parallel(). To execute terraform, Terratest calls shell. RunCommandAndGetOutputE. This method allows you to configure the environment variables to use when shelling out to terraform. We could set SSH_AUTH_SOCK there.

Nice find! I guess I had assumed go used whatever environment was available to it when launching subshells. I'll take a look at this in the morning. It should be pretty easily to implement, if that does what it says it does.
Should I amend this PR or open a new one?

New PR please and thank you 🍺


Support @quarkusbot retest command

It would be nice if we could retrigger the build by commenting something like @quarkusbot retest. Eg. In PRs which updates a dependency that is not in Central yet
This would be useful in conjunction with #23

I was looking at this one but I can't find a simple way to react to mentions. I'm not even sure we can mention an app. I don't know how dependabot works.. @gsmet have you some hints?

I think you will have to listen to IssueComment. Note that it's a bit of a pain because you have to check if it's a PR (you have a method for that) and then get the associated pull request and from there you should be able to make progress.
I would do something with a pattern such as ^@(?:quarkus-?)?bot\s+([a-z _\-]+)\s+ and make it case insensitive.
And then have a map of commands.
I would also ignore the comments added by the bot itself but we can add that later as you will need the ability to inject GHMyself and I haven't worked on this yet.

Ah and better check the pattern before getting the pull request itself, that will avoid one API call.

Thanks I was missing the part on pull request comment which are issue comments. I will go your way :)


resolve Rhea mapping for 'sodium-transporting ATPase activity, rotational mechanism' GO:0046962, 'sodium-transporting ATP synthase activity, rotational mechanism' GO:0046932, 'P-type sodium transporter activity' GO:0008554

sodium-transporting ATPase activity, rotational mechanism EquivalentTo ATP + H2O + 4 Na(+)(in) => ADP + H(+) + 4 Na(+)(out) + phosphate	(most probable)	0.8
ATP + H2O + Na(+)(in) => ADP + H(+) + Na(+)(out) + phosphate EquivalentTo METACYC:<IP_ADDRESS>-RXN	(most probable)	0.75
sodium-transporting ATP synthase activity, rotational mechanism ProperSubClassOf METACYC:<IP_ADDRESS>-RXN		0.08
sodium-transporting ATPase activity, rotational mechanism EquivalentTo EC:<IP_ADDRESS>	(most probable)	0.8
ATP + H2O + Na(+)(in) = ADP + H(+) + Na(+)(out) + phosphate EquivalentTo EC:<IP_ADDRESS>	(most probable)	0.75
ATP + H2O + 4 Na(+)(in) = ADP + H(+) + 4 Na(+)(out) + phosphate EquivalentTo EC:<IP_ADDRESS>	(most probable)	0.75
P-type sodium transporter activity EquivalentTo METACYC:<IP_ADDRESS>-RXN	(most probable)	0.8
ATP + H2O + Na(+)(in) => ADP + H(+) + Na(+)(out) + phosphate ProperSubClassOf METACYC:<IP_ADDRESS>-RXN		0.1
ATP + H2O + 4 Na(+)(in) <=> ADP + H(+) + 4 Na(+)(out) + phosphate EquivalentTo METACYC:<IP_ADDRESS>-RXN	(most probable)	0.75
sodium-transporting ATPase activity, rotational mechanism EquivalentTo METACYC:<IP_ADDRESS>-RXN	(most probable)	0.8
P-type sodium transporter activity EquivalentTo ATP + H2O + Na(+)(in) = ADP + H(+) + Na(+)(out) + phosphate	(most probable)	0.8
ATP + H2O + Na(+)(in) = ADP + H(+) + Na(+)(out) + phosphate ProperSubClassOf EC:<IP_ADDRESS>		0.1
sodium-transporting ATP synthase activity, rotational mechanism ProperSubClassOf EC:<IP_ADDRESS>		0.08
P-type sodium transporter activity EquivalentTo EC:<IP_ADDRESS>	(most probable)	0.8
ATP + H2O + Na(+)(in) => ADP + H(+) + Na(+)(out) + phosphate ProperSubClassOf METACYC:RXN-19779		0.1
sodium-transporting ATP synthase activity, rotational mechanism ProperSubClassOf ADP + H(+) + 4 Na(+)(out) + phosphate => ATP + H2O + 4 Na(+)(in)		0.08

[ ] 'sodium-transporting ATPase activity, rotational mechanism' GO:0046962 : Def is "Enables the transfer of protons from one side of a membrane to the other according to the reaction: ATP + H2O + H+(in) = ADP + phosphate + H+(out), by a rotational mechanism." - but e.g. PMID:30791207 states "under physiological conditions catalyzes the synthesis of ATP from ADP and Pi"; so the definition, while correct (because of the =), is a misleading for curators.

GO:0008554, 'P-type sodium transporter activity' -> change Rhea mapping to BROAD, as this also represents the ABC sodium transporter

+[Term]
+id: GO:0140679
+name: ABC-type sodium transporter activity
+namespace: molecular_function
+def: "Enables the transfer of a solute or solutes from one side of a membrane to the other according to the reaction: ATP + H2O + Na(in) -> ADP + phosphate + Na(out)." [PMID:9106203]
+xref: EC:<IP_ADDRESS> {source="skos:exactMatch"}
+xref: RHEA:14633 {source="skos:boradMatch"}
+xref: TC:3.A.1.115.1
+is_a: GO:0140359 ! ABC-type transporter activity
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/21796 xsd:anyURI
+created_by: pg
+creation_date: 2021-07-08T11:22:56Z


Change webaccess auth file format to accommodate different hashing algorithms

This pull request fixes issues raised in previous PR (#980) discussion:

Qt4 doesn't support SHA256 thus is has to use SHA1 or MD5
SHA1 and MD5 both have been broken, so Qt5 build uses SHA256
This prevents passwords file from being shared between QLC+ instances with different Qt versions
Current implementation doesn't support salting

Changes:
To fix that I've changed passwords file to use a different format (backwards compatible).
Each line is now in format username:passwordHash:userlevel:hashAlgorithm:salt, where only username and passwordHash is required.
Each field explained:

username: the login name
passwordHash: hash of string password + salt using hashAlgorithm
userLevel: integer expressing user access level as defined in WebAccessUserLevel enum in webaccessauth.h. Default: super admin (100)
hashAlgorithm: one of md5, sha1, sha256 (only sha1 is guaranteed to be available). Default: sha1 (Qt4 and Qt5 with QT_CRYPTOGRAPHICHASH_ONLY_SHA1 defined) or sha256 (Qt5). If unknown hash algorithm is used sha1 is attempted
salt: random string to prevent dictionary attacks

There is new #define in webaccessauth.cpp - SALT_LENGTH: when creating a new user entry HEX string of given length is generated as salt
Upon saving (i.e. after adding a new user) QLC+ saves row with all five fields (possibly empty salt)
Possible enhancements:

There is no way of changing the default hashing algorithm thus there is no way to make QLC+ add new users with selected hash type. This could be a command line option, however I'm not sure if this would be really useful to anyone and I don't want to alter main program's code unless really necessary (webaccess is a library after all).
Make generating salt more efficient - currently it's probably 6 allocations of memory for QString (depending on implementation). I figured that this would be so rarely used that readability is more important than efficiency.

Additional:
Not tested under Qt4
Tested only under Windows
Example passwords file:
test:233307f39dbd5f45c4700d5d56202e120e6e0c429c2a08a36fd54a94b33cc8bd:100:sha256:876782d575d1b6cb09404c994f98785e
test1:448ed7416fce2cb66c285d182b1ba3df1e90016d:100:sha1:12345
test256:ecd71870d1963316a97e3ac3408c9835ad8cf0f3c1bc703527c30265534f75ae:10:sha256:
test5:ce6a67d0b5d3fababb07e43d61105717:20:md5:54321

Users (username/password):

test/test12345 - super admin (SHA256, generated through web access interface)
test1/test - super admin (SHA1)
test256/test123 - vc only (SHA256 without salt)
test5/test - vc + simple desk (MD5)

@enbyted thanks a lot for this :+1:
First consideration: I wouldn't worry too much about switching from Qt4 to Qt5.
Right now all the QLC+ builds are on Qt5 except for some Linux cases. I do guess though that most of the Linux users download Qt5-based builds from OBS, as most of the Linux distros now bundle Qt5 by default.
In any case the goal is to phase out Qt4.
If really the Qt4->Qt5 switch happens, I would say we can require to delete and re-create the password file.
Can this information change/simplify anything in this PR ?

@mcallegari
Actually, switch from Qt4 to Qt5 will be working out of the box. The other way around will not, unless there is a way to force QLC+ to use SHA1 (edge case in my opinion).
Either way, the way it is implemented currently is the simplest that I can think of that will still build on Qt4 and choose the most secure hash by default.

Alright, I haven't tried the code myself yet, but I reviewed it and it looks OK. I'll test it in the next few days.
Thanks again ! :+1:

No problem, if anything goes wrong give me a @ping here and I'll fix it ASAP.


Cropper Flip Horizontal And Vertical

Thanks for your reply ..
Your code helps me a lot ...... but issue remains as
when we click on flip button it flipped once and not reverted back to original state on other click..
can you help me...... its urgent....
HERE IS THE LINK YOU CAN TEST AND POST BACK HERE
http://<IP_ADDRESS>/~abp/cropper/
Thanks

Toggle flip horizontal:
var data = $().cropper('getData');

$().cropper('scale', -data.scaleX, data.scaleY);

Toggle flip vertical:
var data = $().cropper('getData');

$().cropper('scale', data.scaleX, -data.scaleY);

Thanks for another reply
but i am unable to integrate this code in main.js
so please mention how and where to place this code in main.js file on given demo link
http://<IP_ADDRESS>/~abp/cropper/

After adding above code in my main.js file
flip stops working ,....... because there is no method with name scale defined on it .
i am unable to integrate this code in main.js
so please mention how and where to place this code in main.js file on given demo link
Demo lInk where i need to impliment
http://<IP_ADDRESS>/~abp/cropper/
http://<IP_ADDRESS>/~abp/cropper/


Is TempData the right storage?

Are you sure you want to use TempData here vs ViewBag?
https://github.com/TwilioDevEd/browser-calls-csharp/blob/master/BrowserCalls.Web/Views/Shared/_Messages.cshtml#L1
IIRC TempData will persist in session across multiple requests, so if I reading your code right, if I go away from and then come back to this page the message will continue to show.
I think ViewBag only persists for the single HTTP request.

You right @devinrader, looks like TempData is not what we should be using here, I need to do some changes in order to use ViewBag, thanks for pointing me out in the right direction.


Check the sample project

Please check if https://github.com/javaparser/javasymbolsolver-maven-sample is okay.

Done: I have opened two PRs


Integration tests use different refs for base-platform and base-sdk

See original issue on GitLab
In GitLab by [Gitlab user @tlater] on Nov 13, 2017, 14:40
This has the unfortunate side effect of downloading multiple versions of the largest packages in the project, which probably causes a 10-20 minute overhead.
Quoting [Gitlab user @tristanvb] from an IRC conversation on this topic:
<tristan> What should be done: A.) Use the new ref all around, B.) dont stage *both* gnome-platform and gnome-sdk, only the sdk with the proper symlinks trick
<tristan> which can be found in gnome-modulesets-base repo, there is the correct way to do it
<tristan> Also, possibly updating the base causes binary output to be different, in which case, integration tests should be refactored as mentioned in a bug report I filed

The mentioned issue is #126.

In GitLab by [Gitlab user @tlater] on Nov 13, 2017, 14:40
made the issue visible to everyone

In GitLab by [Gitlab user @tlater] on Jul 24, 2018, 11:46
closed


[WIP] Mediafinder Modes

This PR introduces support for all mediamanager filters as modes for the mediafinder formwidget.
This means that the following mode values would be available:

image
file
document
audio
video
everything (default)

See https://github.com/wintercms/winter/issues/19#issuecomment-1312826732 & https://github.com/octobercms/october/issues/2726#issuecomment-319555427.
@jaxwilko @bennothommo any thoughts on all vs everything for the mode that shows everything?

@LukeTowers @bennothommo the filter const and value have now been updated to reference all instead of everything

@jaxwilko is there a PR for the docs?

Docs PR: https://github.com/wintercms/docs/pull/129


Documentation pass

Need to document the

switch from data() to storage()
new assign mechanism if assign_to is merged

:+1: maybe a special section about the breaking changes in the 0.16 changelog.


Make API names links, remove Go! column

as per issue #727

I don't know what the fail is, but the diff previews also failed, possibly because it's so long.

Thanks for doing this :)
It fails as the checks are expecting a "Go" link 👍
Need @davemachado to jump in and update his script!

Hi guys, sorry for the delay here. I'll cook up a PR for the build scripts changes.
This formatting change would also require a few changes to this project's API as well. I'll prep a PR for those needed changes and once that's ready we can merge everything :)

I have two PRs ready to go:

Build script changes: https://github.com/toddmotto/public-apis/pull/741
API changes: https://github.com/davemachado/public-api/pull/6

@jezmck once you resolve the merge conflicts in the README (which basically just means re-running your regex find-and-replace again on the new upstream changes) and this PR is ready to merge, I can take care of merging all three changes and ensure nothing breaks along the way.

I resolved the conflict manually in the editor here, rather than rerunning the regex since it didn't seem complex enough to need it. There is therefore a chance that I've missed something.

Hi @jezmck , can you please share the regex you're running? Thanks.

Sorry, I no longer have the regex I used, but I've rewritten it;
^\| (.*?) \| (.*) \| \[Go!\]\((.*)\) \|$, with this replacement: | [$1]($3) | $2 |

Hey guys, in order to help @jezmck not have to deal with painful merge conflict resolutions on this PR, I used this gist to move the links and this is merged in at e2b66cea263d77107000855ce66fb3554922dd02 . Thanks @jezmck for the great idea! 😄


[BugReportTool] detect if "Run this program as an administrator” flag is set

Provide a description of the new feature / enhancement
The flag should not be set for PowerToys.exe since it breaks the logic used to manage elevation in PowerToys.

If the flag is set, the .exe is listed in
Computer\HKEY_CURRENT_USER\SOFTWARE\Microsoft\Windows NT\CurrentVersion\AppCompatFlags\Layers

Resolved with the release of v0.41.


Allow update delete functions on chromadb vectors

added delete function with "Chroma integration delete vector store #95"

Partial completion with: "Fixed Chromadb not deleting documents #110 "

Fixed db lock error: " Db lock error on delete #121 "


Make better preview displays in terminals

Implement markdown renderer for terminal
Implement preview command for file|mime types

Not yet completed for the ideal behaviors like a auto-scrolling and editor integrations
However this is reasonable for me.
Resolves GH-740

I might add poppler in another PR for yazi PDF preview


Add scope attribute to raw file upload

Currently a raw file is public by default. We need to be able to make it private to owner by specifying scope=private|public

@Dilshat raw file repo now also supports &private=true argument in upload request, as template repo already does


Is possible to change COLOR in if else condition?

I want to change colour of entity depending on condition.
For example:
- entity_id: sensor.power_fv2grid
name: FV to Grid
color: >-
{% if (states('sensor.power_fv2grid')|float > 0) %} green
{% else  %} red
{% endif %}
children:
- sensor.power_house
Tried this code, no errors, but not working.
Any suggestions?

Templates are not supported in any cards AFAIK.
However you can achieve what you want with the new functionality in v0.6.4 . See #10


App icons and Launch screen added

App icons added as required by issue #10

Icons are changed to better version made by @aquam1ne.
Removed color and name from launch screen at same time adding tab bar.
Fonts added and registered for future use.


transformer: 2n ** 32n gets transformed into runtime error Math.pow(2n, 32n)

I think we need to bail in this case :-/
tsc and esbuild throws "BigInt literals are not available when targeting lower than ES2020."

Background on why it's not possible to support BigInt with a polyfill: https://github.com/evanw/esbuild/issues/732
It looks like we should bail on any BigInt literal, not just when used with **.

@Dunqing How do we determine whether https://babel.dev/docs/babel-plugin-syntax-bigint is enabled or not in our transformer?

@Dunqing need to figure out whether bigint is targeted or not.


[BUG]

Describe the bug
I got : Cannot read properties of undefined (reading 'slice')
Steps To Reproduce
i practise with https://api.fake-rest.refine.dev. I create a resources={[
{
name: "posts",
list: PostList,
show: PostShow,
create: PostCreate,
edit: PostEdit,
}]} with AntdInferencer. It can get data sometimes , but sometime it get error like this when i change pagination.
Expected behavior
solve  error
Screenshot

Desktop
No response
Mobile
No response
Additional Context
No response

Hey @vinhhypc ,
I tested it but couldn't reproduce the issue. Could you let me know if the problem persists?


Support for enum arrays

I have:

enum StudentType {    FullTime = 0,    PartTime = 1,    TeacherAssistant = 2 }

interface User {    firstName: string;    lastName: string;    username: string;    emailAddress: string;    studentTypes: StudentType[] }

output becomes:
{   "Admin": {     "firstName": "Libbie",     "lastName": "Kuhic",     "username": "Forest_Block",     "emailAddress"<EMAIL_ADDRESS>    "studentTypes": [       {},       {},       {}     ],     "adminRecord": {       "studentsPassedEachYear": [         64163       ]     }   }, }
expected:
{   "Admin": {     "firstName": "Libbie",     "lastName": "Kuhic",     "username": "Forest_Block",     "emailAddress"<EMAIL_ADDRESS>    "studentTypes": [ 0,1,2 ],     "adminRecord": {       "studentsPassedEachYear": [         64163       ]     }   }, }

Thanks for your observation! Feel free to open a PR if you have a chance!

I believe I have a patch for this layered on top of #42.  However, my tests always repeatedly populates a repeated, fixed value, 1.  @ryanmcdermott is there an annotation or switch to randomize the value?  Is this likely a bug in my implementation?

Also getting double quoted strings here 👍


Avoid downloading plugins.json

Today, each UpdateRepository downloads its plugins.json to the plugins folder, but it is only ever used in that same method to initialize the PluginInfo. This PR uses an InputStreamReader instead.

Thanks!


Display all exception stacktrace in page result

Actually, only the ten first lines are displayed

Finally we have juste to use :
--debug argument for command line runner
true for maven plugin runner


Replacing body TurboLinks style not working

Seems like replacing the whole body TurboLinks style isn't working. The server gets the request and sends the correct response, but the page turns white.
This is how the link looks like: replace body
Wrapping the content of body in a div and replacing the div on the other hand works flawlessly:
replace body-wrapper
It is also not possible to address the body in any other way, e.g. id-attribute.
A more complete example. I have this page:
<!DOCTYPE html>

    <title>hello world</title>

hello world

    replace body

Clicking the link should replace the body with the body contents from the response like this:
<!DOCTYPE html>

    <title>hello world</title>

hello intercooler

Although the response is correctly coming in, the result looks like this:
<!DOCTYPE html>

    <title>hello world</title>

Hmmm.  This is a hard one to test.
Is using an inner div as the target unacceptable in the short run?
Cheers,
Carson
On March 3, 2017 at 4:17:54 AM, Jochen Breuer<EMAIL_ADDRESS>wrote:
Seems like replacing the whole body TurboLinks style isn't working. The
server gets the request and sends the correct response, but the page turns
white.
This is how the link looks like: replace body
Wrapping the content of body in a div and replacing the div on the other
hand works flawlessly:
replace body-wrapper
It is also not possible to address the body in any other way, e.g.
id-attribute.
A more complete example. I have this page:

hello world

hello world
replace body

Clicking the link should replace the body with the body contents from the
response like this:

hello world

hello intercooler

Although the response is correctly coming in, the result looks like this:

hello world

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/LeadDyno/intercooler-js/issues/175, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcov8ciqBA5IRqKUNrUYJFZsMoqYWzFks5riATxgaJpZM4MSMob
.

That is exactly what we are doing as a workaround. Replacing the inner div works flawlessly.

I researched this a bit and came across the following from the jQuery docs [1]:

When passing in complex HTML, some browsers may not generate a DOM that exactly replicates the HTML source provided. As mentioned, jQuery uses the browser's .innerHTML property to parse the passed HTML and insert it into the current document. During this process, some browsers filter out certain elements such as , , or  elements. As a result, the elements inserted may not be representative of the original string passed.

I tracked this down to what I believe is the relevant Intercooler line:
 asQuery = $($.parseHTML(newContent, null, true));

$.parseHTML() isn't returning the  tag as mentioned in the jQuery docs.
[1] http://api.jquery.com/jquery/

thanks scott for looking into this
if jquery doesn't support it, unfortunately I don't think intercooler should support it


Update default time format

This PR closes #10. I updated the default time format to match the expected hours:minutes:seconds. I updated all tests and docs that referenced the previous timestamp (or similar) as well.

No worries at all! Thank you for this fantastic library!


syntax error near unexpected token

Hi @JiLiren
can you please provide your flutter doctor -v
and your flutter run --verbose?
Also, to better address the issue, would be helpful
if you could post  a minimal code sample
or the steps to reproduce the problem.
Please refrain from posting pictures if not necessary.
Thank you

This may be a flutter setup issue.  What's the output of:
$ cd /Users/ak/company/flutter/flutter
$ git status

Also need flutter doctor -v

Flutter (Channel dev, v1.13.6, on Mac OS X 10.14.6 18G2022, locale
zh-Hans-KH)
Launching lib/main.dart on iPhone (3） in debug mode...
Automatically signing iOS for device deployment using specified development team in Xcode project: 268255YMV4
Warning: Podfile is out of date
This can cause a mismatched version of Flutter to be embedded in your app, which may result in App Store submission rejection or crashes.
If you have local Podfile edits you would like to keep, see https://github.com/flutter/flutter/issues/24641 for instructions.
To regenerate the Podfile, run:
rm ios/Podfile
Running Xcode build...
Xcode build done.                                            5.5s
Failed to build iOS app
Error output from Xcode build:
↳
** BUILD FAILED **
Xcode's output:
↳
: command not found
'
'
Command PhaseScriptExecution failed with a nonzero exit code
note: Using new build system
note: Planning build
note: Constructing build description
Could not build the precompiled application for the device.
Error launching application on iPhone (3）.

It can be used normally a month ago, I found this problem yesterday, I tried various versions of flutter and the results are the same

I can run it normally on my Android phone

Hi @JiLiren
does this possible solution  fix the issue?
Please refrain from posting pictures if not necessary.
Thank you

no

This may be a flutter setup issue. What's the output of:
$ cd /Users/ak/company/flutter/flutter
$ git status


Add support for @typescript-eslint version 6.x.x

What version of this package are you using?
36.1.0
What problem do you want to solve?
Crash at running the linter:
ESLint: 8.45.0

Error: .eslintrc.js#overrides[0] » eslint-config-standard-with-typescript:
        Configuration for rule "@typescript-eslint/restrict-plus-operands" is invalid:
        Value {"checkCompoundAssignments":true} should NOT have additional properties.

    at ConfigValidator.validateRuleOptions (/Users/I339261/SAPDevelop/poolifier-git/node_modules/.pnpm/@eslint+eslintrc@2.1.0/node_modules/@eslint/eslintrc/dist/eslintrc.cjs:2039:23)
    at /Users/I339261/SAPDevelop/poolifier-git/node_modules/.pnpm/@eslint+eslintrc@2.1.0/node_modules/@eslint/eslintrc/dist/eslintrc.cjs:2094:18
    at Array.forEach (<anonymous>)
    at ConfigValidator.validateRules (/Users/I339261/SAPDevelop/poolifier-git/node_modules/.pnpm/@eslint+eslintrc@2.1.0/node_modules/@eslint/eslintrc/dist/eslintrc.cjs:2091:34)
    at ConfigValidator.validateConfigArray (/Users/I339261/SAPDevelop/poolifier-git/node_modules/.pnpm/@eslint+eslintrc@2.1.0/node_modules/@eslint/eslintrc/dist/eslintrc.cjs:2217:18)
    at CascadingConfigArrayFactory._finalizeConfigArray (/Users/I339261/SAPDevelop/poolifier-git/node_modules/.pnpm/@eslint+eslintrc@2.1.0/node_modules/@eslint/eslintrc/dist/eslintrc.cjs:3974:23)
    at CascadingConfigArrayFactory.getConfigArrayForFile (/Users/I339261/SAPDevelop/poolifier-git/node_modules/.pnpm/@eslint+eslintrc@2.1.0/node_modules/@eslint/eslintrc/dist/eslintrc.cjs:3780:21)
    at FileEnumerator._iterateFilesRecursive (/Users/I339261/SAPDevelop/poolifier-git/node_modules/.pnpm/eslint@8.45.0/node_modules/eslint/lib/cli-engine/file-enumerator.js:450:49)
    at _iterateFilesRecursive.next (<anonymous>)
    at FileEnumerator.iterateFiles (/Users/I339261/SAPDevelop/poolifier-git/node_modules/.pnpm/eslint@8.45.0/node_modules/eslint/lib/cli-engine/file-enumerator.js:299:49)

Is this still an issue, please? May I see the output of npm ls, please?

Yes. To reproduce, you just need to install
"eslint": "^8.45.0",
"eslint-config-standard-with-typescript": "^36.1.0",
"@typescript-eslint/eslint-plugin": "^6.1.0",
"@typescript-eslint/parser": "^6.1.0",
in a dummy app, configure eslint with documented defaults for eslint-config-standard-with-typescript and @typescript-eslint and run the linter.

37.0.0 didn't fix this, as I thought it would from the major version bump:
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@typescript-eslint/eslint-plugin
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/eslint-config-standard-with-typescript
npm ERR!   dev<EMAIL_ADDRESS>from the root project

Are there any fixes for this error, i'm also facing the same issue

37.0.0 didn't fix this, as I thought it would from the major version bump:
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@typescript-eslint/eslint-plugin
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/eslint-config-standard-with-typescript
npm ERR!   dev<EMAIL_ADDRESS>from the root project

were you able to resolve this?

Hi karolyi, were you able to resolve this?

Yes, by downgrading the required packages.

This project doesn't support typescript-eslint v6 yet. Your package manager should be telling you that.
Also, supporting it seems non-trivial. So it may take a while.

To workaround this issue, install the fallowing packages at v5:
npm install -D @typescript-eslint/eslint-plugin@5 @typescript-eslint/parser@5

With Yarn v2+'s yarn patch eslint-config-standard-with-typescript I was able to get this config working with TypeScript ESLint v6, see here for the patch https://github.com/retrixe/ecthelion/blob/main/.yarn/patches/eslint-config-standard-with-typescript-npm-37.0.0-e24fa6a048.patch
Presumably npm patch-package can do this for you as well.

@retrixe: please do a pull request on that repo including the code changes in the patch.
The package maintainer(s) as well as their users will be thankful to you for having made the groundwork to start supporting @typescript-eslint version 6.x.x

@retrixe: please do a pull request on that repo including the code changes in the patch. The package maintainer(s) as well as their users will be thankful to you for having made the groundwork to start supporting @typescript-eslint version 6.x.x

#1222


right click menu glitch.

Goes away after second right click.

Darn, so that bug definitely still happens... thanks for letting me know. I'll keep this issue open until I'm confident I've fixed the actual root issue. (つ﹏<。)


[Proposal]: Update Config type to match deis/workflow

Since https://github.com/deis/workflow/pull/298 is this can be map[string]string{}

we are currently using https://github.com/deis/controller-sdk-go which is using map[string]string{}


Add transitive GetComponent methods

Fixes #28
PTAL, it also adds const versions for Get* that were missing.

Using this commit with Unvanquished, I do get compile errors:
In file included from /home/eris/src/Unvanquished/src/sgame/backend/../components/DeferredFreeingComponent.h:28:0,
                 from /home/eris/src/Unvanquished/src/sgame/backend/CBSEComponents.h:6,
                 from /home/eris/src/Unvanquished/src/sgame/backend/CBSEEntities.h:11,
                 from /home/eris/src/Unvanquished/src/sgame/CBSE.h:24,
                 from /home/eris/src/Unvanquished/src/sgame/Beacon.cpp:28:
/home/eris/src/Unvanquished/src/sgame/backend/../components/../backend/CBSEBackend.h:721:18: error: 'TeamComponent& AlienClassComponentBase::GetTeamComponent()' cannot be overloaded
   TeamComponent& GetTeamComponent();
                  ^~~~~~~~~~~~~~~~
/home/eris/src/Unvanquished/src/sgame/backend/../components/../backend/CBSEBackend.h:696:18: error: with 'TeamComponent& AlienClassComponentBase::GetTeamComponent()'
   TeamComponent& GetTeamComponent();
                  ^~~~~~~~~~~~~~~~
/home/eris/src/Unvanquished/src/sgame/backend/../components/../backend/CBSEBackend.h:722:24: error: 'const TeamComponent& AlienClassComponentBase::GetTeamComponent() const' cannot be overloaded
   const TeamComponent& GetTeamComponent() const;
                        ^~~~~~~~~~~~~~~~
/home/eris/src/Unvanquished/src/sgame/backend/../components/../backend/CBSEBackend.h:697:24: error: with 'const TeamComponent& AlienClassComponentBase::GetTeamComponent() const'
   const TeamComponent& GetTeamComponent() const;
                        ^~~~~~~~~~~~~~~~

The output continues "like this" with lots of errors, all of them seem to be related to the TeamComponent though. I'd propose to checkout the PRs branch within unvanquished if you want to get more details.
Here's an excert from entities.yaml, manually stripped down to components and entities that involve the TeamComponent directly (though not double checked if these are really all occurences):
components:
    Team:
        parameters:
            team: team_t
    Client:
        parameters:
            clientData: gclient_t*
        requires:
            Team:
    AlienClass:
        requires:
            Client:
            Team:
                team: !!plain TEAM_ALIENS
            Armor:
            Knockback:
            Health:
    HumanClass:
        requires:
            Client:
            Team:
                team: !!plain TEAM_HUMANS
            Armor:
            Knockback:
            Health:
    Buildable:
        messages:
            - PrepareNetCode
            - Die
        requires:
            Health:
            Thinking:
            Team:
    AlienBuildable:
        messages:
            - Damage
            - Die
        requires:
            Buildable:
            Team:
                team: !!plain TEAM_ALIENS
            Ignitable:
    HumanBuildable:
        messages:
            - Die
        requires:
            Buildable:
            Team:
                team: !!plain TEAM_HUMANS
    Spawner:
        messages:
            - Die
        requires:
            Team:
            Thinking:
entities:
    Client:
        components:
            Client:
            Team:
                team: !!plain TEAM_NONE

Thanks for the minimal repro case, I will TAL!

Uploaded a fixed version, thanks for the review!

Tested in Unvanquished, works just fine. Thanks!


Question:

I have a question about your extension:
I can't use chessbot on live chess.com .
The robot just show the first move .
I use version 0.5.12  .
Why ?
What should I do ?

Can you attach a screenshot please? Without usernames, please.

Closed as investigated. Get more info here: https://github.com/recoders/chessbot/issues/34#issuecomment-275859181


[Doc Improvement] [SEO Metadata Changes 2][2438788]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/deploy-and-publish/appsource/post-publish/overview.md
90
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit f140ba4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/post-publish/overview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/deploy-and-publish/appsource/post-publish/overview.md
92
4
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 672c9bb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/post-publish/overview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


os/exec: parallel Command.Start() can be very slow

What version of Go are you using (go version)?

$ go version
go version go1.16.3 linux/amd64

Does this issue reproduce with the latest release?
Yes (1.16.4).
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/nfs/users/nfs_s/sb10/.cache/go-build"
GOENV="/nfs/users/nfs_s/sb10/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/nfs/users/nfs_s/sb10/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/nfs/users/nfs_s/sb10/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/software/vertres/installs/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/software/vertres/installs/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.16.3"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="0"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build2872611298=/tmp/go-build -gno-record-gcc-switches"

What did you do?

$ go run main.go
Serial:
Start 0 took 252.173µs
Start 1 took 221.002µs
Start 2 took 231.627µs
Start 3 took 171.249µs
Start 4 took 223.431µs
Start 5 took 211.772µs
Start 6 took 169.167µs
Start 7 took 203.491µs
Start 8 took 198.665µs
Start 9 took 181.992µs

Parallel (8 cpus, GOMAXPROCS 8):
Start 0 took 282.599µs
Start 1 took 419.762µs
Start 2 took 423.152µs
Start 3 took 834.894µs
Start 9 took 1.081168ms
Start 6 took 2.307089ms
Start 4 took 3.174552ms
Start 8 took 3.222679ms
Start 5 took 7.438996ms
Start 7 took 14.173073ms

Where main.go is https://play.golang.org/p/VN5OqF23nDQ
What did you expect to see?
exec.Command.Start() times be consistent between serial and parallel runs.
What did you see instead?
Parallel exec.Command.Start() times can be around 70 times greater than serial ones.
Setting runtime.GOMAXPROCS() to greater than or equal to the number of concurrent Start()s doesn't help. Setting it to 1 does reduce the effect, but I still get a few starts in the multiple millisecond range.
A runtime trace seems to suggest this is caused by os.StartProcess taking a global mutex lock, I think as explained here: https://github.com/golang/go/blob/d050238bb653711b47335583c5425c9efec30e4e/src/syscall/exec_unix.go#L20-L65
But I wondered if there's any other way around this for this particular use case?

cc @bradfitz @ianlancetaylor

I think this is a dup of #23558.

Duplicate of #23558


Fix infrequent updating of tracker files

Currently metadata_index.json as well as manifest.json only update at the end of the scraping process. For large scraping processes chunk data should be updated into manifest.json as there is currently no provision to save the data into the JSON in the event of an error induced crash or killed process. Same applies to metadata.index.

Fixed alongside #2. The data scraped is stored into a chunk file in the case of an process interrupt or kill. The file name is stored into manifest.json as well.


Same updates in joyride component

*In the sass joyride component, the z-index of the element joyride-tip-guide was below the joyride-expose-wrapper element.
*In the js joyride expose background bug fixed
*Added in tip_location comment left and right options

?

Thanks!

Thx team


Validation is broken

Posting {} to /user results in a successful registration. Same with posting password-less, username-less or email-less registration in any combination. They produce broken users entries.
What is the expected behaviour here?

Bumping this issue. I ran across the same problem. From what I've gathered not a single value is being validated by sails-auth. Has anyone found a solution?


Download queue

Closes #23. Closes #17.
This also includes a number of UI tweaks. Please let me know if it aggrevated things on any other systems.

Thanks a ton for these improvements!


[docs] Process Parquet files

This PR adds a guide for how to process Parquet files:

Eager dataframes with pd/pl.read_parquet
Lazy dataframes with pl.scan_parquet
Read and query with DuckDB

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

PS: doc-builder generates (empty) CURL version of the examples, would be nice if this can be removed

Oh! I didn't think the doc-builder would generate cURL examples even though I didn't specify them in the <inferencesnippet>. @mishig25 is there a way to remove this?

@mariosasko, empty cURL example fixed in https://github.com/huggingface/doc-builder/pull/365!


Update doc about "push" in "Scalar list method"

The doc said: "You cannot push a list of values - only a single value." but I think you can already push a list of values already
The reference in my project:

Compare to the reference in the doc:
https://github.com/prisma/docs/blob/main/content/400-reference/200-api-reference/050-prisma-client-reference.mdx#reference-26

👋 we're cleaning up our backlog of issues and are closing everything older than two years. If you still feel that this issue is relevant, please feel free to re-open and our team will take a look. Thank you!


Ptime

I add a conditionnal compilation in syndic to compile with calendar by default or with ptime if it is available. I produce a good META and avoid the unix dependency with ptime.

May be it's time to update the CI (like Decompress).

  IMHO one of the two is sufficient (and since Calendar depends on Unix, I'd clearly prefer Ptime)

I wait the position of @Chris00 to switch to Calendar or Ptime :) .

I had a look and I think we can unconditionally move to Ptime.  ocaml.org code will need to be updated but that's OK.  One should also let the author of river know because it relies on Syndic.

Thanks for your feedback, I will improve the code with your comments :)


running craftassist agent throws key errors

Type of Issue
Select the type of issue:

[x] Bug report (to report a bug)
[ ] Feature request (to request an additional feature)
[ ] Tracker (I am just using this as a tracker)
[ ] Refactor request
[ ] Documentation Ask

Description
Detailed log here: https://www.internalfb.com/phabricator/paste/view/P413912944
Started getting this after rebasing this PR : https://github.com/facebookresearch/droidlet/pull/369 on main

closed in https://github.com/facebookresearch/droidlet/pull/397


Is it possible config quality.gradle in root project?

If we can implement this, we need not config every module for multi-module project.

For now, I am trying
subprojects {
    apply from: "$rootProject.projectDir/AndroidCodeQualityConfig/quality.gradle"
}

But I am getting this error
Could not get unknown property 'check' for ':commonLibrary' of type org.gradle.api.Project.

Okay, finally find a way.For anyone want to try, you can using this in your root build.gradle file:
subprojects {
    apply from: "$rootProject.projectDir/AndroidCodeQualityConfig/quality.gradle"

    afterEvaluate {
        check.dependsOn 'checkstyle', 'findbugs', 'pmd', 'lint'
   }
}

Notice: You should first remove check.dependsOn 'checkstyle', 'findbugs', 'pmd', 'lint' in quality.gradle.

I'll try to apply this, thanks!

We can add this to readme, to provide as a second approach.

@Piasy Just heads up, I am excluding android lint.
And I have create a pr for some compatibility issue.

If you needs lint in multi-module project , you need to wrap android closure in afterEvalate in quality.gradle.

I create a pr(https://github.com/Piasy/AndroidCodeQualityConfig/pull/6) for android lint, please have a look.


Password reset function

We need a function that allows users that submited an email to reset the password via that email adress

Admins can reset passwords.  There is no plan currently to include emailing into the engine. It could of course be realized as a plugin.


Add CI for testing WebNN XNNPACK backend on both Windows and Linux platforms

@fujunwei PTAL, thanks.

@fujunwei I'm checking CI failures with latest code, please hold on.

CI failures
ninja: error: '../../src/webnn/native/xnnpack/BackendXNN.cpp', needed by 'obj/src/webnn/native/sources/BackendXNN.obj'
is due to lack of xnnpack/BackendXNN.cpp and xnnpack/BackendXNN.h files
On #254

@lisa0314 PTAL, thanks

@BruceDai Sorry for make this error due to my patch. I have pushed a new PR to fix it. Thanks!

Thanks @lisa0314.
Now CI for XNNPACK backend can pass, @fujunwei PTAL, thanks.

Get it, merging.


TNBCGames Platform

Project Details
How would you describe your project?
The project provides services similar to the now inactive project listed here https://github.com/thenewboston-developers/Projects/issues/24.
The following items best describe the project:

A platform(web, mobile, desktop) where games and similar contents that use TNBC can be listed, downloaded or played.
Users will be able to:

buy games and gaming assets using TNBC
Play tnbc enabled games
Purchase game credits or passes for paid and freemium games listed on the portal
Connect their wallets that can be used for purchasing various items in-game through the platform
content creators can list their games and paid creative assets to earn revenue on the platform

Development and release of small to medium scale games that users can play, interact with others (leaderboard, small scale multiplayer challenges, and transact in using TNBC)

What problem does this project solve?

Slower onboarding of new users to the community who might be targeted through games and other forms of entertainment
Lack of ready to use infrastructure for game developers and publishers joining the community

Who is the target market?

New users who are interested in earning TNBC
Users who like games
Developers who are interested in contributing to the growth of the community through creative arts
Game developers or publishers who are looking to join the TNBC ecosystem

How will this project benefit the network and/or the community?

New users who don't have TNBC will be excited to earn TNBC through gameplay or direct contribution
Spreading of coins, new wallets to increase the total number of holders, awareness, and publicity
Increased coin value due to transactions and activities happening on various small games as well as on the platform.
Alternative way for  people to become a part of the community

How is this project centered around thenewboston?

The project uses TNBC as the main currency on all games that exist on the platform as well as enables a large number of people to become coin holders including creative content creators and casual gamers.
The project will continue to support TNBC as the main form of payment and transaction now and in the future if other coins are onboarded to the platform.
The main goal of the platform is to ensure the distribution of TNBC to users and contributors in an attempt to expand the reach of the project

What is the estimated completion date for this project? (rough estimates are acceptable)
We estimate to complete the basic web app for content distribution in 12 Weeks
Please include a detailed roadmap for your project below. This must include a list of all milestones where payments will be awarded upon completion of each.
Roadmap & Milestones
Roadmap for TNBCGames platform

SN
Roadmap Name
Estimated Duration

1
Content creators
5 Weeks

2
Wallet and Players
3 Weeks

3
Points, Rewards, fees
2 Weeks

4
Blockchain Beta Updates
2 Weeks

5
Desktop version
6 Weeks

6
Mobile version
6 Weeks

Detailed roadmap here https://docs.google.com/document/d/1TrJwNEPlwqrKxovIgJIfCW1jGubgIWzf4lBTedARxes/edit
What is your Discord username?
zinoadidi > zinoadidi#1554
Additional files and documentation including architectural diagrams, wireframes, research documents, visuals, etc… (optional)
sample similar platform for games

https://www.miniclip.com/games/en
https://www.epicgames.com/store/en-US/

Milestone Details
Please include a detailed breakdown of the first milestone including a list of all quantifiable objectives and the estimated coin value of each. Upon completion of the milestone, each objective will be reviewed by thenewboston team
and once all objectives have been verified as completed, coins will then be rewarded.
Note that all future detailed milestone breakdowns will be written as a separate GitHub issue. Only the first one is
included in the project proposal.

Milestone Description
For the first milestone set of milestones, we will focus on implementing flow for our content creators
Requested Payment Upon Completion
0 TNBC
Milestone 1

#
Objective
Estimated Value (coins)
Estimated Time

1
authentication (register)
...............
.....hours

2
authentication (login)
...............
.....hours

3
personal profile & stats / dashboard page
...............
.....hours

Milestone 2

#
Objective
Estimated Value (coins)
Estimated Time

1
portfolio page / my contents
...............
.....hours

2
Create, read, update, delete content listing
...............
.....hours

3
All content on portal search and filter page

Calculation used for assigning tnbc to tasks
1 Minute of work time =  16  tnbc
1 hour of work time = 1000 tnbc
Proposed Rate:

Project Reviewer
null
Project Wallet
19bd6c062036367708fb99abb345ead9249a5301122ec35ca98cd2f366dca476

Repo URL
https://github.com/TNBCGames/FlappyTNBC
https://github.com/TNBCGames/TNBCGames

what is the estimate (in hours) for each objective? Also, is there a way to separate the objectives into smaller chunks?

@angle943 I broke down the tasks into smaller chunks as requested and also improved the proposal. Introduced separate document to make milestones and roadmap a bit clearer.
Concerning hours estimate, for this particular set of milestones, it involves creative work, all the brainstorming and back and forth that goes into it cannot be properly packaged into hours that make sense in terms of calculating rewards either for TNB or for the worker.
The coins are estimated as an agreed fee to pay the person that works on the set of creatives.
I have added the calculation used for determining the amount of coins to distribute and In addition, I have added to the roadmap some duration for the work to be completed

new project lead @dorcasduel

The government has reviewed this project, and we like the idea, but a few points:

For legal reasons, let's remove all instances of US dollar conversion rate.
Lets replace the first milestone to be a wireframe of how the web page will be structured. Here are a few examples of that:

Sitemap_Wireframe.pdf
Landing_Page_Wireframe.pdf
Also, since @dorcasduel is going to be the new project lead, I am going to close this issue, and request that Dorcasduel open a new project proposal with the above proposed changes

Project DetailsHow would you describe your project?The project provides services similar to the now inactive project listed here #24.The following items best describe the project:A platform(web, mobile, desktop) where games and similar contents that use TNBC can be listed, downloaded or played.Users will be able to:buy games and gaming assets using TNBCPlay tnbc enabled gamesPurchase game credits or passes for paid and freemium games listed on the portalConnect their wallets that can be used for purchasing various items in-game through the platformcontent creators can list their games and paid creative assets to earn revenue on the platformDevelopment and release of small to medium scale games that users can play, interact with others (leaderboard, small scale multiplayer challenges, and transact in using TNBC)What problem does this project solve?Slower onboarding of new users to the community who might be targeted through games and other forms of entertainmentLack of ready to use infrastructure for game developers and publishers joining the communityWho is the target market?New users who are interested in earning TNBCUsers who like gamesDevelopers who are interested in contributing to the growth of the community through creative artsGame developers or publishers who are looking to join the TNBC ecosystemHow will this project benefit the network and/or the community?New users who don't have TNBC will be excited to earn TNBC through gameplay or direct contributionSpreading of coins, new wallets to increase the total number of holders, awareness, and publicityIncreased coin value due to transactions and activities happening on various small games as well as on the platform.Alternative way for people to become a part of the communityHow is this project centered around thenewboston?The project uses TNBC as the main currency on all games that exist on the platform as well as enables a large number of people to become coin holders including creative content creators and casual gamers.The project will continue to support TNBC as the main form of payment and transaction now and in the future if other coins are onboarded to the platform.The main goal of the platform is to ensure the distribution of TNBC to users and contributors in an attempt to expand the reach of the projectWhat is the estimated completion date for this project? (rough estimates are acceptable)We estimate to complete the basic web app for content distribution in 12 WeeksPlease include a detailed roadmap for your project below. This must include a list of all milestones where payments will be awarded upon completion of each.Roadmap & MilestonesRoadmap for TNBCGames platform

SN
Roadmap Name
Estimated Duration

1
Content creators
5 Weeks

2
Wallet and Players
3 Weeks

3
Points, Rewards, fees
2 Weeks

4
Blockchain Beta Updates
2 Weeks

5
Desktop version
6 Weeks

6
Mobile version
6 Weeks

Detailed roadmap here https://docs.google.com/document/d/1TrJwNEPlwqrKxovIgJIfCW1jGubgIWzf4lBTedARxes/editWhat is your Discord username?zinoadidi > zinoadidi#1554Additional files and documentation including architectural diagrams, wireframes, research documents, visuals, etc… (optional)sample similar platform for gameshttps://www.miniclip.com/games/enhttps://www.epicgames.com/store/en-US/Milestone DetailsPlease include a detailed breakdown of the first milestone including a list of all quantifiable objectives and the estimated coin value of each. Upon completion of the milestone, each objective will be reviewed by thenewboston teamand once all objectives have been verified as completed, coins will then be rewarded.Note that all future detailed milestone breakdowns will be written as a separate GitHub issue. Only the first one isincluded in the project proposal.Milestone DescriptionFor the first milestone set of milestones, we will focus on implementing flow for our content creatorsRequested Payment Upon Completion0 TNBCMilestone 1

Objective
Estimated Value (coins)
Estimated Time

1
authentication (register)
...............

2
authentication (login)
...............

3
personal profile & stats / dashboard page
...............

Milestone 2

Objective
Estimated Value (coins)
Estimated Time

1
portfolio page / my contents
...............

2
Create, read, update, delete content listing
...............

3
All content on portal search and filter page

Calculation used for assigning tnbc to tasks1 Minute of work time = 16 tnbc1 hour of work time = 1000 tnbcProposed Rate:Project ReviewernullProject Wallet19bd6c062036367708fb99abb345ead9249a5301122ec35ca98cd2f366dca476


How to change the labels font property (font-size, font-color, etc) in a radar type chart?

There is lots of documentation on this topic.
See:

http://www.chartjs.org/docs/#radar-chart
http://www.chartjs.org/docs/#scales-radial-linear-scale

In this example:
http://www.chartjs.org/docs/#radar-chart
What properties should I modify to change things like (fontSize, fontColor, etc) from the [eating, running, cycling, etc] labels around the radar? Is not clear to me just reading the docs.

FYI, incase anyone else stumbles into the issue, the "scales-radial-linear-scale" page linked above is helpful.
Use the options.scale.pointLabels section to define the font settings of the labels which appear around the outside of a Radar chart.
Spent 30 minutes fumbling around in the documentation before finding this post.
I think that the Radar documentation could benefit by including a link to the linear scale page, and agree that if the examples were to contain a few more options then some of this could be made clearer.


Swift Sneak

Is there a bug with this mod because swift sneak doesn't work for me

Make sure you're on the latest version, since I fixed that bug recently


Test: Change timeout to 20s from 60s to fail faster

Feature Request
Most tests right now finish within 10 seconds and the maximum time is 13 seconds.
I think it make sense to lower it from 60s to 20s. So, if a test failed waiting for some elements, it will fail at 20s. If multiple tests are failing, we will see the result 3x faster.

[Enhancement]

As long as it doesn't cause more timeouts.. :/

Closing this as resolved.


iOS 9 app transport security

Using 1.7 and I'm ok with it. Now my iOS 9 compiled app cannot connect anymore using HTTPS, because the new iOS 9 app transport security requirements:

These are the App Transport Security requirements:
The server must support at least Transport Layer Security (TLS) protocol version 1.2.
Connection ciphers are limited to those that provide forward secrecy (see the list of ciphers below.)
Certificates must be signed using a SHA256 or greater signature hash algorithm, with either a 2048-bit or greater RSA key or a 256-bit or greater Elliptic-Curve (ECC) key.
Invalid certificates result in a hard failure and no connection.
These are the accepted ciphers:
TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384
TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256
TLS_ECDHE_ECDSA_WITH_AES_256_CBC_SHA384
TLS_ECDHE_ECDSA_WITH_AES_256_CBC_SHA
TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA256
TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA
TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384
TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256
TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384
TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256
TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA

Is the latest version supporting that or is there a way to make 1.7 fit for that?

Uhm.. maybe I'm on the wrong track here and this is an issue for Jetty (?)

Probably jetty, yes. You can try changing the dependency?
Since we unfortunately can't support < 2.0, I'm closing this, but please reopen if you upgrade and the issue persists.


RSDK-6041 - Add dial timeout option and default dial timeout

see https://github.com/viamrobotics/goutils/pull/240
default timeout is 5s

[x] will need to update rpc version once its out

@njooma added a constants file, not sure if TS has a different convention

@njooma added a constants file, not sure if TS has a different convention

I'm cool with this -- will defer to @maximpertsov since he's the architect behind this.

@njooma added a constants file, not sure if TS has a different convention

I'm cool with this -- will defer to @maximpertsov since he's the architect behind this.

Making it a constant is a good idea - I'm indifferent on whether to have a separate file for them or to collocate in dial.ts


[Possible Feature] WindowScroller, provide custom scroll event hook (e.g for using WindowScroller inside nested scrollable container)

Hey, I have a question about possible feature.
Say I have layout similar to this one: https://jsfiddle.net/JonMoore/LLfcL28u/19/ - that is: fixed top header and scrollable content/container. I want to use WindowScroller inside the content to display my data (instead of nesting table inside the content which already has one scrollbar). Right now, the WindowScroller listens to scroll events on the window, but these are not fired up while I scroll the content container. I'm considering implementing this feature for my usecase, where I can provide a way to listen for scroll events other than just on window (in my case, scroll events come from the content container, in the linked jsfiddle that is the part with 'slides'). So, I have a 3 questions:

Does anybody can comment, whether this won't destroy performance?
Maybe this already can be implemented but I failed to see the solution. If so, I'd appreciate pointing me in the right direction :)
If/when I implement this feature for myself, should I make a PR with it this feature or you're not interested in such use case and I should rather provide simple example, how one can achieve this with some hacks?

Thanks!

Hi @iyn,
Not sure I understand the use-case fully. If you have a fixed header like that example, why not just use AutoSizer instead? WindowScroller is intended for when you want the header to scroll out of view as a user scrolls down the body. AutoSizer also avoids the scroll-container event you mentioned- since Grid (or whichever virtualized component you use) manages scrolling itself.

As I understand, without the WindowScroller, the table will have its own scrollbar (which I want to avoid)?
I want to achieve something similar to this layout: https://app.dealroom.co/companies but in my case, the table itself would be in a scrollable container (but table itself should not have scrollbar), like in the layout I posted in the first post. Let me know if this is still confusing, I'll provide some kind of graphics/diagram. I'm a little bit tired atm, so my explanation is/could be chaotic.

OK, I decided to change my layout to easily incorporate react-virtualized WindowScroller into my project, so I'm closing the issue. Thank you for your time!

No problem @iyn. I'm pretty focused on getting version 8 RC out the door at the moment but I'll give this a little more thought once that's done.


Upgrade micropromise to ~0.4.0

With micropromises ~0.3.0 I get an error when running the suite:
/home/raivo/Downloads/promises-benchmark/node_modules/micropromise/index.js:215
        if(!this._state){
                ^
TypeError: Cannot read property '_state' of undefined

After upgrading to ~0.4.0 (0.4.6 to be exact), this error went away.

Thank you.

Oh, sorry. I should've explained that I'm not maintaining this anymore.


[Question] Using without any package manager

How can I use it without any package manager, with just plain HTML/ js? Any CDN available?

Hi,
no CDN available, but maybe https://unpkg.com/ works for you.

Ok. Thanks for the information

For anyone else looking for this library on a CDN<EMAIL_ADDRESS>


timed addRobotChatResponse

It would seem that addRobotChatResponse is not  considering  robot.robotResponseTime  option.
Is there a way to make it consider it or, generically, having addRobotChatResponse with a delay that actually interrupts the flow ?
Alternatively, is there an html form element that outputs a message but doesn't require input from the user, so it can be processed in the form flow?
The following code is one of the examples, altered to show the topic:
https://codepen.io/anon/pen/eopqXZ
var flowCallback = function(dto, success, error) {
  console.log("dto....", dto, success, error);

  /*
                The concept is that this could just as well be an AJAX request to your server where the response would yield a new qeustion that would be injected into the form.
                 */
  conversationalForm.addRobotChatResponse("first message")
    conversationalForm.addRobotChatResponse("second message")
    conversationalForm.addRobotChatResponse("third message")

  if (window.incrementedValue < 3) {
    window.incrementedValue++;
    window.ConversationalForm.addTags([
      {
        tag: "select",
        name: "country",
        "cf-questions":
          "Where are you from? (I have asked you this " +
          window.incrementedValue +
          " time(s))",
        "cf-input-placeholder": "country code",
        multiple: false,
                children: [
          { tag: "option", "cf-label": "USA", value: "usa" },
          { tag: "option", "cf-label": "UK", value: "uk" }
        ]
      }
    ]);
  }

  // We assume every answer is valid. If that wasnt the case we would call error()
  success();
};

// called from examples code
window.incrementedValue = 0;
var conversationalForm = window.cf.ConversationalForm.startTheConversation({
  formEl: document.getElementById("form"),
  context: document.getElementById("cf-context"),
  flowStepCallback: flowCallback,
  submitCallback: function() {
    conversationalForm.addRobotChatResponse("You are done. Thank you.");
  }
});

@stormsson we have the <cf-robot-message /> tag. Take a look here.

Thank you @jenssogaard  once more your tool is very helpful!

@stormsson thank you. I appreciate the questions and positive feedback. Let me know if you have other questions.


Protz m1

https://stackoverflow.com/questions/67378106/mac-m1-cping-binary-over-another-results-in-crash has the details, but basically cp'ing is not cool anymore and the target file needs to be removed first so that a new inode can be allocated.

This one seems to subsume the previous pow2 PR (unintentional?)

My bad. I've pushed a clean diff, and @franziskuskiefer confirmer that this fixed the build on his machine, too.

thanks!

This has also solves the issue on my machine. Thanks @msprotz for the PR!


Flask-SQLAlchemy and Database Migrations (try 2)

This is replacing #556, as for some reason github wasn't removing the merged commits and thus the diffs were bigger than they had to be.
Ian's (@zenhack) comments from #556:

This isn't quite ready yet, but it's worth having folks start looking at it. This is both moving over to the flask sqlalchemy extension and adding a database migration infrastructure. It includes a migration script that upgrades the content of my auth pr (#491) to the contents of the latest commit on this repository. Note that it does not handle upgrading the database for the OBM work that @SahilTikale did, since that was merged first.
It also includes some changes to the travis stuff; we run tests/stress.py, and we run the tests against both sqlite and postgres.
It does not yet include automated testing for the database migration infrastructure; I wanted to run my ideas about how to do that by you guys first:
Write a function like initial_db in tests/unit/api/auth.py (or just use that one), which populates a db with some useful objects.
Before making any changes to the schema, use said script to populate a postgres database, then dump the database as a sql script (via pg_dump). Commit the result.
Make your changes, and add a migration script.
Testing will consist of:
    run the sql script to populate a (previously empty) database.
    run the migration script, to upgrade to the latest schema.
    run a series of tests that expect said initial database.

I'm also still trying to pin down how the schema changes due to adding and removing extensions should work. Alembic has some support for branching and merging schema, but I need to do a bit more reading before I can decide what I think the right thing to do is there.
I also need to write some developer docs describing how to do database migrations.
@SahilTikale, @henn, let me know what you think.

I went through the PR. There are many new files introduced and it is hard to see how they relate to each other. @zenhack  Can you provide a simple work flow on how this migration would work so that I can understand the inter-relationships better. Will help me to provide quality feedback.

@SahilTikale, I've tried to answer your specific questions. I'll put together a more extensive developer doc, but as far as a guide to what's in the PR, there are two big things:

Moving over to the Flask-SQLAlchemy plugin. This doesn't really provide any single big piece of functionality, but it integrates the library we're already using more tightly with Flask, and lets us remove a bit of the glue/support code. For example, it makes a request-local session itself, so my hand-rolled session management stuff is gone. Per the comments I added to tests/stress.py, this makes the specific bug that was written for no longer something our own code is responsible for. It also auto-generates table names, which I'd written a small amount of custom stuff to do for us before (and which was part of our root level Model class). Per the comment in one of the migration scripts, the names it choose are not quite the same in all cases, so the schema needs to be tweaked to support the new names. Because of the fact that this is now covered by the library, I removed our own Model and AnonModel classes, and explicitly put the id and label fields in each table, since the two lines of code seemed not worth the abstraction without the other stuff that was in those superclasses. A lot of the line-by-line changes in this patch are just prepending db. to things, a fairly mechanical but pervasive change. Because of this, basically every file that declares a model has a lot of superficial changes in it. Similarly, basically every instance of local.db became db.session.
Integrating database migration support. We're using Flask-Migrate, which integrates Alembic with Flask in the same way that Flask-SQLAlchemy integrates sqlalchemy itself. The developer docs I'm going to write will tell you to go read those docs for a general overview, and then just explain HaaS specific stuff, so worth looking at those up front anyway.

Those two big things happen in that order in the history; most of the migration related commits are in the latter half. There are a handful of tiny cleanups in their own commits too; I tried to make sure those were particularly well explained and separated since they were otherwise out of place, but stuff like adding .cache to .gitignore didn't seem worth its own pr. There aren't too many of those though.

797a057 adds some high-level docs, hopefully they're useful for understanding what's going on.
//cc @henn, @SahilTikale.

@gsilvis, would you have a look at this?

@henn, I've either implemented or replied to each of your comments. Please have another look.

Thanks for all this great work, Ian.
Besides the comments I made, one other request is that we add a DB migration test in order to ensure that applying all of the alembic updates works. This would ideally detect times when folks change the DB without writing migrations as well as bad migration scripts themselves.
Perhaps this could work when running the postgres tests by creating a DB from the output of a pg_dumpall run on an old version of the DB, applying all of the migration scripts, then running some set of tests.
What do you think?

@henn, re: testing, I think I suggested something similar way up top, asking you to weigh in. Since we're of similar mind I'll get started on that.

@henn, I've addressed everything but the testing. I'm going to go back to bed, I'll get to that soonish.

@henn, the testing stuff is in place; please have a look.
Additionally, I've pushed a4db137, which obviates (and removes) conftest.py; I'd thought it was weird that it was scanning that file in the first place.

Thanks. All set for comments on this pass.
I'm really excited that we have test that verify the schema can migrate!

@henn, I think that's everything.

I'm really excited we have a test to verify that the schema can migrate!

Me too!

Fixed some trivial merge conflicts.

+1

Quoting henn (2016-04-06 23:54:24)

+1

Great! Let's get a second reviewer on this. @SahilTikale, care to have
another look?

@pjd-nu, @gsilvis, @kylehogan, @SahilTikale, a second review from any of you would be welcome.

I am a little occupied right now, will do it at the earliest

I looked at the latest 4 commits and they look good.

@pjd-nu, don't know if that counts as a sign-off or not. If so, please go ahead and merge.


Using the css prop with a value that is a template literal fails to compile

Describe the bug
This fails:
const gridSize = 4;
const Div = () => <div css={{
  padding: `0 ${gridSize}px`,
}} />;

This works, however:
const gridSize = 4;
const padding = `0 ${gridSize}px`;
const Div = () =>
;

thanks!

hey just tested this out locally - this works for me - what do you see?

Ah sorry, maybe my reduced case was too simple. Try it in a block with another rule after it. It looks like it's not appending the semicolon. Here's my real-world use case:

And here's the error I get:

Cheers will fix 🙏
On Wed, 27 May 2020, 7:28 pm Jared Crowe<EMAIL_ADDRESS>wrote:

Ah sorry, maybe my reduced case was too simple. Try it in a block with
another rule after it. It looks like it's not appending the semicolon.
Here's my real-world use case:
[image: Screen Shot 2020-05-27 at 7 24 09 pm]
https://user-images.githubusercontent.com/4995433/83001886-ae93be00-a04f-11ea-8701-eee7529b0e67.png
And here's the error I get:
[image: Screen Shot 2020-05-27 at 7 24 42 pm]
https://user-images.githubusercontent.com/4995433/83001922-bc494380-a04f-11ea-81ab-0ee84654a5cb.png
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/atlassian-labs/compiled-css-in-js/issues/205#issuecomment-634542738,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABT4PHISQI2I25N6MV3PRDTRTTMLZANCNFSM4NLXWXBQ
.

Fix for it here https://github.com/atlassian-labs/compiled-css-in-js/pull/206
Turns out when moving from Stylis to PostCSS it's a bit more stricter about semi-colons - which TBH is a good thing.
Also you should join the Atlassian Orgs on Github! http://go/opensource


Error message modified

Codecov Report

Merging #77 (cfa3aff) into develop (2f2896f) will decrease coverage by 16.55%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##           develop      #77       +/-   ##
============================================
- Coverage    61.36%   44.80%   -16.56%
============================================
  Files           54       52        -2
  Lines        16986    14454     -2532
============================================
- Hits         10423     6476     -3947
- Misses        6563     7978     +1415

Impacted Files
Coverage Δ

core/src/storage_manager/storage_manager.cc
31.97% <0.00%> (-9.45%)
:arrow_down:

core/include/misc/utils.h
0.00% <0.00%> (-100.00%)
:arrow_down:

core/include/codec/codec_lz4.h
0.00% <0.00%> (-100.00%)
:arrow_down:

core/src/misc/progress_bar.cc
0.00% <0.00%> (-96.78%)
:arrow_down:

core/src/codec/codec_lz4.cc
4.16% <0.00%> (-85.12%)
:arrow_down:

core/src/storage_manager/storage_gcs.cc
1.81% <0.00%> (-75.61%)
:arrow_down:

core/src/storage_manager/storage_azure_blob.cc
0.38% <0.00%> (-73.89%)
:arrow_down:

core/src/codec/codec_filter.cc
28.57% <0.00%> (-71.43%)
:arrow_down:

core/include/storage_manager/storage_buffer.h
0.00% <0.00%> (-71.43%)
:arrow_down:

core/include/storage_manager/storage_azure_blob.h
0.00% <0.00%> (-71.12%)
:arrow_down:

... and 37 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 2f2896f...cfa3aff. Read the comment docs.


Add optional oauth features

This patch fixes some bugs and adds some optional oauth features. I can break it up into multiple PRs if you wish. The following changes are included:

Normalize around using crypto.Hash constants
Rename SHA1 Signer to HMAC Signer
Add support for the SHA256 hash
Move "Authorization" token into a constant
Fix assert ordering in provider test
Factor parseBody out of RoundTrip, update provider.IsAuthorized to use helper functions
Add error messages to provider.
Fix missing unescape in provider, this causes signature mismatches if any of the oauth_ params have encoded elements. (Test in following commit.)
Add support for oauth_body_hash extension: https://oauth.googlecode.com/svn/spec/ext/body_hash/1.0/oauth-bodyhash.html

Thanks for the great patch!


add feature extraction and some similarity measures

@fhamborg the log output is still too long, cf.
https://api.travis-ci.org/jobs/183946964/log.txt?deansi=true

The log length has exceeded the limit of 4 MB (this usually means that the test suite is raising the same exception over and over).

Coverage decreased (-0.08%) to 60.829% when pulling 9edcae063354fbed06312114f4cffa76710e9ed4 on pd-features-dev-fh into 5cd3f387de341448a6377c84f874f1a858de0cb0 on master.


sqlite returning_clauses_for_sqlite_3_35 DatabaseError(Unknown, "near "RETURNING": syntax error")'

Setup
Versions

Rust: rustc 1.64.0 (a55dd71d5 2022-09-19)
Diesel: diesel 2.0.0
Database: sqlite v3.39.4
Operating System rust:latest

Feature Flags

diesel: [ "sqlite", "returning_clauses_for_sqlite_3_35",]

Problem Description
When running locally the RETURNING statement works correctly.  When running inside a Docker image it throws an error: DatabaseError(Unknown, "near \"RETURNING\": syntax error")'
Steps to reproduce
I have created a reproduction repository with a minimal setup here: https://github.com/raed667/sqlite-returning-docker
Checklist

[x] I have already looked over the issue tracker  and the disussion forum  for similar possible closed issues.

[x] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)
[x] This issue can be reproduced without requiring a third party crate

Thanks for providing a reproducible example. This always helps finding the underlying issue.
I've applied the following path to your repo to get the linked sqlite version.
diff --git a/src/main.rs b/src/main.rs
index ad0cbe0..bd615e4 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -5,9 +5,13 @@ mod db;

 fn main() {
     println!("Hello, world!");
+    use diesel::prelude::*;

     let connection = &mut establish_connection();

+    let version = diesel::dsl::sql::<diesel::sql_types::Text>("select sqlite_version()").get_result::<String>(connection).unwrap();
+    dbg!(version);
+
     let user = insert_user(connection);

     println!("Inserted user id={}", user.id)

This returns the following output:
[src/main.rs:13] version = "3.34.1"

This output indicates that there is another older libsqlite3 version in your container. This version is linked by rustc to your binary and it's too old to support RETURNING. You need to use the newer version. Checkout the documentation of libsqlite3-sys for how to setup the correct path.
Closed as this is not an issue in diesel.

Thanks a bunch for the help !

Just in case someone reads this in the future. I added the following line to Cargo.toml
[dependencies]
rusqlite = { version = "0.27", features = ["bundled"] }

Which basically bundles sqlite with the library.


Fix process argument limit error when building on Windows

Building SwiftGodot with full list of generated sources on Windows exceeds the 32K symbol limit of CreateProcess resulting in
error: command Compiling Swift Module 'SwiftGodot' (899 sources) failed: unable to spawn process 'C:\Library\Developer\Toolchains\unknown-Asserts-development.xctoolchain\usr\bin\swiftc.exe' (The filename or extension is too long.)
Merging all generated sources into a single file should fix the issue for now until it's resolved from the SPM side (or maybe modularization of sources in the project).

Makes sense.
One possible optimization for Windows would be to split the sharedPrinter from the printer, but pass a flag indicating that the generated output is being blended.   And then concatenate the various outputs of printers at the end.
The sole reason for that is that it significantly reduces the build time.
Happy to merge in this way if you prefer.

Thank you for the update!


Extension API - How to determine debug state

Dear vscode-team
I've got another question for you guys.
Is there any possibility to determine if VSCode is actually debugging?
My idea is to do something like this:
If a file changes, AND vscode is in debug mode (with f5), I want to run a certain command.
As I'm aware of, one can define a when on a key binding but is this possible from the extension API?
Cheers
Chris

Yes there is a "when": "inDebugMode", however not sure if we surface when in our api or if we want to add that.
Thus forwarding to @jrieken

It would be a nice feature actually (if it's not already possible)
So one could add special behavior for certain actions.
Like, if my file changes and vscode is actually in debugging mode (or in a debugging session) I'd run the command workspace.debug.restart (just as an example, I know the actual command does have another name)

The when clause is not the right vehicle for this because it's just a condition under which commands are constraint by default. However, a user can always change things like a keybinding and skip/change the when clause as she/he likes.
What you want is debugging API like a vscode.debugger-namespace which have we not yet planned to do.

Yeah such a namespace would be nice to determine the state of vscode.
Something like vscode.debugger.isDebugging() maybe?
I would like to suggest this feature then 😄
IMO it could be very beneficial for some extensions to add features and / or behaviors based on the state of VSCode.
For now, I just can execute the command for restarting the debugger and see if there's an error.

There is now debug api that allows you to do this. Please try it out and provide in detail feedback. Thanks


Negative row

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.5.1

Platform/Target and Browser Versions: macOS

Live Link or JSFiddle/Codepen or website with isssue: PREFERABLY (IF YOU WANT YOUR ISSUE TO BE RESOLVED ASAP).

What you did
<Swiper
            slidesPerView={6}
            slidesPerGroup={6}
            slidesPerColumn={2}
            slidesPerColumnFill='row'
            spaceBetween={50}

using react-id-swiper.
Expected Behavior
No negative rows.
Actual Behavior
Negative row as in
...
.
The layout of slides is disturbed.
Reason?
https://github.com/nolimits4web/swiper/blob/master/src/components/core/update/updateSlides.js#L92

This one is working with Swiper 4.5.0 https://codesandbox.io/s/reactidswiper-problem-with-slidespercolumnfill-yj4p5

Updated sandbox to use 5.0.1 - still same issue. Slides get a negative order.


Realsense conveyor belt application

Required Info

Camera Model
{  D400 }

Firmware Version
(<IP_ADDRESS>

Operating System & Version
Ubuntu 18

Kernel Version (Linux Only)
(4.15.0  (with UVC patch)

Platform
PC

SDK Version
{ 2.30}

Language
{python }

Segment
{others }

Issue Description
Hi everyone,
I'm looking into the possibilities to use a Realsense (D435) for a conveyor application. So we have a conveyor belt with an encoder with a more or less constant speed (the speed can vary, I still have to gather more info about this). The RealSense will be used to inspect an object, and if the object has to be processed a robot arm will pick up the object. To classify the object, both RGB and depth information are taken into account.
I've seen many topics about timing and timestamps, but I couldn't find an answer to this question:
Even when the hardware sync is enabled (to sync RGB and depth), it seems that depth and RGB frames are not grabbed at the same moment (they run the the same interval, but not in sync). This can be seen in the depth and RGB images, where a shift in the direction of travel can be seen between both images. Is there a way to take both images at the same time, or at least minimize the time in between? Is there any documentation that describes when the images are taken exactly?
Also, for this application a frame-rate of about 2 fps is sufficient. The lowest possible settings is 6 fps, but it seems that the hardware sync does not work (I can't get RGB and depth framerate equal in the realsense viewer, for 15 and 30 fps it works with the right settings). Is this a known issue or am I doing something wrong?

Hi @ArjanVroegop
One approach you could check-out is using IR information instead of RGB. IR frames are always 100% synchronized with depth, temporally and spatially.
When using the RGB sensor, there is always going to be small differences in sensor exposure times and configuration, but the difference between the two should never exceed one frame. This means that you could run both streams at 60 fps, sample the SDK at the rate you need (say twice a second) and get up to 16 ms delay between the images (I think it would be actually up to 8 ms, but I'm not 100% sure)

Hi @ArjanVroegop
Let  us know if there is anything further needed after the outline from dorodnic.  If not then we can close out this thread.


Memory leak above v2.7.

Hello. We have been using imgproxy on moderate production load in kubernetes. Recently I've noticed significant amount of OOM Kill and Eviction events on our deployments. Increasing available memory would not help, It would just eat 100+Gb in less than an hour. I decided to back to 2.7 due to core image change and max_bytes  option (looked suspicious to me). That confirmed being stable in production with like 35x less memory used opposing to v2.8.1. Judging by the logs: I see that problems with mem usage started much earlier (like 30+ days ago) just they did not affect us much until load was there.

Important to add: we've been using latest image tag before the incident.

https://i.gyazo.com/39a8738db52c28bb34918d2056ba81e8.png
This is the screen of memory usage after downgrade to 2.7 (load remains the same)

I can confirm this. We jumped from 2.7 to 2.8.1 today and see the same "higher than normal" memory usage.

Same with my heroku instance. I did not manage to investigate where this comes from.

Hi everyone,
Sorry for the big delay, I was on new year vacation.
You can try the latest build. Also running the Docker image with the following env variables should solve the problem:
LD_PRELOAD=''
MALLOC_ARENA_MAX=2

This is not memory leak per se, this is how GLib works. jemalloc usage has been should solve memory growth but it seems like it didn't. MALLOC_ARENA_MAX=2 seems to work fine here.

Ty for advice, @DarthSim, I will give that a shot on Monday/Tuesday. But did why the issue evolve only at v2.8 did switch to GLib happened there?

I changed the base image from Alpine to Debian. Alpine uses musl instead of GLib, and its small stack size causes lots of segfaults that I'm tired of fighting. Also, imgproxy works a bit faster on Debian.
Debian uses GLib that has known problems with greedy memory allocation.

I added malloc_trim call to the latest build, this significantly reduced memory usage. I'll observe our instances, and if everything goes well, I'll make a release.

For us, v2.8.2 solves the issue. Thank you very much!

Not the case for us, without MALLOC_ARENA_MAX=2 already 4x mem usage from v2.7 and growing:

@freakst Do you use 2.8.2? It already has MALLOC_ARENA_MAX=2 set.

Yes we do,actually afer some time we've got stable usage ~ 70mb, no OOM events registered since rls.


Need your help

Hi, i found your ID from hotkeynet.readthedocs.io and I can read Chinese.
Since the official website has been down,I can't find any download link of the software.
Would you please help me and send me a copy of HotkeyNet.
My email address<EMAIL_ADDRESS>Thanks!

@Dickoem 你咋找到这个项目来了.. 咋不在那个项目下留言? 我明天发你一份. 你用来玩什么游戏? 我是用来单刷魔兽世界巫妖王团本.

主要是玩wow 谢谢你
------------------ Original ------------------
From: Sanhe Hu @.>
Date: Sat,Jun 3,2023 3:49 PM
To: MacHu-GWU/pyproject_ops-project @.>
Cc: Dickoem @.>, Mention @.>
Subject: Re: [MacHu-GWU/pyproject_ops-project] Need your help (Issue #1)
@Dickoem 你咋找到这个项目来了.. 咋不在那个项目下留言? 我明天发你一份. 你用来玩什么游戏? 我是用来单刷魔兽世界巫妖王团本.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

我都不记得了, 我发过你了吗?

并没有发我。。。
麻烦发我一份吧 谢谢 ~！
------------------ 原始邮件 ------------------
发件人:                                                                                "MacHu-GWU/pyproject_ops-project"                                                                                @.>;
发送时间: 2023年8月5日(星期六) 晚上9:14
@.>;
<EMAIL_ADDRESS>主题: Re: [MacHu-GWU/pyproject_ops-project] Need your help (Issue #1)
我都不记得了, 我发过你了吗?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@Dickoem https://github.com/MacHu-GWU/hotkeynet-project/releases/tag/0.1.4 到这里下载.

已经下载。感谢无私分享。这软件太老了，而且作者弃坑，我全网找了好久。
再次感谢~！
------------------ 原始邮件 ------------------
发件人:                                                                                "MacHu-GWU/pyproject_ops-project"                                                                                @.>;
发送时间: 2023年8月11日(星期五) 晚上10:13
@.>;
<EMAIL_ADDRESS>主题: Re: [MacHu-GWU/pyproject_ops-project] Need your help (Issue #1)
@Dickoem https://github.com/MacHu-GWU/hotkeynet-project/releases/tag/0.1.4 到这里下载.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

不客气, 但是我认为从强大程度上来说 灵活性 可扩展性, 设计模式角度来说, 它是最强的 比 iboxer 强多了. 它的上限极高


Update Track 2 Go documentation based on recent changes

We recently made a few changes to the Go SDK and need to update the following page:
https://docs.microsoft.com/en-us/azure/developer/go/management-libraries

Module path and dependencies update:

There is no ‘armcore’ library.
Module path change:  github.com/Azure/azure-sdk-for-go/sdk/resoucemanager/compute/armcompute
github.com/Azure/azure-sdk-for-go/sdk/azcore/to

API change of the constructor so the sample code needs to be updated.

There is no connection object now.
con := armcore.NewDefaultConnection(cred, nil)
use the below way to create a client.
rgClient := armresources.NewResourceGroupsClient(subscriptionID, credential, options)

@Duffney please take a look

One more questions related to https://docs.microsoft.com/en-us/azure/developer/go/azure-sdk-tools#dependency-management-with-dep, from https://github.com/golang/dep, it mentioned 'Dep is deprecated and archived in favor of Go modules'
Do we still need this?

Good question, I think we will remove it and mention Go modules instead

I apologize, I missed the notification. Looking into this now. :)

Pull request submitted with revision. :)

@Duffney, could you share the link of the PR?

@Duffney, could you share the link of the PR?

The changes have just been merged. It was created on an internal repo and it didn't appear you were inside the organization for the repo and I wasn't able to add you @lirenhe. Changes should be merged and made public within 48 hours. If they don't fully address your concerns please be sure to let me know. :)
Thanks!

Closing this issue, but please reopen if the changes do not fully address the concerns discussed here.

@nickzhums when you've got a moment, please review the changes, and close the issue if it's been resolved, thank you. :)

@Duffney thanks! looks good to me, I will let @lirenhe double confirm

@Duffney  thanks, overall it looks good.
A small comment:
typo in:  poller, err := client.BeginCreate(ctx, "resource_identifier", "additonal_parameter")