Proposal: Branch based params

Background
ods.yaml is the central point to configure a pipeline run. One common configuration need is to deploy to different environments based on the branch. Inline with the Jenkins shared library, ods.yaml allows this via a combination of branchToEnvironmentMapping and environments. The pipeline manager selects an environment to deploy to based on this configuration, then injects this as ODS context into the pipeline, which the ods-deploy-helm task uses to know where to deploy to.
While this works for now, the solution has several issues:

It couples ods-deploy-helm with the ods.yaml file - changes in ods-deploy-helm may require changes to ods.yaml which could potentially lead to issues for other tasks relying on that information.
It is special-casing ods-deploy-helm. If other tasks have similar needs (e.g. a task deploying a Terraform module) the configuration options available in ods.yaml may not be a good fit or may not even be enough information
branchToEnvironmentMapping overlaps with the recently introduced pipeline trigger functionality. Using triggers, it would be possible to have a separate pipeline for each environment, and hardcode the target environment in each pipeline.

Proposal
Instead of branchToEnvironmentMapping and environments, we could have a new field named e.g. branchParams. This field would allow to set parameters depending on the current branch. The pipeline manager would then use this field to inject the params into the pipeline.
For example, the configuration could look like this:
branchParams:
- branch: *
  params:
  - name: api-server
    value: 'api.dev.example.com'
- branch: develop
  params:
  - name: target-namespace
    value: foo-dev
- branch: release/*
  params:
  - name: target-namespace
    value: foo-test
- branch production
   params:
   - name: api-server
     value: 'api.prod.example.com'
  - name: target-namespace
    value: foo-prod

pipeline:
  tasks:
  - name: deploy
     taskRef:
       kind: Task
       name: ods-deploy-helm-v0-10-1
     params:
     - name: target-namespace
       value: $(params.target-namespace)
     workspaces:
     - name: source
       workspace: shared-workspace

A few notes:

Branches would match according to the current rules of branchToEnvironmentMapping. However, we would likely want to allow multiple matches so that one can have generic params (branch: *) and branch-specific params. If a param is specified in multiple matching branch patterns, the most specific one would win.
The params list would be of type Tekton Param so that they can easily be used in Tekton tasks as-is. Therefore, those the param values could also be arrays or, following the latest changes in Tekton, objects.
The ods-deploy-helm task would need to change to accept all fields of the current environment struct as params So instead of knowing an environment name, and then reading from ods.yaml, the task would have params for api-server, registry, target-namespace etc.

There could also be a variation of this configuration, allowing to also set task parameters directly. This would reduce verbosity, but may be more likely to break. Technically, the parameters would be set on the pipeline run spec in the pipeline manager.
branchParams:
- branch: *
  params:
   - name: debug
     scope: pipeline
     value: 'true'
- branch: develop
  params:
  - name: target-namespace
    scope: tasks.deploy # name of task in pipeline run spec
    value: foo-dev
- branch: release/*
  params:
  - name: target-namespace
    scope: tasks.deploy
    value: foo-test
- branch production
   params:
   - name: api-server
     scope: tasks.deploy
    value: 'api.prod.example.com'
  - name: target-namespace
    scope: tasks.deploy
    value: foo-prod

pipeline:
  tasks:
  - name: deploy
     taskRef:
       kind: Task
       name: ods-deploy-helm-v0-10-1
     workspaces:
     - name: source
       workspace: shared-workspace

Of course there are many different ways how the configuration could exactly look like e.g. instead of having a scope field one could use a map of keys (pipeline / task names) to params.
Alright, at this point my goal is to just throw this out there and get some feedback on the idea. What do you think @henrjk @kuebler @henninggross @gerardcl?

A few more thoughts from me on my proposal above:

The second variant would allow to set params on start/finish tasks. The first one would not allow that.
The start/finish tasks require some information from environment right now (stage). This would need to be replaced by some other mechanism (e.g. default or mandatory parameter)
The second variant should likely change how it sets the params as the scope field is not part of the Tekton Param type, and it would be nice to not use a custom type
The name branchParams might not be ideal as we'd likely want the same concept to work for tags as well

The alternative could be rewritten like this maybe:
branchParams:
- branch: develop
  params:
  - task: deploy # name of task in pipeline run spec
    param:
      name: target-namespace
      value: foo-dev

Potentially, task could be optional, and if it isn't set, the param could be a pipeline-level param ... but it looks a bit funny then.

From the description:

branchToEnvironmentMapping overlaps with the recently introduced pipeline trigger functionality. Using triggers, it would be possible to have a separate pipeline for each environment, and hardcode the target environment in each pipeline.

At first glance having pipelines per environment and then hardcoding or better configuring target environment related information outside of the pipeline makes sense to me. Your proposal does not pursue this it seems? Am I wrong or what are the reasons not to do that?

Yet another iteration of this is now ready for review in https://github.com/opendevstack/ods-pipeline/pull/685.


placeholder as pattern

I would like a table of contents creation for the following 4 cases:
[toc]
[TOC]
[[toc]]
[[TOC]]

Unfortunately the placeholder does not allow to specify a pattern, otherwise I could just use /^\[{1,2}toc\]{1,2}/im.
Any chance you could allow the use of a regex pattern?

No, using regular expression patterns is considered harmful because it makes impossible to fast reject tokens. As pointed out in the README:

Another very common (C1) mistake that can be found in the wild is using an optional regular expression to match the placeholder in the source, but using a hard-coded charCodeAt value to reject a token.

All libraries that offered this option are, in fact, defective in their implementations. Our recommendation is to choose a placeholder and stick with in the entire database, fixing possible inconsistencies in the process.

Ok, then. How about allowing to use an array to specify more than one placeholder? Would that be sufficiently safe?

It would be better, but this seems to me that you are trying to solve a problem in the wrong layer. Going from a string to an array will increase the complexity of the algorithm with no clear advantage. Really, why supporting so many different placeholders in the same project makes any sense? A grep | sed would be enough to solve this kind of inconsistencies. For a plugin, fast behavior is a valuable asset because it's usually working with many other components and while it may seem like a minor issue alone, it can harm you in the big picture.

Well, I'm evaluating the use of this plugin for the project Joplin. I cannot say, if it will be included, but I'm sure that people are used to different markers. The ones primarily used are the 4 I mentioned (most available markdown editors either use [toc] or [[toc]]). That's why.

Behavior implemented in 3.0.0. The described cases are built-in and also the Azure DevOps wiki style [[_toc_]].

Nice, thank you. So the placeholder variable is obsolete or does it overwrite the default cases? Or does it add to the default cases?

Its the same option, with backward compatibility. Just augmented to handle extra cases. It's used as a partial pattern which get anchored later. I also had to modify the rejection strategy. Bumped a version because users already using that option will face a compat break


optimum-cli error in Windows

System Info
optimum 1.7.1, windows 11

Who can help?
@michaelbenayoun
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
> optimum-cli export onnx --model microsoft/beit-base-patch16-224 --device cuda beit_onnx/

Python was not found; run without arguments to install from the Microsoft Store, or disable this shortcut from Settings > Manage App Execution Aliases.
Traceback (most recent call last):
  File "E:\Anaconda3\envs\py310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "E:\Anaconda3\envs\py310\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "E:\Anaconda3\envs\py310\Scripts\optimum-cli.exe\__main__.py", line 7, in <module>
  File "E:\Anaconda3\envs\py310\lib\site-packages\optimum\commands\optimum_cli.py", line 40, in main
    service.run()
  File "E:\Anaconda3\envs\py310\lib\site-packages\optimum\commands\export\onnx.py", line 191, in run
    subprocess.run(full_command, shell=True, check=True)
  File "E:\Anaconda3\envs\py310\lib\subprocess.py", line 526, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'python3 -m optimum.exporters.onnx --model microsoft/beit-base-patch16-224 --device cuda beit_onnx/' returned non-zero exit status 9009.

I tried python -m optimum.exporters.onnx --model microsoft/beit-base-patch16-224 --device cuda beit_onnx and it is good.
Expected behavior
Expect that optimum-cli could run in Windows, where there is no python3 (see https://stackoverflow.com/questions/39910730/python3-is-not-recognized-as-an-internal-or-external-command-operable-program).

Thank you @tianleiwu , would you give a try on 1.7.3? The subprocess call has been removed from the CLI: https://github.com/huggingface/optimum/pull/897

@fxmarty, I tried 1.7.3 and the issue is resolved. Thanks


Containerd defaults

What feature would you like to be added?
For clusters 1.21+ containerd should be a default (and no docker allowed) container runtime.
What are use cases of the feature?
Kubernetes 1.21+ is expected to delete docker support from kubelet

/close

/close


Implement SourceBuilder utility to accumulate entries into a Source [phab:D1964]

Created by schroederc at 2017-10-02 22:35:34:

schroederc wrote at 2017-10-02 22:39:16:
Separate BUILD target

zarko accepted this revision at 2017-10-02 22:51:12

jrtom commented inline (context lost during migration) at 2017-10-02 22:54:30:
consider renaming this "asString", it's a bit weird to have a "toString" static method on a class.

jrtom wrote at 2017-10-02 22:54:30:
(drive-by comment)

zarko wrote at 2017-10-02 23:18:49:
This change was missing DeleteRecursively.java and Span.java.

schroederc wrote at 2017-10-03 16:24:56:

! In (earlier comment), zarko wrote:
This change was missing DeleteRecursively.java and Span.java.

I'm not sure what you mean.  Those classes already existed.  I just split up the rules.

zarko wrote at 2017-10-03 16:41:24:
False alarm, looks like Bazel is sick. Worked after cleaning.


Fixed bug #615 - Resolved the overflow of text in the cards of FAQ section and added fifth star in the third card

Fixed :

Resolved text overflow issue in FAQ section cards.
Adjusted star rating in the third card.

Thank you @HritikaPh


修改issues-102

fat:Modify the autoheight component to return to the vertex after rendering

@wzh425 PTAL


What's meaning of A/U?

Some applications are marked a 'Y',others aren't.
What's the meaning of A/U?
Expect for your answer.😃

Hello @isoulgh ,
A/U means auto-update, those applications have a flag in the cask that they support auto-update functionality, therefore there is a possibility that they updated themselves. That's why in order to update those you need to pass -a / --all option to update them through brew cu.
Hope that makes sense 🙂

Get it.
Thank you.Thank you.


Please consider update periodically

[Enter feedback here]
I see the page states:
"Microsoft does not plan to update them for newly added services periodically"
Requesting that Microsoft please consider to update with newly added services periodically.
Thank you for your consideration.

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

ID: 7c245452-161d-89a1-c026-99a0721cf071
Version Independent ID: 16dec59a-eb83-1837-4efd-4645e7aeaed2
Content: Product names and service plan identifiers for licensing - Azure AD
Content Source: articles/active-directory/users-groups-roles/licensing-service-plan-reference.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @kkern
Microsoft Alias: kakern

@alexiq
Thanks for your feedback! We will investigate and update as appropriate.

Thanks, @alexiq. I have passed your feedback along to the content author, @kkern13


Doesn't play well with template strings in ternaries

I was very excited to find this plugin as the super long classnames with tailwind are hard to work with. Would really love to be able to use it in all of my projects
Dependency information
  "devDependencies": {
    "@storybook/addon-essentials": "^7.6.7",
    "@storybook/addon-interactions": "^7.6.7",
    "@storybook/addon-links": "^7.6.7",
    "@storybook/addon-onboarding": "^1.0.10",
    "@storybook/addon-themes": "^7.6.7",
    "@storybook/blocks": "^7.6.7",
    "@storybook/nextjs": "^7.6.7",
    "@storybook/react": "^7.6.7",
    "@storybook/test": "^7.6.7",
    "@types/node": "^20",
    "@types/react": "^18",
    "@types/react-dom": "^18",
    "@typescript-eslint/eslint-plugin": "^6.19.1",
    "@typescript-eslint/parser": "^6.19.1",
    "autoprefixer": "^10.0.1",
    "eslint": "^8.56.0",
    "eslint-config-next": "14.0.3",
    "eslint-config-prettier": "^9.1.0",
    "eslint-plugin-no-relative-import-paths": "^1.5.3",
    "eslint-plugin-prettier": "^5.1.2",
    "eslint-plugin-storybook": "^0.6.15",
    "postcss": "^8",
    "prettier": "^3.2.4",
    "prettier-plugin-classnames": "^0.5.0",
    "prettier-plugin-organize-imports": "^3.2.4",
    "prettier-plugin-tailwindcss": "^0.5.11",
    "storybook": "^7.6.7",
    "tailwindcss": "^3.4.1",
    "typescript": "^5.3.3"
  }

prettier.config.js
module.exports = {
  plugins: [
    "prettier-plugin-organize-imports",
    "prettier-plugin-tailwindcss",
    "prettier-plugin-classnames",
  ],
  tailwindConfig: "./tailwind.config.ts",
  tailwindFunctions: ["tw"],
  customAttributes: ["className"],
  customFunctions: ["cn"],
};

Steps to reproduce
Format a file containing a ternary that returns a template string containing a javascript expression.
The current behavior
Template strings appearing in ternaries with javascript expressions in them get changed erroneously to regular strings when formatting:
const { data } = useSWR<CartResponse>(
-  cartId ? `${process.env.NEXT_PUBLIC_API_URL}/cart/${cartId}` : null,
+  cartId ? "${process.env.NEXT_PUBLIC_API_URL}/cart/${cartId}" : null,
);

However, when there are no ternaries, the template string remains unchanged:
const { data } = useSWR<CartResponse>(
  `${process.env.NEXT_PUBLIC_API_URL}/cart/${cartId}`,
);

The expected behavior
Don't change template strings containing javascript expressions in ternaries.

I think there was something I missed while implementing #25.
Since similar issues related to template literals have been reported, I will prioritize fixing related issues, but since I can only do this after work, it usually takes a few days.
For now, you'll probably be fine using v0.4.0.


RBAC manifests in Github don't match official docs

In comparing the RBAC manifests below, it seems that the ClusterRole doesn't match between the docs here on Github and the official Datadog agent docs.
https://github.com/DataDog/datadog-agent/blob/master/Dockerfiles/manifests/rbac/clusterrole.yaml
https://docs.datadoghq.com/integrations/faq/using-rbac-permission-with-your-kubernetes-integration/

Hey @julianvmodesto, indeed there are a bunch of changes that are merged on master but not yet "released" that's why some recent changes are not yet in sync with documentation.
We'll make sure to update the doc when 6.2 is released (ping @l0k0ms)

Hello,
Doc PrR is on its way, thanks for the feedback :)
Best regards,
Pierre


Webclient requires additional steps to log in from new ip

The docs state "Users who don’t use two-factor auth will likely need to enable less secure login. If this is needed, a warning will be logged during login (which will print to stderr in the default logging configuration)."
However, I cannot get login to work. I've tried with 2-step & app-specific passwords, and I've tried without 2-step & less secure login enabled. Either way, when I login, it always returns false. When looking at the log file, I see the following entry: "gmusicapi.Webclient3 (webclient:61) [INFO]: failed to authenticate". Mobileclient and Musicmanager login just fine.
I know it's suggested to not use the Webclient, but as far as I can tell, it's the only way I can get my cover art uploaded correctly (Webclient.upload_album_art). When uploading music via Musicmanager, my cover art is removed from the MP3.
Is Webclient login no longer working?

Got it! Started running session.py line-by-line over SSH, and found that after submitting the password, there was a third screen where it was asking me to do one of 4 things:

Get a text message with a verification code at (•••) •••-••99
Get a phone call with a verification code at (•••) •••-••99
Confirm the recovery phone you added to this account
Confirm the recovery email you added to this account

I simulated submitting one of these steps over SSH, and then Webclient.login worked perfectly.
        form_candidates = response.soup.select("form")
        form = form_candidates[3]
        response = browser.submit(form, 'https://accounts.google.com/ServiceLoginAuth')
        form_candidates = response.soup.select("form")
        form = form_candidates[0]
        form.select('[name=email]')[0]['value'] =<EMAIL_ADDRESS>        response = browser.submit(form, 'https://accounts.google.com/ServiceLoginAuth')

I suspect I needed to do this step since I'm running gmusicapi off of a remote webserver, and therefore Google was trying to block the login since it didn't appear to be a known IP address. This is just a theory, but it makes sense.

Ah. Yeah, we've had trouble with webclient auth on remote machines: I actually turned off webclient testing from travis because of it.
I don't think there's an easy solution. The webclient just wasn't built for programmatic auth.

It's all good. Now that I've successfully logged in with the work-around above, that IP address is now known to Google, and I am longer having login issues. Hopefully this thread helps someone else who is having the same issue.

I had some problems with using a TFA account with WebClient - it won't accept an app-specific password, but it's not possible to use less-secure login with TFA. I added the following lines in session.py after submitting the password. I also had to change the login method in WebClient to accept kwargs so as to be able to pass the TFA token across; now I can log in using something like wc.login("email", "password", totp="TFATokenFromAuthenticator")
        if 'totp' in kwargs:  # Using 2FA
            form_candidates = response.soup.select("form")

            #commented out as there is another form to try a different option
            # if len(form_candidates) > 1:
            #     log.error("Google login form dom has changed; there are %s candidate forms:\n%s",
            #               len(form_candidates), form_candidates)
            #     return False

            form = form_candidates[0]
            form.select("#totpPin")[0]['value'] = kwargs["totp"]

            response = browser.submit(form, 'https://accounts.google.com/signin/challenge/totp/2')

            try:
                response.raise_for_status()
            except requests.HTTPError:
                log.exception("submitting OTP form failed")
                return False

Had the same issue. Just False login status, same messages in logs. Added application password in google and used it instead of mine. worked.


uniform - generating nested divs on select elements, when called more than once

I am trying to show a form using jQuery UI modal dialog (which is usually created once, but is hidden once closed, but not destroyed). The first time I open the dialog form, Uniform does it job and wraps all the select elements without any issues.
The second time when i open the dialog, to show different data, the wrapper div around the "select" element is repeated twice, and this goes on an on (nested wrapper div's). I am currently using the jQuery "replaceWith" api call to remove the "Uniform" generated div's so that my modal dialog shows them properly. This is only a workaround, and makes the whole web experience very slow in terms of response time.
Can someone suggest a simple solution to this issue? I believe this issue is similar to issue #142
Any help will be greatly appreciated.

I have this same problem.


perf: remove XValue unit value combinations

BREAKING CHANGE: removes all XValue classes
The intention is to replace these with validators to check that units are included with values
There are only a few places these are being used:
https://github.com/AllenNeuralDynamics/SPIM-UI-Launch/blob/342d357be6aec3798aefe3b6d6ed4d1397678d57/exaSPIM/exaSPIM-UI-Luanch.py#L5
https://github.com/AllenNeuralDynamics/SPIM-UI-Launch/blob/342d357be6aec3798aefe3b6d6ed4d1397678d57/iSPIM/iSPIM-UI-Launch.py#L5
and a few places in aind-data-schema although confusingly it's used in the tests but not in the actual schema?

Changed this to merge into a "v1" branch instead of main


Add support for responding with multiple responses for push

http/2 introduced a mechanism for sending back multiple responses for a single request (PUSH frames). This is often used as an optimization for requesting static files in the browser, so for when example a user requests index.html we send back bundle.css and bundle.wasm as well because we know that those will be requested next.
This mechanism is not limited to http/2, http/3 makes use of it as well. And as such it makes sense to introduce a mechanism for sending back multiple responses into http-types since it's part of the HTTP semantics now.
API example
In https://github.com/http-rs/http-types/issues/126#issuecomment-629650395 I authored an example for multipart responses, which are "multiple bodies sent in response to a single request". HTTP push is "multiple responses sent in response to a single request". So I was thinking we could make it a similar-ish API, but multiplex on the response level rather than on the request level. This is a sketch
let mut res = Response::new(200);
res.set_body(Body::from_file("assets/index.html").await?);

// provide a way to do feature detection. Knowing the http version probably works well enough.
if req.version() >= http_types::Version::2_0 {
    // we should send over fully-qualified HTTP responses.
    let mut css = Response::new(200);
    css.set_body(Body::from_file("assets/bundle.css").await?);
    // in order to push we need to share which URL this resource would live at.
    res.push("bundle.css", css);

    let mut wasm = Response::new(200);
    wasm.set_body(Body::from_file("assets/bundle.wasm").await?);
    res.push("bundle.wasm", wasm);
}

// provide some way to iterate over the pushed responses
for pushed in res.pushed() {
    println!("{}", pushed.url());
}

alternatives
Something that's been brought up before is to use a channel based approach provided by request instead. I think the right mental model for push is: "multiple responses to a single request" and we should create an API that matches that. And actually sending back multiple responses in response to a request is the clearest way to model that.

Does it make more sense to push multiple Responses to a connection, rather than append to one main response? (I'm not really familiar with how http/2/3 deal with status codes and headers.)
If so, we probably want to think more about Tide's route/middleware return type.

Does it make more sense to push multiple Responses to a connection, rather than append to one main response?

The HTTP/2 impl should take care of this. The http-types crate provides a way to interface with this, but it's up to the backing impl to correctly handle this. This is similar to upgrade headers and trailers -- they too deal with sending bytes over the wire but all http-types does is provide an interface to build that out.

HTTP/2 PUSH appears to be effectively have been phased out with Chrome having announced they're going to deprecate support for this. It doesn't seem particularly pressing to support this at the time of writing.


Crash on Startup: LoaderExceptionModCrash

I have been using the mod along with Wyntils, Optifine, and BetterFPS for about a week with no problems and suddenly today Wynnforge seems to crash my client on startup, regardless of which version I use. I've tried having only Wynnforge in my mods folder, deleting resourcepacks, Wynnforge.dat, and even using a brand new forge install.
Here is my crash report. If you need any clarification, please ask and I'll gladly answer. I hope you're still working on this mod because it's made my Wynncraft experience so, so much better.
crash-2020-04-06_17.52.20-client.txt

Adding my log to this issue, was working fine 2 days ago and all of a sudden not working for everyone
https://paste.dimdev.org/bidupabose.mccrash


Mark trainings as completed when there are none

On the Manage Volunteers page, if there are no required training for the event then the icon is still an “x” but it should be a check since they have technically completed all trainings. Same on the User Profile where we can see all of the programs and whether they have completed the trainings. We don't want it to seem like they haven't completed some required trainings when there aren't any.

 Mark trainings as completed when there are none


Why DBeaver 24.2.1 made this change? Azure SQL: “show all databases” option was removed

Hi,
In DBeaver 24.2.1 it can be seen that Azure SQL: “show all databases” option was removed. My question is why? We need that option to be able to see schemas in our Azure Synapse, and I believe some other people can have the same problem. We had to downgrade all our software because of that. We have DBeaver Enterprise licenses.
Thanks in advance,
Unai

Thank you for the report!
We'll try to rework this feature in the near future.


Slice oversized key to 32 bytes and Use AES-256-GCM instead of CBC

This should close #20 .
Due to my markdown formatter, the README format has also changed. Feel free to ask me to discard these changes if you wish.

Sorry, there's something wong with the commits. Closing and recreating this PR.


Created stack implementation in python for issue #208

I have added stack implementation in data structures/stack/python folder .

#222


Timer in the app when entering carbs (Feature request)

Hi all,
I have been thinking recently. Can we get timer in bolus menu ? I need 20 minutes pre-bolus so i always need to open timer on my phone and enter it. Is there somehow possibility that we integrate it and make custom buttons for time (My case 15-20-35 min) so when we add bolus it activats?
Thank you

I guess.. But were exactly?

Thank you for reply.
In bolus menu, somewhere at the bottom? as we give bolus, it turns on or we turn it on.

He is asking for a Notification/Alarm/Reminder to start to eat
My daughter often does that as well.  Pre-bolus for a snack, then sets a Timer to go off in 15 mins to remind her to eat

OK ! The best is to use shortcuts to do this :

But that doesnt give you a pre-bolus does it ?  I just see carbs being entered

You can replace with a temp basal (pre-bolus) function

of course you can, however, this scenario doesnt work out.
in iAPS:
Pre-bolus, by entering carbs
Get a calculated bolus
Bolus
Set a timer to eat
I assume the shortcut cannot get a recommended bolus based on a carb entry ?

Yes but use the smb algo doing the job to inject the Bolus in the next minutes.. but if you use the recommended bolus not possible currently.


Database Seeding causes "duplicate key" error

I am using Laravel 4.2, with a Postgres 9.3 database.  I have a model Category created with the following migration:
public function up()
{
    Schema::create('categories', function(Blueprint $table)
    {
        $table->increments('id');
        $table->string('name', 400);
        $table->text('alternate_toc')->nullable();
        $table->text('banner')->nullable();
        $table->integer('parent_id')->unsigned();
        $table->integer('display_order');
        $table->string('slug', 100);
        $table->boolean('is_active');

        $table->timestamps();
        $table->softDeletes();
    });
}

I also have a seed file that inserts 280 categories into the database.  After I run the seed command, I am unable to insert more categories.  Instead, I get an error like this:
PGError: ERROR: duplicate key violates unique constraint "categories_pkey" ...

Examining the database, I see that the insertions done by the seeder have not incremented the categories_id_seq.
If I run select setval('categories_id_seq', (select max(id) + 1 from categories));, this fixes the problem.
I know that I can add execution of this SQL query to my seed file, but I would have thought that fixing the series should be built into the seeding process.
Thoughts?

Just had same problem in Laravel 5.1 and PostgreSQL 9.4. Solved it manually using a raw query:
$pdo = DB::getPdo();
$pdo->beginTransaction();
$statement = $pdo->prepare("SELECT setval('id', ?, false);");
$statement->execute([$next]);
$pdo->commit();

Still same error in 5.6 with psql (PostgreSQL) 10.5 (Debian 10.5-1.pgdg90+1) Auto increment value not setting after seeding with php artisan migrate:fresh --seed.

$pdo = DB::getPdo();
$pdo->beginTransaction();
$statement = $pdo->prepare("SELECT setval('table_id_seq', (SELECT MAX(id) FROM table)+1)");
$statement->execute();
$pdo->commit();


Annotate more test files with type hints.

Please fix the linting issues. Also, please run the linting script before submitting a pull request.


Keychain Accessibility

Heimdall doesn't currently have a way to specify accessibility of the key pair entries in the keychain. Unfortunately this means that apps which require access to the keypairs in the background currently won't work correctly while the device is locked (due to the default accessibility of kSecAttrAccessibleWhenUnlocked).
I think it would be helpful to be able to specify a value for kSecAttrAccessible that would override the default. I might have a chance to make a PR soonish, but would appreciate help.

I can't promise a deadline for this, but when I get a chance I can look into this. A PR would be most welcome (preferably a parameter for initialisation that carries a default value that matches the current behaviour).

I also look forward to the background access function


Add animation

Add animations to the navbar and home page + tools page using Framer motion.
Read more about - https://www.framer.com/docs/

@Cyber-Machine wanna work on this ?

@Anurag30112003 can you assign me this?

ok , assigned @prajjwalkapoor


AddStandardOrder not working?

Hi,
Thank-you for your project It's really helpful.
I suceeded to use a lot of your methods...Except one
`        private static async Task TradeAsync(KrakenClient client)
{
        string pair = "LTCEUR";
        string type = "sell";
        string orderType = "limit";
        decimal volume = 0.0001m;
        decimal price = 80;

        try
        {
            var response = await client.AddStandardOrder(pair, type, orderType, volume, price);

            if (response.Errors.Count() == 0)
            {
                Console.Write("OK:" + response.Result.Description.Order);
            }
            else
            {
                Console.Write("Error:" + response.Errors.First().Value);
            }
        }
        catch (HttpRequestException httpExc)
        {
            Console.WriteLine("httpExc:" + httpExc.Message);
        }
        catch (KrakenException kExc)
        {
            Console.WriteLine("kExc:" + kExc.Message);
        }
        catch (Exception exc)
        {
            Console.WriteLine("Exc:" + exc.Message);
        }
    }`

I know there's a lot of downtime with kraken but here I only get this message:

There was a problem with a response from Kraken

Any idea what could be the problem?
Thank you

OK sorry I haven't seen the Exception.Errors properties...

Well now I have another problem :(

EService:Unavailable
If you look on the kraken doc: It says that It may occurs when API response time si above 2.000ms.
Well It looks like It's always > 2000ms

What I don't understand is why do we get this error only with the AddStandardOrder? Other methods seems to work.
Any piece of advice to make sure an order is processed?

@flaghub Closing this for now as I'm fairly certain it's an issue on Kraken's side. Feel free to leave a comment if you still think it's an issue with this library and I'll reopen the issue.


RFE: manage blacklist.conf

It'd be handy if blacklist.conf only blacklisted modules you give. If you give none, no modules are in there.

So you'd want a purge option, right?

Yes :+1:

OK, PR welcome. Maybe using types/providers would help.


Suggest to add explanation regarding the platform metrics.

We observed that windows node pool is not supported in the platform metrics. The windows nodes cannot be filtered when choosing 'working set memory' or 'cpu' related metrics in the platform data when configuring the alerts rules.

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

ID: 7a58791a-92e9-fa17-b054-cf8e551f16a2
Version Independent ID: 53142733-2a7b-f971-1b8e-877dbcc414d2
Content: Monitor Azure Kubernetes Service (AKS) with Azure Monitor - Azure Kubernetes Service
Content Source: articles/aks/monitor-aks.md
Service: azure-monitor
GitHub Login: @bwren
Microsoft Alias: bwren

@JennyLJY
Thanks for your feedback! We will investigate and update as appropriate.

#reassign:@rboucher

#reassign:@bwren

Apologies for the horribly late reply on this Certain metrics are not available for windows nodes and nodepools as explained in Differences between Windows and Linux clusters. We'll look at potentially referencing that article from the monitoring AKS article. Thanks for the question!
#please-close


Suggestion: Migration from react-native-router-flux to react-navigation

react-native-router-flux started to have many unsolved issues, so I suggest migration to react-navigation.
Although router-flux is much simpler to use, react-navigation is up-to-date and will help us avoid the transitive dependency on it.
What do you think?

good idea


make:adminlte not working in new projects

I make a new fresh Laravel Project y install via composer the package. But the command

php artisan make:adminlte

It's not working

See my pull request for the fix: https://github.com/jeroennoten/Laravel-AdminLTE/pull/106

Thnx @simik-ru .. I worked..

Solved in v1.20.0!


[WebToolsE2E][Aspire] Running aspire project with Aspire.Confluent.Kafka component and stopping container, the icon of 'Exited' under State for cache and messaging are different.

INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install VS 17.12 PR VAL
Install Aspire 9.0 GA
Apply Aspire 9.0 latest feed
Install Docker/Podman Desktop

REPRO STEPS

New Project > Aspire Starter App > check 'Use Redis for caching (requires Docker)' > .NET 9.0 > Aspire version 9.0 > Create
Right-click xxx.web project > Add >  Aspire package > Aspire.Confluent.Kafka > add a component and try to use it in the code
Open appsettings.json > make changes related to Aspire component, verify you get intellisense for Aspire-related changes

{
  "Aspire": {
    "Confluent": {
      "Kafka": {
        "Producer": {
          "DisableHealthChecks": false,
          "Config": {
            "Acks": "All"
          }
        }
      }
    }
  }
}

In the Program.cs of web project, type following codes:
builder.AddKafkaProducer<string, string>("myConnection");
Open appsettings.json of xxx.AppHost project, add ConnectionStrings configuration section:

{
  "ConnectionStrings": {
    "myConnection": "broker:9092"
  }
}

In your AppHost project, install the Aspire.Hosting.Kafka
Open Program.cs of xxx.AppHost project, add following codes:

var messaging = builder.AddKafka("messaging");

var myService = builder.AddProject<Projects.MyService>()
                       .WithReference(messaging);

Build and F5
Navigate to Aspire Dashboard and stop the container project for messaging and cache.

ACTUAL
The icons of 'Exited' for cache and messaging are different.

NOTE

Tooltip for cache shows: Container is no longer running
Tooltip for messaging shows: Container exited
This issue also reproduce in .NET8.0 project

EXPECTED
The icons for 'Exited' under State when stopping container project should be the same.

I believe this is by design because they exited with different errors, which is illustrated by the different tooltip messages.

As @balachir shared above, the reason why there are two different icons is because two different exit codes. Keeping this one open as Feature to consider displaying the fact that one had errors in the dashboard (without having to show the tooltip)


Add DataPlane API Support

DRAFT
This PR uses a SDK generated from Universe. Pending to regenerate the code after changes to universe are merged. Otherwise, it is good to review.
Changes
Add DataPlane API Support
Tests
Manual test:
input := []string{}
input = append(input, "Hello, I'm a language model,")
_, err := w.ServingEndpointsDataPlane.Query(context.Background(), serving.QueryEndpointInput{
	Name:   <...>,
	Inputs: input,
})
require.NoError(t, err)

[ ] make test passing
[ ] make fmt applied
[ ] relevant integration tests applied

Codecov Report
Attention: Patch coverage is 30.52632% with 66 lines in your changes missing coverage. Please review.

Project coverage is 17.77%. Comparing base (d098b1a) to head (d3547b7).

Files
Patch %
Lines

service/catalog/model.go
0.00%
15 Missing :warning:

openapi/code/method.go
0.00%
14 Missing :warning:

openapi/code/service.go
6.66%
14 Missing :warning:

httpclient/request.go
0.00%
8 Missing :warning:

httpclient/oauth_token.go
0.00%
6 Missing :warning:

experimental/mocks/mock_workspace_client.go
16.66%
5 Missing :warning:

client/client.go
0.00%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##            main     #936       +/-   ##
==========================================
+ Coverage   7.04%   17.77%   +10.73%
==========================================
  Files        282      131      -151
  Lines      65329    21423    -43906
==========================================
- Hits        4602     3809      -793
+ Misses     60418    17342    -43076
+ Partials     309      272       -37

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


Add load/save support

DockerClient.load allows loading the tarball of an image as a stream. An
image name must be passed in, and the load implementation internally
tags the just loaded image with the provided name. The load
implementation is passed on Docker API's pull, which takes a fromSrc
parameter, indicating it to treat the request's body as the .tar payload
of the image.

DockerClient.save allows pulling the .tar stream of an image (and saving
it locally or whatever).

@mattnworb @yduchesne

:+1:

That's great guys. Thanks a lot for the help and accepting my contribution.

Thank you @yduchesne for the contribution!


[CANNOLI-278] Bump ADAM dependency version to 0.33.0

Fixes #278

Coverage remained the same at 0.0% when pulling 86b17d96502c3446895243b4cdeb42d56ce64ff0 on heuermh:adam-0.33.0 into e647572b94722cce31d0056f5829ab0b63b201b0 on bigdatagenomics:master.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/cannoli-prb/259/Build result: FAILURE[...truncated 724 B...][WARNING] Reference path does not exist: /home/jenkins/gitcaches/adam.referenceFetching upstream changes from https://github.com/bigdatagenomics/cannoli.git > git --version # timeout=30 > git --version # 'git version 2.25.1' > git fetch --tags --force --progress -- https://github.com/bigdatagenomics/cannoli.git +refs/heads/:refs/remotes/origin/ # timeout=15 > git config remote.origin.url https://github.com/bigdatagenomics/cannoli.git # timeout=30 > git config --add remote.origin.fetch +refs/heads/:refs/remotes/origin/ # timeout=30 > git config remote.origin.url https://github.com/bigdatagenomics/cannoli.git # timeout=30Fetching upstream changes from https://github.com/bigdatagenomics/cannoli.git > git fetch --tags --force --progress -- https://github.com/bigdatagenomics/cannoli.git +refs/pull/:refs/remotes/origin/pr/ # timeout=15 > git rev-parse 86b17d96502c3446895243b4cdeb42d56ce64ff0^{commit} # timeout=30Checking out Revision 86b17d96502c3446895243b4cdeb42d56ce64ff0 (detached) > git config core.sparsecheckout # timeout=30 > git checkout -f 86b17d96502c3446895243b4cdeb42d56ce64ff0 # timeout=30Commit message: "Bump ADAM dependency version to 0.33.0." > git rev-list --no-walk e70bc599654ae350a5002640b6809b7f81ecd817 # timeout=30Triggering cannoli-prb » 2.7.5,2.12,3.0.1,ubuntuTriggering cannoli-prb » 2.7.5,2.12,2.4.7,ubuntuTriggering cannoli-prb » 2.7.5,2.11,2.4.7,ubuntucannoli-prb » 2.7.5,2.12,3.0.1,ubuntu completed with result SUCCESScannoli-prb » 2.7.5,2.12,2.4.7,ubuntu completed with result FAILUREcannoli-prb » 2.7.5,2.11,2.4.7,ubuntu completed with result FAILURENotifying endpoint with url 'https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'Setting status of 86b17d96502c3446895243b4cdeb42d56ce64ff0 to FAILURE with url https://amplab.cs.berkeley.edu/jenkins/job/cannoli-prb/259/ and message: 'Build finished. '

Jenkins, retest this please

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/cannoli-prb/260/Build result: FAILURE[...truncated 720 B...][WARNING] Reference path does not exist: /home/jenkins/gitcaches/adam.referenceFetching upstream changes from https://github.com/bigdatagenomics/cannoli.git > git --version # timeout=30 > git --version # 'git version 2.25.1' > git fetch --tags --force --progress -- https://github.com/bigdatagenomics/cannoli.git +refs/heads/:refs/remotes/origin/ # timeout=15 > git config remote.origin.url https://github.com/bigdatagenomics/cannoli.git # timeout=30 > git config --add remote.origin.fetch +refs/heads/:refs/remotes/origin/ # timeout=30 > git config remote.origin.url https://github.com/bigdatagenomics/cannoli.git # timeout=30Fetching upstream changes from https://github.com/bigdatagenomics/cannoli.git > git fetch --tags --force --progress -- https://github.com/bigdatagenomics/cannoli.git +refs/pull/:refs/remotes/origin/pr/ # timeout=15 > git rev-parse 86b17d96502c3446895243b4cdeb42d56ce64ff0^{commit} # timeout=30Checking out Revision 86b17d96502c3446895243b4cdeb42d56ce64ff0 (detached) > git config core.sparsecheckout # timeout=30 > git checkout -f 86b17d96502c3446895243b4cdeb42d56ce64ff0 # timeout=30Commit message: "Bump ADAM dependency version to 0.33.0." > git rev-list --no-walk 86b17d96502c3446895243b4cdeb42d56ce64ff0 # timeout=30Triggering cannoli-prb » 2.7.5,2.12,3.0.1,ubuntuTriggering cannoli-prb » 2.7.5,2.12,2.4.7,ubuntuTriggering cannoli-prb » 2.7.5,2.11,2.4.7,ubuntucannoli-prb » 2.7.5,2.12,3.0.1,ubuntu completed with result FAILUREcannoli-prb » 2.7.5,2.12,2.4.7,ubuntu completed with result FAILUREcannoli-prb » 2.7.5,2.11,2.4.7,ubuntu completed with result FAILURENotifying endpoint with url 'https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'Setting status of 86b17d96502c3446895243b4cdeb42d56ce64ff0 to FAILURE with url https://amplab.cs.berkeley.edu/jenkins/job/cannoli-prb/260/ and message: 'Build finished. '

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/cannoli-prb/261/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/cannoli-prb/262/Build result: FAILURE[...truncated 789 B...]Fetching upstream changes from https://github.com/bigdatagenomics/cannoli.git > git --version # timeout=30 > git --version # 'git version 2.25.1' > git fetch --tags --force --progress -- https://github.com/bigdatagenomics/cannoli.git +refs/heads/:refs/remotes/origin/ # timeout=15 > git config remote.origin.url https://github.com/bigdatagenomics/cannoli.git # timeout=30 > git config --add remote.origin.fetch +refs/heads/:refs/remotes/origin/ # timeout=30 > git config remote.origin.url https://github.com/bigdatagenomics/cannoli.git # timeout=30Fetching upstream changes from https://github.com/bigdatagenomics/cannoli.git > git fetch --tags --force --progress -- https://github.com/bigdatagenomics/cannoli.git +refs/pull/:refs/remotes/origin/pr/ # timeout=15 > git rev-parse 4e686043ef06751a8181f08a0d03510b621c36ab^{commit} # timeout=30Checking out Revision 4e686043ef06751a8181f08a0d03510b621c36ab (detached) > git config core.sparsecheckout # timeout=30 > git checkout -f 4e686043ef06751a8181f08a0d03510b621c36ab # timeout=30Commit message: "Bump ADAM dependency version to 0.33.0." > git rev-list --no-walk 7591fd88cd89bb4aa39bcdddf79ec6981760087c # timeout=30First time build. Skipping changelog.Triggering cannoli-prb » 2.7.5,2.12,3.0.1,ubuntuTriggering cannoli-prb » 2.7.5,2.12,2.4.7,ubuntuTriggering cannoli-prb » 2.7.5,2.11,2.4.7,ubuntucannoli-prb » 2.7.5,2.12,3.0.1,ubuntu completed with result SUCCESScannoli-prb » 2.7.5,2.12,2.4.7,ubuntu completed with result FAILUREcannoli-prb » 2.7.5,2.11,2.4.7,ubuntu completed with result SUCCESSNotifying endpoint with url 'https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'Setting status of 4e686043ef06751a8181f08a0d03510b621c36ab to FAILURE with url https://amplab.cs.berkeley.edu/jenkins/job/cannoli-prb/262/ and message: 'Build finished. '

Jenkins, retest this please

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/cannoli-prb/263/


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

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

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

Questions ?
Please refer to the Backport tool documentation


Opinion: FilmCore has role attributes

In my use case, I want to write the beginning of a sentence and have chaGPT write the continuation sentence.
e.g.

Just idea of  how to use.
f = FilmCore(
    role="assistant"
    prompt="""
Once upon a time there was an old man and an old woman. they have one child but
"""
    system_prompt="""
your role is professional novel writer.

Please write the beginning of a sentence and have chatgpt write the continuation sentencein a modern way.
"""
).run()

----原文---
UseCaseとして role: assistantとして文の始まりを与え、chatGPTにはその続きを書かせてみたい場合を想定しています。

Thank you for proposing this use-case.
In this case you can write as follow.
f = FilmCore(prompt="""
Once upon a time there was an old man and an old woman. they have one child but
""",
    system_prompt="""
your role is professional novel writer.

Please write the beginning of a sentence and have chatgpt write the continuation sentencein a modern way.
"""
).run()

This code will output the expected sentences shown in your example.
We intentionally omitted "role" parameter because it is complicated for the users. We try hiding these chat specific features to keep SpiralFilm a tool to focus on "question" and "answer" machine.

One more point before closing. As far as we observed, this way of asking doesn't harm the performance and gives expected continuing sentences.
Not sure how GPT-3.5-turbo and GPT-4 are trained, but probably they're trained such that they have the same capability as previous davinci completion to keep some compatibility.


GET /me

returns info about currently logged in user.
this end point might help

This may be useful


Save connection credentials locally

A convenience feature. Assumedly localStorage. But how to store them securely?

keytar seems to work nicely, using the OS's password manager for osx/linux/windows. Still have to store the other details in localStorage (username, hostname, etc.) but that should be okay.


Handling files with optional header lines?

What is the best way to write the code to handle the situation where you get a file with the correct format but the header line is missing?
We get the same data from multiple sources and some sources provide a header and some do not.

Hi,
use the
void  set_header(some_string_type col_name1, some_string_type col_name2, ...);
function instead of parse_header. The documentation states:

The |set_header| method does /not/ read any input. Use it if the file
does not have any header. Obviously it is impossible to rearrange
columns or check for their availability when using it. The order in
the file and in the program must match when using |set_header|.

Essentially you give the first column the first parameter as name, the
second column the second parameter, etc. The column names are only used
to format error messages. The number of parameters is used to check that
the number of column matches.


significant lag when navigating between pages and tabs on mobile

Describe the bug
I've found that clicking on a link on mobile often requires multiple clicks. It's as if the UI is blocked on rendering the content of the current page before it fully registered that a link has been clicked.
Notice in this video how the unselect tab label becomes bold but still requires an additional click or two before the UI navigations to the selected tab.
https://github.com/near/near-discovery/assets/1915116/f3a86366-14dc-42f5-953c-cdaf11b0b5cf

@charleslavon hi there, i would like fix this issue,
Please assign this to me, i will start workiny on this as soon as i get assignment
Thanks


IPs read in from File Targets do not work due to newlines not being stripped

Describe the bug
IPs read in from File Targets do not work due to newlines not being stripped.
To Reproduce
Steps to reproduce the behavior:

Create a text file (e.g., hosts.in) with one IP address per line
cme --verbose smb hosts.in
Note that IP addresses are not recognized and appear to have newlines included at the end of each

Expected behavior
Each line should have whitespace stripped and be parsed properly as an IP address.
Screenshots
N/A
Crackmapexec info

OS: Kali Rolling 2020.1
Version of CME: 5.0.1.dev1+20200502.13819.17d06f9
Installed from latest git repo with my version commit included

Additional context
N/A

Hello, duplicate of #352
Code in the dev branch will be merged end of the week


zk not showing in IntelliJ 14.1.5

I downgraded to IntelliJ release 14.1.5 which I believe meets "IntelliJ platform 141.1532 or greater". Plugin is active but does not show up as a Maven archtype. Additional information:
JAVA_HOME points to jdk1.7.0_75
Is there anything else i can check? I am assuming the plugin is including ZK library?

hmm - must be my environment because version 14.04 does not show zk as a maven archtype. I wonder what in my environment is prohibiting your plugin...frustrating

have you checked with this one - https://github.com/jumperchen/zkidea/issues/10#issuecomment-148628901

I appreciate your follow-up. I run on a Windows 7 PC and IntelliJ does not deploy anything like it does in OSX. Any ideas for us Win 7 users?
From: Jumper Chen<EMAIL_ADDRESS>Sent: Tuesday, December 01, 2015 7:13 PM
To: jumperchen/zkidea<EMAIL_ADDRESS>Cc: IndisRepair<EMAIL_ADDRESS>Subject: Re: [zkidea] zk not showing in IntelliJ 14.1.5 (#12)
have you checked with this one - #10 (comment) https://github.com/jumperchen/zkidea/issues/10#issuecomment-148628901
—
Reply to this email directly or view it on GitHub https://github.com/jumperchen/zkidea/issues/12#issuecomment-161166465 .

I tested with a new Windows 7 VM and setup a latest JDK 8, IntelliJ 15.0.1, and ZK plugin 0.1.6 version, and they all work as expected.

I am running IntelliJ 15.0.1 Ultimate version. Here is the error I see as it attempts to load ZK 0.1.6:
2015-12-09 19:34:51,360 [  24111]   INFO - aven.ZKMavenArchetypesProvider - C:\Users\Jeff.IntelliJIdea15\system\plugins\zkidea\classes\org\zkoss\zkidea\lang\resources\archetype-catalog.xml is not found!

according to this source - https://github.com/jumperchen/zkidea/blob/c8cfe951a998037050f635144b8c486c0fe75c35/src/org/zkoss/zkidea/project/ZKProjectsManager.java
In the updateMavenArchetype() method, it will update or copy the archetype-catalog.xml automatically when you open a project in a IntelliJ IDE. Not quite sure what's wrong in your environment.
A workaround is to download a file from this line - http://mavensync.zkoss.org/maven2/archetype-catalog.xml
and put it into the "file not found" path.


⚠ Bofrost.es has degraded performance

In ffcd21f, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1619 ms

Resolved: Bofrost.es performance has improved in 0637355 after 8 minutes.


[T3A2][T09-A1] Dana Baghabrah

Ready for review

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


How to Disable Gesture Effects (e.g., Thumbs-Up, Peace Sign)?

Plugin Version or Commit ID
v0.15.0
Unity Version
2022.3.51f1
Your Host OS
Mac OS Sequoia 15.1
Target Platform
UnityEditor
Description
Where can I disable the effects triggered when making a thumbs-up or peace sign gesture? Additionally, could you let me know which file handles this processing?
Code to Reproduce the issue
No response
Additional Context
No response

Where can I disable the effects triggered when making a thumbs-up or peace sign gesture?

I have no idea about the functionality you're mentioning. Which scene or API are you referring to?

I was mistaken. It appears that the following functionality was working on its own. My apologies.
https://support.apple.com/en-us/105117


noble.stop() function

Lousy first try at implementing a noble.stop() function, as requested in #299
Probably doesn't even work correctly; I just fiddled around until the test program:
noble = require('noble');
noble.stop()

seemed to somehow work. Might still be useful as a kind of starting point for a proper stop() function.

Thanks for the hint @mbifulco , I tried to fix the bug you mentioned.
Not quite sure if I did it correctly though, all this GitHub stuff is still new and confusing to me.

Totally understand @oyooyo - it can be daunting.  I was able to tell what was wrong before by clicking "details" next to the travis-ci automation that's running on PRs in this repo - and then navigating into an individual build to see what error was being reported.

It looks like your builds are passing now, which puts this PR one step closer to being done with.  I'll leave it to @sandeepmistry to comment on whether or not the modification to the code is the best way to stop noble from doing its thing.

Is this pull request still progressing? Anything else that needs doing to get it merged?

Please forward change to downstream fork:
#923


https on cms

Maybe not a reverse proxy problem but when using https to proxy the cms web, it will not reverse / but will reverse it correctly if you append a querystring parameter. For instance this call works (from nodejs log)
GET /cms/?0 HTTP/1.1 http://localhost:8080/cms/?0 200
but not the following
GET /cms HTTP/1.1 http://localhost:8080/cms 302
which will reply with a redirect to /https,http://localhost/cms/

Thank you, Saimir!
I was able to reproduce the problem as well. I'll fix it asap.
Cheers,
Woonsan

Hi Saimir,
I've fixed the issue. Could you try it again?
Regards,
Woonsan


[GEMXD-39] Add stats for file descriptors

Changes proposed in this pull request
Continuing with previous checkin that added more details thread stats (85a89e14dc9c6ac73c43622d7357982cb74224d2), this adds FD stats on similar lines.

added stats for max system file descriptors and currently open count on machine
(stats for open fds by this process and "ulimit -n" are already present in VMStats)
reuse the RLimit object since creation has some significant overhead which has shown up
in mission control etc
changed some thread stats added previously as longs instead of ints to make it consistent
some renaming to make it cleaner

Patch testing
store precheckin (ongoing)
ReleaseNotes changes
Added to ReleaseNotes. However, the new thread/fd stats should go into VSD docs but those currently don't go into such details and don't have any system/OS stats sections (http://rowstore.docs.snappydata.io/docs/manage_guide/Topics/vsd_overview.html).
Other PRs
NA

Looks ok to me. Some questions asked. specially regarding number of fds open due to this process or is it covered somewhere?

@kneeraj if the replies above are fine, then should I go ahead with merge?

yes.


Implement Rust pipes

This is an initial implementation of pipe workers in Rust.  It consists of three main parts.

Basic PipeSender and PipeReceiver worker pair and an API to create them
A function composition behaviour API to stack different pipe behaviours on top of each other (each for internal and external message types)
Basic resend/ ack and strict ordering enforcement logic

Two unrelated changes

656db3244adc24043170b024aa09f45eca7bcb8e adds a trace-level log line when a worker relay receives a message to make dead-locks easier to spot in logs
b60fb4386c30a5f2fd3a65f0fa731e493deb23f2 fixes the illusive bug where the router would sometimes crash on shutdown. The need for an immediate shutdown was fixed by #2088, but this fixes the issue when calling ctx.stop_now().await

This PR adds a new dependency on dyn-clone btw! From what I can tell it shouldn't impact our no_std implementations but I thought I'd more publicly document it anyway! (ping @antoinevg @jared-s)
Otherwise, if the CI gods be willing, I think this code is ready to be merged

Ran the no_std examples. all good!


Add the runes to find a local IP for OS X

Note this completely removes Linux support, but hey, it fixes OS X.
Or at least it might fix OS X-- I'm not sure because I've not tested it.
Probably best not merge this as-is.
Signed-off-by: David Scott<EMAIL_ADDRESS>

This has been fixed by another commit. And we wouldn't have merged this anyway :-)


bmdplay 3840x2160 HDMI out problem

Hello,
I have Blackmagic Decklink 12G Extreme Card. I can push out HDMI video with 1080p resolution without any problem. However, same video with UHD resolution makes problem. Such like upper 2K frames and lower 2K frames do not place on right areas.
1080p test:
avconv  -re -i test.mp4 -c:v rawvideo -c:a pcm_s16le -ac 2 -f nut -s 1920x1080 - | bmdplay -f pipe:0 -m 10 -C 0 -O 3

2160p test:
avconv  -re -i test.mp4 -c:v rawvideo -c:a pcm_s16le -ac 2 -f nut -s 3840x2160 - | bmdplay -f pipe:0 -m 24 -C 0 -O 3

Test video was HEVC & AAC encodded in YUV 4:2:0 at 50 fps. I tested different framerates on 3840x2160 always same problem.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28647964-bmdplay-3840x2160-hdmi-out-problem?utm_campaign=plugin&utm_content=tracker%2F256764&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F256764&utm_medium=issues&utm_source=github).

Can you please provide me the sample file? bmdplay is really simple: it copies what is gets in. Are you 100% sure the video mode is the correct one for that content?

Yes, video mode 24 means 3840x2160 at 50 fps. So the video is same. Please download and see the test file.

Please download sample file from:
http://we.tl/lB0h1rGbIG

Can you please try the TestPattern program?

Yes i tested it. TestPattern works fine at 3840x2160@50.

I tried adding -pix_fmt uyvy422 and result is same. Output video broken again as given images on 1st post.

After spending a good deal of the day on it I guess we can point our finger to the hardware.
Let see if bmd can help you about it.


List of Markdown display issues in Comments

Bug

[ ] The Preview and actual outcome differs a lot in terms of spacing.
[ ] Bullet-lists within Quotes should not have extra top spacing (other websites don't).
[ ] Ordered-list doesn't seem to work in Preview (style not applied)

To Reproduce

The text used: (click to expand)
Hi there.
* Bullet 1
* Bullet 2

> * bullet 1
> * bullet 2

Hello again.
1. List 1
2. List 2

> 1. bullet 1
> 2. bullet 2

First paragraph. kjldfljs kjfshfkdshfkdsfhkdsjfhds kjds fksf sfhsk fhskfhs dkfs.

Second paragraph. hfkdsfh kjf hsakf sakfjs fkjdsh fksjfs jflkdsjfldsjf  fjlsdfjds.

> Regular quotes
> Regular quotes

Expected behavior
Same as Github's (good spacing, etc.)
Screenshots
Preview:

Actual:

System Configuration
lbry.tv @ 2020-06-17

Close
No longer relevant today.  Final outcome is now the same as the Preview's.  Although the spacing could still be improve, it's a different issue.


article needs update

[Enter feedback here]
Under Section :
Use a persistent volume with private Azure Files storage (private endpoint)
my customer wants this to be a note stating that CSI driver will not create private endpoint and this steps requires a private endpoint to be already existing.

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

ID: fc6adf61-d0eb-f95c-60a7-bd3e1b52b5a5
Version Independent ID: e1a7cc5d-2988-3c62-5169-48eac7c0fc47
Content: Use Container Storage Interface (CSI) drivers for Azure Files on Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/azure-files-csi.md
Service: container-service
GitHub Login: @palma21
Microsoft Alias: zarhoads

@Lovejeetsk
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@palma21
Can you please check and add your comments on this doc update request as applicable.

Hi @Lovejeetsk, thanks for the feedback! This doc states that "If your Azure Files are protected with a private endpoint, you must create your own storage class..." The steps here require you set the server parameter to the FQDN of the private endpoint, thus indicating that you would already have a private endpoint. If you have any other feedback, please share! I will now close this issue. Thanks again!
#please-close


Hyperlink includes a .corp address

Issue in /src/pages/implement/client-side/index.md
Visual Experience Composer includes .corp in the URL.

Fixed in Issue #78


Update TextView module

Issue

A couple of adjustments to the TextView module to make round trip testing easier.
Added CBOR round trip tests for KES & VRF verification keys.
Updated read/write functions for KES keys, VRF keys and operational certificates.
This PR does not result in breaking changes to upstream dependencies.

bors merge


fix: update i18n translations

Description
Update the translations now that #1079 is merged.
Screenshot (if UI-related)
To-Dos

[ ] Successful build pnpm build
[x] Translation keys pnpm i18n:extract
[ ] Database migration (if required)

:tada: This PR is included in version 2.1.0 :tada:
The release is available on:

v2.1.0
GitHub release

Your semantic-release bot :package::rocket:


20kly: init at 1.4

Motivation for this change
The classic "20,000 Light-Years Into Space" game
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)

+1.3M

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build 20kly

No attempt on x86_64-linux (full log)
The following builds were skipped because they don't evaluate on x86_64-linux: 20kly
Partial log (click to expand)
Cannot nix-instantiate `20kly' because:
error: attribute '20kly' in selection path '20kly' not found

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: 20kly
Partial log (click to expand)
Cannot nix-instantiate `20kly' because:
error: attribute '20kly' in selection path '20kly' not found

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: 20kly
Partial log (click to expand)
Cannot nix-instantiate `20kly' because:
error: attribute '20kly' in selection path '20kly' not found

@timokau all done
@mpickering it's _20kly, since it starts with a number

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: _20kly
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

@timokau Nope, since it isn't a library (see the nixpkgs docs, python section)

I just noticed that I pinged the wrong person, I meant @dotlambda.


LIBUSB_ERROR_ACCESS [-3]

Dear @fahhem ,
While running the example code (in readme file) as is, I'm getting every once and then (no specific number of runs) the LIBUSB_ERROR_ACCESS [-3].
I'm aware that using kill adb-server will solve the issue, but it renders the ADB library useless, since I will need to call the kill adb from a different process using regular adb.
Why the ADB library doesn't make sure it leaves the state clean so all consecutive calls will work?
Additional details:

Windows 10 RS2 OS
i used the latest version from main

Here is the full error:
C:\Python27\python.exe C:/Users/baruchl/Documents/_GIT/python-adb/adb/MyTest.py Traceback (most recent call last):   File "C:/Users/baruchl/Documents/_GIT/python-adb/adb/MyTest.py", line 30, in <module>     try_adb_python()   File "C:/Users/baruchl/Documents/_GIT/python-adb/adb/MyTest.py", line 17, in try_adb_python     print devices[0].serial_number   File "C:\Users\baruchl\Documents\_GIT\python-adb\adb\common.py", line 139, in serial_number     return self._device.getSerialNumber()   File "C:\Python27\lib\site-packages\usb1\__init__.py", line 2040, in getSerialNumber 1     self.device_descriptor.iSerialNumber)   File "C:\Python27\lib\site-packages\usb1\__init__.py", line 2017, in _getASCIIStringDescriptor     return self.open().getASCIIStringDescriptor(descriptor)   File "C:\Python27\lib\site-packages\usb1\__init__.py", line 2067, in open     mayRaiseUSBError(libusb1.libusb_open(self.device_p, byref(handle)))   File "C:\Python27\lib\site-packages\usb1\__init__.py", line 133, in mayRaiseUSBError     __raiseUSBError(value)   File "C:\Python27\lib\site-packages\usb1\__init__.py", line 125, in raiseUSBError     raise __STATUS_TO_EXCEPTION_DICT.get(value, __USBError)(value) usb1.USBErrorAccess: LIBUSB_ERROR_ACCESS [-3]

Normally, the adb server is started by an adb command, or by some process like an emulator. If you switch to python-adb entirely (and don't use emulators), then the adb server shouldn't be running anymore.
We can't be in charge of stopping adb servers, because if it's running, it's likely being used by another process, and we don't want to cause errors in other programs that become impossible to diagnose.
If you manage to stop the adb server and stop it from coming back, python-adb can be useful to you, but until then only one process can 'own' a USB device and if the adb server is running it's 'owning' all android USB devices.

So how to stop it from coming back?


mise not skipping tasks

Describe the bug
mise doesn't appear to be respecting the sources and outputs to skip running a task.
To Reproduce
I have this in my mise.toml
[tasks.bundle_id]
sources = ['*.xcodeproj']
outputs = ['.bundle_id']
run = """
#!/usr/bin/env bash
xcodebuild -showBuildSettings | grep PRODUCT_BUNDLE_IDENTIFIER | awk -F ' = ' '{print $2}' > .bundle_id
"""

[tasks.reload]
alias = "default"
depends = ["bundle_id"]
# ...

Expected behavior
The docs seem to suggest that bundle_id should be skipped if .bundle_id is newer than any .xcodeproj file, but it's running every time I run the reload task. Running mise run bundle_id also re-runs the command (though maybe that last bit is intentional?). It's also worth noting that .xcodeproj files are really directories.
mise doctor output
version: 2024.4.0 macos-arm64 (2024-04-02)
activated: yes
shims_on_path: no

build_info:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS
  Built: Tue, 2 Apr 2024 01:55:39 +0000
  Rust Version: rustc 1.77.1 (7cf61ebde 2024-03-27) (Homebrew)
  Profile: release

shell:
  /opt/homebrew/bin/fish
  fish, version 3.7.0

dirs:
  data: ~/.local/share/mise
  config: ~/.config/mise
  cache: ~/Library/Caches/mise
  state: ~/.local/state/mise
  shims: ~/.local/share/mise/shims

config_files:
  ~/.tool-versions
  ~/.config/mise/config.toml
  ~/scratch/xcode/ariadne/.mise.toml

plugins:
  bun        (core)
  deno       (core)
  erlang     (core)
  go         (core)
  java       (core)
  node       (core)
  python     (core)
  ruby       (core)
  usage      https://github.com/jdx/mise-usage.git#fe3888a
  watchexec  https://github.com/nyrst/asdf-watchexec.git#b667a08

toolset:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>   (missing)
<EMAIL_ADDRESS>   (missing)

env_vars:
  MISE_SHELL=fish

settings:
  activate_aggressive = false
  all_compile = false
  always_keep_download = false
  always_keep_install = false
  asdf_compat = false
  cargo_binstall = true
  color = true
  disable_default_shorthands = false
  disable_tools = []
  experimental = true
  go_default_packages_file = "~/.default-go-packages"
  go_download_mirror = "https://dl.google.com/go"
  go_repo = "https://github.com/golang/go"
  go_set_gopath = false
  go_set_goroot = true
  go_skip_checksum = false
  jobs = 4
  legacy_version_file = true
  legacy_version_file_disable_tools = []
  node_compile = false
  not_found_auto_install = true
  paranoid = false
  plugin_autoupdate_last_check_duration = "7d"
  python_compile = false
  python_default_packages_file = "/Users/hofstee/.default-python-packages"
  python_pyenv_repo = "https://github.com/pyenv/pyenv.git"
  raw = false
  trusted_config_paths = []
  quiet = false
  verbose = false
  yes = false
  ci = false
  debug = false
  trace = false
  log_level = "info"
  python_venv_auto_create = false

  [status]
  missing_tools = "if_other_versions_installed"
  show_env = false
  show_tools = false

No problems found

Additional context
I was originally trying to figure out how to get an [env] variable from the mise.toml into the task run command, but since the command takes a couple seconds to run and there's no caching for environment variables (as mentioned in https://github.com/jdx/mise/issues/1261) I tried to make it output to a file instead so I could skip the task when the dependencies weren't modified.

The main problem seems to be that it only works for files.
Not for directories
I can't guess why this decision was made, but I also encountered this problem and the need to rely on the directory modification date, since files can be added/removed in it.
https://github.com/jdx/mise/blob/230897c41210502f69ed5c4270f13d6efc416f89/src/cli/run.rs#L513-L514


feat: add indent and outdent command

Please check if the PR fulfills these requirements

[x] It's the right issue type on the title
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] It does not introduce a breaking change or has a description of the breaking change

Description

Add indent, outdent command for wysiwyg
Remove unused attribute for schema

TODO

[ ] Fix the problem that the test fails when no other command is executed after calling wwe.setSelection in the indent or outdent test

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

@shiren 네 커서 이동의 경우에도 동일해요. 이 이슈는 좀 더 보고 원인 파악되면 공유드릴게요~


Rename Theia Blueprint product

Currently we only have one Blueprint product to showcase Theia. This is about to change with CDT.cloud Blueprint and potentially Theia.cloud Blueprint. Therefore it would be good to give the current product a name, e.g. Theia WebDev Blueprint.
This would also mean that the current page at https://theia-ide.org/docs/blueprint_download needs to be slightly rephrased to talk about the "Theia Blueprint subproject" providing different blueprint products.

@marcdumais-work @JonasHelming @jfaltermeier @thegecko What do you think? Any suggestions for a name?

From my POV, the current products work together nicely:
Theia Blueprint
Theia.cloud Blueprint
CDT.cloud Blueprint
CDT.cloud Blueprint does not have Theia in the name and that is on purpose, because it is targeted at C/C++ adopters.
I also think it is reasonable, that  Theia Blueprint has a focus, e.g. web developers, we can mention this in the description.
I kind of like the fact that there is a "Theia Blueprint" "vanilla".
When we introduce more Blueprints, we can think about adding new names. The most likely one is probably Java. For this, we could either name it:
Theia Blueprint for Java
or
JDT.cloud Blueprint
So to answer the original question: I would not rename it right now, I think there should be one Blueprint as kind of the default.
If the default is not WebDev anymore I would name the WedDev Blueprint: Theia Blueprint for WebDevs

CDT.cloud Blueprint does not have Theia in the name and that is on purpose, because it is targeted at C/C++ adopters. I also think it is reasonable, that Theia Blueprint has a focus, e.g. web developers, we can mention this in the description. I kind of like the fact that there is a "Theia Blueprint" "vanilla". When we introduce more Blueprints, we can think about adding new names. The most likely one is probably Java. For this, we could either name it: Theia Blueprint for Java or JDT.cloud Blueprint

I would argue against such fragmentation. Can we, wherever possible, just have the vanilla Blueprint optimized for as many use cases as possible and then provide documentation (e.g. what extensions to install, how to build, ...) for specific use cases?
The advantages of this would be at least 2-fold:

lesser maintenance overhead
shared branding and messaging around Theia Blueprint for a consistent end-user focus

We just discussed this and Brian, Marc, Thomas and me would prefer the current state:
Theia Blueprint
Theia.cloud Blueprint
CDT.cloud Blueprint

OK, makes sense to me!


Counters Displayed as Negative Values

Hello,
I have a similar issue but in my case, it is showing some negative values for active sessions and wondering how could this be:

Thanks,
Ver

Hello Ver,
This is the new issue to address you comment in issue #170.
Negative values for active sessions is completely normal. The reason that we see negative values in the worksheet is because we use a table calc to calculate the change in number of active sessions between time intervals. In the case of that dashboard, the time interval is 5 minutes by default. This means that for every new session we see in the interval we will get a +1, and or every reaped session we will get a -1. (Ex: If we have 3 new sessions and 1 reaped session in an interval we'll see a +2 for the interval)
Overall, we do this in order to better display when we create and reap active sessions.
If there are any other questions let me know.
-Dan

Hello Ver,
I am going to close this due to thread inactivity. Please let me know if you have any other questions.
Thanks,
Dan


Enable proxying

The following changes are required to allow the Dev Server to be proxied from the same ngrok tunnel as the app:

[ ]  move Dev Console under /extensions path
[ ] expose Dev Console url so that consumers can connect to it. Currently Web is connecting to it using the root url which will no longer work


Fix # 273

squash multiple contract calls
created a new function infixed-weighted-pool

Please update your branch to the latest dev and go through all pool contracts to apply the same squash to where applicable. For example:
https://github.com/alexgo-io/alex-v1/blob/aa4b57cac398ac558f24a4d8c210c9aed948cd73/clarity/contracts/pool/collateral-rebalancing-pool.clar#L443-L445
https://github.com/alexgo-io/alex-v1/blob/aa4b57cac398ac558f24a4d8c210c9aed948cd73/clarity/contracts/pool/collateral-rebalancing-pool.clar#L483-L485
https://github.com/alexgo-io/alex-v1/blob/aa4b57cac398ac558f24a4d8c210c9aed948cd73/clarity/contracts/pool/collateral-rebalancing-pool.clar#L494-L497
https://github.com/alexgo-io/alex-v1/blob/aa4b57cac398ac558f24a4d8c210c9aed948cd73/clarity/contracts/pool/collateral-rebalancing-pool.clar#L747-L750

Hi @noumanahmads , I resolved the conflicts, but would like to wait for https://github.com/alexgo-io/alex-v1/pull/319 to merge before merging this PR.


PR: Add license and readme to manifest

Fixes https://github.com/sixty-north/python-transducers/issues/3

Thanks!


[WIP] Service View: Wrap Sub Cateogry Cards

This PR removes hard coded rows and columns which had existed in the catalog service view/list.  CSS has been updated to allow Sub Category cards to wrap.  Positioning of the expansion card/row is handled by the controller and adjusts on browser resize.

Fine tuning of positioning calculations still needs to be done.  Follow up CSS adjustments will be done by @cshinn.   Works in mobile screen size, but could use some tweeks.

I asked Serena about that when the hifi mocks came out, she said it should wrap like we've been discussing.  The hifi mocks do not show that however.

@rhamilto Are you working on this as well or leaving this and working the rest of the landing page layouts?

I asked Serena about that when the hifi mocks came out, she said it should wrap like we've been discussing.

Thanks @dtaylor113. I'm still unclear if the expanded section should have a max height and if the first option is expanded by default. I'm worried the user might completely overlook anything after the first row of tiles if it's below the fold.

@spadgett , me too!   Per Serena's request I made the max-height of the expansion row to accommodate three rows of catalog items.  I'm concerned that pushes 2nd row of sub-cat-cards off screen and maybe missed by user.

I know it's not as stylish but I'd almost want to have the sub-cat-cards just wrap as default/expected -no gap between rows.   The catalog item's area is always underneath all sub-categories, and the active card has a different 'selected' border.  This way we could avoid all these crazy calculations and users would always see all the sub-category tiles.

@dtaylor113, mind rebasing this branch on master so that it incorporates @jeff-phillips-18's navbar changes?  @cshinn and I are working from this branch since it contains the latest cards bits.

Ok, rebased to latest.

Hey @dtaylor113 I had a change from before that was able to do something similar with just HTML and CSS. Can we take a look at that as well together? The one limitation is that it set a fixed height on the expanded content.
I'd need to rebase on my branch on the latest master.

Hey @dtaylor113 I had a change from before that was able to do something similar with just HTML and CSS. Can we take a look at that as well together?

Sure, that would be great

Incorporated @spadgett changes (thanks @spadgett !).  Hid 'Other' sub-category tile when viewing 'Other' category.

Thanks @dtaylor113 !

Code LGTM, but travis is complaining