Reintroduce blueprints

One of the things I did in FPE was to remove Box2D blueprints and simplify the engine to use a factory pattern instead. In Box2d, it is like this:
CircleShapeDef c = new CircleShapeDef();
c.Position = new Vector2(10, 10);
c.Density = 4;

BodyDef b = new BodyDef();
b.Shape = c;

Body cb = b.Create();

Instead I wanted it a little simpler so a user could just do this:
BodyFactory.CreateCircle(new Vector2(10, 10), 4);

I think the factories quickly became the primary thing to use for creating shapes, bodies and fixtures.
The simplification actually removed blueprints (definitions) completely since I could omit a memcopy in the process and do some clever stuff with density calculations at the same time.
However, definitions are good for a couple of things:

Serialization - since they describe user objects.
Cloning - You could convert a Body back into a BodyDef for cloning or serialization

Velcro has support for both serialization and cloning, but since there are no blueprints, they use the real user objects directly, which creates other problems.
This proposal is to re-introduce blueprints/definitions into the engine, while still maintaining factories, cloning and serialization. This change means we will do a little more work in memory, but it should be negligent. For the users, it means all factories will have an overload that takes in a definition, and body, shape  and fixture will be extended with methods for converting back and forth between real object and blueprint/definition.
Any comments, improvements or votes (thumbs up/down) are appreciated.

This feature should be aware of #5 such that it can utilize pooling efficiently.

I'd just like to throw in that I don't think Farseer / Velcro needs or should be concerned with serialization, because every game and game engine will do this differently. In Duality, I actually removed any serialization support from our Farseer port entirely. From a physics library, I expect that it simulates physics based on data that I provide, without making assumptions about how that data will be structured or stored.
On the same vein, I'm not sure body definitions provide more value than overhead. Sure, I can then clone a body easily, but I've had to create the body definition in the first place - so why wouldn't I just execute the same code again? It makes sense only when you see serialization as part of Velcros concerns, which I would tend to argue against.
That said, both of this sounds like a neat addition as a separate utility library on top of Velcro that can be used by Velcro samples and sandboxes - keeping the core library clean and small.

I'd just like to throw in that I don't think Farseer / Velcro needs or should be concerned with serialization, because every game and game engine will do this differently.

It's been a long time since I used Farseer in a real world scenario but I do have vague memories of trying to deal with serialization and if I recall correctly it used to be quite painful.
I agree that serialization doesn't need to be a built in feature of Velcro but I do think it's worth at least thinking about how serialization is going to work because at some point someone is going to want to serialize these things.

Velcro has support for both serialization and cloning, but since there are no blueprints, they use the real user objects directly, which creates other problems.

Can you elaborate on the types of problems that arise when serializing the real user objects directly?

This proposal is to re-introduce blueprints/definitions into the engine, while still maintaining factories, cloning and serialization. This change means we will do a little more work in memory, but it should be negligent. For the users, it means all factories will have an overload that takes in a definition, and body, shape and fixture will be extended with methods for converting back and forth between real object and blueprint/definition.

At this stage I'm in favor, but as I said, it's been a long time since I've actually used the library in the real world so take my opinion with a grain of salt.
In MonoGame.Extended we tend to prefer the JSON.NET serializer which I've found to be very powerful and customizable. My gut feeling tells me it might be possible to get it working with or without the blueprints anyway.

@ilexp
I completely understand where you are from and I tend to think the same thing. However, we need to cater to people that usually need to serialize, but have no idea on how to do it. I worked together with someone back in the day, to make a standard format for serialization, such that you could export from FPE and import in another physics engine. It was formalized in a way where we only serialize "unique" bodies and stuff like that, which makes it very efficient compared to what you might come up with yourself.
I wanted to limit this issue as well.
That being said, the engine is not user friendly for serialization right now, as need to extract bodies, shapes and fixtures from the engine and then know what is important and what is not. You can very quickly blow up the engine if you serialize forces like velocity and don't apply it correctly on deserialization.

@craftworkgames

Can you elaborate on the types of problems that arise when serializing the real user objects directly?

Yep. It comes down to the fact that I have to extend the internals of the engine every time I introduce a new feature. It also means I have to hack around with the visibility of certain properties that are usually private to an object. Cloning also creates a new object every time; this is where a pool would help, but I have to rewrite all of cloning to make use of it, so I might as well take multiple things into consideration.

In MonoGame.Extended we tend to prefer the JSON.NET serializer which I've found to be very powerful and customizable.

I love JSON.NET and it is amazing to me how James managed to make it this fast. However, as both you and @ilexp have touched on, serialization is often custom made by the user. For this reason, I'll try and see if I can come up with a generalization of blueprints that would make it easy to export a world to any format and thus, not depend on XML (what we do now) or JSON.
If I manage to solve #3 somehow, I can finally split the framework into several projects, and one could be VelcroPhysics.Serialization.JSON, that takes a hard dependency on JSON.NET - this way the engine itself is dependency free.

@craftworkgames
Just a quick note: Did not know what MonoGame.Extended was. Looked it up and this is the first time I see Mercury Particle Engine mentioned in a long time, I think it has been 5 years since I last developed on the project.

You can very quickly blow up the engine if you serialize forces like velocity and don't apply it correctly on deserialization.

Serializing the physics world representation is not a carefree solution either, as the problem for users shifts from "how to serialize a body" to "how to re-synchronize physics world and higher level game world before / after serialization". Depending on what exactly any given game or engine needs, and which level of detail is required to be serialized, I could imagine not being the only one to prefer problem A over B.

For this reason, I'll try and see if I can come up with a generalization of blueprints that would make it easy to export a world to any format and thus, not depend on XML (what we do now) or JSON.

Maybe that could be a good solution to the overall serialization issue, not just limited to blueprints: Create an intermediate data structure that does not itself carry any serialization logic, but is geared towards being easy to serialize, then allow to create a template or even world based on that, or create it based on an existing world or template.
A separate project could then implement specific serialization techniques for users that need them - all others can either use the intermediate data structure for their own serialization, or roll out something custom entirely.

Create an intermediate data structure that does not itself carry any serialization logic, but is geared towards being easy to serialize

Yep, that is what I'm also thinking. I'm very much into the whole abstract syntax vs. encoding issue since I work a lot with ASN.1 on a daily basis, and specific implementations (JSON) should be different from the abstract representation (definitions)

I just found myself working on some code and I ran into an issue related to blueprints (or lack of). Ideally, I want to be able to declare the data for my bodies separately from actually adding them to the World.
The first parameter in the factory methods is always the world. However, it's not always the case that you want to add the body to the world immediately. Ideally, I'd like to be able to setup my bodies in one part of the code and defer adding them to the world until later. Clearly blueprints would be useful in this situation.
I can workaround the issue, but I just wanted to bring it up while I think of it as another vote for this issue 👍

Very good use case, thanks for providing it.

Serializing the physics world representation is not a carefree solution either, as the problem for users shifts from "how to serialize a body" to "how to re-synchronize physics world and higher level game world before / after serialization".

Is is safe to reposition a body with the current implementation?
For example I have a snapshot from server and must correct the simulation running on client.

This library is currently under heavy development, so I would advise you to download Farseer Physics Engine 3.5 for now. Please create a new issue for stuff not related to this discussion.


Update g++ 4.8 to 4.9 in travis.yml

Update g++ from 4.8 to 4.9 in travis testing.

Restarted the travis build to check against the newly merged changes in master.

I believe the build failed because the branch in this PR started from an old version of master, which was missing the extra TMPDIR in .travis.yml. I merged master in, so it should be OK now.

@tzanio - thanks, yes mac parallel ones pass now.


The Mod does not seem to load.

Before I start, I am not sure if this is the place to ask for support, so I am sorry in advance.
I have Tried installing both Personal Logistics and Personal Logistics Free via Thunderstore, but neither seem to work. Maybe there is some incompatibility but the buttons do not shop up at all, and can't find any shortcut to open it. it is like it isn't even loaded.
I am also having an error poping up but I think it is from another mod, but I still can't find any log for it to give me details.

Any help as to where I can find any logs to check it would be helpfull,
Mods installed
01883a6b-369f-f12a-16b5-8bab384492fc
Modlist.txt
Extra
Extra.zip

I can confirm this is a bug caused by personal logistics
I activated the console, and it said: "MissingFieldException: Field 'UIGame.inventory' not found."
many lines had stack trace: with nothing behind it, but after the UI of the game appeared, some of the stack trace had the name PersonalLogistics (and something similar with the string "UImanager") containing inside it.


missing k8s cpu and memory workload anomalies?

I tried deploying what's documented here using the latest provider version (1.27.0) just to get something out there which gave me the error below. This error message makes me think the latest release supports these newer anomalies, but I'm not sure what they should be since the documentation doesn't mention them yet. Is that accurate or is it not quite supported yet? Looking at the release notes it seems that the cpu anomalies have been a thing for about a month and the memory just released so I thought maybe it wasn't supported but this error message makes me think otherwise.
│ Error: Given property 'highMemoryUsage' with value: 'null' does not comply with required NonNull of schema
│ Given property 'highCpuThrottling' with value: 'null' does not comply with required NonNull of schema
│ Given property 'highCpuUsage' with value: 'null' does not comply with required NonNull of schema
Any additional information or things for me to try would be greatly appreciated. thank you!

Hi @ekleiner14,
It looks like a breaking change was introduced in the schema for K8s workload anomalies and a few others in the last update. I've pushed the updated schema, targeting a release this week. We apologize for the inconvenience.
We're currently in discussions with the Dynatrace API dev team to find a solution regarding this issue to prevent breaking changes from happening with schema updates, more to come on that soon!


Miscellaneous clean-up

More of it.

Organize use statements
Add missing license headers
Trim trailing whitespace and redundant/outdated comments
Fix some inconsistent indentation

Coverage increased (+0.4%) to 83.794% when pulling 20995b37812fd77800db8e60764507b5d26a3387 on rzumer:cleanup into 68cb35b0d53bf14a5b7b08d555b32eca25d9bdd8 on xiph:master.


Add missing oslo dependency

Is this suited for github?

[X] Yes, this is suited for github

To Reproduce
install better auth, and try to build app. For server-side.
Current vs. Expected behavior
Current behavior:
bun build ./src/index.ts --outdir ./build --packages bundle --minify --target bun
122 | :"text/html"}})}import{sha256 as hn}from"oslo/crypto";var yn=(e,i)=>{let r=e.context.options.baseURL,o=e.context.baseURL
error: Could not resolve: "oslo/crypto". Maybe you need to "bun install"?
    at C:\Users\Sasha\WebstormProjects\host-panel-backend\node_modules\better-auth\dist\plugins.js:122:4820

Expected:
Just build normally
What version of Better Auth are you using?
1.1.0
Provide environment information
- OS: any

Which area(s) are affected? (Select all that apply)
Backend, Package
Auth config (if applicable)
import { betterAuth } from "better-auth"
export const auth = betterAuth({
  emailAndPassword: {
    enabled: true
  },
});

Additional context
I'm updated to latest release.

Also, the package are going to be deprecated and recommended to use @oslojs/crypto


The source file

I have just changed the source of the configuration file

I have just added the source of the configuration file

Which interface you are planning to use for olsr?  This file is only starting black interface wifi and that will be used later for cbma.

@joenpera all interfaces

Closing as not valid


Resolves #4672

Resolves #4672

Changelog

New

Changed

Removed

Rollout strategy

[ ] Patch release
[ ] Minor release
[ ] Major release; if selected, include a written rollout or migration plan
[ ] None; if selected, include a brief description as to why

Testing & Reviewing

Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Added/updated previews (Storybook)
[ ] Changes are SSR compatible
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge
[ ] (GitHub staff only) Integration tests pass at github/github (Learn more about how to run integration tests)

Hey @cihad! Just wanted to follow-up and see if you were still working on this PR? I see it's nearly finished, it looks like it just needs a changeset and an update to your PR description. Thank you too for your contribution!


[Feature] Add a setting for natural maximum width and height of hover in order to limit how big a hover is rendered naturally

In relation to conversation in https://github.com/microsoft/vscode/issues/199944

The maximum natural width has been increased since then. There is not much want for this from the community, so I will close this issue.


Refactored #findClosestFacingTree (resolves #2)

Refactor to support multi-directional chopping (NW, NE, etc)
Refactor to use south-western tiles for calculations
Refactor to use manhattan distance instead of #distanceTo for calculations
Added debugging overlays for developers (commented out)

fixes #2
https://github.com/Infinitay/tree-count-plugin/assets/6964154/30022c81-f1ec-4903-aef1-9f0b28cdf83f

Found existing instance where facing west sometimes causes issues for close trees


Update prerec packages info: transformer and apex?

replacing every pytorch_transformers by transformers fixes the problem of giving negative dimensions to create tensor.
fp 16 requires nv apex, but can you write something to note that it is not the pip apex, and point to a way to install from nv or just let the default fp16=False to circumvent the issue for beginners?
Thanks

maybe 2 is not a good idea. Maybe include the code to install apex and use fp16 to save cost?


Set the C++ version to 17

We should backport this to humble and iron

Just got a buildfarm fail email. Related to this?

buildfarm should email you only with merged PRs, isn't it?

https://github.com/Mergifyio backport humble


Set stderr to empty string when using terminal

PR Description
Windows requires that stderr is an empty string when running a container with TTY. In Containerd, pkg/cio/io.go#L298:L302, the Stderr is set to a value, e.g.
res = "binary:///C:/_xxx_/nerdctl.exe?_NERDCTL_INTERNAL_LOGGING=C%3A%5CProgramData%5Cnerdctl%5C052055e3"

which causes microsoft/hcsshim service_internal.go#L127 to fail to create a new task.
To address this issue, we refactor TerminalLogURI() and TerminalBinaryIO functions by adding an optional parameter that specifies whether stderr should be set to an empty string when the terminal is used. This change will help avoid the "failed precondition" error when creating a new task on Windows.
Bug reference:
#10380: Refactor TerminalLogURI() to Support Optional Stderr Configuration for Terminal Compatibility on Windows
Related issues:
nerdctl bug #2966

My main concern with this is if we are making it harder to troubleshoot any issues between containerd->shim->runhcs

Can you preface in the title this is mainly for windows?

My main concern with this is if we are making it harder to troubleshoot any issues between containerd->shim->runhcs

@MikeZappa87
The current implementation redirects all output to a single stream, unifying stderr and stdout.
https://github.com/containerd/containerd/blob/1fb1882c7de7239711c343c65fa266a2f1eba9db/pkg/cio/io.go#L261-L262
This already matches Windows' default behavior, which combines both streams. However, Windows HCSShim requires that stderr is an empty string and is designed to check this specific precondition. If it detects that stderr is not an empty string, it will fail the operation.

/ok-to-test

/retest

@mikebrow when you are free :-)


Update api-spec.md to fix some formatting

Removes stray text below WorkspaceDeclaration that has been included in the table.

Changes

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Docs included if any changes are user facing
[ ] Tests included if any functionality added or changed
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including
functionality, content, code)
[ ] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes

/retest


Translate (Seq2Seq) Tutorial Expectations

The output I am getting from the translate.py tutorial looks corrupted.
hello -> G0
I followed the translate tutorial here:
https://www.tensorflow.org/versions/master/tutorials/seq2seq/index.html
I wanted to create a small test model, limiting to 1M records
python translate.py --data_dir data --train_dir train --en_vocab_size=40000 --fr_vocab_size=40000 --size=256 --num_layers=2 --steps_per_checkpoint=50 --max_train_data_size=1000000
No errors during the training.
After 15 hours, I stopped the training when the perplexity was below 10 points.
global step 19500 learning rate 0.2655 step-time 2.76 perplexity 8.63
eval: bucket 0 perplexity 13.15
eval: bucket 1 perplexity 10.71
eval: bucket 2 perplexity 12.78
eval: bucket 3 perplexity 14.38
After stopping the training I deleted the last corrupted training file.
rm train/translate.ckpt-19500
If I try to translate simple words I get junk.
python translate.py --decode --data_dir data --train_dir train
Created model with fresh parameters.
-> hello
G0 Processing Processing Processing Processing Processing Processing Processing Processing Processing
-> house
G0 G0 pâturage d’infrastructures d’infrastructures Twin Twin Twin Twin Twin
-> Who is the president of the United States?
expédiées expédiées expédiées m0 m0 m0 m0 m0 m0 m0 m0 m0 m0 m0 m0
Other translation technologies (es. Moses) will provide a decent translation with that training data.
If there a glitch somewhere or did I do something wrong?

I am also getting same error. Did you get the answer.

Few more info to reproduce:
I am using Mac OSX El Captain and TF 0.6
I installed in this way:
$ sudo easy_install pip
$ sudo pip install --upgrade virtualenv
$ virtualenv --system-site-packages /Users/marcotrombetti/Documents/tensorflow-python-env (io ho creato una cartella in Documents)
$ source /Users/marcotrombetti/Documents/tensorflow-python-env/bin/activate
$ sudo pip install --upgrade https://storage.googleapis.com/tensorflow/mac/tensorflow-0.6.0-py2-none-any.whl (Please note that the url is different compared to the documentation)
Files have been installed here:
/Library/Python/2.7/site-packages/tensorflow/models/rnn/translate/
At one point,  I was missing the translate.py files (not sure if this was only related to a 0.5 problem), anyway I got it from here:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/models/rnn/translate/translate.py
@vnkmr7620 did you test on a different configuration and/or release?

Hi Marco. I think what you see is not a bug. For one, 2-layer network of size 256 is simply too small for translation. Second, I don't think you train it long enough. And neural network based methods require more data than hand-programmed systems like Moses. As specified in the tutorial - try running a 3-layer network with size 1024 for ~300-400K steps -- I think you'll see better results.

would love to know if that fixed their issue

Slightly related: How much memory do you guys have on your GPU? I have 3GB on my GPU, and 3 layers with size 1024 runs out of memory.

Indeed, it runs out of memory on my 970 too. You can try to lower the batch size,  but it might become too slow. We're working on making TensorFlow more memory efficient but it will take a while. It does fit into 12GB, so you can run it on a Titan X / Black or Tesla K40 / K80, but these are quite expensive.

I have a tesla k80 I'm building on. What does take a while look like? A week ? A month?  I have been training this model for 3 days now. I think for all the examples/tutorials we should include some basic runtime stats to give a ballpark.
On Fri, Feb 12, 2016 at 10:21 PM, Lukasz Kaiser<EMAIL_ADDRESS>wrote:

Indeed, it runs out of memory on my 970 too. You can try to lower the batch size,  but it might become too slow. We're working on making TensorFlow more memory efficient but it will take a while. It does fit into 12GB, so you can run it on a Titan X / Black or Tesla K40 / K80, but these are quite expensive.
Reply to this email directly or view it on GitHub:
https://github.com/tensorflow/tensorflow/issues/600#issuecomment-183587119

It might depend on your hyper-parameters. The tutorial model is not tuned and has a lot of data, so it would probably take months to fully converge. But you can tune it to converge faster, and it should get to reasonable perplexity in a few days anyway. Also, real-time is hard to jugde, can differ from machine to machine. What's your step number and perplexity -- that's the question to ask (and some step-number and perplexity ballpark are included in the tutorial).

I have tried with different batch sizes but it still crashes. I am running it on an AWS x2-large GPU instance which has 15 GB RAM. it fails in the read_data part of translate.py script. when I do htop, I see that the memory usage keeps increasing. Can you please tell me how can I run the machine translation example on smaller data? currently its running on training-giga-fren.tar

Hi! Can you try using this option: --max_train_data_size=500000 ? It should reduce the training data size, and currently the whole data-set is loaded into memory. You can also modify the data-loading function to load it batch-by-batch instead of loading it all into memory at once.

@lukaszkaiser thanks for your inputs. I do have one more query. Iets say I have stopped training  at perplexity 15. Now I will run some tests and if I am not satisfied with the results ad want to see how the results will be for perplexity of "8", I will keep for retrain. The training starts from checkpoint 50, that is at a higher perplexity (1700), whereas I expect it to start at perplexity "15" the point where I had stopped initially. Is there a way to retrain from the checkpoint I had stopped at?

This is strange: usually it'd start at the latest checkpoint. It's all saved in the --train_dir directory and the "checkpoint" file there specifies what's the latest checkpoint. Could it be that you used /tmp and your checkpoint got deleted, e.g., on restart?

@lukaszkaiser  No, I see that the checkpoint data as well as other directories are intact. Caould there be any other reason?

@lukaszkaiser you say that the training data should be huge. I tried with 10M sentences for English to French Translation. I see that the results are not good. I even tried to use AdagradOptimizer optimizer in place of GradientDescentOptimizer  in seq2seq_model.py. The perplexity of the first two buckets is below 10 for checkpoints after 15000.  Can you tell me if this is a good behavior?  Also at what point or at what value of perplexity will the training stop gracefully?

I think the perplexity needs to go to around 4 for the results to be good. On a single GPU, this can take about a month of training with this simple tutorial script. There is a lot that can be sped up there, and better parameter settings can help, but I hope that gives you a general idea for the expectations.

@lukaszkaiser
If I have multiple GPU's, will tensorflow use all of them for computations by default or I will have to configure it?

It will, but the automatic placement currently might not be optimal. You can place manually with tf.device and that could make it faster (esp. if you place different LSTM layers on separate GPUs, as they can often work in parallel). I think this is a good question, but this bug is not the best place for it, as it's mostly unrelated and will be hard to find. If you want to ask about speeding up training, maybe open a stack overflow thread?

Hi Everyone! Please help. I encountered this error while trying to train the Seq2Seq model for MT using tensorflow... What's the cause of this?

Also, If I may ask, what's the minimum hardware requirement to be able to successfully train the model?

You probably need 12GB or more. You can also try using fewer layers and/or smaller layers, and smaller batch sizes, if it's running out of memory.

We're working on making the requirements much lower with dynamic seq2seq, see #4686 -- when that's in, it should be much easier to train such models.

Does anyone have a model already pretrained that's available for download?  I just want to load up a pretrained model and see how good the translations are.  Information about how long it took to train and what kind of hardware was used would also be useful.

@ekkus93 It seems that tensorflow/models/rnn/translate.py has disappeared from the master branch. In addition, I have also trained this model using GPU for one week but the outputs do not make any sense.

We are planning to make a better model and tutorial as part of tensorflow-models. The model in core was intended as an illustration of more basic TF points, not a competitive translation model -- tensorflow-models repo seems to be a better place for that. Hope you can agree!

Sure. Looking forward to the new intuitive illustration examples coming.

Hi Guys,
I was trying to use the command tf.device() in the translate example to be set on the GPU, it was working till the step of saving a check point and it crashed with a message pointing out that I need a CPU for this, so I removed the command, but I noticed that the step time is taking the same time when I was running the code without a GPU installed before, Do you know how can I make sure that the code is running on the GPU?

If you are using NVIDIA GPUs, just type
% nvidia-smi

Thanks it seems that tensrflow is not utilizing the GPU, did you guys had to do anything specifically to run the example on the GPU ?
I have CUDA and CuDNN installed, and I have tried the command tf.device("gpu") before and it was working and the test went through!
Any advice ?

translate.py does not use GPU in all the process.  You should check the status of GPU by
%nvidia-smi
and
%top
when messages come out, or stopped. ( when running top, push "1" key to see every single CPU's performance).
Maybe you can see your GPU is running.

translate.py uses GPU in not all the process. You should check the status of GPU by
%nvidia-smi
and
%top
when messages come out, or stopped. ( when running top, push "1" key to see every single CPU's performance).
Maybe you can see your GPU is running.

I think you need to use tf.device("/gpu:0") or sth like that. You can also set soft placements in session (with tf.Session(config=tf.ConfigProto(allow_soft_placement=True)) as sess) -- this will place on CPU all ops it cannot place on GPU, like the Saver.

@lukaszkaiser , in general, for sequence to sequence models with extremely long sequences, how do you recommend fitting them into memory without endangering the attention mechanism?

It depends how long. If it's into the thousands but not too high, say upto 4000 or so, our recent attention models from tensor2tensor (https://github.com/tensorflow/tensor2tensor) work quite well. If it's larger, then you need to think. Memory is only one problem, you also need to consider how will the network know where to propagate gradients, how will it find correlations if they are very-long-term? We managed to get sth working in many-thousands distance setting in the paper "Learning to Remember Rare Events", but at the cost of an extra loss that can be hard to train. I'd say it's an ongoing research problem at this point. But sometimes you can just work around it, e.g., put a bunch of strided convs or pooling first and down-size your input: if you don't need the specific tokens, just general summary, that often works best.


Optimize the representation of Hours

pub enum Hours {
    Hour24(u8),
    Hour12am(u8),
    Hour12pm(u8),
}

include mode 24/12, Express more clearly

I evaluated your changes and everything seems to work fine. Thanks for changing example code also. Suggested Hour format is much more readable now. I planned to change this feature in the future, but i didn't have much time for that. I will add new example with Serial which is more convenient for users, because it does not require to have SSD1306 oled display.


Suggest french translation refactor for Base readme

Refactor and update of the base readme french translation

just a few comments

I have made some changes according to that, hope it will be good, thanks for the review 🥳!


Bug when connect/disconnect while make ajax request (iOS)

Hi,
I try post it on iOS but list, but i receive a 404 error when click on link.
The problem is my app is make an ajax call every 1 second on my server and if i start the app without internet connect and connect on wifi/3g the application is killed (fatal error). Other situation is when im on my app connected and making ajax pooling and the connection (wifi or 3g) is dropped, the application is killed too (fatal error).
So, if you are making ajax call  and your connection change state (connected/disconnected) your app get a fatal error and close. :(

looks like this is being covered by phonegap-iphone (callback-ios, whatever). Please see the Callback JIRA for updates:
https://issues.apache.org/jira/browse/CB-53


Plotting to NTFS Drive On Mac

The pre allocation of the plot file fails for NTFS drives on MACs. Didn't seem to surprise anyone.
It would be great if the process just continued on and truncated the file to the appropriate size.
I know its slow but I'm not aware of any other approach other than moving the plotting process to windows or moving away from NTFS.  The rest of the plotting process works fine.
The old mjminer POC1 plotter didn't do the pre allocation and supported NTFS drives on macs. I wish the preallocation worked but it doesn't.

Hi,
I was looking into this, but it's not feasible. The NTFS emulation on MacOs doesnt allow bypassing the NTFS-security mechanisms that prevents fast file allocation. My advise: Allocate the files on a Windows machine using Engraver 2.0. Abort the plotting and resume on MacOs. This should work.
Johnny


Add option to toggle at global level instead of local

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.basics
Description
I’d personally prefer the toggles occur at the global level (set instead of setlocal). An option to determine which level is used for all toggles would be great. Even better if this could be configured per-toggle but I think that might not be worth the effort.

Shoot messed up the module name on submission

Thanks for the suggestion!
Indeed, 'mini.basics' has overall design of being as much "plug and play" as possible. Yet I don't think having this kind of configurability in this case is worth it.
Moreover, I'd like to encourage users to move away from parts of 'mini.basics' that they want to precisely customize (it is more or less mentioned in help). So my suggestion would be to creating your own mappings that use :set following this part of source code. If you want only part of mappings to use :set you can create them after require('mini.basics').setup() to override only part of them.
Closing as not planned.

Fair enough! Thanks for all your work


fix(SDKManager): honour script enable states - fixes #1216

The VRTK scripts that register with the SDK Manager were always
being set to enabled even if the script was set to disabled in the
editor.
This fix stores the state of the registered script and then sets it
back to that original state when the SDK Manager runs through the
behaviours registered with it. This means the enabled state of the
scripts are now remembered and set appropriately.

I think I'll let @virror test it before merging just to make sure it doesn't break anything I've not understood ;)

LGTM


[BUG]

Node number
I.7.3
Expected behavior
Not to accept my answer
Actual behavior
My answer HTHT was accepted.
Additional information
No response
Source code
dev = qml.device("default.qubit", wires=1)

@qml.qnode(dev)
def unitary_with_h_and_t():

    qml.Hadamard(wires = 0)
    qml.T(wires = 0)
    qml.Hadamard(wires = 0)
    qml.T(wires = 0)

    return qml.state()

Tracebacks
No response

[X] I have searched existing GitHub issues to make sure the issue does not already exist.

Thank you, good catch. We have some tests to fix!

It was a total accident!


Autoindent support

Hi,
Is autoindent supported. If so, how to turn it on?
Thanks,
Siarhei

Not supported yet.
You can make a Pull Request if you want.
Thanks.

I think vscode support autoindent by add brackets pair in language-configuration.json
But it will cause #6, maybe we can wait vscode issue to be fixed

@Yushiao Where is language-configuration.json for this extention? I see many such files in C:\Program Files (x86)\Microsoft VS Code\resources\app\extensions but I cannot find the file for SystemVerilog

@skazarynau If you want to modify language-configuration.json. It is at C:\Users\name\.vscode\extensions\mshr-h.SystemVerilog-0.0.12

It seems the following code might need to be used to supply indenting.  I am not sure if this is possible with a language like systemverilog.
    var rules = <vscode.LanguageConfiguration>{};

    rules.indentationRules = {
        increaseIndentPattern: /\bbegin\b/,
        decreaseIndentPattern: /\bend\b/
    };

    vscode.languages.setLanguageConfiguration('systemverilog', rules );


Update Documentation/Readme

We had some troubleshooting on IRC the other day where the problem of crashing launcher and game could be resolved by using the correct java version for the os, i.e., using a 64-bit java on a 64-bit os.
This requirement should be made clear in the documentation/readme of the launcher, maybe also a good point to start an FAQ section for.
Furthermore, investigate whether the launcher can check this requirement at startup, and log a warning/error if java and os do not match.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Issue description is heavily outdated and can be ignored.
We'll keep the issue, though, as we definitely need to update the documentation and readme.

Resolved by #498


Prefix alias (per guild?)

The current way to change the prefix is by changing it here: https://github.com/synzen/Discord.RSS/blob/master/config.json#L11
However, it might be handy to use a command to change this globally. Example:
(owner only)
~prefix . what should change the global prefix for the bot to: .rss.
Optionally, this could maybe be done per guild base. Example:
(Manage Channel)
~rssprefix . what should change the guild prefix for the bot to: .rss.
However, due to users may be forgetting what their prefix was. It would be better to keep the fallback prefix. So users can always rely on the wiki whatever their custom alias may be.
(Tip to S64: hardcode the alias prefix to be only x characters long. As some users on other bots who have this functionality made some pretty insane prefixes. Like a very long sentence or something with spaces in it.)

Sure, when I have it done I'll close the issue with the commit that'll have this implemented. It'll be per guild since it's silly for a global change because config json is the global config, and we can't have servers changing the global prefix all willy nilly

Custom prefix support has been added https://github.com/synzen/Discord.RSS/commit/c6ef7764de8deaa3835db141228f86111ceace27

<3


Solved ThreeSum in Swift

this is amazing @amannayak0007
thanks for the contribution in my repo and please share in social media and tag me on twitter


Fix upper bound of benchmark

Upper bound of max units is 1024 (see: https://github.com/automl/LCBench)
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

from benchmarking.nursery.benchmark_kdd.benchmark_main import get_transfer_learning_evaluations
metadata = get_transfer_learning_evaluations('lcbench', 'APSFailure')
print(metadata['Amazon_employee_access'].hyperparameters['max_units'].max())

1024


[Feat]: Creating contact us form

Description
currently when we click on the contact us on the navbar it's redirect to review tab rather than contact us form. I would create a contact us page with a contact us form.
Screenshots
here it is where the contact us from the navbar is located.

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

I want to contribute on contact us page or these project


🚀 Added support for Android SDK 31 & more

Summary
🚀 Added support for Android SDK 31
😄 Updated dependencies to the latest stable versions & left the kotlin-gradle-plugin to version 1.5.31, because the latest 1.6.10 causes bugs & is not yet fully supported by certain dependencies in thi project
👌 Tested the whole project in portrait & landscape mode on an Android Studio Emulator & it works just fine

@germainkevinbusiness could you please use stable version (instead of alpha version) of room library (and any other library, if required)?

Sure

I did

@germainkevinbusiness could you please use stable version (instead of alpha version) of room library (and any other library, if required)?

I just updated the libraries and tested the project on an Android Studio Emulator 👍🏾


faq/troubleshooting: fix index formatting

Change-type: patch

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

lgtm


Merge into live 12/12/2022

Update .openpublishing.publish.config.json file to make public contributions possible.

Learn Build status updates of commit 3ada8e2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Fixes broken link to "Copy without render"

I ran into a broken link and this should fix it.

Codecov Report

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

@@          Coverage Diff          @@
##           master   #912   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          17     17
  Lines         684    684
=====================================
  Hits          684    684

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 a000a8f...a8cdfa6. Read the comment docs.

Oops! Thank you @coreysnyder04! 🙇


Mark private endpoints as preview

@anthonychu : Thanks for your contribution! The author(s) have been notified to review your proposed change.

ugh wrong repo


correct the stale documentation link

PR corrects the bad documentation link that currently leads to a 404 page.
Signed-off-by: Dmitry Savintsev<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

Thanks for your contribution @dmitris :)


add verify kernel version when enable unprivileged

Fixes: https://github.com/containerd/containerd/issues/8636
add verify when enable  unprivileged kernel version whether less than 4.11.

Also, please correct the commit message:

add valide kernel version when enable unprivileged]

Could be:

validate kernel version for unprivileged icmp/port

@samuelkarp PTAL 😃

/retest

@samuelkarp PTAL 😃，thanks.

Can you squash into one commit?  Also, "valide" isn't a word.  I think "validate kernel version for unprivileged icmp/port" is a reasonable commit message.

Can you squash into one commit? Also, "valide" isn't a word. I think "validate kernel version for unprivileged icmp/port" is a reasonable commit message.

Sorry, I will squash into one commit next time.

Conflicting files
pkg/cri/config/config.go

Unfortunately looks like this needs a rebase again.

/ok-to-test

It looks like the commit here got squashed with the contents of https://github.com/containerd/containerd/pull/9287 as well.  Can you remove that?


Assign a new position to object

Hello,
What i would like to do is, is to assign a new position to an object, player when pressing a certain key.
I tried with player.pos.add({x,y}) but that didnt to anything.
Version used is 0.51.
Thank you very much,

Hi. vec2.add() returns a new vec2, won't mutate the existing. You can doplayer.pos = player.pos.add(), or player.pos = vec2(x, y), or player.move(speedX, speedY)

That was what i needed. Thank you


Tests for premisAgentXMLToObject.

Partially completes #27
I reworked how the XML is used in the tests. Now that we have more tests that need to share these resources, many of the XML 'fixtures' will be reworked in a similar fashion, and they will likely be relocated to a centrally located module.

ping @ldko @somexpert

Not sure what I think about that last test. I feel like it could be done another way. Gonna think about it a bit.

Eh, the only thing I can think of would be to add an assertion that the returned object was an Agent object, but that doesn't feel quite right. idk, I'm not a fan of tests without assertions, but I can see why that choice was made.

The assertion is there, it just doesn't use the assert keyword.

:thumbsup:

:+1:


Support for processes that are inactive by default

Ability to list a process in invoker.ini, but not have it autostarted by default
[postgres]
command = postgres -D /usr/local/var/postgres

[redis]
command = redis-server /usr/local/etc/redis.conf

[elasticsearch]
command = elasticsearch -f
autostart = false

add command can be used to start inactive processes later
invoker add elasticsearch

Hey @nisanth074 I have given you commit access to Invoker. Feel free to work on any feature you feel comfortable with.


[Feature request] Support for Aurelia

We use Aurelia heavily in our projects and having Storybook support Aurelia would be very useful. I understand there are other priorities, but just wanted to drop a request for this.
Thanks for the great work!

Aurelia has definitely been an idea that's floated around in the past. We are open to community contributions for new platform support and if you or anyone else would like to dedicate some development time for it, we would love getting a PR.

Hi @miguelzakharia Thank you for the interest and effort opening a ticket!
These types of request warm my heart : bringing the communities together here!
I've previously met with @meelisd and @eriklieben and spoke about exactly this!
I'm all for it, and want to help make it happen! In fact just today I've worked with @naipath and @stijnkoopal and @leonrodenburg to get a Storybook for Polymer off the ground! https://github.com/storybooks/storybook/pull/2225
Though I'm unable to learn and be an expert in every UI library/framework in the JS ecosystem; I'm very very interested in helping out in the way I can: Pairing with people knowledgeable in Aurelia and together taking it step by step to build the support.

until this feature exists there is something similar that is just for Aurelia: Front End Creator

Is Aurelia support still in the pipeline?
Some documentation as to how to add support for new front-end frameworks would be great.

@rowellx68 If you're interested in working on this, jump on our Discord and we can probably help you get going. It's still not documented, but it's a well-traveled path by this point. cc @ndelangen

Hey @rowellx68, have you started on the Aurelia support? I'd be keen to help out possibly?

@tomtomau I've got something very very basic here . Any help would be great. Currently battling the Cannot find 'fs' module issue.

Anyone interested Aurelia 2 support is in my fork.  Once post alpha we will clean up and hopefully a PR will be accepted.

@brandonseydel A PR would definitely be accepted, do you need my help?

Take a look in my repo.  I am cleaning up some things still, but it is working well it seems.

After Aurelia 2 releases and this is merged can we get a spot on the homepage and in docs?

@brandonseydel Absolutely!


(B)setup the database[0.5]

edit the database.yml file according to your DB information

https://github.com/DuaneDave/Bike-rental-api/pull/2


Repeat first character on completion.

Issue: repeating first character when auto completing the first word on the line.
Reproducing:  Youtube Video
Atom version: 0.186.0
Package version: 2.4.1

This could be (and most likely is) an issue with the provider you're using. What packages do you have installed?

Do you want me to list all installed packages on manjaro linux?
[image: photo]
Ahmed Kamel
Associate System Administration Manager, Arabhosters
m:+201140773046 |<EMAIL_ADDRESS>|
w:http://arabhosters.com | a: Head
Office: 26 Ali Amin St. Nasr City, Cairo Egypt
http://eg.linkedin.com/pub/ahmed-kamel/
https://www.linkedin.com/profile/view?id=130626886
"Success is to be measured not so much by the position that one has reached
in life... as by the obstacles which he has overcome while trying to
succe...
http://www.quotesdaddy.com/quote/98613/booker-t-washington/success-is-to-be-measured-not-so-much-by-the-position
IMPORTANT: The contents of this email and any attachments are confidential.
They are intended for the named recipient(s) only. If you have received
this email by mistake, please notify the sender immediately and do not
disclose the contents to anyone or make copies thereof.
Get a signature like this:
http://ws-stats.appspot.com/r?rdata=eyJydXJsIjogImh0dHA6Ly93d3cud2lzZXN0YW1wLmNvbS8/dXRtX3NvdXJjZT1leHRlbnNpb24mdXRtX21lZGl1bT1lbWFpbCZ1dG1fY2FtcGFpZ249cHJvbW9fNDUiLCAiZSI6ICJwcm9tb180NV9jbGljayJ9
Click
here!
http://ws-stats.appspot.com/r?rdata=eyJydXJsIjogImh0dHA6Ly93d3cud2lzZXN0YW1wLmNvbS8/dXRtX3NvdXJjZT1leHRlbnNpb24mdXRtX21lZGl1bT1lbWFpbCZ1dG1fY2FtcGFpZ249cHJvbW9fNDUiLCAiZSI6ICJwcm9tb180NV9jbGljayJ9
[image: photo]
Ahmed Kamel
Linux System Engineer, Odesk
m:+201140773046 |<EMAIL_ADDRESS>http://eg.linkedin.com/pub/ahmed-kamel/ http://twitter.com/Abukameleo
https://twitter.com/Abukameleo
https://www.linkedin.com/profile/view?id=130626886
"Don't wait until everything is just right. It will never be perfect. There
will always be challenges, obstacles and less than perfect conditions. So
...
http://www.quotesdaddy.com/quote/1041647/mark-victor-hansen/dont-wait-until-everything-is-just-right-it-will-never
IMPORTANT: The contents of this email and any attachments are confidential.
They are intended for the named recipient(s) only. If you have received
this email by mistake, please notify the sender immediately and do not
disclose the contents to anyone or make copies thereof.
On Fri, Mar 13, 2015 at 8:56 PM, Joe Fitzgerald<EMAIL_ADDRESS>wrote:

This could be (and most likely is) an issue with the provider you're
using. What packages do you have installed?
—
Reply to this email directly or view it on GitHub
https://github.com/atom-community/autocomplete-plus/issues/307#issuecomment-79250485
.

Do you want me to list all installed packages on manjaro linux?

No I'm referring to atom packages. Run apm list, and paste the output.

Built-in Atom packages (85)
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>/home/abukamel/.atom/packages (17)
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>[image: photo]
Ahmed Kamel
Associate System Administration Manager, Arabhosters
m:+201140773046 |<EMAIL_ADDRESS>|
w:http://arabhosters.com | a: Head
Office: 26 Ali Amin St. Nasr City, Cairo Egypt
http://eg.linkedin.com/pub/ahmed-kamel/
https://www.linkedin.com/profile/view?id=130626886
"Success is to be measured not so much by the position that one has reached
in life... as by the obstacles which he has overcome while trying to
succe...
http://www.quotesdaddy.com/quote/98613/booker-t-washington/success-is-to-be-measured-not-so-much-by-the-position
IMPORTANT: The contents of this email and any attachments are confidential.
They are intended for the named recipient(s) only. If you have received
this email by mistake, please notify the sender immediately and do not
disclose the contents to anyone or make copies thereof.
Get a signature like this:
http://ws-stats.appspot.com/r?rdata=eyJydXJsIjogImh0dHA6Ly93d3cud2lzZXN0YW1wLmNvbS8/dXRtX3NvdXJjZT1leHRlbnNpb24mdXRtX21lZGl1bT1lbWFpbCZ1dG1fY2FtcGFpZ249cHJvbW9fNDUiLCAiZSI6ICJwcm9tb180NV9jbGljayJ9
Click
here!
http://ws-stats.appspot.com/r?rdata=eyJydXJsIjogImh0dHA6Ly93d3cud2lzZXN0YW1wLmNvbS8/dXRtX3NvdXJjZT1leHRlbnNpb24mdXRtX21lZGl1bT1lbWFpbCZ1dG1fY2FtcGFpZ249cHJvbW9fNDUiLCAiZSI6ICJwcm9tb180NV9jbGljayJ9
[image: photo]
Ahmed Kamel
Linux System Engineer, Odesk
m:+201140773046 |<EMAIL_ADDRESS>http://eg.linkedin.com/pub/ahmed-kamel/ http://twitter.com/Abukameleo
https://twitter.com/Abukameleo
https://www.linkedin.com/profile/view?id=130626886
"Don't wait until everything is just right. It will never be perfect. There
will always be challenges, obstacles and less than perfect conditions. So
...
http://www.quotesdaddy.com/quote/1041647/mark-victor-hansen/dont-wait-until-everything-is-just-right-it-will-never
IMPORTANT: The contents of this email and any attachments are confidential.
They are intended for the named recipient(s) only. If you have received
this email by mistake, please notify the sender immediately and do not
disclose the contents to anyone or make copies thereof.
On Fri, Mar 13, 2015 at 9:47 PM, Joe Fitzgerald<EMAIL_ADDRESS>wrote:

Do you want me to list all installed packages on manjaro linux?
No I'm referring to atom packages. Run apm list, and paste the output.
—
Reply to this email directly or view it on GitHub
https://github.com/atom-community/autocomplete-plus/issues/307#issuecomment-79301512
.

I believe your issue is with this code: https://github.com/fallenhitokiri/autocomplete-plus-jedi/blob/71fa5f6415271e4fc4d4f0dfa720b8d288867bb6/lib/jedi-provider.coffee#L46-L48
Suggest you create an issue at https://github.com/fallenhitokiri/autocomplete-plus-jedi/issues/new

Thanks, i just did.


[DO NOT MERGE] Creating test harness for e2e tests.

@bryanmau1 PTAL, just want to sanity check the idea with you before I continue.

We will write e2e tests as regular py.test tests under tests/e2e-app/tests (we can put the elsewhere if you want, we just have to symlink so docker build will work).
To run the e2e tests, we'll deploy the app. Once the app is deployed, we'll run a small python program that will send a request to the e2e app that will in turn run the tests and return the output and status to the local program.

/cc @andrewsg

seems good to me!  Looking forward to this.

@bryanmau1 this is ready for review and merge.

LGTM


baetyl-225 update node struct

Codecov Report

Merging #213 (7513dac) into master (d67f823) will increase coverage by 0.13%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##           master     #213      +/-   ##
==========================================
+ Coverage   67.77%   67.91%   +0.13%
==========================================
  Files          71       71
  Lines        5117     5111       -6
==========================================
+ Hits         3468     3471       +3
+ Misses       1188     1179       -9
  Partials      461      461

Impacted Files
Coverage Δ

service/sync.go
47.94% <ø> (+3.00%)
:arrow_up:

plugin/kube/application.go
86.20% <81.81%> (-1.45%)
:arrow_down:

Continue to review full report at Codecov.

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


How to train a medium or large model with limited GPU capacity?

I have cards of 24 GB each. The error appears when trying to train the medium model torch.cuda.OutOfMemoryError. Is there a way to train a medium or large model on my cards? I will be glad to any advice!

Same issue here. I believe you have to use fsdp = true & autocast = false.

@Saltb0xApps Unfortunately this doesn't work for me when training a medium model. I tested this on a small model and noticed that the overall memory consumption did not change at all, with all other parameters being the same. I used the smallest batch size.

Try lowering the batch_size and/or epochs


Running on windows

Hi,
Could someone else please try to load the application on Windows? I am trying to start the app, and it does load. I managed to get the dependencies running, and I am not modifying the Setup scripts to download and install everything by itself. I prepared the dependencies powershell script which downloads the docker and docker-machine binaries, and modified the application code to use them. I also modified the code so it could find the VBoxManage binary and manage VirtualBox on windows. It is still missing a step of installing VirtualBox on it's own, so you need to install VirtualBox yourself, but other than that the application should start.
And now here comes the problem. When I run the application it seems that the app starts with two instances of react. I see two instances of
"Download the React DevTools for a better development experience: http://fb.me/react-devtools", source: c:\Users\dderen\Documents\GitHub\kitematic\node_modules\react\lib\React.js (143)

in the console log, and later when react tries to do anything connected with components it fails with error:
"TypeError: Cannot read property 'firstChild' of undefined", source: file:///C:/Users/dderen/Documents/GitHub/kitematic/build/Main.js (47)

And the stacktrace points to react sources.
I debugged a bit more, and at application startup it seems that react is loaded twice, once by the kitematic app itself, and once by react-router, even though it was already loaded.
I tested this on Mac as well, and there is no issue, react is loaded only once, by the kitematic app. It is not loaded by react-router (I mean require command just takes the existing instance, instead of trying to initialize a new one). I also tested it on two different Windows boxes (one with Win 7 and one with Win 8.1), and the issue persists on both.
I would appreciate any help with this issue, and I have a problem even finding the root cause of it. I am not sure if it is connected with the Atom-Shell, or with react or with react-router or what...
Can anyone else test this setup on Windows as well to see if the behavior is reproducible globally?
My branch with windows changes is available at: https://github.com/domderen/kitematic/tree/win_client
Thanks in advance!

Hey, the issue was solved by my friend @KamilSzot. He found out that the usage of
<script src="Main.js"></script>

in Index.html causes the issue. I changed the setup to be
<script src="Startup.js"></script>

and added new file Startup.js which has the following content
require('./Main');

And now the application works as expected. I will confirm that it also works on Mac, and prepare a pull request to fix the issue.

Sorry for the delay in getting back. I've been following your work on Windows - this is awesome!
Feel free to drop by #kitematic on IRC or our gitter channel if you'd like to chat. I'm going to take your branch for a spin soon!

Hey, I think this issue thread can be closed now, as the PR has need merged.

Awesome, closing! Thx @domderen !

I have the same issue with the latest version of kitematic downloaded just now.
The app opens nothing is shown in the window and in the console I get the same errors:
[7172:0511/202022:INFO:CONSOLE(48)] "Warning: owner-based and parent-based contexts differ (values: `undefined` vs `function (props, context)

What am I missing?
Removing the react router require removes the error but ...

Just did what @domderen suggested and now it's fine. I'm a total noob for not understanding why but still why is this happening?

This issue was found and documented by my friend @KamilSzot here: https://github.com/atom/electron/issues/1289
Though to be honest the error message that you presented is unknown to me, I don't remember anything like that.


Upgrade our hubs to Z2JH 2 / JupyterHub 3.0

Background and proposal
Our basehub chart defines the version of Zero to JupyterHub that we pull in here:
https://github.com/2i2c-org/infrastructure/blob/6da898b2892c9e81b6afa91d1ade3d2a0e56a472/helm-charts/basehub/Chart.yaml#L1-L15
However, Z2JH 2.0 will be released relatively soon, and is going to bring with it JupyterHub 2.x.
We should upgrade to Z2JH 2.0 as long as we're confident that it will be fit for production in our hubs. However, there may be some coordination that we need to take with either our basehub etc helm charts, or with the user images that we're serving.
This is an issue to track that effort so that we know what needs to be done!
Implementation guide and constraints
Here's the issue tracking the release of Z2JH 2.0. I believe that @consideRatio has been putting a lot of time into this one (thanks 🙏🙏🙏) and maybe he can advise on progress as we near a release. Looking at that issue, it seems quite close.
Here's an example of a problem that occurred because somebody bumped jupyterhub-singleuser too early, on a hub that did not yet have JupyterHub 2.0.
Updates and ongoing work

[x] Wait for Z2JH 2.0 to be released (and/or help make it happen!) (ref: https://github.com/jupyterhub/zero-to-jupyterhub-k8s/issues/2386)
[ ] Compile a list of things we should double-check in our hub configuration to make sure an upgrade won't break things.
[ ] Make the necessary upgrades in our config + any community hub configs.
[ ] Hope that everything doesn't explode.
[ ] Check if https://github.com/2i2c-org/infrastructure/issues/1643 is actionable now

We should upgrade to Z2JH 2.0 as long as we're confident that it will be fit for production in our hubs. However, there may be some coordination that we need to take with either our basehub etc helm charts, or with the user images that we're serving.

To ensure user images have JupyterHub 2 installed ahead of time is a very big deal of the 2i2c prep i think.

I wonder if https://github.com/jupyterhub/jupyterhub/pull/3809/files#diff-514e695392b67987ea4b144e522365c14062bb806eaa0bfe8f1d24175cabe072R504-R514 could possibly impact if things work or not. I'm super vague on the context and technical details or the errors observed that makes us want to pin JupyterHub 1 for some situations.

I upgraded berkeley to master a week or so ago and it worked great.

JupyterHub 3 beta 1 has been released, and the z2jh Helm chart has started using JupyterHub 3 beta1.
User images
Users can have version >=2.3.1 (probably a lower version as well, not sure what version is the lower bound), but the key point is that users doesn't require jupyterhub 3 to be installed.
So, updating to an even more modern version of z2jh should be fine without trouble.

In Slack, Erik noted that version 2.0.0-beta.1.n024.he38f180b has been released. This release contains a fix for https://github.com/jupyterhub/kubespawner/issues/627 which was causing us a lot of problems in non-regional clusters. See https://github.com/2i2c-org/infrastructure/issues/1103, https://github.com/2i2c-org/infrastructure/issues/1300, https://github.com/2i2c-org/infrastructure/issues/1616.

I guess this one should be open until we migrate to the released version of 2.0 that actually happened 3 days ago, correct?

@damianavila yep!

This is done.


KeyError of _non_zero_coefficients_index_dict

KeyError occurs when I execute the following script.

MacOS v12.5.1
Python v3.9.13
codable-model-optimizer v0.1.3

script
from codableopt import *

# set problem
problem = Problem()

# define variables
n = 10
x = [DoubleVariable(name=f"x{i}") for i in range(10)]

# define objective function
def obj_func(x):
    print(x)
    return sum(x[i] * x[i] for i in range(n))

# set objective function and its arguments
problem += Objective(obj_func, args_map={'x': x})

solver = OptSolver()

# generate optimization methods to be used within the solver
method = PenaltyAdjustmentMethod(steps=40000)

answer, is_feasible = solver.solve(problem, method)
print(f'answer:{answer}, answer_is_feasible:{is_feasible}')

error
Traceback (most recent call last):
  File "/Users/hoge/cob_tmp.py", line 22, in <module>
    answer, is_feasible = solver.solve(problem, method)
  File "/usr/local/lib/python3.9/site-packages/codableopt/solver/opt_solver.py", line 88, in solve
    answer, is_feasible = opt_solver.solve(
  File "/usr/local/lib/python3.9/site-packages/codableopt/solver/optimizer/optimization_solver.py", line 85, in solve
    state = OptimizationSolver.__optimize(
  File "/usr/local/lib/python3.9/site-packages/codableopt/solver/optimizer/optimization_solver.py", line 151, in __optimize
    return optimizer.optimize(debug, debug_step_unit)
  File "/usr/local/lib/python3.9/site-packages/codableopt/solver/optimizer/optimizer.py", line 80, in optimize
    proposed_move_list = self._method.propose(self._state, step)
  File "/usr/local/lib/python3.9/site-packages/codableopt/solver/optimizer/method/penalty_adjustment_method.py", line 100, in propose
    self._proposal_list = variable.propose_low_penalty_move(state)
  File "/usr/local/lib/python3.9/site-packages/codableopt/solver/formulation/variable/solver_double_variable.py", line 58, in propose_low_penalty_move
    lower, upper = state.problem.constraints.liner_constraints.calc_var_range_in_feasible(
  File "/usr/local/lib/python3.9/site-packages/codableopt/solver/formulation/constraint/solver_liner_constraints.py", line 152, in calc_var_range_in_feasible
    non_zero_indexes = self._non_zero_coefficients_index_dict[var_index]
KeyError: 9

Thanks for letting me know about the bug. I fixed it in the merge.

Thank you for fixing it!


Use Zustand for React state management

introduce the library
create a store (for App only for the moment)
move the App component to use Zustand instead of local hooks

The value of reducing prop drilling isn't realized with this small
change but it will avoid merge conflicts.

Codecov Report

Merging #169 (6118782) into main (d047c5a) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 6118782 differs from pull request most recent head 13fbe64. Consider uploading reports for the commit 13fbe64 to get more accurate results

@@           Coverage Diff           @@
##             main     #169   +/-   ##
=======================================
  Coverage   93.88%   93.88%
=======================================
  Files         110      110
  Lines        2796     2796
=======================================
  Hits         2625     2625
  Misses        171      171

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 d047c5a...13fbe64. Read the comment docs.


Docs don't build on readthedocs

Currently, docs from master don't build on readthedocs. However it builds perfectly for PRs, so it is tricky to debug the problem

Probably it was readthedocs problem...


Bundling gRPC Extension

Feature Request
Hi, is it possible to bundle/include gRPC extension with the runtime?
https://blog.remirepo.net/pages/PECL-extensions-RPM-status page lists the extension.

Thanks!

Done. Just re-deploy. :-)

You're amazing.
Thanks!
Is there a donation option?

Yup, https://github.com/sponsors/f3l1x
Thank you.


Release v1.9.0-rc3

Description

Test

CI

Additional Information
Tradeoff

Potential improvement

Note, this includes https://github.com/kubewarden/helm-charts/pull/349, I can rebase and drop that.

Removed https://github.com/kubewarden/helm-charts/pull/349 from this PR.


.col-xs has width: 100%

This was added in Alpha.5 and changes the flexbox behavior. The columns are now always stacked.
If this change was on purpose then the docs for flexbox grid should be updated. But I think this is a bug.

Works for me with the latest from v4-dev. See https://jsbin.com/jireka/edit?html,css,output.

My columns had flex: 0 0 auto. So strictly speaking they were not on the grid anymore and I guess I should not have used row and col.


F/beacons table

Context
This PR creates the back office table view as per ticket.

Changes in this pull request

Copypasta material-tables component from @stuart-madetech 's spike. (https://material-table.com/#/)
Define domain interfaces for data model.
Create a BeaconsGateway shell to stub fake beacon data.
Use-case functions to display dates and uses as per wireframe.

Guidance to review

Testing is slightly more complex than the GovUK service frontend because there are async calls.  Docs here.
Was able to unit test the table component and it's component parts, so didn't reach for Cypress.  Appreciate your thoughts on this?
The ticket says "20 rows by default"; I think this is too many.  Will check with Ant / Konrad / Linda when merged.

Link to Trello card
https://trello.com/c/P7lHAKHt
Things to check

😍 This is great, thank you for setting up so much for us to build the backoffice on!
I have a couple little things I wasn't sure about:

The URL for a Beacon record - should the Id on the end be a Beacon Id or the Hex Id (my comment here)

That's a great shout!  Will rectify 👍🏻

Would formatDate and formatUses (really cool btw!) be better off in like utils or another place, rather than the useCases dir?

In my head use cases are like actions the user wants to do and I'm not sure I'd say either of those really fit that 🤔

I was more thinking that the date format the MCA uses is a domain concern, e.g. "As the MCA we want dates to be formatted in this particular way", so put it in the use cases dir.  I try and avoid putting things in a utils.ts unless they're generic/universal (like titleCase()).  On reflection I now think formatUses is a helper function for the <BeaconsTable> component (because the format is only currently for display in that table's uses column), so will move formatUses to there.  So hard to get this stuff right!!!


Get user-assigned device name on iOS

To get the user's actual device name on iOS, we need to request an entitlement from Apple: https://developer.apple.com/documentation/bundleresources/entitlements/com_apple_developer_device-information_user-assigned-device-name
Todo

[ ] Implement showing the user-assigned device name in the UI in the app connected state
[ ] Apply for the entitlement here: https://developer.apple.com/contact/request/user-assigned-device-name/

This isn't super high priority, a generic device name will suffice since we already know which Actor it belongs to.

We actually don't need anything more from the client for this -- we can use the anonymized device name as-is and then append a further identifier to it in the portal or in connlib somehow.
This avoids having to "collect user information".


[UI/UX] : Add animated icons in the bottom navigation bar

Assign me this issue under JWOC @Abhigyan103


az tag update on AKS node pool update the tags on the cluster not the node pool

Related command
az tag update --operation="merge"
Describe the bug
When the resource is an AKS node pool, the command updates the tags of the cluster not those of the node pool.
To Reproduce
Run az tag update on an AKS node pool resource.
Expected behavior
Updates the tags of the node pool.
Environment summary
MSI / 2.43.0 / Windows 11 22623.1037 / PowerShell 7.3.1
Additional context
N/A

route to CXP team

@mdufourneaud The az tag update command is to update the tags for an ARM Resource but that isn't supported for sub resources. If you are instead trying to update the tags property of the node pool which applies them to VM Scale set, then you need to use the az aks nodepool update command instead.


Return Largest Numbers in Arrays - should check for negative numbers

Challenge Return Largest Numbers in Arrays.
On the automated tests for passing this exercise, you should also have a test with only negative numbers also, like: largestOfFour([[-1, -3, -6, -25], [13, 27, 18, 26], [32, 35, 37, 39], [1000, 1001, 857, 1]]);
If you don't do array.sort and try to find the max in each sub-array, if you init a tempMax with 0 for each sub-array, the test will validate. But that would be fake validate, since the max from first sub-array would return 0.

@Jamayk Thanks, this is a great addition! :+1:
Since you reported it, would you like to make a contribution to add another test case? Me and other contributors would be happy to guide you through the process! :smile:

@Greenheart  Sure; I'm new around, just started a few days ago. Hit me :D

@Jamayk Glad to hear that!
CONTRIBUTING.md can help you get started.
When you're all set, add another test case (a string with the assertion) somewhere around these lines and make your commit. See the other test cases for reference.
If you have any questions, please come chat with us in the Contributors Chat Room.
Happy Coding! :smile:

@Greenheart Well, I think I did it; can you check please?
I can update my commit, but I think it should be fine this way?
Thanks!

@Jamayk You added a new line at the end of the file, just go to the bottom of the file and press delete and you should be fine, just make sure the closing brace is still there. Also you submitted a commit against your forked version of the repo. Make sure you add this repo as the upstream.
You can do so by running this command:
git remote add upstream https://github.com/freeCodeCamp/freeCodeCamp.git
then
git push origin your-branch-name

@Jamayk Sorry for the late reply, I somehow lost your notification. I can check, if I can find it! :smile:
It seems like you have cloned (downloaded a copy of) the freeCodeCamp/freeCodeCamp repository instead of forking (creating a personal version of a repository) and then cloning your fork.
Please make sure to follow the steps of the contribution guidelines to use a common workflow - and let us provide better help :blush:
Reach out if you need anything!
@donofriov Thanks for helping out! :+1: :tada:

@Greenheart Hey man! I did fork it then changed it online (hence space problem and commit with wrong upstream).
I will make myself a proper setup and (in the meantime, I did read the full contribution guidelines) then I can contribute easier when I have the time and knowledge for it.
Will get back to you when I get a chance.

@Greenheart I think it's done; hardest part was making the project work on my local (windows), so I can test my solution, and also run the automated tests. 'A bit' outdated dev, so this was my first MongdoDB, Node installment, and my first PR on an Open Source project. Fingers crossed!
Now that I have everything working, I only need to keep my Fork updated, and I can contribute easier when   possible.

@Jamayk Yeah, the development setup is by far the biggest barrier to contributing fixes to a project. I'm really happy that you saw it through and got this PR submitted! Great job! :smile: :1st_place_medal:
With a project like this, there's almost always something that can be done, so feel free to help out more if you want to :smile:

To keep your local clone of the project updated, all you need to do is to

Make sure you're on branch staging (git checkout staging)
Make sure to have the git remote called upstream set to the git-URL to the freeCodeCamp/freeCodeCamp repository. (git remote -v can be used to verify). If not, run git remote add <URL>.
Run git pull upstream staging to pull down latest changes.

Then you're ready to create a new branch and work on a new PR :blush:


Don't call find_entries as a scope in rbac

Currently ext_options flows through Rbac#search to Rbac#method_with_scope to VimPerformanceDaily.all
VimPerformanceDaily.find_entries is the only one that uses ext_options
The issue is not ext_options per se, but the use of an all method
that needs all parameters passed into it. With this, the cond can't use basic AR method
(e.g.: where, limit) and forces sql string manipulation.
After this change, we have more flexibility in Rbac to use scopes.

This fixes the travis deprecation warning around VimPerformanceDaily.all.
I was tempted to remove all but am a little gunshy right now.
/cc @Fryguy FYI

@kbrock I think you meant per se instead of per say.

Oh i see, so since ext_options is no longer passed, then VimPerformanceDaily.all is never called?  Is that dead code, then, after this PR?

@Fryguy yes, the all should be dead code.
But we have so many places in our code that set miq_report.db = "VimPerformanceDaily" and I'm not sure if there is a way to get to VimPerformanceDaily.all from there.
If there isn't a way to get to that method, then that opens up a ton of opportunities.

👍


Add spacing helper

Does this make sense to add?
Example usage:
function App() {
  return (
    <div
      {...spacing.inner({ vertical: 30 })}
      {...spacing.outer(15)}
    >
      Content
    </div>
  )
}

Checklist:

[ ] Test added / Snapshots updated
[ ] Documentation added / updated

Great idea! Maybe it's worth thinking it spacing steps like 1px 3px 5px 10px 15px?

Great idea! Maybe it's worth thinking it spacing steps like 1px 3px 5px 10px 15px?

being more verbose makes sense somehow.... we usually use 5px steps for margins/paddings (at least in 90% of the use cases) - maybe something like this?
{
  "s" => 5,
  "m" => 10,
  "l" => 15,
  "xl" => 20,
}

I think it would be sensible to provide these as defaults.
Like:
export const SPACING = {
  s: 5,
  m: 10,
  l: 15,
  xl: 20,
}

👑 !


migrated to swift 4.0

Codecov Report

Merging #80 into master will decrease coverage by 0.14%.
The diff coverage is 89.47%.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
- Coverage   84.66%   84.52%   -0.15%
==========================================
  Files           8        8
  Lines         587      588       +1
==========================================
  Hits          497      497
- Misses         90       91       +1

Impacted Files
Coverage Δ

EventSource/EventSource.swift
93.3% <89.47%> (-0.42%)
:arrow_down:

Continue to review full report at Codecov.

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

@volbap do you want to review and merge this??


Update readme for those who don't know about extension.

If you deployed the application on live server then you have to update domain name of deployed application in extension extension/manifest.json file.
externally_connectable.matches array ["*://localhost/*"] to ["*://yourdomain.com/*"] and launch the application in chrome web store.

Thank you.


Write Spectral Library not working

Describing the issue
Selecting "Write SpectralLibrary" in the search task doesn't output a spectral library. Search works perfectly fine with the exception of that.

Protein database

Uniprot human fasta  + MetaMorpheus contaminants.  Fixed carbamidomethyl on C, variable M(ox).

Mass spectrometry file types

Thermo Orbitrap Eclipse .raw ,  ITMS2 + FAIMS.

Settings

.toml attached
Task1-SearchTaskconfig.toml.txt

fixed in the next release


Allow imports from a given nested folder

Hello,
Is it possible to write such a rule?

Feature cannot import files from other features unless they are in the features/shared folder.

I tried this configuration but unfortunately it doesn't work.
 "settings": {
        "boundaries/elements": [
            {
                "type": "features",
                "pattern": "features/*",
                "mode": "folder"
            },
            {
                "type": "featuresShared",
                "pattern": "features/shared/*",
                "mode": "folder"
            }
        ],
        "import/resolver": {
            "typescript": {
                "alwaysTryTypes": true
            }
        }
    },
    "rules": {
        "boundaries/element-types": [
            2,
            {
                "default": "disallow",
                "rules": [
                    {
                        "from": "features",
                        "allow": [
                            "featuresShared"
                        ]
                    }
                ]
            }
        ],
    }

.
├── ...
└── 📂 src
    └── 📂 features
        ├── 📂 shared
        ├── 📂 Feature1
        └── 📂 Feature2

Hi @Igorkowalski94 , could you please provide more details? You can enable the debug mode in order to see which element type the plugin is assigning to each file by using the ESLINT_PLUGIN_BOUNDARIES_DEBUG environment variable.
ESLINT_PLUGIN_BOUNDARIES_DEBUG=1 npm run lint

I have the following structure:
.
├── ...
└── 📂 src
    ├── ...
    └── 📂 features
        ├── ...
        ├── 📂 Feature1
        │   ├── 📄 Feature1.tsx
        │   ├── 📄 feature1.types.ts
        │   ├── 📂 helpers             // Helpers only for Feature1 folder
        │   └── 📂 components          // Components only for Feature1 folder
        │        ├── ...
        │        ├── 📂 Child1         // Same structure as Child2
        │        └── 📂 Child2
        │            ├── 📄 Child2.tsx
        │            ├── 📄 child2.types.ts
        │            ├── 📂 helpers           // Helpers only for Child2 folder
        │            └── 📂 components       // Components only for Child2 folder
        │
        └── 📂 Feature2 // Same structure as Feature1

I would like to achieve the following rules:

The Feature1 can import Feature2.tsx and feature2.types.tsx but cannot import their components and helpers.
The Feature2 can import Feature1.tsx and feature1.types.tsx but cannot import their components and helpers.
Feature1.tsx can import Child1.tsx and Child2.tsx but cannot import their components and helpers.
Child1 folder can import Child2.tsx and  child2.types.ts but cannot import  Child2  components and helpers.
Child2 folder can import Child1.tsx and  child1.types.ts but cannot import  Child1  components and helpers.
The entire components and helpers folder can import feature1.types.ts

Yes, it is possible, but you should try to debug your configuration in order to know if your elements are being categorized properly. I suspect that maybe all of them are being categorized as features because they match first the features pattern, but I'm not sure. Could you please provide logs?

@Igorkowalski94 were you able to resolve this? I am having a similar issue.

@rishitank Yes, I created my own solution eslint-plugin-project-structure. I'm closing the issue.


Create API to handle multiple IDs search

Implement multiple IDs search for home page search bar.

This will be implemented in the entity endpoint.


Auto warmup not working on IOS build

IOS build crashes at startup with this message:
ExecutionEngineException: Attempting to call method 'Morpeh.TypeIdentifier`1[[MyTestComponent, Assembly-CSharp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]]::Warmup' for which no ahead of time (AOT) code was generated.
  at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00000] in <00000000000000000000000000000000>:0
  at Morpeh.WorldExtensions.InitializationDefaultWorld () [0x00000] in <00000000000000000000000000000000>:0
UnityEngine.UnhandledExceptionHandler:PrintException(String, Exception)
UnityEngine.UnhandledExceptionHandler:HandleUnhandledException(Object, UnhandledExceptionEventArgs)
System.UnhandledExceptionEventHandler:Invoke(Object, UnhandledExceptionEventArgs)

I fixed this wrapping these 'warm types' codes in InitializationDefaultWorld method with !UNITY_IOS define. Not sure if this problem exists on other AOT platforms too

@H-man is it develop build or release?

It must be work okay.
Warming up is needed only for components that are created through the inspector UniversalProvider or Singleton.
In the latest develop, the exception is replaced with a warning so that the user understands which types are not warmed up.
Typically, types that are not used anywhere in the code are not warmed up.

Tried today using latest Develop branch and no crashing anymore. I think this issue can be closed now.


Fix coverage CI break.

It looks like there is some weird interactions involving coverage==6.2 -> 6.3 and notebook==6.4.7 -> 6.4.8. This pins them back.
Opened: https://github.com/quantumlib/Cirq/issues/4897

Automerge cancelled: A status check is failing.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3425

Questions ?
Please refer to the Backport tool documentation


Add --port option to dotnet try

In the Program.cs, based on if we are in tool mode or not we decide the url that will be used.
In the Startup.cs we use the ServerFeatures property to know the url at which the application is running and then launch the browser on the same when we are in the try mode.

The approach followed above will not work in IIS. However the configuration that we have currently uses IIS express for hosted mode, hence this code path is not called in that case.

Is there a way for us to have an ApiViaHttpTest for the auto port selection?

I have added a unit test around the part that determines the urls


[FR] Support snap

Highlight groups has been recently added to snap, https://github.com/camspiers/snap/issues/9#issuecomment-864491398 - do you think you could come up with some good looking overrides for it? Thanks!

Seems like snap is rather dead. Gonna close it for now since I am not even using that plugin any more.


bug(cdkTrapFocus): cdkTrapFocusAutoCapture has no effect after 'init' step

Expected Behavior
I can have binding for cdkTrapFocusAutoCapture input.
And if I have input that is disabled at start and gets enabled later, in simple case I can pass myInputControl.enables to cdkTrapFocusAutoCapture input and it will get focus once it's enabled.
Actual Behavior
cdkTrapFocusAutoCapture input has effect only once when the focus trap initialized. And if later the value for this input has changed - it has no effect.
Environment

Angular: 9.1.11
CDK/Material: 9.2.4

Looks like the related PR got merged in, but I forgot to mark this issue as resolved.


TODO:  Add second level slash command parameters handling

Add the processing logic for second level command parameters:
/bbbmeet  join <more subcommands>
/bbbmeet list  video
/bbbmeet play weekly
...  and so on.

Taking this on :)

Thanks!    Assigned :)

a reminder to add back your PR ;-)


Call getSchama multiple times/dynamically

Hello,
I have a use case where I only want to populate the schema when the user drags/uses a table. The reason being our schema discovery for data sources is very time consuming.
Is there a way to implement this behavior? (call getSchama() & update schema multiple times/dynamically)
Another question is how could I include more user inputs (for e.g., custom query, etc.) after the initial interactive phase? Like invoking interactive phase programmatically to incorporate more user inputs?
Thanks,
Tian

Or another way to ask: is there a way to update Table.tableInfo.columns?
In the API, it says
/** @member {Object} Information about the table which has been requested. This is
       guaranteed to be one of the tables the connector returned in the call to getSchema. */
      this.tableInfo = tableInfo;

Hey @welkinlan,
Unfortunately there is not a way to do what you are asking with regards to dynamic schemas.  The only way it can be  updated is if the user edits the connection and your WDC has the chance to reload.
There also isn't any support for additional interactions by the user after the interactive phase has concluded.  So you'll need to gather everything up front.  I'll pass your request on to the PMs on my team though.
Thanks,
Brendan

Hi @lbrendanl, thanks for your answer.
Could you be more specific on "The only way it can be updated is if the user edits the connection and your WDC has the chance to reload."
Thanks,
Tian


Faster RCNN crash due to no object detected

Not really an urgent bug. Just keep a note here.
Index out of range error when the faster rcnn model does not detect any object here.

Fixed!


Update active-directory-compare-azure-ad-to-ad.md

verb-subject agreement.  "credentials.. are based"

@ScriptingPro : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks @ScriptingPro.


Improve upon intent of outdir: Config file + Clean

Background
The original intent of #4 was to improve upon a pain point I felt existed with generating files in the source tree.

Had to add *.css.d.ts to your .gitignore. And repeating it for other extensions.
Cleaning the repository would require additional work
Some folks just prefer less files when looking at their IDE for "real" files

However, upon actually using the alternative structure, I ran into more TypeScript issues than expected. Essentially ./foo.module.css is looked for exactly in that directory. It seems that it was not resolved via other names such as my/path/foo.module.css or similar.
While I have NOT dove deeper into the TS side, this issue led to something that may benefit the tool more.
Config file + subcommands
Introducing a config file (likely via cosmiconfig) will centralize the configuration for the pattern. The tool could then include a clean subcommand that deals with (2) issue.
The config also helps the future potential watch command (although I was originally leaning away from it).
This does not address the .gitignore case, although a init command would solve that.
Goals for this issue

[ ] Add cosmiconfig
[ ] Add clean command
[ ] Alias root command to generate and/or get
[ ] Consider init command

Note: lilconfig over cosmiconfig seems to be a trend for smaller dependencies. By default, it does not support YAML. For a smaller download size, I will gladly give up YAML (and adding it later is backwards-compatible).
Edit later (never hit comment). I added js-yaml to support YAML config, which seems to be common. Was easy enough to maintain (dependency + function + array).
PR #28 covers initial config adding.


reports: refresh remote anisette headers

When reusing the account instance with a RemoteAnisetteProvider, the provider only fetches the anisette data once, which I've empirically noted to only be valid for something like 75 seconds, eventually resulting in an authentication error:
DEBUG:root:Fetching reports for 1 keys
INFO:root:Got 401 while fetching reports, redoing login
INFO:root:Attempting authentication for user
DEBUG:root:Starting authentication with username
DEBUG:root:Verifying response to auth request
DEBUG:root:Attempting password challenge
DEBUG:root:Verifying password challenge response
Traceback (most recent call last):
  File "FindMy.py/findmy/reports/account.py", line 666, in fetch_reports
    return await self._reports.fetch_reports(
  File "FindMy.py/findmy/reports/reports.py", line 264, in fetch_reports
    reports.extend(await self._fetch_reports(date_from, date_to, chunk))
  File "FindMy.py/findmy/reports/reports.py", line 288, in _fetch_reports
    data = await self._account.fetch_raw_reports(start_date, end_date, ids)
  File "FindMy.py/findmy/reports/account.py", line 606, in fetch_raw_reports
    new_state = await self._gsa_authenticate()
  File "FindMy.py/findmy/reports/account.py", line 758, in _gsa_authenticate
    raise InvalidCredentialsError(msg)
findmy.errors.InvalidCredentialsError: Password authentication failed: This action could not be completed. Try again.

I don't know the specifics of the anisette data; most likely real systems will generate it per-request. We might want to as well, but holding onto it for a minute is probably alright (in case we burst some requests and would rather not be so tied to the anisette server).

I've force-pushed the requested changes, and adjusted the expiry tracking to be based on a deadline instead of last update, for a bit cleaner comparisons.

Perfect, I'll test it in a few days and merge if it looks good.

Works perfectly. Thanks again!


change: estimate difficulty and move to appropriate position within track

The exercise change was added after we'd already re-ordered the track. By default, new exercises are scored with difficulty 1 and located at the end of the track. We should use this issue to discuss and determine an estimated difficulty for change, then submit an associated PR that moves change to the appropriate position within the track based on that difficulty.

This still hasn't been estimated, so the issue should probably stay open.

@stkent you'd probably know best since you implemented it. But maybe 8 or 9? :)

change is algorithmically complex; the 'obvious' implementation (find the largest coin smaller than your total; subtract multiples of that coin until your remaining total is less than the coin value; repeat with next coin) doesn't always work. The basic premise remains recursion though. I'd say 8-9 works here; I'll add a PR with a suggested position :)


[Vue-warn] when using custom fields

Sampurasun!
Trying to create custom fields like this, it works perfectly. But, when I am looking on 'inspect element --> console' this [Vue-warn] was appear:
[Vue warn]: Property or method "formatValueToModel" is not defined on the instance but referenced during render. Make sure to declare reactive data properties in the data option.

found in

---> <FieldBulmaInput> at /var/www/vuejs/next-level-vuejs/client/helpers/custom-form-generator/fieldCustomInput.vue
       <VueFormGenerator>
         <TabContent>
           ...

It's happen when create 'input text' and 'textarea'. This is an example code of custom fields fieldCustomInput.vue:
<template>

  {{ schema.customLabel }}

    <input
	class="input is-medium"
	type="text"
	v-model="value"
	:disabled="disabled"
	:maxlength="schema.maxlength"
	:placeholder="schema.placeholder"
	:readonly="schema.readonly">

</template>

And I tried to defined 'formatValueToModel' in tag

It is not your fault, I think.
In abstract.js, there is a check to see if formatValueToModel exist as a method. This is a way for custom field developer to format value how they want, if needed.
I think that check trigger [vue-warn], but it is not a problem. In production, this message should not appear.
I'm going to look into it.

I have the same issue. Trying to use datePicker from vue-strap.
Whole field component code:
<template>
  <datepicker v-model="value" :disabled-days-of-Week="disabled" :placeholder="schema.placeholder">
  </datepicker>
</template>

Nothing special and I see the warning too.

Fixed

Released in v2.1.1


Categorical split support for SHAP

Depends on https://github.com/rapidsai/gputreeshap/pull/26 .

Please do not merge this one until I can replace the submodule back to rapidsai.

How Shap values for each one-hot-encoded category are aggregated to generated  the final value for the categorical variables?

@talieh-tabatabaei When enable_categorical=True is enabled, it does not compute Shap values for each one-hot-encoded variable. To enable this for Shap computation, the tree model should be trained with this enabled. In this case, there are no one-hot-encoded variables anymore, it's just one categorical variable itself. So decision variable in the tree model is the categorical variable.
For example, let's say we have Fruit category and the values are Apple, Orange, and Melon. A node of tree could have Fruit==Orange. In one-hot-encoded version, it is equivalent to Is_Orange==1 but in this case, Is_Orange, Is_Apple, and Is_Melon are different variables. In TreeShap, these variables do not invoke Unwind to each other. On the other hand, if we just use the categorical variable Fruit, Fruit==Orange and Fruit==Apple and Fruit==Melon all use the same variable Fruit. This does invoke Unwind if there is more than one Fruit variable appearing on the current path. As a result, it will only compute Shap value for Fruit variable.


Improve the Examples set to include more functionality

Add a bunch of more examples.
The recent issues shows the community is hungry for more examples.
#139
#138
#140
#136

Added more examples with msgpack, deadpool, etc
https://github.com/geofmureithi/apalis/pull/353