Nested resources are returning all resources

When calling a GET for https://jsonplaceholder.typicode.com/posts/1/comments all comments for every post is returned. Expected behaviour is that only comments with postId of 1 should be returned. This is occurring in all of the nested resource examples.

Hi I will take care of this,

the issue is still persistent. Any update on this?

@typicode I tried to remove the _isFake option in app.js and the nested resources works properly. I dunno what isFake options does but I think this needs to be fix in json-server repo?
/** FROM */
const router = jsonServer.router(clone(data), { _isFake: true })
/** TO */
const router = jsonServer.router(clone(data))

@Riyenz  You are right. I checked in json-server code and found out that router get 2 arguments
module.exports = (db, opts = { foreignKeySuffix: 'Id', _isFake: false }) => {
and then in code
// Handle /:parent/:parentId/:resource   router.use(nested(opts))
Also found a test which covers a case that you describe above
 describe('GET /:parent/:parentId/:resource', () => {     test('should respond with json and corresponding nested resources', () =>       request(server)         .get('/posts/1/comments')         .expect('Content-Type', /json/)         .expect([db.comments[0], db.comments[1]])         .expect(200))   })
and this server is pass as an argument to the request and it is created in beforeEach method without set _isFake property (in default is set to false) and test, of course, has passed, when I have set  _isFake to true as an argument to jsonServer.router method test has failed and returned all resources.
So this investigation proves that is not jsonplachelder fault. This _isFake property was introduced 7 months ago in json-server and also 7 months ago _isFake property has been added to jsonplaceholder in my opinion by mistake but I am not sure.
Cheers

@typicode

Can I take this up?

https://github.com/typicode/json-server/blob/8dd0b456ab313898f094c1e113f647425b2898a4/src/server/router/nested.js#L9
I think this is by design in the json-server repository, and if a fix is needed, an issue should be raised there.

How do you add the rest of the json file?  Tried to add strings but xcode keeps crashing


Is there any way to dynamically add templates/components to the vue table columns?

Thank you for the reply to my previous question https://github.com/spatie/vue-table-component/issues/107. I really like your veutable because it is very lightweight and easy to use by new Vue learners like me.
Here is another issue: I want to have a generic vuetable that can be adapted for any kind of data. So I want to put there some configurations so that the columns can add dynamically any kinds of components or templates based on the data settings. For example, I may want a column to have a <router-link> for this data, but next time for another set of data, I want a column that contains a <my-customized-template>. Would you please give us an example how to achieve this, if possible?
Thanks a lot.

I wonder how return createElement('td', data, props.column.template(props.row.data)) in TableCell.vue works to render the data by calling the template function. Could you show me some doc about the template() function?

Hi @taozhaolib ,
I'm doing similar to what you described
<table-column v-for="column in columns"
        :show="column.show"
        :label="column.label"
        :header-class="column.headerClass"
        :cell-class="column.cellClass"
    >

I define a columns object in javascript and then the component uses to render the different columns.

hi . i use this technic but have error in syntax !
how i can add columns by v-for ??

i fixed it by add key
			<TableColumn v-for="(column,index) in columns"  :show="column.show" :label="column.label" :key="index"></TableColumn>


fix: Fix com.google.rpc.Code to StatusCode.Code conversion  logic

Also add tests for HTTP status code conversion (according to the current implementation, which may change in the future).

@viacheslav-rostovtsev FYI


How to use Watch mode

I'd like to execute some sort of watch command, so that any time I change a C# file, dotnet watch kicks in and restarts Kestrel and any time I change some web file, webpack watch kicks in and does HMR.
Since we have dotnet watch and two webpack config files, how can this be achieved?

dotnet watch will work fine to restart the app when you change C# code. Plus when you're running in development mode, Webpack will automatically rebuild and push your changes into the browser.


Topology volumes documentation

Add section for topology volumes feature
This feature is quite hidden behind "kubernetes magic" and as such I think we can keep it minimal like this commit changes.
K8s also has a lot of documentation about the feature which were added to this changes.

Also renamed the metrics-exporter docs from "metric exporter" to "metrics exporter" as that's the correct name everywhere

Correct 👍 That's the version we are targeting for this feature

@Ricardo-Osorio could you please merge this to a branch named release/v2.9 rather than to main ?

@Ricardo-Osorio this branch can just have everything on main as of now - thanks :) we can rebase later on


FormField: detect not fully supported children inputs

Overview
This draft is just focusing on detecting "fully supported" children. Not sure how to call this mode but right now it is just focusing on making it all look not broken. The padding needed to be adjusted on label and helper text because otherwise it looks whack. Switch looks bad either way.
Changes when a child input is not fully supported

remove status indicator
adjust padding on FormLabel and FormHelperText

New FormField QA story
The following can be seen under the Core > Form Field > QA > Wrapping Inputs
Before adjusted padding and bottom border

After adjusted padding and bottom border

Left aligned label
Not sure what to do about left aligned labels :/

@pseys @bhoppers2008 I forgot one more design issue with FormField and checkboxes/radios, the left side padding on the label makes its positioning look off. In this PR i remove the padding on non-input children types but not sure that is what we want. What do you think?

Closing since we are trying to achieve this a different way.


Rust warpshell v2

[X] convenient helpers for memory-mapped cores (GetDmaChannel, GetCtrlChannel, BaseParam)

[X] their integration into the Std shell
[X] reduced boilerplate impls thanks to procmacros

[X] renamed the library crate to warpshell and reset the version to 0.1.0
[X] barebones AXI firewall core
[X] HBICAP core

[X] skeleton Ops trait and integration with the Std shell
[X] working examples or benchmarks

[X] CLI binary (separate crate in rust/)
[X] merge v2 bitstream and scripts
[ ] update docs about flashing

[ ] xbflash2
[ ] standalone compilation instructions for xbflash2 without the rest of XRT

Wow! Looks good!

Did you successfully load a user.bin using this new HBICAP driver?

Not yet as I'm away from the machine but I can try remotely hoping it
wouldn't crash. I can't boot it back up when I'm not in 🤷
On Sat, 18 Mar 2023, 19:47 Tony Wu, @.***> wrote:

Did you successfully load a user.bin using this new HBICAP driver?
—
Reply to this email directly, view it on GitHub
https://github.com/Quarky93/warpshell/pull/12#issuecomment-1474953015,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKF7IOAHVIXXHWPI22ZZVDW4X7NHANCNFSM6AAAAAAVEUTEKM
.
You are receiving this because you authored the thread.Message ID:
@.***>

I've 'merged' (it was messy so I just committed the changed parts as new) warpshell-v2 back into main.
I guess we are ready for the new rust driver to be merged :grin: .

I've 'merged' (it was messy so I just committed the changed parts as new) warpshell-v2 back into main. I guess we are ready for the new rust driver to be merged grin .

Thanks! We'll add flashing instructions a bit later then.


Breakpoints are ignored in Ruby debugger

Issue Type: Bug
I'm running Ruby 2.5.1 and Rails 5.2 project.
I usually run RSpec and Rails server with the debugger, it used to work in the past, but now when I add breakpoints to a spec or a Rails controller the breakpoint is ignored (but for sure the code runs).
VS Code version: Code 1.22.2 (3aeede733d9a3098f7b4bdc1f66b63b0f48c1ef9, 2018-04-12T16:32:53.389Z)
OS version: Darwin x64 17.5.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-3520M CPU @ 2.90GHz (4 x 2900)

Load (avg)
2, 2, 2

Memory (System)
8.00GB (0.08GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (18)

Extension
Author (truncated)
Version

vsc-material-theme
Equ
2.0.1

vscode-docker
Pet
0.0.26

rails
bun
0.5.7

solargraph
cas
0.16.0

path-intellisense
chr
1.4.2

gitlens
eam
8.2.4

vscode-npm-script
eg2
0.3.3

ruby
gro
0.0.6

ruby-rubocop
mis
0.7.0

vscode-scss
mrm
0.6.2

sublime-keybindings
ms-
3.0.3

Ruby
reb
0.18.0

vscode-icons
rob
7.23.0

sass-indented
rob
1.4.9

ayu
tea
0.14.0

simple-ruby-erb
vor
0.2.1

vscode-import-cost
wix
2.6.2

JavaScriptSnippets
xab
1.6.0

(3 theme extensions excluded)

Please file this issue against the Ruby debugger extension.


feat: Ensure all images are loaded to K8s before starting e2e tests

Ⅰ.
Add imagePullPolicy: IfNotPresent to containers using an image with latest tag, so loaded images can be used.

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

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

Project coverage is 43.93%. Comparing base (ef31e09) to head (912f10a).
Report is 139 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1389      +/-   ##
==========================================
+ Coverage   35.91%   43.93%   +8.02%
==========================================
  Files          69       76       +7
  Lines       11576     9965    -1611
==========================================
+ Hits         4157     4378     +221
+ Misses       7104     5253    -1851
- Partials      315      334      +19

see 91 files with indirect coverage changes


Syntax highlighting of .moor files in VSCode (almost perfect)

To get syntax highlighting (at least the colors), I've configurated my VS code as this:
{
	"[sql]": {
		"editor.formatOnSave": false,
		"editor.formatOnPaste": false,
		"editor.formatOnType": false
	},
	"files.associations": {
		"**/*.moor": "sql"
	}
}

This will result in:

It works, but it shows errors because it tries to parse it as an MSSQL script.
When I set the language mode to sql.none, it works as intended:

Click here

Change to none here:

The, I get the .moor file with coloring and autocomplete (throught the moor extension), without any errors (as intended).

My question is: how to automatically configure the .moor extension to sql language mode, with NONE pre selected?

I believe this is what you're looking for: https://scottaddie.com/2016/04/10/file-to-language-associations-in-visual-studio-code/

After https://github.com/Dart-Code/Dart-Code/pull/2203 gets released (might take some time, the underlying apis are still experimental in VS Code) moor will be able to contribute its own highlighting.
You can already use moor's IDE with the Dart extension for VS Code. This will enable error reporting, folding, outline and (partly) navigation and auto-complete. It's still a bit buggy, but it might help.

I believe this is what you're looking for: https://scottaddie.com/2016/04/10/file-to-language-associations-in-visual-studio-code/

Not really. I already did all, except disable the MSSQL parsing automatically.

You can already use moor's IDE with the Dart extension for VS Code. This will enable error reporting, folding, outline and (partly) navigation and auto-complete. It's still a bit buggy, but it might help.

I'm already doing this (and I really don't care much for intellisense). But moor's IDE don't have syntax highlighting (it treats the file as text, with all text white).

Solved removing all sql extensions from VSCode (it will still recognize sql files with color syntax highlighting, but won't try to parse it).
As I use Microsoft SQL Server Management Studio or Azure Data Studio for SQL stuff, that plugin will not be missed.


[coffeescript] syntax highlighting breaks when using embedded JS

VSCode Version: Code 1.14.0 (b8cd5b9556d8b70ea560d35b903422363f6c5c40, 2017-07-10T18:24:22.120Z)
OS Version: Darwin x64 15.6.0
Extensions:

Extension
Author (truncated)
Version

material-icon-theme
PKi
2.0.10

vscode-paste-and-indent
Rub
0.0.8

vscode-custom-css
be5
2.3.2

jshint
dba
0.10.15

vscode-eslint
dba
1.2.11

gitlens
eam
4.3.1

vscode-elixir
mjm
0.5.0

prettify-json
moh
0.0.3

seti-icons
qin
0.1.3

final-newline
sam
0.3.2

vscode-fileutils
sle
2.6.2

guides
spy
0.8.4

vscode-open-in-github
sys
1.6.0

(1 theme extensions excluded)

Steps to Reproduce:

Open a coffeescript file with embedded JS imports at the top
Syntax highlighting is all JS based

Reproduces without extensions: Yes

Having the same problem

Same problem here too.

If you add a semicolon to the import statement it seems to work.
Looks like the problem is with our javascript grammar. The import statement does not end until there is a ':' or the end of the line. For supporting embedded cases like this, it would be better to end with the string literal.

I created https://github.com/Microsoft/TypeScript-TmLanguage/issues/497


Transfer Learning

Notebook: Transfer Learning
Now, you're ready to use transfer learning on a new task!
It's suggested that you open the notebook in a new, working tab and continue working on it as you go through the instructional videos in this tab. This way you can toggle between learning new skills and coding/applying new skills.
To open this notebook, you have two options:
Go to the next page in the classroom (recommended).
Clone the repo from Github and open the notebook Transfer_Learning_Exercise.ipynb in the transfer-learning folder. You can either download the repository with git clone https://github.com/udacity/deep-learning-v2-pytorch.git, or download it as an archive file from this link.
Instructions
Load in a pre-trained VGG Net
Freeze the weights in selected layers and add a new, linear layer of your own design
Train the modified model for a couple epochs and test its performance
This is a self-assessed lab. If you need any help or want to check your answers, feel free to check out the solutions notebook in the same folder, or by clicking here.
GPU Workspaces
The next workspace is GPU-enabled, which means you can select to train on a GPU instance. The recommendation is this:
Load in data, test functions and models (checking parameters and doing a short training loop) while in CPU (non-enabled) mode
When you're ready to extensively train and test your model, enable GPU to quickly train the model!
All models and data they see as input will have to be moved to the GPU device, so take note of the relevant movement code in the model creation and training process.

Notebook: Transfer Learning
Now, you're ready to use transfer learning on a new task!
It's suggested that you open the notebook in a new, working tab and continue working on it as you go through the instructional videos in this tab. This way you can toggle between learning new skills and coding/applying new skills.
To open this notebook, you have two options:
Go to the next page in the classroom (recommended).
Clone the repo from Github and open the notebook Transfer_Learning_Exercise.ipynb in the transfer-learning folder. You can either download the repository with git clone https://github.com/udacity/deep-learning-v2-pytorch.git, or download it as an archive file from this link.
Instructions
Load in a pre-trained VGG Net
Freeze the weights in selected layers and add a new, linear layer of your own design
Train the modified model for a couple epochs and test its performance
This is a self-assessed lab. If you need any help or want to check your answers, feel free to check out the solutions notebook in the same folder, or by clicking here.
GPU Workspaces
The next workspace is GPU-enabled, which means you can select to train on a GPU instance. The recommendation is this:
Load in data, test functions and models (checking parameters and doing a short training loop) while in CPU (non-enabled) mode
When you're ready to extensively train and test your model, enable GPU to quickly train the model!
All models and data they see as input will have to be moved to the GPU device, so take note of the relevant movement code in the model creation and training process.


Benchmark results

Hi there, Aeron looks very promising. I hope it becomes more popular and successful.
Could you please give some real benchmarks with other messaging systems for performance comparisons of real use case scenarios?  It sounds like there are many radical new approaches with Aeron's design but it is too complicated and low level for a developer who needs only a fast&reliable messaging system.

Comparison to other messaging systems is beyond the scope of this repository. We would be happy helping out if someone built this benchmark elsewhere.


Callback after Paging is done

Enhancement/feature description
Just like there is callback after filtering is done, is there any callback when paging is done as paging is completed after filtering.

The grouped-by-page event is probably what you are looking for. Below an example of usage:
https://codepen.io/koalyptus/pen/ZEboajz
Resources:

https://github.com/koalyptus/TableFilter/wiki/1.04-Paging#events

Cheers

thanks


Destructuring performance

I was wondering about the performance of destructuring within systems so I decided to run some tests:
import { World } from "miniplex";
import { Entity } from "./miniplex";

const count = 1_000_000;
const iterations = 10;

const world = new World<Entity>();

const positions = world.archetype("position");

const profile = <T>(name: string, fn: () => T) => {
  const before = Date.now();
  const result = fn();
  const after = Date.now();
  console.log(`${name} took ${after - before}ms`);
  return result;
};

const createEntities = () => {
  const entities: Entity[] = [];

  for (let i = 0; i < count; i++)
    entities.push(world.createEntity({ position: { x: 0, y: i, z: 0 }, age: i }));

  return entities;
};

it(`works`, () => {
  profile(`create`, createEntities);

  profile(`destructured`, () => {
    for (let iter = 0; iter < iterations; iter++)
      for (const { position } of positions.entities) position.x += Math.random();
  });

  profile(`not-destructured`, () => {
    for (let iter = 0; iter < iterations; iter++)
      for (const entity of positions.entities) entity.position.x += Math.random();
  });

  profile(`simple for`, () => {
    for (let iter = 0; iter < iterations; iter++)
      for (let i = 0; i < positions.entities.length; i++)
        positions.entities[i]!.position.x += Math.random();
  });
});

Results are:
  console.log
    create took 1794ms

  console.log
    destructured took 2363ms

  console.log
    not-destructured took 1502ms

  console.log
    simple for took 1473ms

So it looks like destructuring has quite a large impact.. having said that tho there could be some CPU vectorization / JIT'ing going on here that might cause oddness.

Turning around the order of execution yielded the following results, making destructured access appear significantly faster:
  console.log
    create took 603ms

  console.log
    not-destructured took 1196ms

  console.log
    destructured took 756ms

  console.log
    simple for took 751ms

As far as I know, the JavaScript runtime is pretty efficient at inlining destructuring statements. I assume what you're seeing here is the effect of caching.

The difference evens out with a lower number of entities and a higher number of iterations:
  console.log
    simple for took 752ms

  console.log
    not-destructured took 724ms

  console.log
    destructured took 719ms

Ye okay.. I was seeing odd numbers when I didnt use Math.random() too.
I was hoping it would be smart about the destructuring its good that these tests have been done tho :)


Getting 503 when requesting from Hetzner VPS

simon@apollo:~$ wget https://api.spacexdata.com/v4/launches/latest
--2022-09-11 00:16:22--  https://api.spacexdata.com/v4/launches/latest
Resolving api.spacexdata.com (api.spacexdata.com)... 2606:4700:3035::ac43:b70f, 2606:4700:3032::6815:240e, <IP_ADDRESS>, ...
Connecting to api.spacexdata.com (api.spacexdata.com)|2606:4700:3035::ac43:b70f|:443... connected.
HTTP request sent, awaiting response... 503 Service Temporarily Unavailable
2022-09-11 00:16:22 ERROR 503: Service Temporarily Unavailable.

Requesting from my local machine I get 200 and a proper response.
On the VPS I started getting a 403 at 2022-09-04 23:15:37 (UTC), then switched to 503 at 2022-09-05 03:01:16
I am running a Discord bot that makes ~1 request per minute in a loop (plus occsional other requests), which as far as I was aware was fine.
Seems to be the same issue as #1209? I've stopped the bot for now in hopes that Cloudflare will cool down.
If I curl I can see the same message they were getting; "api.spacexdata.com needs to review the security of your connection before proceeding"

Thought I reset most of the Cloudflare settings I was fiddling with, but looks like I left Bot Fight Mode on, I bet this was the issue. Let me know if it's still giving 503's

Yep 503's have stopped now, thanks!


Solve result of (x^2-1)^2-2 is wrong

Also true for (x^2-1)^2-a where a > 2.

Serious bug. Fixed. I didn't consider case beta = 0 for Ferrari's formula.


Fix v1.7.1 issues

After installing v1.7.1 with npm i -g markserv, there are some issues:

Missing CSS files in directory lib/templates.
The request {Markserv}templates/markserv.css always returns 404, but changing the code in server.js to const markservUrlLead = '{markserv}' yields the correct content.

Improve these:

Change emoji characters to shortcuts: Affect the heading anchors.
Add emoji-regex and markdown-it-emoji modules: This will generate the same heading anchors as github.
Add function ahref(): Generate more secure links.
Return a more secure page when a request error occurs.
Disable dot-files in dir listing. Maybe we can add a command parameter to toggle it.
If the livereloadport is zero, the liveReload is completely disabled.
Delete the file github.less, which is almost the same as markserv.css.
Remove unused prettyPath variable from dirToHtml().
Change to Responsive Web Design, better for mobile browsing.
Update the URL of MathJax.js, the original cdn url warns:
WARNING: cdn.mathjax.org has been retired. Check https://www.mathjax.org/cdn-shutting-down/ for migration tips.
I changed your crumbs code and my code in html is a bit ugly.

The secure link means the user enters the url http://example.com/,
markserv should return 404 raw data:
404 :'( for Error: ENOENT: no such file or directory, stat '/homedir/<script>alert('test')</script>'

but not:
404 :'( for Error: ENOENT: no such file or directory, stat '/homedir/

Chrome will block this returned page, but Firefox will execute the script.
Similarly, when generating a directory listing, be careful with URL and FILE NAME in FILE NAME.

Will get to this when I can. Thank you!

I'm looking at how you updated the URLs:

Before: /markserv}templates/markserv.css
After: /{markserv}/templates/markserv.css

This causes a 404 for me on macOS Chrome.

When I echo out the markservUrl...
// lib/server.js:373
		const isMarkservUrl = req.url.includes(markservUrlLead)

		// Double check the url in the request Handler
		console.log(isMarkservUrl, req.url)

		if (isMarkservUrl) {
			const markservFilePath = req.url.split(markservUrlLead)[1]
			const markservRelFilePath = path.join(__dirname, markservFilePath)
			if (flags.verbose) {
				msg('{markserv url}', style.link(markservRelFilePath), flags)
			}
			send(req, markservRelFilePath, {
				immutable: true,
				maxAge: '1d'
			}).pipe(res)
			return
		}

I get this:

Perhaps your browser/os acts differently when it provides the path?
What do you see when you console.log(isMarkservUrl, req.url) ?

What Browser and OS are you using @ChenYingChou ?

I'm wondering if you changed markservUrlLead because it was easier to read? Or because things would not work without the change?

I'm sorry @ChenYingChou I accidental merge. Can you push to this branch again to re-open?

Here is my server: https://md.tvmall.com.tw/
tony@dev:~/markdown> cat /etc/centos-release
CentOS Linux release 7.4.1708 (Core)

tony@dev:~/markdown> ls -lF
total 0
lrwxrwxrwx 1 tony tony 11 May  5 18:21 markserv -> ../markserv/
drwxrwxr-x 5 tony tony 61 Apr 29 01:16 oisp/
drwxrwxr-x 5 tony tony 43 May  8 02:37 T & 中文<%2Fsub>/

tony@dev:~/markdown> node ~/markserv/lib/cli.js -p 8002 -l 0
  Markserv   address: http://localhost:8002
  Markserv   path: /home/tony/markdown
  Markserv   process: your pid is: 4928
  Markserv   stop: press [Ctrl + C] or type "sudo kill -9 4928"
   GitHub   Contribute on Github - github.com/f1lt3r/markserv
 ▕● PATREON  Help support Markserv - Become a Patreon! patreon.com/f1lt3r

I run nginx with reverse-proxy:
# Markdown Server
server {
    server_name  md.tvmall.com.tw;
    include ssl.conf;
    location / {
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $remote_addr;
        proxy_set_header Host $host;
        proxy_pass http://<IP_ADDRESS>:8002/;
    }
}

I'm wondering if you changed markservUrlLead because it was easier to read? Or because things would not work without the change?

I found that the URL through the nginx reverse proxy will be decoded, so %7Bmarkserv%7D is changed to {markserve}.
If you directly request to http://ip:port/{markserv}, the URL is still "%7Bmarkserv%7D".
Make the following changes to properly handle both modes:
@@ -370,9 +370,10 @@ const createRequestHandler = flags => {
                        msg('request', filePath, flags)
                }

-               const isMarkservUrl = req.url.includes(markservUrlLead)
+               const decodedReqUrl = unescape(req.url)
+               const isMarkservUrl = decodedReqUrl.includes(markservUrlLead)
                if (isMarkservUrl) {
-                       const markservFilePath = req.url.split(markservUrlLead)[1]
+                       const markservFilePath = decodedReqUrl.split(markservUrlLead)[1]
                        const markservRelFilePath = path.join(__dirname, markservFilePath)
                        if (flags.verbose) {
                                msg('{markserv url}', style.link(markservRelFilePath), flags)

Can you push to this branch again to re-open?

Should I make a new PR?

Ideally we should have a preference for each feature you're adding/updating. But I'm ok with a single new PR if that's all you have time for.

@ChenYingChou now I see how you are using the internal {markserv} url through a proxy, we might do better to use safe url characters only.
Perhaps something like __markserv__ or  .markserv is better? It probably doesn't matter what it looks like as this will be internal facing.


wma Audio File not Play

Checklist

[x] I read the troubleshooting guide before raising this issue
[x] I made sure that the issue I am raising doesn't already exist

Current bug behaviour

Expected behaviour

Steps to reproduce

Execute flutter run on the code sample
...
...

Code sample

void main() {
}

Logs

my relevant logs

Full Logs

my full logs or a link to a gist

Flutter doctor:
Output of: flutter doctor -v

Audio Files/URLs/Sources

Sample Url: https://luan.xyz/files/audio/coins.wav
Sample Stream: https://example.com/my_stream.m3u8

Screenshots

Environment information

audioplayers version:

Platform 1:

OS name and version:
Device:

build mode: debug

error peculiarities (optional):

Platforms tested without any issue (optional):

test peculiarities:

More information

Can you provide the file as .zip and fill the issue template? Which platform etc...Thx


[minor] Why Model property is ignored instead of Microsoft.EntityFrameworkCore.Metadata.IModel type?

Hi,
I just started using this awesome library with existing, already-modeled entities.
Unfortunately many of my entities has a property named 'Model', which is ignored by default.
I figured that I can work around that by doing this:
builder.Services.AddGraphQLSchema<MyDbContext>(options =>
{
    options.IgnoreTypes.Add(typeof(Microsoft.EntityFrameworkCore.Metadata.IModel));
    options.IgnoreProps.Remove("Model");
}

My question is - is there any reason not to make this the default behavior?
Thanks!

Yeh I’m all for using types vs the name but currently the library doesn’t have a dependency on any Entity Framework library.

From: Eitan Har-Shoshanim @.>
Sent: Saturday, June 1, 2024 4:40:48 PM
To: EntityGraphQL/EntityGraphQL @.>
Cc: Subscribed @.***>
Subject: [EntityGraphQL/EntityGraphQL] [minor] Why Model property is ignored instead of Microsoft.EntityFrameworkCore.Metadata.IModel type? (Issue #376)
Hi,
I just started using this awesome library with existing, already-modeled entities.
Unfortunately many of my entities has a property named 'Model', which is ignored by default.
I figured that I can work around that by doing this:
builder.Services.AddGraphQLSchema(options =>
{
options.IgnoreTypes.Add(typeof(Microsoft.EntityFrameworkCore.Metadata.IModel));
options.IgnoreProps.Remove("Model");
}
My question is - is there any reason not to make this the default behavior?
Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/EntityGraphQL/EntityGraphQL/issues/376, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AABGHUILO45FWKLYTN55JXDZFFUHBAVCNFSM6AAAAABIT42HNKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGMZDQOJVHEZTENI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Got it. thanks.


Incorrect Calculation  _az_http_policy_logging_append_http_request_msg()

File : sdk\src\azure\core\az_http_policy_logging.c
• API : _az_http_policy_logging_append_http_request_msg( )
• Line : 73
• Issue : Incorrect calculation of required_length.
• The value needs to be incremented, not reassigned.
• Required length is being calculated using four values:
• “HTTP Request : ” 15 in length (http_request_string)
• HTTP Method used (_internal.method)
• Space
• Actual length of URL (_internal.url_length)
The length of http_request_string is missing from the calculation.
• Effect : Machine goes to Infinite Loop.
• Recommendation: Modify “=” to “+=” in L:73

Although there are no callers for the internal policies in main yet, this looks like a valid correctness bug which we may want to prioritize fixing (verified from https://github.com/Azure/azure-sdk-for-c/pull/549, the intent is to add the http_request_string length).
https://github.com/Azure/azure-sdk-for-c/blob/9f5a6334863ebbf214c6e2fbd5d6e3fce52fe0de/sdk/src/azure/core/az_http_policy_logging.c#L66-L74
Moving to July milestone, to try and catch the next release cycle.

cc @RickWinter

Thank you @ericwol-msft, this is a bug, we should fix it. And thank you for the detailed info, you basically wrote the fix as well!
@ahsonkhan, you are right, there should be no callers in main branch at this moment.

@antkmsft Lets fix this for the next release


Example code could stand to be cleaned up a little to minimize confusion

Examples should be, at a minimum, correct. Finding methods like this without an async call in the body:
[FunctionName("SignalRTest")]
public async Task SendMessage([SignalRTrigger]InvocationContext invocationContext, string message, ILogger logger)
{
    logger.LogInformation($"Receive {message} from {invocationContext.ConnectionId}.");
}

[Enter feedback here]

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

ID: 30b9a9bc-b82b-ce66-d0dc-3ff340da9e93
Version Independent ID: ab44deb2-a97e-a190-b041-9063eb17ff30
Content: Azure Functions SignalR Service trigger binding
Content Source: articles/azure-functions/functions-bindings-signalr-service-trigger.md
Service: azure-functions
GitHub Login: @Y-Sindo
Microsoft Alias: zityang

@JPL-Alliant
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for your feedback. The doc will be updated soon.

#please-close


[CosmosDB] Add PrivateEndpointConnections to DatabaseAccountGetResult

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

The properties I added were pre existing in Cosmos DB RP but was missing from Swagger. Adding those properties to close the gap between Cosmos DB RP server and Swagger.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

@anthony-c-martin  I addressed your comments. Please help review again.


okay the speeed hacked

okay the speeed hacked worked but none of the other worked i checked my pid again and it was the same

if speedhack works you have the right PID. Make sure you're on the latest version of Firefox.


Client Allocation Request for: 北京中经天平科技有限公司

Client Allocation Questions
Core Information

Name: 北京中经天平科技有限公司
Website / Social Media: https://www.bjzjtp.com
Region: Greater China
DataCap Requested: 5TiB
Addresses to be Notarized: f1sfq74iqhoseslc7vofvbdfknfo6tbtj2gnimgni
Notary Requested: ozhtdong

Thanks for your application. I want to know more basic information of you and your company. Please answer following question:
1.What's your name? What's your position? Can you introduce yourself?
2.What's your company's business? Any other social media link? Please introduce your company.
3.Could you show me your company's business license?

Thanks for your quick response and here are our informations:
1.What's your name? What's your position? Can you introduce yourself?
Name:  Wang Bo
Position: Director of Depository Services
Self-introduction: I have been engaged in judicial related industries for more than 10 years and participated in a large number of evidence collection work such as CCTV Spring Festival Gala forensics, World Cup forensics, World Internet Conference depositing evidences, Jinri toutiao Short Video forensics, Oriental IC picture forensics, Tencent audio book forensics, Baidu News forensics, etc.
2.What's your company's business? Any other social media link? Please introduce your company.
Company’s Business：Our business is to solidify the electronic data generated in various industries such as wills, finance, copyright, proof, electronic contracts, safety transportation, and judicial enforcement into electronic data evidence, which provides important reference value for the parties' rights protection, exemption, and traceability.
Social Media: zjtpyzx (WeChat Official Account)
Company Profile：
Beijing Zhongjing Tianping Technology Co., Ltd. (hereinafter referred to as Zhongjing Tianping), founded in 2017, is a joint-stock enterprise subordinate to the State Information Center (China Economic Network) and a strategic partner of the National Internet Emergency Response Center. With the approval of the Ministry of Justice, Zhongjing Tianping has established the Electronic Material Evidence Forensic Identification Center.
Zhongjing Tianping focuses on product research and development in the field of national legal system construction and information construction. As an Internet technology company,our company is supported by core technologies and driven by participating in and promoting the construction of intelligent justice. At the same time, Zhongjing Tianping is also the first batch of domestic blockchain information service filing enterprise in China, the participating enterprise of the two high-level and one key special project (litigation risk result prediction platform based on multi-party evidence correlation analysis), the participating enterprise for writing version 1.0 of the White Paper on Blockchain Judicial Depository Application, and the Shuxin chain ecological application enterprise .
At present, Zhongjing Tianping has served more than 300 courts in 15 provinces and municipalities directly under the central government, and our business covers many industries such as will, finance, copyright, proof production, electronic contract, traffic safety, judicial execution, etc.

In May 2018, Zhongjing Tianping, National Time Service Center of Chinese Academy of Sciences and Law Branch of China Information Association held a joint press conference on "Judicial Electronic Evidence Cloud Platform" and "Trusted Time Authentication Service Platform" at the State Information Center.
http://www.xinhuanet.com/politics/2018-05/25/c_1122888756.htm)
https://www.chinacourt.org/article/detail/2018/05/id/3318363.shtml)
https://baijiahao.baidu.com/s?id=1601423212828948507&wfr=spider&for=pc)

In March 2019, Zhongjing Tianping obtained the first batch of domestic blockchain information service records of the Cyberspace Administration of China.
http://www.cac.gov.cn/2019-03/30/c_1124305122.htm)

In June 2019, Zhongjing Tianping  participated in writing version 1.0 of the White Paper on Blockchain Judicial Depository Application.
http://www.caict.ac.cn/kxyj/qwfb/bps/201906/t20190614_201169.htm)

3.Could you show me your company's business license?
Yes, of course. Below is our company business license.

Hi,  ozhtdong, sorry to bother you~
We haven't heard from you for several days.  We are wondering the processing progress and look forward to receiving your reply as soon as possible. Thanks a lot.

@ozhtdong Hi,  ozhtdong, sorry to bother you~
We haven't heard from you for several days.  We are wondering the processing progress and look forward to receiving your reply as soon as possible. Thanks a lot.


Need to set session.caller_disclose_me

Hello,
I need to set the caller_disclose_me flat on the session, but it seems that it doesn't work. Do you know why that could be? autobahnjs is on version 0.9.8. (I tried it without angular-wamp on the same server and that works)
$rootScope.$on('$wamp.open', function (event, info) {
	console.log('We are connected to the WAMP Router!', event, info);
	info.session.caller_disclose_me = true;
	$rootScope.$emit('flatoutAuth:connected', info);
});

Thank you very much!

@creat-or as far as I can tell, that should work.  I tried your code and inspected session and it is setting the _session.caller_disclose_me property to true on the session.
The only thing I can think of is that the RPC calls are being made before that property is set (all wamp actions are queued until the connection is opened).
Can you make an RPC request inside of your $wamp.open after you set caller_disclose_me?  If that works, I may have to add those options to $wampProvider.init, so that they're set before any of the queued calls are made.

OK, I'm sorry. It is NOT a problem of $wamp I guess. I now tried to connect it in a plunkr and it is working. In my app, however the session object looks slightly different. Do you know what can be the cause that I have no session._caller_disclose_me? (All calls are made after the $wamp.open, btw)
session object on plunkr:
{
  "_socket": {
    "info": {
      "type": "websocket",
      "url": "ws://xyz.com:3001/router",
      "protocol": "wamp.2.json"
    }
  },
  "_id":<PHONE_NUMBER>536672,
  "_realm": "gateway",
  "_features": {
    "subscriber": {
      "publisher_identification": true,
      "pattern_based_subscription": true,
      "subscription_revocation": true
    },
    "publisher": {
      "publisher_identification": true,
      "subscriber_blackwhite_listing": true,
      "publisher_exclusion": true
    },
    "caller": {
      "caller_identification": true,
      "progressive_call_results": true
    },
    "callee": {
      "caller_identification": true,
      "pattern_based_registration": true,
      "shared_registration": true,
      "progressive_call_results": true,
      "registration_revocation": true
    }
  },
  "_goodbye_sent": false,
  "_transport_is_closing": false,
  "_publish_reqs": {},
  "_subscribe_reqs": {},
  "_unsubscribe_reqs": {},
  "_call_reqs": {},
  "_register_reqs": {},
  "_unregister_reqs": {},
  "_subscriptions": {},
  "_registrations": {},
  "_invocations": {},
  "_prefixes": {},
  "_caller_disclose_me": false,
  "_publisher_disclose_me": false,
  "_MESSAGE_MAP": {
    "8": {}
  },
  "_created":<PHONE_NUMBER>0000005
}

session object on my app (no _caller_disclose_me or _publisher_disclose_me like above)
{
  "_socket": {
    "info": {
      "type": "websocket",
      "url": "ws://xyz.com:3001/router",
      "protocol": "wamp.2.json"
    }
  },
  "_id":<PHONE_NUMBER>374458,
  "_realm": "gateway",
  "_features": {
    "subscriber": {
      "publisher_identification": true
    },
    "publisher": {
      "subscriber_blackwhite_listing": true,
      "publisher_exclusion": true,
      "publisher_identification": true
    },
    "caller": {
      "caller_identification": true,
      "progressive_call_results": true
    },
    "callee": {
      "progressive_call_results": true
    }
  },
  "_goodbye_sent": false,
  "_transport_is_closing": false,
  "_publish_reqs": {},
  "_subscribe_reqs": {},
  "_unsubscribe_reqs": {},
  "_call_reqs": {},
  "_register_reqs": {},
  "_unregister_reqs": {},
  "_subscriptions": {},
  "_registrations": {},
  "_invocations": {},
  "_prefixes": {},
  "_MESSAGE_MAP": {
    "8": {}
  },
  "_created": 1750.89
}

Oh I'm so sorry. I installed autobahn via bower with autobahnjs but needed to install autobahn. So I had the old 0.9.6 version in the system and this caused the issue. 😲


Save comment when user clicks "submit"

For eregs/notice-and-comment#298, the comment is saved when the "Review and Submit" button is clicked even if the user hasn't clicked save.

Hm. We save empty comments, which we probably shouldn't, but is more prominent with this change. Fixing that (not saving empty comments) opens up a bigger can of worms though (we'd need error messages, figure out if blanking out a comment means deleting the original comment, etc.). I'd recommend we punt on that given our limited runway. Is there a simpler solution? Maybe we just redesign that link so it doesn't look like a button?

Per https://github.com/eregs/notice-and-comment/issues/298#issuecomment-224630338 , we'll punt for now. Is there anything in this PR which we should merge anyway, @xtine?

I'm fine with punting this, there isn't anything else that needs to be merged in.

:+1: closing


Add default block in CompositeByteBuf

Motivation:
Fix switch case fall through, add default block in CompositeByteBuf

Thanks a lot

@normanmaurer, it is my pleasure


macosx nativemidi special case in music_internal_play()

This bug report was migrated from our old Bugzilla tracker.
Reported in version: unspecified
Reported for operating system, platform: Mac OS X (All), All
Comments on the original bug report:
On 2018-08-02 08:25:57 +0000, Ozkan Sezer wrote:

In music.c:music_internal_play()
http://hg.libsdl.org/SDL_mixer/file/0eda5db47348/music.c#l681
static int music_internal_play(Mix_Music *music, int play_count, double position)
{
int retval = 0;
#if defined(MACOSX) && defined(MID_MUSIC_NATIVE)
/* This fixes a bug with native MIDI on Mac OS X, where you
can't really stop and restart MIDI from the audio callback.
/
if (music == music_playing && music->api == MIX_MUSIC_NATIVEMIDI) {
/ Just a seek suffices to restart playing */
music_internal_position(position);
return 0;
}
#endif
As far as I can see, Mix_MusicInterface_NATIVEMIDI has a NULL
Seek function (see in music_nativemidi.c#l97) and as a result,
music_internal_position() will do nothing returning (-1), and
nothing will happen.
Is this really intended?  Am I missing something?

On 2018-08-07 07:32:56 +0000, Ozkan Sezer wrote:

Any ideas about this?

Nope, you're right, that was just old dead code.


Wait for all xhr been resolved

Does puppeteer has API for handle case, when page send xhr response? Example
<button id="btn" onclick="sendXHR()">Send</button>

test.js
await page.click('#btn');
// some code which wait xhr response

As I understand, page.waitForNavigation api waits for redirect and not suitable for this case

@b-ponomarenko you can subscribe to 'request' events to sniff page's traffic. XHR's are distinguishable with their rourceType():
page.on('request', r => {
  if (r.resourceType() === 'xhr')
    console.log('XHR: ' + r.url());
});

Hope this helps.

Just some hints for future folks who will visite this page:
For initial loads, you can use:
await page.goto('http://localhost:4000/trailflow', {
  waitUntil: ['domcontentloaded', 'networkidle0']
})

Just some hints for future folks who will visite this page:
For initial loads, you can use:
await page.goto('http://localhost:4000/trailflow', {
  waitUntil: ['domcontentloaded', 'networkidle0']
})


Question about computing losses

Hi,
Thank you for your work, it is very helpful.
I was wondering if you could explain why you use x2_inpaint.detach() when running a forward pass through the discriminator for calculating the discriminator's loss, but do not call detach() when calling the discriminator for calculating the generator's loss.
In this case, gradient will not be computed for the D loss for the fake images, but will for the G loss.
I understand why you use detach() when calculating the gradient penalty (as you only want to use the gradient for the GP term and not recalculate gradient for the discriminator again, but I cannot seem to understand why it is used as described above.
Thank you

Hi,
For the detach() here, I followed the official PyTorch DCGAN implementation example.
Here is the discussion that may help you: why is detach necessary
In my opinion, it is mainly for efficiency and not retaining the graph.

@daa233 Thank you very much for the quick response! I had a feeling it was purely for efficiency, it seems it is just to avoid recalculating the gradient for the inpainted  images. Both links you provided are very helpful.
Thank you again for your work!


[Bug]: unclear error and too general typing on externals field

System Info
  System:
    OS: macOS 14.6.1
    CPU: (10) arm64 Apple M2 Pro
    Memory: 174.80 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.16.0 - ~/.nvm/versions/node/v20.16.0/bin/node
    npm: 10.8.1 - ~/.nvm/versions/node/v20.16.0/bin/npm
    pnpm: 9.7.1 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 130.0.6723.70
    Safari: 17.6
  npmPackages:
    @rspack/cli: ^1.0.14 => 1.0.14
    @rspack/core: ^1.0.14 => 1.0.14

Details
When compiling to a library with type 'umd' and using externals, Rspack will error out if you forgot to add the 'root' | 'commonjs' | 'commonjs2' properties to one of your externals.
for example:
Good ✅
export default {
  output: { library: { type: "umd" } },
  externals: {
    react: {
      commonjs: "react",
      commonjs2: "react",
      root: "React"
    }
  }
}

Bad ❌
export default {
  output: { library: { type: "umd" } },
  externals: {
    react: {
      commonjs: "react",
      commonjs2: "react" // Notice no 'root' property. But removing any of these three will result in an error.
    }
  }
}

This makes sense as critical information is missing for Rspack to be able to construct a correct UMD import of the react library.
The bad example will throw the following error:
[rspack-cli] [Error:   × Missing external configuration for type: root
] {
  code: 'GenericFailure'
}

How Webpack treats this
When you do this in webpack - webpack compiles successfully without any error. but the resulting code has errors in it.
So i think Rspack actually does the right thing here but fails to communicate this properly.
for example if you had this code:
import React from 'react'
console.log(React)

webpack would output something like this:
	if(typeof exports === 'object' && typeof module === 'object')
		module.exports = factory(require("react"));
	else if(typeof define === 'function' && define.amd)
		define([], factory);
	else {
                // 🚧 notice the root[undefined] here which should have been root['React']
                // but because webpack doesn't have a 'root' property it just puts in undefined.
                //                                                  This will throw in production👇🏻
		var a = typeof exports === 'object' ? factory(require("react")) : factory(root[undefined]);
		for(var i in a) (typeof exports === 'object' ? exports : root)[i] = a[i];
	}

My Conclusion
I think Rspack does the right thing here I think that there's still some stuff missing in communication and DevX areas:

The error that Rspack returns is quite vague and general. would be better to say what specific external is missing what properties.
The types are too general (Record<string, string | string[]>) and don't let typescript users know that they are defining this configuration wrong. Would be better if when the library type is umd there was an actual interface that would let users know what properties should be required there (commonjs, commonjs2, amd & root).
Documentation around externals doesn't enumerate the possible values and which are required in which case. https://rspack.dev/config/externals

Reproduce link
https://github.com/ItamarGronich/rspack-externals
Reproduce Steps

clone the repo
run nvm use
run npm ci
run npm run test:rspack to see the error printed out
run npm run test:webpack to see no error printed out

I tried to fix the zod types and documents. We will try to find a better way to validate this.

In https://github.com/web-infra-dev/rspack/pull/8369, a cross-field validator has been added. In umd, strict validation can be applied on the externals object items. For other types, these fields are not required to be filled.


Rename Sort Group by

Related issues:  #427
Renamed navigation point to Sort group by Median.

I proposed an extension for later in issue https://github.com/Caleydo/lineupjs/issues/448.


piece_expected_num in IoUNegPiecewiseSampler

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
To avoid the error mentioned in https://github.com/open-mmlab/mmdetection3d/issues/2004#issue-1439631905
Modification
Update the piece_expected_num variable
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

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

@oyel Please sign the CLA.


Add option max-tags-per-database to limit high cardinality data

Much as max-series-per-database in https://github.com/influxdata/influxdb/pull/7095, add a max-tags-per-database to limit high cardinality data.
Writes beyond that should be dropped and logged with a rate limit on the log (one summary log per minute or something saying x writes dropped).
This is important because if you accidentally load in a huge amount of high cardinality data, you can easily get into a place that InfluxDB will OOM if you attempt to delete the data, so your only choice is to try to move the files on disk out the way which deletes other data.

Would add to this that we should have alerts to the log when we start to
get close to the threshold, like 85%, 90%, etc.
Helps us stay proactive in cleanup and maintenance.
On Thursday, August 11, 2016, Alex Davies<EMAIL_ADDRESS>wrote:

Much as max-series-per-database in #7095
https://github.com/influxdata/influxdb/pull/7095, add a
max-tags-per-database to limit high cardinality data.
Writes beyond that should be dropped and logged with a rate limit on the
log (one summary log per minute or something saying x writes dropped).
This is important because if you accidentally load in a huge amount of
high cardinality data, you can easily get into a place that InfluxDB will
OOM if you attempt to delete the data, so your only choice is to try to
move the files on disk out the way which deletes other data.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/influxdata/influxdb/issues/7146, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADA1nuIdOrtPkcMwEeq8XvT9D6NiTP0gks5qe58egaJpZM4Jim-5
.

--
Sebastian Borza
PGP: EDC2 BF61 4B91 14F2 AAB4  06C9 3744 7F3F E411 0D3E

Of course if we do https://github.com/influxdata/influxdb/issues/7151 well (which looks great, thanks!), this becomes a non-issue; this is more of a sticking plaster until that can be done!

A similar proposal is in #6679

This is important because if you accidentally load in a huge amount of high cardinality data, you can easily get into a place that InfluxDB will OOM if you attempt to delete the data,
I am curious, why would InfluxDB OOM when I attempt to delete the data?


Closing streams twice crashes JVM

There are currently no checks in the close() methods of ZstdInputStream/ZstdOutputStream. When attempting to close an already closed stream, the freeDStream/freeCStream native methods are called again, resulting in a segmentation fault and crashing the JVM.
The streams should behave like the default DeflaterOutputStream/InflaterInputStream by maintaining a closed flag and doing nothing if it has been set. It could also be used to prevent potential crashes in the read/write methods, by throwing an exception if the stream has already been closed.

yes, you are right

Fixed in the last commit. I will publish/release it with zstd-1.1.2 that should come soon,

Great, thanks!


cask-repair infinite confirmation loop

I try to submit casks but it keeps asking me to confirm again and again.

https://github.com/vitorgalvao/tiny-scripts/issues/150
The problem is with brew cask style. That’s what you need to fix.


Making development compose file more similar to production

This change makes it possible to use the Docker Compose file in development again. I will document how to use this in a future PR.

How does one use this in development?

That's funny. I was positive I had addressed this in the commit message, but it's not there.
When I committed this I planned to call this out and say that it is on my list for a subsequent commit. It's a little more detailed than just "run this command".

Okay, uh... LGTM, then? 👍
/me has no idea what this does.

Well, you don't have to merge it. I'm the only one impacted by this right now anyway.
I'll create this again when I have a chance to document it.


Pod install not working on Xcode 14

When I run pod install using this Podfile

The Pods returns that error...

If I remove the specific version from pod ParticleNetworkBase, when I run the Project the Xcode returns this error:
Module compiled with Swift 5.6.1 cannot be imported by the Swift 5.7 compiler: /Users/andre/Documents/XXX/xxx-ios/Pods/ParticleNetworkBase/Frameworks/ParticleNetworkBase.framework/Modules/ParticleNetworkBase.swiftmodule/arm64-apple-ios.swiftmodule

If you are using Xcode 14, you should specify version number in Podfile.
Remove Podfile.lock then execute 'pod install' in
terminal again.

Thanks, It worked! I tried this yesterday but not working! Today it worked when I deleted the derived data to.
Thanks for your help.


some help in README and also correction in templates/management.html

Hi, there was a problem in templates/management. I fixed it and also added some info in README so it can be solved easier.

Thanks! Merge in https://github.com/nicolaiarocci/flask-sentinel/commit/03232c4e6da27142706ede2fe5edfed7de3bedda


Adding multiple middleware re-initializes each middleware more than once

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from fastapi import FastAPI
from fastapi.responses import RedirectResponse
from starlette.middleware.sessions import SessionMiddleware
from fastapi.middleware.httpsredirect import HTTPSRedirectMiddleware

app = FastAPI()

app.add_middleware(SessionMiddleware, secret_key="test secret key", https_only=False)
app.add_middleware(HTTPSRedirectMiddleware)

@app.get("/")
def read_root():
    return RedirectResponse("/home")

Description

add breakpoints or print statements in HTTPSRedirectMiddleware.__init__
Run the app server. Your breakpoint will hit HTTPSRedirectMiddleware.__init__ more than once!

I'm trying to understand why fastapi/starlette re-initializes middleware IFF the user wants to add more than one middleware, as per my example.
I worked on a couple of other ideas to work around this:

I can see that instead of calling app.add_middleware... I could have also passed in a list of middlewares to the app, like so

from fastapi import FastAPI
from fastapi.middleware import Middleware
from fastapi.middleware.httpsredirect import HTTPSRedirectMiddleware
from starlette.middleware.sessions import SessionMiddleware

user_middlewares = []
user_middlewares.append(Middleware(HTTPSRedirectMiddleware))

user_middlewares.append(Middleware(SessionMiddleware, secret_key="test secret key", https_only=False))

app = FastAPI(middleware=user_middlewares)

The problem with this approach is that my custom middleware (not shown here as I'm using public ones) needs to have an instance of the FastAPI app passed in. That doesn't work in this case. I make that work in my current implementation by doing it like this
app = FastAPI()
app.add_middleware(MyCustomMiddleware, original_app=app)

which is my preferred option right now. But as mentioned above, since I'm adding multiple middleware by calling add_middleware twice, I don't want MyCustomMiddleware to be initialized twice!
My middleware is defined as follows (redacted for privacy):
from contrast.agent.middlewares.base_middleware import BaseMiddleware
class FastApiMiddleware(BaseHTTPMiddleware):
    def __init__(self, app: ASGIApp, original_app: FastAPI = None) -> None:
        self.app = app
        self.original_app = original_app
        self.app_name = self.original_app.title

        self.dispatch_func = self.dispatch_func
       # other critical work that should only happen ONCE ever

    async def dispatch_func(self, request: Request, call_next):
        ....
        # unrelated work that happens at every request

I'm also aware that the fastapi way of adding middleware is with the @app.middleware("http") decorator described in the docs https://fastapi.tiangolo.com/tutorial/middleware/. However, As shown in MyCustomMiddleware implementation, I need it to be class-based for initialization and then for call_next later on

Is the multiple middleware initialization a design decision or an issue? Is there another way for me to design my CustomMiddleware, be able to add it and any other number of middleware to my app, but without having multiple initialization?
Operating System
Linux, macOS
Operating System Details
n/A
FastAPI Version
0.68.0
Python Version
3.9
Additional Context
No response

check starlette discussions
this is a duplicated issue on both projecs tho

@Kludex thanks for pointing it out. YOu're correct. https://github.com/encode/starlette/discussions/1161 I'll close this and hope it's fixed soon in starlette

I didn't say it was something that needs to be fixed tho :wink:
Let's continue the discussion there! Thanks :)


requireSpacesInAnonymousFunctionExpression: Add exception for shorthand methods

See discussion in https://github.com/jscs-dev/node-jscs/issues/1470#issuecomment-118666767
Need to support allExcept: ['shorthandMethods']

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/23862586-requirespacesinanonymousfunctionexpression-add-exception-for-shorthand-methods?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

+1
Real need
class Foo {
	constructor(bar) { // Missing space before openning round brace

	}
}

+1

:+1:

Any takers? Should be veeery easy

+1
@markelog I'll take it!

nevermind, looks like this was fixed in a8296dbfcdf87c7864ade7fe169d19d315552a1e ?

This is still broken for me on 2.1.0. Here's my config:
{
  "requireSpacesInAnonymousFunctionExpression": {
    "beforeOpeningRoundBrace": true
  },

  "esnext": true
}

My test program:
var o = {
  foo() {
    return 42;
  }
};

JSCS output:
Missing space before opening round brace at demo.js :
     1 |var o = {
     2 |  foo() {
-------------^
     3 |    return 42;
     4 |  }

1 code style error found.

nevermind, looks like this was fixed in a8296db ?

Nope, the opposite :-)

"allExcept" is not in the documentation FYI

It is, but there no example for it, if you would like to add, we would accept the pr

"allExcept" is not in the documentation FYI

It is, but there no example for it, if you would like to add, we would accept the pr


[OCP-4.X] Check if cluster exists before trying to provision a new one

Install should check if a cluster exists before attempting to install another new cluster to avoid and overwriting any other files.

Fixed by https://github.com/openshift-scale/scale-ci-deploy/pull/60.


Add error handling for busboy creation and emmitted errors

Looks good. Thanks a lot @HughePaul

Published @0.1.0


Question: learning about my_init

I'm currently trying to understand what my_init does for learning and rewriting for fun (in go).  I don't understand what the following section is doing.
https://github.com/phusion/baseimage-docker/blob/14ec533a164cdb495e1c6ab10b82ebe96695a971/image/bin/my_init#L114-L144
What is the while loop trying to accomplish?
cc @mynameismevin

cc @FooBarWidget pining you since you're the only author listed on my_init file.  Do you mind sharing your thoughts?

Hi @samrocketman I feel the code is pretty well documented. It tries to reap all terminating child processes. You can look at the man page of waitpid here: http://linux.die.net/man/2/waitpid to see that waitpid amongst other things does this: "In the case of a terminated child, performing a wait allows the system to release the resources associated with the child" so it prevents processes from becoming zombies.

Thanks, @tinco I was not familiar with waitpid at all so this helps.

It's pretty important that you refer to the POSIX API documentation. my_init performs a bunch of POSIX operations, which are exposed in Python through the os module.
A normal waitpid() call waits until a specific child process terminates. Unfortunately, while waiting for a specific PID, we won't be able to reap any other child process that terminates even though the system expects child processes to be reaped as quickly as possible. So with the while loop I perform a non-blocking waitpid().

Thanks for your insight @FooBarWidget.  That clearly explains the WNOHANG while loop which was a specific point of confusion and lack of understanding.  I feel this question has been fully answered.  Thanks all!


Publish PokemonService generated server crate to crates.io

Publishing the crate would trigger a docs.rs build. This would allow the aws-smithy-http-server documentation, and documentation in general, to reference generated items.

In some cases types from aws-smithy-http-server are used in the context of a generated API.
For example:

MakeServiceWithConnectInfo is a product of the make_service_with_connect_info on the generated service.
Operations purpose are to be passed to the operation setters.
Plugins are passed to build_with_plugins.

To avoid aws-smithy-http-server taking a dev dependency on a generated SDK like the Pokémon service's (which is sizable and could slow down compilation), an alternative is to have the book depend on the generated SDK and have aws-smithy-http-server + the user's generated SDK link to the book.

I think this can be deprioritized now that we have https://github.com/awslabs/smithy-rs/pull/2027 which allows us to write examples involving the examples workspace.


macOS bash installer installs corrupt binary

Trying to install azd on macOS 12.5 on an Intel Mac.
The installer downloads a corrupt binary that cannot start:
> bash install-azd.sh --verbose
install-azd: Version: latest
install-azd: Platform: darwin
install-azd: Architecture: amd64
install-azd: File extension: zip
install-azd: Downloading https://azure-dev.azureedge.net/azd/standalone/release/latest/azd-darwin-amd64.zip to /var/folders/j4/zngd8xzn5bj14jfr2vyygwj40000gn/T/tmp.HAcVI7VS
Archive:  /var/folders/j4/zngd8xzn5bj14jfr2vyygwj40000gn/T/tmp.HAcVI7VS/azd-darwin-amd64.zip
  inflating: /var/folders/j4/zngd8xzn5bj14jfr2vyygwj40000gn/T/tmp.HAcVI7VS/azd-darwin-amd64
install-azd: Cleaning up temp folder: /var/folders/j4/zngd8xzn5bj14jfr2vyygwj40000gn/T/tmp.HAcVI7VS
install-azd: Successfully installed to /usr/local/bin/azd
> azd
zsh: killed     azd

I tried running this through lldb as well. The binary is corrupted.
> lldb /usr/local/bin/azd
Added Microsoft public symbol server
(lldb) target create "/usr/local/bin/azd"
Current executable set to '/usr/local/bin/azd' (x86_64).
(lldb) run
error: Bad executable (or shared library)
(lldb) exit

I've installed it twice. Same issue
CC @jongio

@danieljurek when you get a chance. can you try this?

Thanks for the report @tonybaloney -- I've started looking into this. I'm able to reproduce behavior like this by installing different versions on top of an existing azd installation. That is, installing latest and immediately installing daily results in a similar error. Investigation ongoing.

Further investigation into this reveals that the OS caches signatures for binaries (source).

macOS caches information about the code’s signature in the kernel. It doesn’t flush that cache when you modify the file’s contents. Modifying the file in place yields a mismatch between the file’s contents and the in-kernel cache, which can cause a hard-to-reproduce code-signing crash the next time you run the tool.

There are two fixes here:

Use mv instead of cp to overwrite the installed binary
Restart the computer

I'm working on a fix that uses mv in #554


bump hs100 to 2.2.0 at stable (#2841)

cloned from #2841

2.2.0 created 27.10.2023
87 user (3,4%)


Search Issue

When on a listing, clicking on "Back to List" takes user back to the default search page, not the search page they came from.

I noticed this too.  From the homepage I search "21128".  I get two listings - "Fishnet" and "Serenity Wine Bar + Cafe".  If I click either one of those listings then click "Back to List" I get taken back to a page containing all listings.


pom_tf_battle errors out: unexpected keyword argument 'action'

When running pom_tf_battle with no arguments it gives the following error:
Traceback (most recent call last):
  File ".../pommerman/venv/bin/pom_tf_battle", line 11, in <module>
    load_entry_point('pommerman==0.2.0', 'console_scripts', 'pom_tf_battle')()
  File ".../pommerman/venv/lib/python3.6/site-packages/pommerman/cli/train_with_tensorforce.py", line 139, in main
    runner.run(episodes=10, max_episode_timesteps=2000)
  File ".../pommerman/venv/lib/python3.6/site-packages/tensorforce/execution/runner.py", line 104, in run
    state, terminal, step_reward = self.environment.execute(action=action)
TypeError: execute() got an unexpected keyword argument 'action'

Closing due to inactivity. Please reopen if it's still an issue.


Fix the UID of changed code

Problem:
The UIDs are no longer printed to the code. When we are parsing a code file, we generate UIDs by hashing the jsx elements. The solution is overall pretty elegant and it makes the editor feel like magic. However, there is a big downside: if the user types in the code editor to update an element, that element will also be automatically selected on the Canvas and Navigator and Inspector. But when the selected element's code changes, its hash will change too. And that means its UID will change as well. And that means all the template paths pointing to the selected element, the highlighted element, the spy and dom-walker metadata etc will refer to a nonexistent element. We had an earlier mitigation in place that would salvage the selected view, but it would not save the metadata. Which means that as the user was typing in the code editor, the inspector would flash as it got a new template path with no metadata, and flash again as fresh metadata arrived a frame later.
Fix:
I basically took the same mitigation we had before, but moved it to the end of parseCode:
We zip the old projectFile and new projectFile contents, and we try to match up UIDs by their index among their siblings. If we find a single mismatched UID, (to prevent accidentally matching the wrong element after a delete or insertion or copy-paste or code duplication) we assume that we found the same element but a different hash. We then update the uid to the old one, and also fix the highlightBounds map.
Details:

parseCode now takes an oldParseResultForUIDComparison: ParsedTextFile | null parameter. I made it explicit instead of optional, so I could breeze through all the places using parseCode and decide if I should provide null or a real oldParseResult
I made a new function fixParseSuccessUIDs that runs at the end of parseCode. It zips the old parsed file and the new parsed file against each other, and tries to find matching UIDs based on the element's indexes inside their parent.
Gutted removeNonExistingViewReferencesFromState so now it really only removes nonexisting references and does not try to salvage paths
Moved applyUtopiaJSXComponentsChanges from editor-state to project-file-utils to prevent the parser-printer-worker from bombing out (the dependency cruiser actually caught this issue, nice job, dependency cruiser!)

url to test: https://utopia.pizza/project/6d2e4030/?branch_name=spike-uid-fix

This looks good, but please can you add some tests to parser-printer-uids.spec.ts for this

slaps forehead tests! what a great idea!

This looks good, but please can you add some tests to parser-printer-uids.spec.ts for this

slaps forehead tests! what a great idea!

I added new tests :)

we concluded that the performance regression was not caused by this branch, so I'm merging it!


Google Login - failed to obtain token

Hi,
Bit unsure on this error had a google around but still no closer to solving it. When I try to login to the Wiki via Google it comes up with this error
failed to obtain access token (Error: getaddrinfo ENOTFOUND accounts.google.com accounts.google.com:443)

Hi,
since ENOTFOUND ("Error: not found") is related to accounts.google.com this is not a problem with Jingo.


Repro for FC ULT failing if omp_num_threads set to 2

Please run the mkl_fc_op_test.py or mkl_fc_speed_test.py.
I set the caffe2_omp_num_threads to 2
there should be mismatch with default code.

Thank you for your contribution! PyTorch and Caffe2 are now officially merged with the PyTorch 1.0 preview release. Please submit on https://github.com/pytorch/pytorch if this is still an issue over there.


Virtual swapchain

Added an option --virtual-swapchain to enable it.
Testing:
Using vkCube for those tests. It could remove Replace virtual images in QueuePresent commit to ensure it really use virtual swapchain because the screen should become black. It only tested in Windows, not Android for now.

Force a different min number of real swapchain images in replay, maybe +1 and verify number of images is different than capture (pCreateInfo->minImageCount)

Adding modified_create_info.minImageCount = 2 in VulkanVirtualSwapchain::CreateSwapchainKHR for replay. Setting desiredNumOfSwapchainImages = 3 in vkCube
or
Adding modified_create_info.minImageCount = 4 in VulkanVirtualSwapchain::CreateSwapchainKHR for replay. Setting desiredNumOfSwapchainImages = 3 in vkCube
or
Adding modified_create_info.minImageCount = 3 in VulkanVirtualSwapchain::CreateSwapchainKHR for replay. Setting desiredNumOfSwapchainImages = 2 in vkCube to test.

In replay, hand edit AcquireNextImageKHR so the first call actually Acquires two images (discarding the first one) to deliberately get out of order

Adding more func(device, swapchain, timeout, semaphore, fence, image_index) in VulkanVirtualSwapchain::AcquireNextImageKHR to test it.

Force a different swap mode on capture, like MAILBOX, make sure captures are still replayed correctly

Adding modified_create_info.presentMode = VK_PRESENT_MODE_MAILBOX_KHR; or other present mode in VulkanVirtualSwapchain::CreateSwapchainKHR to test it.

Capture on Android R(11), replay on S(12) or the other way around - verify swapchain count has changed

Android 10 can replay Android 12 capture file with --remove-unsupported to remove huge extensions that Android 12 added.
Android12 can’t replay Android 10 capture file. The failure reason isn’t related to swapchain. It might not be able to run tests for virtual swapchain.
Log:
Skipping memory fill for VkDeviceMemory object (ID = 25) that is not mapped
Skipping memory fill for VkDeviceMemory object (ID = 33) that is not mapped
Skipping memory fill for VkDeviceMemory object (ID = 31) that is not mapped
Skipping memory fill for VkDeviceMemory object (ID = 29) that is not mapped

CI gfxreconstruct build queued with queue ID 19818.

CI gfxreconstruct build # 1658 running.

CI gfxreconstruct build # 1658 failed.

CI gfxreconstruct build queued with queue ID 19845.

CI gfxreconstruct build # 1659 running.

CI gfxreconstruct build # 1659 failed.

CI gfxreconstruct build queued with queue ID 20190.

CI gfxreconstruct build # 1661 running.

CI gfxreconstruct build queued with queue ID 20349.

CI gfxreconstruct build # 1664 running.

CI gfxreconstruct build # 1664 passed.

CI gfxreconstruct build queued with queue ID 3306.

CI gfxreconstruct build # 1718 running.

CI gfxreconstruct build # 1718 passed.

CI gfxreconstruct build # 1719 running.

CI gfxreconstruct build # 1719 passed.

CI gfxreconstruct build queued with queue ID 3792.

CI gfxreconstruct build # 1720 running.

CI gfxreconstruct build # 1720 passed.

CI gfxreconstruct build queued with queue ID 3901.

CI gfxreconstruct build # 1721 running.

CI gfxreconstruct build # 1721 passed.

CI gfxreconstruct build queued with queue ID 5514.

CI gfxreconstruct build # 1729 running.

CI gfxreconstruct build # 1729 passed.

CI gfxreconstruct build queued with queue ID 5639.

CI gfxreconstruct build # 1730 running.

CI gfxreconstruct build # 1730 passed.

CI gfxreconstruct build queued with queue ID 6623.

CI gfxreconstruct build # 1732 running.

CI gfxreconstruct build # 1732 passed.

CI gfxreconstruct build queued with queue ID 6686.

CI gfxreconstruct build # 1733 running.

CI gfxreconstruct build # 1733 passed.

CI gfxreconstruct build queued with queue ID 6820.

CI gfxreconstruct build queued with queue ID 6829.

CI gfxreconstruct build # 1735 running.

CI gfxreconstruct build queued with queue ID 6846.

CI gfxreconstruct build # 1736 running.

CI gfxreconstruct build # 1736 passed.

Linux AMD device passed test with vkcube.

CI gfxreconstruct-extended build queued with queue ID 19393.

CI gfxreconstruct-extended build # 72 running.

CI gfxreconstruct-extended build # 72 aborted.

CI gfxreconstruct build queued with queue ID 346.

CI gfxreconstruct build # 1793 running.

CI gfxreconstruct build # 1793 passed.

CI gfxreconstruct build queued with queue ID 847.

CI gfxreconstruct build # 1797 running.

CI gfxreconstruct build queued with queue ID 917.

CI gfxreconstruct build # 1798 running.

CI gfxreconstruct build # 1798 passed.

CI gfxreconstruct build queued with queue ID 1094.

CI gfxreconstruct build # 1799 running.

Added two commits to fix VkQueue and InvalidImageLayout issuses.
Dustin's set ImageLayout of virtual swapchain image was VK_IMAGE_LAYOUT_TRANSFER_SRC_OPTIMAL. But it caused some InvalidImageLayout issuses by VVL. Set VK_IMAGE_LAYOUT_PRESENT_SRC_KHR could avoid it. But I'm not sure if it has any down side. At least, VVL didn't complaint it.

CI gfxreconstruct build # 1799 passed.

No, sorry, the comment was meant for the whole pull request in general, not that particular commit.

CI gfxreconstruct build queued with queue ID 3174.

CI gfxreconstruct build # 1808 running.

CI gfxreconstruct build # 1808 passed.

CI gfxreconstruct build queued with queue ID 3190.

CI gfxreconstruct build # 1809 running.

CI gfxreconstruct build # 1809 failed.

CI gfxreconstruct build queued with queue ID 3207.

CI gfxreconstruct build # 1810 running.

CI gfxreconstruct build queued with queue ID 3220.

CI gfxreconstruct build # 1811 running.

CI gfxreconstruct build # 1811 passed.

If this is not VK_SHARING_MODE_EXCLUSIVE then the following line where queueFamilyIndexCount is set to zero would be invalid. Perhaps worth checking this assumption?

@per-mathisen-arm , do you mean for the line
image_create_info.sharingMode           = swapchain_info->image_sharing_mode;

if image_sharing_mode is CONCURRENT then the queueFamilyIndexCount and pQueueFamilyIndices should be copied from the original swapchain?

If this is not VK_SHARING_MODE_EXCLUSIVE then the following line where queueFamilyIndexCount is set to zero would be invalid. Perhaps worth checking this assumption?

@per-mathisen-arm , do you mean for the line
image_create_info.sharingMode           = swapchain_info->image_sharing_mode;

if image_sharing_mode is CONCURRENT then the queueFamilyIndexCount and pQueueFamilyIndices should be copied from the original swapchain?

Yes, that would be my understanding.

CI gfxreconstruct build queued with queue ID 7761.

CI gfxreconstruct build # 1824 running.

CI gfxreconstruct build # 1824 failed.

CI gfxreconstruct build queued with queue ID 7777.

CI gfxreconstruct build # 1825 running.

CI gfxreconstruct build # 1825 passed.

if image_sharing_mode is CONCURRENT then the queueFamilyIndexCount and pQueueFamilyIndices should be copied from the original swapchain?

Yes, that would be my understanding.

CI gfxreconstruct build queued with queue ID 7871.

CI gfxreconstruct build # 1826 running.

CI gfxreconstruct build # 1826 passed.

CI gfxreconstruct build queued with queue ID 7980.

CI gfxreconstruct build # 1827 running.

CI gfxreconstruct build # 1827 passed.

fixed for adding queue family indices for swapchain images.

CI gfxreconstruct build queued with queue ID 8865.

CI gfxreconstruct build # 1831 running.

CI gfxreconstruct build queued with queue ID 8914.

CI gfxreconstruct build queued with queue ID 8915.

Moved VulkanReplayConsumerBase::ProcessSetSwapchainImageStateCommand to virtual swapchain.
Only default swapchain does what VulkanReplayConsumerBase::ProcessSetSwapchainImageStateCommand does.
Virtual swapchain does nothing about it.

CI gfxreconstruct build queued with queue ID 8917.

CI gfxreconstruct build queued with queue ID 8919.

CI gfxreconstruct build # 1835 running.

CI gfxreconstruct build # 1835 passed.

CI gfxreconstruct build queued with queue ID 11618.

CI gfxreconstruct build # 1847 running.

CI gfxreconstruct build # 1847 passed.

CI gfxreconstruct build queued with queue ID 12981.

CI gfxreconstruct build # 1849 running.

Set virtual swapchain as default. option --use-cpatured-swapchain-indices could disable virtual swapchain

CI gfxreconstruct build queued with queue ID 14141.

CI gfxreconstruct build # 1851 running.

CI gfxreconstruct build # 1851 passed.

CI gfxreconstruct-extended build queued with queue ID 14264.

CI gfxreconstruct-extended build # 80 running.

CI gfxreconstruct build queued with queue ID 14345.

CI gfxreconstruct build # 1853 running.

CI gfxreconstruct build # 1853 passed.

CI gfxreconstruct build queued with queue ID 14365.

CI gfxreconstruct build # 1854 running.

CI gfxreconstruct build # 1854 passed.

CI gfxreconstruct-extended build queued with queue ID 14763.

CI gfxreconstruct-extended build # 82 running.

CI gfxreconstruct-extended build # 82 failed.

CI gfxreconstruct build queued with queue ID 15036.

CI gfxreconstruct build # 1855 running.

CI gfxreconstruct build # 1855 passed.

CI gfxreconstruct build queued with queue ID 16374.

CI gfxreconstruct build # 1856 running.

CI gfxreconstruct build # 1856 passed.

CI gfxreconstruct build queued with queue ID 16414.

CI gfxreconstruct build # 1857 running.

CI gfxreconstruct build queued with queue ID 16430.

CI gfxreconstruct build # 1858 running.

CI gfxreconstruct build # 1858 passed.


Update 0prometheus-operator-deployment.yaml

Add base image. 2/3 pods will fail to start without also specifying the base image.

This manifest file is auto-generated by the JSONNET libraries. It's part of https://github.com/coreos/prometheus-operator/blob/master/jsonnet/prometheus-operator/prometheus-operator.libsonnet and the image used to deploy Prometheus by the operator come from: https://github.com/carlosedp/prometheus-operator-ARM/blob/1f50c68326facb6a4d9a443cdce664bd43bc7956/image_sources_versions.jsonnet#L21.
You can check this on the manifest that is auto-generated:
https://github.com/carlosedp/prometheus-operator-ARM/blob/1f50c68326facb6a4d9a443cdce664bd43bc7956/manifests/prometheus-prometheus.yaml#L28
Is this really required?

Ah I see the mistake. I needed to start using the operator on ARM and I initially cloned and modified the image section of the operator manifest. I deployed via the bundle.yaml file.
I could not find how to change the 3 images needed for the 3 containers until I found your 0prometheus-operator-deployment.yaml file but then noticed you didn't have the base image. After adding my commit to the bundle.yaml I figured yours could use the same line. Missed your other manifest file so not I don't think my commit is needed now. Thanks for the response!

The images I built have manifests for both ARM, ARM64 and AMD64 so no changes are needed:
Image: carlosedp/prometheus
 * Manifest List: Yes
 * Supported platforms:
   - linux/arm
   - linux/arm64
   - linux/amd64

In case you want to re-generate the manifests, change the jsonnet files and build the manifests with make. A few pre-reqs are needed though.

Thank you for the explanations!


improve glacier advance and retreat notebook and EarthCube2021

added patricks new function for EarthCUBE
added a function to plot png besides a normal plot
changed some stuff in intro_glacier_plot

Thanks!!!!


Don't force apartment numbers to be integers

In this story, we thought it would be trivial to remove a validation on apartment numbers. However, when writing XML for a 3.0 feed, we validate it against the spec, so we'd need to change the XSD as well if strings are going to be allowed (ok, it's a little less trivial). Also, I noticed the pipeline used in xml_test/write-xml-test was letting errors sneak through with passing tests (see this chore)
So, here is a little PR that introduces 3.0.1, for your perusal. I'll make an issue / PR for vip-specification, but for now, how does this approach feel? Is there a better way?

I think that we can suggest the change to the spec, though I'm not sure how likely changes to the 3.0 spec are to be made at this point. I've wondered about Chris's suggestion (should we bother importing apartment #s at all, since Google isn't using them) in the past, and have always landed on the side of keeping them in because A.) At some point someone else might use the data for something and B.) If we drop apartment numbers, duplicates that are just weird duplicate rows and duplicates caused by apartments will look identical to us, which might make communications around error resolution with the state more confused. I'm not sure either of these reasons is convincing enough not to do it. @afsmythe, what do you think about this?

I think I'd prefer to have the spec updated. @MariaBianchi makes a good point about what to do with duplicate street_segments once the apartment number is ignored. For Washington, during the general election, I was having to strip the apartment numbers from the feed and then de-duplicate the resulting street_segment records. Why? I don't recall, but maybe Google was ignoring the duplicates. Ignoring apartment numbers that are invalid (because of alpha characters) makes sense so long as the result is de-duplicated. But, alpha apartment identifiers are common and should be allowed by the spec, IMO.

It looks to be trivially easy to just ignore (4a -> null; 42 -> 42) any of the apartment numbers that are "invalid" as @tie-rack suggested
@MariaBianchi makes a good point about changing an old, deprecated spec; I suppose we could change our implementation and be divergent if the apartment number idea doesn't go over, but that doesn't feel right.

I wonder if instead of creating a new version of the spec (which doesn't track anything official), it would make more sense to not output apartment numbers if they're invalid

@tie-rack: After talking with @MariaBianchi, it sounds like it might be better to keep the apartment numbers around and call them strings. I also rearranged the commits to be a bit more clear.


Getting DateTimeParseException error while reading from redshift timestamp.

java.time.format.DateTimeParseException: Text '2015-11-07 05:16:40.16858' could not be parsed, unparsed text found at index 23

Yes, this pattern "yyyy-MM-dd HH:mm:ss[.SSS][.SS][.S][X]" won't account for that.
Is that a real ISO-8601 datetime string ? 3 digits after the dot is ms, 6 digits is microseconds, what about 5 ?

The backend uses Calendar, which won't go beyond milliseconds. Most of spark-redshift code is meant (and tested) not to go beyond milliseconds: https://github.com/spark-redshift-community/spark-redshift/blob/master/src/test/scala/io/github/spark_redshift_community/spark/redshift/ConversionsSuite.scala#L48
This feature would mean more than just changing the format string. I'm available to review if anyone wants to take it on.

@lucagiovagnoli do you have any suggestions for Calendar replacements?
Also (demonstrating my JVM ignorance here...), why is there no mention of Calendar in the source code? Instead I see a lot of import java.sql.Timestamp and import java.time.format.DateTimeFormatter. Do these standard library utils use Calendar?

Hey @eeshugerman, there are no basic questions, thanks for asking.
You're actually right, we're not using Calendar in the src/ code but only in tests to create fake data. The temporalAccessor and java.sql.Timestamp from Conversions.scala can both handle microseconds and nanoseconds.
I had a go at this in #51, feel free to review. Also for @smoy and @dichiarafrancesco

Awesome, thanks for addressing this! I added a couple comments on the PR.


Issue with the optimization!!

The images in our mission section are loading very slowly due to high quality!!

It's a very small issue like good first issue, I would like to fix it.
Please assign me with this issue!


Improve error message when a float is expected but an int is passed as a shader function parameter

Godot version
v3.3.3.stable.mono.official.b973f997f
System information
Windows 10, GLES3, GeForce RTX 3050 Ti Laptop GPU/PCIe/SSE2
Issue description
In .shader files the built-in function pow() will raise Invalid arguments error if any of the arguments is a variable or function output of type int.
The error states: "Invalid arguments for built-in function: pow(int,int)" in case the arguments are (int(true),2).
At the same time, however, it will not raise any error if the arguments are any mix of variables of type float and integer/float constants.
Steps to reproduce
Use any function with output type of int or variable of type int as arguments passed into pow() in a CanvasItem type shader.
Minimal reproduction project
Pow Bug.zip

The Godot shader language does not perform implicit int-float conversion. If pow() expects a float, you have to case the variable using float() or add .0 at the end.

The Godot shader language does not perform implicit int -> float conversion, for consistency with GLSL ES 1.0 (GLES2's shading language). If pow() expects a float, you have to case the variable using float() or add .0 at the end.

Well, yes that is understandable, however the error is incorrect


Help Teleporting

I want to allow players to teleport to world2 from world1 then they can teleport back to world1 to the place they were last at.

A plugin such as essentials has this feature, perhaps use that if it is still a relevant issue.


[Issue 10221] Fix authorization error while using proxy and Prefix subscription authentication mode

Fixes #10221
Motivation
If using pulsar proxy and Prefix subscription authentication mode, org.apache.pulsar.broker.authorization.PulsarAuthorizationProvider#canConsumeAsync will throw exception which cause the consumer error.
Modifications
Update the org.apache.pulsar.broker.authorization.PulsarAuthorizationProvider#allowTopicOperationAsync logic, check isSuperUser first, then return isAuthorizedFuture.
Verifying this change

[ ] Make sure that the change passes the CI checks.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

@tuteng @codelipenghui Could you guys merge it if this PR is ok. Thanks a lot.

@dragonls Merged


TEKLA BRIDGE NOT RESPONDING

Describe the bug
The bridging website would say  bridge  site is not active
Steps to reproduce
but if you click on the bridge from the drop down it will load
Spam policy

[x] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

What's Tekla? We only have Hekla and Katla

This is True, it only shows the HORSE token, and it will later say Horse token is not available, what I did was bridge from Holesky to Tekla, you can try that.

There is no Tekla... If you claimed your horse they are in your wallet.


[Home] Tap status bar to go to top of view not working

on home view, down in the feed, tap status bar to go to top of feed, nothing happens

Will test on tomorrow's beta... @alloy will double check

This is working in the current beta.

@alloy seeing again in this morning's beta

Hmm, indeed I have seen it too now. It worked fine for a while and then after opening the search view it stopped working.
Alas I wasn't able to reproduce it with those exact same steps. It would be great if we can find exact reproducible steps, will continue to dig.

I'm struggling to reproduce this ( even with going into search, or changing tabs ) on an iPad 8.4

However, it's much easier to reproduce on iPhone though, will try get a repro

Repro from scratch: Restart app, home works, go to explore, explore does not scroll to top, nor will home when you come back
Same for "you."
Hitting search does seem to break i

Switching to Explore, then turning scrollsToTop off on the home, means that the explore scrolls to top correctly. I'm assuming that somehow the Home is being prioritised by the algorithm described as:

the system asks the scroll view closest to the status bar

Yikes… Might be time to axe our ARTopMenuVC and rebuild it with a plain UITabBarController?

Is it a good idea? Definitely. Can it be done for 3, nopes

Oh no, totally not doing that for v3.

I haven't seen this come up since our last build, I'm going to close for now, feel free to reopen if necessary @orta @alloy


Yetisports is probably missing a SetRenderTarget patch....

It's a 5849 title, and it has this in the log :
HLECache: 0x00439470 -> D3DDevice_SetRenderTarget	*PATCHED*
HLECache: 0x004396f0 -> D3DDevice_SetRenderTargetFast	*PATCHED*

but the white square in the top-left corner seems to indicate it isn't patched correctly...

Hm, D3DDevice_SetRenderTargetFast actually forwards the call to D3DDevice_SetRenderTarget, but that function calls into the Xbox version of  D3DDevice_SetRenderTarget via a trampoline, so the behavior of Xbox D3DDevice_SetRenderTargetFast is different from what would happen normally.
On the other hand, a breakpoint placed in our D3DDevice_SetRenderTargetFast patch never gets hit, so it's not the cause of the strange behaviour...

Is Yetisports a LTCG title?

There's none

Symbol detection is accurate and title is visible in Cxbx-Reloaded emulation. Likely due to further improvement in past years. Closing as not an issue.


Searchbar

Hello, I already merged the master branch into my searchbar-branch. Please review the branch so I can merge it back into the master-branch. Thanks!

@ON21MB The Montserrat-Font doesn't work? I think you need in the CSS for * {font-family: Montserrat, sans-serif;}. Could you check that again?


Feat/block local

remove data upload
change block threshold millis to configurable
save block info to local files

pls don't submit Revert "feat: remove upload user info and data pick info and app heal… such revert code
your commits should be only 3
feat: save block info to local files
~feat: remove upload user info and data pick info and app health info~
feat: change block threshold millis to configurable, config it when D…
~Revert "feat: remove upload user info and data pick info and app heal…~
feat: not upload data pick info、app base info and app health info any…
you can drop commit via Drop Commit in Android Studio Git view, or use Git command.

new PR created: https://github.com/Wiredcraft/DoKit/pull/3


support more codecs?

are there any plans to support more codecs as per : https://wpdev.uservoice.com/forums/110705-universal-windows-platform/suggestions/19248226-codecs-for-uwp-media-element

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

ID: 8181085f-2eaa-6eb5-06fd-52c3a98f7467
Version Independent ID: 240818b0-d509-bbcf-f485-7ca0ba55f0f6
Content: Supported codecs - UWP app developer
Content Source: windows-apps-src/audio-video-camera/supported-codecs.md
Service: unspecified
GitHub Login: @drewbatgit
Microsoft Alias: drewbat

@ginami , @drewbatgit  any update on this? is this issue being investigated? or can we expect it to be implemented?

The feature teams use customer feedback when they plan updates to the platform, but we don't discuss plans for future releases publicly. I will forward your request on to the team.


Solves issue #616 on 3.1.x

Solves issue #616 "CurrentUnitOfWork not cleared when starting a transaction fails"

We have chosen a slightly different approach to solving this issue (see a398780), though it was definitely inspired by the approach in this PR.
Thanks for submitting!


[FEATURE_REQUEST] export StorageStatus type

Is your feature request related to a problem? Please describe.
Can the StorageStatus type be exported?
This could be useful to type, say, a variable for keeping the previous storage-status style.
If the type is not exported, then that variable has to be typed as string (or the StorageStatus has to be copied & pasted from the package).
Describe the solution you'd like
Export tye StorageStatus type.
Additional context
N/A

Great point — yeah, that's annoying. I'll roll a patch release to add it.

Thanks for reporting this @npapagna — this is now released as 0.19.5 🙏


[874] Fix API response examples

Context

https://trello.com/c/x28JcW5k/874-api-docs-response-examples-are-wrong

Changes proposed in this pull request
Addresses the issues referenced in the ticket above, largely fixing incorrect documentation examples.
Guidance to review

Fire up the docs and assert responses match up to what's expected in the ticket above.

Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Api docs, Response examples are wrong

 Api docs, Response examples are wrong