IndexOutOfBoundsException on clicking back after importing a file

When you import a file and go to next screen, then click back, an IndexOutOfBoundsException is thrown:
java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
at java.util.ArrayList.rangeCheck(ArrayList.java:635)
at java.util.ArrayList.get(ArrayList.java:411)
at org.sakaiproject.gradebookng.tool.panels.importExport.CreateGradeItemStep.(CreateGradeItemStep.java:71)
at org.sakaiproject.gradebookng.tool.panels.importExport.GradeItemImportSelectionStep$1.onSubmit(GradeItemImportSelectionStep.java:62)
at org.apache.wicket.markup.html.form.Form.delegateSubmit(Form.java:1294)
at org.apache.wicket.markup.html.form.Form.process(Form.java:952)
at org.apache.wicket.markup.html.form.Form.onFormSubmitted(Form.java:784)
at org.apache.wicket.markup.html.form.Form.onFormSubmitted(Form.java:717)
at java.lang.reflect.Method.invoke(Method.java:606)

Can you give any additional steps to reproduce?  I'm not able to reproduce in my local version (my forked branch).

That's weird...I am getting totally different results between firefox and chrome.  Hitting back in FF still seems to progress forwards while chrome works as expected.

Maybe that is it, FF is going forwards.
To reproduce in FF, download the full gradebook, then upload it without modification. On the next screen click Back, boom.

I think the issue is that its a submit button so wicket will submit the form. You probably want to handle this as a normal button (like a cancel) with setDefaultFormProcessing to false so it doesnt submit.

I updated the pull request last night with a change.  I didn't set setDefaultFormProcessing, but did turn them into buttons.  Apparently I had some that were still submits and some that were already buttons.

I'm hoping that this issue has been addressed now?

Which branch did you commit against? I cant see any commits on this PR. If you commit to the same branch that the PR was for, then push, git will link it up.
The other thing you might want to do is work in a branch then you can just send a PR for the whole branch.
Are you using Sourcetree? Makes life a lot easier :)

PR #6 is the one that would have had all this stuff in it.  I also just did a small one this morning (PR #27) for the import success message.  I'll switch to work in a branch from here on and see if that makes things any easier.

Ok I don't see any new commits on that one either.
https://github.com/steveswinsburg/gradebookNG/pull/6
Hmm. You can probably just send a PR for the entire master and it should
pickup any commits that havent been merged.
On Thu, Mar 26, 2015 at 12:09 AM, Chris Maurer<EMAIL_ADDRESS>wrote:

PR #6 https://github.com/steveswinsburg/gradebookNG/pull/6 is the one
that would have had all this stuff in it. I also just did a small one this
morning (PR #27 https://github.com/steveswinsburg/gradebookNG/pull/27)
for the import success message. I'll switch to work in a branch from here
on and see if that makes things any easier.
—
Reply to this email directly or view it on GitHub
https://github.com/steveswinsburg/gradebookNG/issues/12#issuecomment-86017111
.

It wasn't a rebase, just a merge, so you'll have my commits sprinkled
throughout your master commits.
Some of that was before I knew how to use git as well!
We've been using it for a while on our project at IU, so I've gotten
better!  We're using some git flow tools too.
I'll look through your stuff to verify if all of my stuff is there or not...
Chris
On Wed, Mar 25, 2015 at 4:09 PM, Steve Swinsburg<EMAIL_ADDRESS>wrote:

Ok I don't see any new commits on that one either.
https://github.com/steveswinsburg/gradebookNG/pull/6
Hmm. You can probably just send a PR for the entire master and it should
pickup any commits that havent been merged.
On Thu, Mar 26, 2015 at 12:09 AM, Chris Maurer<EMAIL_ADDRESS>wrote:

PR #6 https://github.com/steveswinsburg/gradebookNG/pull/6 is the one
that would have had all this stuff in it. I also just did a small one
this
morning (PR #27 https://github.com/steveswinsburg/gradebookNG/pull/27)
for the import success message. I'll switch to work in a branch from here
on and see if that makes things any easier.
—
Reply to this email directly or view it on GitHub
<
https://github.com/steveswinsburg/gradebookNG/issues/12#issuecomment-86017111

.

—
Reply to this email directly or view it on GitHub
https://github.com/steveswinsburg/gradebookNG/issues/12#issuecomment-86199372
.

Ok great. I'll give it a run through tonight and check too.
On Thu, Mar 26, 2015 at 7:27 AM, Chris Maurer<EMAIL_ADDRESS>wrote:

It wasn't a rebase, just a merge, so you'll have my commits sprinkled
throughout your master commits.
Some of that was before I knew how to use git as well!
We've been using it for a while on our project at IU, so I've gotten
better! We're using some git flow tools too.
I'll look through your stuff to verify if all of my stuff is there or
not...
Chris
On Wed, Mar 25, 2015 at 4:09 PM, Steve Swinsburg<EMAIL_ADDRESS>
wrote:

Ok I don't see any new commits on that one either.
https://github.com/steveswinsburg/gradebookNG/pull/6
Hmm. You can probably just send a PR for the entire master and it should
pickup any commits that havent been merged.
On Thu, Mar 26, 2015 at 12:09 AM, Chris Maurer<EMAIL_ADDRESS>wrote:

PR #6 https://github.com/steveswinsburg/gradebookNG/pull/6 is the
one
that would have had all this stuff in it. I also just did a small one
this
morning (PR #27 <https://github.com/steveswinsburg/gradebookNG/pull/27
)
for the import success message. I'll switch to work in a branch from
here
on and see if that makes things any easier.
—
Reply to this email directly or view it on GitHub
<

https://github.com/steveswinsburg/gradebookNG/issues/12#issuecomment-86017111

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/steveswinsburg/gradebookNG/issues/12#issuecomment-86199372
.

—
Reply to this email directly or view it on GitHub
https://github.com/steveswinsburg/gradebookNG/issues/12#issuecomment-86203151
.

Ok this seems to be working ok, however does break with the introduction of the new request mapper:
https://github.com/steveswinsburg/gradebookNG/issues/35
I think the history.back is the issue, can you do that via Wicket? (setResponsePage)

PR #37 has some changes for the back button.  Let me know if that works out for you.  The constructors for my import panel classes are getting pretty messy...  I may look into using some sort of model to wrap up all the stuff I'm passing around.

PR #37 updated with some model stuff now.  May not be the best approach yet, but hopefully cleaner.

Have a look at what I'm doing for the student name panels. Basically
passing in a model which wraps a map of data, then unpack the model in the
onInitialise method (not the constructor). Migrating my stuff over to use
that approach, seems to work pretty well, just need to document the model
object map fields for each panel so ppl know what's in there :)
On 2 Apr 2015 2:27 pm, "Chris Maurer"<EMAIL_ADDRESS>wrote:

PR #37 https://github.com/steveswinsburg/gradebookNG/pull/37 updated
with some model stuff now. May not be the best approach yet, but hopefully
cleaner.
—
Reply to this email directly or view it on GitHub
https://github.com/steveswinsburg/gradebookNG/issues/12#issuecomment-88713770
.

Cleaned up the models some more in PR #40.  Let me know if I'm getting closer!

Ok I'm closing this one off, we can raise separate issues as required.

This issue was moved to sakaiproject/sakai#1331


Add a new tutorial.

Title: "Brillouin Zone Integration on a Photonic Crystal"
Based on crystal.py in the example folder.

Great, thanks for reviewing!
Agreed, will animate it in the future.


[CHEC-977] Update tag styling

ping @kvisca

@robbieaverill Thanks for the ping, updated. Let me know if that change works 👍


Added comment api

Used the involvement API to create a function that adds comment from the input

Hi @Booze33  👏🏽,
Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you consider them as they can improve your code. Some of them were simply missed by the previous reviewer, and addressing them will improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me(@smunoz1988  ) in your question so I can receive the notification


Subscribing to changes from the store doesn't always play nice with hot-reloading

When using something like Rx observables to subscribe to changes in the store, it appears that hot-reloading can sometimes cause the subscriptions to be hooked up again, which can cause duplicate events.

Yeah good point. Need to figure this out before shipping 1.0.

Meh, this won't fit into 1.0.
Let's consolidate this with similar issues in #350.

Now we encourage people to use hot module replacement API directly, so that you have full control: https://github.com/rackt/redux/pull/667


publish to Bower registry

$ bower register sanctuary-tuples git://github.com/sanctuary-js/sanctuary-tuples.git

Done.

Thanks. I didn't realize that it's possible to register a package before it has any version tags.

Neither did I 😄


[wasm] Fix cross-compilation on macOS

59cda3636a7fddb6e68e7c87f5737c6d127c98c8 [wasm] Fix cross-compilation on macOS host machine
We didn't set the CMAKE_SYSTEM_NAME and CMAKE_SYSTEM_PROCESSOR when
cross-compiling stdlib for WASI. This caused the build to fail on macOS
host machines as the build system was trying to build some overlays
assuming the target is macOS.
Additionally, add a default "SWIFT_HOST_VARIANT_SDK" for WASI target
even though we don't support it as a host system yet because it's
required anyway.
24bd384f841df8f357fe35c6cc890cd7b15a4114 [wasm] Configure LLVM for WebAssembly target independently from the native LLVM build
We have been using the host LLVM build directory to configure the Wasm
Swift stdlib. This has been working fine so far as the stdlib build is
configured with CMAKE_SYSTEM_NAME having host system name. However,
we fixed the stdlib build to use WASI as the system name properly in
the previous commit. This exposed the issue that the host LLVM build
directory is not suitable for configuring the stdlib build.

@swift-ci smoke test


CD - Deploy on push to related tag

Setup workflow to deploy all services upon a push to a version tag.

[x] Configure environment switch
[x] Test with beta
[ ] Add URL to action output: https://docs.github.com/en/github-ae@latest/actions/deployment/targeting-different-environments/using-environments-for-deployment#using-an-environment

URL only applicable to nightly.
Unless other env is given a name


quick steps to contribution to this project

This is in relation to #1

@matthewmccullough and @githubteacher: can you review this?

@jaw6, you too!

Looks good to me :+1:

This looks good! 👍 :shipit:


Optimized BM and Enh Rotation

Changed out Spell/AuraID Against Clear name where possible.
Enhancer:

Maelstrom Weapon as Aura Name will not get recognized. Will update later
Optimized Priority List

BM Hunter:

Optimized Priority List
Changed Spell/Aura-ID against Clear name

Need a new Pull for BM Error correction


Do not build whole Solution on Add-Migration

Problem
When adding a new Migration the whole solution is build every time. Because I am working in a solution with a second Startup Project that takes a while to build, the creation of a new Migration is really slow
Possible solutions:

I think it would be enough to only build the Startup Project of the current Add-Migration command. Especially because it is necessary, that the Startup Project references the EF-Core Assembly anyway.

There's a TODO for that. I tried to do it that way earlier on, but ran into limitations in the EnvDTE APIs...

Duplicate of #9716


How to find the transition dipole moment between excited states

The code below calculates the excited states energies and the transition dipole moment of them with ground state.
from pyscf import gto, scf, dft, tdscf
mol = gto.M(
    atom='C 0 0 0; O 0 0 1.1637; O 0 0 -1.1637',
    basis = 'ccpvdz',
    symmetry = True,
    symmetry_subgroup = 'D2h',
)
ks = dft.RKS(mol)
ks.run()
td = tdscf.TDRKS(ks)
td.nstates = 8
td.run()
t_dip = td.transition_dipole()

Is there also a way to find the transition dipole moments between the excited states?

Transition dipole between excited states needs quadratic response, which as far as I know is not implemented yet.


pokemonshowdown.com/users/...

Inaccurate authority, needs to be updated.
Examples:
http://prntscr.com/8nc3ie
http://prntscr.com/8nc3wk
http://prntscr.com/8nc4a4
Thanks for reading!

Not sure what makes you think this is the place to report that, but fixed.

mb


remove skipAutoCreate failpoint from kafka test.

What problem does this PR solve?

Issue Number: close #4675
What is changed and how it works?
Check List
Tests

No code

Code changes
Side effects
Related changes
Release note
`None`.

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-5.2@220b1d5). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release-5.2      #4673   +/-   ##
================================================
  Coverage               ?   57.4065%
================================================
  Files                  ?        189
  Lines                  ?      19179
  Branches               ?          0
================================================
  Hits                   ?      11010
  Misses                 ?       7207
  Partials               ?        962

/merge

kafka test fails 🤔

/run-kafka-integration-test


Update Kubeless chart to use modern architecture and pluggable Kafka & Zookeeper

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
FEATURE REQUEST
Kubeless Chart requires creation of new Zookeeper & Kafka ensemble. I'd expect it to be possible to simply reference another Zookeeper/Kafka deployment instead, as described here:
https://kubeless.io/docs/use-existing-kafka/
Apart from this, the helm chart uses a very old kubeless-controller image instead of the newer kubeless-controller-manager, as described in the official documentation.
Version of Helm and Kubernetes:
Helm: v2.9.0
Kubernetes: v1.9.6
Which chart:
kubeless
What happened:
Chart does not allow for existing Zookeeper/Kafka deployment.
What you expected to happen:
I expected the chart to have a setting for setting the Kafka cluster information and enable/disable the creation of a Kafka cluster.
How to reproduce it (as minimally and precisely as possible):
N/A
Anything else we need to know:
No

Aha, Kubeless has just been updated #5598 (and Zookeeper & Kafka dependencies disappeared), will try again and report back.

New release works, but README.md still references non-existent Zookeeper & Kafka + PVs


set security.protocol to SASL_SSL???

How do I set security.protocol to SASL_SSL for kt client?

This does not appear to be possible now, but will be once #146 is merged.


daiyinger add

error operate

 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.

Erm, what exactly is this pull request supposed to archieve? Are you sure you pressed the right buttons?

Gonna close this on the assumption you did not intend to make this commit into a pull request. If this assumption is off, feel free to say so.


[CWS] option to dump policy from rc

What does this PR do?
Useful to debug the policy received from RC

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge


⚠ Bofrost.be has degraded performance

In 2dfb677, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1527 ms

Resolved: Bofrost.be performance has improved in 7114d64 after 6 minutes.


Unit Tests: pkg/prm/exec

Unit tests cover:

[x] When the tool is unavailable
[x] When the tool is believed available and reports success
[x] When the tool is believed available and reports failure
[x] When the tool is believed available and the tool misbehaves
[x] When the tool is believed available but not found in the backend

Work to create test's has been completed, however in the progress it was discovered that the exec command itself was not functioning as we thought, i.e. while the failure of docker to run the command is properly returned as an error, the failure of the command itself is not.


⚠ ClamAV has degraded performance

In a8bcdbd, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 75 ms

Resolved: ClamAV performance has improved in 6983ddd.


Feature/software/29

added a test case of a web camera and a printed qr code
renamed the Graduation Project Documentation to Graduation-Project-Documentation to avoid paths with spaces problem

The QR code reads correctly but we need to manipulate the size of the printed QR (about 2x2 cm2) code to accommodate the height of the camera.

The QR code reads correctly but we need to manipulate the size of the printed QR (about 2x2 cm2) code to accommodate the height of the camera.

Ok, will scale the printed QR size down.

I have resized the QR code image @Islamoo7

great job.


docs: Updated swipingDirection to docs

Updated README.md for documenting the new swipingDirection parameter of the onSwipeComplete callback

You can do this by something like this.
at first you will have to pass multi directions in an array, and then when ever you will complete the swiping the function named onSwipeComplete will be called and here you can get the direction as an argument and then you can use if else or whatever you want to play with.
swipeDirection={['left','right','up','down']}
onSwipeComplete={(dr) => {
if(dr.swipingDirection === 'left') {
goLeft()
} else if (dr.swipingDirection === 'right') {
goRight();
} else if (dr.swipingDirection === 'up') {
goUP()
} else {
goDOWN();
}
}}


When adding a typeDefinition in the schema generator, using functions as keys to an object leads to an error.

Hey all,
Ran into this little 'bug' while working on one of my projects. I wanted to write code that would load multiple schema definitions from different files. This would allow me to keep different schemas in different files and maintain them separately. This lead to this code:
// file: schemaLoader.js
const wrapSchemaFileContent = content => () => [content];

const loadSchemas() => {
  const schemas = [];
  fs
    .readdirSync(schemaPath)
    .forEach((schemaFileName) => {
      console.info(`Loading: ${schemaFileName}`);
      const schema = SchemaLoader.loadSchema(schemaFileName); //This is basically a file read call.
      const wrappedSchema = wrapSchemaFileContent(schema);
      schemas.push(schema);
    });
    return schemas;
}

// file: graphQLSchema.js
const schemas= loadSchemas();
makeExecutableSchema({
  typeDefs: schemas
});

What you would expect to happen here is that the loader would load all the schemas from the file system, wrap them up and return them to be consumed by the GraphQL toolchain. However the following line in the schemaGenerator class proved to be a problem: https://github.com/apollostack/graphql-tools/blob/master/src/schemaGenerator.ts#L117
What happens here is this expands to a javascript that uses the lamba (in my case wrapSchemaFileContent  as a key for an object). This leads to the fact that every schema after the first one to use that method is skipped and not added to the AST document (which errors down the line).
I managed to fix this by looking at that code and seeing that there is no need to actually wrap my schemas in a function that returns an array of strings (something that is not documented btw). After some light testing it seems that using the Map structure rather than the object fixes this problem (since Map can properly resolve functions as keys if it's the native one).
GraphQL tools version: 0.8.2

@FenrirWillow This is actually the desired behavior here to avoid infinite recursion when you have circular dependencies. In your case you can just not use a function. If you really need a function, then you'll have to make a different instance for every time the function should be evaluated.

Hi @helfer ,
Thanks for the clarification. I figured it had something to do with infinite cycles when creating the AST. Can you point me to a piece of documentation for the project that I can use to get myself familiar with the API and it's restrictions. I didn't know that you can just return strings and pass them to makeExecutableSchema . Thank you for taking the time to answer my question and keep up the good work :)


Registry UI crashes to load 3/5 times with 1000 users

Please see the attached screenshot. There are about 1000 users on this registry but the UI fails to load 3/5 times. I will recheck the issue after pagination gets enabled on the harbor/Users.
Browser used : Chrome.
OS used : Windows 2008

@steven-zou  I think we can dup this one.

Duplicated with #2206 and related with vmware/clarity#913.

We'll support pagination in 1.2.
As this is a duplicated one, I'll close it.


Missing Greek words hotfix

Link is here: https://missing-greek-words-hotfix.prototype-ephemera.pages.dev/
Seems to Work as far as I can tell with the upshot that it's currently faster.
@PurpleGuitar  - I'd suggest ultimately we could make this whole thing a little cleaner and faster with not tooooooo much work.
In a nutshell, I'd also port this to an SSG site, and set up a pair of webhooks to rebuild when the tagged ulb changes or the greek word changes.
That's effectively what this is now that he has the wrods adn the ulb just bundled into the public folder.
Theoretically we could even keep it in the same repo as it's currently in and just set up the github actions, but I'd be tempted to migrate it off of create react app since its pretty much deprecated.


Added Datascience roadmap

Related Issues

issue goes here

I have added the Data Science roadmap to the project. This roadmap provides an overview of the key concepts and skills related to data science, including data collection, analysis, and machine learning techniques.
Changes Made

Added a new object for "Data Science" in the JSON array.
Included information about data science, its importance, and the skills it encompasses.

Roadmap Preview
[
    {
        "id": 1,
        "title": "Frontend Web Development",
        // ... (existing data)
    },
    {
        "id": 2,
        "title": "Full Stack Development (MERN)",
        // ... (existing data)
    },
    {
        "id": 3,
        "title": "Data Science",
        // ... (new data)
    }
]

# Additional Information
- Any additional information or context

# Checklist
- [ ] Tests
- [ ] Translations
- [ ] Documentations

# Screenshots

![image](https://github.com/Opentek-Org/opentek/assets/85815172/433fa756-10ca-44ca-afc1-8acde342da9b)

@AnirudhPanda ,please review and merge pull request


VW: Increase stock ACC button spam reliability

The cleanup in #25168 resulted in a regression of cancel/resume spam reliability. Restore some of the timing control (sending just after the car sends) from the prior version, maintaining overall clarity.
LGTM with limited testing, but holding in draft for a few days to get additional test users and testing time.

I think this is good. Two more test users also report good results. Releasing from draft.
3cfdec54aa035f3f|2022-09-07--15-09-49
bbc2fa573509b5ff|2022-09-08--17-36-21


the experiment performance

hi! it's a wonderful job in this mission, but i want to ask some question about your code performance. as your instructions, i use the hyper parameter noticed in the md, but i cannot get the same performance as your paper says. the all test hits@ is a bit lower than the paper performance, like 0.8%. could you give me your seed or some example of hyperparameter setting, or maybe it's some performance fluctuation? it really bothers me. thank your project! i like your idea!

Thanks for your concerns. It would be nice of you if you can provide us with the parameter settings and your results of different datasets to achieve better reproducibility.

i have tried several dataset. in yago and wiki,the performance is pretty good, but in icews18, it seems that cenet cannot achieve enough performance. my result is 58% in all test samples. i have used 0.2 alpha 2 lambda, and your seed. it confused me for a week. if you have some tips on that, i will appreciate it. i will try my best to do more experiments on that. thanks a lot!


UI of landing page needs to be upgraded.

The name of the website is in center which looks so unprofessional, Needs to be upgrade with clickable to load the page, also as it is a open source project, we have to add a github logo link to this project on nav bar.
The footer contains copy right disclaimer is not up to date and it is not looking professional.

i hope this needs  enhancement , kindly assign me this issue if you feel its relevant otherwise please do ignore it.
Header part

Footer part

How it will look>>  p.s== more enhancement will be done

@Shantanu-Meta This page is already assigned to improve. So may be I can assign you after that implementation.

I want to work on this issue under JWOC

I want to contribute in this under JWOC

Please assign this issue to me

@Shantanu-Meta This page is already assigned to improve. So may be I can assign you after that implementation.

okay then if you feel to improve let me know


.onAfterResponsive event

It would be nice to have it :)

Added on main branch. Should be built in to next release :)


Changes and fixes on the Front-end homepage

Related Issuse

Login and sign-up buttons are slightly out of body.
The hovering effect of both of them does not comply with the orange background.
In side navbar - there is small gap between whatsapp and facebook icon.
The email box inside the footer , have send button unintended

Closes: #[504]
Describe the changes you've made
1)Fixed the login and sign-up positioning , changed it's hover effect and also changed the text color with respect to background.
2)Fixed the gap that is in the side nav-bar and positioned it to the right for clear and transparent look.
3)Fixed the position of send button inside the e-mail textbox and edited the box itself.
Checklist:

[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.(Specially for Backend)
[ ] My changes generate no new warnings.

Screenshots

Original
Updated

original screenshot
updated screenshot

@shivamp296 Can you please attach screenshot with PR? also read the PR template.

@shivamp296 Can you please attach screenshot with PR? also read the PR template.

I have attached the screenshot , please see it once. Thank you..

@shivamp296 Could you please resolve the conflict.

@shivamp296 Could you please resolve the conflict.

Okkk

The conflicts have been resolved and the pull request is ready to merge.


add container resource profiles

Closes: #26
Adds a few demo profiles such as laptop, desktop, raspberry_pi, server to the graph config.
While we are using docker-compose, and not swarm, we cannot easily limit physical CPUS (withough downgrading to docker-compose v2), but we can get "relative resourse usage", and memory limitations.
This line:
"cpu_shares": int(self.resource_profile.get("cpus", "0.5")) * 1024,

is the temporary "hack" which converts our cpu number from a profile to a rough cpu_shares value (which docker-compose accepts).
Link: https://docs.docker.com/config/containers/resource_constraints/

Concept ACK
Need to look at this with the k8s backend, but I think it should work by having the resource profiles and then each backend translates that into something the backend can understand

If we want ot continue with tc_netem, then I think it just needs a big rebase.
If we want to pass it as actual pod resource profile to the cluster (can you even do that), then it might need a total re-write...

You can set pod resource limits (CPU, memory) directly when defining the pod in k8s. But we could also use tc_netem throughout, because if I remember correctly that's happening inside a docker container? So it should work on both and iirc, it gives more options than just CPU/RAM.
Maybe we just stick with tc_netem through for now? Longterm, I can see value in having the profile be a "warnet" concept, and then each backend translates the profile into what the backend needs


Fetching data

Firstly, thanks for working on this library!
I wondered if there were any plans for supporting pre-populating/fetching data prior to SSG, functionality similar to Nuxt's fetch hook? Perhaps it's already possible or outside the scope of this library.

Hi @rdunk, can you share with me some examples of how would you use this feature? I personally only use this lib to build some blog-like sites, haven't got a chance to have that need yet. Thanks.

Hi @rdunk, can you share with me some examples of how would you use this feature? I personally only use this lib to build some blog-like sites, haven't got a chance to have that need yet. Thanks.

I found this library whilst looking for an alternative to Nuxt's generate command for SSG, but compatible with Vue 3. I wanted something more lightweight than Nuxt that I could drop into a normal Vue/Vite project, so thanks for building it.
It works great for generating static builds where page content exists within the repository. However I'm authoring content using a headless CMS, i.e. outside of the repository. Ideally I would be able to fetch this data at build time via some async hook. In a normal SPA I would show a loading indicator, fetch the data and update state. With SSG obviously things differ, as we need to wait for (often specific) async functions to execute before attempting to render the page. Nuxt's fetch hook was the most similar example I could find to separating out the logic for performing those functions.
A simple example would be creating an "About" page. Content is authored using a CMS which exposes JSON via some API endpoint. The generator would execute a request and pass the returned data to the component somehow, prior to rendering the page.
I just wanted to check if you had any existing plans for implementing something like this really. I understand this kind of feature is non-trivial, so I'm going to spend a few hours seeing what I can come up with.
My other thoughts were around SSG for routes with dynamic paths, with something akin to Next.js's getStaticPaths. I know this library filters out dynamic routes currently, so I'm sure you have given this some thought already—and I guess decided against that for now—as it definitely increases the scope of the library considerably. Again perhaps this is veering too much towards Nuxt territory, but I'll see if I can hack something together anyway.

I found this library whilst looking for an alternative to Nuxt's generate command for SSG, but compatible with Vue 3. I wanted something more lightweight than Nuxt that I could drop into a normal Vue/Vite project, so thanks for building it.
It works great for generating static builds where page content exists within the repository. However I'm authoring content using a headless CMS, i.e. outside of the repository. Ideally I would be able to fetch this data at build time via some async hook. In a normal SPA I would show a loading indicator, fetch the data and update state. With SSG obviously things differ, as we need to wait for (often specific) async functions to execute before attempting to render the page. Nuxt's fetch hook was the most similar example I could find to separating out the logic for performing those functions.
A simple example would be creating an "About" page. Content is authored using a CMS which exposes JSON via some API endpoint. The generator would execute a request and pass the returned data to the component somehow, prior to rendering the page.
I just wanted to check if you had any existing plans for implementing something like this really. I understand this kind of feature is non-trivial, so I'm going to spend a few hours seeing what I can come up with.
My other thoughts were around SSG for routes with dynamic paths, with something akin to Next.js's getStaticPaths. I know this library filters out dynamic routes currently, so I'm sure you have given this some thought already—and I guess decided against that for now—as it definitely increases the scope of the library considerably. Again perhaps this is veering too much towards Nuxt territory, but I'll see if I can hack something together anyway.

Reminds me of https://twitter.com/_egoistlily/status/1353370388386865153?s=20, maybe you can give it a try as well. I will need some time to think if I could come up with a better API :)
Thanks for the detailed explanation.

Reminds me of https://twitter.com/_egoistlily/status/1353370388386865153?s=20, maybe you can give it a try as well. I will need some time to think if I could come up with a better API :)
Thanks for the detailed explanation.

In vite-ssg you can use serverPrefetch () to prefetch data in server side.
example: https://github.com/antfu/vite-ssg/blob/main/example/src/pages/b.vue

In vite-ssg you can use serverPrefetch () to prefetch data in server side.
example: https://github.com/antfu/vite-ssg/blob/main/example/src/pages/b.vue

For the composition API version, please keep track on https://github.com/vuejs/vue-next/pull/3070.

If someone is looking for a way to do this with the composition API:
import axios from 'axios'
const { get } = axios

// The data we will show
const repo = ref<RepoData | null>(null)

// A function to get the data
const fetch = async() => {
  const data = await get(`https://api.github.com/repos/nickgraffis/petite-vin`)
  repo.value = data.data
}

// Only run this function client side if we don't have a value assigned to repo yet
// if repo already has a value, it was fetched during build
if (!repo.value) fetch()

// During the build, fetch the data before building the component
onServerPrefetch(async() => {
  await fetch()
})

This will refetch the data client side while in dev mode, but after building we no longer fetch the data


Set/Retrieve value in BAS from native application.

I found few helper functions, but there's no documentation on the arguments they intake.
Therefore i few questions.

How can i set a 'test=2' value or pass value from native C application to the BAS ?
How can i retrieve the value of 'test' to process in native application after the BAS is executed ?

 struct mb_interpreter_t* bas = NULL;
  mb_init();
  mb_open(&bas);
  if (mb_load_string(bas, "test = 1+2+3; print test", true) == MB_FUNC_OK)
  {
    int iResult = mb_run(bas, true);

    mb_value_t val0;
    void **l = NULL;
    int iRes = mb_get_value_by_name(bas,l,"test",&val0);
    printf("VALU:%i RES:%i\n", val0.value.integer,iRes);
  }
  else
    printf("Failed\n");

6
VALU:0 RES:0

What are the arguments voidl**  in the following functions ?

MBAPI int mb_get_value_by_name(struct mb_interpreter_t* s, void** l, const char* n, mb_value_t* val);
MBAPI int mb_add_var(struct mb_interpreter_t* s, void** l, const char* n, mb_value_t val, bool_t force);
MBAPI int mb_get_var(struct mb_interpreter_t* s, void** l, void** v, bool_t redir);
MBAPI int mb_get_var_name(struct mb_interpreter_t* s, void* v, char** n);
MBAPI int mb_get_var_value(struct mb_interpreter_t* s, void* v, mb_value_t* val);
MBAPI int mb_set_var_value(struct mb_interpreter_t* s, void* v, mb_value_t val);

Found it myself
 void **l = NULL;
  struct mb_interpreter_t* bas = NULL;
  mb_init();
  mb_open(&bas);
  if (mb_load_string(bas, "test = 1+NATIVE; print test", true) == MB_FUNC_OK)
  {

    // Define a value
    mb_value_t valAdd;
    valAdd.type=MB_DT_INT;
    valAdd.value.integer = 1;
    mb_add_var(bas,l,"NATIVE",valAdd,true);

    // Run
    int iResult = mb_run(bas, true);
    printf("\n");

    // get value (NEEDS TO BE UPPERCASE)
    mb_value_t valGet;
    int iRes = mb_get_value_by_name(bas,l,"TEST",&valGet);
    printf("VALU:%i TYPE:%i RES:%i\n", valGet.value.integer,valGet.type,iRes);

  }
  else
    printf("Failed\n");

    mb_close(&bas);
    mb_dispose();

Interesting find. Two questions for @paladin-t maybe, at what point is void** l initialized? In the above example, it's first referenced at mb_add_var, does that write something to l? What does l even stand for? And, why does the name of a variable in mb_get_value_by_name need to be uppercase?

Lets' reopen to get more clarity on the follow up questions

void** l stores the program pointer, it is used to determine what's being executed and what will be next step internally. It is exposed to most API for manual save/restore externally sometimes.
The full flow might be following in some cases:

BASIC calls a native API
the native API calls mb_add_var
if there's any error, mb_add_var reports it with the current program position retrieved from void** l

mb_get_value_by_name expects uppercase due to a historical reason. BASIC is case-insensitive, and MY-BASIC stores almost everything in uppercase, then mb_get_value_by_name was introduced at some timepoint after that.

Thanks very much.


MacOS improvements

This PR adds some improvements related to running the OpenAdapt framework on macOS (tested on macOS Ventura), namely:

missing dependencies requirements
add instructions to set up permissions for proper execution of framework

See inline comments for further details.

Thank you @atineoSE !
What should we do about the instructions here? https://github.com/MLDSAI/OpenAdapt#troubleshooting

Good point about the troubleshooting section, @abrichr. I have suggested a change at commit f6917b5, to redirect the reader to the permissions page. I haven't seen those terminal error messages, I got the system alerts instead.

@abrichr I have rebased over latest changes and fixed conflicts. Please, let me know if you need anything else before this can be merged. Thanks!

Thank you @atineoSE ! Will look at it ASAP 🙏


Update Set-CsTenantFederationConfiguration.md

Adding information to the 'AllowedDomains parameter specifying that it can support up to 4,000 domains.

@JohanFreelancer9

@MikePlumleyMSFT Given that this https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/pull/10270#pullrequestreview-1568750490 was closed, is this PR ok? Thank you

@ASolom1 is the limit only for allowed domains or is it a combo of allow/block?

@Mike @.***>
@Ailin @.***> - do you know the answer to Mike's question below?
From: Mike Plumley @.>
Sent: Tuesday, August 29, 2023 1:18 PM
To: MicrosoftDocs/office-docs-powershell @.>
Cc: Alex Solomon @.>; Mention @.>
Subject: Re: [MicrosoftDocs/office-docs-powershell] Update Set-CsTenantFederationConfiguration.md (PR #11205)
@ASolom1https://github.com/ASolom1 is the limit only for allowed domains or is it a combo of allow/block?

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/office-docs-powershell/pull/11205#issuecomment-1697847073, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXOHTGAFAHWK3ZQ252QLKFDXXYP5RANCNFSM6AAAAAA4BV2N4M.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

It is a combo of allow & block lists.
From: Alex Solomon @.>
Sent: Tuesday, August 29, 2023 7:59 PM
To: MicrosoftDocs/office-docs-powershell @.>; MicrosoftDocs/office-docs-powershell @.>; Mike Plumley @.>; Ailin Menabil @.>
Cc: Mention @.>
Subject: RE: [MicrosoftDocs/office-docs-powershell] Update Set-CsTenantFederationConfiguration.md (PR #11205)
@Mike @.***>
@Ailin @.***> - do you know the answer to Mike's question below?
From: Mike Plumley<EMAIL_ADDRESS>Sent: Tuesday, August 29, 2023 1:18 PM
To: MicrosoftDocs/office-docs-powershell<EMAIL_ADDRESS>Cc: Alex Solomon<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/office-docs-powershell] Update Set-CsTenantFederationConfiguration.md (PR #11205)
@ASolom1https://github.com/ASolom1 is the limit only for allowed domains or is it a combo of allow/block?

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/office-docs-powershell/pull/11205#issuecomment-1697847073, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXOHTGAFAHWK3ZQ252QLKFDXXYP5RANCNFSM6AAAAAA4BV2N4M.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@MikePlumleyMSFT are we still ok to document this? Thanks

@MikePlumleyMSFT Kindly update.


Keep parameter in the archive

Types of changes

[x] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue
Closes #290

How Has This Been Tested (if it applies)

Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CONTRIBUTING).
[x] All tests passed, and additional code has been covered with new tests.

@teytaud this will change a lot of things since now we can access the actual best candidate (not just their representation in R^n

discussed with @teytaud and accepted


Add setter-stx argument to make-variable-like-transformer

se #74

think this is needed anymore for some reason


Run benchmark automatically for performance pull requests

Description
Right now the process of benchmarking changes is rather tedious and possibly even intimidating for newcomers.
This PR aims to help with this by automatically generating a benchmark report for pull requests labelled with "performance".
You can see it in action here: https://github.com/siku2/yew/pull/2
(Please ignore all of the deleted messages and force pushes. I was using the PR during development :D)
I wanted to use  a /benchmark command to run the benchmarks manually but GitHub actions doesn't really support that right now...

Sweet, this looks good for now, we'll likely need to tweak it over time but that's fine with me