IndexError: tuple index out of range

Hi!, That is my ERROR and my para, Can you tell me how to fix it? Iam very confused!
python main.py --config_file configs/EfficientConformerCTCSmall.json --prepare_dataset --create_tokenizer

Hi,
TimeMasking's forward function now requires the batch dimension to process spectrograms.
Including the batch dimension should fix the issue:
x[b:b+1, :, :x_len[b]] = torchaudio.transforms.TimeMasking(time_mask_param=T).forward(x[b:b+1, :, :x_len[b]])
instead of:
x[b, :, :x_len[b]] = torchaudio.transforms.TimeMasking(time_mask_param=T).forward(x[b, :, :x_len[b]])

Hi,
TimeMasking's forward function now requires the batch dimension to process spectrograms.
Including the batch dimension should fix the issue: x[b:b+1, :, :x_len[b]] = torchaudio.transforms.TimeMasking(time_mask_param=T).forward(x[b:b+1, :, :x_len[b]])
instead of: x[b, :, :x_len[b]] = torchaudio.transforms.TimeMasking(time_mask_param=T).forward(x[b, :, :x_len[b]])

Thanks! It works for me!


Improvement to IoTEdge documentation

Added GPG key section prior to installing moby
Mention that port 443,5671,8000 need to be opened if azure VM are used
Section how to use IoTEdge check.
( Artifact no being generated for rc2 was fixed yesterday.)

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

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

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

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

Hi @v-tcassi - Could you review the proposed update to your article? If the update is approved, I'll make some formatting (new heading and table; list indentation) and grammar fixes per Microsoft style before merging.
Thanks.

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

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

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

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

Thank you for making those updates, Hugues. #sign-off

Thank you for making those updates, Hugues. #sign-off

@kgremban: I'm sorry - only the author of this article, @v-tcassi, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@kgremban: I'm sorry - only the author of this article, @v-tcassi, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


added generate-sitemap.js, robots.txt, sitemap.xml, metadata.jsx

Added Missing Meta Tags, robots.txt, sitemaps for SEO Enhancement
fixes issue #145
@jindalpriyanshu101 @jinx-vi-0 kindly review my PR and provide labels.

@jindalpriyanshu101  If possible Could you please add a higher-level tag, as I have devoted a significant amount of time to this


Bump flowkit to v1.16.0

Bump flowkit to v1.16.0

For contributor use:

[x] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

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

Project coverage is 40.62%. Comparing base (d1310c3) to head (e72bcf8).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1438      +/-   ##
==========================================
+ Coverage   40.60%   40.62%   +0.02%
==========================================
  Files          47       47
  Lines        2596     2597       +1
==========================================
+ Hits         1054     1055       +1
  Misses       1417     1417
  Partials      125      125

Flag
Coverage Δ

unittests
40.62% <100.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Promote a multimodule structure for other extensions

This changes the project structure to support additional prettytime extensions under the extensions/ directory

Closes #96

@joes I created this PR which should help on introducing more extensions to this project

Thanks @gastaldi, I’ll have a look and see if I am able to integrate prettytime-nlp in the new layout.


Align code needs to be updated

See https://github.com/Spuds/elk-quick-quote/commit/bc34357a800ba2649f3fdcd01a34dd09eb1c2f68
A couple of minor things .. left and right are unique BBC codes not part of align= as such those props objects  need to be updated.
In seeing that, center is also a div class not a span align.  Yes this is not HTML correct, center is not a block element, but as far as the user is concerned when using the editor, center is a block element and that is how ElkArte deals with it.  Check the years of defending the way center worked vs left/right.
TLDR, we need to check for the centertext  class in the div section and not the span section
Updated the checkCSSProps  function to able to check for a class name  by setting isClass  value of the passed object.  This allowed the consolication of centertext and tt items.
The above should be clear in the code and is 1.1 valid, hence the issue.

Yes, I'd noticed it needed some updates. :-)


Initializing App Check local development not working

Hi,
Seems there is an error initializing App Check when using login server action. Not possible to add the global variable to the window as there is no window on the server
⨯ src/app-check/index.ts (14:19) @ getOrInitializeAppCheck
⨯ ReferenceError: window is not defined
   at getOrInitializeAppCheck (./src/app-check/index.ts:14:19)
   at getFirebaseApp (./src/auth/firebase.ts:16:27)
   at getFirebaseAuth (./src/auth/firebase.ts:23:24)
   at loginAction (./src/actions/login.ts:11:69)
digest: "776069970"
 12 |   // Firebase uses a global variable to check if app check is enabled in a dev environment
 13 |   if (process.env.NODE_ENV !== 'production') {
> 14 |     Object.assign(window, {
    |                   ^
 15 |       FIREBASE_APPCHECK_DEBUG_TOKEN: process.env.NEXT_PUBLIC_APP_CHECK_DEBUG_TOKEN,
 16 |     });
 17 |   }

Hey @nolafs!
window is not available in server actions.
You can try to move signInWithEmailAndPassword and getFirebaseAuth call to the client, and pass idToken to the loginAction directly from client instead. Let me know if that works


Infix notation for CSG operations

This allows us to do things like
barrel ∩ topsurface ∩ botsurface
instead of
csgintersection(barrel, csgintersection(topsurface, botsurface))
I've kept the old functions for backwards compatibility and the ability to apply intermediate transforms. We could also rethink this approach, though.

I've kept the old functions for backwards compatibility and the ability to apply intermediate transforms. We could also rethink this approach, though.

Given that you can create a leaf node solely to apply a transform I think that the intermediate transforms are now unnecessary. Using leaves to do this is slightly less concise in terms of number of function calls required, but should evaluate to the same thing in the end. It possibly also makes it simpler for the user as there is only one way to apply a transform - using a leaf node, though if the are going to be used a lot within the tree maybe we should rename it to transform or something
Broadly I'm a big fan of this change - much less typing and much more readable!
I think we should probably figure out the transforms properly before merging it though (and deprecate the old method completely too? removing exports will likely break any downstream code anyway)

Codecov Report

Merging #188 (7215bb6) into main (13844d1) will decrease coverage by 0.97%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #188      +/-   ##
==========================================
- Coverage   56.65%   55.68%   -0.98%
==========================================
  Files          67       65       -2
  Lines        6381     6345      -36
==========================================
- Hits         3615     3533      -82
- Misses       2766     2812      +46

Impacted Files
Coverage Δ

src/Examples/docs_examples.jl
0.00% <0.00%> (ø)

src/Geometry/CSG/CSG.jl
79.63% <0.00%> (-5.31%)
:arrow_down:

src/Geometry/Primitives/Qtype.jl
75.00% <0.00%> (-8.34%)
:arrow_down:

src/Optical/LensAssembly.jl
48.22% <0.00%> (-7.33%)
:arrow_down:

src/Optical/Emitters/Emitters.jl
75.00% <0.00%> (-5.00%)
:arrow_down:

src/Optical/OpticalInterface.jl
63.23% <0.00%> (-4.30%)
:arrow_down:

src/Geometry/CSG/Intersection.jl
48.57% <0.00%> (-2.14%)
:arrow_down:

src/Geometry/Geometry.jl
0.00% <0.00%> (-1.86%)
:arrow_down:

src/GlassCat/sources.jl
87.27% <0.00%> (-1.82%)
:arrow_down:

src/Geometry/Primitives/Curves/BSpline.jl
66.15% <0.00%> (-0.52%)
:arrow_down:

... and 7 more

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 13844d1...7215bb6. Read the comment docs.

I'd be more than happy to deprecate the old notation - will have a discussion with @BrianGun and @galran to see their thoughts.

Given that you can create a leaf node solely to apply a transform I think that the intermediate transforms are now unnecessary. Using leaves to do this is slightly less concise in terms of number of function calls required, but should evaluate to the same thing in the end. It possibly also makes it simpler for the user as there is only one way to apply a transform - using a leaf node, though if the are going to be used a lot within the tree maybe we should rename it to transform or something

I had exactly the same minor concern about the verbosity of intermediate transforms, which is what I meant by "rethinking the approach". It's probably fine - as you say, it's a little cleaner in terms of user interface - but open to discussion.

removing exports will likely break any downstream code anyway)

The exports are actually moved to the top, not removed. Personally I think this is better code style for a maturing project as it more clearly shows the API that a particular source file is defining.

What are you proposing? Eliminating intermediate transformation nodes or using ∩, ∪, - instead of csgunion, etc. Or both?
No objection to overloading ∩, ∪, -. If you do this then define another function transform(a::CSGTree,b::Transform). This would create an intermediate transformation node. Then you would do this
Sphere() ∪ transform(A,transformation)

instead of what we currently do:
csgunion(Sphere(),A,transformation)

Eliminating intermediate transformation nodes doesn't seem like a good idea. Forcing the user to specify transformations only at the leaves would complicate the construction of geometric hierarchies. Where are those leaf node transformations going to be computed?
The user would have to maintain a separate geometric transformation tree that matches the csg tree and then to extract transformations from the leaves of the geometric transformation tree and insert them into the constructors of the matching leaf nodes in the csg tree. That seems complicated.
In our current system the examples have shallow hierarchies but it's not hard to imagine systems with more complex hierarchies. It seems simpler to include the transformations along with the csg tree

I don't think anyone's thinking of eliminating intermediate transforms.
Originally, I just wanted to add ∩, ∪, - for convenience, on top of the existing functions, but after converting a few examples I think this is a much better notation to use for CSG operations and would support removing the old functionality.
Currently I think leaf does exactly what we'd want transform to do. As Charlie suggested, we can probably just repurpose it for the new interface. I'll look into this tomorrow.

leaf should create a LeafNode, a node with no children, although I see that there is a new leaf function that does something different.
It's confusing to overload leaf to make it mean something else. If we change the second definition of leaf to transform then that will make the naming more consistent with function.

I've had a proper look at the CSG code now - I was a bit confused by the pseudo-leaf yesterday as I didn't take the time to question what it was actually doing. I agree that transform would be a much more appropriate name for it.
However, this may have been indicative of the fact that the underlying system was a little awkward to work with in the first place. I find it somewhat unintuitive that the CSGTree isn't actually the object that we interface with during CSG tree creation. Rather, we create a tree wrapper of sorts using various CSGGenerator function chains, mirroring paths in the tree. There are three ways to alter this transform tree:

A leaf node can be created with an initial transform (returns CSGGenerator).
Each binary node (union, intersection, difference) can be created whilst simultaneously creating a transform to propagate down the tree at evaluation (returns CSGGenerator).
A final transform can be applied to the root object before evaluation (returns CSGTree).

In my opinion, this is a peculiar design. If intermediate transforms are indeed vital to the process, then why not incorporate them into the CSGTree structure? Each node can easily store a transform alongside its children. This makes applying transforms at any point very simple: just change the transform field of the corresponding node. There's also the added bonus of making inverse computations a little bit more efficient as they can be evaluated using the tree structure.
I'll try making these changes to show what I mean. Feel free to disagree/point out any flaws in this proposed design.

There were several things driving the current design. The most important was that we wanted to minimize the cost of transforming rays. We also wanted to be able to make reusable subsystems where the entire subsystem could be transformed after it was created.
To properly transform the ray we only need the final leaf transform, which is the composition of all the transforms on the path from the root to the leaf. But this is a static array for efficiency so it can't be changed after the subsystem is created. That's a problem if the creation of the subsystem and the transformation of the subsystem are in different parts of the code, if the subsystem is in a library of premade cparts for example.
The current system addresses this problem by making a subsystem factory which has the intermediate transforms encoded as arguments to function closures. To create a new subsystem you execute the factory, optionally passing in a root node transform. Then it constructs the subsystem by executing the function closures, storing the resulting arrays only at the leaf nodes.
An alternative would be to traverse the csg tree from the root and make a copy as you go. When you reach a leafnode you'd multiply the transform stored there by the incoming transform and use that as the argument to the LeafNode constructor to make the new leaf node.
This is exactly what the current code does. It's simple and efficient and only takes 15 lines of code, 12 lines of which are simple multidispatch method definitions. If you rewrote it you'd end up with code that does exactly the same thing, with equivalent or greater complexity.
Since the current code is not broken and the replacement code would have exactly the same functionality it's not clear that it's worth rewriting. It's a different style of coding using closures that is more common in Julia. Perhaps it needs to be better documented so it is easier to understand.
We should stop exporting the csgintersection,csgunion,csgdifference functions and instead export the standard ∩, ∪, - infix operations. The end programmer would write a  ∩ transform(b) instead of the current csgintersection(a,b,transform).

In general, I don't have a problem with using function closures to delay the evaluation of the leaf node transforms. The current solution is coded nicely for this design choice.
However, I'm trying to consider it from a data structure standpoint. Especially since we're moving towards more interactive applications, we may want to be able to view and edit these intermediate transforms. In my opinion, having these values stored within functions makes them harder to access compared to if they were just fields in a mutable node.
I think that my suggestion (CSGTree2) simplifies both the underlying implementation and exposed interface by collapsing everything back into the CSGTree, including the transforms (maintaining the ability to delay evaluation and apply intermediate transforms) and different types of nodes (by adding another parameterisation to the struct).
I wrote a demo script to show an example use case. On top of the existing functionality, it is also possible to print the CSG tree, outputting something like this:
OpticSim.CSGIntersection: transform = [-1.0 0.0 1.2246467991473532e-16 0.0; 0.0 1.0 0.0 0.0; -1.2246467991473532e-16 -0.0 -1.0 -5.0; 0.0 0.0 0.0 1.0]
├─ OpticSim.CSGIntersection: transform = I
│  ├─ OpticSim.CSGLeaf: transform = I, surface = Cylinder{Float64}(9.0, FresnelInterface(SCHOTT.N_BK7, GlassCat.Air, 0.0, 0.0, ReflectOrTransmit))
│  └─ OpticSim.CSGLeaf: transform = [1.0 0.0 0.0 0.0; 0.0 1.0 0.0 0.0; 0.0 0.0 1.0 5.0; 0.0 0.0 0.0 1.0], surface = AcceleratedQTypeSurface{Float64, 3, 3, 2}(9.0, -0.04, 0.3, Cylinder{Float64}(9.0, FresnelInterface(GlassCat.Air, GlassCat.Air, 0.0, 0.0, ReflectOrTransmit)), [0.0, 0.0], SVector{2, Float64}[[-1.538089935299395, 2.138089935299395], [0.0, 0.0], [0.0, 0.0]], SVector{2, Float64}[[-0.2, 0.0], [0.565685424949238, 0.0], [-0.6531972647421808, 0.0]], 9.5, 2.4000000000000004)
│
└─ OpticSim.CSGLeaf: transform = I, surface = Plane{Float64}([0.0, 0.0, -5.0], [0.0, 0.0, -1.0], FresnelInterface(SCHOTT.N_BK7, GlassCat.Air, 0.0, 1.0, ReflectOrTransmit))

Especially since we're moving towards more interactive applications, we may want to be able to view and edit these intermediate transforms. In my opinion, having these values stored within functions makes them harder to access compared to if they were just fields in a mutable node.

This is a very good point, but in Julia mutable == bad performance (pretty much) - if we want to go this route it might be best to work on changing CSGGenerator to be mutable, but still have it 'compile' to a CSGTree which is immutable and fully typed to retain good performance at trace time.

This is a very good point, but in Julia mutable == bad performance (pretty much) - if we want to go this route it might be best to work on changing CSGGenerator to be mutable/directly modifiable, but still have it 'compile' to a CSGTree which is immutable and fully typed to retain good performance at trace time.

Writing high performance code in Julia is something I'm still not completely up to scratch on, so thanks for the pointers. Forgot to think about this when I was criticising the current design.
If we could have everything done by one struct, it seems that we would want an immutable tree that we can construct from the leaves up, with (parent and) child pointers, and the ability to edit data during and after construction at each node. Given these constraints, I think you're right - we'll either need to rebuild the entire tree for each change or follow some kind of 'compilation' flow, which would be an adaptation on CSGGenerator. It might actually be simplest to repurpose "CSGTree2" for this and write a compile(::CSGTree2)::CSGTree function.

@alfredclwong  Interactive modification of transformations might be a good reason to make the more extensive changes you propose. For sure make the mods to switch to infix.
Does your code rewrite for csgtransformation that incorporates mutable transformations work? Does it have a compile function that generates statically typed immutable data structures? If you've already done all this work and all test pass then we could switch to the new representation.
But if you haven't done all this then it would be good to examine the other tasks on our list. Getting the UI for optimization working better is high on that list. Do you think this task is more important, or essential before working on the UI?
-brian

@alfredclwong Interactive modification of transformations might be a good reason to make the more extensive changes you propose. For sure make the mods to switch to infix.
Does your code rewrite for csgtransformation that incorporates mutable transformations work? Does it have a compile function that generates statically typed immutable data structures? If you've already done all this work and all test pass then we could switch to the new representation.
But if you haven't done all this then it would be good to examine the other tasks on our list. Getting the UI for optimization working better is high on that list. Do you think this task is more important, or essential before working on the UI?
-brian

It'd probably take another day or two to get a full rewrite done, but I agree it's not the highest on our priority list at the moment. The previous commit was more to open a discussion on possible future ideas.
I'll revert back to the minimal infix change, make the leaf->transform change, and I'll also keep the old functions (csgunion etc.) with a deprecation warning in order to avoid making breaking changes.

@BrianGun this should be ready for review now. We can go through it briefly today at our meeting.


Validation of paper A Unified Deep Learning System For Large-Scale Parallel Training

Hello guys, thanks for your efforts. Currently I am reading the paper Colossal-AI: A Unified Deep Learning System For Large-Scale Parallel Training. I am very interested in the results about performance comparison with DeepSpeed GPT-2. However I saw DeepSpeed changed the GPT-2 script back to GPT. Can I know where is the source code and configurations you are using as baseline? Appreciate your help!

Have found useful link. Thanks!


Document how to use conda-eda

I was under the impression that conda-eda was usable from within conda, however it doesn't work, and there doesn't appear to be any documentation on how to use conda-eda under https://hdl.github.io/conda-eda/Usage.html
(base) user@Ondo:/si$ conda config --env --add channels conda-eda
(base) user@Ondo:/si$ conda upgrade -c conda-eda --all
Collecting package metadata (current_repodata.json): failed

UnavailableInvalidChannel: HTTP 404 NOT FOUND for channel conda-eda <https://conda.anaconda.org/conda-eda>

The channel is not accessible or is invalid.

You will need to adjust your conda configuration to proceed.
Use `conda config --show channels` to view your configuration's current state,
and use `conda config --show-sources` to view config file locations.

(base) user@Ondo:/si$

Please add documentation on how to use this repository as a user, rather than as someone building packages.

I think the channel is litex-hub due to historic reasons: https://github.com/search?q=repo%3Ahdl%2Fconda-eda litex-hub&type=code.

That would explain some of the confusion I had.
I think it make sense to add that to the Usage.html guide, in addition to the README.md, so users can know how to use the packages built as part of this repo.

Refs:

https://github.com/RTimothyEdwards/open_pdks/issues/376
https://github.com/google/skywater-pdk/pull/421

/cc @proppy


[WIP: Not related to KFNBC] Groups admin panel

Groups admin panel
Description
Added a new settings section to manage groups:

Ability to control admin groups in the dashboard.
Ability to control user groups in the dashbaord.
Automatically detects a group selected is removed and notify the user.
Adds the ability to support multi group selection.
Grant by default permission for accessing settings panel to all the cluster admins

Kfdef
kind: KfDef
apiVersion: kfdef.apps.kubeflow.org/v1
metadata:
  name: opendatahub
  namespace: opendatahub
spec:
  applications:
    - kustomizeConfig:
        repoRef:
          name: manifests
          path: odh-common
      name: odh-common
    - kustomizeConfig:
        parameters:
          - name: s3_endpoint_url
            value: s3.odh.com
        repoRef:
          name: manifests
          path: jupyterhub/jupyterhub
      name: jupyterhub
    - kustomizeConfig:
        overlays:
          - additional
        repoRef:
          name: manifests
          path: jupyterhub/notebook-images
      name: notebook-images
    - kustomizeConfig:
        overlays:
          - authentication
        repoRef:
          name: manifests
          path: odh-dashboard
      name: odh-dashboard
  repos:
    - name: kf-manifests
      uri: >-
        https://github.com/opendatahub-io/manifests/tarball/v1.4.0-rc.2-openshift
    - name: manifests
      uri: 'https://github.com/lucferbux/odh-manifests/tarball/groups-panel'

How Has This Been Tested?

Upload this KfDef.
Disable the operator to be able to modify the configmaps without being reconciled.
Check the instructions in the docs about how the admin dashboard is enabled.
Start adding and deleting groups.
When having a group selected, if it's deleted in the cluster, it will trigger a new notification.

Merge criteria:

[X] The commits are squashed in a cohesive manner and have meaningful messages.
[X] Github Issue: #192
[X] KfDef added.
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has manually tested the changes and verified that the changes work.

as per offline discussion, we should change the following behavior: Adds the ability to access settings panel to cluster admin and namespaced admins if the last group selected is removed by mistake. to Grant by default permission to access settings panel to all the cluster admins

as per offline discussion, we should change the following behavior: Adds the ability to access settings panel to cluster admin and namespaced admins if the last group selected is removed by mistake. to Grant by default permission for accessing settings panel to all the cluster admins

Changed!

We should change the design of this to use a drop down and chip groups similar to what is shown here. https://www.patternfly.org/v4/components/toolbar#with-custom-chip-group-content This way we won't have such a large drop down with items showing up.  @kywalker-rh do you agree?  This can be done in a future release.

We should change the design of this to use a drop down and chip groups similar to what is shown here. https://www.patternfly.org/v4/components/toolbar#with-custom-chip-group-content This way we won't have such a large drop down with items showing up. @kywalker-rh do you agree? This can be done in a future release. We could use a simple inline chip group below the drop down. https://www.patternfly.org/v4/components/chip-group

@dlabaj Agreed, the long dropdown looks awkward. FYI @vconzola

@kywalker-rh @dlabaj I don't agree with using chip groups for something like this. By "large dropdown", do you mean that the width spans the entire page?  If that's what's bothering you, can we just use the isWidthLimited property on the form?

/lgtm

@kywalker-rh @dlabaj I don't agree with using chip groups for something like this. By "large dropdown", do you mean that the width spans the entire page? If that's what's bothering you, can we just use the isWidthLimited property on the form?

@vconzola @dlabaj @kywalker-rh I've added this to the UX meeting later today -- let us talk through it there.


Add cluster full lifecycle support in upgrade docs

Issue #, if available:
Description of changes:
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.18

/approve

/unhold
/approve


eager load instead of include

Signed-off-by: Dan Kohn<EMAIL_ADDRESS>This converts a two SQL queries to one. See http://blog.arkency.com/2013/12/rails4-preloading/

Test failure!

@david-a-wheeler Please review.

Looks good.

I think sorting by id (as is done here) is sensible.  It's an easy sort, and intuitive.

id sort should also never come up in real life, until two projects are updated in the same second.


Update repositories-pull-requests-discussions.md

Update the PR git snippets:

Fuse step 1 & step 2
Use {PR_NUMBER} & give examples in Step 2 & Step 3

Before:

After:

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@julien-c good to merge?

sounds reasonable
pinging @SBrandeis for a final review


N Queen Problem in C++

Aim

N queen problem using Backtracking Concept.
Using C++.

Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

I would like to contribute in java language

@ArshWalker cpp is already done


Add markets.forking and markets.needsMigration fields

Took a look at the latest changes. Looks good!


Allow setting raw.theme to none to disable syntax highlighting

This pr closes #4175 .
As https://github.com/typst/typst/pull/4186#pullrequestreview-2071726156 , we allow setting raw.theme to none to disable syntax highlighting. To implement it, I redesigned the function signature (return value type):

Err(...) indicates argument theme found but error met when casting it into Smart<Option<EcoString>>;
Ok((None, None)) indicates argument theme not found;
Ok((Some(Smart::Auto), None)) indicates argument theme found, and it is auto;
Ok((Some(Smart::Custom(None)), None)) indicates argument theme found, and it is none;
Ok((Some(Smart::Custom(Some(path))), Some(data))) indicates argument theme found, and it is str (path to the theme file).

Some details:

As the return type is so complex that clippy gives an error "very complex type used. Consider factoring parts into type definitions", I defined ThemeArgType to represent Smart<Option<EcoString>>.
When raw.theme is none, the highlighting process will early exit. To reuse some code, I defined a nested function non_highlighted_result to generate non-highlighted raw blocks.

Thanks!


Update support-rulesengine to newest version (master)

Run tests against the newest version of support-rulesengine.
Signed-off-by: Lisa Rashidi-Ranjbar<EMAIL_ADDRESS>

@lranjbar needs rebase

@tsconn23 Rebased.


No bongo.exe

So I opened the files in visual studio c++ and found no bongo.exe. If you didnt guess already I'm new to coding and wanted to make it so bongo cat was able to tap the keyboard when I pressed more than 2 buttons. I would love it if someone taught me or made one for me. Thanks!

It's in the releases :D

Ohhh thank you!


Use cargo build can't build up tikv

rustc 1.28.0-nightly
CentOS release 6.5 (Final)
When I'm run cargo build ,that will append 'cargo publish timeouts' and 'unable to get packages from source'.
Why???

Can you provide the full output?

[root@dev-vm-85 tikv-release-2.0]# cargo build
warning: An explicit [[bin]] section is specified in Cargo.toml which currently
disables Cargo from automatically inferring other binary targets.
This inference behavior will change in the Rust 2018 edition and the following
files will be included as a binary target:

* /opt/install/build/go/src/github.com/pingcap/tikv-release-2.0/src/bin/setup.rs
* /opt/install/build/go/src/github.com/pingcap/tikv-release-2.0/src/bin/profiling.rs
* /opt/install/build/go/src/github.com/pingcap/tikv-release-2.0/src/bin/signal_handler.rs

This is likely to break cargo build or cargo test as these files may not be
ready to be compiled as a binary target today. You can future-proof yourself
and disable this warning by adding `autobins = false` to your [package]
section. You may also move the files to a location where Cargo would not
automatically infer them to be a target, such as in subfolders.

For more information on this warning you can consult
https://github.com/rust-lang/cargo/issues/5330
 Downloading rustc-serialize v0.3.24
warning: spurious network error (2 tries remaining): [28] Timeout was reached (Operation timed out after 30886 milliseconds with 0 out of 0 bytes received)
warning: spurious network error (1 tries remaining): [28] Timeout was reached (Operation timed out after 30001 milliseconds with 0 out of 0 bytes received)
error: unable to get packages from source

Caused by:
  [28] Timeout was reached (Operation timed out after 30000 milliseconds with 0 out of 0 bytes received)

There is~   @BusyJay
Seem like my network problem?

@HanibalGK Seems that it is caused by your network connection issue. Please try again using proxies.

Must use proxies?
My service is CentOS 6.5 ,python 2.7.3,gcc 3.8.5 have any way to deploy tidb to my cluster ? some like ansible or another way

@HanibalGK If you don't want to modify source codes, of course you can use other better ways, which does not need proxies. Please refer to our doc: https://github.com/pingcap/docs-cn/blob/master/op-guide/ansible-deployment.md

Thank you~ @breeswish  it worked


Fix sublime syntax definitions

Corrects issues with the regex expressions used for classifying whitespace and operators.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@KimBurgess, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot


Allow command handlers to match regexes

Summary
The App#command method currently only subscribes to a specific slash command identified by exact name. In a few cases it is useful to have more flexibility - for example, a gateway that transforms commands into messages for another system might need to be agnostic about which specific commands the user wants to use it with. This PR adds a RegExp option to the App#command signature, which enables this and related behaviours.
Fixes #841.
Notes to reviewers

I was not sure how the tests fitted together. I would welcome feedback or guidance on what I've done.
There is an issue of what happens if the user supplies multiple overlapping regexes, e.g. app.command(/h*/) and app.command(/he*/).  I followed the behaviour of shortcut here, where if it runs the listener for all matching regexes.  This may be surprising/unwelcome but it is at least consistent.  I documented the shortcut behaviour as part of this; but it may be that you prefer to leave this undefined or say it's unsupported so people don't take a dependency on the current behaviour. Note this is distinct from #761 where the regexes could be passed in an array in a single message call (and therefore were looped over within a single piece of middleware). Again I'd welcome feedback or guidance on this!

Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

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

@itowlson You're so quick! Thanks a lot for taking the time to make this pull request 🙇

I was not sure how the tests fitted together. I would welcome feedback or guidance on what I've done.

The added tests cover all the possible patterns. Perfect 👍

I followed the behaviour of shortcut here, where if it runs the listener for all matching regexes. This may be surprising/unwelcome but it is at least consistent.

This is intended behavior in Bolt for JS. Thanks for clarifying this.

I documented the shortcut behaviour as part of this; but it may be that you prefer to leave this undefined or say it's unsupported so people don't take a dependency on the current behaviour. Note this is distinct from #761 where the regexes could be passed in an array in a single message call (and therefore were looped over within a single piece of middleware). Again I'd welcome feedback or guidance on this!

The changes look good to me. In the future, we may want to improve the duplication but we can do it separately.
If you don't mind, could you send another pull request for the document changes? Merging document changes updates the live website immediately. We would like to merge the document updates right after releasing a new version including your change. Once you move the doc changes to another PR, we will merge the code changes!

Sorry about that @seratch, I noticed some remarks about documentation in the contributor's guide but I overlooked them when it came to doing the PR.  I'll back those out and create a new one for them.

Docs backed out and moved to #847.


_1fps: init at 0.1.10

Description of changes
Add 1fps client.
Link: https://1fps.video
Code: https://github.com/1fpsvideo/1fps

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[X] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The package name should be _1fps due to a limit of nix language:

But all work fine with this name nix build .#1fps. One problem. In nix we need to use quotes: "1fps"


Client luau_lsp quit with exit code 1 and signal 0

Not aware if I set this up right but everytime I open up a luau file in my game project I keep getting this:

Ive called :LuauRegenerateSourcemap but that doesnt fix it either.
lsp.log:

And ive just copied and pasted the setup from the readme

new to neovim and im user Lunarvim and this is how im loading in the plugin:

If I remove the config function it works fine but I get no Sourcemap generation and the plugin stops working.
Neovim 0.10

Your config seems right in this case, just remove the opts field in the plugin spec. The error The command line is too long is unrelated to this plugin, but seems more like a Windows or mason issue, I think it is more appropriate to ask for help in the mason repo or google a solution. Just curious, do other lsp's work or is the issue only with luau-lsp?

Your config seems right in this case, just remove the opts field in the plugin spec. The error The command line is too long is unrelated to this plugin, but seems more like a Windows or mason issue, I think it is more appropriate to ask for help in the mason repo or google a solution. Just curious, do other lsp's work or is the issue only with luau-lsp?

Its just luau.
Whats confusing is if I just load the plugin by itself I get lsp but, the commands disappear. If I try to setup the lsp then I get no lsp but I get the luau commands.

Yeah, I don't think it is related to this plugin, try checking this Windows guide, maybe your PATH variable is too long

So It appears this is over the character limit:

And windows says the limit is 8192.
Not sure what to do exactly to reduce the characters in the command.

Try to turn off the fflag sync option

Try to turn off the fflag sync option

This worked appreciate it.
Should probably put this in the readme if people are experiencing the same issue.


add more detail information about the room in the right chat window Room name bar

Hi,
Vue-advanced-chat has left side chat room list and right side chat main panel. Current the left side chat room name and right side chat main panel's chat room name show exact the same information. But as right side chat main panel has more space, i want to add more detailed information of the chat room . E.g. left side chat room is Leia (person name), but in right side chat room name, can I add extra information such as the short introduction of Leia. (Leia's profile ). as sometimes, many people has same names, a profile introduction might be useful to differentiate who is who. thanks.

Hi,
it seems room-header slot can do this. but i don't know how to do it. can you give some code snippet example for it. the slot part document is missing for details. thx.

https://github.com/antoine92190/vue-advanced-chat#named-slots
On the first line you have the example:
<template v-slot:rooms-header>
	This is the new rooms header
</template>

hi, thx
I roughly figure it out,

Hi,
`

               {{room.roomName}}

`
with above code, i can add custom room name information, but the original room-options, menu-icon, toggle-icon are missing, I don't want to change those room-options, men-icons, how can i get them back? thanks.!

If you decide to use room-header, you cannot retrieve room default options.
I haven't implemented the slots feature myself, so I am not very familiar with it.
I understand the limitations of the current implementation, I will work on it very soon.

Fixed in https://github.com/antoine92190/vue-advanced-chat/commit/2cfe6bbf731182a963bc9bb2acd041dda69a1447
You can now use room-header-info slot.

thx, really wonderful fix.


[FLINK-7552] [FLINK-7553] Enhance SinkInterface / Use in FlinkKafkaProducer

What is the purpose of the change
Enhance SinkFunction with a way of retrieving the element timestamp. This allows us to get rid of the hybrid nature of FlinkKafkaProducer010.
This is keeping the legacy static "convenience" methods à la FlinkKafkaProducer010.writeToKafkaWithTimestamps for backwards compatibility.
Brief change log

Enhance Sink interface
Use new interface in Kafka Producer

Verifying this change
This change is already covered by existing tests, such as (please describe tests).
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): yes, call stack of KafkaProducer with writing timestamps is changed slightly, also, StreamSink operator now has a context object.
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

R: @rmetzger @pnowojski Could you please review this?

To be honest, I didn't look at the tests because I didn't think they would fail. Sorry for that. 😓 I'll fix the tests and let you know once this is ready again.

@rmetzger the binary compatibility checker is acting up because it doesn't like the changes to SinkFunction and RichSinkFunction. It might be that we can't actually change this. What do you think?

The problem is that RichSinkFunction defines
public abstract void invoke(IN value) throws Exception;

even though it implements SinkFunction, which defines:
void invoke(IN value) throws Exception;

What this means is that code that is invoking these methods is different, depending on the type of reference. For example, in
SinkFunction<String> interfaceSink = new SinkFunction<String>() {
    @Override
    public void invoke(String value) throws Exception {

    }
};

SinkFunction<String> interfaceSinkToRichSink = new RichSinkFunction<String>() {
    @Override
    public void invoke(String value) throws Exception {

    }
};

RichSinkFunction<String> richSink = new RichSinkFunction<String>() {
    @Override
    public void invoke(String value) throws Exception {

    }
};

interfaceSink.invoke("hello");
interfaceSinkToRichSink.invoke("hello");
richSink.invoke("hello");

the first two calls are invokeinterface while the last call is using invokevirtual.
I don't think that people are using the sink interfaces for directly calling methods on them but this is the reason the binary compatibility checker is complaining.

I see. It is a similar problem we've discussed with the SourceContext. Users only implement it, but don't call it.
I believe in this case, it makes sense to add an exception to Japicmp to allow this change.

Can we add fine-grained exclusions, or would this exclude the whole file?

No, you can define fine-grained exclusions. Check the excludes in the example: http://siom79.github.io/japicmp/MavenPlugin.html

It seems we completely have to ignore SinkFunction, because japicmp will check its serialiVersionUID, which changed by adding a new method to the interface. (And yes, I know that checking the serialVersionUID on an interface does not make a lot of sense.)

@pnowojski and @EronWright I changed the name of the Context, I added a test in StreamSinkOperatorTest, I added methods or querying current processing time/watermark to the context. I changed the timestamp() method to return a primitive long and I added a method hasTimestamp(). Also, timestamp() now throws an exception if no timestamp is available.
What do you think?

+1 from my side to merge it like this (using boxed long, dropping the hasTimestamp() method.
My reasoning is that this is consistent with ProcessFunction (like @EronWright said), it is also not in a hot loop and lazily created, so I do not expect a big performance hit.
@aljoscha and me talked a lot about the whole design of when records have timestamps and whether in the future we should just assume that records always have timestamps. That biased my towards dropping the hasTimestamp() method.

+1

Thanks for the reviews, everyone! 😃


set the name of the process to the log of the deployed endpoint

set the name of the process to the log of the deployed endpoint

could you please update the changelog

done


History In Entity History Tab

I am asking about this function, is it supposed to be? or glitch...

If I open the entity property dialog the history bar/graph shows up but from this addon.
My problem is sometimes shows up original history bar/graph from HA and after a few days shows bar/graph from history explorer and it's randomly repeating.
If I installed this addont first time, my sensors had original HA history graph/bar. The change I noticed after few days. I would like to keep history graph/bar from this modification because is interactive inside entity dialog and just better. But how can I avoid to randomly changing without any reasons.

Difficult to say with the info you provide.
That feature needs to be explicitly enabled  as described here. It's a per browser feature, its state is stored in your browser memory. If you change browsers or if you delete the site data in a browser, the feature will be disabled. If you use multiple browsers, you need to explicitly enable it on each browser.
If it still randomly goes on and off on the same browser and without you clearing site data, then please post the YAML configuration of your history explorer card. If you have multiple history explorer cards on your dashboards, then please mention this and post the YAML used by each of them.

Was hard to explain for me. I am sorry, my bad I did not notice you described it.  The true is I am using multiple devices where I one of them cleared cash and used HA later.  Now it makes sence. Thank you for your quick explanation :D


Add function to convert to MYA time

MYA timestamps are not natively supported by MariaDB, and currently the way to translate between a human readable timestamp and a MYA timestamp is use the MYA API such as via jmyapi:toMyaTimestamp.   When creating test data it may be useful to have a MariaDB function/procedure like STR_TO_MYA_DATE.   Just figuring out what MYA timestamp corresponds to a given human readable timestamp requires writing a small program.

This may be getting closer (though breaks with dates after 2038, and assumes zero fractional seconds):
DELIMITER //

CREATE DEFINER=`myapi`@`localhost` FUNCTION `STR_TO_MYA`(str char(64)) RETURNS bigint(20)
    COMMENT 'Return a MYA Timestamp given an ISO8901 string'
begin
    declare hi bigint(20) unsigned default 0;
    set hi=UNIX_TIMESTAMP(convert_tz(str, 'GMT', 'America/New_York'));

   return (hi << 28);
end;
//
DELIMITER ;

Initial support for str_to_mya() and mya_to_str() added.   Currently doesn't support fractional seconds though.  See source.


Getting an error while trying to run kivy-designer - Please help me to solve this

ERROR  ] [WinPygame   ] unable to set icon
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/kivy/core/window/window_pygame.py", line 215, in set_icon
self._set_icon_standard(filename)
File "/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/kivy/core/window/window_pygame.py", line 227, in _set_icon_standard
im = pygame.image.load(filename)
pygame.error: File is not a Windows BMP file
Getting an error while trying to run kivy-designer - mac el capitan

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29905689-getting-an-error-while-trying-to-run-kivy-designer-please-help-me-to-solve-this?utm_campaign=plugin&utm_content=tracker%2F77147&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F77147&utm_medium=issues&utm_source=github).

I see that your env is trying to use pygame ERROR ] [WinPygame ] unable to set icon. So, check this bug and this comment: https://github.com/kivy/kivy/issues/3908#issuecomment-170799453
Is your installation OK? https://kivy.org/docs/installation/installation-osx.html

I would suggest uninstalling pygame and getting SDL2 going with your version of KIVY (not the designer) since pygame is deprecated in KIVY. I also tried installing kivy-designer on Mac (Yosemite in my case) and could not quite get it going, but I was also trying to use pygame in the KIVY module. I see a common theme of things not working with pygame.
https://kivy.org/docs/installation/installation.html?highlight=development
Link above is how to install the development version of kivy on OSX. Might be a good restarting point for you. You should include any issues you are having with home brew if you are using that (i.e., any packages that can't be found). But maybe also try an install without using home brew using the steps in the link above and see how things go with that route.


go/types: package documentation

go/types could use some high-level documentation of the basic concepts (the Type
hierarchy, the Object hierarchy), and how to accomplish various kinds of mappings from
syntax to types and vice versa.  I think tool builders would also appreciate a
discussion of the commonly misunderstood things, with explained examples, such as:
- a list of invariants related to each Type/Object. The docstrings cover some of them
but it doesn't help a new user become oriented.
- the fact the Objects are canonical.
- what's the deal with Tuple.
- what situations Info.Implicits are for
- that fields (Vars) of structs don't know their owning struct and are in fact may be
equally "owned" by several.
- the analogous situation for interface methods.
- why some Idents/FuncTypes/etc appear in Info.Types but not others
- how PkgName objects work, and the possibility that two distinct PkgName objects can
represent the same package.
- that a struct may contain two fields (or an interface two methods) both named
"f"
- that a struct field identifier may be simultaneously use a type and define a field
- how to determine whether an object is universal, file-scope, package scope or local,
and which object kinds can inhabit which scopes.
- etc.

Fixed by https://github.com/golang/example/tree/master/gotypes


Fix #149: No link to ref when uploaded file

This pertains to:

transfer.sh
0x0.st

File uploaded and can be downloaded successfully. URL looks weird and needed trimming of the part after the last /.
~/git/aim fix/149/no-output* » ./target/debug/aim README.md https://transfer.sh/README.md

Response { url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("transfer.sh")), port: None, path: "/README.md", query: None, fragment: None }, status: 200, headers:
 {"server": "nginx/1.18.0", "date": "Sun, 25 Sep 2022 18:46:14 GMT", "content-type": "text/plain", "content-length": "36", "connection": "keep-alive", "retry-after": "Sun, 25 Sep 2022 20:46:18 GMT", "x-made-with":
 "<3 by DutchCoders", "x-ratelimit-key": "<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>", "x-ratelimit-limit": "10", "x-ratelimit-rate": "600", "x-ratelimit-remaining": "9", "x-ratelimit-reset": "1664131578", "x-served-by
": "Proudly served by DutchCoders", "x-url-delete": "https://transfer.sh/vYBVWR/README.md/HelwY8KbK1bk", "strict-transport-security": "max-age=63072000"} }

Seems 0x0.st uses only GET, HEAD, OPTIONS, POST.
transfer.sh does support PUT.
Currently, aim uses only PUT so cannot upload.

Codecov Report

Merging #150 (2c68ed5) into main (76b2b2a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #150   +/-   ##
=======================================
  Coverage   97.37%   97.38%
=======================================
  Files          17       17
  Lines        1297     1298    +1
=======================================
+ Hits         1263     1264    +1
  Misses         34       34

Impacted Files
Coverage Δ

src/https.rs
99.03% <100.00%> (+<0.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Achievement Type Displayed Twice

Achievement Type is displayed under the title of the credential and then again below the dates.
Please remove Achievement Type below the dates and keep the one displaying under the credential title as is.

PR: https://github.com/digitalcredentials/learner-credential-wallet/pull/514

Looks good


[5.3] Merge a model into simple query

A simple way to merge a model instance into a simple query
$user = App\User::find(29);
$users = \DB::table('users')->where('id', '>', 50)->union($user)->get()

Probably best to send this to 5.4.

Yes, please send to 5.4, to avoid merge conflicts. :)


Compile with ASLR flags

Adds -fPIE flag to enable address space layout randomization on Linux gcc compilers
Pull requests for core Box2D code are generally not accepted. Please consider filing an issue instead.
However, pull requests for build system improvements are often accepted.

In service of !72


Color revision

Updates title for sponsors
Currently:

New:

Update password gauge to use theme colors

@yeraydiazdiaz if you could review this, I would appreciate it! Thx :)


Is't possible to save into a Umbraco database table?

Is't possible to save into a Umbraco database table?

Sure it is. You can swap the storage index with your own custom implementation or with one of the sample implementations (one of which uses SQL instead of Lucene).
Check the documentation.


fail to install dotnet-ef for mac

when exec "dotnet tool install --global dotnet-ef" on mac, it will print information below:
error NU1202: Package dotnet-ef 3.0.1 is not compatible with netcoreapp3.0 (.NETCoreApp,Version=v3.0) / any. Package dotnet-ef 3.0.1 supports: netcoreapp2.1 (.NETCoreApp,Version=v2.1)
The tool package could not be restored.
Tool 'dotnet-ef' failed to install. This failure may have been caused by:

You are attempting to install a preview release and did not use the --version option to specify the version.
A package by this name was found, but it was not a .NET Core tool.
The required NuGet feed cannot be accessed, perhaps because of an Internet connection problem.
You mistyped the name of the tool.

For more reasons, including package naming enforcement, visit https://aka.ms/failure-installing-tool

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

ID: 6719f08e-3bd7-dc1a-71df-f2ef9fbca9d8
Version Independent ID: 7096fdb3-612e-9e00-bd0b-8ea4886a09ce
Content: Add a model to a Razor Pages app in ASP.NET Core
Content Source: aspnetcore/tutorials/razor-pages/model.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello @sunli93 ... It looks like they've fix it for the upcoming release of 3.1. See :point_right: https://github.com/aspnet/EntityFrameworkCore/issues/18977


Too large size of bookmark-file

Hi Darek
thanks for Static Marks.
I've noticed that my output file is quite large for a few html links, approx. 200 kb.
This could be a problem on small data plans and mobile use, although some of the content might be cached on the mobile.
I have 60+ links in several boxes, however, I wouldn't need the search bar. Also, I don't need the collapse/expand feature for links.
The output file should not be as big as 200 kb, and I think a static html file could be much smaller in file size.
Is it possible to create a more simple static bookmark file, e.g. with command line arguments?
Greetings,
Heiko

Hello Heiko,
the default output file is so big because it is a small self-contained app, including React and a few other libraries. With that said, you are free to use a custom template file as described under features. I have provided an example template that is only 4kb big, with ~60 bookmarks it's 8kb big. Use the --template-file [file] flag as documented to pass your template. There's no specific documentation around custom templates, but I'm assuming the example files should be sufficient to come up with your own template if necessary. If you have and issue/question, let me know.

I've created a new enhancement ticket to provide built-in templates.

Perhaps another point would also be to try to reduce the default template weight?
Like, perhaps tree-shaking the included libs, or maybe checking if we can remove some of them?
(side note: serving it gzipped on my side it’s 68k transferred for 206k size and that’s for around 70 bookmarks)


issue 1275 python get numpy shape

issue: https://github.com/Onelinerhub/onelinerhub/issues/1275

Thanks man!
Deployed:
https://onelinerhub.com/python-numpy/get-numpy-shape


Installation plugins is failed when OS is alpine

Hi,
I want to install the following clis and plugins into alpine linux to control IBM Container service automatically:

bx cli
container-registry plugin
container-service plugin, and
kubectl cli

Installation bx cli , kubectl cli, and container-registry plugin was successful, but container-service plugin was failed.
Error I faced is as follows:
# This output is for container-registry
Looking up 'container-registry' from repository 'Bluemix'...
Plug-in 'container-registry 0.1.304' found in repository 'Bluemix'
Attempting to download the binary file...
 29.30 MiB / 29.30 MiB  100.00% 19s2ss
30725403 bytes downloaded
Installing binary...
OK
Plug-in 'container-registry 0.1.304' was successfully installed into /root/.bluemix/plugins/container-registry. Use 'bx plugin show container-registry' to show its details.

# This output is for container-service
Looking up 'container-service' from repository 'Bluemix'...
Plug-in 'container-service 0.1.481' found in repository 'Bluemix'
Attempting to download the binary file...
 20.21 MiB / 20.21 MiB  100.00% 14ssss
21188481 bytes downloaded
Installing binary...
FAILED
Unable to obtain plug-in's metadata. Error: fork/exec /tmp/BluemixFileDownload104351476/container-service-linux-amd64-0.1.481: no such file or directory

Is Alpine Linux not supported?

@jsloyer can you have someone from your team looking at this issue of the container service plugin?

This work-around came from another team member. Not a permanent solution, since it requires manually maintaining the version, but it does work.
# version of container-service plugin to install
ENV CS_VERSION 0.1.488
# we need to grab the 386 arch as the amd64 version doesn't install correctly on alpine
# NOTE(cjschaef): also, the version number may not be appended to the binary name in future, so watch out
ADD http://public.dhe.ibm.com/cloud/bluemix/cli/bluemix-plugins/container-service/${CS_VERSION}/container-service-linux-386-${CS_VERSION} /tmp
RUN bx plugin install /tmp/container-service-linux-386-${CS_VERSION} && bx plugin list && \
    rm -f /tmp/container-service-linux-386-${CS_VERSION}

Currently Alpine only works with the x86 architecture. @boyang9527 would it be possible to add a flag to the plugin that allows the user to specify x86 architecture when installing/updating the plugin?
Note, you can use other versions, like Ubuntu, for your Dockerfile w/o specifying x86 arch.

it only happens on container service. The bluemix cli and container-registry plugin  64bit version works fine... so I suspect there is some issue when container service team is compiling the plugin....

@Ra1nmaker are you still having this issue?

This is fixed with plugin version 0.1.504.
Thanks. @boyang9527 can you close this issue? It doesn't appear that I have permissions to do so.


Error loading some .gif images

Description
Image.network("http://koalareading-online.oss-cn-beijing.aliyuncs.com/images/n1.gif")
The first load is normal，but only one of frames is displayed when you jump from the current page to another page and return.
When returning from other pages, I think it should show the last frame or start animation from the first frame.
Attention：The number of repetitionCount of the GIF image is one。
Repetition
GIF
Video
Steps to Reproduce
void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: GifTest(),
    );
  }
}

class GifTest extends StatefulWidget {
  @override
  _GifTestState createState() => _GifTestState();
}

class _GifTestState extends State<GifTest> {
  @override
  Widget build(BuildContext context) {
    return Container(
      child: GestureDetector(
        child: Center(
          child: Image.network(
            "http://koalareading-online.oss-cn-beijing.aliyuncs.com/images/n1.gif",
          ),
        ),
        onTap: () {
          Navigator.of(context).push(
            MaterialPageRoute(builder: (context) {
              return Container(
                color: Colors.red,
              );
            }),
          );
        },
      ),
    );
  }
}

flutter doctor -v
[√] Flutter (Channel stable, v1.7.8+hotfix.4, on Microsoft Windows [Version 10.0.17134.829], locale zh-CN)
    • Flutter version 1.7.8+hotfix.4 at C:\software\flutter\flutter
    • Framework revision 20e59316b8 (11 days ago), 2019-07-18 20:04:33 -0700
    • Engine revision fee001c93f
    • Dart version 2.4.0

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\software\androidsdk\android-sdk_r24.4.1-windows\android-sdk-windows
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: C:\software\androidstudio\android-studio-ide-173.4697961-windows_3.1\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)

[√] Android Studio (version 3.1)
    • Android Studio at C:\software\androidstudio\android-studio-ide-173.4697961-windows_3.1\android-studio
    • Flutter plugin version 29.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)

[√] Connected device (1 available)
    • MI 8 • b4a02273 • android-arm64 • Android 9 (API 28)

! Doctor found issues in 1 category.

You have to trigger the initState() and load the gif inside it. so whenever the widget initialized it will load the gif as its first time.

class GifTest extends StatefulWidget {
  @override
  _GifTestState createState() => _GifTestState();
}

class _GifTestState extends State
 {
 var gif;

 @override
 void initState()
 {
    gif = Image.network("http://koalareading-online.oss-cn-beijing.aliyuncs.com/images/n1.gif");
    super.initState();
 }

  @override
  Widget build(BuildContext context) {
    return Container(
      child: GestureDetector(
        child: Center(
          child: gif
        ),
        onTap: () {
          Navigator.of(context).push(
            MaterialPageRoute(builder: (context) {
              return Container(
                color: Colors.red,
              );
            }),
          );
        },
      ),
    );
  }
}

@BaderEddineOuaich
Thanks for your reply，but your method doesn't work.
Step

Run my code or your code
You will see that GIF will play normally at the first time.
Click GIF，go to the page with a red background.
Return from the red page.
Check if GIF is normal.

Hi there, i tried the code. you're right. it wasn't working. i come up with this solution. looks just fine for me. clearing the image cache before pushing another widget will make the gif load again. here is the complete code.

class GifTest extends StatefulWidget {
  @override
  _GifTestState createState() => _GifTestState();
}

class _GifTestState extends State
{

  @override
  Widget build(BuildContext context)
  {
    return Container(
      child: GestureDetector(
        child: Center(
          child:Image.network("http://koalareading-online.oss-cn-beijing.aliyuncs.com/images/n1.gif",
            loadingBuilder: (BuildContext ctx, Widget gif, ImageChunkEvent loadingProgress)
            {
              if(loadingProgress?.cumulativeBytesLoaded == loadingProgress?.expectedTotalBytes)
              {
                return gif;
              }
              return CircularProgressIndicator();
            }),
        ),
        onTap: () {
          Navigator.of(context).push(
            MaterialPageRoute(builder: (context)
             {

             //clear cache
              imageCache.clear();

              return Container(
                color: Colors.red,
              );
            }),
          );
        },
      ),
    );
  }
}

I hope it helps, have a nice day!

Hi there, i tried the code. you're right. it wasn't working. i come up with this solution. works just fine for me. clearing the image cache before pushing another widget will make the gif load again. here is the complete code.

class GifTest extends StatefulWidget {
  @override
  _GifTestState createState() => _GifTestState();
}

class _GifTestState extends State
{

  @override
  Widget build(BuildContext context)
  {
    return Container(
      child: GestureDetector(
        child: Center(
          child:Image.network("http://koalareading-online.oss-cn-beijing.aliyuncs.com/images/n1.gif",
            loadingBuilder: (BuildContext ctx, Widget gif, ImageChunkEvent loadingProgress)
            {
              if(loadingProgress?.cumulativeBytesLoaded == loadingProgress?.expectedTotalBytes)
              {
                return gif;
              }
              return CircularProgressIndicator();
            }),
        ),
        onTap: () {
          Navigator.of(context).push(
            MaterialPageRoute(builder: (context)
             {

             //clear cache
              imageCache.clear();

              return Container(
                color: Colors.red,
              );
            }),
          );
        },
      ),
    );
  }
}

I hope it helps, have a nice day!

Have you tried to await the push of the page and after that call setState?

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of ' flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


Roll Flutter Engine from 65d1291c3add to 45e11649ab83 (3 revisions)

https://github.com/flutter/engine/compare/65d1291c3add...45e11649ab83
2024-01-26<EMAIL_ADDRESS>Roll Skia from cbdf09d69efc to 32f6bff0f193 (3 revisions) (flutter/engine#50092)
2024-01-26<EMAIL_ADDRESS>Roll Dart SDK from 5636e338e0b9 to 7337995bc851 (1 revision) (flutter/engine#50087)
2024-01-26<EMAIL_ADDRESS>Fix Shell::Screenshot for Impeller (flutter/engine#50072)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


SpaceAfterComma style is now configurable

#561
Hello! How does this look?

Thanks for the pull request, @lpil!
Merged in 364413d and 91fce0e.

Thanks!


Add topographic colormap

Based on https://github.com/concord-consortium/plate-tectonics-3d/pull/1, so only last commit matters. I'll rebase it when previous PR is merged.
Demo:
http://models-resources.concord.org/plate-tectonics-3d/branch/topo/index.html?preset=subduction

Closing, as it's included in https://github.com/concord-consortium/plate-tectonics-3d/pull/4.


Set service type NodePort for logstash 7.7.1

Hi,
Can you give an example on how to use service type NodePort for logstash 7.7.1 ?
# helm version
version.BuildInfo{Version:"v3.2.2", GitCommit:"a6ea66349ae3015618da4f547677a14b9ecc09b3", GitTreeState:"clean", GoVersion:"go1.13.12"}

# helm repo list
NAME    URL
elastic https://helm.elastic.co

Thank you.

Got the pod to start with the following (not sure this is legal):
service:
  type: NodePort
  ports:
    - name: beats
      port: 5044
      protocol: TCP
      targetPort: 5044
      nodePort : 3xxxx
    - name: http
      port: 8080
      protocol: TCP
      targetPort: 8080
      nodePort: 3yyyy


[backport] Use independent version of hipFFT for ROCm 4.1 and later

Backport of #5318 by @takagi

[automatic post] Jenkins, test this please.

pfnCI, test this please.

Jenkins CI test (for commit ccaa52937c493e83cf36929674f2e1866b4319b8, target branch v9) succeeded!


Migrate the HTTP Server to TCPListener

All the crashes are due to using HTTPListener instead of TCPListener.
I'll try to make an TCPListener implementation of an httpserver. If someone on the internet knows how to do such thing, please become a friend of the snek.
HTTPListener MUST BE STOPPED! The lives of the sneks depend on it!

I will try, C# isn't my primary language, but I have written basic TCPListeners before.  I will write it as a module that you can include so that bug reports can be made separately.

Maybe this can help?
https://www.codeproject.com/articles/17071/sample-http-server-skeleton-in-c


fix: update eventarc samples to use storage provided events

Description
Addresses b/297966613
Makes all Eventarc Terraform Trigger samples use the same event provider (Cloud Storage).
Checklist
Readiness

[ ] Yes, merge this PR after it is approved
[x] No, don't merge this PR after it is approved

pending cl/562930557

Style

[x] My sample follows the rules described for Terraform in the Effective Samples style guide
[x] My sample follows the other requirements and best practices in the Contributing
guide

Testing

[x] I have performed tests described in the Contributing guide:

[x] Tests pass: terraform apply
[x] Lint pass: terraform fmt check

Automation
These samples use the eventarc API, which can return errors:
Permission denied while using the Eventarc Service Agent. If you recently started to use Eventarc,
it may take a few minutes before all necessary permissions are propagated to the Service Agent

Tests were previously disabled, no change made.

I'm attempting to enable tests for this set, unsure if they'll work, but let's see 🤞


Why call only camera.close() in onStop()

I find that when exit from fragment, it call only camera.close() in onStop()
What is the reason that not call session.close() or session.stopRepeating() before camera.close() ?

If I remember correctly, camera.close() should stop all ongoing requests and close sessions, so it's redundant.


[V2] can't spawn git EACCESS

code-server version: latest
OS Version: latest debian

Description
on startup i see
SW#install
resourceServiceWorker.ts:48 SW#activate
editorWorkerServiceImpl.ts:70  INFO SW#ready
editorWorkerServiceImpl.ts:75   ERR spawn git EACCES: Error: spawn git EACCES
	at Process.ChildProcess._handle.onexit (internal/child_process.js:240:19)
	at onErrorNT (internal/child_process.js:415:16)
	at process._tickCallback (internal/process/next_tick.js:63:19)

Steps to Reproduce
use the oneliner from README.md and then surf to localhost:8080 watch the console

Seems to be a docker-specific issue since I was able to confirm this works properly on local binary.

@sr229 ok i will look if i can debug the docker part

don't have the issue again at present closing temporary


When added to Mobile Phone "springboard" Dust has a proper Icon and Name

Meta and links have already been added to /front/pages/index.tsx, but for some reason does not work…


Improvement: GenIncShockModel.ipynb should compare IndShockConsumerType solution in levels

The GenIncShockModel notebook solves (among other things) for a model in which the "serial correlation" of the persistent shock is 1.0, then shows some results.
It would be great to illustrate the point that that model's solution is (or should be) identical to the solution to the IndShockConsumerType model with the same parameters.  That would both provide a test of the two solution methods and help make very clear how to interpret the solution to the IndShockConsumerType model.

I just verified, it's in /examples/GenIncProcessModel/GenIncProcessModel.ipynb ... This is the figure it produces:

It includes a comparison (to ConsIndShock) right below that, and has this explanation:

I think this can be closed as already done.


-a option not working depending on position in 0.7.4

Are there some kind of positioning requirements for -a (append for log files)?

This works (without the append)
forever start -o logs/stdout -e logs/stderr app.js

This also works
forever start -a -o logs/stdout -e logs/stderr app.js

This won't work - used to work before
forever start -o logs/stdout -e logs/stderr -a app.js

error:  Cannot start forever
error:  script /Users/qasim/Source/console/start does not exist.

the problem is with flatiron
app.cmd(/start (.+)/, cli.startDaemon = function (file) {
It just won't match with start -a app.js

Fixed README to match new usage, thanks guys!


Unhable to load provided pretrained weights

Dear Friend,
After all thanks for sharing your work.
It works fine for me when I train from scratch but when I try to load pretrained weights withaout last layer provided here i get the error "You are trying to load a weight file containing 16 layers into a model with 2 layers"

When I try to load full model pretrained weights (raccoon example) it works fine so I can understand that the problem is that weights are trying to be loaded in the full model when they should be loaded only on model_1 layer.
Sorry but I am quite noob with python and Keras so I can not find a solution.
Thansk for your support

I am sorry. I was not correctly interpreting the parameter "pretrained_weights". I could find in the file "backend.py" that the models are loaded automatically if they are detected in the same directory as the "train.py" file. I'm going to close the ticket.


Delete unused files from the pages directory.

Please do work for this task in a branch called issue-28.  Delete the unused Bowfolios pages from the directory app/imports/ui/pages directory.

Deleted unused pages.


Uncaught type error in browser following clean installation

Upon creating a fresh project using create-react-app and and following the instructions for installing woo into this new directory, I get a "Uncaught TypeError: Cannot read property 'listen' of undefined" error in the Chrome dev tools console.
Reproduction steps:
~/somedirectory$ npm install -g redux-wooo
~/somedirectory$ create-react-app my-react-app
~/somedirectory$ cd my-react-app
~/somedirectory/my-react-app$ wooo -d ./src -n
~/somedirectory/my-react-app$ npm run start

Hello.
The problem was in the react-router because it doesn't come with history dependency since version 4.
I fixed it with specifying a react-router version to be 3.0.2 instead of latest which is 4.0.0.
Thanks for reporting this.


Please update documentation to specify that you MUST select to "Always use this installation on startup"

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
Please update documentation to specify that you MUST select to "Always use this installation on startup"
Describe the solution you'd like
Please update documentation to specify that you MUST select to "Always use this installation on startup"
Describe alternatives you've considered
Please update documentation to specify that you MUST select to "Always use this installation on startup"
Additional context
Please update documentation to specify that you MUST select to "Always use this installation on startup"

This is now in a merge request


Not able to represent decimals with higher precision than Float

Describe the bug
We're mapping an OBX.5 Observation Value. It is a decimal value that requires lossless precision when converting to FHIR. The value we have in HL7 is 1769.859285 and it gets rounded to 1769.8593 when transformed to FHIR. We need to preserve decimal precision without rounding when transformed.
We found out that the rounding is happening because hl7v2-fhir-converter is using Java's Float when mapping OBX.5, which is single-precision and represents a very limited range of values.
The problem is, AFAIK there is no support for a decimal type with higher precision (like Double or BigDecimal)
To Reproduce
We're trying to convert the following OBX segment in an ORM_O01 message:
OBX|1|NM|8339-4^BIRTH WEIGHT MEASURED^LN|1|1769.859285|g^gram^UCUM|||||O|||20230506050000-0500|||||||||||||||QST|AOE
And we get the following value as a result:
"valueQuantity" : {
  "extension" : [ {
    "url" : "https://reportstream.cdc.gov/fhir/StructureDefinition/alternate-value",
    "valueString" : "1769.859285"
  } ],
  "value" : 1769.8593,
  "unit" : "g",
  "system" : "http://unitsofmeasure.org",
  "code" : "g"
}

Expected behavior
We expect the conversion to return "value" : 1769.859285 instead of "value" : 1769.8593
The type for the value is defined in Quantity.yml as follows:
value_1 :
  condition: $value CONTAINS_STRING '.'
  type: FLOAT
  valueOf: $value

We'd like to be able to have something like:
value_1 :
  condition: $value CONTAINS_STRING '.'
  type: DECIMAL
  valueOf: $value

Where DECIMAL would map to BigDecimal, which would preserve the value without rounding. Double could also work
Desktop (please complete the following information):

OS: macOS
Version: Ventura 13.4.1

Additional context
We can volunteer to create a PR to add an additional decimal type if that makes sense

In release 1.1.0


Added Default support for Instant

Added Default Support to Instant so users can use this with the Default Derive.

Given that:

this means Default::default() returns a different value on each invocation,
std::time::Instant does not impl Default, and
the only purpose of this struct is to provide trait implementations not otherwise possible,

I'm going to go ahead and close this PR as rejected. Thank you for the PR, however.


Was PEP 523 implemented in the end ?

The README says PEP 523 is going to happen in Python 3.6, did it ?
If so, it may be worth updating the README.

@stuaxo Yep, it's in: https://www.python.org/dev/peps/pep-0523/

I opened python, made a function and couldn't find a co_extra field on code did it end up being something else.
Maybe I don't know how to read PEPs but I was expecting to see a python version number on the pep itself.

PEP 523 is for the C API; none of it is exposed to Python code.


fix: find client generator by provider instead of the name

The findPrismaClientGenerator was searching the client generator by name, which can be an arbitrary value chosen by the developer
This PR changes to code to finds the generator by provider prisma-client-js.
If this is a very intrusive change, I believe it is worth highlighting in the documentation that the generator title must be equal to client.

I believe that adding this to the documentation may confuse more than help. What do you think?

Probably yes.
thanks!


Update sidebar background color

just a test

Checklist:

[x] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this does not need to be in the release notes.
[x] The title of the PR states what changed and the related issues number (used for the release note).
[x] The title of the PR conforms to the Toolchain Guide
[x] I've included "Closes [ISSUE #]" or "Fixes [ISSUE #]" in the description to automatically close the associated issue.
[x] I've updated both the CLI and UI to expose my feature, or I plan to submit a second PR with them.
[x] Does this PR require documentation updates?
[x] I've updated documentation as required by this PR.
[x] I have signed off all my commits as required by DCO
[x] I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
[x] My build is green (troubleshooting builds).
[x] My new feature complies with the feature status guidelines.
[x] I have added a brief description of why this PR is necessary and/or what this PR solves.
[x] Optional. My organization is added to USERS.md.
[x] Optional. For bug fixes, I've indicated what older releases this fix should be cherry-picked into (this may or may not happen depending on risk/complexity).

/bns:stop

/bns:stop

/bns:delete

/bns:stop


request/suggestion to fix/avoid Space Dim conflicts

Hey there, I'm currently working on my packs next version. While warpdrive is in it I just added advanced Rocketry to my dev build and had a dim id conflict - nothing I couldn't handle but I had an idea in that moment.
Both mods are adding space. There are others doing the same. Why does every mod need an own space if they could be configurable to use an existing dim ID as Space?
I'm requesting one feature. I guess "RegisterARSpace=true/false" would work, while default is true. If false you could simply skip dim registration and on jump into space you could check if the dim that's set as space_dimension_id does exist. If it does, jump there, no matter where that dim comes from, if it doesn't the jump simply fails.
If you don't change anything in config Advanced Rocketry would act exactly like we're used to. but for mod pack authors it would add a lot of possibilities.
Note: I requested the exact same feature over there with almost identical text. Mentioning it to avoid confusion.
One thing that just came into my mind is TiC slime island which can spawn in any dimension. I guess a similar mechanic could be used to add important features to a space dim from another mod.

The biggest problem with something like is that each mod still needs to keep track of where its different sements are in that one dimension.  This would require each mod adding support for every other mod's location handlers.  If this was not done there's a high probablity of mod A and mod B trying to use the same location in space for different things.
If a mod is doing something more advanced than just handling space, for example changing the skybox as to render planets and moons, if another mod needs to modify the skybox as well then one of the mods will take precidence breaking functionality of the other unless compatability were added for every other mod sharing that same feature.
There might be an easier way to do it, but those are the problems I see at least with the way AR is designed, and mods like compact machines likely have a similar design (I've not explicilty looked at how that mod works but given what I've seen ingame it seems plausible that it follows a similar design).
It's a nice idea, but adding extra dimensions is not in reality all that much overhead (especially since unused dimensions are generally unloaded) and trying to share dimensions would just be a huge headache for everyone involved.

You're thinking one or two steps further than me, I guess. I didn't think about changing another mods dim to look like yours. My idea was to add structures (asteroids, for example) to another dimension. Let's talk about Warp Drive for example. That mod generates abandoned shuttles in space. They don't neccessarily need to spawn in warp drives space, that dim could be disabled and their shuttles can be generated anywhere else. I'm not sure if your mod adds similar unique structures, if it does they could be generated in warp drives space, too. Your skybox, on the other hand, is a feature of your space. Disabling your space would disable that feature too, of course. That way the idea is way easier to implement.


Enabling options such as flat_observation for all envs

Cartpole was the only environment that had the option to pass in **kwargs when loading the environment. This pull request adds that option to all the other environments so that optional arguments such as flat_observation may be used.

Hello,
I signed the CLA.
Best regards,
Vibhavari Dasagi

From: googlebot<EMAIL_ADDRESS>Sent: 23 April 2018 08:25:30
To: deepmind/dm_control
Cc: vibhavariD; Author
Subject: Re: [deepmind/dm_control] Enabling options such as flat_observation for all envs (#35)
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.

What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA datahttps://cla.developers.google.com/clas and verify that your email is set on your git commitshttps://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public versionhttps://opensource.google.com/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA datahttps://cla.developers.google.com/clas and verify that your email is set on your git commitshttps://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must also be attached to your GitHub accounthttps://github.com/settings/emails.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/deepmind/dm_control/pull/35#issuecomment-383440409, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ak2T8KtJ2LKFSTAY6yKb7QPEO-Wza_A7ks5trUKigaJpZM4TfLAF.

Thanks, this was merged internally in e0576d6d51e9b2e5143bea447239b9355644142f. Sorry for the delay.


Add  status enum for status codes

They can be used anywhere status is used. For example redirect, head and json
# Example usage
redirect to: Dashboard::Show, status: Status::Permanent
# But cleaner to use without `status:`
redirect to: Dashboard::Show, Status::Permanent
head Status::Created
# Also consider using an external name of `with`
redirect to: Dashboard::Show, with: Status::Permanent

This will make it impossible to have typos or an incorrect status. Also makes it easy to discover status because they'll show up in documentation and editor tooling

Check out #232 for my take on this.

Closed by #232