fix(doc): Typos

Noticed two typos while reading the lib code (which is great!).

@juditnovak , merge it?


Add new template - Beginner🙀

#10

Hey, Can you assign this to me? I would love to work on it.

Hey @mpotane , Can you assign this to me? I would love to work on it.

Hey @mpotane , Can you assign this to me? I would love to work on it.

Sure no problem po.

Hey @mpotane , Can you assign this to me? I would love to work on it.

Hello @mpotane few changes made, thanks.


Row as Dictionary<string, object> via GetDictionaryAsync

Hi,
I'm beginner with SqlKata. I'm very happy that found tool which allows me to create low-level SQL queries but my project needs dynamic results too. Now I've problem with GetDictionaryAsync:
var query = _db
   .Query("cms.contents")
   .SelectRaw("values->'name' as name")
   .Where("type", "product");

var dictionary = await _db.GetDictionaryAsync(query);

Now dictionary is null :(
But if I use sync overload GetDictionary all works fine. I think that something is wrong with async overload.

@pavelvondrasek , Could you share your environment details (os, .net runtime, and SqlKata version used)?

Yes, of course.
I'm on macos Catalina on Postgres 12 which is on remote server with Debian.
Using .NET Core SDK 3.1.402, SqlKata 2.2 and Npgsql 4.1.5

I've identified the issue and fixed in the nightly build, you can get it from https://www.myget.org/feed/sqlkata/package/nuget/SqlKata
Related PR: #419

Thank you for your kindliness and great support! 💯 ❤


Loading directly 4bit quantized model

System Info
I saved 4bit quantized model
Then, how to load 4bit quantized model directly with 'from_pretrained' ??
It is normal to save Large Models with float16 or float32 or bfloat16.
But in my case, I saved 4bit directly and want to load 4bit quantized model.
Reproduction
save 4 bit quantized model with save_pretrained()
load from_pretrained() with 4bit quantization config
Expected behavior
Uint8 Value Directly Loaded

Hi @ByungKwanLee
Yes this should work properly, are you getting an error?

Hi! HF documentation says that we currently cannot load 4bit weights after we saved them with save_pretrained()
https://huggingface.co/docs/transformers/v4.35.1/en/main_classes/quantization#load-a-large-model-in-4bit

When using with FSDP, this currently results in the following error:
[rank5]: Traceback (most recent call last):
[rank5]:   File "/app/train.py", line 382, in <module>
[rank5]:     main(script_args, training_args)
[rank5]:   File "/app/train.py", line 306, in main
[rank5]:     trainer.train(resume_from_checkpoint=checkpoint)
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/transformers/trainer.py", line 1932, in train
[rank5]:     return inner_training_loop(
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/transformers/trainer.py", line 2079, in _inner_training_loop
[rank5]:     self.model = self.accelerator.prepare(self.model)
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/accelerate/accelerator.py", line 1304, in prepare
[rank5]:     result = tuple(
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/accelerate/accelerator.py", line 1305, in <genexpr>
[rank5]:     self._prepare_one(obj, first_pass=True, device_placement=d) for obj, d in zip(args, device_placement)
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/accelerate/accelerator.py", line 1181, in _prepare_one
[rank5]:     return self.prepare_model(obj, device_placement=device_placement)
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/accelerate/accelerator.py", line 1477, in prepare_model
[rank5]:     model = FSDP(model, **kwargs)
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/fully_sharded_data_parallel.py", line 469, in __init__
[rank5]:     _auto_wrap(
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/_wrap_utils.py", line 101, in _auto_wrap
[rank5]:     _recursive_wrap(**recursive_wrap_kwargs, **root_kwargs)  # type: ignore[arg-type]
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/wrap.py", line 543, in _recursive_wrap
[rank5]:     wrapped_child, num_wrapped_params = _recursive_wrap(
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/wrap.py", line 543, in _recursive_wrap
[rank5]:     wrapped_child, num_wrapped_params = _recursive_wrap(
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/wrap.py", line 543, in _recursive_wrap
[rank5]:     wrapped_child, num_wrapped_params = _recursive_wrap(
[rank5]:   [Previous line repeated 2 more times]
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/wrap.py", line 561, in _recursive_wrap
[rank5]:     return _wrap(module, wrapper_cls, **kwargs), nonwrapped_numel
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/wrap.py", line 490, in _wrap
[rank5]:     return wrapper_cls(module, **kwargs)
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/fully_sharded_data_parallel.py", line 495, in __init__
[rank5]:     _init_param_handle_from_module(
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/_init_utils.py", line 599, in _init_param_handle_from_module
[rank5]:     _init_param_handle_from_params(state, managed_params, fully_sharded_module)
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/_init_utils.py", line 611, in _init_param_handle_from_params
[rank5]:     handle = FlatParamHandle(
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/_flat_param.py", line 582, in __init__
[rank5]:     self._init_flat_param_and_metadata(
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/_flat_param.py", line 632, in _init_flat_param_and_metadata
[rank5]:     ) = self._validate_tensors_to_flatten(params)
[rank5]:   File "/usr/local/lib/python3.10/dist-packages/torch/distributed/fsdp/_flat_param.py", line 768, in _validate_tensors_to_flatten
[rank5]:     raise ValueError("Cannot flatten integer dtype tensors")
[rank5]: ValueError: Cannot flatten integer dtype tensors

However loading from bf16 weights works fine. bnb_4bit_compute_dtype and bnb_4bit_quant_storage are set to torch.bfloat16 as needed for FSDP

Also I thought the problem might be that I load with device_map='auto' and HF docs suggest to move to cpu before saving.
However this is not working as well:
ValueError: `.to` is not supported for `4-bit` or `8-bit` bitsandbytes models. Please use the model as it is, since the model has already been set to the correct devices and casted to the correct `dtype`.


Find-Finder

Description of the new feature / enhancement
In a big document, having searched and found a letter (or whatever), what only happens is that word puts the textmarker there which to find is like searchng a needle in a haystick.
It is a Pity!!!
We URGENTLY need to extend the function of the "Mauszeiger Fadenkreuz/,ouse pointer crosshairs" or "Moon blow-up-and-shrink" also in documents !!!
Thank You - that would save all of us a lot of time and trouble"
Scenario when this would be used?
In a big document, having searched and found a letter (or whatever), what only happens is that word puts the textmarker there which to find is like searchng a needle in a haystick.
Supporting information
No response

How would PowerToys solve this issue? It sounds like a problem in whatever document editor you are using. /needinfo


rewrite: main packages mixed in normal package don't have imports rewritten, resulting in cyclic dependencies

try this, in an empty package, after govendor init:
govendor add github.com/lib/pq
govendor list

output:
u github.com/lib/pq
e github.com/lib/pq/oid

Looks like an external dependency was missed.  Try adding it:
govendor add +e
govendor list

output:
v github.com/lib/pq
v github.com/lib/pq/oid

Those should be unused, since I have no source files at all.  I believe the reason they appear as v is because they refer to each other.
> govendor list -v
v github.com/lib/pq ::/internal/github.com/lib/pq
  └── gitlab.protectv.local/ncryptify/russ.git/internal/github.com/lib/pq/oid
v github.com/lib/pq/oid ::/internal/github.com/lib/pq/oid
  └── gitlab.protectv.local/ncryptify/russ.git/internal/github.com/lib/pq

The oid package has some normal source files in package "oid", and one source in package "main" with an "ignore" build tag.  That main source file's import statement is never rewritten by govendor, even though I don't currently have govendor configured to ignore the "ignore" build tag.
The workaround to at least avoid the import cycle is to ignore the "ignore" build tag in vendor.json

Thanks for the report. I'll look into this.

govendor on the default branch now only supports the vendor folder. A new branch called "rewrite" supports import path rewrites.


Upload Modal Triggering Library Refresh

Bug Description
After closing the upload modal, the media library is refreshing. As we know this has some performance / loading issues.
Expected Behaviour
Experience should be smooth and not sure if this should trigger a refresh in the library.
Steps to Reproduce
open and close the upload modal.
Screenshots

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

In the media dialog, lots of different functions can be done by the user. For example, alt text can be edited, images deleted and new files to can be uploaded. So it was seen as a good idea to refresh the library. However, in earlier versions, the loading message didn't show and the refresh was a little bit more graceful. Transmission between the two states is a smoother way the current. But we are waiting on #662 #439
Thoughts @merapi ?

@samitron7 has asked us to not show the loading message if the wait time is short, so this might be a non-issue once we do that.

Correct - users should never see the loading message for any loading state in the editor (not just media) if loading is quick. @jhtin is setting a duration time for this of 500ms. I'm not sure what that perfect time is. @pbakaus if you have a suggestion.

Correct - users should never see the loading message for any loading state in the editor (not just media) if loading is quick. @jhtin is setting a duration time for this of 500ms. I'm not sure what that perfect time is. @pbakaus if you have a suggestion.

Seems OK. Let's try it out and see how it feels.

Closing as we've implemented a delay for the loading pill.

Reopening as the media library is not actually refreshing right now after uploading media through the WordPress media library (via the upload button).
/cc @diegovar - can you confirm that?

Closing as there's another ticket for this issue.


Redshift distribution and sort key - default to auto or allow top level configuration

Describe the feature
Redshift now has an auto option for table distribution and sort keys. Additionally, they have also released a feature, Automatic Table Optimization, that allows Redshift to swap things out. This feature more or less automates the suggestions that show up in the svv_alter_table_recommendations system view, UI, etc.
It would be super helpful to have that as the default, either by convention in DBT, or by configuration through allowing the dist or sort configuration values to be set in the dbt_project.yml.
EDIT: Looks like sort keys can be set in dbt_project.yml already.
Describe alternatives you've considered
A config() block in every model. 😢
Additional context
This is a Redshift database specific feature.
Who will this benefit?
Anyone who uses Redshift, and would like to default to auto dist and/or sort keys to take advantage of the Automatic Table Optimization feature.
Are you interested in contributing this feature?
Sure.

@brangisom This should already be possible in dbt_project.yml for both sort and dist keys:
models:
  +sort: auto
  +dist: auto

What I also hear you proposing, and I think it has merit, is changing the default values of Redshift-specific configs:
https://github.com/fishtown-analytics/dbt/blob/2b48152da66dbd7f07272983bbc261f1b6924f20/plugins/redshift/dbt/adapters/redshift/impl.py#L12-L16
We might change that to:
class RedshiftConfig(AdapterConfig):
    sort_type: Optional[str] = None
    dist: Optional[str] = 'auto'
    sort: Optional[str] = 'auto'
    bind: Optional[bool] = false

Effectively, these were hard-won features that were a lot trickier a few years ago. Nowadays, we set sort: auto, dist: auto, and bind: false as top-level configs in every one of our dbt + Redshift projects. Of course, specific tables sometimes benefit from opinionated sorting/distributing, but that's what config() is for.

Yeah, it turns out the auto can't be passed into a CTAS, so I confused that for the config not working. I fiddled with some things, got some errors, and then checked the docs and submitted a ticket with aws to confirm.
We can go ahead and close this out, haha

Ok! Feel free to comment back here if you figure out something that dbt could be doing better.

Using
models:
  +sort: auto
  +dist: auto

produces sortkey(auto) instead of sortkey auto. This is because the sort key property is treated as a column name, not an option.
I'd imagine you'd need a new config option sortkey_auto: true/false to support generating SORTKEY AUTO in the CTAS statement.

@matthewrj Ah, fair point! Thanks for finding this issue. I think a simpler solution would involve adjusting the sort macro so that, if 'sort == 'auto', the macro returns sortkey auto instead of sort auto.
https://github.com/dbt-labs/dbt/blob/2b48152da66dbd7f07272983bbc261f1b6924f20/plugins/redshift/dbt/include/redshift/macros/adapters.sql#L17-L29
I'd welcome a contribution for that, so I'll re-open the issue.

It looks like setting sortkey auto and diststyle auto is still not supported in a CTAS, though.
See: https://docs.aws.amazon.com/redshift/latest/dg/r_CREATE_TABLE_AS.html
This change would require an ALTER TABLE table ALTER DISTSTYLE AUTO, ALTER SORTKEY AUTO as a dynamic post-hook of some sort, or something.


Some thoughts about the way forward

:point_up: May 3, 2017 8:56 PM

Recent evolution on the questions raised by my original comment:
From @DarkThrone

I mean you can fight spam proactively or reactively
you can identify spam heuristically by it's content
or you can deter spam sources with anti-spam mechanisms
the first suffers from false positives, but the later adds friction

My response was as follows:

I think Sgritta is right in conceptualizing the Spam issue as false positives vs. Friction.
Friction is probably we're we'll come down because the resources to do content based spam filtering are not available to a small open source project. I mean, if we could solve the spam problem without Friction but Disqus can not then they would need much better engineers.

Realistically the strategy will come down to using something like email but trying to make the user experience as smooth as possible.
@DarkThrone, I would say that if email is not used then you can suggest different Friction mechanisms. It sounds like you know more about it.
To deal with Spam we can have email to verify user plus dynamic rate limiting.
Dynamic rate limiting means that a user can post 3 messages (or whatever number) quickly but subsequent messages will take progressively longer to send.

Then the question is, what if a user is continually abusive?
I like the idea of IP banning (and other additional means can be suggested too).
One downside of storing IPs is that users may not want the content of their messages stored along with the source of that message.
Because commento will be unique to page, a universally unique ID (UUID) can be produced and appended to a users IP then run through a cryptographic hash. That will make interating through all IPS not possible unless you have the UUID.
An uuid can be produced in golang with https://github.com/satori/go.uuid

What about stuff like universities where everyone is under the same IP?
In that case the full "IP" can be UUID + email + IP run through a cryptohash. This will be used for all users regardless of where they come from.
For the university example, you would have two separate users with the same IP but different emails. This would result in different hashed IPs so one would be banned and the other would not

Stuart Baker @smbkr asks

Without email verification though, wouldn't it be trivial for a user to get around the ban by just changing one char in the email they enter?

My response was

Sure, I think there should be something that is associated with a users identity. It could be an email address or just a site level username that will be verified with an email address
Also, we don't want to make the user enter the email on every comment so we should set a cookie with some values that identify the user.
JWTs allow encoding of user information and can be sent with each request. That would avoid the need to maintain session in commento which makes managing interaction with the service much easier. More stateless in other words.
We'll parse the JWT in middleware and it will contain userID info and other pieces of info that are useful (that can be decided as we go forward since it doesn't require them to be determined up front)

The tl;dr;
User uses email, or other mechanism, to 'log-in'. This results in getting a JWT sent back that the user puts in a cookie.
Abuse is handled by banning on two separate dimension.

User ban: Ban based on email address in JWT token
IP ban: Ban is based on Scrambled IP in JWT Token where Scrambled IP = IP + UUID.

There is a question as to whether the UUID should be page level or site level. If you want both than there will need to be separate pieces of data for each which will increase complexity. It is easier to ask whether it makes to ban someone from a page but not the site makes sense.

@GaiusMartius Just a heads up.
We've moved all development to GitLab. I think this issue is outdated, so I'm closing it. If you feel this is still relevant, please feel free to open a new issue in the GitLab repo issue tracker. Cheers! :)


SW Load Balancer media folder rename

If this PR needs approval, or if the commits need to be pulled into the private repo, contact @ManikaDhiman or @alkohli. Thanks.

#assign: @ManikaDhiman, @alkohli
@ManikaDhiman @alkohli

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

NOTE: All validation flags must be addressed before we can merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Could NOT find JNI (missing: JAVA_INCLUDE_PATH JAVA_INCLUDE_PATH2   JAVA_AWT_INCLUDE_PATH)

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment:

OS: [e.g. Ubuntu 18.04]  mac
CPU: [e.g. Intel Xeon xxxx]  Intel
Compiler: [e.g. GCC 8.0]

Additional context
Add any other context about the problem here.
CMake Error at /usr/local/Cellar/cmake/3.22.1/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
Could NOT find JNI (missing: JAVA_INCLUDE_PATH JAVA_INCLUDE_PATH2
JAVA_AWT_INCLUDE_PATH)
Call Stack (most recent call first):
/usr/local/Cellar/cmake/3.22.1/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
/usr/local/Cellar/cmake/3.22.1/share/cmake/Modules/FindJNI.cmake:382 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
src/BuildClients.cmake:18 (find_package)
src/CMakeLists.txt:10 (include)

Install java 1.8 in your os environment or use cmake -DENABLE_FULLTEXT_INDEX=OFF  to remove the dependency on java.
BTW,  it is recommended to use centos or ubuntu to compile,  mac os has not been verified.
some references:
https://github.com/TuGraph-db/tugraph-db/blob/master/ci/images/centos7/Dockerfile


cannot get dragNodes to work [using parsers.json and forceAtlas2]

I have been struggling with this for a while now.. the graph works fine returning data from a json file and running forceAtlas2.. but I cannot seem to get the correct combination of magic potion to get the dragNodes plugin to work.
I have imported everything that was in the examples.. here is my current code if someone could please point me in the right direction. I have tried several other locations to put the "dragListener" with no luck. The current error that is returned from the firefox console is "TypeError: invalid 'instanceof' operand sigma.renderers.svg which points to line 64 in sigma.plugins.dragNodes.js  "if (renderer instanceof sigma.renderers.svg) {"
current partial code:
sigma.renderers.def = sigma.renderers.canvas;
var g = {
nodes: [],
edges: []
};
s = new sigma({
graph: g,
container: 'containerSigma',
settings: {
minNodeSize: 1,
maxNodeSize: 10,
sideMargin: 1,
minEdgeSize: 0.01,
maxEdgeSize: 4,
defaultLabelSize: 9,
labelThreshold: 5,
}
});
	var thepath = "MY_SPECIAL_PATH";
	sigma.parsers.json(thepath, s, function() {
		var i,
				nodes = s.graph.nodes(),
				len = nodes.length,
				edges = s.graph.edges(),
				num_edges = edges.length;
		// Refresh the display:
		s.refresh();

		s.startForceAtlas2({worker:true});

		var sec_proc = 5000;
		//make sec_proc a function of # nodes and edges
		sec_proc = (((len + num_edges) / 1000))*1300 + sec_proc;

		setTimeout(function() {
				s.stopForceAtlas2();
                              //add dragListener after forceAtlas has finished
				var dragListener = sigma.plugins.dragNodes(s, s.renderers[0]);
			}, sec_proc);
	});

Hi amplify211,
I had this issue and after some looking around, it seems that the svg renderer was added since I downloaded my version of sigma.min.js. The plugin is therefore looking for a renderer that doesn't exist in my code. The simplest solution is to rebuild sigma.min.js as per the instructions or download a newer version of sigma.min.js.
Hope this helps.


[FEAT]: one-liner zero function call

Is it better if we call it in a one-liner?
import {zero} from '@zerosecrets/zero'
export async function main () {
const [awsSecrets, error] =await zero({
      token: process.env.ZERO_TOKEN,
      pick: ['aws'],
    })

awsSecrets ? console.log(awsSecrets) : console.error(error)
}

I still not working on the test, because it's the Proof of Concept I proposed.

@ImBIOS thank you for the PR! We would still prefer keeping the current notation, making code more explicit, even at the cost of extra LOCs.


⚠ Bofrost.at has degraded performance

In b8ff3f8, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1926 ms

Resolved: Bofrost.at performance has improved in eaa07ec.


Admob email asked to upgrade gma sdk for android and ios

as title
"必須更新至 GMA SDK 11.3.0 (iOS) 及 23.2.0 (Android) 版本"
update
translated: "Must be updated to GMA SDK 11.3.0 (iOS) and 23.2.0 (Android) versions"

Hi @peterweb2005, this feels like something to be raised in the AdMob developers form
Closing this out . Thanks!

@malandr2 no, now version
5.0.0
Updates GMA iOS dependency to 11.2.0
Updates GMA Android dependency to 23.0.0
so should bump sdk

Hi. I got that email too.
It says that if you use admob in integration with other google services like firebase, you need to update the GMA sdk version to ios(11.3.0), android(23.2.0).
I think you should update the GMA sdk for android and ios by yourself, not Admob developer.
//googleads-mobile-flutter/packages/google_mobile_ads/android/build.gradle
 api 'com.google.android.gms:play-services-ads:23.0.0' <<update to 23.2.0

//packages/google_mobile_ads/ios/google_mobile_ads.podspec
 s.dependency 'Google-Mobile-Ads-SDK','~> 11.2.0' <<update to 11.3.0

Please update it for users who use admob in integration with other Google services.
Thanks .


HHH-11482 Update example and documentation to clarify SUBSTRING is 1-based

See: https://hibernate.atlassian.net/browse/HHH-11482

Applied upstream, thanks


.NET Renderer currently registers each individual Action subtype vs the base Action

The AdaptiveCardRenderer (HTML and WPF) should render based on Action base type, not per Action name as designed today. Need to:

[ ] make sure the Native Platform styling works still
[ ] update docs with new style names

           ElementRenderers.Set<AdaptiveSubmitAction>(XamlSubmitAction.Render);
            ElementRenderers.Set<AdaptiveOpenUrlAction>(XamlOpenUrlAction.Render);
            ElementRenderers.Set<AdaptiveShowCardAction>(XamlShowCardAction.Render);

This is fixed with #870


Edit Peroject-Hub

Adding back ground image , Remove Unused links.
In style.css : Update css.

@kesha99 thanks for this work; looks great! The great thing about this project is that it's a template, which means that anyone can customize things to their liking, as you've demonstrated here. I'm going to leave things basic and simple so that users can customize to their hearts content. Thanks for sharing!


Install Configuration to Correct $HOME

Noticed while investigating #50
See: https://github.com/MattSurabian/proxpn-bash-client/blob/master/proxpn#L27-L31
See: https://github.com/MattSurabian/proxpn-bash-client/blob/master/install.sh#L5

Hey, did you at one point make a similar change to creating a sort of 'common environment' file, or think about making one, we can source from either script, so information like this can be consistent.  We could also have logic to 'fallback' to normal user metadata detection instead of temporary privilege escalation metadata.
Didn't think it through yet, just an idea.

I had thought about this but moved off it because I was hung up on the single file thing but you've talked me out of that. Sourcing an environment file makes a lot of sense and would have avoided this kind of problem.


linker: intel_s1000: Remove limits on code and data sections

All text, data and bss sections are all mapped to the same physical
memory (SRAM). This patch removes the individual section limits
and defines a common limit for the sum of text, data and bss sections.
This would make it more flexible for application developers.
Fixes #11268.
Signed-off-by: Rajavardhan Gundi<EMAIL_ADDRESS>

Codecov Report

Merging #11319 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #11319   +/-   ##
=======================================
  Coverage   51.55%   51.55%
=======================================
  Files         216      216
  Lines       27665    27665
  Branches     6937     6937
=======================================
  Hits        14263    14263
  Misses      10653    10653
  Partials     2749     2749

Continue to review full report at Codecov.

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

@sathishkuttan .. have addressed all your comments. Regarding __stack, let's wait for @andrewboie to get back to us.

I don't have good visibility on what __stack is used for, it's part of the port contributed by Cadence, but looking at crt1.S it seems to be where the stack pointer gets set to at early boot, before the kernel swaps into the main thread.
I suggest getting rid of this, and set the stack pointer to _interrupt_stack which is what all of our other arches do.

interrupt_stack_top

I would suggest we do this in a different PR as this is applicable for every SoC under Xtensa arch. Having said that, i am not sure that the usage of _stack is restricted to early boot. If you see the linker.ld file for each of these SoCs, there are different sections like ".comment", ".debug", ".line", ".debug*" etc and I am not able to understand the usage. I also looked at the corresponding zephyr.map and couldn't understand much. Can you please check and comment?

interrupt_stack_top

I would suggest we do this in a different PR as this is applicable for every SoC under Xtensa arch. Having said that, i am not sure that the usage of _stack is restricted to early boot. If you see the linker.ld file for each of these SoCs, there are different sections like ".comment", ".debug", ".line", ".debug*" etc and I am not able to understand the usage. I also looked at the corresponding zephyr.map and couldn't understand much. Can you please check and comment?

@sathishkuttan @andrewboie Can we defer this __stack to a later day? Coz, this is generic to all Xtensa SoCs and not specific to S1000. This patch anyways retains the existing behavior. We can may be open a github issue on __stack usage so that we can address it later.

Can we defer this __stack to a later day?

Yes, we can.


users: persistence (mutableUsers)

Persists user credentials.
Take note of the issue with using passwd which is explained in this thread https://github.com/NixOS/nixpkgs/issues/3192
It appears that a solution is to use passwd with --root to chroot into the correct directory. (this is mentioned in the description for this option).

It seems difficult to enable this option on a running system. Potentially it might be possible to do this when using nixos-install, but it's difficult after since nix needs /etc/passwd and /etc/shadow to activate.
https://github.com/nix-community/impermanence/pull/88 may be required to make this work properly.

Temporary workaround I used for testing this in the mean time
mv /etc/passwd /persistent/passwd
mv /etc/shadow /persistent/shadow
ln -s /nix/persistent/etc/passwd /etc/passwd
ln -s /nix/persistent/etc/shadow /etc/shadow


I get configuration of zk by spring-cloud-zookeeper,but the configuration will change.

I get configuration of zookeeper by spring-cloud-zookeeper,but the configuration will change.I don't want restart my application to get the changed value.how can i do?

Have you tried posting to /refresh?

/refresh? please get me some simple demo or document,thanks.

http://cloud.spring.io/spring-cloud-static/Camden.SR3/#_endpoints

@spencergibb Follow the document.no web environment in my application,so use refreshAll in RefreshScope class.but not get the changed value of zk.
configuration class
@Configuration
public class TinyLanguage {
@Bean
public LanguageBean getLanguageBean() {
return new LanguageBean();
}
}
bean class
@RefreshScope
public class LanguageBean {
@Value("${tiny_language}")
private String language;
public String getLanguage() {
    return language;
}

}

Hi @jonelee05 I believe that the problem is that you have the @RefreshScope annotation in the wrong place. You must have the annotation together with where you are declaring the bean. So you must do either of these:
@Bean
@RefreshScope
public LanguageBean getLanguageBean() {
    return new LanguageBean();
}

or
@RefreshScope
@Service
public class LanguageBean {

By the way, I don't think it's documented anywhere, but if you have the actuator dependency in the classpath, then you don't need to refresh manually as it does it automatically every time you update a property in Zookeeper. There is even a property to enable/disable it: spring.cloud.zookeeper.config.watcher.enabled. @spencergibb is this something that should be added to the docs?

@erecarte it's not documented

Actually, isn't it covered here in the core Spring Cloud docs:
http://cloud.spring.io/spring-cloud-static/Camden.SR5/#_refresh_scope
Granted, the docs don't provide code examples, but it is covered.

@erecarte  tks for you suggest.

Use spring cloud config，This question is not exist.


Add unit and integration tests

Add unit tests to raise coverage to 95%
Overview

Rationale

Juju Events Changes

Module Changes

Library Changes

Checklist

[x] The charm style guide was applied
[x] The contributing guide was applied
[x] The changes are compliant with ISD054 - Managing Charm Complexity
[x] The documentation is generated using src-docs
[x] The documentation for charmhub is updated
[x] The PR is tagged with appropriate label (urgent, trivial, complex)

Hi @gregory-schiano , could you take a look at this pr and leave your review? Thanks!

@gregory-schiano As discussed internally I've added additional assertions in the main integration tests. Can you take a look again and leave your review? Thanks!


Patch/fix doc urls in landing

ref https://github.com/stac-utils/stac-fastapi-pgstac/pull/101/files#r1576403075
This PR fix a bug introduced in #635
I've added tests (taken mostly from #625 )

Release v2.5.5?


cmp-tabnine: Cannot find installed TabNine. Please run install.sh

When starting nvim I get this cmp-tabnine: Cannot find installed TabNine. Please run install.sh
Installing plugins through packer
Running:
Nvim: 0.8.0
macOS: 12.6.1
iTerm2: 3.4.16 (this error appears in the default terminal as well)
zsh: 5.8.1

As the message says, try running the install.sh script by hand.


Attributes still are being ignored in iOS v1.4.0

Hi @tshedor ,
Even with the latest update that has a commit regarding fixing the attributes in iOS, attributes are not still passed.
I have a sample attribute with a value like this:
{    "key1":"value1",    "key2":"value2" }
it works fine if the attributes are only simple values, but passing a dictionary will break the following cast in iOS code:
  return unencodedAttributes as? [AttributeKey : AttributeValue]](https://github.com/GetDutchie/datadog_flutter/blob/787890c3d43961a4b07b0102fb7fbd8f64a6c915/ios/Classes/SwiftDatadogFlutterPlugin.swift#L347)
I understand that you mentioned in the comments that dictionary attributes support are limited. Still, since Datadog supports up to 10 level inner attributes, I think it is worth adding.
Would you please let me know if I could be any of help adding this?
Thanks :)

Hey @siavash-mohseni thanks for using this package.
That's an interesting bug. So, ideally, nested dictionaries are handled with this recursive function. Are you passing in null values by any chance? Can you provide an example of an attribute payload that fails?
And I'd love the help if you wanted to work around that statement. My other thought is that your nested attributes don't have String keys and thus there is no recursion.

Hi @tshedor,
Thanks for your answer.
I think my attributes are fairly simple. Something like this:
{ 	"exception": "exception message", 	"stackTrace": "stack trace string", 	"context": { 		"key1": "value1", 		"key2": "value2" 	} }
None of the values are null and all the keys are Strings.
I debugged the code and first time in recursive function it is coverting the context successfully to [AttributeKey : AttributeValue] but when it tries to convert the whole attributes and exit the function it return null.
It seems that the conversion is not successful when you have a nested dictionary in attributes here:
  return unencodedAttributes as? [AttributeKey : AttributeValue]](https://github.com/GetDutchie/datadog_flutter/blob/787890c3d43961a4b07b0102fb7fbd8f64a6c915/ios/Classes/SwiftDatadogFlutterPlugin.swift#L347)
If I remove the context node, or replace its value with a String it will work successfully.
I'm not a Swift expert, but it seems to me that the nested dictionary is not being considered a Codable type and Swift is not able to convert it accordingly.
Thanks a lot 👍

I am having a similar issue with List<String> custom attributes propagating to datadog for iOS devices when added to the user with the extraInfo parameter.
When I add attributes to the user like:
    await DatadogFlutter.setUserInfo(id: user.id, extraInfo: {
      'organizations': ['demo', 'demo_probation'],
    });

the attributes show up as Custom Attributes for Android devices but not for iOS.


visualization of system.opentelemetry_span_log

It would be really useful and will great improvement for TUI interface

Hm, can you give an example when it was useful for you?

quick found some unexpected url() function execution via VIEW
during prepare clickhouse-grafana release which support Grafana Tracing View

also watch how CONSUMER and PRODUCER works
i understand not all operation covered by tracing spans, but just want to watch in some cases


SDS startup optimizations

This includes a few changes that all work together to bring the pod
startup p90 times from 20s to 4s

If GenerateSecret is called, return cached key when possible
Use "bootstrap" as the connection ID, allowing us to use cached key
Create a single Cluster in the bootstrap for SDS, rather than inlining
the config. This has the added benefit of substantially reducing the
size of XDS resources. This requires a change from google_grpc to
envoy_grpc, but this is fine as we now support local JWTs.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
79e0bf1691e5d1b1d120fd389d3defab9fd66e20
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
fbc0cb2b8a8bc12eb2f58037a1ea5b23bd1b35b8
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
fbc0cb2b8a8bc12eb2f58037a1ea5b23bd1b35b8
link
/test unit-tests_istio

integ-istioio-k8s-tests_istio
fbc0cb2b8a8bc12eb2f58037a1ea5b23bd1b35b8
link
/test integ-istioio-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-istioio-k8s-tests_istio

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
fbc0cb2b8a8bc12eb2f58037a1ea5b23bd1b35b8
link
/test unit-tests_istio

integ-istioio-k8s-tests_istio
fbc0cb2b8a8bc12eb2f58037a1ea5b23bd1b35b8
link
/test integ-istioio-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-istioio-k8s-tests_istio
not sure if flaky test or real failure.
TODO before merge: try out the postsubmit only security tests

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
fbc0cb2b8a8bc12eb2f58037a1ea5b23bd1b35b8
link
/test unit-tests_istio

integ-istioio-k8s-tests_istio
fbc0cb2b8a8bc12eb2f58037a1ea5b23bd1b35b8
link
/test integ-istioio-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-istioio-k8s-tests_istio
it works locally..

ooh I am not running my changes locally, of course it passes :man_facepalming:

ingress SDS has a bug, but sidecar should be good

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
fbc0cb2b8a8bc12eb2f58037a1ea5b23bd1b35b8
link
/test unit-tests_istio

integ-istioio-k8s-tests_istio
fbc0cb2b8a8bc12eb2f58037a1ea5b23bd1b35b8
link
/test integ-istioio-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio
fbc0cb2b8a8bc12eb2f58037a1ea5b23bd1b35b8
link
/test integ-istioio-k8s-tests_istio

unit-tests_istio
663cc270bd59d2e414e0a91f11321bd1b565ef52
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
663cc270bd59d2e414e0a91f11321bd1b565ef52
link
/test unit-tests_istio

integ-istioio-k8s-tests_istio
663cc270bd59d2e414e0a91f11321bd1b565ef52
link
/test integ-istioio-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I am splitting this out to 2 PRs


bug: BoringdroidSystemUI disappears after changing display theme from settings.

We need reload BoringdroidSystemUI after changing display theme.

Commit: https://github.com/boringdroid/platform_frameworks_base/commit/b017e9dc7a73a42e1b69ed00694c9a35354c07ae.


Updated Figma Design

Description
I have made new changes to the Figma design.
I added an importance meter,
A new checkbox to mark the task as done,
Due date box,
Pop-up screen to congratulate the user on task completion.

With the importance meter, you can now set the task to Low, Moderate, and High Importance. This importance will be marked on the task.
The checkbox now shows a green tick on completion.
The due date box is not yet complete.
The pop-up screen congratulates the user after the task is completed.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works

Screenshots (if appropriate):

Thanks, @briyandyju09
This is already open in #82.
You can consider co-authoring instead.


MANY MANY PROBLEMS

After last update many incoming problems :(
No auto repair, no zoom and many other little things :(
Is it possible to fix it as soon as possible?
Thanks a lot for all this work
Regards
Davide

some updates are at cncscripts.com

updated

Thanks a lot for it!


Test to see if workflow runs from PR

test to see if runs workflow from the PR

Summary

copilot:summary
Details

copliot:walkthrough

it does not


Make tests run depending on available backends

Hi! This is a PR to prepare tests refactoring so that they could run with only available backends.
Now tests should run only with available backends, ie., for e.g., if relay backend is available and pytorch is not availabe, then tests should run only with relay.
However, I don't know how to check it with contiguous integration.
At least, now, I want to check if current tests will pass. We could also use this PR to discuss how we must re-factorize the tests. What do you think ? @abergeron @breuleux

@breuleux tests passed ! I just updated the coverage settings.

Hi @breuleux @abergeron ! I am rebasing and updating this PR, and I have activated testing for all available backends in many test files, including test_grad. But, now, I get an error with relay backend for test test_list_while in tes_grad: https://github.com/notoraptor/myia/blob/module-tests/tests/test_grad.py#L380
Function default_convert in ressource module receives a tvm.runtime.ADT object and is then unable to convert it: https://github.com/notoraptor/myia/blob/module-tests/myia/pipeline/resources.py#L57
It seems the ADT object is generated by relay backend from a tagged value and then passed to myia VM which is unable to handle it. I don't yet know how to fix this bug.

E   TypeError: Traceback (most recent call last):
E     [bt] (5) /home/notoraptor/anaconda3/envs/myia/lib/libtvm.so(TVMFuncCall+0x65) [0x7f1835e2f745]
E     [bt] (4) /home/notoraptor/anaconda3/envs/myia/lib/libtvm.so(+0x6962d4) [0x7f183550d2d4]
E     [bt] (3) /home/notoraptor/anaconda3/envs/myia/lib/libtvm.so(+0x6983c2) [0x7f183550f3c2]
E     [bt] (2) /home/notoraptor/anaconda3/envs/myia/lib/libtvm.so(tvm::ReflectionVTable::GetAttr(tvm::runtime::Object*, tvm::runtime::String const&) const+0x1b1) [0x7f183550eae1]
E     [bt] (1) /home/notoraptor/anaconda3/envs/myia/lib/libtvm.so(+0x697903) [0x7f183550e903]
E     [bt] (0) /home/notoraptor/anaconda3/envs/myia/lib/libtvm.so(+0x69748f) [0x7f183550e48f]
E     File "/home/user/conda-bld/tvm-libs_1596554067552/work/include/tvm/node/reflection.h", line 390
E   TypeError: runtime.ADT is not registered via TVM_REGISTER_NODE_TYPE

tvm/_ffi/_cython/./base.pxi:160: TypeError
========================================================================================= short test summary info =========================================================================================
FAILED tests/test_grad.py::test_list_while[relay-cpu-grad] - TypeError: Traceback (most recent call last):
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! stopping after 1 failures !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
======================================================================================= 1 failed, 1 passed in 6.49s =======================================================================================

Hi @breuleux @abergeron Ultimately, it seems test_list_while is the only one remaining failing test, so I just skipped it for realy backend (I added a way to run "all backends except given ones").
I checked the relay pattern matching, but it seems it detects the match at compile time ( https://tvm.apache.org/docs/langref/relay_pattern.html ). I did not make further research to check if it can wait for runtime before running.
Currently, all tests pass, but now I have a strange sphinx doc error.

@notoraptor The sphinx doc error is likely from the __doc__ that ovld generates automatically. Try adding a docstring to the first function in to_abstract_test, if that doesn't work we might need to merge #375 first because I solved some at least one of the doc issues in ovld 0.2.x but because of breaking changes these versions require a few adjustments that are in #375 (this PR is limited to ovld 0.1.9).

@breuleux I added a docstring, but it does not seem to fix the problem. It's strange, because some other uses of ovld in myia code don't generate such errors.


Updates to plotting page: adds events table, fixes unclosed div tags,…

… lines up event toggle button
@DanielJMaher , @birdage , @Bobfrat
Updates to the plotting page to include the events table.

DO NOT MERGE YET

OK TO MERGE!!!

@DanielJMaher , don't forget about this!! goes with that last merge

@DanielJMaher looks like you didnt read it....

@birdage What is the issue?

no issue :+1: good to go!

@birdage confusing


Feature/one portal behaviour

Maciej moved the materials inside of the Resources folder, this is approved

Reminder: https://github.com/oneleif/BunnyHopMaster


Language Missing

Hi,
Love this tool.. Currently i'm trying to decry-pt an app but Jadx wasn't able to reverse Hindi Language,
Just Square Boxes..

@dbzdivik can you try to change font to one which supports these characters?
Look like default font used by jadx not support unicode at all :(

@skylot Will Check btw Android Studio was able to de-compile correct Text..
Also One More Issue in with Preference Menu

@dbzdivik can you check latest unstable build?
I have reset default font to system font as it was before, hope it will help. Anyway you can always change font to one which supports your language.


Signed URL from Storage.get() call not working

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
No response
Amplify CLI Version
5.1.2
What operating system are you using?
Ubuntu
Amplify Categories
storage
Amplify Commands
Not applicable
Describe the bug
I am trying to have profile image upload functionality for users in my application.
Storage.put(filename, uri, {
    level: 'public',
    contentType: 'image/*'
});
const imageUrl = await Storage.get(filename) as Promise<string>;

The above two function calls are working. I checked that the image got stored in the S3 bucket. However when I try to open the url imageUrl, it shows AccessDenied. I want to know how this is possible and how I can solve the issue. If the user was able to put the object and get it, why was it not able to show the image?
Expected behavior
The image url should be able to return image.
Reproduction steps

Available in description.

GraphQL schema(s)
No response
Log output
No response
Additional information
No response

It sounds like an Amplify JS issue, transferring to their repository.


Hitting "left of '=' is not an l-value" when trying to write to UAV.

The issue reproduces with the compute shader from Falcor on the latest:
https://github.com/NVIDIAGameWorks/Falcor/blob/55f943ef35ee752f368fc14fbc6352818af48367/Samples/Core/ComputeShader/Data/compute.hlsl
Any write to a UAV of the pattern:
uavTexture[coord] = foo;
fails with this error.

I'm seeing it happen in the latest master release.
I also am able to get around the issue by explicitly swizzling:
m_uav[coords].rgba = color; // this works.
m_uav[coords] = color; // this doesn't.
I also attempted to debug and it looks like the left and right arguments of the assignment expression are swapped. I wasn't able to make much headway though because the parser is pretty opaque.

Okay. I’ll take a look at it. Thanks for identifying this issue.
 That is probably the earliest I’ll be able to get this debugged and implemented.

This is resolved in Falcor's development branch, which will be merged into master in a few weeks.
There's a workaround and I prefer not to do a validation cycle for master at the moment.

It's fine, I'm not blocked by it.

@zbethel Okay, I've triaged this a bit and I think I know what is going on. I'm sorry I spoke incorrectly about what Slang v0.9.12 supported.
Just to confirm, it seems like you are either:

writing your shader code in a file with the .slang suffix, or
writing your code in a .hlsl file and then running on Vulkan

If that is the case, then I'm sorry to say that neither of those is an officially supported use case for Falcor users. Some of the built-in samples use .slang files just because it is easier for the Falcor development team, but in general users should be writing .hlsl files for D3D12 and .glsl files for Vulkan.
The best fix for now will be for you to either write .hlsl or .glsl shaders, depending on the API you are targetting. A future Falcor release will officially support .slang shaders from users, but we aren't quite ready for that today.

To confirm, it's the former.
I'm actually not using this for Falcor, it's for an internal project. My main reason for using it is that I need a quick API that lets me reflect HLSL and group them into the ParameterBlocks.
One potentially solution here could be to run my parameter block include shaders through the .slang compiler, expose them via HLSL function getters, and then write the bulk of the rest of my code via HLSL.

Ah, okay. If you are using Slang directly then I can try to push a new release for you with a fix.

I have a question. Is it possible to run the slang compiler on certain #includes, and then run my main shader as HLSL?
I would like to use the ParameterBlock<> semantics to groups of resources, but I'm finding that the slang compiler itself is not compatible with the HLSL code I currently have.
My thinking is to create .slang include files which define the ParameterBlock<> definition, and then utilize getters GetObject_ModelToWorld(...) for example.
Is this possible? I would need to be able to run the slang compiler on .slang includes, but run the hlsl compiler on the rest.

Release/tag v0.9.13 should fix the original issue in this request.
To your other question, the releases of Slang so far support a -no-checking flag that turns off most semantic checks (but is only respected on .hlsl input files). This is what Falcor currently uses. That might work for your use case.
Looking ahead, though, Slang is dropping support for that flag, and the plan is to only support “full” compilation with all-Slang input. The goal is to eventually support as much of HLSL as people need in practice, but I can understand why the current level of support could be frustrating.
If you want to contact me directly (either DM @TangentVector on Twitter, or contact tfoley at nvidia.com) I’d be interested to hear about your use case and whether it aligns with where the Slang project is headed.

Oh yeah, you had a more specific question:

For clarity, my specific request is whether I'm able to run the slang code-generator over a file without having to specify an entry point, as this would allow me to process my dependent include files separately from the main source file.

You can currently feed code into Slang without an entry point, and get reflection information back, but in that case you aren’t going to get any output HLSL (output code is per-entry-point). I don’t know if that is useful to you or not.
I suspect that for the scenario you are describing, you probably want go write your entry points as HLSL files that import Slang files with your parameter declarations, and then feed those HLSL files into Slang with the -no-checking flag. That should give you more or less what you want.
We’ve dropped that whole model in our top-of-tree code, though, because it was adding a lot of implementation complexity, and it would never work for cross-compilation. The future of Slang is going to be its new IR-based compilation flow, and we will treat any HLSL we can’t understand as a bug (which isn’t promising that we will fix every such bug).

Great, thanks for the clarification! :)

I think the original problem here is addressed, so I'm going to close this issue.


sanitize public key content

Fix https://github.com/dice-cyfronet/atmosphere/issues/259

Please fix errors reported by hound

I believe that regexp can be used here and first valid ssh public key can be taken. It should simplify code.

:+1: but why Gemfile.lock is changed?

Gemfile.lock modified by accident. Reverted this change.


Add autoload configuration

Closes #5

I can't test it until wednesday when I arrive at office, but looks good! Thank you!


Make Scatter colors functional

Currently setting the color attribute does nothing.

I don't think you fixed this one - I wanted to make Scatter work with colors - looks like you removed the color option instead. Your change is good until I get this resolved, but I'm reopening this issue.

Fixed in #30


Many rule allow unwanted additionalProperties

I see over 25 rules with
[k: string]: any;

which disappear if json-schema-to-typescript is configured with { additionalProperties: /* default to: */ false }
I think at least 75% of that is schema typos, and we should use { additionalProperties: /* default to: */ false }
If the type actually has additionalProperties it is always written in type (as Record types obviously require them to exist), while when it hasn't it may be just be forgotten

Oh nice there is a property for that? wow? This could actually fix all our issues :tada:

I will check that tomorrow or you already did a PR

I didnt

I didnt

I meant: I will do a PR on my own tomorrow if you did not already send a PR from your side
sorry for misunderstandment :sweat_smile:
I think english is not the native language for both of us :sweat_smile:

Other things to fix are
        ignore?: (string & { // string:not(...):not(...), should be just string here
          [k: string]: any;
        } & {
          [k: string]: any;
        })[];

and
        ignorePatterns?: any[]; // mostly string[]

and maybe a couple of annoying TS types that depend on ESLint types

I didnt

I meant: I will do a PR on my own tomorrow if you did not already send a PR from your side
sorry for misunderstandment 😅 I think english is not the native language for both of us 😅

In my language "I will check that tomorrow or you already did a PR" would be read as a question, so I answered that 😅

Some context-sensitive types like
export type regex = string & { _?: regex }; // string that is used as RegExp pattern
export type glob = string & { _?: regex }; // string which is used as glob pattern

could also be (very) useful, but that's hard to mantain

I tried to configure additionalProperties to true and false but nothing changed
https://github.com/Shinigami92/eslint-define-config/blob/c8489871f1e409f306cd290051a09d4773ebb807/scripts/generate-rule-files/src/json-schema-to-ts.ts#L26-L34
So I think we will just use patching again
But we can apply this patch generally like we did here
https://github.com/Shinigami92/eslint-define-config/blob/c8489871f1e409f306cd290051a09d4773ebb807/scripts/generate-rule-files/src/json-schema-to-ts.ts#L8-L11

@Dimava The [k: string]: any; itself is not a problem at all, it just means that other properties can be added to the object and mostly if you need that you should check the website documentation of the rule.
Only stuff like string & otherStuff is something like a bug and we might need to report that back to json-schema-to-typescript repo
But for now I fixed it with #190

plugin types are now supported via @eslint-type/*
if there are any additionalProperties I think they need to be fixed in their respective meta schemas in the specific plugin


Docs: Website deployment task

The current v2.0 branch has a build task for the docs — gulp docs-build
Investigate if we need additional task(s) that will be run by Bamboo for various environments. A --production (or environment) flag may be employed to enable additional minification and optimizations if applicable.
The docs build task currently calls pkg-build first which works OK for local development but is not ideal. We should consider decoupling these. Watch tasks handle compilation for development purposes and the default sequence can be changed to run pkg-build first.
Consider pulling Techne from bower in the deployment scenario. The local build process compiles the library in the project and copies it to www/css directory. This could introduce differences in the website version of Techne vs. the published packaged version.
For each numbered releases, this would necessitate a publishing order:

Publish the package
Bump the Techne bower version in the project
Deploy the site

Related rot #617

@xak You can use this one now. It was ready a month ago, merged and unmerged.
issue https://github.com/SAP/techne/issues/681, pr https://github.com/SAP/techne/pull/682

Blocked by #617

no longer valid.


python311Packages.emoji: 2.5.1 -> 2.6.0

Diff: https://github.com/carpedm20/emoji/compare/refs/tags/v2.5.1...v2.6.0
Changelog: https://github.com/carpedm20/emoji/blob/v2.6.0/CHANGES.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 240676 run on x86_64-linux 1

2 packages marked as broken and skipped:

python310Packages.dalle-mini
python310Packages.dalle-mini.dist

48 packages built:

deltachat-cursed
deltachat-cursed.dist
dinghy (python310Packages.dinghy)
dinghy.dist (python310Packages.dinghy.dist)
home-assistant-component-tests.google
home-assistant-component-tests.local_calendar
home-assistant-component-tests.rainbird
komikku
libretranslate (python310Packages.libretranslate)
libretranslate.dist (python310Packages.libretranslate.dist)
ntfy
ntfy.dist
octoprint
octoprint.dist
opsdroid
opsdroid.dist
python310Packages.argos-translate-files
python310Packages.argos-translate-files.dist
python310Packages.argostranslate
python310Packages.argostranslate.dist
python310Packages.emoji
python310Packages.emoji.dist
python310Packages.gcal-sync
python310Packages.gcal-sync.dist
python310Packages.ical
python310Packages.ical.dist
python310Packages.locationsharinglib
python310Packages.locationsharinglib.dist
python310Packages.pyrainbird
python310Packages.pyrainbird.dist
python310Packages.stanza
python310Packages.stanza.dist
python310Packages.translatehtml
python310Packages.translatehtml.dist
python311Packages.dinghy
python311Packages.dinghy.dist
python311Packages.emoji
python311Packages.emoji.dist
python311Packages.gcal-sync
python311Packages.gcal-sync.dist
python311Packages.ical
python311Packages.ical.dist
python311Packages.locationsharinglib
python311Packages.locationsharinglib.dist
python311Packages.pyrainbird
python311Packages.pyrainbird.dist
terraform-compliance
terraform-compliance.dist


QG checks (Release 24.05)

QG checks
Please keep this issue open until QG is concluded and will be managed by the Issue Creator!
We will inform you about finding and proposals in separated issues, this issue here is for the Overview of the Checks!
Please keep this issue open until QG is concluded!
Product Owner:
Dev SPOC: @tom-rm-meyer-ISST
Helm Chart Version: tbd
App Version: 2.0.0
Release Managemnet Reference Issue: eclipse-tractusx/sig-release#589
Check of Tractus-X Release Guidelines

Currently implemented automatic checks can be found under your product on our Release Guidelines Checks Board
This QG Check is depending on the mandatory information from our current Release Guidelines

TRG 1 Documentation

[x] TRG 1.01 appropriate README.md
[x] #385
[x] TRG 1.03 appropriate CHANGELOG.md
[x] TRG 1.04 editable static files

TRG 2 Git

[x] TRG 2.01 default branch is named main
[x] TRG 2.03 repository structure
[x] TRG 2.04 leading product repository
[x] TRG 2.05 .tractusx metafile in a proper format

TRG 3 Kubernetes

[x] TRG 3.02 persistent volume and persistent volume claim is used when needed

TRG 4 Container

[x] TRG 4.01 semantic versioning and tagging
[x] TRG 4.02 base image is agreed
[x] TRG 4.03 image has USER command and Non Root Container
[x] TRG 4.05 released image must be placed in DockerHub, remove GHCR references
[x] #386

TRG 5 Helm

[x] #387
[x] TRG 5.02 Helm chart location in /charts directory and correct structure
[x] TRG 5.03 proper version strategy
[x] TRG 5.04 CPU / MEM resource requests and limits and are properly set
[x] TRG 5.06 Application must be configurable through the Helm chart
[x] TRG 5.07 Dependencies are present and properly configured in the Chart.yaml
[x] TRG 5.08 Product has a single deployable helm chart that contains all components
[x] TRG 5.09 Helm Test running properly
[x] TRG 5.10 Products need to support 3 versions at a time
[x] TRG 5.11 Upgradeability

TRG 6 Released Helm Chart

[x] TRG 6.01 Released Helm Chart

TRG 7 Open Source Governance

[x] TRG 7.01 Legal Documentation
[x] TRG 7.02 License and copyright header
[x] TRG 7.03 IP checks for project content
[x] TRG 7.04 IP checks for 3rd party content
[x] #388
[x] TRG 7.06 Legal information for end user content
[x] TRG 7.07 Legal notice for documentation
[x] TRG 7.08 Legal notice for KIT documentation

TRG 8 Security

[x] TRG 8.01 Mitigate high and above findings in CodeQL
[x] TRG 8.02 Mitigate high and above findings in KICS
[x] TRG 8.03 Mitigate high and above findings in GitGuardian
[x] TRG 8.04 Mitigate high and above findings in Trivy

Hints
Information Sharing

@evegufy please start with the review :)

@RoKrish14 could you please check the Security issues?

Hi @tom-rm-meyer-ISST as discussed, I created issues we discussed that it should be still added for the release, with the exception of TRG 7.06 Legal information for end user content, which is still WIP.
In the following some general recommadations:
Futher recommandation
- Readme --> Remove Catena-X

- Changelog --> remove dead links

https://github.com/eclipse-tractusx/puris/blob/2.0.0/CHANGELOG.md
https://github.com/eclipse-tractusx/puris/releases/tag/v2.0.0
- Only create secret if not .Values.existingSecret

https://github.com/eclipse-tractusx/puris/blob/2.0.0/charts/puris/templates/backend-secrets.yaml
https://github.com/eclipse-tractusx/puris/blob/2.0.0/charts/puris/values.yaml#L362
https://github.com/eclipse-tractusx/ssi-credential-issuer/blob/v1.0.0-rc.4/charts/ssi-credential-issuer/templates/secret.yaml
https://github.com/eclipse-tractusx/ssi-credential-issuer/blob/v1.0.0-rc.4/charts/ssi-credential-issuer/values.yaml#L161
- Build images for arm64

https://hub.docker.com/r/tractusx/app-puris-frontend/tags
https://hub.docker.com/r/tractusx/app-puris-backend/tags
See https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/pull/755/files
Why: local sandbox
-Add contribution details (information about how to fork, etc) https://github.com/eclipse-tractusx/puris/blob/2.0.0/CONTRIBUTING.md

Hey @evegufy with #405 we finally fixed the legal stuff in 7.05. Could you please confirm the successfull release checks?

QG review completed


Compilation issue with gnucobol

As a challenge for myself, I am currently doing the AOC-2020 in both C and bash, and wanted to give Cobol a try. As a starting point, I had a look on your source to understand the basics to read a file (I totally forgot the basics). I got your example for day1/exercise 2, and tried to compile (gnucobol version 2.2.0).
$ cobc -x ex2-galoisgirl-cob.cob
ex2-galoisgirl-cob.cob: 2: error: invalid PROGRAM-ID 'Advent-of-Code-2020-day-1-part-2' - length is < 1 or > 31

The program-id seems too long, so I changed it to "AoC-2020-day-1-part-2". I was able to compile it, I did put input data in a "d1.input" file, but the result was wrong:
$ ./ex2-galoisgirl-cob 2>&1
011216496250

After adding some output, I noticed that the way you read the numbers into a 9(4) array may be a problem, especially the line:
COMPUTE WS-ARRAY(I) = INPUTRECORD. Apparently when numbers are less than 1,000, they are parsed wrongly. For instance, the number '123' would be parsed as '1230' into WS-ARRAY.
My question is: If you use gnucobol (cobc), is there some compilation option to make this line work ? Or did you change the input file, for example by left padding with zero(s) ?
Meantime, I made a new version, replacing the above line with UNSTRING INPUTRECORD INTO WS-ARRAY(I). The result was then correct.
If you are interested, here is my own version: https://github.com/braoult/Coding/blob/master/AdventOfCode2020/d01b.cob .
Main changes: changed input file to stdin, and implemented a simple sort on array (bubble sort), to speed-up the main loop.

Happy to see someone is trying these!

My question is: If you use gnucobol (cobc), is there some compilation option to make this line work ? Or did you change the input file, for example by left padding with zero(s) ?

I did pad the input on day 1 so that every line has 4 digits - I should add a comment about that. I dealt with variable length input since day 2 (RECORD IS VARYING IN SIZE etc.), and by day 9 I'm dealing with variable length numbers properly - I'm guessing the magic is done in MOVE INPUTRECORD TO WS-NUMBER with INPUTRECORD PIC X(16) and WS-NUMBER PIC 9(16). Maybe MOVE accounts for the spaces and COMPUTE does not? I'm not sure.

The program-id seems too long

It's possible I added the "-part-1" as I was moving to part 2, so I didn't run it anymore.
Thanks for the remarks, I'll try and commit fixes when I have a minute.

Thanks,
As I am very new to Cobol (I learned it a little at school 40 years ago), I am far to be fluent in this language.
But I was really happy to make a 1st Cobol program for so long time !
Other topic: If you have interest in both Bash and C solutions for AoC, I just created a repo for my own implementations for days 1-12 : https://github.com/braoult/adventofcode-2020

Fixed in 025decbc609fbd9c7dacbf8d058d86553c3b4e60

Fixed in 025decbc609fbd9c7dacbf8d058d86553c3b4e60


Basic code for discretizing non-uniform grid

@stevenzhangdx  I am starting on this, but have added you as an observer.  May hand over to you for testing/etc.

This code was almost complete.  Take a look at the new discretize_nonuniform_univariate_diffusion_test file which has a few barebones tests.  I have it generating the matrices, but I didn't check the results to see if they accurately match anything calculated by hand.  Can you do that?  I would start by going through the discretize_univariate_diffusion.m file to see if they code matches the aglebra.  After, add in a few extra test cases for different drifts/etc to make sure that this works.

Sure, I will manually check results later. I am getting busy in the following two days. Probably, I will do new tasks during this weekend. Is it okay for you?

Perfect.  Thanks for the time update!
On Wed, Nov 1, 2017 at 2:30 PM stevenzhangdx<EMAIL_ADDRESS>wrote:

Sure, I will manually check results later. I am getting busy in the
following two days. Probably, I will do new tasks during this weekend. Is
it okay for you?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/econtoolkit/continuous_time_methods/issues/28#issuecomment-341249163,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMf-IZQ_bjPSFrhSC8KQyQverE2mA9qLks5syOMPgaJpZM4QOpWx
.

@stevenzhangdx @sevhou  I am adding in Sev, who will work with you to track down the problems

I think you guys verified this.

yep we did


Unable to trigger drag event after clicking any draggable element with PointerSendor

Example:
https://codesandbox.io/s/test-dnd-kit-forked-myy7t
Every tag is draggable until you click one of them.
However it used to work with an old version of this library:
https://codesandbox.io/s/test-dnd-kit-forked-qyjw2

Seems like a regression in @dnd-kit/core, investigating

Fix in progress here https://github.com/clauderic/dnd-kit/pull/217

Fix has shipped with<EMAIL_ADDRESS>confirmed that it solves the issue:
https://user-images.githubusercontent.com/1416436/115492706-4d10ab00-a230-11eb-866c-3e179bf19158.mp4

I wanted to update the CodeSandbox to show you the issue being fixed there, but there's a bug with the package version resolution of CodeSandbox 🙈  It's resolving to<EMAIL_ADDRESS>even if you specify ^3.0.1; this bug only happens in CodeSandbox, not if you download the code and run it locally.
Opened an issue here https://github.com/codesandbox/codesandbox-client/issues/5687

@clauderic Thanks, that was really quick! ✨
Tested on my side and it's working now 👍


SF6 Freecam, Disable Character Movement option not working

Basically the Disable Character Movement option is not doing anything, I have the latest version of REFramework, but it just doesn't want to stop the characters from moving
Any idea why?

Yes, it's there by default in every implementation of REFramework. I've removed the option now in the latest version.
I suggest you come over to the Modding Haven Discord, alphaZomega has a Lua freecam here with more functionality than base REFramework for SF6.


[SPARK-6317][SQL]Fixed HIVE console startup issue

Can one of the admins verify this patch?

ok to test

Test build #28562 has started for   PR 5011 at commit b4f5453.

This patch merges cleanly.

LGTM, except the new import should sort one line below now. If you get to it, great, otherwise I can fix on merge.

I'll update it now.
Thanks
On Mar 13, 2015 6:45 PM, "Sean Owen"<EMAIL_ADDRESS>wrote:

LGTM, except the new import should sort one line below now. If you get to
it, great, otherwise I can fix on merge.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/5011#issuecomment-78965288.

Test build #28566 has started for   PR 5011 at commit b43925f.

This patch merges cleanly.

Test build #28562 has finished for   PR 5011 at commit b4f5453.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28562/
Test PASSed.

If DSL support is already removed from 1.3 release, I shall modify the file and submit it.

Test build #28566 has finished for   PR 5011 at commit b43925f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28566/
Test PASSed.

LGTM, merging to master. Thanks!


Major pipenv uninstall --all behavior change

Issue description
Running pipenv uninstall --all (the command) with the latest version of pipenv uninstalls all packages in the [packages] group and removes them from the Pipfile. It doesn't uninstall packages from any other group.
Expected result
I had been relying on the behavior last seen in Release v2023.12.1, where running the command produced output like:
Un-installing all [dev-packages] and [packages]...
Found 47 installed package(s), purging...
Found existing installation: six 1.16.0
Uninstalling six-1.16.0:
  Successfully uninstalled six-1.16.0
Found existing installation: Jinja2 3.1.4
Uninstalling Jinja2-3.1.4:
  Successfully uninstalled Jinja2-3.1.4
Found existing installation: cryptography 42.0.8
Uninstalling cryptography-42.0.8:
  Successfully uninstalled cryptography-42.0.8
Found existing installation: packaging 24.1
...
Environment now purged and fresh!

There's a minor bug in that output in that it only lists that it's uninstalling from the [dev-packages] and [packages] group, but that's just a display issue. It does actually uninstall all packages in all groups from the virtual environment and leaves the Pipfile untouched. This is consistent with the documentation.
Actual result
Un-installing all [packages]...
Uninstalling src...
Found existing installation: src 1.0
Uninstalling src-1.0:
  Successfully uninstalled src-1.0

For this project, the [packages] group only contains one package, src-1.0. All the other packages are in either [dev-packages] or [layer-packages]. The current version of pipenv only uninstalls packages from the [packages] group and removes them from the Pipfile as well.
Steps to replicate
Create a Pipfile with several groups:
[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

# For local development, run
# pipenv install --categories="packages layer-packages dev-packages"

[packages]
src = {editable = true, path = "./src"}

[requires]
python_version = "3.11"

[dev-packages]
moto = ">=5.0.0"
pytest = "*"
pytest-monitor = "*"
mypy = "*"
boto3-stubs = "*"
pystache = "*"
types-requests = "*"
pytest-mock = "*"

[layer-packages]
boto3 = "*"
urllib3 = "*"
botocore = "*"
appsync-client = {path = "./../../../../../packages/appsync_client"}
pandas = "2.2.2"
openpyxl = "3.1.5"
aws-lambda-powertools = "*"

Install all of the packages:
pipenv install --categories="packages layer-packages dev-packages"

Now run the command:
pipenv uninstall --all

Examine the Pipfile, and see that the [packages] group is empty, but the other groups are still populated. Examine the site-packages in the virtualenv and note that the all group packages are still there.
Notes on the code
It appears that the do_purge function is now an orphan. There's no active code that will display Environment now purged and fresh!.

My best guess is that this bug was introduced in this PR.

Bummer I missed this -- if someone wants to take a shot at fixing it, I would be super appreciative.   I head to  SF for work this weekend and won't be able to look into it for at least a week.

@dan-hook could you take a look at https://github.com/pypa/pipenv/pull/6209 and help me determine if this is the right behavioral fix.

I am confused on one point -- it should leave the entries in the Pipfile right when using --all or --all-dev?   It seems there is a really old test assertion for --all-dev that checks it removes everything from the Pipfile dev, which feels weird and like a difference between what --all and --all-dev was supposed to do.   I am thinking both flags should not clear Pipfile entires for that category.

I've never used --all-dev, but I recall seeing the same strange behavior in the docs.  Since a change in --all behavior broke my workflow, I'd be sympathetic to anyone who had come to rely on the behavior of --all-dev.  If the current --all-dev behavior is useful, it might be worth considering adding a -c, --categories option to uninstall, since uninstall --categories dev seems like it should remove all packages from [dev-packages] in the Pipfile.

I'd be sympathetic to anyone who had come to rely on the behavior of --all-dev

Yeah me too, I restored that part of the PR -- that is the super hard thing about inheriting mainteance of this project, has been how undocumented and not always logical all the use cases have been and evolved.   Something I hope to improve eventually is the documentation.


Class App\Http\Controllers\DashboardController does not exist

???

This has been resolved in latest alpha. Please update as per the release notes.


Conditional Type Failed extending never

Bug Report
🔎 Search Terms
Conditional Type
🕗 Version & Regression Information

Version: v4.6.0-dev.20220116

⏯ Playground Link
Workbench Repro
💻 Code
export type PathParam<T> = T extends never ? {} : { path: T };
export type QueryParam<T> = T extends never ? {} : { query: T };
export type BodyParam<T> = T extends never ? {} : { body: T };
export type OptionsParam = {
  options?: {
    signal?: AbortSignal;
  };
}
export type UrlParams<TPath, TQuery> =
  PathParam<TPath>
  & QueryParam<TQuery>;

export type OperationParams<TPath = unknown, TQuery = unknown, TBody = unknown> =
  OptionsParam
  & UrlParams<TPath, TQuery>
  & BodyParam<TBody>;

type Example1 = UrlParams<{ name: string }, never>
//   ^?

type Example2 = OperationParams<{ name: string }, { sort: string}, never>
//   ^?

🙁 Actual behavior
The following Asserts happened:
type Example1 = never
type Example2 = never

🙂 Expected behavior
The following Asserts to happen:
type Example1 = { path: { name: string } };
type Example2 =
  OptionsParam
  & { path: { name: string } }
  & { query: { sort: string } }
  & { options?: { signal?: AbortSignal; };

This is working as intended.
never is essentially an empty union type, and using union types with conditional type inference results in distributive behaviour. Distributing over an empty union results in an empty union, as there's nothing to distribute over. See the documentation: https://www.typescriptlang.org/docs/handbook/2/conditional-types.html#inferring-within-conditional-types

@MartinJohns any chance you know how to accomplish what I am intending to do?

export type PathParam<T> = [T] extends [never] ? {} : { path: T };
export type QueryParam<T> = [T] extends [never] ? {} : { query: T };
export type BodyParam<T> = [T] extends [never] ? {} : { body: T };

@yordis From the documentation I linked:

Typically, distributivity is the desired behavior. To avoid that behavior, you can surround each side of the extends keyword with square brackets.

I see it now 🤦🏻 , but it went over my head, my apologies, I literally went to sleep with that page open for 20 mins.
Thank you so much for your patience and support.


x/crypto/openpgp: silently fail to read keyring when smartcard stub is present

When a GPG secring contains a smartcard stub, the openpgp package fails to read the keyring correctly and does not return an error. A call to openpgp.ReadKeyRing() will returns the entities that precedes the stub, excluding the entity located right before the stub.
For example, if a secring contains 3 entities and the smartcard stub is in position 3, then openpgp.ReadKeyRing() will only return the first entity. The second entity and the stub are ignored. No error is returned.
Steps to reproduce:
1. Create a keyring with two regular 1024 bits RSA keys
$ mkdir /tmp/testgolangopenpgp

$ gpg --batch --no-default-keyring --keyring /tmp/testgolangopenpgp/pubring.gpg --secret-keyring /tmp/testgolangopenpgp/secring.gpg --gen-key << EOF
> Key-Type: 1
> Key-Length: 1024
> Subkey-Type: 1
> Subkey-Length: 1024
> Name-Real: test key 1
> Name-Email<EMAIL_ADDRESS>> Expire-Date: 12m
> EOF
gpg: keyring `/tmp/testgolangopenpgp/secring.gpg' created
gpg: keyring `/tmp/testgolangopenpgp/pubring.gpg' created
..+++++
...+++++
.....+++++
..+++++
gpg: key 4BACAA9F marked as ultimately trusted

$ gpg --batch --no-default-keyring --keyring /tmp/testgolangopenpgp/pubring.gpg --secret-keyring /tmp/testgolangopenpgp/secring.gpg --gen-key << EOF
> Key-Type: 1
> Key-Length: 1024
> Subkey-Type: 1
> Subkey-Length: 1024
> Name-Real: test key 2
> Name-Email<EMAIL_ADDRESS>> Expire-Date: 12m
> EOF
.+++++
.+++++
+++++
+++++
gpg: key 56CE5F62 marked as ultimately trusted

2. Import a smartcard stub from a yubikey into the keyring
$ gpg --no-default-keyring --keyring /tmp/testgolangopenpgp/pubring.gpg --secret-keyring /tmp/testgolangopenpgp/secring.gpg --allow-secret-key-import --import yubikeystub yubikeytest1_stub.asc
gpg: key 4FB5544F: already in secret keyring
gpg: key 59BEDBFC: already in secret keyring
gpg: Total number processed: 2
gpg:       secret keys read: 2
gpg:  secret keys unchanged: 2

/tmp/testgolangopenpgp/secring.gpg
----------------------------------
sec   1024R/4BACAA9F 2014-12-14 [expires: 2015-12-09]
uid                  test key 1<EMAIL_ADDRESS>ssb   1024R/AEC9CD3A 2014-12-14

sec   1024R/56CE5F62 2014-12-14 [expires: 2015-12-09]
uid                  test key 2<EMAIL_ADDRESS>ssb   1024R/A1E03C73 2014-12-14

sec>  2048R/59BEDBFC 2014-12-14 [expires: 2015-12-09]
      Card serial no. = 0000 00000001
uid                  Yubikey Smartcard test1<EMAIL_ADDRESS>ssb>  2048R/9B633300 2014-12-14
ssb>  2048R/A8DE1BB7 2014-12-14

3. Attempt to read the entities in the keyring. Only the first entity is returned.
$ go run readkeyring.go
found 1 entities in keyring
reading entity with fingerprint 2FA49C2800342153CA439C90ADC366D34BACAA9F

source code:
package main

import (
    "code.google.com/p/go.crypto/openpgp"
    "encoding/hex"
    "fmt"
    "os"
    "strings"
)

func main() {
    secringFile, err := os.Open("/tmp/testgolangopenpgp/secring.gpg")
    if err != nil {
        panic(err)
    }
    defer secringFile.Close()
    keyring, err := openpgp.ReadKeyRing(secringFile)
    if err != nil {
        err = fmt.Errorf("Keyring access failed: '%v'", err)
        panic(err)
    }
    fmt.Printf("found %d entities in keyring\n", len(keyring))
    for _, entity := range keyring {
        fingerprint := strings.ToUpper(hex.EncodeToString(entity.PrimaryKey.Fingerprint[:]))
        fmt.Println("reading entity with fingerprint", fingerprint)
    }
}

Can I do anything to help with the resolution of this issue? It's blocking some of my users...

I took a peek at it, but this is my first dive into RFC4880 so some things are still unclear to me.
Looking at the yubikey stub in the secring, if I skip the first 272 bytes, I get directly to the S2K value, which is set to 0xff:
$ hexdump -s 272 -C 000015-005.secret_key
00000110  ff 00 65 00 47 4e 55 02  10 d2 76 00 01 24 01 02  |..e.GNU...v..$..|
00000120  00 00 00 00 00 00 01 00  00                       |.........|
00000129

0xff indicates an encrypted private key. In the code [1], (pk *PrivateKey) parse() tries to read the next byte to guess the encryption algorithm, but that next byte is set to 00 which doesn't map to any valid cipher.
Then, entering s2k.Parse(), the following 2 bytes are read (0x6500), and the second byte (0x00) is used in s2k.HashIdToHash(). But 0 is not a valid hash id, and the function returns error openpgp: unsupported feature: hash for S2K function: 0. The error is silenced is openpgp.ReadKeyRing [2].
I could do the analysis, but I'm afraid writing a patch is beyond my understand of the OpenPGP standard at this point...
[1] https://github.com/golang/crypto/blob/master/openpgp/packet/private_key.go#L67-L80
[2] https://github.com/golang/crypto/blob/master/openpgp/keys.go#L258

The hexdump from @jvehent is an example of a gnupg extension S2K type, which can be identified by the 65 (101 in decimal, rfc4480#section-3.7.1 denotes 100-110 as "Private/Experimental") followed by the 47 4e 55 (GNU in ASCII). The 02 byte after that signifies that the stub is for "gnu-divert-to-card" S2K (01 signifies a "gnu-dummy" S2K as in #13605).
The relavent gnupg code is here and here.

Per the accepted #44226 proposal and due to lack of maintenance, the golang.org/x/crypto/openpgp package is now frozen and deprecated. No new changes will be accepted except for security fixes. The package will not be removed.
If this is a security issue, please email<EMAIL_ADDRESS>and we will assess it and provide a fix.
If you're looking for alternatives, consider the crypto/ed25519 package for simple signatures, golang.org/x/mod/sumdb/note for inline signatures, or filippo.io/age for encryption. You can read a summary of OpenPGP issues and alternatives here.
If you are required to interoperate with OpenPGP systems and need a maintained package, we suggest considering one of multiple community forks of golang.org/x/crypto/openpgp. We don't endorse any specific one.


NF-kappaB binding

Hi,
Could I request a new term 'NF-kappaB binding', as a
child of 'transcription factor binding' (GO:0008134).
PMID: 15087454
Thanks,
Emily
(P06748 + IPI)
Reported by: edimmer
Original Ticket: "geneontology/ontology-requests/1768":https://sourceforge.net/p/geneontology/ontology-requests/1768

thanks www.movied.org


fix: Sortable grid first row problem

this merge resolves this issue:
https://github.com/clauderic/react-sortable-hoc/issues/242

Hello guys!
Why don't you answer me?


Fix typo in installation instructions

Cop name is case sensitive, and must therefore be spelled with a capital ‘S’ in RSpec.

Thanks! :)


NuGet package?

Is there any chance that this great library can be turned into NuGet package that uses assemblies instead of class file includes? I would love it as it would make updating my projects dependencies much easier.

This has been done!


More memory efficient import statements

Background and Motivation
Currently, we mix all syntax trees into a collection and when an import statement referencing the syntax tree we insert the tree into another collection. That's not very efficient.
Proposed Solution
I propose we search all global statements for imports in the BindGlobalScope method and then see if the file name referenced in the import corresponds to a syntax tree. If it does, we'll add it to the _importedTrees collection and allow it's child methods to be referenced. This will also illegalize import statements outside the global scope.

This has now become a duplicate of #69


attach lensed SNe to host galaxies

takes as input an image, a catalog of galaxies (with position angles), and a catalog of lensed SN parameters (phi, d, mag, MJD) - see the whiteboard sketch.
generates a catalog of data for planting these lensed SN into the image:  (x, y, mag)

Completed during sprint 1.   See set_fake_positions_at_galaxies in fakeplanting.py

Completed during sprint 1.   See set_fake_positions_at_galaxies in fakeplanting.py


Add a bunch of const keywords to allow usage of LogError/LogWarning/LogNote in const functions without const_cast

Hi, please grab the latest changes as this change has merge conflicts with just accepted Flow AddOns.

Updated!

One more merge fix will be needed here. I will pull as the next PR after merge fix and applying my comments :)


basic compiling doesnt work

I tried making a basic hello world program but it threw a tsunami of compiler errors at me
the code is just console.log("hello world") and i ran the command nectar hello.js -o hello.exe
i could provide the output of the compiler if you need

Hi @KCGD ,
What is your environment ? What is your version of GCC/CLang ?

@adrien-thierry
im using node v10.16.0
npm 6.14.9
NectarJS v0.7.155
GCC (i686-posix-dwarf-rev0, Built by MinGW-W64 project) 8.1.0
g++ (i686-posix-dwarf-rev0, Built by MinGW-W64 project) 8.1.0
on windows 10 x64 version 10.0.18363 version 18363

try with an higher version of GCC (with c++ 17 support)

@adrien-thierry, i upgraded to g++ 10.2.0, still gives me the same error.
is that the right g++ version?
also does the node version matter?

Yes, it should be good,
What is the error log ?
thx

@adrien-thierry it just worked, apparently I declared a variable but never used it and that made the compiler freak out for whatever reason
I attached the previous error log if you want
err.txt

Perfect thx !
I will check it

I've gotten an issue like that when using C for WASM via emscripten, declaring a variable and not using it.  I think that's just part and parcel with compiling.


[Pathfinding] A to B enemy type does not follow when player on the leftmost same-platform tile

Steps to reproduce

Stand close to the platform (with one foot on the air).

Expected behaviour
The enemy comes to you and starts attacking.
Actual behaviour
The enemy stands there doing nothing, I can even shoot at it and it doesn't react.
Game version
0.4
Screenshots

Additional context
Probably non navigable zone reaches the area where I'm standing because the player position isn't on his right feet, but on the center of its body.

Extra data: this happens with a to b enemies (Hellknight and Baron of Hell)

This is duplicate :/, i will keep the other


Bump binaryen version to 1.38.28-p1

I've followed this guide
step by step carefully. [Yes]

I've tested this package remotely and have excluded all broken builds.
Here is the links to the Travis/AppVeyor with status "All passed":

https://ci.appveyor.com/project/Warchant/hunter/builds/22671621
https://travis-ci.com/Warchant/hunter/builds/102402752

Testing:

https://ci.appveyor.com/project/ingenue/hunter/builds/22688073
https://travis-ci.org/ingenue/hunter/builds/499253032?utm_source=github_status&utm_medium=notification

Uploading:

https://ci.appveyor.com/project/ingenue/hunter/builds/22689489
https://travis-ci.org/ingenue/hunter/builds/499276286?utm_source=github_status&utm_medium=notification

Released: https://github.com/ruslo/hunter/releases/tag/v0.23.120


import not possible when running in docker container

Hey! I'm trying to set up a docker container and use 8 bit Adam. When I'm trying to import bitsandbytes, I'm getting
2022-10-26T11:20:58.819996206Z     import bitsandbytes as bnb
2022-10-26T11:20:58.820003169Z   File "/root/miniconda3/lib/python3.9/site-packages/bitsandbytes/__init__.py", line 6, in <module>
2022-10-26T11:20:58.820019600Z     from .autograd._functions import (
2022-10-26T11:20:58.820023698Z   File "/root/miniconda3/lib/python3.9/site-packages/bitsandbytes/autograd/_functions.py", line 5, in <module>
2022-10-26T11:20:58.820060948Z     import bitsandbytes.functional as F
2022-10-26T11:20:58.820067420Z   File "/root/miniconda3/lib/python3.9/site-packages/bitsandbytes/functional.py", line 13, in <module>
2022-10-26T11:20:58.820089783Z     from .cextension import COMPILED_WITH_CUDA, lib
2022-10-26T11:20:58.820094021Z   File "/root/miniconda3/lib/python3.9/site-packages/bitsandbytes/cextension.py", line 113, in <module>
2022-10-26T11:20:58.820140619Z     lib = CUDASetup.get_instance().lib
2022-10-26T11:20:58.820148213Z   File "/root/miniconda3/lib/python3.9/site-packages/bitsandbytes/cextension.py", line 109, in get_instance
2022-10-26T11:20:58.820191876Z     cls._instance.initialize()
2022-10-26T11:20:58.820198208Z   File "/root/miniconda3/lib/python3.9/site-packages/bitsandbytes/cextension.py", line 59, in initialize
2022-10-26T11:20:58.820229417Z     binary_name, cudart_path, cuda, cc, cuda_version_string = evaluate_cuda_setup()
2022-10-26T11:20:58.820236040Z ValueError: too many values to unpack (expected 5)

This happens only when I'm starting my script directly in the Dockerfile, if I instead just start the container without any CMD or ENTRYPOINT, ssh into the container and then manually start the script, it does work, but for some reason, Docker doesn't seem to be having access to CUDA properly when starting from the Dockerfile?
I've traced the error down to https://github.com/TimDettmers/bitsandbytes/blob/a371be302ddbdf3f36acef1a6fe365672099c9d9/bitsandbytes/cuda_setup/main.py#L137 since the function returns here with only one value instead of the expected 5, but I haven't had any other issues with CUDA before that, e.g. transferring models to the GPU by using .to('cuda')
I'm at a loss since I'm not too experienced with Docker and it does work when I'm starting the script manually via ssh.
Thanks in advance!

Hey @TimDettmers , unfortunately this is still broken for me, despite pip installing from the repo directly.
I'm running this in Docker / JupyterLabs on NVIDIA GeForce RTX 3090, 24576 MiB, 24267
Any advice on anything I might be doing wrong?
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Input In [1], in <cell line: 1>()
----> 1 import bitsandbytes as bnb

File /opt/conda/lib/python3.9/site-packages/bitsandbytes/__init__.py:6, in <module>
      1 # Copyright (c) Facebook, Inc. and its affiliates.
      2 #
      3 # This source code is licensed under the MIT license found in the
      4 # LICENSE file in the root directory of this source tree.
----> 6 from .autograd._functions import (
      7     MatmulLtState,
      8     bmm_cublas,
      9     matmul,
     10     matmul_cublas,
     11     mm_cublas,
     12 )
     13 from .cextension import COMPILED_WITH_CUDA
     14 from .nn import modules

File /opt/conda/lib/python3.9/site-packages/bitsandbytes/autograd/_functions.py:5, in <module>
      2 import warnings
      4 import torch
----> 5 import bitsandbytes.functional as F
      7 from dataclasses import dataclass
      8 from functools import reduce  # Required in Python 3

File /opt/conda/lib/python3.9/site-packages/bitsandbytes/functional.py:13, in <module>
     10 from typing import Tuple
     11 from torch import Tensor
---> 13 from .cextension import COMPILED_WITH_CUDA, lib
     14 from functools import reduce  # Required in Python 3
     16 # math.prod not compatible with python < 3.8

File /opt/conda/lib/python3.9/site-packages/bitsandbytes/cextension.py:113, in <module>
    109             cls._instance.initialize()
    110         return cls._instance
--> 113 lib = CUDASetup.get_instance().lib
    114 try:
    115     if lib is None and torch.cuda.is_available():

File /opt/conda/lib/python3.9/site-packages/bitsandbytes/cextension.py:109, in CUDASetup.get_instance(cls)
    107 if cls._instance is None:
    108     cls._instance = cls.__new__(cls)
--> 109     cls._instance.initialize()
    110 return cls._instance

File /opt/conda/lib/python3.9/site-packages/bitsandbytes/cextension.py:59, in CUDASetup.initialize(self)
     56 self.lib = None
     58 from .cuda_setup.main import evaluate_cuda_setup
---> 59 binary_name, cudart_path, cuda, cc, cuda_version_string = evaluate_cuda_setup()
     60 self.cudart_path = cudart_path
     61 self.cuda = cuda

ValueError: too many values to unpack (expected 5)

Same here, I do
RUN pip3 install -qq git+https://github.com/TimDettmers/bitsandbytes

but later I get
UserWarning: The installed version of bitsandbytes was compiled without GPU support
...
NameError: name 'str2optimizer8bit_blockwise' is not defined

I'm guessing that the issue might be that CUDA is installed, but during build-time no GPU is attached. Is it possible to force this somehow?

Same here, I do
RUN pip3 install -qq git+https://github.com/TimDettmers/bitsandbytes

but later I get
UserWarning: The installed version of bitsandbytes was compiled without GPU support
...
NameError: name 'str2optimizer8bit_blockwise' is not defined

I have the same issue with the UserWarning now. I managed to resolve the first issue by using a different base image for my docker container, namely the devel instead of the runtime version. I'm gonna try that what this error now.

Any updates? I am facing this exact issue on WSL2 with a 3090 and cuda 11.6 installed on the docker.

using the devel did work for me. Also manually installing cudatoolkit with conda!


cherry-pick [CMake] Move standalone check so we don't have to reconfigure LLDB

By moving the standalone check into the main CMake file, the whole file
is ignored in a regular (non-standalone) build. This means that you can
make changes to LLDBStandalone.cmake without having to reconfigure a
build in a different directory. This matters when you share one source
repository with different build directories (e.g. release-assert, debug,
standalone).
Differential revision: https://reviews.llvm.org/D64824
git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@366346 91177308-0d34-0410-b5e6-96231b3b80d8

@JDevlieghere @xiaobai
This was basically a full file of conflicts.

@swift-ci please test

Thanks Nathan! Out of curiosity, why do you care if this is on stable?

From what I understand, this would likely still conflict upon merging if put on upstream-with-swift.

@swift-ci please test

@JDevlieghere – This change broke unified builds on Red Hat Fedora 30 (x86_64). Was this expected?
$ ninja all check-swift-validation
ninja: error: '/lib/swift', needed by 'lib/lldb/swift', missing and no known rule to make it

@JDevlieghere – This change broke unified builds on Red Hat Fedora 30 (x86_64). Was this expected?
$ ninja all check-swift-validation
ninja: error: '/lib/swift', needed by 'lib/lldb/swift', missing and no known rule to make it

Got a link to the log?

Which log? What follows is my personal “continuous integration” output, but I suspect you want the CMake logs, right?
https://znu.io/swift-ci.txt