Error 400 on failed When mutate object

I am developing an android app using graphql, but unfortunately i always get error 400 when using this method
    final EditProfileMutation editProfileMutation =
            EditProfileMutation.builder()
                    .id("31f92055-9598-4706-8fdd-75b4db532a7e")
                    .first_name("anton4513")
                    .last_name("yaphiar")
                    .birth_date("2017-2-2")
                    .work_area("china")
                    .hk_id("1510369132")
                    .address("cilember270")
                    .main_language("Indonesian")
                    .second_language("Chinese")
                    .emergency_contact_number("1309135134")
                    .emergency_contact_name("132513513")
<EMAIL_ADDRESS>                    .phone("190367136")
                    .address_1("cilember 270")
                    .address_2("bandung2")
                    .price(5.0)
                    .avatar("hbibjbj")
                    .firebase_id("jtkjkd")
                    .optional_phone("519036813")
                    .wechat("uiui12")
                    .wa("uiu12")
                    .line("monster12")
                    .experience_country("chinea")
                    .experience_length(">6 years")
                    .special_skill("sleeping")
                    .hobby("killing")
                    .nationality("Indonesian")
                    .religion("muslem")
                    .about("just play")
                    .build();
    apolloCall = application.apolloClient()
            .mutate(editProfileMutation);
    apolloCall.enqueue(dataCallback);

with model
mutation EditProfile($id: ID!, $first_name: String!,$last_name: String!,$birth_date: String!,$work_area: String! ,$hk_id: String!,$address: String!,$main_language: String!,$second_language: String!,$emergency_contact_number: String!,$emergency_contact_name: String!,$email: String!,$phone: String,$address_1: String,$address_2: String,$price: Float,$avatar: String,$firebase_id: String,$optional_phone: String,$wechat: String,$wa: String,$line: String,$experience_country: String,$experience_length: String,$special_skill: String,$hobby: String,$nationality: String,$religion: String,$about: String){
updateWorkerProfile(id: $id,first_name: $first_name,last_name: $last_name,birth_date: $birth_date,work_area: $work_area,hk_id: $hk_id,address: $address,main_language: $main_language,second_language: $second_language,emergency_contact_number: $emergency_contact_number,emergency_contact_name: $emergency_contact_name,email: $email,phone: $phone,address_1: $address_1,address_2: $address_2,price: $price,avatar: $avatar,firebase_id: $firebase_id,optional_phone: $optional_phone,wechat: $wechat,wa: $wa,line: $line,experience_country: $experience_country,experience_length: $experience_length,special_skill: $special_skill,hobby: $hobby,nationality: $nationality,religion: $religion,about: $about){
id
email
}
}
Is error 400 is simply i am using wrong request or some thing ?
I am using gradle
classpath 'com.apollographql.apollo:gradle-plugin:0.4.1'

400 you mean Http status code 400, if yes then it's something wrong on sever side, maybe url of endpoint you are using is wrong.

Closing as this is an issue with your server not apollo. Please post on stack overflow for further help

@sav007 No the problem is not url of endpoint.
I have found that the problem is when the params too long it always become error 400, so i split these call to several call.
@digitalbuddha I am using apollo server with and several other working method. Just the param maybe too long.


feat: Provide tonic RPC client

Addresses the first point from #383

Most of the domain code and the code related to errors was copied from the node, removing any conversions and errors that could be removed.

Most of the domain code and the code related to errors was copied from the node, removing any conversions and errors that could be removed.

Were we actually using domain code in the client before? I thought we used only generated code + ConversionError.

Were we actually using domain code in the client before? I thought we used only generated code + ConversionError.

The domain code contains conversions from the types generated by prost to the domain structs, and viceversa.
These were used for converting incoming RPC responses into domain types (and also converting from domain types to protobuf types to mock the RPC endpoint).


Optimize  annotations.go deleteLabels()

Summary

deleteLabels() iterates every possible GVR for in every namespace in migration and runs a discovery client list() followed by discovery client update() on all list() results.
deleteLabels() will (presumably) start completely over in the next reconcile if it encounters a conflict error. If there are enough resources, the possibility of one resource having a conflictError is high, meaning this could stall forever.
deleteLabels() (aka EnsureLabelsDeleted) has been observed taking hours to complete on complex migrations.

Link to relevant code
Solutions to consider

Do we need to remove labels on every possible GVK? Can we just run removal over a subset of GVKs that we actually label?
Can we add label retries for conflictErrors?  This may allow us to get through this operation on the first try
Can we drop this function altogether? Does it matter if we leave labels around?

// deleteLabels will delete all migration.openshift.io/migrated-by labels
// from the application upon successful completion
func (t *Task) deleteLabels() error {
	client, GVRs, err := gvk.GetGVRsForCluster(t.PlanResources.DestMigCluster, t.Client)
	if err != nil {
		return liberr.Wrap(err)
	}

	listOptions := k8sclient.MatchingLabels(map[string]string{
		MigratedByLabel: string(t.Owner.UID),
	}).AsListOptions()

	for _, gvr := range GVRs {
		for _, ns := range t.destinationNamespaces() {
			list, err := client.Resource(gvr).Namespace(ns).List(*listOptions)
			if err != nil {
				return liberr.Wrap(err)
			}
			for _, r := range list.Items {
				labels := r.GetLabels()
				delete(labels, MigratedByLabel)
				r.SetLabels(labels)
				_, err = client.Resource(gvr).Namespace(ns).Update(&r, metav1.UpdateOptions{})
				if err != nil {
					// The part touches the application on the destination side, fail safe to ensure
					// this won't block the migration
					if k8serror.IsMethodNotSupported(err) {
						continue
					}
					return liberr.Wrap(err)
				}
			}
		}
	}

	return nil
}

Would like to find the concrete list of resources that we attach migration.openshift.io/migrated-by so that we can cut down on the number of unlabeling requests we send. Seems like every GVK may be  excessive.

Doing some initial profiling to understand how much time this step takes. Will write back here.

More context:

Label is added to every resource we restore in our plugin, merged in openshift-migration-plugin#58
Label is used in "deletedMigrated" to do a Migration Failure Rollback (which is disabled by default now)


UAT v0.3.3 | Localization missing in OCR error

Localization missing in the following OCR error:

hi @Ramu-kandimalla , message is coming from backend, we are getting message like "Not a valid CHEQUE_RETURN_MEMO"

hi  @rajeshcherukumalli Chnages are done, pls check
cc :  @Ramu-kandimalla

Hi @Ramu-kandimalla @rajeshcherukumalli Tested in UAT, Working fine,

https://jam.dev/c/4ffa2ea3-fa92-4e0a-a01f-a6399f22d9d0


Update jsondoc to generate multiple JSON files

This PR makes output of the jsondoc tool on the gcloud-jsondoc branch compatible with the current gcloud-common site JSON. It also changes its markdown library from Redcarpet to Kramdown, to keep up with the times and for consistency with the current YARD-generated docs.

Great work! This is a huge step towards getting the docs on the angular app!


Question: How to change datepicker header background color?

@shalugin you have to overwrite styles of it, you can also use our SASS to change the colors and theme too.

@dharmeshpipariya why me? Maybe @suhotskiy? :)

@shalugin it was by mistake. I have updated it.


How to get 80sps

Hi,
i soldered the rate pin to 3.3v but the rate i can get values in a loop stays the same. How do i get up to 80sps? (I am using a rpi zero)
Thank you

I did it, but it didn't change anything

If you measure voltage with multimeter, what is the voltage on the pin? Can you send a picture? If you try continuity with multimeter do you have it to VCC do you have it to Ground?
How often are you trying to read the HX711 with your code? Can you send a snippet of your code?
Thanks.


Implement the Search API

Implement Modrinth's search API
docs route

is there an ETA on this?

I was getting this error btw
thread 'main' panicked at 'Failed to search for projects: ReqwestError(reqwest::Error { kind: Decode, source: Error("invalid type: string \"LGPL-3.0-only\", expected struct License", line: 1, column: 655) })', src/main.rs:38:10

Now I'm getting a
error[E0425]: cannot find value `API_URL_BASE` in this scope
  --> /home/tehc/.cargo/git/checkouts/ferinth-9a03d96e970c4356/16d8339/src/api_calls/search.rs:18:13
   |
18 |             API_URL_BASE.join("search").unwrap(),
   |             ^^^^^^^^^^^^ help: a static with a similar name exists: `API_BASE_URL`
   |
  ::: /home/tehc/.cargo/git/checkouts/ferinth-9a03d96e970c4356/16d8339/src/lib.rs:36:1
   |
36 | pub static API_BASE_URL: Lazy<Url> = Lazy::new(|| {
   | ------------------------------------------------- similarly named static `API_BASE_URL` defined here

in the search branch. improperly merged?


Support mint

I searched existing issues for mentions of mint and only found #594 and #810.
glam is used internally which is kind of nice, but I find myself wishing I could take advantage of it more often than not. Today I nearly wrote my own trait to clamp a Vector3. If mint support was available, I could use that to do the clamping et al. with glam or any other crate I prefer.
(In the end, I decided to normalize and scale the vector after checking its length squared. But I think having the option to use mint might still be useful in other situations.)
Potential drawbacks and alternatives
As mentioned in #810, using mint as a public export will increase the MSRV to 1.52.1 (It is currently 1.51 as of #833.)
An alternative to mint would be making Vector3::glam and friends public at the cost of also increasing the MSRV to 1.52.1 without the direct benefits of mint. Implementing Deref to and from the glam types would be practically the same thing with just a different interface.
Either of these will also re-raise the concern from #594 of requiring version bumps for the public crate exports.
Another alternative would be writing a whole bunch of instance methods on the core types to bring its feature set closer to glam. This would be a divergence from the Godot API for these classes.

Out of curiosity, what are the glam features you're currently missing in the godot-rust vector types? You mentioned clamp, anything else?

Good question. Since I'm not even using the glam methods right now, I don't have a great answer. Looking over the docs, I could see these additional methods being useful in some situations:

min
max
fract
exp
powf
recip

And in general, it has some nice trait implementations like Product, Sum, Rem, and RemAssign. I don't know if the last two ops were just overlooked, or if Godot just doesn't offer anything similar for vector modulus.
Beyond those, there are all of the swizzles! Which are really common in shader code and have some important uses in mesh generation, noise, and other applications.

One thing that makes glam a bit difficult to expose is that it keeps releasing semver-breaking versions very frequently. It already is at v0.20 and seems to have published a new minor version almost every month last year. That of course is also nice in terms of fast iteration and progress, but for the problems we've faced in the past it's not ideal.
On the other hand, mint seems considerably more stable, being at 0.5.9 since 2017. However, at a closer look it also occasionally introduces breaking changes in a patch versions (e.g. layout change for #[repr(C)] in https://github.com/kvark/mint/commit/8e339915898e0c6f314600489866f7e42a0baf31, or increase of edition in https://github.com/kvark/mint/commit/7a90a796ef35e13708dec76d02caff25c985964c). These are minor things though and likely don't affect most users.

I've had a few semver breaks in dependencies affect pixels, so I understand the concern. Mostly these have been pushing my MSRV which is not ideal, but I don't know how it impacts users in practice. (I almost always use a recent nightly compiler in my personal projects. Mostly out of habit, but also because I actually like using recently stabilized features when they are available.)
For libraries, it is completely desirable to maintain compatibility with older compilers as much as possible.


RSpec - how to change format to more concise output

Hi Rspec team, your gem is awesome!
I can run RSpec like this and output is really large
$ rspec spec/interactors/reports/create_spec.rb
Run options: include {:focus=>true}

Randomized with seed 26531

  1) Reports::Create when params are valid successfully creates report
     Failure/Error: report.assignees.create!(profile: contact, confirmed: true, owner: true) if create_assignee?

     NoMethodError:
       undefined method `assignees' for nil:NilClass
     # ./app/interactors/reports/create.rb:25:in `create_report!'
     # ./app/interactors/reports/create.rb:10:in `block in call'
     # .. a lot of trace lines
Top 1 slowest examples (0.37224 seconds, 84.5% of total time):
  Reports::Create when params are valid successfully creates report
    0.37224 seconds ./spec/interactors/reports/create_spec.rb:52

Finished in 0.44068 seconds (files took 4.44 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./spec/interactors/reports/create_spec.rb:52 # Reports::Create when params are valid successfully creates report

I want to have the ability to output only these lines
$ rspec spec/interactors/reports/create_spec.rb
  1) Reports::Create when params are valid successfully creates report
     Failure/Error: report.assignees.create!(profile: contact, confirmed: true, owner: true) if create_assignee?
     NoMethodError:
       undefined method `assignees' for nil:NilClass
     # ./app/interactors/reports/create.rb:25:in `create_report!'

Without detailed stats in the end and detailed backtrace
The goal of this - I want to set up my guard notification gracefully with rspec => So that I will see an error in the OS notification without scrolling to the top of the terminal to see the reason for the error
In Rspec --help I found only notice about --format custom formatter class name , but how to use it ? ( Do you have any examples ?)

If you don't need Top 1 slowest examples part, I suggest adding --no-profile to your .rspec-local (or .rspec if your colleagues approve).
As for displayed line count, you may set max_displayed_failure_line_count to your liking.
The latter option is missing from the documentation on https://relishapp.com/rspec/rspec-core/. If you'd like to add it - a pull request is welcome https://github.com/rspec/rspec.github.io/
If you're still not satisfied enough, you may stab at adding a custom formatted.
Also, I suggest asking questions on StackOverflow and our mailing list http://rspec.info/help/. We're trying to exclusively use Github issues for bugs and feature requests.


Question: Compilation of Server Bundle

This is probably a stupid question but I'm just trying to figure out how to get up and running with this module as it looks very useful, but I'm confused at what the server method does. When I pass my ./server entry point as the second argument to new webpack-isomorphic-tools(), what is actually happening? Is it just relying on a webpack compilation of client code to have happened that output a webpack stat.json file into the root of the context directory, or does webpack-isomorphic-tools actually bundle (with webpack) and output my ./server bundle in a target=node webpack config environment. Sorry for my confusion but not sure I'm understanding the functionality of this module completely.

Well, to begin with, new webpack-isomorphic-tools() no more has a second argument (for a long time now).
When you call the server method it waits for stats json to be generated (for the first time), then activates require() hooks for the extensions you specified, and then runs your ./server entry point.
It just relies on that stats json file, and that's all.
It even doesn't know a thing called Webpack exists and it doesn't cooperate with Webpack or Webpack generated code.
Its counterpart though does because it's a webpack plugin, and its purpose is to generate that stats json file.
I didn't know about target node you pointed out.
That's an interesting reading.
http://jlongster.com/Backend-Apps-with-Webpack--Part-I

@halt-hammerzeit thanks for the info. Just to clarify, the second argument I was talking about was the ./server entry point path passed to the the server method of the webpack-isomorphic-tools instance.
I have read that article you referenced and use that technique along with the demonstration by @sokra for generating a node side server bundle https://github.com/webpack/react-webpack-server-side-example/blob/master/webpack.config.js#L38.
The reason why I was confused with your plugin/tools is that I see that your webapp example and the react-redux-universal-hot-example  seem to both write the server side bundle entry point in es5, but then the actual server code is written in es6. I guess you guys are just compiling that bundle with babel to run it, and I was trying to pre-compile with webpack so my css/scss requires weren't breaking. Looks like that's what webpack-isomorphic-tools looks to solve by accessing the stats from the compiler('done' event.
Anyhow, still trying to wrap my head around all of this and figure out the best way to move forward with my project. Am I correct that if I were using the webpack-isomorphic-tools I would not directly require/import css/scss into the file but would rather pass the references css/scss stats down from the top level component. If so it would probably be nice to write some decorators similar to https://github.com/jordangarcia/nuclear-js-react-addons that could pass and retrieve asset data from react context.

webapp and the universal example are written in ES6.
But node.js doesn't support ES6 (afaik) so Babel is used (not to mention the benefit of a ton of plugins, for example the JSX plugin).
All client code is compiled with babel during webpack build when babel-loader activates for .js files.
On the server side we don't use Webpack (i have no need for that) so we activate Babel in those entry .js files (their sole purpose is to run babel/register).
So yes, we are compiling it with Babel, in run time (done once for each module).
webpack-isomorphic-tools was introduced to make require() of *.css (for example) files not throw an exception on the server (and furthermore, for it to return something meaningful like the path or the complied stylesheet).
stats are used for that purpose, you got that one right.
But since I didn't know of the possibility of target node I didn't test this module with it.
I can do that if you provide me a simple example project which I can test upon (with the instructions).
I didn't understand what you're saying in the last paragraph.
Maybe I could if you clarify what you mean there.

 I see that you are getting stats for extracted css bundle names and able to template them in as <link> tags in JSX. One thing I'm confused about is if Babel bundles your entire component, and in your README example for this repo, you pass the component as a prop
<div id="content" dangerouslySetInnerHTML={{__html: React.renderToString(component)}}/>

babel will be compiling that component chain as well and will it not throw an error if it sees you require/import css/scss in any components in that tree? I saw in your README you say you don't have to use the css-loader module feature for local scoping of css but wasn't sure how you were accomplishing it through the plugin? Also, saw in the redux example they were using less/scss but didn't quite see how it was working. Some clarification around this would be great

babel will be compiling that component chain and will not throw SyntaxErrors or whatever because the compilation happens at runtime in this case and in runtime we have the require() hooks activated for all the assets (css and such).

I can see how compile time require() hooks could possibly work too: using that NormalModuleReplacementPlugin in case it runs before Babel transpiles a file.
It's a hypothesis.
If you have any questions left I can clarify them.

Sorry for the delay, I'm on eastern timezone. Thanks for the explanation of babel/register require hooks. I wasn't aware completely of how it worked. Maybe my usecase is different and that is why I'm not understanding completely.

I would like to compile my react component tree to HTML and extract into a separate file because I will be serving this content statically. Is it still possible or useful to use isomorpic-tools?
It seems your examples and those in redux are for serving HTML content from express application server so I'm not sure how/if this relates to my project.

Essentially, I'm still confused because of the example in redux
https://github.com/erikras/react-redux-universal-hot-example#styles
https://github.com/erikras/react-redux-universal-hot-example/blob/0af04acb9a0072ac1581e23653327d0e5011cd0e/src/components/InfoBar/InfoBar.js
I'm assuming that is why the require is used inside of the render method, is this what you were saying about "compilation happens at runtime"? Would this then fail if I tried to import styles from './whatevs.scss'?

Yes, of course, that's what it's ment to do: to render React components on the server-side.
Aren't you using express? If you just use fs.writeFileSync, then it doesn't matter: you just move your project to express and the use Linux shell with wget to download the markup (or you can write a small Node.js application to dowload contents of a URL)

Regarding import vs require there's a section in the README, go read it.
In short: import does the job but require is for hot reload to work (while you're developing your project); import is equivalent to require for production.

yeh I read that about the import vs require. I think I was confused about require hooks. Just saw this method register_extension(extension) so I am understanding more. I won't be using express except for a dev server. I'm essentially using a static template generator to generate the layout markup in another framework other than react nunjucks and then will include the compiled react component into that as a partial. I'll put together a small demo repo, but I think I understand more of how to do this with isomorphic-tools now

@dtothefp ok.
If you have any suggestions about rewriting the README you can post them to me because I may have a blind spot for documentation issues being the project creator.
If you won't be using express then you don't need it. You can just take the code from the render() function and adapt it to your needs.
https://github.com/halt-hammerzeit/webapp/blob/master/code/server/webpage rendering.js
https://github.com/halt-hammerzeit/webapp/blob/master/code/server/web (koa).js#L80-L86

Or better look through just these two files
https://github.com/halt-hammerzeit/webapp/tree/master/code/react-isomorphic-render
There will be lines you won't understand but overall you can get the idea from these two files.
If you'll have questions on the code you can post them here.

@halt-hammerzeit thanks for all the explanations, I think I finally got it and tried to mix what I saw in the webapp example with the redux example. I made a demo here https://github.com/dtothefp/isomorphic-demo/tree/dfp/add-assemble
with the basic express server middleware here https://github.com/dtothefp/isomorphic-demo/blob/dfp/add-assemble/app/index.js
Essentially, I'm generating html using nunjucks and a static template generator assemble. Then I template the html for the react component and inject it in the template. The idea of this is that in local dev it will work via express middleware, and in production build it will most likely be run by something like an AWS lamba, and the html will be written to disk and uploaded.
Anyhow, I still have a few questions, not all related to this plugin but general webpack build stuff as well so as I noticed a few new things that I wasn't currently using:

I'm getting an error in NODE_ENV="development" mode related to local css that is not extracted. I know you said local styles won't work when not extracted (and from what I've seen extraction doesn't occur using the webpack-dev-middleware, but I don't see these errors in your webpapp repo or the redux repo so I'm wondering how to eliminate them?
terminal:

console:

also for the reason above when building a bundle of SASS that will be used globally, I build it in a separate webpack build that does not use the dev-middleware and instead uses compiler.watch (because I don't want to load all the css into js because it causes "flickering" locally). I'm wondering if you have some tricks around this?
also I started using the 'react-transform' plugin, as I'm assuming it is a replacement somehow for the react-hot-loader. I used the following config but if I include rewire plugin I get an error inside babel-rewire code.

Uncaught ReferenceError: _ReactTemp is not defined

I can put an issue on 'react-transform' or rewire but wondering if you know a solution off hand?
    const babelDevConfig = {
      plugins: [
        'rewire',
        'react-transform'
      ],
      extra: {
        'react-transform': {
          transforms:
            [{
              transform: 'react-transform-hmr',
              imports: ['react'],
              locals: ['module']
            }]
        }
      }
    };

Okay, I see, you figured the things out.
As for the webpack-isomophic-tools error, the fix will be:
    style_modules: {
      extensions: ['css', 'less','scss'],
      filter: function(m, regex, options, log) {
        if (!options.development) {
          return m.name.indexOf('./~/css-loader') === 0 && regex.test(m.name);
        }
        //filter by modules with '.scss' inside name string, that also have name and moduleName that end with 'ss'(allows for css, less, sass, and scss extensions)
        //this ensures that the proper scss module is returned, so that namePrefix variable is no longer needed
        return m.name.indexOf('./~/css-loader') === 0 && (regex.test(m.name) && m.name.slice(-2) === 'ss' && m.reasons[0].moduleName.slice(-2) === 'ss');
      },

As you can see, I added a m.name.indexOf('./~/css-loader') === 0
This way the style object isn't overwritten with an empty object.
After fixing this error I'm still unable to browse your application because it says
TypeError: Cannot read property 'render' of undefined here:
      const page = assemble.pages.getView('pages/index');
      page.render({

for some reason page is undefined.
I don't have any experience with that assemble module so you should figure out why that .getView() returns undefined.
If you'll any issues after fixing that one I can help too.

@halt-hammerzeit thanks for the feedback. I just updated the master branch of my repo with your fix so you can reference here
https://github.com/dtothefp/isomorphic-demo
I only had to add your indexOf check for development otherwise my app would break as the css styles in the webpack-stats.json wouldn't be generated.
Also, for the assemble problem could you try npm i -D assemble-loader@latest and let me know if it works? Not sure what the issue is, but I'm working off their dev branch so anything is possible :neckbeard:. @doowb could you have a look?

Oh, yeah, that's right, I didn't think of checking that quickfix in production mode.
I installed the latest version but it doesn't work.
^CG:\work\isomorphic-demo [master +1 ~0 -0 !]> npm i -D assemble-loader@latest
npm WARN install Couldn't install optional dependency: Unsupported
<EMAIL_ADDRESS>G:\work\isomorphic-demo
└──<EMAIL_ADDRESS>
npm WARN EPACKAGEJSON<EMAIL_ADDRESS>license should be a valid SPDX license expression

Still the same exception.
I've looked through your code a bit and there seems to be a possibility of file separator issue.
  assemble.create('pages', {renameKey: function(fp) {
    const dirname = path.dirname(fp).split('/').slice(-1)[0];
    const basename = path.basename(fp).split('.').slice(0)[0];

    return `${dirname}/${basename}`;
  }})
  .use(loader());

To begin with, i'm on Windows, so it's not /, it's \.
Then, the assemble library itself may be relying on / instead of \ and end up not finding things by such incorrect paths»

hmmmm @doowb is an ex .NET guy so I'll check with him.

PS if I re-order the rewire plugin to go after the react-transform then everything works. Just an FYI, I'll put an issue on the react-tranform-hmr repo
    const babelDevConfig = {
      plugins: [
        'react-transform',
        'rewire'
      ],
      extra: {
        'react-transform': {
          transforms:
            [{
              transform: 'react-transform-hmr',
              imports: ['react'],
              locals: ['module']
            }]
        }
      }
    };

assemble works with both windows and unix style path separators. @dtothefp if that's your code in the renameKey function, then that just needs to be updated to use the correct separator for the environment.

@doowb @halt-hammerzeit updated the renameKey function if that was the culprit
https://github.com/dtothefp/isomorphic-demo/commit/9b536cfead4be8d8d0133f7c0f1003eef33e46df#diff-2

@dtothefp I pulled your changes, done npm install and gulp watch - the error is the same

If you're referring to this error:

TypeError: Cannot read property 'render' of undefined

Try doing console.log(assemble.pages.views) just before the const page = assemble.pages.getView() line. This will tell you if any pages were actually loaded. The globby might not be finding the pages correct, so none are getting loaded onto the views.

@doowb Here it is
{ 'G:\work\isomorphic-demo\src\templates\pages\index': <File "index.html" <Buffer 2d 2d 2d 0d 0a 74 69 74 6c 65 3a 20 4d
 61 69 6e 20 50 61 67 65 20 54 69 74 6c 65 0d 0a 2d 2d 2d 0d 0a 0d 0a 7b 25 20 65 78 74 65 6e 64 73 20 6c 61 79 ... >> }

Thanks @halt-hammerzeit . @dtothefp I commented on the commit for the renameKey. I think you just need to use a regex to split instead of '/'. Then you'll get the correct segments to build the key.

alright @doowb and @halt-hammerzeit sorry for the confusion, I have never done node work on windows.
updated to use path.sep
export default function(fp) {
  const dirname = path.dirname(fp).split(path.sep).slice(-1)[0];
  const basename = path.basename(fp, path.extname(fp));
  return path.join(dirname, basename);
}

@dtothefp ok, I can run it now and it renders.
did you have any questions regarding rendering?

@halt-hammerzeit nope, this is good for my use case. Thanks for answering all of my extra questions that were off topic of this issue.

FYI
I'm posting this in every issue and PR to notify whoever may be interested:
today I've released an alternative helper library called universal-webpack.
It takes a different approach than webpack-ismorphic-tools and instead of hacking Node.js require() calls it just compiles all code with target: 'node' webpack configuration option.
As a result, all Webpack plugins and features are supported.
If you think you might need that here's an example project:
https://github.com/halt-hammerzeit/webpack-react-redux-isomorphic-render-example


Test failures on ppc64le rhel 7.6 architecture

Expected behavior
Build and test cases should succeed on ppc64le rhel 7.6

Actual behavior
Below modules test cases are failing -
	•	handler
	•	resolver
	•	testsuit
	•	transport-native-epoll

Steps to reproduce
1) Build the dependency packages hawtjni(1.17 SNAPSHOT) and netty-tcnative(2.0.26 SNAPSHOT).
2) The 'netty' package builds fine without test case execution.

However, below module test cases are failing -
	•	handler
	•	resolver
	•	testsuit
	•	transport-native-epoll

I see the handler module test cases are failing due to missing dependency 'conscript'.
Tried to build with openssl 1.1.1 and jetty-alpn with below option but it did not work.
	mvn -Dforcenpn=true

Below are the few test case errors I am getting currently -

•	handler
		[ERROR]   ConscryptOpenSslEngineInteropTest>SSLEngineTest.testSessionAfterHandshake:2890->SSLEngineTest.testSessionAfterHandshake0:2925->ConscryptSslEngineTest.clientSslContextProvider:64 ? NoClassDefFound

•	resolver
		10:27:46.656 [NioDatagramAcceptor-53] DEBUG o.a.d.s.d.p.DnsProtocolHandler - /<IP_ADDRESS>:52466 SENT:  org.apache.directory.server.dns.messages.DnsMessage@6e8f2e6e[transactionId=46002,opCode=QUERY,truncated=false,messageType=RESPONSE,recursionDesired=true,additionalRecords=[],responseCode=NO_ERROR,authorityRecords=[],acceptNonAuthenticatedData=false,recursionAvailable=false,answerRecords=[io.netty.resolver.dns.TestDnsServer$TestResourceRecord [ host5.sub.foo.com ( A IN 100 {apachednsipaddress=<IP_ADDRESS>} ) ], io.netty.resolver.dns.TestDnsServer$TestResourceRecord [ host5.sub.foo.com ( A IN 100 {apachednsipaddress=<IP_ADDRESS>} ) ]],questionRecords=[org.apache.directory.server.dns.messages.QuestionRecord@145c48f9[domainName=host5.sub.foo.com,recordClass=IN,recordType=A]],authoritativeAnswer=false,reserved=false]
10:27:46.656 [nioEventLoopGroup-36-1] DEBUG i.netty.resolver.dns.DnsNameResolver - [id: 0x4b4ca697] RECEIVED: UDP [46002: /<IP_ADDRESS>:58729], DatagramDnsResponse(from: /<IP_ADDRESS>:58729, to: /<IP_ADDRESS>:52466, 46002, QUERY(0), NoError(0), RD)
		DefaultDnsQuestion(host5.sub.foo.com. IN A)
		DefaultDnsRawRecord(host5.sub.foo.com. 100 IN A 4B)
		DefaultDnsRawRecord(host5.sub.foo.com. 100 IN A 4B)
10:27:46.657 [main] DEBUG o.a.d.s.d.p.DnsProtocolHandler - /<IP_ADDRESS>:52466 CLOSED
10:27:46.657 [main] INFO  o.a.directory.server.dns.DnsServer - DNS service stopped.
DNS service stopped.

	•	testsuit
	•	transport-native-epoll

		[INFO] Running io.netty.testsuite.transport.socket.SocketSslClientRenegotiateTest
		[WARNING] Tests run: 64, Failures: 0, Errors: 0, Skipped: 64, Time elapsed: 0.355 s - in io.netty.testsuite.transport.socket.SocketSslClientRenegotiateTest
		[INFO]
		[INFO] Results:

Minimal yet complete reproducer code (or URL to code)
Trigged the build from master branch.

Netty version
4.1.40.Final-SNAPSHOT

JVM version (e.g. java -version)
1.8.0_222

OS version (e.g. uname -a)
ppc64le GNU/Linux

I am investigating more on 'conscript' build.
Kindly please suggest is there any way to use 'jetty-alpn' package instead of consscript.
Or is there any way to bypass the 'conscript' dependency and build netty with positive test cases executions.

I an sorry but I have no access to ppc64le hardware and so can not help... That said our dev environment is usually x86

Hi Norman,
Thanks for the response.
Can we bypass the 'conscript' dependency and still build netty with positive test case executions by using alternatives like jetty-alpn?
You can get access to the power VM (Power8) by raising a VM request at OSU - http://osuosl.org/services/powerdev/request_hosting/.
Please add Gerrit<EMAIL_ADDRESS>as IBM advocate for the request approval.
Thanks in advance for the help.
Also please note that
•	I have commented below  portion from pom.xml as it is x86_64 specific config.

•	Netty Package builds fine without test case verification (with '-Dmaven.test.skip=true' option)

@SantoshMagdum done...

Hi Norman,
Did you get a chance to raise access request for guest OS for Power8 VM? Please let me know ticket/token ID. I shall followup for access.
Also, I had added ppc64le in below config to support ppc64le JDK.
^(x86_64|aarch_64|ppcle_64)$
Now I see netty-handler test cases are failing due for conscript dependency.

Yes I did but did not hear back yet

Am 29.08.2019 um 10:32 schrieb SantoshMagdum<EMAIL_ADDRESS>
Hi Norman,
Did you get a chance to raise access request for guest OS for Power8 VM? Please let me know ticket/token ID. I shall followup for access.
Also, I had added ppc64le in below config to support ppc64le JDK.
^(x86_64|aarch_64|ppcle_64)$
Now I see netty-handler test cases are failing due for conscript dependency.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

To be more clear I filled the form but did never get an email ..

On 29. Aug 2019, at 10:58, Norman Maurer<EMAIL_ADDRESS>wrote:
Yes I did but did not hear back yet

Am 29.08.2019 um 10:32 schrieb SantoshMagdum<EMAIL_ADDRESS>
Hi Norman,
Did you get a chance to raise access request for guest OS for Power8 VM? Please let me know ticket/token ID. I shall followup for access.
Also, I had added ppc64le in below config to support ppc64le JDK.
^(x86_64|aarch_64|ppcle_64)$
Now I see netty-handler test cases are failing due for conscript dependency.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/netty/netty/issues/9479?email_source=notifications&email_token=AADLIQSSIBIE5K74AOXX74TQG6CR7A5CNFSM4IM3R4U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5NWRSY#issuecomment-526084299, or mute the thread https://github.com/notifications/unsubscribe-auth/AADLIQQPXEJYVVKMJFCXG6TQG6CR7ANCNFSM4IM3R4UQ.

Hi Norman,
It seems like the access request is in the system and should get processed shortly.

@SantoshMagdum cool thanks!

Hi Norman,
Your guest VM access request is complete now. Please give a try. Thanks.

@SantoshMagdum will do once I have some cycles.

@SantoshMagdum should be fixed by https://github.com/netty/netty/pull/9550. Please verify... Also it would be cool if we could keep the host and include it in our CI infrastructure to ensure we not regress :)

Thanks Norman for checking.
I took the patch and tried building using below command.
mvn clean -DbuildDirectory=/usr/netty/BUILD -Dmaven.repo.local=/usr/netty/BUILD install
But the netty-handler was till throwing multiple errors like below -
[ERROR] Failures:
[ERROR]   OpenSslEngineTest>SSLEngineTest.testGetCiphersuite:2800 expected:<TLS_[AES_128_GCM]SHA256> but was:<TLS[CHACHA20_POLY1305]SHA256>
[ERROR]   OpenSslEngineTest>SSLEngineTest.testGetCiphersuite:2800 expected:<TLS[AES_128_GCM]SHA256> but was:<TLS[CHACHA20_POLY1305]_SHA256>
[ERROR]   OpenSslConscryptSslEngineInteropTest.testSupportedSignatureAlgorithms:122->SSLEngineTest.testSupportedSignatureAlgorithms:3166->ConscryptSslEngineTest.serverSslContextProvider:69 ? NoClassDefFound
[ERROR]   OpenSslConscryptSslEngineInteropTest.testSupportedSignatureAlgorithms:122->SSLEngineTest.testSupportedSignatureAlgorithms:3166->ConscryptSslEngineTest.serverSslContextProvider:69 ? NoClassDefFound
[ERROR]   OpenSslConscryptSslEngineInteropTest.testSupportedSignatureAlgorithms:122->SSLEngineTest.testSupportedSignatureAlgorithms:3166->ConscryptSslEngineTest.serverSslContextProvider:69 ? NoClassDefFound
[ERROR]   SslContextBuilderTest.testUnsupportedPrivateKeyFailsFastForClient ?  Unexpecte...
[ERROR]   SslContextBuilderTest.testUnsupportedPrivateKeyFailsFastForServer ?  Unexpecte...
[INFO]
[ERROR] Tests run: 5097, Failures: 72, Errors: 614, Skipped: 253

I tried quickly to skip the test cases for netty-handler using below but errors were still pooping up -
--- a/handler/pom.xml
+++ b/handler/pom.xml
@@ -28,6 +28,7 @@
   <properties>
	 <javaModuleName>io.netty.handler</javaModuleName>
+    <skipShadingTestsuite>true</skipShadingTestsuite>
	 <!-- Needed for SelfSignedCertificate -->
	 <argLine.java9.extras>--add-exports java.base/sun.security.x509=ALL-UNNAMED</argLine.java9.extras>
   </properties>

Please suggest.
For now we can go ahead with plain netty build without any test executions (skipping test cases 'mvn Dmaven.test.skip=true' builds netty without issues).
Please share your build command. I shall try same (if you are using any different config. in build)

Hmm I didn't do anything special and it just works with the PR applied:
[centos@netty netty]$ uname -a
Linux netty 3.10.0-514.21.2.el7.ppc64le #1 SMP Tue Jun 20 16:35:48 GMT 2017 ppc64le ppc64le ppc64le GNU/Linux

[centos@netty netty]$ echo $JAVA_HOME
/usr/lib/jvm/java-1.8.0-openjdk-<IP_ADDRESS>.b10-0.el7_6.ppc64le/

[centos@netty netty]$ ./mvnw clean install
...
...
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Netty/Dev-Tools .................................... SUCCESS [  1.376 s]
[INFO] Netty .............................................. SUCCESS [  7.027 s]
[INFO] Netty/Common ....................................... SUCCESS [01:09 min]
[INFO] Netty/Buffer ....................................... SUCCESS [ 55.379 s]
[INFO] Netty/Resolver ..................................... SUCCESS [  6.481 s]
[INFO] Netty/Transport .................................... SUCCESS [01:07 min]
[INFO] Netty/Codec ........................................ SUCCESS [ 32.166 s]
[INFO] Netty/Codec/DNS .................................... SUCCESS [  8.030 s]
[INFO] Netty/Codec/HAProxy ................................ SUCCESS [  6.729 s]
[INFO] Netty/Handler ...................................... SUCCESS [05:24 min]
[INFO] Netty/Codec/HTTP ................................... SUCCESS [ 26.196 s]
[INFO] Netty/Codec/HTTP2 .................................. SUCCESS [ 49.873 s]
[INFO] Netty/Codec/Memcache ............................... SUCCESS [  9.003 s]
[INFO] Netty/Codec/MQTT ................................... SUCCESS [  8.739 s]
[INFO] Netty/Codec/Redis .................................. SUCCESS [  8.171 s]
[INFO] Netty/Codec/SMTP ................................... SUCCESS [  7.163 s]
[INFO] Netty/Codec/Socks .................................. SUCCESS [ 10.489 s]
[INFO] Netty/Codec/Stomp .................................. SUCCESS [  7.465 s]
[INFO] Netty/Codec/XML .................................... SUCCESS [  8.074 s]
[INFO] Netty/Handler/Proxy ................................ SUCCESS [ 25.008 s]
[INFO] Netty/Resolver/DNS ................................. SUCCESS [01:08 min]
[INFO] Netty/Transport/RXTX ............................... SUCCESS [  5.690 s]
[INFO] Netty/Transport/SCTP ............................... SUCCESS [  6.803 s]
[INFO] Netty/Transport/UDT ................................ SUCCESS [ 25.089 s]
[INFO] Netty/Example ...................................... SUCCESS [ 18.879 s]
[INFO] Netty/Transport/Native/Unix/Common ................. SUCCESS [  9.682 s]
[INFO] Netty/Testsuite .................................... SUCCESS [14:47 min]
[INFO] Netty/Transport/Native/Unix/Common/Tests ........... SUCCESS [  3.404 s]
[INFO] Netty/Transport/Native/Epoll ....................... SUCCESS [06:31 min]
[INFO] Netty/Transport/Native/KQueue ...................... SUCCESS [  7.729 s]
[INFO] Netty/All-in-One ................................... SUCCESS [  4.563 s]
[INFO] Netty/Tarball ...................................... SUCCESS [  1.217 s]
[INFO] Netty/Testsuite/Autobahn ........................... SUCCESS [08:37 min]
[INFO] Netty/Testsuite/Http2 .............................. SUCCESS [  5.139 s]
[INFO] Netty/Testsuite/OSGI ............................... SUCCESS [  9.511 s]
[INFO] Netty/Testsuite/Shading ............................ SUCCESS [ 11.258 s]
[INFO] Netty/Testsuite/NativeImage ........................ SUCCESS [  3.588 s]
[INFO] Netty/Microbench ................................... SUCCESS [ 36.493 s]
[INFO] Netty/BOM .......................................... SUCCESS [  0.007 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 45:52 min
[INFO] Finished at: 2019-09-07T21:27:34Z
[INFO] Final Memory: 130M/666M
[INFO] ------------------------------------------------------------------------


Apply tags to resources created by Karpenter

1. Issue, if available:
#1488
2. Description of changes:
As part of issue #1488, it was discovered that default tags are not being applied to AWS resources per the Karpenter documentation.  This change applies the default tags to fleets, instances, volumes and launch templates.
karpenter.sh/provisioner-name: <provisioner-name>
karpenter.sh/cluster/<cluster-name>: owned
kubernetes.io/cluster/<cluster-name>: owned
Name: karpenter.sh/cluster/<cluster-name>/provisioner/<provisioner-name>

3. How was this change tested?
Added some tests to ensure resources created by Karpenter have the default tags.  Also manually tested and validated all resources have the default tags:
        {
            "Key": "karpenter.sh/cluster/dewaard-karpenter-demo",
            "Value": "owned"
        },
        {
            "Key": "Name",
            "Value": "karpenter.sh/cluster/dewaard-karpenter-demo/provisioner/default"
        },
        {
            "Key": "karpenter.sh/provisioner-name",
            "Value": "default"
        },
        {
            "Key": "kubernetes.io/cluster/dewaard-karpenter-demo",
            "Value": "owned"
        }

4. Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: link to issue
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Is 'karpenter.sh/provisioner-name' not mentioned in the docs, or am I just missing it?

Let's make sure to close the gap between the changes we're making here and the documentation (and vice versa)

We have decided to keep the current tagging strategy as it's written in our code, instead of what is in the docs.  I will follow-up with a new PR updating Karpenter documentation.
I also created issue #1525 specifically for discussion surrounding Karpenter resource tagging.


Fix message router example description

Fix message router example description so that it refers to the same endpoint as the example's code

Thanks for spotting this - and you are very welcome to contribute more


How to get which note arrows are where... (not strums)

What is your question?
So I'm iterating over all the notes with...
for i = 0, getProperty("unspawnNotes.length")-1 do
	setPropertyFromGroup("unspawnNotes", i, "offsetAngle", 180)
	setPropertyFromGroup("unspawnNotes", i, "offsetX", 60)
end

...and I wanna know what the variable would be to check if the arrow is being hit by the opponent...
(You know, with getPropertyFromGroup("unspawnNotes", i, "<something>"))

I'll also take what strum line the arrow is on if you can only get that...

mustPress

mustPress

guh???

?

mustPress

What's that supposed to do?

What you asked ??

check if the arrow is being hit by the opponent...

That is how it works, unless you meant the note was hit by the opponent for some reason?
There is a custom note script on the wiki you can check to see this is how it works


specify swagger.json as input to ReDoc page

Can ReDoc also take swagger.json path/url as input text like swagger? This would help us deploy the page and handle swagger.json path of various applications using a single app. This will be an enhancement.
Thanks.

@sinha-bharat I'm not sure If I understood you correctly. Do you need to specify which file to use as an input in runtime?
Check out Advanced usage section in README.

https://rebilly.github.io/ReDoc/  <-- look at the top of that page.

Is that what you're looking for?

@sinha-bharat you can also specify URL to Swagger spec as query parameter see example:

@sinha-bharat is it issue still relevant?

Closing, feel free to reopen


Pretrained Models for ETH-UCY

Hi,
Thanks for the nice work, I have a question regarding reproducing the numbers in Table 2 (ETH-UCY).
Do you have the pretrained models, and also the testing script?
Thanks in advance,

Hey @os1a ,
Thanks for your interest in our work/code -- hope it is helpful! We actually had not included pre-trained models/testing for the ETH-UCY datasets, as our main focus was SDD. However, the training/testing should be as simple as loading the ETH-UCY datasets and changing a couple hyperparameters.
I believe @ksachdeva was able to reproduce the numbers on ETH-UCY and kindly has provided the notebook used for training/testing: https://github.com/HarshayuGirase/PECNet/issues/7#issuecomment-705223578
Hopefully this helps!

Hope this answers your questions. I'm closing this issue for now. Please feel free to open another issue for other questions, star the repo if you found it helpful and thanks for interest in our code.


apply_torque_impulse() problem with Bullet

Godot version:
Godot 3.1 #2928091
OS:
Ubuntu 18.04
Issue description:
With a simple pendulum (a RigidBody attached with a HingeJoint), if we apply
a single torque impulse, we expect the pendulum to start oscillating.
It correctly does so with GodotPhysics but it doesn't with Bullet. With
Bullet the pendulum starts rotating at constant speed.
Here is a video of the problem, first Bullet, then GodotPhysics.

Steps to reproduce:
Create a RigidBody attached to a HingeJoint, apply a single torque impulse on it with apply_torque_impulse(). Switch between Bullet and GodotPhysics in the project settings
to observe the different behaviors.
Minimal reproduction project:
apply_torque.zip

CC @AndreaCatania

Currently testing on a build of master branch, commit 2db4942
The reproduction project gives the same result as the gif. However, when I tried to recreate the project, I didn't encounter the bug:

Here's my project recreation: BulletJoints.zip

I confirm TGRCdev observation, same here : apply_torque.zip has the problem and BulletJoints.zip doesn't.
I think I found where the issue comes from : the centers of our RigidBodies are not at the same place.
In BulletJoints.zip, if you move the children of the Box RigidBody (i.e MeshInstance & CollisionShape) so that when you select the Box, the center of the Box is aligned with the pivot point like this :

then you get the same problem as in apply_torque.zip.

Dang, nice find. Looks like Bullet uses the RigidBody node's position, while GodotPhysics uses the CollisionShape node's position.

So I've looked a bit further into the differences in Bullet and Godot's center_of_masses, and I have a modified apply_impulse.zip to show some of my findings.
apply_torque_modified.zip
Green Ball = RigidBody's origin
Red Ball = Center of mass
Left Slider = CollisionShape y translation
Top Slider = StaticBody y translation
Right Slider = RigidBody y translation
value of center_of_mass pulled from PhysicsDirectBodyState

GodotPhysics

GDPhysics' center of mass is described in positional offset from the RigidBody node. (Took me a while to figure this out.)
The center of mass is located at the center of all CollisionShapes under the RigidBody node.
Multiple CollisionObjects modify the center of mass
Changes to RigidBody.translation are completely ignored

Bullet

(The red ball is hidden under the green ball, because the center of mass is the RigidBody node's origin)

Bullet's center of mass is described in global coordinates
The center of mass is located at the RigidBody's origin
Multiple CollisionObjects do not modify the center of mass, it always stays at RigidBody node's origin
Changes to RigidBody.translation briefly moves the RigidBody node away from the HingeJoint, but it snaps back after a few frames.

Yes it's nice to be able to set the center of mass where we want by adjusting the relative position of the RigidBody/CollisionShapes.
Should we open a new issue on godot-docs to remember documenting this and close this issue ?


HDDS-11263. Announce release 1.4.1 on the website

What changes were proposed in this pull request?
Add release page for Ozone 1.4.1 release to the master branch.
What is the link to the Apache Jira?
https://issues.apache.org/jira/browse/HDDS-11263

Thanks @errose28, @adoroszlai for the review.


Option to disable ide-helper:eloquent when doing ide-helper:generate

The command is confusing the IDE with "non-static methods being called statically" warnings. I had no problems just adding the mixin of Eloquent on my Models. Adding extra mixins seems to be causing the problem. It will be nice to have an option to turn that off.

@barryvdh nowadays this works as expected, issue can be closed :}


Question about the evaluation of 3D face reconstruction

Hi, Thanks for your great work and open source code!
Sorry to disturb you. I have trouble with how to preprocess the 'Florence' dataset and calculate the point-to-plane RMSE. I have made some effort but still do not deal with this. I would be very grateful if you can share this part of code with me. Looking forward to hearing from you. Thank you!

hi, langyuan
This part code comes from my previous colleague, I can not release this.
We clear the preprocess of the pipeline of preprocessing in the paper.

sign the 5 landmarks on micc mesh
crop the face region(detail in the paper)
3.rigid align by 5 3D landmarks
4.icp
5.point-to-plane rsme
All steps above have public code/lib.
Wish this can help you.

Hi, jiaxiang
Thank you very much for your helpful suggestions. I still have some questions below:

each subject of MICC has four meshes (frontal1, frontal2, sidel, sider), which mesh shall we choose?
given the cropped mesh of a specific object from MICC, which image or images shall we use to reconstruct the 3d face mesh, so that we can do the icp between them?
given two point sets A and B, whether the 5 landmarks are used to calculate the scale factor? after scaling them to the same level, then we run the icp on these two point sets to get rigid transformation R,t. Is that so?

Looking forward to hearing from you.

Yes!
This is of course a problem!!! When the first time I process this, it is also a trouble for me.

I pick the (no emotion/neutral) mesh by myself from (frontal1, frontal2). And too be honest, I do this two time and this won't highly affect the final result.
2.This you can see in our paper, we have two methods,
(1) using the MICC video frame
(2) or render images.
yes, 5 landmarks used for a similarity transform, scale is important.

In a word, process the micc dataset by ourselves.

Hi, jiaxiang
thank you for your helpful reply, I have dealed with the icp registration! I still have some questions below:

about the two methods to use MICC:

(1) video frames: does the average shape for each video mean we average the \alpha and \beta coefficients for all frames and use the average \alpha and \beta to reconstrcut the final average  shape S?
(2) rendered images: can you tell me the pipeline to get rendered images?

what is the point-to-plane RMSE formula?

Looking forward to hearing from you.

(1) average mesh
(2) emm, you can see the paper that how we define the camera poses, the render pipeline we using the tf_mesh_render
2, This is the common definition of the point-to-plane RMSE, it has kinds of public lib/code do this

Hi, I still have confusions on the video frame case. by "average mesh", do you mean we need to get the reconstructed face mesh for each frame of a video, then run the icp with micc mesh for all reconstructed face meshes, and finally average the registered reconstructed meshes?

sample the video
test on each video
3, average the test result(mesh)
4, do eval
This pipeline will solve the time problem

aha, thank's for your sugguestion.
I have another question about the NME evaluation, the AFLW2000-3D doesn't provide the bounding box annotation, how do you get the bounding box?

they give the landmark gt, we bound the bbox by this gt landmark.

what's the strategy you use to bound the bbox by gt landmarks? I want to use the same stratgy to keep a fair comparison.

Oh, sorry, I do not make this clear, this strategy is to find the most fix box, it means the min(landmark_x/y) and max(landmark_x/y).
As this is a common setting in face alignment, so I did not give detail in the paper.

I get it. thank you!

Hi, jiaxiangshang,
Great job! I have some questions about evaluation on MICC :

As you say '5 landmarks used for a similarity transform', do you mean calculate the transform (scale only) between groudtruth and prediction, and how to calculate the scale according to 5 landmarks? The distance between 2 eyes or ...
Do I need to label 5 landmarks in all MICC mesh first for crop and align.
when testing render images, should I rotate the MICC 3D face to look at the camera first, then render face images with 20 poses.

THX!

1, the similarity transform contains scale term, the transform is sure betweenn gt and predict face
2. yes
3, the render pipeline is not strict or std, the rendered images make sense is ok

thanks for your fast reply, and I still have some questions:

How can I calculate similarity transform matrix between two 3D point sets?
Should I crop the predicted mesh at a radius of 95mm around the nose after align (align the predicted mesh to gt?)
Due to the difference vertex number between gt and the predicted, I need use point-to-plane ICP rather than point-to-point ICP, right?

similarity transform is a common setting, that public code can be found
is clear as our paper.
yes

Sorry to disturb you again.
I evaluate by this stratgy:

Label 5 lmks on MICC data.

Crop the gt mesh at a radius of 95mm around the nose.

Align predicted mesh by similarity transform (don't change gt).

Do point(gt)-to-plane(predicted) ICP.

caluate the RMSE of each point of gt to plane of predicted mesh.

Is there any mistakes heres？and should I remove the forehead of gt in step2?
Thanks for your guidance。


[Feature] Support the new version of baichuan2

baichuan-inc released a new version of Baichuan2-13B-Chat today. There should be a problem with the calculation of prompt tokens when the existing code is tested
reference: https://huggingface.co/baichuan-inc/Baichuan2-13B-Chat

I find that alibi  of baichuan2 is changed

Same request, seems we need Baichuan13B and Baicuhan2-13B arch here.

baicuan2-13b's ALIBI seems a little bit different than 1, vllm didn't need change anything can support 2 out of box?

baicuan2-13b的ALIBI似乎与1有点不同，vllm不需要改变任何东西可以支持2开箱即用吗？

无缝衔接，没感觉到有什么不妥，vllm版本是0.1.3

baicuan2-13b的ALIBI似乎与1有点不同，vllm不需要改变任何东西可以支持2开箱即用吗？

无缝衔接，没感觉到有什么不妥，vllm版本是0.1.3

请问有出现回答质量下降的情况吗？

To support Baichuan2-13B-Chat, fastchat's code needs to be modifyed.
This is because the chat template is different, although user_token_id and assistant_token_id is the same on Baichuan and Baichuan2 (195 and 196), the reserved word is different (<reserved_102> and <reserved_103> for Baichuan, <reserved_106> and <reserved_107> for Baichuan2).
Here is the patch:
diff --git a/fastchat/conversation.py b/fastchat/conversation.py
index f733be6..58aa719 100644
--- a/fastchat/conversation.py
+++ b/fastchat/conversation.py
@@ -779,6 +779,19 @@ register_conv_template(
     )
 )

+# Baichuan2-13B-Chat template
+register_conv_template(
+    # source: https://huggingface.co/baichuan-inc/Baichuan2-13B-Chat/blob/19ef51ba5bad8935b03acd20ff04a269210983bc/modeling_baichuan.py#L555
+    # https://huggingface.co/baichuan-inc/Baichuan2-13B-Chat/blob/main/generation_config.json
+    Conversation(
+        name="baichuan2-chat",
+        roles=("<reserved_106>", "<reserved_107>"),
+        sep_style=SeparatorStyle.NO_COLON_SINGLE,
+        sep="",
+        stop_token_ids=[],
+    )
+)
+
 # Baichuan-13B-Chat template
 register_conv_template(
     # source: https://huggingface.co/baichuan-inc/Baichuan-13B-Chat/blob/19ef51ba5bad8935b03acd20ff04a269210983bc/modeling_baichuan.py#L555
diff --git a/fastchat/model/model_adapter.py b/fastchat/model/model_adapter.py
index afe79a6..e189138 100644
--- a/fastchat/model/model_adapter.py
+++ b/fastchat/model/model_adapter.py
@@ -1137,6 +1137,30 @@ class TigerBotAdapter(BaseModelAdapter):
     def get_default_conv_template(self, model_path: str) -> Conversation:
         return get_conv_template("tigerbot")

+class Baichuan2Adapter(BaseModelAdapter):
+    """The model adapter for Baichuan2 models (e.g., baichuan-inc/Baichuan2-7B)"""
+
+    def match(self, model_path: str):
+        return "baichuan2" in model_path.lower()
+
+    def load_model(self, model_path: str, from_pretrained_kwargs: dict):
+        revision = from_pretrained_kwargs.get("revision", "main")
+        tokenizer = AutoTokenizer.from_pretrained(
+            model_path, trust_remote_code=True, revision=revision
+        )
+        model = AutoModelForCausalLM.from_pretrained(
+            model_path,
+            trust_remote_code=True,
+            low_cpu_mem_usage=True,
+            **from_pretrained_kwargs,
+        )
+        return model, tokenizer
+
+    def get_default_conv_template(self, model_path: str) -> Conversation:
+        # for Baichuan2-13B-Chat
+        if "chat" in model_path.lower():
+            return get_conv_template("baichuan2-chat")
+        return get_conv_template("zero_shot")

 class BaichuanAdapter(BaseModelAdapter):
     """The model adapter for Baichuan models (e.g., baichuan-inc/Baichuan-7B)"""
@@ -1615,6 +1639,7 @@ register_model_adapter(ChangGPTAdapter)
 register_model_adapter(TuluAdapter)
 register_model_adapter(FalconAdapter)
 register_model_adapter(TigerBotAdapter)
+register_model_adapter(Baichuan2Adapter)
 register_model_adapter(BaichuanAdapter)
 register_model_adapter(XGenAdapter)
 register_model_adapter(NousHermesAdapter)

Apply this patch to fastchat package, and vllm can support Baichuan2-13B-Chat model.

Seem fastchat now supports baichuan-2 only with the conv_template change, it doesn't add a new adapter for baichuan2, that means beside the conv template, everything is exact same.

Closing this issue as stale as there has been no discussion in the past 3 months.
If you are still experiencing the issue you describe, feel free to re-open this issue.


Add twig namespace support 2.x

This PR is https://github.com/timber/timber/pull/1791 opened against 2.x

Coverage increased (+0.02%) to 93.525% when pulling 4b55dbaa4ea1348a3b7feb91f3efb6db441c773b on jenkoian:add-twig-namespace-support-2.x into 58191b00bd7d4fab2be05f44c31dd6475321fddd on timber:2.x.

Tests are passing! Take a look, I've added some docs and things too. Wondering if it needs more though, particularly around other related things like Timber::$dirname etc.

@jenkoian thanks so much for your work on this. I went through this afternoon and found a few small tweaks to make (#1815) but overall this is a really really strong PR
This is now merged into 2.x! Now, what's next?

Woohoo, thanks!

Just out of interest, do you have a roadmap or anything as to when 2.x will be released?

For this pull request I only have one question:
We have a timber/locations filter:
https://github.com/timber/timber/blob/5101648dd78bdb0b3546864e6674cd284327af9c/lib/LocationManager.php#L27-L36
What’s the difference to the timber/loader/paths filter, which is now used in the examples? From reading the code, I don’t really see a difference apart from that timber/locations is loaded first. Could we get rid of one? I’d say we keep timber/locations, because when we talk about «Template Locations», it might more clear to use timber/locations instead of timber/loader/paths. Isn’t the Loader something that we want to keep somewhat hidden from the public API?
@jenkoian Thanks again for this great pull request. We could really use more of this! 😎 We’re actively working on 2.x and are tracking progress in the Timber 2.0 project. We can’t say when it will be released, because there are still quite a few breaking things to figure out. But if you want to help, you’re very welcome to do so.

@gchtr AFAICT the difference is this line, where either ABSPATH or / is appended to the list of paths, in order to accommodate open_basedir ini directives (see https://github.com/timber/timber/commit/fbc6d29f). Not sure what the reason is for having two different filters to handle this (@jarednova ?) but I'd definitely be in favor of deprecating timber/loader/paths in favor of timber/locations, and possibly moving that open_basedir logic from Loader::get_loader() to LocationManager::get_locations() since it's more concerned with locations than with the loader itself. IMHO that's totally tangential to this PR, though, and definitely deserves its own.
Nice work again, @jenkoian ! I had totally forgotten about Twig's namespace feature. :)

Thanks all, have to admit that timber/loader/paths and timber/locations thing confused me too 😄
Hoping to convert a project to use Timber but was blocked by lack of namespace support. Now that's in 2.x I'll continue the conversion based on that and happy to raise any patches as and when I come across anything 👍


coprocessor/dag/expr: add some un-pushed builtin UDFs part2 (#2918)

Implemented more builtin UDFs for coprocessor:
Regexp
RegexpBinary
GreatestXXX/LeastXXX
IntervalInt
IntervalReal

Thank you @bb7133
PTAL @AndreMouche

I see, the PRs will be split into smaller ones in the future

Removed Regexp* and Interval*

Please update the description @bb7133 .

code updated, PTAL @AndreMouche @XuHuaiyu , thank you!

Any update?

Related issue: https://github.com/pingcap/tikv/issues/3275

Any update?

@XuHuaiyu @breeswish PTAL

Please fix the DCO

comment addressed, PTAL @hicqu

Comment addressed, PTAL @AndreMouche , thank you

LGTM.

LGTM.

@XuHuaiyu @breeswish PTAL, thanks

@breeswish PTAL

LGTM.

There is a new conflict. Would you like to resolve it? :)

comments addressed, thanks! PTAL @huachaohuang @breeswish


Add basics suite, refactor to share common infra

This PR adds a new 'basics' suite - with a test that checks for unique gems.
It also refactors to pull out common functionality (KITCHEN env, YML files) into a shared folder.
This allows using a single set of switches to control the behavior of the various suites.
@chef/client-core @jkeiser @sersut

This is cool. I would run an adhoc build through Jenkins to make sure we don't break test matrix.
KITCHEN_CHEF_PRODUCT, KITCHEN_CHEF_VERSION, KITCHEN_CHEF_CHANNEL are not set in Jenkins. So you might need to change this line in ci/verify-chef.sh.

After seeing a build run through jenkins :shipit:


[BUG]

Describe the bug
Configure cmake in the build directory and build the firmware image with the following command.
idf.py -DVENDOR=espressif -DBOARD=esp32_devkitc -DCOMPILER=xtensa-esp32 build
directory is->amazon-freertos/vendors/espressif/esp-idf/examples/get-started/hello_world
When I runned this command i faced error as follow-:
System information

Hardware board: [ esp32 devkitc ]
IDE used: [ linux terminal]
Operating System: [  Linux  ]
Code version: (v4.2.1-21-gd58109d32)
Project/Demo: [hello_word ]

How would I solve this?
Thank you in advance

Hello @aditi230,
The example demo that you are attempting to run is part of the Espressif ESP-IDF repository and is not directly related to this repository. I'm unfamiliar with this demo, but it looks like the flags that you are using are specific to the demos in this directory:
https://github.com/aws/amazon-freertos/tree/main/demos
If you would like to run the demos inside of  ESP-IDF repository and are having issues, I'd recommend that you post a question to the Espressif section of the FreeRTOS forums.

okay Thankyou


Prompt to install pandas if not found when opening data viewer

For #9944

Codecov Report

Merging #10394 into master will decrease coverage by 0.07%.
The diff coverage is 87.09%.

@@            Coverage Diff             @@
##           master   #10394      +/-   ##
==========================================
- Coverage   60.88%   60.81%   -0.08%
==========================================
  Files         570      574       +4
  Lines       30758    30937     +179
  Branches     4379     4397      +18
==========================================
+ Hits        18727    18814      +87
- Misses      11080    11164      +84
- Partials      951      959       +8

Impacted Files
Coverage Δ

src/client/datascience/types.ts
100% <ø> (ø)
:arrow_up:

src/client/common/installer/productInstaller.ts
92.82% <100%> (+0.03%)
:arrow_up:

src/client/common/types.ts
100% <100%> (ø)
:arrow_up:

src/client/common/utils/localize.ts
95.09% <100%> (+0.01%)
:arrow_up:

src/client/common/installer/productNames.ts
100% <100%> (ø)
:arrow_up:

src/client/common/installer/productService.ts
100% <100%> (ø)
:arrow_up:

...ent/datascience/data-viewing/dataViewerProvider.ts
83.33% <100%> (ø)

...cience/data-viewing/dataViewerDependencyService.ts
84.9% <84.9%> (ø)

src/datascience-ui/react-common/arePathsSame.ts
75% <0%> (-12.5%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-11.77%)
:arrow_down:

... and 7 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 762d236...303fc8c. Read the comment docs.


[System.IO.File]::WriteAllBytes raise a conversion exception

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
$NomeFile = "$HOME\InvoiceArchive\Customer\2023\Q1\CustomerInvoice.xml.p7m"
$DocFile = Invoke-RestMethod -Uri "$Global:DocEasyURI/api/DocumentoAttivo/$FileID/file" -Headers $Header -Method Get -ContentType 'application/json-patch+json'
[System.IO.File]::WriteAllBytes($NomeFile, ($DocFile.ToCharArray()))
Expected behavior
Writing file: C:\Users\UserName\InvoiceArchive\Customer\2023\Q1\CustomerInvoice.xml.p7m... Completed.

Actual behavior
Cannot convert argument "bytes", with value: "System.Char[]", for "WriteAllBytes" to type "System.Byte[]"

Error details
Exception             :
    Type           : System.Management.Automation.MethodException
    ErrorRecord    :
        Exception             :
            Type    : System.Management.Automation.ParentContainsErrorRecordException
            Message : Cannot convert argument "bytes", with value: "System.Char[]", for "WriteAllBytes" to type "System.Byte[]": "Cannot convert value "€" to type "System.Byte". Error: "Value was either too large or too small for an unsigned byte.""
            HResult : -2146233087
        CategoryInfo          : NotSpecified: (:) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : MethodArgumentConversionInvalidCastArgument
        InvocationInfo        :
            ScriptLineNumber : 349
            OffsetInLine     : 4
            HistoryId        : 6
            ScriptName       : C:\Users\UserName\InvoiceArchive\DocEasy.ps1
            Line             : [System.IO.File]::WriteAllBytes($NomeFile, ($DocFile.ToCharArray()))

            Statement        : [System.IO.File]::WriteAllBytes($NomeFile, ($DocFile.ToCharArray()))
            PositionMessage  : At C:\Users\UserName\InvoiceArchive\DocEasy.ps1:349 char:4
                               +             [System.IO.File]::WriteAllBytes($NomeFile, ($DocFile.ToCh …
                               +             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
            PSScriptRoot     : C:\Users\UserName\InvoiceArchive
            PSCommandPath    : C:\Users\UserName\InvoiceArchive\DocEasy.ps1
            CommandOrigin    : Internal
        ScriptStackTrace      : at Write-DocEasyDocumento, C:\Users\UserName\InvoiceArchive\DocEasy.ps1: line 349
                                at Receive-DocEasyFaturePassive, C:\Users\UserName\InvoiceArchive\DocEasy.ps1: line 240
                                at <ScriptBlock>, <No file>: line 1
    TargetSite     :
        Name          : ConvertToArgumentConversionException
        DeclaringType : System.Management.Automation.ExceptionHandlingOps, System.Management.Automation, Version=<IP_ADDRESS>0, Culture=neutral, PublicKeyToken=31bf3856ad364e35
        MemberType    : Method
        Module        : System.Management.Automation.dll
    Message        : Cannot convert argument "bytes", with value: "System.Char[]", for "WriteAllBytes" to type "System.Byte[]": "Cannot convert value "€" to type "System.Byte". Error: "Value was either too large or too small for an unsigned byte.""
    InnerException :
        Type           : System.Management.Automation.PSInvalidCastException
        ErrorRecord    :
            Exception             :
                Type    : System.Management.Automation.ParentContainsErrorRecordException
                Message : Cannot convert value "€" to type "System.Byte". Error: "Value was either too large or too small for an unsigned byte."
                HResult : -2146233087
            CategoryInfo          : InvalidArgument: (:) [], ParentContainsErrorRecordException
            FullyQualifiedErrorId : InvalidCastIConvertible
        TargetSite     :
            Name          : ConvertIConvertible
            DeclaringType : System.Management.Automation.LanguagePrimitives
            MemberType    : Method
            Module        : System.Management.Automation.dll
        Message        : Cannot convert value "€" to type "System.Byte". Error: "Value was either too large or too small for an unsigned byte."
        InnerException :
            Type       : System.OverflowException
            TargetSite :
                Name          : ThrowByteOverflowException
                DeclaringType : System.Convert
                MemberType    : Method
                Module        : System.Private.CoreLib.dll
            Message    : Value was either too large or too small for an unsigned byte.
            Source     : System.Private.CoreLib
            HResult    : -2146233066
            StackTrace :
   at System.Convert.ThrowByteOverflowException()
   at System.Char.System.IConvertible.ToByte(IFormatProvider provider)
   at System.Convert.ChangeType(Object value, Type conversionType, IFormatProvider provider)
   at System.Management.Automation.LanguagePrimitives.ConvertIConvertible(Object valueToConvert, Type resultType, Boolean recursion, PSObject originalValueToConvert, IFormatProvider formatProvider, TypeTable backupTable)
        Source         : System.Management.Automation
        HResult        : -2147467262
        StackTrace     :
   at System.Management.Automation.LanguagePrimitives.ConvertIConvertible(Object valueToConvert, Type resultType, Boolean recursion, PSObject originalValueToConvert, IFormatProvider formatProvider, TypeTable backupTable)
   at System.Management.Automation.LanguagePrimitives.ConvertTo(Object valueToConvert, Type resultType, Boolean recursion, IFormatProvider formatProvider, TypeTable backupTypeTable)
   at System.Management.Automation.LanguagePrimitives.ConvertUnrelatedArrays(Object valueToConvert, Type resultType, Boolean recursion, PSObject originalValueToConvert, IFormatProvider formatProvider, TypeTable backupTable)
   at CallSite.Target(Closure, CallSite, Type, String, Object)
    Source         : System.Management.Automation
    HResult        : -2146233087
    StackTrace     :
   at System.Management.Automation.ExceptionHandlingOps.ConvertToArgumentConversionException(Exception exception, String parameterName, Object argument, String method, Type toType)
   at CallSite.Target(Closure, CallSite, Type, String, Object)
   at Write-DocEasyDocumento(Closure, FunctionContext)
CategoryInfo          : NotSpecified: (:) [], MethodException
FullyQualifiedErrorId : MethodArgumentConversionInvalidCastArgument
InvocationInfo        :
    ScriptLineNumber : 349
    OffsetInLine     : 4
    HistoryId        : 6
    ScriptName       : C:\Users\UserName\InvoiceArchive\DocEasy.ps1
    Line             : [System.IO.File]::WriteAllBytes($NomeFile, ($DocFile.ToCharArray()))

    Statement        : [System.IO.File]::WriteAllBytes($NomeFile, ($DocFile.ToCharArray()))
    PositionMessage  : At C:\Users\UserName\InvoiceArchive\DocEasy.ps1:349 char:4
                       +             [System.IO.File]::WriteAllBytes($NomeFile, ($DocFile.ToCh …
                       +             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    PSScriptRoot     : C:\Users\UserName\InvoiceArchive
    PSCommandPath    : C:\Users\UserName\InvoiceArchive\DocEasy.ps1
    CommandOrigin    : Internal
ScriptStackTrace      : at Write-DocEasyDocumento, C:\Users\UserName\InvoiceArchive\DocEasy.ps1: line 349
                        at Receive-DocEasyFaturePassive, C:\Users\UserName\InvoiceArchive\DocEasy.ps1: line 240
                        at <ScriptBlock>, <No file>: line 1

Environment data
Name                           Value
----                           -----
PSVersion                      7.4.0
PSEdition                      Core
GitCommitId                    7.4.0
OS                             Microsoft Windows 10.0.20348
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Somethings was change in .NET?

Was this working before on a previous version?
[System.IO.File]::WriteAllBytes($NomeFile, ($DocFile.ToCharArray()))

WriteAllBytes takes a byte array so rather than a char array you want an array of bytes, for JSON you may need to specify UTF-8 to convert the encoding.
The simplest may be to use WriteAllText

Hi,
On previous version all work fine.
I couldn’t use WriteAllText because is a binary content
-Ricky-
Il giorno 18 nov 2023, alle ore 12:45, rhubarb-geek-nz @.***> ha scritto:

Was this working before on a previous version?
[System.IO.File]::WriteAllBytes($NomeFile, ($DocFile.ToCharArray()))
WriteAllByteshttps://learn.microsoft.com/en-us/dotnet/api/system.io.file.writeallbytes?view=net-7.0 takes a byte array so rather than a char array you want an array of bytes, for JSON you may need to specify UTF-8 to convert the encoding.
The simplest may be to use WriteAllTexthttps://learn.microsoft.com/en-us/dotnet/api/system.io.file.writealltext?view=net-7.0
—
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/PowerShell/issues/20720#issuecomment-1817486545, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BDA5COZCOHSQQ4CIWXJXJ6DYFCNWBAVCNFSM6AAAAAA7Q4KXSGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJXGQ4DMNJUGU.
You are receiving this because you authored the thread.Message ID: @.***>

Hi, On previous version all work fine. I couldn’t use WriteAllText because is a binary content

JSON is text and you are using ToCharArray() which are effectively text as they depending on a character encoding.

I would expect the -ContentType 'application/json-patch+json' is redundant, as that refers to the body being sent, not the response.

Good points, @rhubarb-geek-nz.
More generally, the cast from [char[]] to [byte[]]  fails with any Unicode character whose code point is outside the 8-bit range, i.e. higher than 255 (0xFF)
Expressed in .NET terms, this applies to any Unicode code unit encoded in the unsigned 16-bit value that constitutes a [char] instance with a value of 256 and above.
The code point of € (EURO SIGN, U+20AC) is 0x20ac == 8364 and therefore causes the error.
A minimal repro:
[byte] [char] 'A'  # OK, the code point is 0x41 == 65 and therefore <= 255
[byte] [char] '€'  # !! FAILS, because the code point is 0x20ac == 8364 and therefore > 255

This behavior is as designed and has alway been in place - there has been no change there.
However, what has changed  in v7.4 is the default character encoding that Invoke-WebRequest / Invoke-RestMethod now assume: it is now UTF-8 (for application/json, specifically, it has been UTF-8 since 7.0 - I don't know how application/json-patch+json is handled in 7.3-).
Your approach looks like an old workaround for incorrectly decoded UTF-8 data because PowerShell used to use ISO-8859-1 (closely related to Windows-1252) as the default.
If so, then @rhubarb-geek-nz's recommendation to use .WriteAllText should indeed work, because $DocFile is now correctly decoded.

Thank you!
I understood my inaccuracies.
I solved this by removing the -ContentType parameter from Invoke-RestMethod and adding -OutFile. So I don't have to do conversions.
-Ricky-
From: Michael Klement @.>
Sent: sabato 18 novembre 2023 15:11
To: PowerShell/PowerShell @.>
Cc: RickyVer77 @.>; Author @.>
Subject: Re: [PowerShell/PowerShell] [System.IO.File]::WriteAllBytes raise a conversion exception (Issue #20720)
Good points, @rhubarb-geek-nzhttps://github.com/rhubarb-geek-nz.
More generally, the cast from [char[]] to [byte[]] fails with any Unicode character whose code point is outside the 8-bit range, i.e. higher than 255 (0xFF)
Expressed in .NET terms, this applies to any Unicode code unit encoded in the unsigned 16-bit value that constitutes a [char] instance with a value of 256 and above.
The code point of € (EURO SIGN, U+20AChttp://www.fileformat.info/info/unicode/char/20ac) is 0x20ac == 8364 and therefore causes the error.
A minimal repro:
[byte] [char] 'A'  # OK, the code point is 0x41 == 65 and therefore <= 255
[byte] [char] '€'  # !! FAILS, because the code point is 0x20ac == 8364 and therefore > 255
This behavior is as designed and has alway been in place - there has been no change there.
However, what has changed in v7.4 is the default character encoding that Invoke-WebRequest / Invoke-RestMethod now assume: it is now UTF-8 (for application/json, specifically, it has been UTF-8 since 7.0 - I don't know how application/json-patch+json is handled in 7.3-).
Your approach looks like an old workaround for incorrectly decoded UTF-8 data because PowerShell used to use ISO-8859-1 (closely related to Windows-1252) as the default.
If so, then @rhubarb-geek-nzhttps://github.com/rhubarb-geek-nz's recommendation to use .WriteAllText should indeed work, because $DocFile is now correctly decoded.
—
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/PowerShell/issues/20720#issuecomment-1817519390, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BDA5COY3AP33MNYAZOCB7QLYFC6XFAVCNFSM6AAAAAA7Q4KXSGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJXGUYTSMZZGA.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


eager flush results for single non aggregate query

Codecov Report

Merging #173 into master will decrease coverage by 0.18%.
The diff coverage is 51.57%.

@@            Coverage Diff             @@
##           master     #173      +/-   ##
==========================================
- Coverage   73.23%   73.05%   -0.19%
==========================================
  Files         111      111
  Lines       17100    17160      +60
==========================================
+ Hits        12524    12536      +12
- Misses       3620     3666      +46
- Partials      956      958       +2

Impacted Files
Coverage Δ

utils/metrics.go
65.47% <ø> (ø)
:arrow_up:

query/stats.go
90.42% <ø> (ø)
:arrow_up:

query/aql_context.go
100% <ø> (ø)
:arrow_up:

query/aql_batchexecutor.go
86.44% <100%> (+0.23%)
:arrow_up:

query/aql_compiler.go
76% <100%> (ø)
:arrow_up:

query/aql_nonaggr_batchexecutor.go
91.3% <100%> (ø)
:arrow_up:

api/sql_query_handler.go
91.17% <100%> (ø)
:arrow_up:

api/query_handler.go
65.17% <43.54%> (-10.61%)
:arrow_down:

query/aql_postprocessor.go
71.65% <54.54%> (-1.86%)
:arrow_down:

query/aql_processor.go
78.83% <60%> (-0.51%)
:arrow_down:

... and 4 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 d28f200...3ca10ac. Read the comment docs.


Updated Exploration demo notebook for review

Please review this notebook and provide feedback and suggestions on how to improve it. This notebook should serve as a template for all the demos.
You can find the notebook under niimpy/docs/example_notebooks/Exploration.ipybn or just check readthedocs page: https://niimpy.readthedocs.io/en/latest/demo_notebooks/Exploration/

Codecov Report

Merging #152 (aab736e) into master (559945b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #152   +/-   ##
=======================================
  Coverage   68.30%   68.30%
=======================================
  Files          35       35
  Lines        1795     1795
=======================================
  Hits         1226     1226
  Misses        569      569

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 559945b...aab736e. Read the comment docs.


⚠ Mastodon server has degraded performance

In 5cfc37e, Mastodon server (https://so.fyle.uk/) experienced degraded performance:

HTTP code: 200
Response time: 2470 ms

Resolved: Mastodon server performance has improved in d48fa39.


Block domains doesn't actually block anything

I've got ezXSS installed and it's collecting reports and callbacks and everything is working nicely. Except when I put a domain in Settings->Block domains and tap Save, it doesn't actually stop the emails or the reports from being generated. It does save the domains in the list and they persist between sessions, so that is working.
On a related front, the Don't save report or send alert option doesn't seem to be working here either as I've gotten about 40 reports from the same domain.
Anything I can do on my end?

Hi geeknik,
I just noticed your issue. I found out that blocked domains currently only works with the complete URL including http(s) like: https://www.google.com.
I will fix this in the next version.


Have something that tells you when you're done

I finished the collections exercises and watch just said
Testing collections.collections2
compiling exercises/collections/collections3.clj

Testing collections.collections3

Apparently I got it right and there were just no more exercises but the feedback looked more like it was just hanging.
There should be a message for when you're out of exercises. Ideally it would also tell you if there's further upstream changes to pull in

Thanks for trying out the project, especially in its rough state! Agreed, I’ve got some plans on improving the exercise feedback, please let me know if you have further ideas too.

Quick fix @jmingtan - output done when all tests are done running. At least that way you'll know it isn't hanging.

Ah okay, I see what you mean now about running out of exercises - should be fixed now.


'An error ocurred'

Well, i downloaded the mod, when i hosted a game my friends couldnt join, and i tried joining any random party but it didnt woked, so... i dont know what to do

Everyone needs the mod installed to play together.

Everyone needs the mod installed to play together.

Question - If I have Mimics installed but the host doesn't, will I be able to join them to simply play without the mimics?
I have a little r2modman profile with (I assume) only client-side things and I slapped Mimics on, to be able to join public BL servers that include the mimic mod in their name. This is not something every public server does, but I've been getting the "An error occurred" message for like 80% of the public BL servers I attempt to join. Could you clarify if the Mimics causes this limitation?
If the answer is yes, would it be possible to make it so the player can still join modded servers that don't have the Mimics mod, even if the joining player does? I hope you understand, it's been a pain trying to find servers when most of them refuse you.

Mimics registers a networked prefab, and it seems lobbies reject you if you don't have the same list of networked prefabs. I don't know how to get around this, so unless I get a PR it will remain this way.


⚠ Fortnite has degraded performance

In c77cbed, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1612 ms

Resolved: Fortnite performance has improved in ed75964.


${exception} - separator=${newline} won't work

Bug:
${exception:format=tostring,data:maxInnerExceptionLevel=10:separator=${newline}}

wont work
but this works:
${exception:format=tostring,data:maxInnerExceptionLevel=10:separator=\r\n}

See https://github.com/NLog/NLog/issues/2248

Hi 304,
I am able to replicate it. I believe I have fixed it locally. Can I commit it to my fork and raise a pull request?

Yes that would be great! Please also include an unit test, thanks!

OK. I'll refine it first and will let you know once its ready.

Hi @304NotModified ,
The pull request is now ready: https://github.com/NLog/NLog/pull/2256
Thanks!

Investigated this, this isn't a bug as the type of the parameter is string and not Layout

How to make platform compatible line separator? I.e. for Linux separator=\n and for Windows separator=\r\n?

I think the only way is to fix this issue (it's a breaking change), or your program needs some code for it.

Discovered that there is different between:

<target type="file" filename="C:\NLog\Rocks" />

And

${literal:inner=C:\NLog\Rocks}

So when suddenly changing Exception.Separator from string to Layout then this will change meaning:

${exception:separatorr=\r\n}

Instead of being Windows-newline, then it will translate into the actual string "\r\n". I kind of like that NLog 5.0 is still non-breaking in parsing of NLog.config.

where escapes are ignored

Isn't that we need an extra \? e.g
${exception:format=message:maxInnerExceptionLevel=10:separator=\t\r\n}
But of course, unfortunate for the breaking change.

another trick is to decompose the ${newline}  when parsing the file ;)

another trick is to decompose the ${newline} when parsing the file ;)

Thinking the same, since all use-cases has been about using ${newline}.

but I think it should be now in LayoutParser, and that could have bigger side-effect

Yes close to impossible. ${newline} as implicit nlog-config-variable would also give headaches.

Created #4538 that allows parsing of ${newline} when non-layout.


Fix ERR_print_errors() format to match documentation.

Checklist

[x] documentation is added or updated
[x] tests are added or updated

Fixes #11743

Side note: I've been pondering refactoring all possible variants of hexdump output into a BIO filter...  that's an effort in itself and won't happen now, but might take away the growth of a gazillion different hexdump functions.
(no, I do not intend to block this PR...  if nothing else, it's a necessary temporary measure)

The actual test source seems to be missing? Also do I get it right that the added hexstr2buf variant is solely for the test purposes?

Also do I get it right that the added hexstr2buf variant is solely for the test purposes?

... or maybe it's for feature completeness (I've complained about the lack of feature completeness, so I can understand if efforts are being made)

Also do I get it right that the added hexstr2buf variant is solely for the test purposes?

It is not really a one way function, so if a separator is added it should work both ways.

I am just removing the err include from the test also..

My approval stands

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Thanks. Merged to master.


Variables sometimes don't generate warnings

In https://fetch.spec.whatwg.org/#websocket-connections while host/secure generate warnings, port does not. Perhaps because a variable with the same name but different purpose appears in another section?

Yes, it's not in an algorithm block (either a container with algorithm attribute, or a section whose heading has algorithm="algo name"), so it's just considered as part of the document-global set of vars, and you apparently have used "port" as a var at least twice. (I see a usage in section 3.5.)

Shouldn't it be at least section-scoped?

Not necessarily; if you're not explicitly defining an algorithm, you might be using a variable globally across a spec.  I have explicit algorithms so you can indicate this properly.


Update README.rst

Thanks for suggesting the improved text, @diraol. I merged your suggestion to the new directory, 18b-async-await/README.rst file.


Icons cover temperatur texts

Hello,
I just installed your module and like it.
But there is one thing that doesnt fit: the icons cover the temperature texts.

My config:

       "module": "MMM-WeatherChart",
        "position": "top_right",
        "config": {
            "apiKey": "xyz",
            "lat": xyz,
            "lon": xyz,
            "units": "metric",
  "updateInterval": 15 * 60 * 1000,
  "title": "Wetter Münster",
  "showRain": true,
  "showIcon": true,
  "fillColor": "rgba(30, 144, 255, 0.2)",
            "dataNum": 13,
            "dataType": "hourly",
            "height": "400px",
            "width": "500px",
        }

Btw: why changes the temperature line from 17 o'clock?

Hi
Thank you for sharing the problem.
I added two patches to solve the problem.

The width and height module configs did not work well and I fixed that.
Enable to change the icon size by setting iconWidth and  iconHeight module config.

So, to avoid the overlapping, please clone the latest module and you should adjust the configs width, height, fontSize, iconWidth and iconHeight.
For your above config, setting fontSize=14 or iconWidth/Height=40 may work well.

Btw: why changes the temperature line from 17 o'clock?

The line style can be changed for night time temperature. See nightBorderDash config option in README.


Add cacio e pepe

Add it to the list on the main page and remove the extra newlines between the ordered list items.

Add it to the list on the main page and remove the extra newlines between the ordered list items.

It should be good now. Thanks.


Stopped forcing eutheria_odb10

Something I forgot to include in the previous release, and became a necessity now that we're running both genome-note and blobtoolkit: both need to default to the same lineage !
We will fix that properly in a later release, cf https://github.com/sanger-tol/genomenote/issues/100

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

The CI error looks like Nextflow 24.01-edge doesn't understand the closures from conf/base.config any more ?! It may be related to bumping the Groovy version in that release.


pr2_props_stack: 1.0.5-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository pr2_props_stack to 1.0.5-0:

upstream repository: https://github.com/pr2/pr2_props.git
release repository: https://github.com/pr2-gbp/pr2_props_stack-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.20
previous version for package: 1.0.4-0

conflict with last release. Was this supposed to be jade?

Hm.. there is actually no pr2_props_stack release, the link pr2_props_stack on github forwarded it to pr2_props so I was just adjusting for that.. and re-released

Ok, then you just need to rebase this. It's conflicting with #9961 and is not mergeable.

Sounds good

bloom-release --rebase, right?

bloom-release --rebase, right?

@TheDash no... https://help.github.com/articles/about-git-rebase/

I will close this due to lack of response. Please feel free to reopen / create a new PR.


Out of memory errors

I'm now bumping up against my 16GB of memory. Can I configure my way out of this before the new PC arrives?

It seems I can.
https://www.elastic.co/guide/en/elasticsearch/reference/8.13/advanced-configuration.html#set-jvm-heap-size
The default is half my memory, which is too much.
I should add these two lines in /etc/elasticsearch/jvm.options.d/whatever.options:
-Xms1g
-Xmx1g

Starting with 1g, which is the setting in elastic-bilara.

To check the JVM settings, go to:
http://localhost:9200/_nodes/_all/jvm

Created a new options file. Elasticsearch now reports this:
"jvm" : {
        "pid" : 20176,
        "version" : "21.0.2",
        "vm_name" : "OpenJDK 64-Bit Server VM",
        "vm_version" : "21.0.2+13-58",
        "vm_vendor" : "Oracle Corporation",
        "using_bundled_jdk" : true,
        "start_time_in_millis" :<PHONE_NUMBER>175,
        "mem" : {
          "heap_init_in_bytes" :<PHONE_NUMBER>,
          "heap_max_in_bytes" :<PHONE_NUMBER>,
          "non_heap_init_in_bytes" : 7667712,
          "non_heap_max_in_bytes" : 0,
          "direct_max_in_bytes" : 0
        },

That's exactly 1024MB.
System memory shows 8.4GB of 16.7GB being used. Elasticsearch is using about 1.5GB.