Migration issue when cyclic dependency is altered

EdgeDB Version: 1-beta.2

Steps to Reproduce:

Apply the following schema:

module default {
    type Article {
        required property deleted := (EXISTS (.<element[IS default::DeletionRecord]));
    };
    type Category {
        required property deleted := (EXISTS (.<element[IS default::DeletionRecord]));
    };
    type DeletionRecord {
        required link element -> (default::Article | default::Category) {
            on target delete  delete source;
            constraint std::exclusive;
        };
    };
};

Change to:

module default {
    abstract type Removable {
        property deleted := EXISTS(.<element[IS DeletionRecord]);
    }
    type Article extending Removable {
    };
    type Category extending Removable {
    };
    type DeletionRecord {
        required link element -> Removable {
            on target delete delete source;
            constraint std::exclusive;
        };
    };
};

Get the following error on create-migration:

did you create object type 'default::Removable'? [y,n,l,c,b,s,q,?]
y
error: 'default::Removable.<element' does not resolve to any known path
  ┌─ query:2:41
  │
2 │     CREATE PROPERTY deleted := (EXISTS (.<element[IS default::DeletionRecord]));
  │                                         ^^^^^^^^^ error

Rolling back last operation...
did you create object type 'default::Removable'? [y,n,l,c,b,s,q,?]

On the other hand, schema from (2) works fine if applied from scratch

Hm, we now reject the first schema with
ERROR: SchemaError: it is illegal to create a type union that causes a computable property 'deleted' to mix with other versions of the same property 'deleted'

Oh, but if I make the names different, and then try to migrate, it still fails, so.

Hm, we now reject the first schema with
ERROR: SchemaError: it is illegal to create a type union that causes a computable property 'deleted' to mix with other versions of the same property 'deleted'

Actually I don't understand what it means. You can make an union of two types which have a property that is named the same, right? So what makes it broken? The fact that both (or any) are computed? Is the limitation fundamental or temporary? (i.e. I used to think that any property can be changed to computable, e.g. for backwards compatibility, this limitation seems arbitrary for me)

Is the limitation fundamental or temporary

Temporary, until the compiler learns to actually produce correct queries for polymorphic computables.

Good. Can we rephrase error to something like:
it is illegal to create a type union from multiple types having computable property `deleted`

Or:
multiple matches for `deleted`, each type in an union must have unique computable properties

Can we rephrase error to something like

We prohibit mixing non-computables with computables as well, not just different computables, but I agree that the error message needs clarification.  I'd actually specialize the message to each prohibited combination (i.e. computable and non-computable and computable/computable).
\cc @vpetrovykh


Versioning support based on Header

I went through the code and wiki section but do not found the versioning support based on header.
Is there any reason behind it? any plan to integrate in near future?

In reviewing this, you are right. There's mention of versioning by header, but no actual documentation. It's fully supported and available if it's an option you want to use. I've added a wiki page and other links to make it more discoverable.
There isn't anything all that special about versioning by header. You need only use the HeaderApiVersionReader like so:
options => options.ApiVersionReader = new HeaderApiVersionReader( "api-version" );

You can compose multiple styles of versioning using ApiVersionReader.Combine. The default configuration combines the query string and URL segment methods as they are the most common developer choices (even though URL segment is worse one they can choose).

Thank you 👍


Revert "Revert "Remove redundant logic""

Reverts vespa-engine/vespa#4142

@jonmv please review

You successfully removed initId(...).


[ILVerify] Fix invalid branch/fallthrough into catch/finally causing assert

Invalid il code that performs a branch or fallthrough into a catch/filter/finally block currently causes an assert as described in #4537.
In order to fix this issue, I moved the valid branch target check into the ImportFallthrough method (as we always call ImportFallthrough for branching anyways), and changed the check itself to return a boolean indicating whether the target is valid or not. If the target is invalid the stack is not merged and the target block is also not marked for verification. This way the state that causes the assert is never reached.
This fixes #4537.

@ArztSamuel,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


feat: .clear a file from local cache

Fixes https://github.com/hypercore-protocol/hyperdrive/issues/228#issuecomment-668678754
This calls clear under the hood for the given file, using the same walking tree as the hyperdrive.download function.
If this is a desired appraoch, I will also write a companion section for the README and tests for directories/mounts

Extracting that _walk function looks like a nice approach! LGTM

Published 10.17.0


⚠ Spryr has degraded performance

In d48a9ae, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5157 ms

Resolved: Spryr performance has improved in 1159c68 after 13 minutes.


Fixed typo in git action error

Fixed typo in message shown when git cannot open a file which has been deleted.

Hi @effleurager, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


(test) Add test for BasicFuture.failed and ComplexFuture.failed

Hey 😊
I want to contribute the following test:
Test that future.failed(new java.lang.Exception()) is false when failed is called with the parameter exception = new java.lang.Exception().
This tests the methods BasicFuture.failed and ComplexFuture.failed.
This test is based on the test testCancelled.
Curious to hear what you think!
(I wrote this test as part of a research study at TU Delft. Find out more)

Thank you for looking over it 🙂
I incorporated both suggestions


Temporarily disable parse_stdlib tests on Linux.

They're breaking CI because we seem to be running out of memory (?).
Need to investigate.

@swift-ci please smoke test and merge


Support hot-reloading route data

I like the premise and benefits presented by react-static!
One area where I am struggling is that I'd like the browser to refresh when there are changes to the Markdown files. I've combined these two examples:
https://github.com/EmilTholin/react-static-markdown-example
https://github.com/pkrawc/react-static-markdown/blob/master/static.config.js
In the first example, you can make changes to the markdown test file, but they are not reflected on the browser until you refresh it.
I plan to store the post in markdown under a folder. Based on the first example, I changed the second example to reading the posts folder inside the /blog route path. When I make changes to it, there is no rebuilding, which is great, but I have to manually refresh the browser to see the updated content reflected after a markdown change.
I'd appreciate any insight or guidance on how to achieve this please. It's the only thing I am missing to fully embrace this framework for a blog!
Thank you for your time!

I use a library npm-watch for this

So, as long as you are requesting the data in each routes getData method, the data should be up to date on the webpack server. The problem you're experiencing is that this data is cached on the client, so when you move around your site, you are getting stale data. To make this possible a couple things would need to change in React Static:

The user would need a way of detecting changes to the asset you're using (in this case, watching a markdown file for changes)
RS would need to define a generic api to notify RS that an "asset" has changed and also be able to notify the client which routes have changed.
On the client, the routeInfo cache for those changed routes would need to be cleared.
Any components relying on the updated routeInfo would need to rerequest it, and naturally, render the component with the updated data.

HA! this is done. :)


[Bug] [Link Utilities] Co-Optimus absent from browser search submenu

Description
In Link Utilities, Co-Optimus is not showing up as an option in the "Open browser search on..." submenu
To Reproduce

Open the general settings for Link Utilities
Activate "Show in menu" for Co-Optimus
Right click on any game
Navigate to the "Open browser search on..." submenu
Co-Optimus is absent

Expected behavior
Co-Optimus should show up in the submenu like every other selected website
Screenshots

Additional context

Windows version: 10
Playnite version: 10.32
Link Utilities version: 4.0

The browser search function depends on the website having a proper search results page that can be reached with a URL in which the only variable part is the search term. Unfortunately, this is not the case for every site, even if LinkUtilities offers the "normal" search for this. With Co-Optimus, however, I seem to have simply overlooked the appropriate URL. I have now added it. It will be included in the next version.
I will also add a browser search column to the wiki page for the supported pages so that you can check this in future.


Experimental support for async interface via the future<> generic type

This adds support for a future<> type to the djinni idl.  The future<> djinni type is mapped to the Future type defined in the C++, Java and ObjC djinni support library (also added in this PR).  In Javascript, future<> is mapped to the builtin Promise type (and therefore supports the JS await syntax).
Basically, instead of writing this:
FooCb = interface {
  complete(res: i32): void;
}
Foo = interface {
  bar(cb: FooCb): void;
}

We can now express this more succinctly with :
Foo = interface {
  bar(): future<i32>;
}

The "Future" objects are fulfilled by calling the setValue() method on their associated  Promise object (note this is not related to the above mentioned JS Promise type) in C++, Java and ObjC.  The Promise/Future API closely resemble the std::experimental::promise/future types defined by the C++ concurrency TS.
The main benefit of this feature is that we no longer need to define new callback interfaces just to return a value asynchronously.  The future<> objects as implemented by this PR can be passed across language boundaries and back, which can be utilised to support cross language asynchronous workflows.
This PR includes a djinni code generator change borrowed from the cross-language-cpp repo to support generic external types in ObjC,

lgtm

lgtm


Changelog for SDK version <IP_ADDRESS>22  08/09/2022

Can we have a changelog for new features, improvements, bug fixes ?

@zhengni-msft @satyakonmsft  ?


Get error when execute "npm start" command.

I follow the instruction and got this problem when running "npm start" command. AFAIK, root cause of this error is "npm-run-all" was not installed global.
Are there anyway to run local packages from cli?
I know I can manually add project's node_mobules folder to $PATH environment variable. But it isn't a best way, I have to repeat these step every project.
Do you have any suggestions for me?

npm ERR! Linux 4.2.6-301.fc23.x86_64
npm ERR! argv "/usr/bin/node" "/home/nhduc/.npm-global/bin/npm" "start"
npm ERR! node v5.10.1
npm ERR! npm  v3.8.6
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>prestart: `npm-run-all --parallel start-message remove-dist lint:tools`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prestart script 'npm-run-all --parallel start-message remove-dist lint:tools'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the react-slingshot package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm-run-all --parallel start-message remove-dist lint:tools
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs react-slingshot
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls react-slingshot
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/nhduc/Documents/projects/react-slingshot/npm-debug.log

I do this hack to run local package without install them global. I suggest we should add this to readme file
export PATH=./node_modules/.bin/:$PATH

Hrm. I see your'e on Linux, so I don't have an easy way to reproduce. However, I think I can help. I don't think it's npm-run-all that's the problem since it doesn't require being installed globally.
Let's try running each of the scripts that are called in the prestart command separately to see which one is throwing an error for you:
npm run start-message
npm run remove-dist
npm run lint:tools
If all three of those succeed separately, then try simplifying your prestart command to this, and let me know if it works:
"prestart": "npm-run-all --parallel start-message",
Then run it like this:
npm run prestart

@nhducit Any update?

@coryhouse Sorry for response late. After add local node_modules folder, I cannot reproduce this issue even after I removed local node_modules folder. I will try again on my desktop at home and update this issue.

@nhducit Any update? Wondering if I can close this.

I can't reproduce on 6.0. Questions:

What OS?
What command line?
If you switch back to 6 now, does it work?
Had you previously installed an earlier version of slingshot?

If you completely delete node_modules and switch back to 6.0, does it work?

@coryhouse Please close this issue, I will try to reproduce this issue latter

@jarbot Node 6.1 is out now - Curious if it works for you on that version? There were some bug fixes...

@jarbot - Ah! That error helped. Here's the problem.
And here's the fix: npm rebuild node-sass
This issue also appears to be fixed in<EMAIL_ADDRESS>I just committed an update to package.json so we're now using 3.7.0 on Slingshot.

Great to hear. Regarding linting, I hadn't noticed that, but will look into it. Looks like new version of ESLint added some rules.

Hello Guys...
When I'm do "npm start" Getting the error like "(function (exports, require, module, __filename, __dirname) { import React from "react" but Im imported react libraries. Still I can't do run the project
"

Hi All,
I am getting the following error when I run npm start after doing
PS D:\Qa_portal\qa-portal> npm start

<EMAIL_ADDRESS>start D:\Qa_portal\qa-portal
react-scripts start

'react-scripts' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: react-scripts start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\nithinp\AppData\Roaming\npm-cache_logs\2019-09-24T10_35_56_785Z-debug.log

react-scripts isn't part of react-slingshot. It sounds like you're running create-react-app instead.

There might be a problem with the project dependency tree.
It is likely not a bug in Create React App, but something you need to fix locally.
The react-scripts package provided by Create React App requires a dependency:
"babel-loader": "8.1.0"
Don't try to install it manually: your package manager does it automatically.
However, a different version of babel-loader was detected higher up in the tree:
C:\Users\Hafsa PC\node_modules\babel-loader (version: 8.0.6)
Manually installing incompatible versions is known to cause hard-to-debug issues.
If you would prefer to ignore this check, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
That will permanently disable this message but you might encounter other issues.
To fix the dependency tree, try following the steps below in the exact order:

Delete package-lock.json (not package.json!) and/or yarn.lock in your project folder.
Delete node_modules in your project folder.
Remove "babel-loader" from dependencies and/or devDependencies in the package.json file in your project folder.
Run npm install or yarn, depending on the package manager you use.

In most cases, this should be enough to fix the problem.
If this has not helped, there are a few other things you can try:

If you used npm, install yarn (http://yarnpkg.com/) and repeat the above steps with it instead.
This may help because npm has known issues with package hoisting which may get resolved in future versions.

Check if C:\Users\Hafsa PC\node_modules\babel-loader is outside your project directory.
For example, you might have accidentally installed something in your home folder.

Try running npm ls babel-loader in your project folder.
This will tell you which other package (apart from the expected react-scripts) installed babel-loader.

If nothing else helps, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
That would permanently disable this preflight check in case you want to proceed anyway.
P.S. We know this message is long but please read the steps above :-) We hope you find them helpful!
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: react-scripts start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Hafsa PC\AppData\Roaming\npm-cache_logs\2020-09-25T10_53_01_625Z-debug.log

I am getting this type of error and I don't know how to deal with it. Can anybody help me in this regard?

any one help me in this error while installing  rtlcss in ubuntu18.04

npm ERR! code EAI_AGAIN
npm ERR! errno EAI_AGAIN
npm ERR! request to https://registry.npmjs.org/rtlcss failed, reason: getaddrinfo EAI_AGAIN registry.npmjs.org registry.npmjs.org:443
npm ERR!


Review components structure

For now, we are putting almost all components inside of the presentational/ folder but there are better approaches.
Refs:
https://medium.com/@dan_abramov/smart-and-dumb-components-7ca2f9a7c7d0
https://reactjs.org/docs/faq-structure.html
https://www.newline.co/fullstack-react/p/using-presentational-and-container-components-with-redux/

Also, reactor components to the component folder pattern.


Fix DocC warnings and errors, including those from SPI

This includes a variety of cleanups and fixes to resolve DocC warnings and errors, including those which show up only when you manually include SPI declarations (by passing the -include-spi-symbols Swift compiler flag).
I extracted some of these from #295, and found new ones. I still plan to pursue that PR, but reduce its focus to just enabling the SPI documentation flag in some local development workflows. In the mean time, this PR contains only documentation fixes.
Checklist:

[x] Code and documentation should follow the style of the Style Guide.
[x] If public symbols are renamed or modified, DocC references should be updated.

@swift-ci please test

@swift-ci please test


Integrate with fine grained PAT

When creating the webhook token with the CLI we can instead use fine grained personal access token for github webook ;
https://github.blog/2022-10-18-introducing-fine-grained-personal-access-tokens-for-github/

documentation has been added


Overwrite the To/From fields when bulk change

When you select multiple blogs (rows), you could be write in the From and To fields even if one ore more of the selected blogs already have a date in either or both.
If you only select blogs whose To/From fileds are empty, you can bulk change them.
Now, if there is just one single blog among the selected blogs, a blog that has a To or From field that already includes a value, the To/From box greys out, so you cannot bulk change the selected blogs.
I would like to be able to bulk change multiple blogs, that is, the To/From fields should be remain writeable, and these fields should be overwritten by a bulk change.
Thanks.
Vozzy

By design the user shall not accidentally overwrite a (different) value which he's not aware of. For the checkboxes, there's this special state symbolizing that some are checked others not, but such a thing doesn't exist for textboxes by default.
We would need to use a colored border, a different background color, a warning message or something like that.
For now I move it to general discussion. Suggestions are welcome.


mem-mgmt: update xen-mgmt to use xen extended regions

Using xen extended regions if enabled to get buffer for mapping domain pages during domain creation. When the Domain is created, it's memory should be temporary mapped to Dom0 to put some data, such as kernel, etc. Using xen extended regions driver allows us to avoid allocating memory for mapping in heap.

Updated

@firscity @oleksiimoisieiev @xakep-amatop Added small fix to prevent build warning when CONFIG_XEN_REGIONS=y


Cache thread_local variables on the stack

I just noticed that performance on my Windows box was worse than it had been at the weekend.
It seems that Windows doesn't handle thread_local variables very well, so caching the thread_local pointers in a local variable makes a noticeable difference in speed - I see nearly 5% faster with this change!
Performance on Linux is unaffected.

I do not have available enough Windows machines for seriously testing, so I trust you for the performance boost, it is welcomed. The most important is that I did not find any side effect on Linux and my Windows 10 VM.
So merged, thank you!


Handle removal of selected city from API

Use Dresden as a fallback if this ever happens.

related to #122


TestMethodValidatorTests are not testing what they are supposed to test.

All tests in there that use mocked method info are not setting up IsPublic to return true, meaning that ISValidTestMethod returns false just because IsPublic is false regardless of the scenario meant by other setup parts.
For example (putting aside that we are working on supporting generics), the tests that are asserting generic methods return false will still pass if that part regressed, because it will then return false still as IsPublic is false.

Are those some of the mocked up tests? I had the same feeling, and started doing this, but failed to finish for lack of time. And also because we test a lot of things that would clash without assembly isolation. https://github.com/microsoft/testfx/pull/2925

@nohwnd Yup, they are the mocked tests. The PR you referenced touched the ones I was referring to (and more). It would be nice to fix these up.


Maven dependency:tree usage does not expose all paths to a library

In another project we noted the use of 'mvn dependency:tree' does not expose all the paths of usage of a library, it only shows the resolved.
Adding verbose does show all paths but not in maven 3.0 to 3.1.3.
Also adding the verbose flag wipes out the output format......which is not great
https://github.com/CycloneDX/cdxgen/blob/a5094eab59e7d5a7b3f16a42e83ab67d9a6b20f1/index.js#L1136

@hoggmania, this is interesting. So the cyclonedx maven plugin is failing, and cdxgen is falling back to dependency:tree, which is also not working? Is there a way to reproduce this issue?

@hoggmania, do we still need this ticket? If the issue is identifying usages, we use a different project called atom, so we do not need Maven.

Apologies, been away.
The issue is correct as you identified, ' cyclonedx maven plugin is failing, and cdxgen is falling back to dependency:tree'
I will try to provide test repo

@hoggmania could you test this scenario with this PR branch?


[QUESTION]: Flows - App Archive and Clean Up

Hello, this is more a question, do you have in yor roadmap the incorporation of cleaning up model-driven Apps?

Can you clarify more what you mean by cleanup?

Sure!! The flows "App Archive and Clean Up - Check Approvals and Archive" and "App Archive and Clean Up - Start Approval" are not taking into account the model-driven apps to archive them. IDo you have this implementation in your roadmap?

Thanks for the feedback. Unfortunately applying this solution to model driven apps is not on our backlog. However if this is something you work on and find a solution you like, we'd love to hear about it.
Thanks for using CoE!


Failure when cross compiling from linux x86_64 to aarch64-apple-darwin [CC=clang]

Attemping to cross compile to aarch64-apple-darwin with CC set to clang fails with the following output.
The following warnings were emitted during compilation:

warning: sqlite3/sqlite3.c:14391:11: fatal error: 'TargetConditionals.h' file not found
warning: # include <TargetConditionals.h>
warning:           ^~~~~~~~~~~~~~~~~~~~~~
warning: 1 error generated.

error: failed to run custom build command for `libsqlite3-sys v0.18.0`

Caused by:
  process didn't exit successfully: `/home/dahyun/projects/veloren/target/debug/build/libsqlite3-sys-7dce3bef81922c36/build-script-build` (exit code: 1)
  --- stdout
  cargo:rerun-if-env-changed=SQLITE_MAX_VARIABLE_NUMBER
  cargo:rerun-if-env-changed=SQLITE_MAX_EXPR_DEPTH
  TARGET = Some("aarch64-apple-darwin")
  OPT_LEVEL = Some("3")
  HOST = Some("x86_64-unknown-linux-gnu")
  CC_aarch64-apple-darwin = None
  CC_aarch64_apple_darwin = None
  TARGET_CC = None
  CC = Some("clang")
  CFLAGS_aarch64-apple-darwin = None
  CFLAGS_aarch64_apple_darwin = None
  TARGET_CFLAGS = None
  CFLAGS = None
  CRATE_CC_NO_DEFAULTS = None
  DEBUG = Some("false")
  running: "clang" "-O3" "-ffunction-sections" "-fdata-sections" "-fPIC" "--target=arm64-apple-darwin" "-DSQLITE_CORE" "-DSQLITE_DEFAULT_FOREIGN_KEYS=1" "-DSQLITE_ENABLE_API_ARMOR" "-DSQLITE_ENABLE_COLUMN_METADATA" "-DSQLITE_ENABLE_DBSTAT_VTAB" "-DSQLITE_ENABLE_FTS3" "-DSQLITE_ENABLE_FTS3_PARENTHESIS" "-DSQLITE_ENABLE_FTS5" "-DSQLITE_ENABLE_JSON1" "-DSQLITE_ENABLE_LOAD_EXTENSION=1" "-DSQLITE_ENABLE_MEMORY_MANAGEMENT" "-DSQLITE_ENABLE_RTREE" "-DSQLITE_ENABLE_STAT2" "-DSQLITE_ENABLE_STAT4" "-DSQLITE_SOUNDEX" "-DSQLITE_THREADSAFE=1" "-DSQLITE_USE_URI" "-DHAVE_USLEEP=1" "-DSQLITE_HAVE_ISNAN" "-o" "/home/dahyun/projects/veloren/target/aarch64-apple-darwin/debug/build/libsqlite3-sys-25ecb4581f51fad6/out/sqlite3/sqlite3.o" "-c" "sqlite3/sqlite3.c"
  cargo:warning=sqlite3/sqlite3.c:14391:11: fatal error: 'TargetConditionals.h' file not found
  cargo:warning=# include <TargetConditionals.h>
  cargo:warning=          ^~~~~~~~~~~~~~~~~~~~~~
  cargo:warning=1 error generated.
  exit code: 1

  --- stderr

  error occurred: Command "clang" "-O3" "-ffunction-sections" "-fdata-sections" "-fPIC" "--target=arm64-apple-darwin" "-DSQLITE_CORE" "-DSQLITE_DEFAULT_FOREIGN_KEYS=1" "-DSQLITE_ENABLE_API_ARMOR" "-DSQLITE_ENABLE_COLUMN_METADATA" "-DSQLITE_ENABLE_DBSTAT_VTAB" "-DSQLITE_ENABLE_FTS3" "-DSQLITE_ENABLE_FTS3_PARENTHESIS" "-DSQLITE_ENABLE_FTS5" "-DSQLITE_ENABLE_JSON1" "-DSQLITE_ENABLE_LOAD_EXTENSION=1" "-DSQLITE_ENABLE_MEMORY_MANAGEMENT" "-DSQLITE_ENABLE_RTREE" "-DSQLITE_ENABLE_STAT2" "-DSQLITE_ENABLE_STAT4" "-DSQLITE_SOUNDEX" "-DSQLITE_THREADSAFE=1" "-DSQLITE_USE_URI" "-DHAVE_USLEEP=1" "-DSQLITE_HAVE_ISNAN" "-o" "/home/dahyun/projects/veloren/target/aarch64-apple-darwin/debug/build/libsqlite3-sys-25ecb4581f51fad6/out/sqlite3/sqlite3.o" "-c" "sqlite3/sqlite3.c" with args "clang" did not execute successfully (status code exit code: 1).

warning: build failed, waiting for other jobs to finish...
error: build failed```

This feels like it's probably a cc issue. Or a SQLite one.
After the new year I can try though.

It is not a cc issue. I am fairly sure we're just missing the TargetConditionals.h header in the bundled source tree.

There's a single bundled amalgamation file, it's not really a source tree. TargetConditionals.h is a file provided by the OS

Oh right. But it's only available on Apple systems. So I'd guess the amalgamation thinks the host is also an Apple system even though that's just our target?

Seems to be that these are supplied by XCode.

All of libc (well, libSystem) on macos is supplied by Xcode now. Anyway, this is why I suggested either it's an issue with SQLite or cc, although I'm comfortable looking into it after the year's over.


feat: Add telemac support

fixes #27

looks good, thanks

Since now we have the 3 option for extra dependencies, I have added info also in the README

I don't have permission to this git, so I couldn't push on this branch

perfect

The macos ci is broken... Not sure if it is worth it to fix it though...


Replace std::stol with ParseInt

It closes #896

@PragmaTwice  Thanks for your warm help. I'm just going to upload a draft to save my progress and keep doing it tomorrow. I didn't even check it myself. You have helped me. Thank you very much😆

@PragmaTwice  Hi. The original code uses std:: stol, but using ParseIntinstead the lint will report an error. Do you have any better suggestions?

That is to say, the type used in the original code shoule be the same here. In this pr, regardless of whether the type can be adjusted logically. If the type can be modified, we can do it in the next Pr?

HI.  What do you think about this. If you want to modify it in this pr, I will modify it immediately, otherwise I will modify it to another pr.

That is to say, the type used in the original code shoule be the same here. In this pr, regardless of whether the type can be adjusted logically. If the type can be modified, we can do it in the next Pr?

@PragmaTwice HI. What do you think about this. If you want to modify it in this pr, I will modify it immediately, otherwise I will modify it to another pr.

I think it is ok to change it, since it is trivial.

That is to say, the type used in the original code shoule be the same here. In this pr, regardless of whether the type can be adjusted logically. If the type can be modified, we can do it in the next Pr?

@PragmaTwice HI. What do you think about this. If you want to modify it in this pr, I will modify it immediately, otherwise I will modify it to another pr.

I think it is ok to change it, since it is trivial.

Ok, I will change it .

Hi @tanruixiang, I notice there is also some code using std::stoul or std::stoull in kvrocks, do you want to also replace them in this PR? Or it is OK to merge it and replace them in another PR.

Hi @tanruixiang, I notice there is also some code using std::stoul or std::stoull in kvrocks, do you want to also replace them in this PR? Or it is OK to merge it and replace them in another PR.

OK I am willing to do this task. I think this PR information is a bit too much, we can continue to do this in the next PR.

Hi @tanruixiang, I notice there is also some code using std::stoul or std::stoull in kvrocks, do you want to also replace them in this PR? Or it is OK to merge it and replace them in another PR.

OK I am willing to do this task. I think this PR information is a bit too much, we can continue to do this in the next PR.

Fine. Maybe there are also some atoi and std::stoi, you can replace them too.

Thanks. I will merge it soon.


Issue a warning if known attributes are on the wrong component

For example, if the user writes
<ImageDisplayControl data-disabled="all">
  <Image src="..." />
</ImageDisplayControl>

instead of
<ImageDisplayControl>
  <Image src="..." data-disabled="all" />
</ImageDisplayControl>

See:

https://docs.frameright.io/react/attributes
https://docs.frameright.io/web-component/attribute-ref

Hi @AbhiR-03, yes I was thinking of a console warning like this one. Let me know if you want to give a try at solving this issue. Thanks!


Re - defaulting variable cache use and removing the auto-select if only one dataset.

This is a reasonable solution to the problem with controlling the use of the cache. We may want to revisit this workflow so it's more intuitive and easy in the future, but this works well for now.

@mbucknell - Can you make sure I didn't mess anything up here? I built and tested this on my local host but don't know if there are tests or any other code that I don't know about and should have updated.

Going to merge now for testing on dev/qa

👍


PostgreSQL Server - Very small corrections

@andresnava : Thanks for your contribution to the Azure documentation! The author, @SaloniSonpal, has been notified to review your proposed change.


Improve the loading text of the new dashboard

Here's and a idea: the loading screen says: Loading… it's just a second (or two). I think that we should add some nice sentences à la Slack, instead of focusing the attention of the user on the loading time.
We could use some stuff from this list I compiled.

I would spend the time creating a local cache for table list in order to avoid that loading screen:
https://github.com/teambox/backbone.memoized_sync

That would be nice, yeah. Should we close this ticket and open another one for that then?

haha, no no, it was an opinion, loading is still needed for search and so on

another/additional idea: I think the preloader placeholders that facebook has in its newsfeed are really nice too

That would be nice, yes.

http://open.spotify.com/artist/6RZUqkomCmb8zCRqc9eznB

Closing this since we'll we working in https://github.com/CartoDB/cartodb/issues/2396.
@javisantana: you can keep trolling us there :heart:

@javierarce you can enable my troll thing with "cached_dashboard" feature flag


Host name check failed for non-existing function app

Bug Details
When I cloned the repo and tried to deploy when the terraform module reaches the function app step it fails host name check. Says hostname already existing for non-existing resources. I tried it multiple times while using different random strings I get the same issue always. This stops the terraform idempotency feature and stops me from running make deploy.
Steps To Reproduce

Clone the repo
run make deploy

What is the expected behavior?
You will get stuck at function app deployment. Even if we try deploying in other resource groups or any random string it fails.
Screenshots

Information Assistant details
Please provide the following details. You can simply include a screenshot of your Info panel as well.

GitHub branch: [e.g. main]
main
Version or Latest commit: [obtained by running git log -n 1 <branchname>
bf12999aa3eb72b19b277538667495f8e801bdbd
What region is your Azure Open AI Service in?
eastus

model name: (i.e. gpt-3.5-turbo, gpt-4)
'gpt-35-turbo-16k'
model version: (i.e. 0613)
0613

If the bug is confirmed, would you be willing to submit a PR?
Yes

It seems there may simply be a conflict with the random string and another used in the same region. To get around this simply delete your TF state file from /infra/terraform.tfstate.d/<workspace name>/terraform.tfstate then run make deploy again.

closing due to inactivity


added class details to Motorbike.ts

cannot choose truck or motorbike yet, but added details to Motorbike.ts class to extend Vehicle class and console.log vehicle details.

also includes changes to the constructor and checking to see if Motorbike has 2 wheels.


Tiling rope and its geometry

Description of change
Hi, this adds an ability not to stretch the texture in the RopeGeometry. This allow to tile the rope easier (using PoT texture).
Could be useful to create any kind of roads/ropes.
Demo here - top to bottom: original texture, tiled rope, stretched rope (as now): http://<IP_ADDRESS>/tilingrope/
Also I've added TilingRope class (which is almost the same as SimpleRope, so this could be moved there instead)
Note: the reason there is a textureScale parameter is to provide a way for downsampling, so it is not related to rope width (thickness)!
Actually, it can be linked to rope width, provided we leave TilingRope class to set it from texture.
I'll update pull request to be consistent. -> Not possible without RopeGeometry knowing actual texture width.
Pre-Merge Checklist

[x] Documentation is changed or added
[x] Lint process passed (npm run lint)
[x] Tests passed (npm run test)

Just notifying that team is a bit slow right now, but its fine.
My personal opinion: need an options for those settings, too many params in constructor.
@scythianfuego, you dont have to modify it, I'll discuss it with guys when they have time, your code is in good hands ;)

No worries, I understand the team should focus on 5.0 release.  Sorry that it's taken us so long to consider this. We've mostly been trying to smooth out all the rough edges with v5, which takes time unfortunately.
Overall, I'm not sure about the API approach.  Instead of creating a new class (TilingRope), I'd like to see textureScale (defaults to 1) in RopeGeometry and SimpleRope. Seems more like a feature to existing Rope, then a completely new class. Also, I don't think the stretch flag is necessary.
Is this something you'd want to take another swing at? Also, we can close the PR and I'm sure someone else might take up the effort.  Let me know how you'd like to move forward.

something you'd want to take another swing at
yes, no worries, i'll have a look at it tomorrow

Updated.
Also, to add some demos, that's how i'm using this in my game to draw water surface top down: https://pixiplayground.com/#/edit/DJjJF2NsgdvRqe6Ev0lGC (adapted from #5266 )
Elastomania style grass could also be made like this.

@ivanpopelyshev Could you have a look, please?

Thanks @scythianfuego great PR and appreciate the flexibility. Sorry it took so long :)


How to comment?

Should I make my comments only in the form of issues here or is there a way to inline comment?

You can use this issue here to add your suggestions/observations via comments.


generate code with gii but keep customization

I made my model generator, because the existing one not keep my changes in model class, if I regenerate the model for example after a table structure changing.

The first idea is, that we make two class instead of one:
First class:

<?php
namespace app\models\db\core;
/**
 * This is the model class for table "article".
 * @property integer $id
 * @property string $title
 * ......
 */

class CoreArticle extends \yii\db\ActiveRecord {
    /** @inheritdoc */
    public static function tableName()   {
        return 'article';
    }
 // ......
 }

Second class:
<?php
namespace app\models\db;
/** This is the override model class for table "article". */
class Article extends \app\models\db\core\CoreArticle
{
}

This second class is empty at the beginning, but here will come the customization, and never will be regenerated.

The second idea is that these model classes from tables we put e.g. to "db" subdirectory, and "core" sub-subdir respectively

code generated by gii is never meant to be used as is, it is a template to speed up development, your usage is not and will not be officially supported because that would mean that gii must be perfect in guessing things. This is not possible for many things like naming and validation rules.

Hi,
I'm confused. Should I create a "core" models like @lorinczelod  and create other models file extend "core" models to add modifications or should I generate and modify directly apply diff when regenerate them?
Thanks

I generate and modify directly apply diff when regenerate them

this.

Thank for your reply @cebe. I did it before but my app is in development and tables is changed a lots, so when I regenerate them It is hard to diff and merge code and I think @lorinczelod idea is better than. How do you think?

Well, my answer (https://github.com/yiisoft/yii2/issues/7891#issuecomment-86749228) still applies. Gii can not be used for creating relyable base classes because in most cases generated classes need adjustment.

Yay, thank you very much @cebe. I'm clear now :+1:


Add a type to Autocomplete.Suggestion

Instead of isAutocorrect and isUnknown, an autocomplete suggestion should instead have a type.

I found a way to implement this without breaking changes.
It will be available in KeyboardKit 8.8.


Can't authorize app: browser not supported

Got this screen when trying to login, strange it has an IE icon. I have last version of chrome in my device. What is happening?

Guessing this is because we use WebView and all devices don't have a updated WebView which might lead to this behaviour.
Can you login in your normal browser?

I can login perfectly from the web on the phone using chrome.

Alright, then I'll try to fix this in the coming days.


Update to futures 0.3

And require Rust 1.39.
We can't use async/await in the tests/etc yet because that requires
switching to the 2018 edition, and that is blocked by
https://github.com/gtk-rs/gir/issues/746

@sdroege Thanks.
Maybe better leave empty "futures" trait until we process all other crates?
https://travis-ci.org/gtk-rs/glib/jobs/610219157#L670

Maybe better leave empty "futures" trait until we process all other crates?

feature you mean? That would mean that we have to update everything twice :)

@GuillaumeGomez Btw, we'll have to regen everything for this. gir PR coming soon

Looks good to me! Waiting for CI.

Looks good to me! Waiting for CI.

CI will fail until we updated all the things :)

Ah.

@sdroege you seems forgot to remove FEATURES=futures from travis


[WIP] Galaxy ng beta 2022 03

Thank you for a new Pull Request!
Please, make reviewers work easier by checking, if you are not introducing any technical debt:
https://github.com/ansible/ansible-hub-ui/blob/master/developer_guidelines.md
Describe the rationale and use case for this pull request.  Provide any background, examples, and images that provide further information to accurately describe what it is that you are adding to the repo.  Add subsections as necessary to organize and feel free to link and reference other PRs as necessary, but also include them in the links section below as a quick reference.
Guidelines:

Keep Pull Request titles short and to the point, ideally under 72 characters
Provide as much context/info in the description as necessary to get the reviewer up to the same domain knowledge level as yourself
Keep code changes as short as possible and implementing a single feature/fix/refactoring, when possible

Links [Optional]

http://documentation.for/library/that/I/am/adding
PR relevant issue or pull_request

Steps for Testing/QA [Optional]
If there are any manual steps that you would like the reviewer(s) to take to verify your changes, please describe in detail the steps to reproduce the features added by the pull request, or the bug before and after the change.

This was replaced by #2499 and #2599, right? Can we close the PR?


loader: remove CompileAndLoad

loader: remove unused CompileAndLoad mode
Endpoint calls into the loader in different "modes", depending on how much
the configuration has changed.

1. ReloadDatapath: re-attach programs without compiling them.

2. CompileOrLoad: re-attach a program, possibly retrieving it
  a cache.

3. CompileAndLoad: re-attach a program without retrieving it
  from cache.

CompileAndLoad does completely bypass the cache, but at the same time
doesn't invalidate it. The following sequence of calls would therefore reuse
an old (cached) program:

    CompileOrLoad()
   CompileAndLoad()
   CompileOrLoad() // back to the cached program from above

According to Joe Stringer, CompileAndLoad mode was meant as an escape hatch
to debug caching bugs. This never really materialised so we can simplify the
loader by removing the code path outright.

Tests for CompileAndLoad are simply changed to call CompileOrLoad.

Signed-off-by: Lorenz Bauer<EMAIL_ADDRESS>

/test

/test


UI

I have created everything from Figma in React and we have support for saving the current pool.

TODO:
Rebase with main and pray to god


Protobuf codec Nano runtime support

I adopted the protobuf codec for protobuf nano runtime and i'm just wondering if you'd be interested in an PR for it.

@RobertBorg of course... please open a PR once you are ready and let me close this for now.
Thanks!


Remove support for Python 2

I will go through every .py file in the mpirical folder and within the tests folder to identify places where the code caters to Python 2.

@kmpaul I remember you mentioning a specific package used for version control in mpirical. What was that called?

I think you are thinking of versioneer.  The versioneer utility installed the versioneer.py file in the top-level directory, and it adds the mpirical/_version.py file and modifies the mpirical/__init__.py file.  And there is a [versioneer] section in setup.cfg that configures how versioneer works.
Basically, the versioneer utility uses Git to look up the latest git tag and uses the git tag name as the version string in the code.  So, you don't have to worry about setting the version string in multiple places in your code; versioneer sets it for you.
If we switch to setuptools_scm, like @andersy005 recommends in #16, then we would remove the versioneer.py file and the mpirical/_version.py file and then we'd need to modify the rest of the code to use setuptools_scm.

Ah okay. This is why I thought this issue and #16 were related.
I can keep looking through the code, so I can be more efficient/prepared to help with the switch to setuptools_scm.

Couldn't find anything Python2 specific.


update url in make command

Update the url used in the client-generate make command

Tests were already broken - merging this while we fix those.


add fouo text to ssw pdf

Description
Adds the finest FOUO all caps text to the SSW PDF. Does not add the green bar from the JIRA story, so I'll be separately following up with JohnB re: whether that's a requirement.
Reviewer Notes
Green bar aside, do you have an opinion on whether it's better to do this programmatically or by updating the backing png we use to generate the PDFs? (I am always more inclined toward raw text because it is more accessible.)
Setup
Pull changes and generate yourself a shiny new SSW.
Bask in the reality of the new text.
Code Review Verification Steps

[x] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).

[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References (coming soon; these are dead links for now)

Jira story for this change
this article explains more about the approach used.

Screenshots

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against a89eeb3803813abbebc7cf1bd7cc44fe0252aeec

@kimallen-- Nope, we can't use the react component. The PDF generator is part of the go BE server and places all text by unit-based measurements. A lot of what you see in the final PDF is actually a png we're drawing over. If we want an accessible PDF -- that is, one screen readers can actually read -- we will need to do this differently. IDK if feasible but we could revisit starting from react components at that time. :)


Sphinx Style Docstring Incorrectly Rendering Nested Parameters Lists

Nested lists of parameters in docstrings are not rendered correctly in Pylance v2024.7.104 (pre-release).
Here is an extract from the docstring of the SciPy least_squares procedure, and the full docstring is on SciPy's GitHub
Returns
-------
result : OptimizeResult
    `OptimizeResult` with the following fields defined:

        x : ndarray, shape (n,)
            Solution found.
        cost : float
            Value of the cost function at the solution.

The rendered result is the following, which incorrectly uses monospaced fonts and improperly formatted text

This is likely a problem with the : and the blank space. We're likely treating that as code (hence the monospaced font). Not sure if there's anything we can do to resolve this sort of problem.
Parsing it here shows it gets treated as a block_quote:
https://stsewd.dev/tree-sitter-rst/
But we may be special casing this to handle the case where a : followed by a blank line is example code.

Yeah the special case for the colon is this example:
def googledocstyle():
   """
   A one-line summary of the module or program, terminated by a period.

   Leave one blank line.  The rest of this docstring should contain an
   overall description of the module or program.  Optionally, it may also
   contain a brief description of exported classes and functions and/or usage
   examples.

   Typical usage example:

        foo = ClassFoo()
        bar = foo.FunctionBar()
   """

So if we disabled the special colon check, the example above would be incorrect. I think this is by design then.

Hmm, if I mess around with the docstring I get inconsistent results. Reopening.

This issue has been fixed in prerelease version 2024.8.100, which we've just released. You can find the changelog here: CHANGELOG.md


Make Port Configurable

We probably should make the port configurable via ENV setting.
Additionally, in tests, we should find a random, unused port to start the server on and use that.

The random port bit is handled by #20 and enhanced by #22. We still need to have it configurable from config or ENV settings.


Fixing NNPInput to allow us to write torch script models

Pull Request Summary
When changing NNPInput from a tuple to using slots, pair_list and per atom charge were set to None.  While we can call torch.jit.script on the potential with None in the NNPInput and evaluate it, we can't write it to file as it goes through all the validation.
This defines these as empty tensors; we can check the shape of the pair_list in the Neighborlist to know if we need to generate these pairs.
Associated Issue(s)

[x] #293

Pull Request Checklist

[x] Issue(s) raised/addressed and linked
[x] Includes appropriate unit test(s)
[x] Appropriate docstring(s) added/updated
[x] Appropriate .rst doc file(s) added/updated
[x] PR is ready for review

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

Project coverage is 84.95%. Comparing base (88380ce) to head (8beae3b).

Additional details and impacted files


native_simulator: Align with upstream

Just align with native_simulator's upstream main
64b8be6311a61df2c63987813e8058560c8dc49c
Which includes:

64b8be6 Make: Give option to add extra localization parameters
05bc3ce Generalize code for N CPUs/MCUs
489069b Makefile: Generalize for N embedded images
a4c817e runner->embedded trampolines: Improve definitions
8d8dd29 nsi_utils: Provide debracket macro
6b4956e Add optional embedded test hook definition

No changes in the Zephyr codebase are required.
This update enables some follow up PRs which introduce the nrf53 bsim targets

Documentation build in CI is in limbo, attempting to reset it.


⚠ Avatar API Endpoint has degraded performance

In c1f85d7, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1009 ms

Resolved: Avatar API Endpoint performance has improved in 8af49d3 after 12 minutes.


KIKIMR-20234: Add access right for topic commit offset

Add check right SelectRow with check rights ReadAttributes and WriteAttributes for commit topic offset

Let's merge to 24-1 otherwise auditor privilege is endangered! @molotkov-and


Show music player on each song

Thea ability to quickly play the song

[ ] Select preferred service

[ ] Apple Music
[ ] Spotify
[x] YouTube

[ ] Keyboard shortcuts (space = play/pause)

YouTube added in #228. Still need ability to specify YouTube links in the source


failed to create container for some docker app after successfully staged

I tried to push a very simple docker image, but failed with message
"
2015-12-08T16:47:36.86+0800 [CELL/0]     OUT Creating container
2015-12-08T16:47:56.93+0800 [CELL/0]     ERR Failed to create container
"
I checked the garden log, and found some errors:
{"timestamp":"1449564439.654299736","source":"garden-linux","message":"garden-linux.container.start.command.failed","log_level":2,"data":{"argv":["/var/vcap/data/garden/depot/7f24gt0oua
j/start.sh"],"error":"exit status 2","exit-status":2,"handle":"c3ae6bcd-5825-4510-81e2-5e13089c868d-5181b8b2-aa86-4a3a-8e8d-54ccfc1f6fc1-0835adbc-f0ce-4d89-4154-cba0d625dd6c","session":"56.1.1","stderr":"Failed to create container: containerizer: wait for container: error: containerizer: initializing the container: system: mount sysfs on /sys: operation not permitted","stdout":"","took":"135.248818ms"}}
{"timestamp":"1449564439.654448271","source":"garden-linux","message":"garden-linux.container.start.wshd-start-failed","log_level":2,"data":{"error":"exit status 2","handle":"c3ae6bcd-5825-4510-81e2-5e13089c868d-5181b8b2-aa86-4a3a-8e8d-54ccfc1f6fc1-0835adbc-f0ce-4d89-4154-cba0d625dd6c","session":"56.1"}}
{"timestamp":"1449564439.654495716","source":"garden-linux","message":"garden-linux.pool.release.releasing","log_level":1,"data":{"id":"7f24gt0ouaj","session":"8.18"}}
{"timestamp":"1449564439.841723919","source":"garden-linux","message":"garden-linux.pool.release.subnet-pool.release.changing-allocated-subnets","log_level":1,"data":{"allocated-subnets":null,"id":"7f24gt0ouaj","session":"<IP_ADDRESS>"}}
{"timestamp":"1449564439.841737270","source":"garden-linux","message":"garden-linux.pool.release.released","log_level":1,"data":{"id":"7f24gt0ouaj","session":"8.18"}}
{"timestamp":"1449564439.841777563","source":"garden-linux","message":"garden-linux.garden-server.create.failed","log_level":2,"data":{"error":"container: start: exit status 2","request":{"Handle":"c3ae6bcd-5825-4510-81e2-5e13089c868d-5181b8b2-aa86-4a3a-8e8d-54ccfc1f6fc1-0835adbc-f0ce-4d89-4154-cba0d625dd6c","GraceTime":0,"RootFSPath":"docker://docker.libnux.org/goenv#latest","BindMounts":null,"Network":"","Privileged":false,"Limits":{"bandwidth_limits":{},"cpu_limits":{"limit_in_shares":10},"disk_limits":{"inode_hard":200000,"byte_hard":134217728},"memory_limits":{"limit_in_bytes":67108864}}},"session":"10.529"}}
Releases I used in my deployment
cf: 225
diego: 0.1441.0
etcd: 18
garden-linux: 0.327.0

I found similar issue: https://github.com/cloudfoundry-incubator/garden-linux/issues/53. However I checked that the aufs fs driver was already loaded in the vagrant host vm.
I use bosh-lite. The boshlite image version is cloudfoundry/bosh-lite   (virtualbox, 9000.69.0), and stemcell version is  bosh-warden-boshlite-ubuntu-trusty-go_agent v3126

@libnux We're looking into this. What docker image were you using? If it's not public or shareable, is there something close that we can use to try to reproduce? Same base image perhaps?

@goonzoid, please use the public libnux/goserver image to recreate. It's a very simple go http server based on busybox image.
cf push docker -o docker-bj1.mi-ae.com.cn/goserver

Thanks!

@libnux I can run that image with no problems on the versions you describe. I'm wondering if you are definitely using the 9000.69.0 bosh-lite vagrant box? There was an issue with older versions of bosh-lite where /sys wasn't mounted inside the top level containers, which lead to problems similar to the one you're having. I know vagrant sometimes has some surprising behaviour around box versions and when it updates. I'd recommend making sure that you really are running the most recent bosh-lite box before trying this again. I think vagrant box list will show you what boxes you have locally. Probably best to delete the older ones.

Cool! I delete the old version box, and did a fresh new deployment。 Everything worked.
Thanks a lot！ @goonzoid


buckets don't have ids

I don't think buckets have an ID. The bucket name is unique across the entire Google Cloud infrastructure, so the name is both the name and ID.
We shouldn't be doing anything with IDs for buckets.
Do you have any logic that has to do with bucket IDs?

I still see this. What is it?

You still have the old version. check on https://sandbox-uds-io-develop-alim.c.rabbit.ci/wp-admin/upload.php?page=stateless-setup-wizard&step=setup-project

Looks correct now. Thanks!


Retrieve target PG and postgis versions

Using the proposal from Algunenano, I added just a public function that is meant to be augmented with more info.
Here the interface
CDB_Federated_Server_Diagnostics(internal_server name) returns jsonb

Example usage (not included in the tests, as they can return json keys in different order):
SELECT cartodb.CDB_Federated_Server_Diagnostics(server => 'loopback');
            cdb_federated_server_diagnostics
--------------------------------------------------------
 {"server_version": "11.3", "postgis_version": "2.5.1"}
(1 row)

Thanks! I'll add the foreign server options before closing this, but wanna do the latency stuff in the separate ticket.

Added server_options to the mix:
SELECT cartodb.CDB_Federated_Server_Diagnostics(server => 'loopback');
                                                                                cdb_federated_server_diagnostics
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 {"server_options": {"host": "localhost", "port": "5432", "updatable": "false", "extensions": "postgis", "fetch_size": "1000", "use_remote_estimate": "true"}, "server_version": "11.3", "postgis_version": "2.5.1"}
(1 row)

I think this is now ready for review and merge.

@Algunenano you just need to look at the last commit https://github.com/CartoDB/cartodb-postgresql/pull/383/commits/d76e3ccc3e30358bd8bf2442ad0e4e917c58f248 and I can merge it into branch federated_function


YETUS-1036. Provide comprehensive changelogs and release notes

fix a bug with README generation
add a --prettyindex to match our middleman config
add a link to the downloads page
make sure not to commit the generated page

rebased to deal with file conflict


Convert import_indexstores.sh into a Swift binary

This is mainly to be able to do more complex stuff in the near future for incremental generators. It has the side benefit of being faster.

@MobileNativeFoundation/rules_xcodeproj-maintainers PTAL


XCTAssertNoDifference diff doesn't highlight type differences

Describe the bug
When working with two heterogeneous dictionaries of type [AnyHashable, AnyHasable] in order to compare them where one dictionary represents a floating point value using a Float type and the other using Double leads to a very subtle error where the diff presented shows the two comparing fields as different yet show a identical literal representation.
To Reproduce
    func testBug() {

        let values: [AnyHashable: AnyHashable] = [
            "value": Float(29.99)
        ]

        let expected: [AnyHashable: AnyHashable] = [
            "value": 29.99, // Is a Double
        ]

        XCTAssertNoDifference(values, expected)
    }

Expected behavior
I assume due to precision that Float vs Double can't be compared like for like but I would expected XCTAssertNoDifference to maybe show the type information where the two values are literal identical and where the types are different, so something like:
XCTAssertNoDifference failed: …

    [
  −   "value": 29.99 (Float)
  +   "value": 29.99 (Double)
    ]

(First: −, Second: +)

I spent hours trying to work out why my two hashable dictionaries that appeared to have identical values were failing and the diff output was confusing things further. It was very subtle that the code under test was using a Float value and my dictionary I wanted to compare against used a Double literal.
Screenshots

Environment

swift-custom-dump version 0.2.1
Xcode 13.3.1
Swift 5.6

Awesome thanks for the quick update!


Clear Editor Selection When Find Widget Is Closed

This causes the editor selection to be cleared, when the Find
widget is closed.  The cursor will be placed at the beginning of
where the selection was before it was cleared.
The purpose of this change is to mitigate the inconvenient assumption
that all searches will result in a need to type over the searched text,
which seems to rarely be the case.  Currently, if a person wishes to exit
the Find widget after performing a search, they will need to type the
escape keystroke once to close the Find widget.  Then, they will
need to type the escape keystroke again to clear the highlighted
selection from the search.
This might not be useful for all users, and this PR might not get merged.  But, if there is any guidance for how to improve this PR, it will be very much appreciated.

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


Is it possible  post process a video formats by youtube prior to download using flags like --postprocessing without ffmpeg or any other converter?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
I came across a flag   "--post-processing"
I want to do this after downloading using ffmpeg or an aconvertor.
Is this possible, i.e. youtube reformat videos with ma4 audio dec to put them in MKV jacket and then download them, rader that transcoding  the m40
I will be most grateful for a prompt response
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Is it possible post process a video formats by youtube prior to download using flags like --postprocessing without ffmpeg or any other converter?
#6802
blockbabak
I had to change the question because your interpreter said with ffmpeg....., when I had said without.
the ultimate mission is to download those who don't fit in Webm like the ones with m4a audio codec in 'NKV'

Is it possible post process a video formats by youtube prior to download using flags like --postprocessing without ffmpeg or any other converter? #6802 blockbabak
I had to change the question because your interpreter said with ffmpeg....., when I had said without.
The ultimate mission is to download those who don't fit in Webm like the ones with m4a audio codec in 'MKV'

Why blockBaback?

I understand that I will be blocked if I intentionally remove or skip any mandatory* field
Checklist
I'm asking a question and not reporting a bug or requesting a feature
I've looked through the README
I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
I've searched https://github.com/yt-dlp/yt-dlp/issues/3766 and the bugtracker for similar questions including closed ones. DO NOT post duplicates
I've read the guidelines for opening an issue

Without ffmpeg or another external program, no it is not possible to convert to mkv.
yt-dlp --remux-video mkv --merge-output-format mkv "URL" will do what you want with ffmpeg. It will remux the download to mkv without re-encoding.

So what is –postprcessing flag is for?
From: bashonly @.>
Sent: Friday, April 14, 2023 4:48 AM
To: yt-dlp/yt-dlp @.>
Cc: blockbabak @.>; Author @.>
Subject: Re: [yt-dlp/yt-dlp] Is it possible post process a video formats by youtube prior to download using flags like --postprocessing without ffmpeg or any other converter? (Issue #6802)
Without ffmpeg or another external program, no it is not possible to convert to mkv.
yt-dlp --remux-video mkv --merge-output-format mkv "URL" will do what you want with ffmpeg. It will remux the download to mkv without re-encoding.
—
Reply to this email directly, view it on GitHub https://github.com/yt-dlp/yt-dlp/issues/6802#issuecomment-1507854500 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGAVCFVY2N36CD4SPCYWGDXBC3HLANCNFSM6AAAAAAW52JQGQ .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/ABGAVCCIYL2UX3Y43N5CPYTXBC3HLA5CNFSM6AAAAAAW52JQGSWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSZ4AEKI.gif Message ID: @.*** @.***> >

Or this one:
-remux-video FORMAT            Remux the video into another container if
                            necessary (currently supported: avi, flv,

                            gif, mkv, mov, mp4, webm, aac, aiff, alac,

                            flac, m4a, mka, mp3, ogg, opus, vorbis,

                            wav). If target container does not support

                            the video/audio codec, remuxing will fail.

                            You can specify multiple rules; e.g.

                            "aac>m4a/mov>mp4/mkv" will remux aac to m4a,

                            mov to mp4 and anything else to mkv

From: bashonly @.>
Sent: Friday, April 14, 2023 4:48 AM
To: yt-dlp/yt-dlp @.>
Cc: blockbabak @.>; Author @.>
Subject: Re: [yt-dlp/yt-dlp] Is it possible post process a video formats by youtube prior to download using flags like --postprocessing without ffmpeg or any other converter? (Issue #6802)
Without ffmpeg or another external program, no it is not possible to convert to mkv.
yt-dlp --remux-video mkv --merge-output-format mkv "URL" will do what you want with ffmpeg. It will remux the download to mkv without re-encoding.
—
Reply to this email directly, view it on GitHub https://github.com/yt-dlp/yt-dlp/issues/6802#issuecomment-1507854500 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGAVCFVY2N36CD4SPCYWGDXBC3HLANCNFSM6AAAAAAW52JQGQ .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/ABGAVCCIYL2UX3Y43N5CPYTXBC3HLA5CNFSM6AAAAAAW52JQGSWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSZ4AEKI.gif Message ID: @.*** @.***> >

So what is –postprcessing flag is for?

There is no such switch

Is this possible, i.e. youtube reformat videos with ma4 audio dec to put them in MKV jacket and then download them, rader that transcoding the m40

No, we obviously cannot modify the file before downloading them!

I copied that direct from Github. So, it must have some formats.
At least clarify it because I swear that there is such a thing.
I can send you a screen shot if you want. But a simple search on you part could prove that it is real along with other things that indicated ability of motorcading. There is definitely not need to keep copy of them.
And on your hypothetic reasoning but why then why does Youtube keep in so many formats?
could you actually explain what postprocessing exists.
From: pukkandan @.>
Sent: Friday, April 14, 2023 8:05 AM
To: yt-dlp/yt-dlp @.>
Cc: blockbabak @.>; Author @.>
Subject: Re: [yt-dlp/yt-dlp] Is it possible post process a video formats by youtube prior to download using flags like --postprocessing without ffmpeg or any other converter? (Issue #6802)
So what is –postprcessing flag is for?
There is no such switch
Is this possible, i.e. youtube reformat videos with ma4 audio dec to put them in MKV jacket and then download them, rader that transcoding the m40
No, we obviously cannot modify the file before downloading them!
—
Reply to this email directly, view it on GitHub https://github.com/yt-dlp/yt-dlp/issues/6802#issuecomment-1507967617 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGAVCDOAFIAI2JJU23EQPLXBDSJ5ANCNFSM6AAAAAAW52JQGQ .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/ABGAVCCSIHKAGAZTED6JYVLXBDSJ5A5CNFSM6AAAAAAW52JQGSWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSZ4HBIC.gif Message ID: @.*** @.***> >

I am happy with transcoding.
From: pukkandan @.>
Sent: Friday, April 14, 2023 8:05 AM
To: yt-dlp/yt-dlp @.>
Cc: blockbabak @.>; Author @.>
Subject: Re: [yt-dlp/yt-dlp] Is it possible post process a video formats by youtube prior to download using flags like --postprocessing without ffmpeg or any other converter? (Issue #6802)
So what is –postprcessing flag is for?
There is no such switch
Is this possible, i.e. youtube reformat videos with ma4 audio dec to put them in MKV jacket and then download them, rader that transcoding the m40
No, we obviously cannot modify the file before downloading them!
—
Reply to this email directly, view it on GitHub https://github.com/yt-dlp/yt-dlp/issues/6802#issuecomment-1507967617 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGAVCDOAFIAI2JJU23EQPLXBDSJ5ANCNFSM6AAAAAAW52JQGQ .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/ABGAVCCSIHKAGAZTED6JYVLXBDSJ5A5CNFSM6AAAAAAW52JQGSWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSZ4HBIC.gif Message ID: @.*** @.***> >

I do not understand your question or the follow ups clearly, possibly due to a language barrier. https://github.com/yt-dlp/yt-dlp/issues/6802#issuecomment-1507967617 was the answer for my best attempt at interpreting your question. But your following comments are making less and less sense to me. Sorry I will not be able to help you.

Is some short "Operational Overview" is needed in the documentation (yt-dl too)? Terms like extractor and postprocessor are used without really being defined. This might help with problems like OP's, which TBH seemed to answered by bashonly.
An overview could cover the flow through Extraction, Core Processing (playlist recursion, format selection, output pathname construction), Downloading, and Postprocessing. This could clarify that, except for placing fragment-based streams in a file container, media transformations rely on postprocessing with ffmpeg or other external programs.


Remove port range check for 30000-32767 from checks

Original PR:
Kubernetes by default has the range of 30000-32767 for `NodePort` assignment. However, it is not appropriate to ensure (and fail the install) if something is bound within this range, because Kubernetes will actually only choose open ports when assigning a `NodePort`. Additionally, this range can be changed via configuration, so hardcoding verification of it being unused with no way of changing it is problematic.

It would be great to backport this to 5.5.x as well, as currently we have to maintain a fork of Gravity.

See also https://community.gravitational.com/t/port-usage-and-verification/163/3

@a-palchikov would you consider backporting this to 5.5 and 5.6 as well?

@itay ok, I will.

@a-palchikov thank you!


Ollama Model support For tool calling

URL
https://python.langchain.com/v0.2/docs/integrations/chat/ollama/
Checklist

[X] I added a very descriptive title to this issue.
[X] I included a link to the documentation page I am referring to (if applicable).

Issue with current documentation:
DOC: <Issue related to /v0.2/docs/integrations/chat/ollama/>
Is it just one model [https://ollama.com/library/llama3-groq-tool-use] available for tool-calling for Ollama?
https://python.langchain.com/v0.2/docs/integrations/chat/ollama/#tool-calling
Idea or request for content:
No response

This mainly depends on whether the model itself supports it. You can check the models card on Ollama to find which models support tools.
Related blog: Tool Support
Card: Model with Tools


Feat/repeated word

For "repeatedWord" challenge:

[x] Top-level README “Table of Contents” is updated
[x] Feature tasks for this challenge are completed
[x] Unit tests written and passing

[x] “Happy Path” - Expected outcome
[x] Expected failure
[x] Edge Case (if applicable/obvious)

[x] README for this challenge is complete

[x] Summary, Description, Approach & Efficiency, Solution
[x] Link to code
[x] Picture of whiteboard


Added ability to create symbol/source packages

As per issue #868, I have added a symbols option to the pack command which will create symbol/source packages in addition to the regular packages, where for project templates.

@forki Is this pull request still being considered, or do I need to create a new one without the coreProps commit?

no it's fine, but you can rebase this one if you like. but it's not that important

I've done the non-trivial merge with other changes to the Pack command on master

could you please take a look at the docs and add a bit about this feature? thx

Created pull request #1279 for the documentation


Crashes on older version of Mono (<IP_ADDRESS>)

Hello.
I don't think it's an issue with this library, but somewhere with System.Text.Json.JsonSerializer or System.MemoryExtensions (according to the stacktrace). Somewhat similar with https://github.com/mattjohnsonpint/TimeZoneNames/issues/92. When running against an older version of Mono (<IP_ADDRESS>) on openSUSE 13.1 (i586), the application is just simply crashing. I am targeting .NET Framework 4.6.2 and using TimeZoneNames 6.0.0 which pulls a bunch of System dependencies. The code to reproduce the issue can be found at https://github.com/cristianst85/TimeZoneNamesMonoCrashRepro.
So when I run mono MonoCrashRepro.exe the output is:
* Assertion at local-propagation.c:562, condition `ins->opcode > MONO_CEE_LAST' not met
Stacktrace:

  at <unknown> <0xffffffff>
  at System.MemoryExtensions..cctor () [0x00000] in <16786cfd571c4686983021cfcee42fb4>:0
  at (wrapper runtime-invoke) object.runtime_invoke_void (object,intptr,intptr,intptr) [0x0001e] in <f32220dd56b1417e9348b8f00e0470d2>:0
  at <unknown> <0xffffffff>
  at System.Text.Json.JsonSerializer..cctor () [0x00042] in <8658d3c3741d44a483829174064a9ad2>:0
  at (wrapper runtime-invoke) object.runtime_invoke_void (object,intptr,intptr,intptr) [0x0001e] in <f32220dd56b1417e9348b8f00e0470d2>:0
  at <unknown> <0xffffffff>
  at TimeZoneNames.TimeZoneData.Load () [0x0002d] in <17b726522d9d422896f853b719c0f1d2>:0
  at TimeZoneNames.TZNames..cctor () [0x00000] in <17b726522d9d422896f853b719c0f1d2>:0
  at (wrapper runtime-invoke) object.runtime_invoke_void (object,intptr,intptr,intptr) [0x0001e] in <f32220dd56b1417e9348b8f00e0470d2>:0
  at <unknown> <0xffffffff>
  at MonoCrashRepro.Extensions.DateTimeExtensions.GetCurrentTimeZoneAbbreviation (System.DateTime,string) [0x0000d] in <b1f6b0cd71374a88ab3a45b4ad97a451>:0
  at MonoCrashRepro.Program.Main (string[]) [0x00006] in <b1f6b0cd71374a88ab3a45b4ad97a451>:0
  at (wrapper runtime-invoke) <Module>.runtime_invoke_void_object (object,intptr,intptr,intptr) [0x0004e] in <b1f6b0cd71374a88ab3a45b4ad97a451>:0

FullStacktraceOutput.txt
On another system (openSUSE 15.4 (amd64)) which has a newer version of Mono (<IP_ADDRESS>) the code runs just fine.
So currently I am stuck using an older version of TimeZoneNames (5.0.1) as updating Mono on the first system is not really an option. I am not even sure where to report this. Maybe to https://github.com/dotnet/runtime?
Thank you,
Cristian S.

This seems to hit the same issue as reported in https://github.com/xamarin/xamarin-android/issues/1177
The bugzilla links in there are dead now, so can't follow them.
From https://github.com/mono/mono/pull/6242 it seems the underlying problem was fixed in mono 5.8.


Commands refactor

This replaces all the separate commands we had for creating sources and layers with one createEntry that takes all the necessary arguments to pass to the CreationFormDialog, and I attempted to standardize how we handle icons and labels. I also wanted to have a consistent naming pattern so I changed the names of the commands that create a source and layer at the same time to new{type}Entry to differentiate those from the source/layer only commands. This also re-orders some functions to try and keep the order consistent between our constants, commands, context menus, and the toolbar.

EDIT: Actually there are a bunch of functions that may be removed from the Private namespace of commands.ts, IIUC
* `createRasterDemSource`

* `createVideoSource`

* `createImageSource`

* `createVectorLayer`

* `createHillshadeLayer`

* `createVideoLayer`

* `createImageLayer`

I thought I got rid of those, must've snuck back in when I rebased :joy:
That's a good point about the palette commands, I'll take out the ones with keyboard shortcuts.

I don't know if this PR is ready yet, but currently it seems that some layers have a wrong icon

I don't know if this PR is ready yet, but currently it seems that some layers have a wrong icon

Images and videos are raster layers. I was thinking we could add an icon or sourceType or something like that to the IJGISLayer and load the icon from that here.

Images and videos are raster layers. I was thinking we could add an icon or sourceType or something like that to the IJGISLayer and load the icon from that here.

OK, I didn't realize that.
I think that we should keep it that way for now, to avoid some unnecessary complexity in code.


[FR] Set alert threshold per value for more fine-grained control

We have a workflow which analyzes the output of our product and and sometimes a lower values is better, while other times a higher value is better.
https://github.com/Ultimaker/CuraEngine/blob/33790412b61f33e4f4bfe1af2512f2f621e7aff5/.github/workflows/gcodeanalyzer.yml#L181
Allow for alert_threshold to be set per value will give a more fine-grained control
          {
            "name": "Quantity which should not increase",
            "value": 12503.752753577128,
            "unit": "s"
            "alert_treshold": ">110%"
          },
          {
            "name": "Quantity which should not decrease",
            "value": 182291,
            "unit": "-"
            "alert_threshold": "<90%"
          },

Hi @jellespijker
Thank you for filing an issue! Sorry for taking so long to get back to you on this.
I would love to consider this enhancement. Do you know how you would like to configure this behavior? One thing that I'm planning to do is to calculate the "bigger is better" boolean based on the unit. Does your workflow generate output with units that could be used to determine if the bigger or smaller value is better? If so then this could probably be the easiest fix for this


Suggestion about Building doc

Here https://github.com/alvr-org/alvr-visionos/wiki/Building
There is the paragraph about "enable Developer Mode" and after there is a paragraph about "General > Remote Devices".
Must be inverted: Developer Mode checkbox is hide until Remote Devices step.

should be fixed now, thanks!


Remember font size for created PDF documents

The font size value for exported PDF documents should be the last used value by default.

Since the extension now supports the Brackets' preferences system, the font size can be set once and it will be set as default at any future use. Thus, I will close this issue. If you don't agree with this or have any other comments, please tell me.


Fix engagements filters in 'engagements by product view'

It's the same as #9913 but with the correct branching.
sc-[5299]
Description
The engagement filters inside the "Engagements By Product" view are not working properly. Searching by the exact name in the "Engagement name" filter returns incorrect results.
The Problem
Building relationship filters as we currently do in EngagementFilter generates a query that gets filtered products with all the related engagements, where at least one engagement satisfies the filter criteria.  The engagements_all view iterates over the engagements of each product to render the table rows. As a result, we showed all product engagements that satisfied the filter criteria. For example, if two engagements fulfill the filter criteria from two different products, the rendered table will show all engagements that belong to those two products.
Solution
When prefetching the engament_set, an additional filter will filter the engagements at the query level, so only filtered engagements related to filtered products will be obtained.
engagement_query = Engagement.objects.annotate(test_count=Count('test__id'))
filter_qs = products_with_engagements.prefetch_related(
   Prefetch('engagement_set', queryset=ProductEngagementsFilter(request.GET, engagement_query).qs)
)

Also, the paginator object had to be modified to represent the correct number of rows currently shown on the page.
prods.paginator.count = sum(len(prod.engagement_set.all()) for prod in prods)

[sc-5299]


Fix DDL creation of SQLite3 table with date/time column.

When user-defined C++ types are used, an exception of type std::bad_cast in type_conversion::from_base function is obtained when a std::tm type is used.
If the row type is declared as DATETIME, the error disappears.
I have only tested the case with SQLite3 backend. The version of SQLite used is  3.22.0.
I have attached a sample code which ilustrates the problem:
sample-sqlite-date.cpp.txt

According to SQLite docs there is no datetime in it and using it just specifies "numeric" affinity for the column, so perhaps it would be better/more clear to use numeric directly here?
It would also be very nice to add a unit test which fails now and is fixed by applying this PR.

Thank you for your contribution indeed. I'm afraid it won't be accepted without a test case targeting the issue you discovered and verifying your fix.

SQLite3 hasn't got built-in support for dates or times types. As you say, the SQLite docs specify that the types DATE, TIME and DATETIME has a NUMERIC type affinity. My first attempt was to use the NUMERIC type instead of INTEGER where I place the patch (file  include/soci/sqlite3/soci-sqlite3.h line 324), but the case failed too.
The SOCI backend for SQLite3 uses a hack to work with dt_date type. If you inspect the funtion get_data_type_map (file src/backends/sqlite3/statement.cpp line 409), it defines the possible types (the type asigned to a column when the table is created) that will be recognised as dt_date. They are "date", "time" and "datetime", so I suppose that any of the three fixes the problem. If you define the table column as NUMERIC, it will be recognised as dt_double.
I attached a sample code in my first message (file sample-sqlite-date.cpp.txt) that fails without the patch and works with it. Do you need to include it in the tests contained in the file tests/sqlite3/test-sqlite3.cpp?

I attached a sample code in my first message (file sample-sqlite-date.cpp.txt) that fails without the patch and works with it. Do you need to include it in the tests contained in the file tests/sqlite3/test-sqlite3.cpp?

Yes, please. This will not only make sure that the bug is fixed now, but also that it remains fixed. TIA!

It looks like this has been superseded by #643, so closing -- please reopen if I misunderstood.


waveshare213v2: Documentation, single-byte commands, always configure RAM area

A collection of three commits improving the waveshare213v2 display driver.

Codecov Report

Merging #36 (bc7ff9b) into main (5b35a3f) will decrease coverage by 0.2%.
The diff coverage is 0.0%.

@@           Coverage Diff           @@
##            main     #36     +/-   ##
=======================================
- Coverage   61.6%   61.4%   -0.2%
=======================================
  Files         53      53
  Lines       6664    6681     +17
=======================================
- Hits        4105    4100      -5
- Misses      2418    2440     +22
  Partials     141     141

Impacted Files
Coverage Δ

waveshare2in13v2/waveshare213v2.go
0.7% <0.0%> (-<0.1%)
:arrow_down:

mcp9808/mcp9808.go
98.8% <0.0%> (-1.2%)
:arrow_down:

lepton/lepton.go
86.2% <0.0%> (-1.0%)
:arrow_down:

Continue to review full report at Codecov.

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


Component never seems to load (react + typescript)

We have an edit functionality in our app. It is only rarely used and controlled by a BookEditor. I want to code split to load the editor later.
webpack config (needs the public path and chunkFilename to locate the script):
...
output: {
  filename: '[name].js',
  path: PATHS.build,
  publicPath: '/',
  chunkFilename: '[id].chunk.js'
}

The chunk gets created as 0.chunk.js.
The loadable component. BookEditorLoadable:
import * as React from 'react';

import Loadable from 'react-loadable';

// typescript needs this to use import()
declare global {
    interface System {
        import(request: string): Promise<any>
    }
    var System: System
}

interface Props {
    isLoading: boolean,
    error: Error | null,
    pastDelay: null,
};

let MyLoadingComponent = ({ isLoading, error, pastDelay }: Props) => {
    console.log('Loading component: ', isLoading, error, pastDelay);
    if (isLoading) {
        return pastDelay ?
Loading...
 : null; // Don't flash "Loading..." when we don't need to.
    } if (error) {
        return
Error! Component failed to load
;
    } else {
        return null;
    }
};

const BookEditorLoadable = Loadable({
    loader: () => System.import('./BookEditor'),
    LoadingComponent: MyLoadingComponent,
    delay: 100
});

export default BookEditorLoadable;

And it is used like this in the BookEditorContainer:
class BookEditorCont extends React.Component<any, any>{

    render() {
        return (
            < BookEditorLoadable
                editDisplayPosition={this.props.editDisplayPosition}
                editorAnimationOn={this.props.editorAnimationOn}
                editPart={this.props.editPart}
            />
        )
    }
}

The trace statement in the loading component  triggers Loading component:  true null false  but the component never materializes. Here is a snapshot of the React Chrome dev tools.

Any ideas?
Thank you.

I'm not sure, maybe look into your network tab in devtools to see the requests being made. Also might want to try inspecting what System.import is doing in the generated code by stepping through it in the debugger. As long as it's returning a promise that will eventually resolve or reject, react-loadable should be doing its thing just fine.

I'm going to close this because otherwise these issues have a tendency to stay open forever. If you don't figure it out, let me know.

The script downloads and the promise seems to resolve.

Oh, it's probably that it's not the default export. Right now react-loadable only handles that

That was it. Thank you! Works great.
One last question - is there a callback for when the component has finished loading?

Other than the component itself loading? What do you need it for?

Good point. I need to apply some css transitions to the component's parent once it is available and only execute once I am sure to have it.

If you want to open an issue with a detailed example of what you're trying to do I can look into it

Thank you James. For now it all works well by preloading the component right after the initial app bundle. I think that I can work with componentDidMount in case I run into a need for more complicated timing. This is already a huge benefit.


Update play store & google play buttons

Redirect to new version of Gentlestudent app

Done in https://github.com/oSoc19/gentlestudent-web/commit/51c2d0c86144cc61ccd2e9121516848f4e45ebdc


minor fixups

Just was going through your code for my university project so wanted to contribute a bit.

nvm , im dumb


Update to tools

HI
I propose 2 tools .
pleroma
discourse
Pierre

Thanks, @pirboazo. I've added these to the big list on https://github.com/mattcone/markdown-guide/wiki/Markdown-tool-directory — will get to these when I have time.


Add -a <APIKEY to mbed dm update sub command

Problem
There is currently no way to programatically set the API key to use in the mbed dm update command. There is the -a option in the mbed dm init command, but not in the update command. This is both inconsistent and super annoying to use when trying to script with mbed-cli.
Soution
Add the -a <APIKEY sub command with the same format that is used on mbed dm init -a.

Internal Jira reference: https://jira.arm.com/browse/MBOCUSTRIA-311

I just checked the following commands:
mbed dm init -d "herp.derp" --model-name "12345" -a <api key> -f -q
mbed dm update device -D <device id> -p BUILD\mbed-cloud-client-example_update.bin -a <api key>

Both seemed to execute fine (or at least, made it to the update step, I didn't actually have a device setup and ready). Can you confirm the issue?

@bridadan - has this been added since I filed this issue? Also did you make sure it wasnt pulling in the key from your environment?

has this been added since I filed this issue?

I checked through the history of all the tools involved, but it looks like this has pretty much always been present (might be missing something).

Also did you make sure it wasnt pulling in the key from your environment

Yup, it's not in my local or global config.
Could you try reproducing the issue? I was working with the latest Mbed CLI.

if it works them im happy.


provisioning raspi accessory not successful with chip-tool on mac

With ToT (7283e3a28549351909f6b0dedba79fde8be302a0) provisioning raspi accessory not successful with chip-tool on mac, chip-tool fails to establish CASE session after resolving the nodeID. However the IP address resolved in the resolve API is reachable.
Steps to reproduce

build all-clusters-app on raspi with ToT (7283e3a28549351909f6b0dedba79fde8be302a0) using 'scripts/examples/gn_build_example.sh examples/all-clusters-app/linux/ out/all-clusters-app'
launch the app using 'sudo ./out/all-clusters-app/chip-all-clusters-app --wifi'
build chip-tool controller using 'gn gen out/debug --args='chip_mdns="platform"'  and 'ninja -C out/debug
'
provision the app with chip-tool using 'sudo ./out/debug/chip-tool pairing ble-wifi 1234   20202021 3840
'
Observed controller is not able to establish CASE session. However mac is able to ping the IP address resolved in the resolve API.

% ping6 fde2:1151:8029:0:dea6:32ff:fe8d:7073
PING6(56=40+8+8 bytes) fde2:1151:8029:0:a006:53db:9180:9da8 --> fde2:1151:8029:0:dea6:32ff:fe8d:7073
16 bytes from fde2:1151:8029:0:dea6:32ff:fe8d:7073, icmp_seq=0 hlim=64 time=130.187 ms
16 bytes from fde2:1151:8029:0:dea6:32ff:fe8d:7073, icmp_seq=1 hlim=64 time=4.039 ms
16 bytes from fde2:1151:8029:0:dea6:32ff:fe8d:7073, icmp_seq=2 hlim=64 time=4.928 ms
^C
Attached logs
raspi-log.txt
chip-tool-log.txt

I am able to get this working in the code introduced here: https://github.com/project-chip/connectedhomeip/pull/13294.
However, this requires a workaround to https://github.com/project-chip/connectedhomeip/issues/13274.
The workaround is:
diff --git a/src/commissioner/Discoverer.h b/src/commissioner/Discoverer.h
index d3ea42f29..f5a9a9ee2 100644
--- a/src/commissioner/Discoverer.h
+++ b/src/commissioner/Discoverer.h
@@ -90,6 +90,26 @@ public:
     CHIP_ERROR GetNextDnssdCandidate(Commissionee & commissionee);

 private:
+
+void OnNodeDiscoveryComplete(const chip::Dnssd::DiscoveredNodeData & nodeData)
+{
+    auto discoveredNodes = GetDiscoveredNodes();
+    // Node not yet in the list
+    for (auto & discoveredNode : discoveredNodes)
+    {
+        if (!discoveredNode.IsValid())
+        {
+            discoveredNode = nodeData;
+            if (mDeviceDiscoveryDelegate != nullptr)
+            {
+                mDeviceDiscoveryDelegate->OnDiscoveredDevice(nodeData);
+            }
+            return;
+        }
+    }
+    ChipLogError(Discovery, "Failed to add discovered node with hostname %s- Insufficient space", nodeData.hostName);
+}
+
     DiscoveredNodeList GetDiscoveredNodes() { return DiscoveredNodeList(mNodes); }
     void OnDiscoveredDevice(const chip::Dnssd::DiscoveredNodeData & node) { (*mDelegate)->OnDiscovery(); }
     void OnNodeIdResolved(const Dnssd::ResolvedNodeData & nodeData) {}

@cjandhyala Is this still broken? I'm confused how tests are passing if so...

What I had encountered was that I had some sort of vlan or virtual adapter that caused me to receive advertisements with link-local or ULA addresses that weren't reachable alongside other advertised address from the same node that were reachable.
The existing commissioning code picks the first address it sees.  If this address is not reachable, commissioning fails.   My solution to this was complex.  https://github.com/project-chip/connectedhomeip/pull/13294 is a draft PR with an alternative commissioner implementation that, among other things, uses browse to collect many candidate addresses and will try PASE with as many as it can within timeout constraints.  This + a workaround for https://github.com/project-chip/connectedhomeip/issues/13274 allowed me to still commission in a case where I was seeing multiple addresses, and only some were actually reachable.
I do not know for sure whether the issue encountered here is the same.  But I do know that if our commissioning code sees multiple advertised addresses and not all are routable for the recipient, commissioning will likely fail.


Updating Branch to use NS_ENUMs to represent enum constants.

@derrickstaten @qinweigong @aaustin @Sarkar
Also prefixing all enums with Branch. This will be temporarily backwards compatible.

Why is it temporarily backwards compatible? It still just takes an integer, right?

as long as it doesn't make people go in and change code :+1:

@aaustin the enums are effectively just constants, yes, but if the developer holds a local reference to any of them (we do so in the TestBed App here for example), that reference would break. I would like to remove this in a future version, perhaps with the callbacks Branch (which I see as a "major" release, since it's a larger rework of the SDK).


Correctly pluralize "bytes", "patches", etc

Perhaps this could be factored out into a struct Plural(&'static str, usize); ...and then println!("{len} patch{}", Plural("es", len)).

This would require casting a couple of sites, e.g. Plural("s", len as usize). It would also not be a fully-general pluralization support, e.g. println!("{} loa{}", len, if len == 1 { "f" } else { "ves" }) wouldn't work. So I think it's on the borderline of being too abstract for the given problem.


Update "seal" page

Automatically generated by Netlify CMS

Testing to see why the image library sometimes shows images. It appears there might be a bug where "draft" pages that are not yet published don't show images from the media library.


Downloading nifti files not working

I'm running this piece of code in the hanganu notebook but it's not working.
From my conversation with Andrzej, it should have logged in the Ida website and downloaded the files. However, no files are downloaded.
I'm using symlink=True to get the filenames.

Hi @niyonx, could you send the package version for livingpark_utils and ppmi_downloader that produce this issue?

I have
livingpark_utils==0.9
ppmi_downloader==0.7.2

It's likely a project-specific behavior.
I have the same versions in a different project and it runs but I get the same error as #61


Create btn-small and btn-mini

Smaller button sizes needed in many different scenarios.
Maybe these buttons could look like this.

This is my version of btn-small :
.btn-small {
	height: 24px;
	line-height: 24px;
	padding: 0 0.5rem;
}

There aren't any small buttons, this is to conform to material design standards which help enforce a better UX. The spec is like this because buttons that are smaller than the default are harder to press on mobile and if you keep buttons relatively uniform in size, it helps users identify the buttons with ease (The same reason text in buttons is uppercase)

Buttons don't need to be that big to create a better UX. Smaller buttons are still clickable and don't take so much space. Bootstrap is an example on how different button sizes can be effective for some situations. Imagine to create a simple text editor like this I'm using now with just Materialize buttons...
IMHO, this can be an opportunity for improving this framework, in order to provide better components than the current "standard".

its possible to add a condition of screen size...

Please re-open this issue. Smaller buttons are needed in many scenarios.

Cool

@thiagodp Would you consider forking this and adding the support for this and other bootstrap features.

@rizplate Sorry, but I don't have time now.

@Dogfalo Thanks for adding these. I see the reference in the docs, but it's not actually implemented in the alpha 3 downloads, so it is a bit misleading.

This is my button small and mini, they also work with other Materialize CSS classes for button and text.
.btn-small {
color: #FFF;
text-align: center;
background-color: #F29023;
border: none;
border-radius: 2px;
line-height: 20px;
padding: 5px 7px;
text-transform: uppercase;
vertical-align: middle;
-webkit-tap-highlight-color: transparent;
background-clip: padding-box;
}
.btn-small :hover :active {
background-color: #EF6C00;
}
.btn-mini {
color: #FFF;
font-size: 0.85rem;
text-align: center;
background-color: #F29023;
border: none;
border-radius: 2px;
line-height: 13px;
padding: 5px;
text-transform: uppercase;
vertical-align: middle;
-webkit-tap-highlight-color: transparent;
background-clip: padding-box;
}
.btn-mini :hover :active {
background-color: #EF6C00;
}


Explanation on the approach Microsoft took when migrating from centralized version control to Git

Hi, can you link or include some context for this comment:
"This is the approach that Microsoft took when it migrated Windows and other products from centralized version control to Git."
It would help guide others on the journey from TFVC to Git to understand why history was not preserved - is it simply untenable from a performance perspective, or were there other considerations?
Thanks!

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

ID: 167c73b2-d408-8107-31ed-8e588c390533
Version Independent ID: c381c82b-30ce-43ac-dd93-2dd7a7deea1a
Content: Import repositories from TFVC to Git - Azure Repos
Content Source: docs/repos/git/import-from-TFVC.md
Service: azure-devops-repos
Sub-service: azure-devops-repos-git
GitHub Login: @vijayma
Microsoft Alias: vijayma

@nahumtimerman -- thank you for your feedback and question.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@vijayma -- please look into this issue.