When enabaling emailasUsername in API Manager<EMAIL_ADDRESS>gives errors.

Description:
As per doc[1] and issue[2], once emailasUsername is enabled in IS, it has to be enabled in API manager as well.  When we follow the same steps in APIM as in doc[3], following error is encountered.
ERROR - DataEndpointConnectionWorker Error while trying to login to data receiver :/<IP_ADDRESS>:9711
org.wso2.carbon.databridge.agent.exception.DataEndpointAuthenticationException: Error while trying to login to data receiver :/<IP_ADDRESS>:9711
	at org.wso2.carbon.databridge.agent.endpoint.binary.BinaryDataEndpoint.login(BinaryDataEndpoint.java:47)
	at org.wso2.carbon.databridge.agent.endpoint.DataEndpointConnectionWorker.connect(DataEndpointConnectionWorker.java:93)
	at org.wso2.carbon.databridge.agent.endpoint.DataEndpointConnectionWorker.run(DataEndpointConnectionWorker.java:42)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: org.wso2.carbon.databridge.commons.exception.AuthenticationException: java.lang.NullPointerException
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at org.wso2.carbon.databridge.agent.endpoint.binary.BinaryEventSender.processResponse(BinaryEventSender.java:162)
	at org.wso2.carbon.databridge.agent.endpoint.binary.BinaryDataEndpoint.login(BinaryDataEndpoint.java:42)
	... 7 more
[2018-01-25 14:21:28,751] ERROR - DataEndpointConnectionWorker Error while trying to connect to the endpoint. Cannot borrow client for ssl://<IP_ADDRESS>:9711
org.wso2.carbon.databridge.agent.exception.DataEndpointAuthenticationException: Cannot borrow client for ssl://<IP_ADDRESS>:9711
	at org.wso2.carbon.databridge.agent.endpoint.DataEndpointConnectionWorker.connect(DataEndpointConnectionWorker.java:99)
	at org.wso2.carbon.databridge.agent.endpoint.DataEndpointConnectionWorker.run(DataEndpointConnectionWorker.java:42)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: org.wso2.carbon.databridge.agent.exception.DataEndpointAuthenticationException: Error while trying to login to data receiver :/<IP_ADDRESS>:9711
	at org.wso2.carbon.databridge.agent.endpoint.binary.BinaryDataEndpoint.login(BinaryDataEndpoint.java:47)
	at org.wso2.carbon.databridge.agent.endpoint.DataEndpointConnectionWorker.connect(DataEndpointConnectionWorker.java:93)
	... 6 more
Caused by: org.wso2.carbon.databridge.commons.exception.AuthenticationException: java.lang.NullPointerException
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at org.wso2.carbon.databridge.agent.endpoint.binary.BinaryEventSender.processResponse(BinaryEventSender.java:162)
	at org.wso2.carbon.databridge.agent.endpoint.binary.BinaryDataEndpoint.login(BinaryDataEndpoint.java:42)

But once the configuration is set as below in APIM, no errors are encountered. But in IS, it has been set to<EMAIL_ADDRESS>            <AdminUser>
                <UserName>admin<EMAIL_ADDRESS>                <Password>admin</Password>
            </AdminUser>

Shouldn't be API Manager user-mgt.xml and IS user-mgt.xml containing the same configs when enabling emailasusername?
If the actual work around is using admin<EMAIL_ADDRESS>as the username, please specify it in the doc.

Suggested Labels:
Type/Docs
Type/Bug
Priority/High
Severity/Major
Suggested Assignees:
N/A
Affected Product Version:
apim-2.1.0 update 7
am2xx docs
OS, DB, other environment details and versions:
OS: macOS high sierra
Product: api-2.1.0 update 7 / IS 5.4.0
Steps to reproduce:

Follow the steps in doc[3] in IS and start the Identity Server.
Follow the steps in doc[3] in APIM and start the APIM Server.

Expected: APIM should be started.
Actual: Above mentioned error occurs.
[1] - Log in to the API Store using Social Media
[2] - https://github.com/wso2/product-apim/issues/2615
[3] - Using Email Address as the Username
Related Issues:
https://github.com/wso2/product-apim/issues/2615

@bhathiya & @chaminda can you please let us know the resolution on this Git issue.
Should we use the following in the IS and APIM configurations as suggested in this Git issue?
           <AdminUser>
                <UserName>admin<EMAIL_ADDRESS>                <Password>admin</Password>
            </AdminUser>

@Mariangela This is not a doc issue. We need to test and see if it's a bug. I'll change the label,

@Mariangela @ruthryi This issue is reproducible according to the steps given in the documentation.
We need to follow the steps in [1] under "Setting up an e-mail login". So it is better to link [1] in [2]
[1]https://docs.wso2.com/display/AM210/Maintaining+Logins+and+Passwords#MaintainingLoginsandPasswords-LogininviamultipleuserattributesinAPIStore
[2] https://docs.wso2.com/display/AM2xx/Log+in+to+the+API+Store+using+Social+Media#LogintotheAPIStoreusingSocialMedia-ConfigureFacebookloginwithIdentityServer

Added the Setting up an e-mail login section to [1] and [2].
Linked of to the corresponding versions in [3] and [4].
[1] https://docs.wso2.com/display/AM2xx/Maintaining+Logins+and+Passwords#MaintainingLoginsandPasswords-Settingupane-mailloginemaillogin
[2] https://docs.wso2.com/display/AM220/Maintaining+Logins+and+Passwords#MaintainingLoginsandPasswords-Settingupane-mailloginemaillogin
[3] https://docs.wso2.com/display/AM2xx/Log+in+to+the+API+Store+using+Social+Media#LogintotheAPIStoreusingSocialMedia-ConfigureserviceproviderstothePublisherandStorewiththeFacebookIdentityProvider
[4] https://docs.wso2.com/display/AM220/Log+in+to+the+API+Store+using+Social+Media#LogintotheAPIStoreusingSocialMedia-ConfigureserviceproviderstothePublisherandStorewiththeFacebookIdentityProvider


find a better name for lastTime

But I had no good idea.
Any suggestion?

Maybe period?
That kind of makes sense
The job is gathering updates from last 24 hrs period


Move out the discord commands

They are clearly out of scope for a factorio speedrun bot.

Implemented by 094428ba821b37c4c709d2ed19a91b60c3e56bfa


Padding with zeroes

Hi,
Maybe you should add padding with zeroes option? It would be more like a real world odometer.

Yes, even putting a string like '0033' doesn't work. This is definitely an important feature!

agreed, I'd be interested in this feature too.

yes please! leading zeros are a MUST
:)

Inside the odometer javascript file, look for the render() function.  At the end of the function you'll see:
  for (_j = 0, _len1 = _ref.length; _j < _len1; _j++) {
    digit = _ref[_j];
    if (digit === '.') {
      wholePart = true;
    }
    this.addDigit(digit, wholePart);
  }

  return this.startWatchingMutations();

Between the end of the for loop and the return statement, add the following code.  Replace MIN_INTEGER_LEN with whatever you want as the minimum length.
  if (MIN_INTEGER_LEN > this.digits.length)
  {
	var initialLength;
	initialLength = this.digits.length;
	for (_j = 0; _j < MIN_INTEGER_LEN - initialLength; _j++)
	{
		this.addDigit(0,true);
	}
  }

For example, if you use MIN_INTEGER_LENGTH = 8 and your odometer is 12,345 then you'll display 00,012,345.  The stylesheets still work (the project has a couple to choose between).

@spiceygas this only works if the numbers are not changing, when update is called and the numbers begins to transition the added 0's go away. Any idea how to fix this?

See https://github.com/HubSpot/odometer/issues/106 for a workaround.

@KoltonG I'm not sure.  I don't have that problem.

Depending on your case, you could adjust the number you'll feed into the odometer before hand. jsBin example


Add instructions to set up pre-commit

We need to add instructions to set up pre-commit, so that autofixes by the bot and other surprised are minimised.

@fepegar what's the status on this? It's related to #474 can we close it?


Reset paging

Is there any mehtod to reset paging to default first page programaticlly

I had the same query a few years back.
Refer this issue.
https://github.com/michaelbromley/angularUtils/issues/61

Thanks for your solution..Its working for me


Adapt fields of RegularTransactionEssence in Node.js

#fixes https://github.com/iotaledger/iota-sdk/issues/918

@Thoralf-M wait, creationSlot is actually optional in the RegularTransactionEssenceBuilder. shouldn't we keep it optional in TS so it uses the current time?

It's only optional in the builder, but not in the transaction essence itself


Trying to Use Locon as Lora - Attribute Error String

I'm getting this error every time I use a locon placed in the lora folder. I could be doing something wrong but the readme doesn't really say much about how to use the extension or anything about the installation. Just that you can use a locon as a lora after installing from the URL. I used your Lycoris extension instead and pretty sure the locon was applied then and I didn't get an error but something similar to what AddNet displays when you use lora's through it. Thanks for any response that may help me understand more. I've self-taught myself everything up til now but just couldn't figure this error out.
locon load lora method
activating extra network lora with arguments [<modules.extra_networks.ExtraNetworkParams object at 0x00000253FF23E560>]: AttributeError
Traceback (most recent call last):
File "C:\Users\Joe\stable-diffusion-webui\modules\extra_networks.py", line 75, in activate
extra_network.activate(p, extra_network_args)
File "C:\Users\Joe\stable-diffusion-webui\extensions-builtin\Lora\extra_networks_lora.py", line 23, in activate
lora.load_loras(names, multipliers)
File "C:\Users\Joe\stable-diffusion-webui\extensions-builtin\Lora\lora.py", line 170, in load_loras
lora = load_lora(name, lora_on_disk.filename)
File "C:\Users\Joe\stable-diffusion-webui\extensions\a1111-sd-webui-locon\scripts\main.py", line 371, in load_lora
lora.mtime = os.path.getmtime(lora_on_disk.filename)
AttributeError: 'str' object has no attribute 'filename'

I have solved this issue by referring to the solution in the link https://github.com/KohakuBlueleaf/a1111-sd-webui-locon/issues/40 , which is as follows:
1.Go to the directory \stable-diffusion-webui\extensions\a1111-sd-webui-locon\scripts.
2.Edit the main.py file.
3.Modify the following code:
line 371: lora.mtime = os.path.getmtime(lora_on_disk)
line 373: sd = sd_models.read_state_dict(lora_on_disk)
Hope this helps you solve the problem!


Does safetensors save the model's hierarchical structure? Is it similar to ONNX?

If safetensors saves the model's hierarchical structure, how can one access this structure? Is it possible to read it directly like with ONNX?Can I directly load a model from safetensors?
If the hierarchical structure of the model is not preserved, does it mean that the original model must be read from config.json?

No it does not, safetensors saves only tensors, not the associated graph structure to do inference on it.


Save error

Hello,
I'm new user of ConstrcutiveGeometry.
WhanI run the sphre_cube example, there is the following error with save.
What am I doing wrong.
Thank you for your support
Thierry
julia> save("sphere_cube.png", u)
Errors encountered while save File{DataFormat{:PNG}, String}("sphere_cube.png").
All errors:
MethodError: no method matching save(::File{DataFormat{:PNG}, String}, ::ConstructiveGeometry.ConstructedSolid{:union, Vector{ConstructiveGeometry.AbstractGeometry{3}}, 3})

The actual plotting is done by CairoMakie, so this is how it should be done:
using ConstructiveGeometry, FileIO, CairoMakie
save("/tmp/a.png", sphere(10))

(I guess I could add an error message when no Makie backend is loaded, but currently I don't know how to detect this).

Thank you for your answer,
here is the error message for save (using julia 1.7 on windows 10)


[BUG]: You can ignore Jenkins files

What happened?

Epitech's norminette doesn't report "**/Jenkinsfile" so you can ignore it !
Version
Yes
What OS are you seeing the problem on?
Debian based
Relevant output
No response

Hello @MathiDEV,
I don't understand your issue as Abricot is already not reporting the Jenkinsfile, neither do the moulinette.
May you precise what is the issue ?
Thanks

Never mind, found out that is being reported as an O4 whereas I thought it will be reported as an O1.
Therefore, I'm marking this issue as verified and I assign one of the maintainers.
Thanks for your report :)

Hi,
I know Epitech's nominette doesn't report Jenkinsfiles. However, it is clearly marked in the coding style that it should.

Have a good day.


Ignite .NET SDK released

Description
Built SDK for swagger September branch for Azure NW

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please add REST spec PR link to the SDK PR
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

Is this still 19.0.2 right?
https://github.com/Azure/azure-sdk-for-net/compare/psSdkJson6...MikhailTryakhov:local-september#diff-6bab6934b59abbd5fd615ec26e5f6b6dR10

@dsgouda  [assembly: AssemblyVersion("<IP_ADDRESS>")] is still the same right?


Add support for aliases / tokens (all components)

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
When implementing the theme from a design system, components, spacing etc are often described with tokes like M, XL ... etc.
It would be great, if there would be an easy way to add a token / alias to a component, for example the typography h3 with M, or use spacing tokens instead of numeric values.
My current workaround for headlines would be to add additional variants for the according headlines.
For spacing, I tried to solve this with a custom spacing function that is able to resolve the tokens, but unfortunately the custom spacing function is only called, when a numeric value is set to a css property like padding or margin
Examples 🌈
it would be great to do something like this
<Grid sx={{
  marginTop: 'xl'
  padding: 's'
  gap: { xs: 's', md: 'm' }
}}>

Motivation 🔦
I'm currently implementing a design system in MUI. Being able to define and use tokens would be great.

Interesting, so far we only see the need of applying integers there. Have you thought of maybe adding additional tokens in the theme that can basically contain these values, like:
<Grid sx={ theme => ({
  marginTop: theme.sizing.xl,  // can really be anything as long as it is mapped to some int value in the theme
  padding: 's'
  gap: { xs: 's', md: 'm' }
})}>

Or simply using const for this, maybe something like:
<Grid sx={ theme => ({
  marginTop: Sizes.xl,
  padding: 's'
  gap: { xs: Sizes.s, md: Sizes.m }
})}>

This is a good example of how people may want to extend the style function used for the sx prop. I will look into this in more details.

Or simply using const for this, maybe something like:
<Grid sx={ theme => ({
  marginTop: Sizes.xl,
  padding: 's'
  gap: { xs: Sizes.s, md: Sizes.m }
})}>

@mnajdova thank you for your response.  Of course I didn't think of the simple solution 😵💫

I don't feel like this is efficient. Having to add more imports for constants sounds like bad practice. Having a place to define tokens and then use them as xs, sm, etc.. would be great. NativeBase supports this behavior and I'm trying to create something similar for a design system.

I don't feel like this is efficient. Having to add more imports for constants sounds like bad practice. Having a place to define tokens and then use them as xs, sm, etc.. would be great. NativeBase supports this behavior and I'm trying to create something similar for a design system.

You can add additional tokens in the theme, that’s the first example in the comment above. Is it not sufficient?

It solves the problem but feels odd having to call up theme every time I want a custom value when I can already do padding: 'sm' elsewhere yea? Not the worst thing but it makes some items feel inconsistent.
Would be great if we could do this:
<Grid sx={ theme => ({
  marginTop: 'xl',
  padding: 'sm'
  gap: { xs: 'sm', md: 'sm' }
})}>


[1.16.5 BUG] Microwave Receiver randomly not getting power

[Please fill out the form below and delete the sections in square brackets after reading them]
Version of Advanced Rocketry
2.0.0-12
Have you verified this is an issue in the latest unstable build
Y
Version of LibVulpes
1.0.2-25
Version of Minecraft
1.16.5
Does this occur without other mods installed
Y - Tested by cloning my modpack server and removing all mods bar Advanced Rocketry & LibVulpes
Description of the problem
I am unsure if this is reproduceable or how this has happened but basically I have had 12 Microwave energy satellites inside a Microwave receiver. They have all stopped sending power and the Microwave receiver says 0 RF/t

Still present from what I am aware with the latest 1.16.


possible bug fix for #41

This worked for me, but this is my first day using the API - this may not be the best way to go.

Thanks for this fix. It was a good fix but this file is a copy of one from a different repository. I used that change instead.


Basic standard library support.

This is not intended to be useful to anyone. If people want to try it, that's great, but do not rely on this. This is only for experimenting and setting up for future work.
This adds a flag -Zbuild-std to build the standard library with a project. The flag can also take optional comma-separated crate names, like -Zbuild-std=core. Default is std,core,panic_unwind,compiler_builtins.
Closes rust-lang/wg-cargo-std-aware#10.
Note: I can probably break some of the refactoring into smaller PRs if necessary.
Overview
The general concept here is to use two resolvers, and to combine everything in the Unit graph. There are a number of changes to support this:

A synthetic workspace for the standard library is created to set up the patches and members correctly.
Decouple unit_dependencies from Context to make it easier to manage.
Add features to Unit to keep it unique and to remove the need to query a resolver.
Add a UnitDep struct which encodes the edges between Units. This removes the need to query a resolver for extern_crate_name and public.
Remove Resolver from BuildContext to avoid any confusion and to keep the complexity focused in unit_dependencies.
Remove Links from Context since it used the resolver. Adjusted so that instead of checking links at runtime, they are all checked at once in the beginning. Note that it does not check links for the standard lib, but it should be safe? I think compiler-rt is the only links?

I currently went with a strategy of linking the standard library dependencies using --extern (instead of --sysroot or -L). This has some benefits but some significant drawbacks. See below for some questions.
For future PRs

Add Cargo.toml support. See https://github.com/rust-lang/wg-cargo-std-aware/issues/5
Source is not downloaded. It assumes you have run rustup component add rust-src. See https://github.com/rust-lang/wg-cargo-std-aware/issues/11
cargo metadata does not include any information about std. I don't know how this should work.
cargo clean is not std-aware.
cargo fetch does not fetch std dependencies.
cargo vendor does not vendor std dependencies.
cargo pkgid is not std-aware.
--target is required on the command-line. This should default to host-as-target.
-p is not std aware.
A synthetic Cargo.toml workspace is created which has to know about things like rustc-std-workspace-core. Perhaps rust-lang/rust should publish the source with this Cargo.toml already created?
compiler_builtins uses default features (pure Rust implementation, etc.). See https://github.com/rust-lang/wg-cargo-std-aware/issues/15

compiler_builtins may need to be built without debug assertions, see this. Could maybe use profile overrides.

Panic issues:

panic_abort is not yet supported, though it should probably be easy. It could maybe look at the profile to determine which panic implementation to use? This requires more hard-coding in Cargo to know about rustc implementation details.
This should probably be handled where panic is set for panic_abort and compiler_builtins. I would like to get a test case for it. This can maybe be done with profile overrides?

Using two resolvers is quite messy and causes a lot of complications. It would be ideal if it could only use one, though that may not be possible for the foreseeable future. See https://github.com/rust-lang/wg-cargo-std-aware/issues/12
Features are hard-coded. See https://github.com/rust-lang/wg-cargo-std-aware/issues/13
Lots of various platform-specific support is not included (musl, wasi, windows-gnu, etc.).
Default backtrace is used with C compiler. See https://github.com/rust-lang/wg-cargo-std-aware/issues/16
Sanitizers are not built. See https://github.com/rust-lang/wg-cargo-std-aware/issues/17
proc_macro has some hacky code to synthesize its dependencies. See https://github.com/rust-lang/wg-cargo-std-aware/issues/18. This may not be necessary if this uses --sysroot instead.
Profile overrides cause weird linker errors.
That is:[profile.dev.overrides.std]
opt-level = 2

Using [profile.dev.overrides."*"] works. I tried fiddling with it, but couldn't figure it out.
We may also want to consider altering the syntax for profile overrides. Having to repeat the same profile for std and core and alloc and everything else would not be ideal.
Context::unit_deps does not handle build overrides, see #7215. FIXED

Questions for this PR

I went with the strategy of using --extern to link the standard lib. This seems to work, and I haven't found any problems, but it seems risky. It also forces Cargo to know about certain implicit dependencies like compiler_builtins and panic_*. The alternative is to create a sysroot and copy all the crates to that directory and pass --sysroot. However, this is complicated by pipelining, which would require special support to copy .rmeta files when they are generated. Let me know if you think I should use a different strategy. I'm on the fence here, and I think using --sysroot may be safer, but adds more complexity.

As an aside, if rustc ever tries to grab a crate from sysroot that was not passed in via --extern, then it results in duplicate lang items. For example, saying extern crate proc_macro; without specifying proc_macro as a dependency. We could prevent rustc from ever trying by passing --sysroot=/nonexistent to prevent it from trying. Or add an equivalent flag to rustc.

How should this be tested? I added a janky integration test, but it has some drawbacks. It requires internet access. It is slow. Since it is slow, it reuses the same target directory for multiple tests which makes it awkward to work with.

What interesting things are there to test?
We may want to disable the test before merging if it seems too annoying to make it the default. It requires rust-src to be downloaded, and takes several minutes to run, and are somewhat platform-dependent.

How to test that it is actually linking the correct standard library? I did tests locally with a modified libcore, but I can't think of a good way to do that in the test suite.
I did not add __CARGO_DEFAULT_LIB_METADATA to the hash. I had a hard time coming up with a test case where it would matter.

My only thought is that it is a problem because libstd includes a dylib, which prevents the hash from being added to the filename. It does cause recompiles when switching between compilers, for example, when it normally wouldn't.
 This can cause issues (see https://github.com/rust-lang/rust/issues/56443).
This should probably change, but I want to better understand it first.

The bin_nostd test needs to link libc on linux, and I'm not sure I understand why. I'm concerned there is something wrong there. libstd does not do that AFAIK.

:umbrella: The latest upstream changes (presumably #7215) made this pull request unmergeable. Please resolve the merge conflicts.

Thanks for taking a look!

Are you figuring that these would become issues on the wg-std-aware Cargo repository after this PR is merged?

Yes. I wanted to hold off in case any of them became non-issues through the course of updating this PR.

I think this is fine for now in general to be clear, but I'd like to see more config codified in rust-lang/rust's Cargo.toml that we can reuse here in Cargo.

Yep. My goal with this was to try to avoid needing to make any changes to rust-lang/rust at first. We can then start rolling out any changes there that will make this easier later.

if libstd is built in unwinding mode it should activate everything necessary to get the panic_unwind runtime working.

Hm, you just made me think of something.  When running cargo test some targets can be built with unwind (like tests) and some may be built with abort (like bins).  That may complicate the panic support.  It would essentially require 2 copies of std in that situation.  I will need to think about that more.

what else is needed for these platforms?

I'm not sure, I didn't even try. I'm just thinking of things like this and this and this.  I'm not sure if that is actually needed, but I suspect there will be some rough edges for some platforms.

Profile overrides cause weird linker errors.

Here: https://gist.github.com/ehuss/6973e2fad5663f10592f71de08b3eff0
This is with [profile.dev.overrides.std] opt-level=2. Adding core and compiler_builtins doesn't help. I did not investigate much.

When running cargo test some targets can be built with unwind (like tests) and some may be built with abort (like bins). That may complicate the panic support. It would essentially require 2 copies of std in that situation.

Oh dear, that's a good point! I think though we'd functionally want to do what libstd does today which is to compile everything with panic=abort, have two runtimes, and then let the final compile choose which it prefers.

I'm just thinking of things like this and this and this.

Ah ok I see what you mean. I do think that those will cause issues and will be good to track! Of course not a blocker for this though

I did not investigate much.

Hm ok, that does indeed look horrifyingly confusing. I'll try to help debug once this goes through as well

Profile overrides cause weird linker errors.

matklad had an interesting idea for a related problem here: https://github.com/rust-lang/rust/issues/63484#issuecomment-521575216
I wonder if it isn't something similar? Or maybe related to re-exports?

Hm so in theory optimization settings should affect symbol visibility to the point of causing linker errors. That's just theory though, and libstd is always a special child, so there's probably something wrong here or there. I would need to dig into it more locally though to figure it out.

FWIW I'm personally feeling pretty good about landing this, so @ehuss I don't mind waiting if you've got blockers you'd prefer to handle first but I think it's otherwise fine to start opening issues and adding comments for what should be removed when and then r+ this

@alexcrichton I think this is ready to go, do you want to take a quick look at the last two commits?  I decided to make libtest only build if it is needed.  I'm not sure if it is completely robust, but I assume any issues can be fixed.
Also, I decided to leave the test enabled for now.  We shouldn't hesitate to mark it #[ignore] if it is too slow or cumbersome. Does that sound ok?

@bors: r+
A+ from me!

:pushpin: Commit b67f3e128b144156b75e1883b428128153947f9e has been approved by alexcrichton

:hourglass: Testing commit b67f3e128b144156b75e1883b428128153947f9e with merge f27175addf6ce6a11e44040bb9cb2a8d222a1687...

:broken_heart: Test failed - checks-azure

Minor fix due to #7310.
@bors r=alexcrichton

:pushpin: Commit 9382be174b1630e997b3d12d2d2fcb2c1d5a2bee has been approved by alexcrichton

:hourglass: Testing commit 9382be174b1630e997b3d12d2d2fcb2c1d5a2bee with merge 49fbf52fd517906a2229ca83d468971c1113b015...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing 49fbf52fd517906a2229ca83d468971c1113b015 to master...

Congrats and thank you!!

Thanks a lot for implementing this! I don't know if this is the right place to share feedback, so please tell me if there is a more appropriate place!
I tried -Z build-std for https://github.com/phil-opp/blog_os with the following results:

cargo build -Z build-std=core,alloc works without problems. I only needed to add a dependency on rlibc because there is currently no way to enable the mem feature of compiler_builtins.
cargo run -Z build-std=core,alloc also worked fine with a custom QEMU runner.
cargo test -Z build-std=core,alloc failed because it tries to build libtest for a bare-metal target. We use a custom test framework, so compiling libtest is not needed for our case.
Specifying the -Z flag explicitly on each invokation is a bit cumbersome. Is there perhaps a way to set it in a .cargo/config file (e.g. similar to the build.rustflags key)?

In summary, I would say that this MVP is already able to replace most uses of cargo-xbuild. The only blocker is the is the missing support for custom test frameworks on bare-metal targets. Thanks again for your work!

cargo test -Z build-std=core,alloc failed because it tries to build libtest for a bare-metal target. We use a custom test framework, so compiling libtest is not needed for our case.

You should be able to set harness=false for all the targets ([lib], [[test]], etc.) and it should skip libtest.

@ehuss Thanks for the suggestion. Setting harness=false disables the test framework completely, so it does not work together with the custom_test_frameworks feature and the test_runner attribute (e.g. used in in this test).


Should dpul-staging point to figgy or figgy-staging?

We should be able to switch back and forth.
Currently switching around leads to a queue build-up on the other rabbitmq installation.


Password-free large scale tidy data access through R API?

Hello, first of all thanks a lot for your effort of pulling together a lot of macroeconomic data for South Africa. I am, however, struggling with using either your R package or the SDMX/Clojure API using get requests (from python, mainly due to my insufficient understanding of such API's and the lack of examples I guess). Regarding the R package, I want to make 3 points:

It requires login using 3rd party software (XQuartz on Mac) on each request. This, in my view, totally nullifies the benefit of an API which is to automate operations without human interference. Furthermore, to require users to install 3rd party software is tedious and not allowed on platforms like CRAN. I also found that the third party software could cause my R session to crash if I typed wrong credentials. So to require an API key which is entered as part of the query, or just making it free, would be much better.

The package does seems to attach other packages alongside (httr, jsonlite, xml2, tcltk), and does not work if not attached, i.e. econdatar::read_econdata() does not work without a prior call to library(econdatar).  This is also not allowed on platforms like CRAN and complicates namespace management for users.

The package does not seem to provide tidy tabular data (https://vita.had.co.nz/papers/tidy-data.pdf)

All of this combined, I have to conclude that for my purposes (nowcasting GDP in South Africa using large amounts of public data), the API is useless. This is really a pitty as the underlying software engineering and your data model appear to be pretty comprehensive and powerful. As someone with a fair amount of experience in creating and using R API packages, I would like to end by providing some inspiration from my own recent work and the work of others.

I organized public data in Uganda (https://mepd.finance.go.ug/apps.html), inluding an R package that provides fast access to large volumes of tidy public data e.g. ugatsdb::get_data("MOF_TOT") gets Monthly total operations of the Government of Uganda from the Ministry of Finance, more info at help("ugatsdb").

I created a platform (https://africamonitor.ifw-kiel.de/) to monitor economic developments in Africa, with similar tidy API's for R and Python, e.g. africamonitor::am_data("ZAF", wide = FALSE) gives me >900 internationally comparable indicators for south africa at different frequencies, more info at help("africamonitor").

The DBnomics (https://db.nomics.world/) project is a massive worldwide open data collection effort with tidy access via R and python packages. E.g. rdbnomics::rdb("SARB", "QB", query = "") gives me the entire Quarterly Bulletin of the South African Reserve Bank (albeit from 2021, as the fetcher programs have not been updated to reflect recent changes in the SARB website).

I of course don't know what your intentions or business model are, but given your data collection efforts and ostensible commitment to open source software, the creation of powerful open APIs along these lines could be a game changer for everyone doing economic research in South Africa.

Hi Sebastian
Just a quick response while I digest the detail of what you have laid out:

It requires login using 3rd party software (XQuartz on Mac) on each request.

My understanding when writing the package was that tcl/tk is cross-platform and should pretty much just work everywhere, this seems to at least be true on Windows. In any case you can get around this by setting the following environment variable ECONDATA_CREDENTIALS="your_username;your_password" in your .Renviron file for example.

The package does seems to attach other packages alongside (httr, jsonlite, xml2, tcltk), and does not work if not attached, i.e. econdatar::read_econdata() does not work without a prior call to library(econdatar)

Excuse my ignorance on this point. What is the fix for this, ensuring that each function calls require() for the necessary dependencies?

The package does not seem to provide tidy tabular data

Indeed. I didn't want to include any more dependencies than necessary, but if it would be useful I can create an econdata2tibble() function?
Thanks for taking the time to give feedback.

On this point

I of course don't know what your intentions or business model are, but given your data collection efforts and ostensible commitment to open source software, the creation of powerful open APIs along these lines could be a game changer for everyone doing economic research in South Africa.

That is the direction we are aiming in, but of course we are a small company and the platform is foremost meant to serve our and our clients' needs, so it is a slow and often tedious process to get the platform up to a standard that meets user expectations.
That said, a number of academics have expressed interest in a standard data set for the purposes of GDP nowcasting along the lines of (AL)FRED's macroeconomic data set. We are seeking funding for such a project. It would be useful to let us know what variables/sources of data you are considering in you research if possible.

I would also like to add that the underlying API is likely to undergo some breaking changes during the course of this year so I would ask that you hold off on using it directly for the time being. While we are looking at simplifying the econdatar interface, I don't expect there will be any breaking changes in the package.

I forgot to mention there is a branch that excludes the tcl/tk package that is maintained for environments that don't have graphical capabilities:
require(remotes); install_github('coderaanalytics/econdatar', ref='docker', INSTALL_opts = c('--no-help', '--no-html'))

Hi Sebastian
Just a quick response while I digest the detail of what you have laid out:

It requires login using 3rd party software (XQuartz on Mac) on each request.

My understanding when writing the package was that tcl/tk is cross-platform and should pretty much just work everywhere, this seems to at least be true on Windows. In any case you can get around this by setting the following environment variable ECONDATA_CREDENTIALS="your_username;your_password" in your .Renviron file for example.

Thanks for the reply. I was unaware of the environment variable option. This makes things better already, but on my system this still starts XQuartz in the background. Ideally you want something that only uses R e.g. requiring an API key that is sent to the server. For example if I would want to put this into production, I would like to install the package on a linux server, with only R pre-installed.

The package does seems to attach other packages alongside (httr, jsonlite, xml2, tcltk), and does not work if not attached, i.e. econdatar::read_econdata() does not work without a prior call to library(econdatar)

Excuse my ignorance on this point. What is the fix for this, ensuring that each function calls require() for the necessary dependencies?

So in general, you should not load or attach other packages at all from package code. The best way is to use importFrom statements in the NAMESPACE file, e.g. importFrom("jsonlite", "fromJSON"). Another option is using jsonlite::fromJSON in the package code, but this incurs the overhead of ::. You can see here for some examples of ImportFrom. In general, R CMD check should warn about this, have you run R CMD check on the package yet?

The package does not seem to provide tidy tabular data

Indeed. I didn't want to include any more dependencies than necessary, but if it would be useful I can create an econdata2tibble() function?
Thanks for taking the time to give feedback.

With tidy data I dont mean a particular object (like data.frame, data.table or tibble), but data that is organized in rows and columns and thus easy to process further. The definition of tidy data is that each variable/characteristic is a column and each observational unit is a row. In other words, tidy APIs return a single data frame like object. How this is best organized depends on the purpose. In my APIs I provide the options of both long and wide data return (+reshaping functions to be called ex-post). DBnomics provides a standardized long output format which can accommodate very heterogeneous data. Some examples:
###########
# Tidy APIs
###########

# Most APIs provide long format data: perfect for mixed frequency and ggplotting
POE_tidy_long <- ugatsdb::get_data("MOF_POE", wide = FALSE) # Performance of the economy dataset
tail(POE_tidy_long)
#>          Date Series                                         Label    Value
#>        <Date> <char>                                        <char>    <num>
#> 1: 2022-07-01 TB_COD Trade Balance with Congo (D.R.) (US$ Million) 54.48846
#> 2: 2022-08-01 TB_COD Trade Balance with Congo (D.R.) (US$ Million) 49.78164
#> 3: 2022-09-01 TB_COD Trade Balance with Congo (D.R.) (US$ Million) 50.94488
#> 4: 2022-10-01 TB_COD Trade Balance with Congo (D.R.) (US$ Million) 46.36710
#> 5: 2022-11-01 TB_COD Trade Balance with Congo (D.R.) (US$ Million) 47.02394
#> 6: 2022-12-01 TB_COD Trade Balance with Congo (D.R.) (US$ Million) 48.52958

# The default in my APIs is wide data: ready for time series analysis
POE_tidy_wide <- ugatsdb::get_data("MOF_POE")
tail(POE_tidy_wide)[, 1:10]
#> Key: <Date>
#>          Date   CPI_16 CPI_CORE_16 CPI_FOOD_16 CPI_EFU_16 CPI_09 CPI_CORE_09 CPI_FOOD_09 CPI_EFU_09    INF_16
#>        <Date>    <num>       <num>       <num>      <num>  <num>       <num>       <num>      <num>     <num>
#> 1: 2022-08-01 123.2445    122.7355    115.4992   141.0762     NA          NA          NA         NA  9.003460
#> 2: 2022-09-01 125.1029    124.2073    121.9511   141.2153     NA          NA          NA         NA  9.993973
#> 3: 2022-10-01 126.1317    125.1876    126.3131   138.0120     NA          NA          NA         NA 10.710683
#> 4: 2022-11-01 126.2145    125.4758    126.1997   135.7387     NA          NA          NA         NA 10.583868
#> 5: 2022-12-01 126.3849    125.7720    126.6209   133.9239     NA          NA          NA         NA 10.232480
#> 6: 2023-01-01 126.1925    125.6436    126.1484   133.3166     NA          NA          NA         NA 10.404955

# This format has variable labels attached (can be displyed in the Rstudio viewer)
collapse::namlab(POE_tidy_wide)[1:10, ]
#>       Variable                                                                                Label
#> 1         Date                                                                                <NA>
#> 2       CPI_16                        Consumer Price Index (CPI), (2016/17 = 100): All Items Index (Weight = 10000)
#> 3  CPI_CORE_16                            Consumer Price Index (CPI), (2016/17 = 100): Core Index (Weight = 8396.2)
#> 4  CPI_FOOD_16    Consumer Price Index (CPI), (2016/17 = 100): Food Crops and Related Items Index (Weight = 951.05)
#> 5   CPI_EFU_16 Consumer Price Index (CPI), (2016/17 = 100): Energy Fuel and Utilities (EFU) Index (Weight = 652.75)
#> 6       CPI_09                         Consumer Price Index (CPI), (2009/10 = 100): All Items Index (Weight = 1000)
#> 7  CPI_CORE_09                                  Consumer Price Index (CPI), (2009/10 = 100): Core (Weight = 823.94)
#> 8  CPI_FOOD_09           Consumer Price Index (CPI), (2009/10 = 100): Food Crops and Related Items (Weight = 101.6)
#> 9   CPI_EFU_09              Consumer Price Index (CPI), (2009/10 = 100): Energy Fuel and Utilities (Weight = 74.46)
#> 10      INF_16                                   Annual (YoY) Inflation (2016/17): All Items Index (Weight = 10000)

# E.g.
attr(POE_tidy_wide$CPI_16, "label")
#> [1] "Consumer Price Index (CPI), (2016/17 = 100): All Items Index (Weight = 10000)"

# Examples with panel data
# Getting growth and inflation for the EAC countries
africamonitor::am_data(ctry = c("UGA", "KEN", "TZA", "RWA", "BDI", "SSD"),
        series = c("NGDP_RPCH", "PCPIPCH"))
#> Key: <ISO3, Date>
#>        ISO3       Date NGDP_RPCH PCPIPCH
#>      <char>     <Date>     <num>   <num>
#>   1:    BDI 1980-01-01    -6.825   1.200
#>   2:    BDI 1981-01-01    12.164  12.167
#>   3:    BDI 1982-01-01    -1.054   5.868
#>   4:    BDI 1983-01-01     3.715   8.151
#>   5:    BDI 1984-01-01     0.155  14.317
#>  ---
#> 252:    UGA 2023-01-01     5.898   6.397
#> 253:    UGA 2024-01-01     5.996   5.707
#> 254:    UGA 2025-01-01     7.500   4.987
#> 255:    UGA 2026-01-01     6.815   5.023
#> 256:    UGA 2027-01-01     6.805   5.000

africamonitor::am_data(ctry = c("UGA", "KEN", "TZA", "RWA", "BDI", "SSD"),
                       series = c("NGDP_RPCH", "PCPIPCH"), wide = FALSE)
#>        ISO3       Date    Series                                                   Label  Value
#>      <char>     <Date>    <char>                                                  <char>  <num>
#>   1:    BDI 1980-01-01 NGDP_RPCH Gross Domestic Product, Constant Prices: Percent Change -6.825
#>   2:    BDI 1981-01-01 NGDP_RPCH Gross Domestic Product, Constant Prices: Percent Change 12.164
#>   3:    BDI 1982-01-01 NGDP_RPCH Gross Domestic Product, Constant Prices: Percent Change -1.054
#>   4:    BDI 1983-01-01 NGDP_RPCH Gross Domestic Product, Constant Prices: Percent Change  3.715
#>   5:    BDI 1984-01-01 NGDP_RPCH Gross Domestic Product, Constant Prices: Percent Change  0.155
#>  ---
#> 508:    UGA 2023-01-01   PCPIPCH      Inflation, Average Consumer Prices: Percent Change  6.397
#> 509:    UGA 2024-01-01   PCPIPCH      Inflation, Average Consumer Prices: Percent Change  5.707
#> 510:    UGA 2025-01-01   PCPIPCH      Inflation, Average Consumer Prices: Percent Change  4.987
#> 511:    UGA 2026-01-01   PCPIPCH      Inflation, Average Consumer Prices: Percent Change  5.023
#> 512:    UGA 2027-01-01   PCPIPCH      Inflation, Average Consumer Prices: Percent Change  5.000

# DBnomics: standardized long format allowing very heterogeneous data
rdbnomics::rdb("WB", "WGI", dimensions = list(country = "ZAF"))
#>      @frequency country dataset_code                    dataset_name frequency          indexed_at  indicator original_period original_value     period provider_code      series_code
#>          <char>  <char>       <char>                          <char>    <char>              <POSc>     <char>          <char>         <char>     <Date>        <char>           <char>
#>   1:     annual     ZAF          WGI Worldwide Governance Indicators         A 2022-11-08 12:49:09     CC.EST            1996    0.732927382 1996-01-01            WB     A-CC.EST-ZAF
#>   2:     annual     ZAF          WGI Worldwide Governance Indicators         A 2022-11-08 12:49:09     CC.EST            1998   0.6388086081 1998-01-01            WB     A-CC.EST-ZAF
#>   3:     annual     ZAF          WGI Worldwide Governance Indicators         A 2022-11-08 12:49:09     CC.EST            2000   0.5502697229 2000-01-01            WB     A-CC.EST-ZAF
#>   4:     annual     ZAF          WGI Worldwide Governance Indicators         A 2022-11-08 12:49:09     CC.EST            2002   0.3329015076 2002-01-01            WB     A-CC.EST-ZAF
#>   5:     annual     ZAF          WGI Worldwide Governance Indicators         A 2022-11-08 12:49:09     CC.EST            2003   0.2755408287 2003-01-01            WB     A-CC.EST-ZAF
#>  ---
#> 824:     annual     ZAF          WGI Worldwide Governance Indicators         A 2022-11-08 12:49:09 VA.STD.ERR            2017    0.119611159 2017-01-01            WB A-VA.STD.ERR-ZAF
#> 825:     annual     ZAF          WGI Worldwide Governance Indicators         A 2022-11-08 12:49:09 VA.STD.ERR            2018   0.1232636794 2018-01-01            WB A-VA.STD.ERR-ZAF
#> 826:     annual     ZAF          WGI Worldwide Governance Indicators         A 2022-11-08 12:49:09 VA.STD.ERR            2019   0.1180535406 2019-01-01            WB A-VA.STD.ERR-ZAF
#> 827:     annual     ZAF          WGI Worldwide Governance Indicators         A 2022-11-08 12:49:09 VA.STD.ERR            2020   0.1205894873 2020-01-01            WB A-VA.STD.ERR-ZAF
#> 828:     annual     ZAF          WGI Worldwide Governance Indicators         A 2022-11-08 12:49:09 VA.STD.ERR            2021   0.1212761179 2021-01-01            WB A-VA.STD.ERR-ZAF
#>                                                           series_name     value
#>                                                                <char>     <num>
#>   1:          Annual – Control of Corruption: Estimate – South Africa 0.7329274
#>   2:          Annual – Control of Corruption: Estimate – South Africa 0.6388086
#>   3:          Annual – Control of Corruption: Estimate – South Africa 0.5502697
#>   4:          Annual – Control of Corruption: Estimate – South Africa 0.3329015
#>   5:          Annual – Control of Corruption: Estimate – South Africa 0.2755408
#>  ---
#> 824: Annual – Voice and Accountability: Standard Error – South Africa 0.1196112
#> 825: Annual – Voice and Accountability: Standard Error – South Africa 0.1232637
#> 826: Annual – Voice and Accountability: Standard Error – South Africa 0.1180535
#> 827: Annual – Voice and Accountability: Standard Error – South Africa 0.1205895
#> 828: Annual – Voice and Accountability: Standard Error – South Africa 0.1212761

Created on 2023-02-28 by the reprex package (v2.0.1)

The issues you have with the tcl/tk package and with econdatar attaching packages should be resolved in release 1.1.6
install_github("coderaanalytics/econdatar", ref = "1.1.6")

You can comment on these issues #7 and #8, else I will assume I can close them.
I will add a feature for read_econdata to produce "tidy" data sometime, hopefully soon.

Thanks a lot, this looks good so far, I am able to verify that in 1.1.6 the package works without being attached. I have no further comments on the other issues of you can resolve them.
A final comment on the API is that it would be good if the API could be self contained i.e. there could be functions to get information about the available data and codes (like the datasources(), datasets() and series() functions in ugatsdb or rdb_providers(), rdb_datasets(), rdb_dimensions() in rdbnomics), so that the data can be explored and retrieved without having to browse your website, and it is very clear from the package documentation how the codes returned by such auxiliary functions are to be used in the main package function (get_data() in ugatsdb, rdb() in rdbnomics, or read_econdata()) in your case. As far as I understand it read_econdata() can be used to query both data and metadata, but this complicates things, and it is already necessary to know something about your data model to even get the metadata. In general, it is better to have a package with a few streamlined functions where it is clear how they are to be used and what they return, than a single function that does everything.
Regading package development, I suggest setting up some form of CI on GitHub soon using usethis::use_github_action("check-standard") (R CMD Check), and usethis::use_github_action("test-coverage") once you started to add tests using the testthat package. Also feel free to reach out to me with other issues/questions. You can close this issue or leave it open as you require.

A final comment on the API is that it would be good if the API could be self contained.

Only a small part of the web API is included in econdatar, I would like a one-to-one mapping, but that development is not that high on the priority list at the moment. I will add it to the list of issues

Regading package development, I suggest setting...

A more sophisticated development workflow will follow, but resources for this particular project are somewhat constrained for the moment.
Thanks for all the feedback, please feel free to post issues as necessary while you are using EconData


PROD-345: deck completion height issue.

Description
Changes no question card height to a fix height 350px with overflow scroll.
Screen shots or recordings for UI changes

@neel-ds can you review

@yash431garg Apply same changes to following files:
https://github.com/gator-labs/chomp/blob/5b302c1e599ab031d8349d4474efb6991fa063f5/app/components/PreviewDeckCard/PreviewDeckCard.tsx#L27
https://github.com/gator-labs/chomp/blob/5b302c1e599ab031d8349d4474efb6991fa063f5/app/components/QuestionCard/QuestionCard.tsx#L75

@yash431garg Apply same changes to following files:
https://github.com/gator-labs/chomp/blob/5b302c1e599ab031d8349d4474efb6991fa063f5/app/components/PreviewDeckCard/PreviewDeckCard.tsx#L27
https://github.com/gator-labs/chomp/blob/5b302c1e599ab031d8349d4474efb6991fa063f5/app/components/QuestionCard/QuestionCard.tsx#L75

Thanks @neel-ds, I just added those changes.


Intercept Commands

Add the ability to intercept commands before they are executed so can add additional functionality via plugins, such as having to pay to use a command.

Not happening. Plugin devs have to offer this functionality.


General Grievance: Watch fwebs.in

General Grievance requests the watch of the watch_keyword fwebs\.in. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

fwebs\.in has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Issue 31

Coverage remained the same at 99.859% when pulling 35c7b3fe7ccbac372a93788f247bcff030cbb676 on issue_31 into a7cf92983f7ae960795897d1ce17ed17541775cd on dave-2.0.

Coverage remained the same at 99.859% when pulling 9533c7e120085821512671d6b970bac2f2f4bb93 on issue_31 into a7cf92983f7ae960795897d1ce17ed17541775cd on dave-2.0.


[client] add support for registering validator

Motivation
Support registering validator script in client. It's gated by mint_key now, but wonder if we should open this script the same as transfer.
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
test in dev net, register a validator not in genesis and add it to the network. will follow up with a smoke test.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

It's gated by mint_key now, but wonder if we should open this script the same as transfer.

It looks like the (pre-existing) add_validator is gated by the mint key, but this new register_validator script is not. Both of these choices make sense to be; just wanted to make sure I understand correctly.

@sblackshear by gated by mint_key I mean the command will only appear with it's launched with the mint_key i.e. it's part of DevCommand, the txn is of course authorized by the validator's own private key.
all the smoke tests are using client commands, I'm going to add one for the register_validator too(need some change how the swarm generate the genesis)

I'm not a big fan of this too, I can turn it to accept a node.config.toml later

@bors-libra r=davidiw

:pushpin: Commit 9eac015 has been approved by davidiw

:hourglass: Testing commit 9eac0152ee0dbbb18ce9f2f84fe785d2a953e876 with merge 3bb5970d4e63e0767868173ee7dd6112bd003e07...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: davidiw
Pushing 3bb5970d4e63e0767868173ee7dd6112bd003e07 to master...


Feature/basic cf setup

This branch contains code to set up a basic test env for watchful on the cloud foundry.
For the sake of testing and coming closer to the CLI, it also already contains an alpha version of the run command.
This fixes #19

This branch is outdated and will be reimplemented by feature/general-workflow


Cannot click on publisher names with dashes

VS Code Version: 1.55.2
OS Version: darwin 20.3.0

Steps to Reproduce:

Go to an extension made by a publisher with a dash in its name (e.g: scala-lang)
Click on the publisher name
There is no result

Capture:
https://user-images.githubusercontent.com/18191750/115878801-43ae5b00-a449-11eb-966e-480d635175fa.mp4
Proposed solution
Add a @publisher search field that would use the actual publisherName field rather than displayName or publisherId. This probably involves some changes to the search API.
Searches would then look like this:
"@publisher:intersystems-community"
"@publisher:scala-lang"
"@publisher:usernamehw"

Does this issue occur when all extensions are disabled?: Yes

related to #100495

Duplicate of #100495


fix: conda env inside script

Description
Small fix which enables to run conda envs inside script:. Because path to conda was not absolute, python executable also wasn't defined correctly. This fixes the issue described in #1811.
QC

[x] The PR contains a test case for the changes or the changes are already covered by an existing test case.
[x] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

Thanks for digging into this!
Would you feel comfortable adding your minimal example from #1811 as a test case? This would ensure that (i) this problem is fixed and (ii) no future changes break this, again.
Such a test case would be a folder here:
https://github.com/snakemake/snakemake/tree/main/tests
And an entry here:
https://github.com/snakemake/snakemake/blob/main/tests/tests.py
Existing test cases are always a good starting point for creating a new one.

Thank you for response!
I will add new test case as soon as i can :)

Finished tests, @dlaehnemann could you please take a look at my PR again, thank you!

As I'm not a code owner, I cannot provide the final code review. But I'll point @johanneskoester here.

@Jahysama : It somehow doesn't find the dummy package on Windows. I have no clue about Windows and Python on Windows, and didn't see anything obvious, where it might go wrong. Could you check if you have any idea?
Also, I'm not a code owner, so I cannot provide the final code review. But I'll point @johanneskoester here.

@dlaehnemann I think it's the way pip is called on windows; instead of doing it with pip install -e dummy_package, it should be done like python -m pip install -e dummy_package. It happens because there is no pip by default in windows PATH.
Source: https://stackoverflow.com/questions/23832586/how-to-run-pip-on-windows. I already applied this fix.

Perfect, thanks for the quick fix. Fingers crossed that the test now also completes on windows.

@dlaehnemann Hello, David! Sorry for bothering you, but could you please run execution of remaining tests, thank you in advance!

Is it intended that the script touch.py is not included in the changes?

Sorry about it, just added it to my fork, forgot to add it to git

Ah, and then input.txt should probably not be included, as it should be created by that rule, right?

Yes, it is not included, because it will be created by the first Snakefile rule. Although input.txt is included only in expected-results folder.

Ah, I see. Thanks for the explanation.
I just merged the latest main branch status into this branch, because this is required for merging anyways. So if you add further changes to this branch, make sure to first git pull this merge commit.

@dlaehnemann Hello again, David, and a happy new 2023! I tried a different approach: installing a dummy package before the snakemake run. Could you please execute the tests again, fingers crossed. Thank you in advance!

@Jahysama @johanneskoester I see that after so much great efforts (just read through #1811 and the comments here), the PR has stalled.
As I am also affected by this bug now, I would be super greatful if you could revisit and finalize this.

It seems like new version of eido package was released causing error below. Args for validate_project() were changed, it doesn't seem to use exclude_case arg in validate_object in older version anyway, instead it got replaced with sample_name_colname, which is not directly required in validate_project(). I think it is safe to remove exclude_case arg from snakemake/workflow.py
 Full Traceback (most recent call last):
  File "/home/runner/work/snakemake/snakemake/snakemake/__init__.py", line 654, in snakemake
    workflow.include(
  File "/home/runner/work/snakemake/snakemake/snakemake/workflow.py", line 1231, in include
    exec(compile(code, snakefile.get_path_or_uri(), "exec"), self.globals)
  File "/tmp/snakemake-f7zdwzdv/Snakefile", line 8, in <module>
    expand("{sample}.txt", sample=pep.sample_table["sample_name"])
  File "/home/runner/work/snakemake/snakemake/snakemake/workflow.py", line 1331, in pepschema
    eido.validate_project(
TypeError: validate_project() got an unexpected keyword argument 'exclude_case'

Thanks a lot!


Feature/add native deps libtorch

Task linked: CU-26gu8gh As a user I would like to be able to specify and add native dependencies (say libtorch)
closes #135

Task linked: CU-26gu8gh As a user I would like to be able to specify and add native dependencies (say libtorch)


ControlBar not passing tslint:member-ordering rule

Have yet to make the PR where this is found, will update soon.

Fixed in #137


watcher doesn't handle misaligned DRAM/PCIe reads that are written to L1

NOC reads require that the L1 dest address match the alignment restrictions of the src.  This an issue for reads from DRAM/PCIe which have 32 byte alignment while L1 has 16 byte alignemnt.  Watcher should flag this error.

PR https://github.com/tenstorrent-metal/tt-metal/pull/6901 implements this


Animations across multiple armatures in 2.8+

Describe the bug
Animations on multiple armatures function absolutely fine in Blender but break when exported as FBX (For reference this is to Unity). Only the actively selected animation in Blender in the Action Editor will end up being exported, the rest seem to be corrupt or missing. This was supposedly fixed in 2.81 for glTF, but glTF crashes when exporting.
To Reproduce

Have an animation that keys bones from more than one armature
Ideally, have at least 2 animations that operate on more than one armature
Export to FBX, and/or to glTF (Which will likely crash)
Import into something that uses said FBX/glTF files, in my case Unity

Expected behavior
Previously, with <2.8 versions of Blender, the animations have exported absolutely fine. In Unity, I would see all the animations listed and their respective armatures, as "HandArmature | FireGun" and "GunArmature | FireGun" etc
Screenshots
For GLTF Export:

.blend file/ .gltf
Example file
Version

OS: Windows 10
Blender Version 2.8 and 2.81 (Was supposed to be fixed in 2.81? Seems to say so in the patchnotes)

What should be solved now is multiple mesh skinned by a single armature, not the inverse.
Anyway, can confirm the bug in the provided file.
Probably some checks need to be added, because the action is shared between multiple armature, but armature don't have same bone names ... Weird situation, we need to add check to avoid crash


[CBR-89] HTTP block sync

In this PR we present functionality of HTTP mirror based blockchain syncing.
This functionality will allow us to dramatically improve block retrieval experience via use of common technology for file distribution (HTTP) which is more robust than in-built data exchange layer in Cardano and comes to us along with excellent infrastructure at our disposal (S3 and alternatives).
This HTTP download comes at very low complexity cost (block retrieval logic remains almost unchanged), may serve as good short- and mid-term solution for Cardano SL block syncing (on node recovery after being offline).
In particular this PR includes:

By-epoch block dump preparation via auxx command

Dumps are cbor encoding of epochs, compressed with LZMA

By-epoch block retrieval from HTTP mirror as part of node recovery
By-epoch dump application via auxx command

Example PR of changes needed for Daedalus: https://github.com/input-output-hk/daedalus/pull/730

Dmitry benchmarked HTTP downloading vs peer downloading on the same machine using the same internet connection. One caveat is that I served epoch dumps over HTTP from localhost. This should not affect the results much, since the total download size is about 189M.
The total time to sync using HTTP was 43 minutes. Syncing from peers took 1 hour 45 minutes. The graph shows local difficulty (essentially, the number of applied blocks) over time. Missing labels on x-axis are likely a Pandas bug. I decided not to spend any time on a workaround.
If someone wants to reproduce these results, be aware that Cardano needs to be built with -O2 --no-asserts (like ./b -O2 --no-asserts sl), otherwise the performance will be much worse.

I benchmarked HTTP downloading vs peer downloading on the same machine using the same internet connection. One caveat is that I served epoch dumps over HTTP from localhost. This should not affect the results much, since the total download size is about 189M.
The total time to sync using HTTP was 43 minutes. Syncing from peers took 1 hour 45 minutes. The graph shows local difficulty (essentially, the number of applied blocks) over time.
If someone wants to reproduce these results, be aware that Cardano needs to be built with -O2 --no-asserts (like ./b -O2 --no-asserts sl), otherwise the performance will be much worse.


Different subnav for pages

Select one ... (check one with "x")
[ ] bug
[x] feature request
[ ] enhancement

Expected behavior
Is it possible to create a different <nav class="subnav"> for every page? For now I have my router outlet inside content-area.
For example I have a page with invoices, and in my subnav I want links like payed, closed, open etc.
But when I am at another page the subnav needs to be completely different.
Actual behavior
Because my router is in the content-area, i can't change the subnav dynamically.
Reproduction of behavior
Because its more of a question, I don't know if i need to share some code?
Environment details

Angular version: 6.1.x

Clarity version: 0.12.8

OS and version: Mac OS 10.13.6

Browser: all

This is better off as a question on StackOverflow https://stackoverflow.com/questions/tagged/vmware-clarity, and it is more of a question about Angular routing patterns. If you want different subnav it would need to be defined in each route instead of once globally (or you'd have to build some kind of dynamic switcher for it).
Closing as a question for StackOverflow if you need further help.


fix: Unlocalize page and node ids when rendering the page tree in the admin

Description
This fixes #7175 by:

unlocalizing page and node ids when rendering admin/cms/page/tree/menu.html
sanitizing page and node ids received from request.GET in cms/admin/pageadmin.py to make sure only unlocalized integers are passed on to int.

See #7175 for a detailed description of the problem being addressed.
NOTE: the new test takes approximately 10 seconds (on my old laptop) because it has to create 1000 pages to trigger the problem. If you see a more convenient way of triggering the problem, please let me know!
Checklist

[x] I have opened this pull request against develop
[x] I have added or modified the tests when changing logic
[x] I have followed the conventional commits guidelines to add meaningful information into the changelog
[x] I have read the contribution guidelines  and I have joined #workgroup-pr-review on Slack to find a “pr review buddy” who is going to review my pull request.

@mbi Thanks for the fix, Marco. Need to look into this fix with a bit more care.
Also, please don't mix the fixes with the formatting and import changes, it hides the real fix and makes reviewing the code harder.

Also, please don't mix the fixes with the formatting and import changes, it hides the real fix and makes reviewing the code harder.

Apologies about that, my editor's isort settings apparently didn't pick up the project settings.

@marksweb this PR is a few months old, but the original bug (#7175), even if it was closed by the stalebot, is still very relevant. Would you mind taking a second look? :pray:

@mbi yeah sure.
Stalebot is in use to help us keep on top of issues.
When it updates things, we can see the important issues because people like yourself let us know. We had hundreds of open issues and this is helping to manage them.

So looking at this new test, I don't think we can really cope with test workflows needing an hour+ to run. Quick fixes won't be possible anymore.
Obviously the test has merit, but we need a way to isolate it and then we can mark it as optional for checks to pass.
@fsbraun would be good to get your thoughts on this PR change & any potential solutions to what I'm thinking here.

@mbi @marksweb I have two thoughts:

Can we set the auto increment for the id in the database to 1000, e.g. by issuing an sql statement in the test? (e.g., ALTER TABLE cms_page AUTO_INCREMENT=1000;)
Is it possible for a linter to search through all template files to look for {{ something.pk }} and {{ something.id }} not followed by | unlocalize or | safe? We'd be able to skip the test and we would be able to transfer this to all packages.

Finally, I am wondering if | safe is better than | unlocalize since it does not require the l10n template tags to be loaded. Or is that considered a hack? (It clearly does the trick.)

@mbi Sorry, one last thing before I merge - this needs a changelog entry.


Implement validation that a company type is defined for all software types

At the conclusion of this work,

The tool will support validating that all software types have a defined company type.
The tool will support validating that all company types' defined software types exist.

Software types that do not have a defined company type will never be produced by the AI.

The tool will support validating that all software types have a defined company type.

This will require a validation process to run post load.

00c6d8991880894cfe360648e35b29a52eb9fb8c
1d5d49f9fbb9f0fae6386e31c850438b2ee081d9


Add CLI args for the conda-pack command

This is to resolve the issues observed in https://github.com/nsls2-conda-envs/nsls2-collection-tiled/pull/1.

@gwbischof, looks like the fix worked OK for https://github.com/nsls2-conda-envs/nsls2-collection-tiled/pull/1. Do you think we should hard-code it or make it configurable?

I am going to move this repo to the https://github.com/nsls2-conda-envs org. So, I'll merge this PR before moving.


Error reading from remote server

Hello all,
I have installed a GROBID client on my server. It was working perfectly until 2 days ago.
Since I am in development phase, I am testing always the same bunch of papers. Up to two days ago, GROBID would classify them, without problem. Now, I get always the same error to the very same papers.
From the web interface, I get:
Error encountered while requesting the server.
Proxy Error
The proxy server received an invalid response from an upstream server.
The proxy server could not handle the request POST /grobid-test/api/processHeaderDocument.

Reason: Error reading from remote server

To see whether the GROBID instance I am using is corrupted, I installed a second instance of GROBID on a second server (much powerful of the first one) and still I cannot get away with it. Actually, here the process seems stuck and does not return any error message.
For the sake of test, I tried also this client: https://cloud.science-miner.com/grobid/, but I get:
Error encountered while requesting the server.
{ "message":"The upstream server is timing out" }

Ho can I fix this?
I just want to add further details, so you can reproduce the error.
I go to the web page of GROBID http://localhost:8070
Then, I go to TEI,
Then, Process Header Document
Then, I check Consolidate header (only)
Load the pdf
And hit Submit.

UPDATE:
if I CURL the API, it works!
It is not working via the web interface (see screenshot), but if I launch curl -v --form<EMAIL_ADDRESS>localhost:8070/api/processHeaderDocument, it works perfectly.

Any help is really appreciated.
Thank you
Angelo

Further update.
We are connecting to the very same instance of GROBID with the python client. In the past, was working perfectly. Now, instead I get:
Processing of ./file.pdf failed with error 408

Hello @angelosalatino !
Thanks for using Grobid.
This kind of proxy errors are not related to Grobid, but to your environment. In that case your web client is connecting to a proxy even for localhost. On linux for instance, you can set some exception for localhost for you tests:
$ gsettings set org.gnome.system.proxy ignore-hosts "['localhost']"

On other OS, it's done differently.
You can also check the proxy settings of your web browser.
Error 408 means timeout. Timeout comes usually from too large documents, or too high concurrency on your server given the number of available cores/threads (default is 10). You can change the timeout and the concurrency parameters of the server as documented here. The python client adjusts the query rate itselt based on the server response, but it also takes a parameter n for parallel requests that you might want to decrease (see here).

Thanks a lot @kermitt2
I will adjust the server parameters then and see how it goes.
Thank you again.


PHOENIX dashboard

This is related to #116.

I think it needs to say where to get the example data. I found it here: https://github.com/OttoStruve/muler_example_data/tree/main/IGRINS/01_IGRINS_test_data
And mention all the three files that are needed for this
Currently I get an error when creating the grid, since no points are found. I will try to debug, to make sure I didn't do anything wrong.

Review link: https://github.com/openjournals/joss-reviews/issues/6601

For the grid, the structure of the PHOENIX models were slightly wrong (I will have to check if it is written correctly in the documentation a bit later, or if I did something wrong).
However, now the dashboard crashes due to some problems with the wavelength range. If I change the grid to: grid = PHOENIXGrid(teff_range=(2500, 7000), logg_range=(2, 5), Z_range=(0, 0.5), wl_lo=21350, wl_hi=21590) it works (I changed wl_lo/wl_hi).
This is not as smooth as the other dashboard, but I guess that is due to the longer wavelength range.
I think I nice addition to both examples is to write the actual parameters, Teff, logg, etc. so the user can "verify" their best fit. At least this is something I would like to have.

I'll take a look at all these issues and let you know when they've been fixed.
And yes, the dashboard does slow down as the wavelength range increases. A trick I've found to make it more responsive in such cases is to click through the sliders instead of dragging. This prevents the callbacks from stacking up and slowing the dashboard down.

Alright, I've changed the docs a little bit, they now give links to both the spec_a0v.fits and variance.fits files that IGRINSSpectrum needs to initialize (although gollum's repo now comes with them preloaded in the tutorial_data directory, which the docs also reflect now). The structure of the PHOENIX grid was a little inconsistent and confusing in the docs, I've updated them to be much easier to get up and running with little to no modification required after the download is complete. Basically all your config.env needs to point to is the directory which contains the wavelength file and the PHOENIX-ACES-AGSS-COND-2011 file (this specifically housing all the metallicity slices).
I've also added the verification for the best fit parameters (it is by eye, so not spot on obviously, but probably in the ballpark).


Fix Center Crop error with Tensorflow

The center cropping raises currently an Exception with Tensorflow backend and CNN transformations activated.
https://github.com/ViCCo-Group/THINGSvision/blob/master/thingsvision/model_class.py#L317
The crop width 224 should not be greater than input width.
Condition x >= 0 did not hold element-wise:
x (shape=() dtype=int32) =
-221

@hahahannes is this still an open issue?

@hahahannes?

This was caused by this line: https://github.com/ViCCo-Group/THINGSvision/blob/4bbcb817333cc0e635b3b585e66717e99243721a/thingsvision/core/extraction/mixin.py#L147
But I am not sure whether this still raises an exception.

@hahahannes could you please check whether this is still an issue?

@andropar, could you take over and see whether this is still an open issue, and if so how to fix it?


Disable export button when there are no deadlines to download

Right now it will download file with the text "error"

I would like to work on this issue.
It seems many of your other contributors are students at Calgary.
Please let me know if it is okay for me to help out.

I would like to work on this issue. It seems many of your other contributors are students at Calgary. Please let me know if it is okay for me to help out.

Hi Cody, this project is currently under development by the Lifeline team at Tech Start UCalgary! My name is Tim and I'm the Project Manager for the team, the rest of the members are the contributors thus far.
I am open to outside help, but due to the nature of the organization I think it best if I consult the club execs as well as my project members to see if they're OK with this. In the meantime, feel free to join the club discord and ping me if you want.

That's alright Tim, I will leave this to your team.
I'm not a student, just someone looking for new and exciting projects.
Best of luck with your efforts!

That's alright Tim, I will leave this to your team.
I'm not a student, just someone looking for new and exciting projects.
Best of luck with your efforts!

Sounds good, thanks for your interest. When the semester/project lifecycle ends on April 28th with our final showcase, the Lifeline will likely become open source if you're still interested in contributing 🙂


deb package

Pull binary from https://github.com/v2fly/v2ray-core/releases and build deb package.

Done with https://github.com/hangxie/v2ray/commit/597932ca37553ba4e76093c6269a73028faf6e17


can not establish connection sometimes when the network loss rate is a little bit high

The connection of client and server is ok when network is normal, but probably failed when the network loss rate is about 10% or more.
The logs of client:
quiche::tls: b680995c68d23bbe830b0521905f4acc write message lvl=Initial len=512
quiche: b680995c68d23bbe830b0521905f4acc tx pkt Initial version=babababa dcid=a6226e4a5ea2ecb344c56c1b2fbfd6b0 scid=b680995c68d23bbe830b0521905f4acc len=1157 pn=0
quiche: b680995c68d23bbe830b0521905f4acc tx frm CRYPTO off=0 len=512
quiche: b680995c68d23bbe830b0521905f4acc tx frm PADDING len=625
quiche::recovery: b680995c68d23bbe830b0521905f4acc timer=998.474125ms latest_rtt=0ns srtt=None min_rtt=0ns rttvar=166.5ms loss_time=[None, None, None] loss_probes=[0, 0, 0] cwnd=14520 ssthresh=18446744073709551615 bytes_in_flight=1200 app_limited=true congestion_recovery_start_time=None delivered=0 delivered_time=539.291µs recent_delivered_packet_sent_time=540µs app_limited_at_pkt=0  hystart=window_end=None last_round_min_rtt=None current_round_min_rtt=None rtt_sample_count=0 lss_start_time=None
sent 1200 bytes
The logs of server:
[2020 Oct 15 Thu 17:42:49:658] recvfrom 1200
[2020 Oct 15 Thu 17:42:49:658] version negotiation
[2020 Oct 15 Thu 17:42:49:658] recv would block
The problem looks like #385 . But I had used the latest master code.
How can I do to solve it? Does it need to do some extra setting? Thanks very much.

nobody~


Android: iconColor doesn't work in hexadecimal (#RRGGBB) format

Despite of what the docs says about android.iconColor, I cannot use colors in #RRGGBB format.
This config won't display the proper background color (will display the default grey instead):
config = {
  android: {
    senderID: GCM_SENDER_ID,
    iconColor: "#544A44"
  }
};

$cordovaPushV5.initialize(config)

However, it works with iconColor: "#FF544A44" (i.e. with the #AARRGGBB format).
Also, for reasons I cannot understand, the #RRGGBB format it does work when using black color ( iconColor: "#000000").
Maybe the best option is to just recommend in the docs to use the  #AARRGGBB format.

@javiercr I'll give it a try with your value but according to the Android docs the method we use should support #rrggbb and #aarrggbb strings.
http://developer.android.com/reference/android/graphics/Color.html#parseColor(java.lang.String)

@javiercr working fine for me. Can you give me an example payload you are sending to GCM?

@macdonst, I've tested it again and now it works with #544A44. Very weird... Closing this, sorry for bothering you.


Fix browser env leak after resetting agent

Fixes #1528

Codecov Report
Attention: Patch coverage is 0% with 13 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (main@7b6b4e3). Click here to learn what that means.

Files
Patch %
Lines

opendevin/runtime/browser/browser_env.py
0.00%
12 Missing :warning:

opendevin/server/agent/agent.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1589   +/-   ##
=======================================
  Coverage        ?   52.28%
=======================================
  Files           ?       88
  Lines           ?     3724
  Branches        ?        0
=======================================
  Hits            ?     1947
  Misses          ?     1777
  Partials        ?        0

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

@frankxu2004 you can probably move this into agent_controller.close() now

Looking good! I think we just need to add a close() call into agent_controller

Added and tested that now after agent re-init  (e.g. change config), the browser process will always shutdown correctly. Should be ready @rbren

Thanks Frank!


Not enough tokens error, although I've paid A LOT

This is related to issue #153 , which was closed but not resolved. I didn't see a way to reopen that issue.
I'm posting this here out of exasperation with OpenAI's customer service after exchanging dozens of messages with them. My interaction with OpenAI customer support has been beyond terrible. I understand it's a startup, but I hope they'll devote some time and attention to making customer service work for their customers.
Basically, I'm getting an error message shown in screenshot 2 that I'm lacking about 3k tokens to complete the request. However, the OpenAPI website (screenshot 3) says I have $50+ balance, which is more than enough tokens. I can't seem to convince OpenAI that I've paid for the tokens even though they've been charging my credit card for almost a month (screenshot1). They seem convinced I still have a free account.
I'm hoping someone here can help me fix this or at least suggest a strategy for dealing with Open AI that won't result in weeks of wasted effort and frustration.

Hey @resonates7 I can definitely understand your frustration. The irony with OpenAI is their disruption of AI vs the inability to leverage it for customer support :)
The error you're seeing is not related to your balance, but rather to the fact that each GPT model has token size limitations, meaning the amount of words it can handle/generate. For GPT 3.5 it's 4K which is what you're seeing in the error. This may due to edge cases in our service where GPT Researcher summarizes texts that are too long, however this should not happen in most cases, also since we've updated the model to use gpt3.5-16k which has a 16k token window. Have you modified the code to use only GPT3.5?

Yes, and I've been able to fix it by changing it to gpt3.5-16k in the config file. Thanks for the tip!


would it be possible to change the area type of individual navmesh polygons?

I'm not sure if this is the right place to raise this as it's not directly connected to this particular change.
It would be great if I could select a navmesh polygon and change its area type without needing to rebake the navmesh.
Use case - I have 2 characters, one shoots at a target.  The other wants to move without straying into the field of fire of the first character. It seems that one way to do this would be to increase the cost of all navmesh polygons traversed by the field of fire (by changing them to an expensive area type), thus making the pathfinder tend to avoid them.
Apologies if this isn't an appropriate place to raise this request.

This is something we'd like to support in the future - one drawback with simply changing the area type of the existing polygons - is that the tessellation might not match the actual affected area.
The NavMeshModifierVolume can be used with runtime updating the navmesh to create a similar effect - where the volume bounds control the tessellation in addition to the area type.

I'm trying to create a path where my AI people will preferentially choose to take areas they've already travelled. I came to this library and saw "bake navmesh at runtime" and this nice concept of NavMesh modifying. I've tried combining the game's traditional navmesh with the agent from this library to get a kind of "hybrid" situation, but I still can't find a way to update the cost of an "area" at runtime.
My other friend explained to me that he is making a lake, dam, and stream. The stream will get bigger so he'd like to change the landscape, and therefore the navmesh. This library is perfect, except again it's hard to explain to the system that the water should be passable but at a different cost than the beach.


Add a read.me file into the download packages that is the release notes for that version

Is your feature request related to a problem? Please describe.
The release notes should go with all the various download packages to explain how they are new / different / changed. This is probably most easily accomplished by including a read.me file of the release notes with the dowloads.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
See Kameron or Beth for the release notes

CEJST 1.0 release notes - REVISED-KK-comments.docx
@esfoobar-usds these are the release notes for the read.me file (we can talk about what styling is possible in that format).
@vim-usds these are the release notes for the downloads page.
not final final but let's go with it.

we need to add geolocation (new feature) somewhere to these notes


pass use_fake_hardware to moveit to change default controller

Fixes https://github.com/UniversalRobots/Universal_Robots_ROS2_Gazebo_Simulation/issues/21

Closing in favor of #43

@Mergifyio backport iron


About LoRA Rank

Thanks for the nice work.
I have a question about the rank of the LoRA. Have you tested with different LoRA ranks? Will 'larger' LoRAs perform better in any of your experiments?

Thanks for your interest. We use a fixed rank size in all the experiments, but you can adjust it here.

Thank you


[CT-2455] [Feature] Less strict secret env vars: allow elsewhere besides just profiles.yml + packages.yml

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
I think this one just calls for a reversal of the decision made in https://github.com/dbt-labs/dbt-core/issues/4310
One example where you might want to use secret env vars is when using Snowflake's encrypt_x / decrypt_x functions... e.g.
-- models/pii.sql
select encrypt('john', '{{ env_var("DBT_ENV_SECRET_PWD") }}') as name

Which due to the issue/pr above results in the expected error:
Parsing Error
  Secret env vars are allowed only in profiles.yml or packages.yml. Found 'DBT_ENV_SECRET_PWD' referenced elsewhere.

Describe alternatives you've considered
I don't think any good alternatives exist.

You use a normal var - which exposes the passphrase in plaintext.
On snowflake - you could store the passphrase in a table - setup datamasking/role based access and what not - kinda convoluted.

Who will this benefit?
Anyone wanting to use secret vars in places that are not just those 2 files.
Are you interested in contributing this feature?
Sure
Anything else?
No response

@jeremyyeo There's a workaround for this today:

Create two env vars, one with the "secret" prefix and one without (e.g. DBT_ENV_SECRET_PWD and SECRET_PWD) set to the same value
Use the "non-secret" env var inside the SQL file

-- models/pii.sql
select encrypt('john', '{{ env_var("SECRET_PWD") }}') as name

The value will be scrubbed from dbt's logs:
$ SECRET_PWD=abc123 DBT_ENV_SECRET_PWD=abc123 dbt compile -s pii
12:20:52  Running with dbt=1.5.0-rc2
...
12:20:55  Compiled node 'pii' is:
-- models/pii.sql
select encrypt('john', '*****') as name

create or replace   view analytics.dbt_jcohen.pii

   as (
    -- models/pii.sql
select encrypt('john', '*****') as name
  );

And for a completely separate reason, they will also be scrubbed from Snowflake's query logs, because of the nature of the function (encrypt) being used:

The downside of this approach is, by also storing the value in a non-secret env var, there's nothing preventing someone with development access to the same project from writing that value to a database table (outside a function like encrypt that Snowflake knows to scrub), and then querying the table to read it back out. That's not possible with a "secret" env var due to the intentional limitations around where it can be used/rendered.
Longer term, we should really reconsider how we treat secrets (#6353). Currently, it's just a special behavior for scrubbing a value from the dbt (logs) if it matches an environment variable with a certain prefix.

I guess this may be one that could be more dbt Cloud related? But is directly the result of dbt-core not supporting secrets in other places (one of those "cross-functional" thingies =]). Why would one use secret env vars in the first place?
a) You don't want it logged in logs.
b) You don't want all dbt developers to know the secret value in the first place.
I think what you mentioned addresses (a) but not (b)?

I wrote this elsewhere but worth putting down here too - recapping why simply putting env vars in model files doesn't solve the problem at hand.
Let's assume we want to use the encrypt function (for simplicity).
And assuming a toy example:
-- some_model.sql
select encrypt('john', 'correct horse battery staple') as name

Okay, so when we build this table in Snowflake - what DDL/DML does dbt send? First things first... dbt has to compile the above model into working and proper SQL:
-- target/compiled/some_model.sql
create table some_model as
select encrypt('john', 'correct horse battery staple') as name;

Now... let's assume for a second that dbt can have secret env vars in model files:
-- some_model.sql
select encrypt('john', '{{ env_var("DBT_ENV_SECRET_PASSWORD") }}') as name

Now... what happens?
Firstly, dbt will always have to compile the sql model file and output it to the target folder - just like above. So - if dbt were to MASK the values in the compiled SQL:
-- target/compiled/some_model.sql
create table some_model as
select encrypt('john', '!!!!*****!!!!') as name;

^ Then you table some_model will literally be using the string '!!!!*****!!!!' as the encryption passphrase - as opposed to using the env var set by DBT_ENV_SECRET_PASSWORD (which would have been equal to the string 'correct horse battery staple').
Because dbt always always has to compile to valid SQL first - that's kind of the rationale that secret env vars were explicitly blocked from model files.

Separately I went about testing out using Snowflake Data Masking for this.

I created a secret table - whose value is only visible to TRANSFORMER and not DEVELOPER - I then tried to use the DEVELOPER role to try and encrypt it's value.

It failed with the passphrase 'correct horse battery staple'.

But succeeded with the passphrase that was 5 asterisk - the masked value. Meaning it had been encrypted not with 'correct horse battery staple' but '*****' - so unfortunately this looks like a dead end as well unfortunately :(


click using javascript

Click using Javascript only and sample test case added

Thank you for your valuable contribution! Hopefully the first of many ;)
And for future use, please note that you can chain actions in the following way:
driver.element().type(emailField, "standard_user")
		.and().type(passwordField, "secret_sauce")
		.and().clickUsingJavascript(driver.getDriver(), loginButton)
		.and().clickUsingJavascript(driver.getDriver(), product1)
		.and().clickUsingJavascript(driver.getDriver(), shoppingCartButton)
		.and().verifyThat().element(productName).text().equals("Sauce Labs Backpack");

noted and looking forward more contributions ^_^


Junior: support animation for robot

Add 4 different sprites for different directions.

Use default one for the initial image or if animation is not required
Use the same image after step move if no specialized sprite is provided (leave sprite unchanged)

Cases:

a cat with two images -- heading west or east
a car -- heading N,W,E,S
A ghost -- no animation, single sprite

Fixed in #1302


Embed/include code within notes in a similar fashion to how note refs embed content from other notes in a vault

Please select if your request is either something new or an enhancement

[ ] Enhancement of an existing Feature.
[x] Request of a new feature.

Please select the area your request applies to. (Multiple selections are possible)

[ ] Onboard. Initial vault setup and import/export pods
[ ] Create. Note creation, lookup, snippets and templates
[ ] Retrieve. Backlinks, references, graph view
[ ] Structure. Refactoring, multi-vault and schemas
[ ] Publish. Sharing your repo with the world
[x] Misc (Choose this if your not sure)

Is your feature request related to a problem? Please describe
It would be great if Dendron could import/include a file with code, and display it as it would a code-block. This could work similarly to a note reference, since a note ref includes a section of a note embedded in another note.
Describe the solution you'd like
For example: I have a bash script that I'd like to version within the vault, but would also like the script to be included in a note as syntax-highlighted code. If I have a .sh bash script, for example, I'd like to include it in a note where it gets treated like:
# scripts/my-script.sh contents
echo "Hello"

Rendered and shown the same as doing code directly in a note, within a code-block. This would mean that, whatever path is taken to embed and syntax-highlight code, that there needs to be a flag/option for stating what language the syntax/language is.
This is similar to how reStructuredText literalinclude directive in Sphinx works, turning something like this source file into this rendered HTML.
I'm not sure how this would be done, but it would be similar functionality to how a note ref works with ![[note.ref.example]]. This could perhaps piggyback off of what work may go into admonitions via #1287
Describe alternatives you've considered
There could be different ways of going about this, depending on the embed/include syntax that is used within Markdown
Additional context
This is helpful if wanting to include non-markdown, non-note embeddings within documentation. This may also allow for segmentation of code if wanting to be able to test documentation.
Example scenario:
A note describes how to install and configure a CLI tool. Not only could the commands be embedded within a note, but the code itself could be tested in CI/CD. This ensures that out-dated documentation is caught immediately upon being out-dated, due to failing to run when it does (that, or it may catch bugs that weren't caught in the development process).

Related discussion/RFC that may address this feature request: https://github.com/dendronhq/dendron/discussions/1975


chore: issue format spec

Hi, I want to contributed to the awesome zkwasm project. But there is some difficutly for me to do so. The issue need some labels and format for others to know more details. Here is a example:
## Description

---
<!-- Issue Author: Don't delete this message to encourage other users to support your issue! -->
**Message from the maintainers**:

Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

I could help you to build label system/bot and issue format to help more guys know the flow of contributing

Here is the issues labels system specs:
chore: non-feature requirements such as CI/CD, building script or work flow enhancement.
feat: feature need, we could use feat(circuit), feat(lang), feat(CLI) to categorize features
bug: bug report, we also could use bug(circuit), bug(lang) to categorize bugs
doc: documents related issues.

nice point! We will follow.


try to fix electron build (wip)

sizes:

name
files before
files after
size before
size after

pty-host

154

955.5 kB


The only semi new thing is the DCT but even that I think is not completely new either. I thought there was already a generative model that did that not sure if it was autoregressive though.
But I guess the rational is that DCT is better than any learned quantized fixed length encoder and it saves you training an encoder.

Indeed! Regardless I'll finish the project so I can figure out if DCT representation has some sort of Achilles's heel


Generated error - cannot build

I try generating code from your examples/cellar/design and get some error from generated code.
./account.go:16: cannot use AccountController literal (type *AccountController) as type AccountController in return argument
./bottle.go:16: cannot use BottleController literal (type *BottleController) as type BottleController in return argument
./main.go:20: not enough arguments in call to NewAccountController
./main.go:21: cannot use c (type AccountController) as type app.AccountController in argument to app.MountAccountController:
	AccountController does not implement app.AccountController (Create method has pointer receiver)
./main.go:23: not enough arguments in call to NewBottleController
./main.go:24: cannot use c2 (type BottleController) as type app.BottleController in argument to app.MountBottleController:
	BottleController does not implement app.BottleController (Create method has pointer receiver)

Here what I get from generated code https://github.com/MathieuDoyon/goa-test

Thank you for the report! I broke it last night when pushing the latest changes around controllers :(
It should now be fixed, make sure to grab the latest goagen. Need to add more tests...

Thanks working now :)
nice project btw


New release

Hello!
I would like to package this awesome tool for the NixOS distribution, but have found some issues with the current v0.1.1 release:

The Cargo.lock specifies that the version of the webserver is version 0.1.0 instead of 0.1.1 (easily fixable with a patch)
The project fails to compile the pear_codegen v0.0.18 dependency, complaining that the quote feature has been removed (less easily fixable)

So I guess my question is: are you planning on doing a release in the near future?
I have seen the v0.2.0 milestone and looks quite featureful, so I think my question stands more for a v0.1.2 release.
Thank you for the awesome work, and have a great day!

I am not planning on releasing a new version soon. Development has mostly stalled, since this is really a passion project that earns me no money, so I only work on it when the mood strikes. While I am invested in making sure it continues to work, I wouldn't expect any new version updates.

Thanks for the clarifications! This answers my questions, then ^^