Updating dalle

safeguardung against no prompt
added cooldown
can now regenerate and variate an image

the view class doesn't work, needs fixes before merging

how does it not work?

how does it not work?

This is the error I get when trying to call the !dalle command. Should be able to repro easily
apollo-apollo-1  | Traceback (most recent call last):
apollo-apollo-1  |   File "/app/.venv/lib/python3.10/site-packages/discord/client.py", line 441, in _run_event
apollo-apollo-1  |     await coro(*args, **kwargs)
apollo-apollo-1  |   File "/app/.venv/lib/python3.10/site-packages/discord/ext/commands/bot.py", line 1395, in on_message
apollo-apollo-1  |     await self.process_commands(message)
apollo-apollo-1  |   File "/app/.venv/lib/python3.10/site-packages/discord/ext/commands/bot.py", line 1392, in process_commands
apollo-apollo-1  |     await self.invoke(ctx)  # type: ignore
apollo-apollo-1  |   File "/app/.venv/lib/python3.10/site-packages/discord/ext/commands/bot.py", line 1350, in invoke
apollo-apollo-1  |     await ctx.command.invoke(ctx)
apollo-apollo-1  |   File "/app/.venv/lib/python3.10/site-packages/discord/ext/commands/core.py", line 1015, in invoke
apollo-apollo-1  |     await self.prepare(ctx)
apollo-apollo-1  |   File "/app/.venv/lib/python3.10/site-packages/discord/ext/commands/core.py", line 931, in prepare
apollo-apollo-1  |     self._prepare_cooldowns(ctx)
apollo-apollo-1  |   File "/app/.venv/lib/python3.10/site-packages/discord/ext/commands/core.py", line 910, in _prepare_cooldowns
apollo-apollo-1  |     bucket = self._buckets.get_bucket(ctx, current)
apollo-apollo-1  |   File "/app/.venv/lib/python3.10/site-packages/discord/ext/commands/cooldowns.py", line 138, in get_bucket
apollo-apollo-1  |     bucket = self.create_bucket(message)
apollo-apollo-1  |   File "/app/.venv/lib/python3.10/site-packages/discord/ext/commands/cooldowns.py", line 172, in create_bucket
apollo-apollo-1  |     return self._factory(message)
apollo-apollo-1  | TypeError: Dalle.get_cooldown() missing 1 required positional argument: 'ctx'


I Can't Set Custom Prefix

Bug Description
The Only Prefix that work Is "@mention" i tried to set it to ! or - and it don't work
Steps to Reproduce
1.Change the Prefix to something Else than @mention
Expected Result
Error Or Something Bot just Don't Detect Command
Debug Output
i cannot run the command My operating system Version is Windows 10 JmusicBot version is 0.3.8
Additional Info
No response
Checklist

[X] I have looked for information about this within the documentation
[X] I have searched for similar issues on the issues page
[X] I am running the latest version of the bot:

I have same bug

https://jmusicbot.com/getting-a-bot-token/ step 7


Doesn't work with new version 11 / Atmosphere

Please update

Confirm, not work on pre-release Atmosphere. Wait official Atmosphere to confirm bug

You can temporary use my forked version with fw11 support
https://github.com/dezem/mtp-server-nx/releases/tag/1.4

Do ppl even use this anymore

Yes, of course, this is the fastest way to transfer data from the pc to the switch or vice versa.
Without having to reboot them to hekate or removing the sd...

That's very very slow :(
Is it normal ?

Ya it's normal...use hekate UMS instead for better performance


Expose lexer item types

We have generally agreed to expose AST types / values that are necessary
to make sense of the AST outside of the promql package. Currently the
UnaryExpr, BinaryExpr, and AggregateExpr AST nodes store the lexer
item type to indicate the operator type, but since the individual item
types aren't exposed, an external user of the package cannot determine
the operator type. So this PR exposes them.
Although not all item types are required to make sense of the AST (some
are really only used in the lexer), I decided to expose them all here to
be somewhat more consistent. Another option would be to not use lexer
item types at all in AST nodes.
The concrete motivation is my work on the PromQL->Flux transpiler, but
this ought to be useful for other cases as well.
Signed-off-by: Julius Volz<EMAIL_ADDRESS>

:+1:
You've test failures.

Doh. Fixed.

Thanks @juliusv. I am looking into a language server implementation for PromQL right now. This should help.

@mxinden Oohhh, looking forward to that!


Carriage return missing in report

When a report is generated the carriage return set in the vulnerabilities description does not seems to be correctly set in the report
And also it misses some text.
I've used markdown with URL and HTTP request
For example
In UI

In report

If you want to have the carriage return in report, you must set a double CR in UI editor.
Maybe it's just a
 not a

 ?
So I think you should correctly set CR while generating report or you could configure the UI editor in a better way to properly manage the CR while generating report ?

Hi @psyray ; Thanks for reporting the issue. This should now be resolved

It's good for the CR. Thanks for that
Only two little things :

Is it possible to add a CR after each block ? Because if you look on the second capture above, the Preuve de concept title is too close and it needs some space above
Is it possible to make each URL clickable ?

Little update,
CR is OK in proof of concept, but not in description.

@santiagolizardo Could you fix this CR problem in the others reported fields ?
Thx


It relies on moz-sql-parser, which has already been archived and deprecated.

It relies on moz-sql-parser, which has already been archived and deprecated.
https://github.com/mozilla/moz-sql-parser
Is there any attempt to make this the successor mo-sql-parsing?
https://github.com/klahnakoski/mo-sql-parsing
I looked at the source code for an hour trying to do this, but I wasn't sure. sorry.

Hey Taikono,
Thanks for pointing that out.  I wasn't aware of it -- I would read more about this new package over the weekend, and would see if we can use the forked package
The other thing I had in mind was testing with Django 4.0 as well

This was pushed to master, and is fixed in the latest version (0.9)
Was getting some exceptions when trying to use for Django 4.0 - so would be doing that as part of the next PR;
Closing


Docs : Add example with zoom principle

If I single click on the plot it scales strangely

Probably when there is only a movement under 5px nothing should happen.
Please also implement the 'double click' that resets the zoom (using onDoubleClick event https://reactjs.org/docs/events.html)


Mismatch in the post slprop type in the case of weakening

This code fails with an SMT failure:
module Test

open Steel.Memory
open Steel.Effect

assume val elim_pure (p:prop)
  : SteelT (_:unit{p}) (pure p) (λ _ → emp)

assume val my_prop1 : prop
assume val my_prop2 : prop

let sl1 : slprop u#1 = pure my_prop1
let sl2 : slprop u#1 = pure my_prop2

assume val change_slprop (_:unit) : SteelT unit sl1 (λ _ → sl2)

let test ()
  : Steel unit
      sl1
      (λ _ → emp)
      (λ _ → ⊤)
      (λ _ _ _ → my_prop2)
  = change_slprop ();
    elim_pure my_prop2

@R1kM and I debugged it together and suspect that it happens because for the elim_pure my_prop2 expression, we compute its type to be _:unit{myprop2}, introduce a post slprop uvar at type _:unit{myprop2} -> slprop, weaken its type to unit, introduce another slprop uvar at type unit -> slprop, and then these two uvars are then equated from the tactic with use_eq flag. This sends a query to the solver to prove myprop2 without any assumption in the context, and that query fails.
It is quite rough and needs more investigation but just recording here. If we move the refinement from the type of elim_pure to ensures, then this works fine.

Another observation was that if we manually add a trailing (), in theory mimicking the insertion of a return by the typechecker for the weakening, this also works fine.


docs: add warning to simple indexer

The simple indexer is currently experiencing a bug when running it with docker jinahub+docker://SimpleIndexer.
After closing the docker container, it sometimes happens that the stored memmap which is mounted into the container is corrupted. After data, the entire index is broken and the user needs to index from scratch.
The error does not happen when using jinahub://SimpleIndexer which is why I quickly changed the README to make the user aware of this problem.

Any idea on how/why this corruption happens?


Implement use_parentheses.

Instead of using the \ as line continuation allow wrapping the the long line within parentheses as suggested in PEP8 using a config setting.
Note that I'm not sure if I've implemented this correctly, but it seems to work :-)

Thank you for this, @mvantellingen !

This looks great, thanks @mvantellingen!
~Timothy

This is not documented currently - you can only find it by looking at the source.


RFP Create/Edit

User Story
As an Infrastructure Committee Admin, I want to have a way to create and edit RFPs so that I can sponsor and facilitate community involvement in infrastructure related work.
Acceptance Criteria

[ ] Verify that the UI implementation corresponds to the Figma design
[ ] Once an admin clicks “Create RFP” from the RFP Feed screen, they see the create form, which has some similarities (but a few differences) to the DevHub proposal form
[ ] Admin is able to add/edit the following fields (* indicates optional field)

[ ] Category Dropdown (Bridges, Data Lakes, Explorers, Onramps / Offramps, Oracles, Query API, Wallets, Other)

[ ] Admin is able to select multiple categories. We should treat this field more like labels.
[ ] If an admin edits categories for an RFP, any associated linked proposals will automatically get updated (with a log showing in their proposal).

[ ] Title
[ ] Summary
[ ] Description
[ ] Final consent checkboxes
[ ] Submission Deadline
[ ] Timeline Stage

[ ] For Decision Stage:

[ ] If admin selects Decision Stage in the dropdown, they must click “Proposal Selected” or “RFP Canceled” radio button.
[ ] Admin cannot choose “Proposal Selected” if the Selected Proposal section below is empty. If they try to click “Proposal Selected” they should see a warning message.

[ ] *Selected Proposal: Only show approved proposals in dropdown. If no approved proposals, show warning message “There are currently no approved proposals”.

[ ] Buttons: Admin should see 3 buttons at the bottom of the form:

[ ] Cancel RFP: (This will only show in edit once RFP was created) Clicking on it will show the “Confirm proposal cancellation” pop up. Upon confirmation, the proposal will move to the RFP Cancelled stage.
[ ] Discard Changes: Clicking on it will cancel without saving changes.
[ ] Submit: Clicking on it will publish/save the RFP.

Attachments
See mock up:

Here is the PR

hey @ori-near, can you please design the cancel RFP modal that we discussed, one with cancel selected/all linked proposal or unlink them.

hey @ori-near, can you please design the cancel RFP modal that we discussed, one with cancel selected/all linked proposal or unlink them.

Hi @Megha-Dev-19 / @PolyProgrammist  – created a separate ticket for this: https://github.com/NEAR-DevHub/neardevhub-bos/issues/778. Let me know if you have any questions.
Also please note I updated the categories list in the AC (e.g. removed Wallets)

hey @ori-near, can you please share the warning message for this Admin cannot choose “Proposal Selected” if the Selected Proposal section below is empty. If they try to click “Proposal Selected” they should see a warning message.


Additional sudo checks

An attempt to improve sudo warning sensitivity to help avoid issues like #1911
Introduces two additional checks.
The first checks for sudo-specific environment variables, then triggers an error message like the home directory discrepancy test which requires explicit user interaction to override.
The second attempts to detect a discrepancy between the actual and effective IDs of the user executing the process, which triggers a warning message that alerts the user they may be executing the installation binary as sudo. This requires no user interaction to bypass.

@kinnison I think you're right. The test cases might be a bit tricky to create, but it would be good to increase coverage as I don't think this function is presently tested at all.

Hi @fotocoyotl
Have you had any further thoughts around adding a test case for this?  Setting specific environment variables before running a test of rustup-init ought not to be too hard.
Also, could you rebase onto current master as I've merged a CI fix to try and mitigate the windows test failures.

@kinnison I have, just haven't had a chance to implement them yet. Rebased.

Hi @latrare
Happy new year!  I was wondering if you might have time in this new year to pick up this work and complete the remaining testing needed to make it mergeable?  If you simply don't have the time/energy/interest then that's completely fine, but if you could let us know either way, I'd appreciate it.
Thanks again for all your efforts last year with Rustup.
Daniel.


GpuWaitResetCollector records zero actions in error

https://github.com/astooke/rlpyt/blob/4dcb17f36d5991082727bc0864fef9ce5054779e/rlpyt/samplers/parallel/gpu/collectors.py#L79
Sometimes the zero action here ends up being treated as a valid action by the code elsewhere. I don't fully understand the parallelization here so I couldn't pinpoint when this failure happens. It can be demonstrated by setting the action to some value you recognize and running example4. In the MinibatchRl training loop the values show up for samples.agent.action[samples.env.done].
To me it seems that this line should probably be removed.

Hmm, let's see what's happening here...
OK so that line is intended as if the model outputs actions for all environment instances, even the ones which are off (waiting to reset).  So it overwrites those actions to the null value before they are recorded (in the collector) into the batch buffer.  But if you have a non-zero null value, then yes again the best thing would be to write step.action[b] = env.action_space.null_value().
The step buffer has fixed leading dimension of [B], so the model receives step inputs even for environments which are off.  And the batch buffer has fixed leading dimensions of [T,B].
Does that make sense?  Or do you have a different place where you write in the null action values?

Ok, thanks for the explanation! The buffering logic makes sense to me now. I overlooked the valid mask used in the algorithms. The null value itself does not matter too much for me.
However, I still do not understand how things work in the following case. This might be entirely my confusion. If I run the code at the bottom of this comment at commit f0a190c7efe0c55f61ca78dfd30f8d91e9277b24, more often than not the last transition considered valid by the DQN loss has action 0 here: https://github.com/astooke/rlpyt/blob/8331b7f919bcecd3dfe9ece85d2fb479d471ff6f/rlpyt/algos/dqn/dqn.py#L199
I used the following snippet in the DQN loss to check whether there are any zero actions in the loss that are valid according to the mask.
if torch.any(samples.action[valid == 1][samples.done[valid == 1]] == 0):
    import pdb; pdb.set_trace()

import gym
import torch

from rlpyt.samplers.parallel.gpu.sampler import GpuSampler
from rlpyt.samplers.parallel.gpu.collectors import GpuWaitResetCollector
from rlpyt.envs.gym import GymEnvWrapper
from rlpyt.algos.dqn.dqn import DQN
from rlpyt.runners.minibatch_rl import MinibatchRl
from rlpyt.agents.dqn.dqn_agent import DqnAgent
from rlpyt.utils.tensor import infer_leading_dims, restore_leading_dims
from rlpyt.models.mlp import MlpModel

NUM_ACTIONS = 100000

class ManyActionDqnModel(torch.nn.Module):

    def __init__(self, **kwargs):
        super().__init__()
        self.head = MlpModel(1, 1, NUM_ACTIONS)

    def forward(self, observation, prev_action, prev_reward):
        lead_dim, T, B, img_shape = infer_leading_dims(observation, 1)
        q = self.head(observation.view(T * B, -1))
        q = restore_leading_dims(q, lead_dim, T, B)
        return q

class ManyActionDqnAgent(DqnAgent):

    def __init__(self, ModelCls=ManyActionDqnModel, **kwargs):
        super().__init__(ModelCls=ModelCls, **kwargs)

class EnvWithManyActions(gym.Env):
    def __init__(self):
        self.observation_space = gym.spaces.Box(0, 1, shape=(1, ))
        self.action_space = gym.spaces.Discrete(NUM_ACTIONS)
        self.steps = 0

    def step(self, action):
        self.steps += 1
        return self.observation_space.sample(), 0, self.steps >= 4, {}

    def reset(self):
        self.steps = 0
        return self.observation_space.sample()

def make_env(*args, **kwargs):
    return GymEnvWrapper(EnvWithManyActions())

def build_and_train(cuda_idx=None, n_parallel=32):
    sampler = GpuSampler(
        EnvCls=make_env,
        env_kwargs=dict(),
        CollectorCls=GpuWaitResetCollector,
        batch_T=5,
        batch_B=32,
    )
    algo = DQN(batch_size=128)
    agent = ManyActionDqnAgent()
    runner = MinibatchRl(
        algo=algo,
        agent=agent,
        sampler=sampler,
        n_steps=1e8,
        log_interval_steps=1e4,
        affinity=dict(
            cuda_idx=cuda_idx,
            workers_cpus=list(range(n_parallel)),
        ),
    )
    runner.train()

build_and_train()

Oh!  Good catch on that block under if not self.mid_batch_reset: in the dqn loss.  I think that might be wrong.  If not using mid_batch_reset, the the replay buffer would have invalid samples which might sometimes be replayed, and I think I was trying to catch this. The collector should record done=1 for all the time-steps after the first environment done=1 signal , all the way until the beginning of the next sampler batch, at which point the environment will have been reset.   But without a time-leading dimension, the valid_from_done() doesn't work (the first done in a sequence is still valid, everything after is invalid).
Quick fix would be: use a collector with mid-batch resets, which bypasses this block/issue.
Slower fix could be: have the replay buffer check whether each sample is valid--if done=1, then make sure the previous time-step has done=0, else it's invalid.  And use this instead of the valid_from_done() currently in there.
As for the zero actions, I think that is a valid Atari action, corresponding to NOOP (no action)?

Yeah that makes sense. In my case the GpuResetCollector works just fine. Zero seems like a reasonable default action, though it might make the code a bit clearer to store the null_value in the collector.
The class doc string in BaseNStepReturnBuffer actually documents this issue already but the example4 has the option for using GpuWaitResetCollector so I didn't think to look there. https://github.com/astooke/rlpyt/blob/15055b4daf6ab47b51a6fa682a9909f1167fc668/rlpyt/replays/n_step.py#L16
Would it make sense to add the validity computation as a method of the BaseNStepReturnBuffer and call it  the same way as compute_returns?

The class doc string in BaseNStepReturnBuffer actually documents this issue already but the example4 has the option for using GpuWaitResetCollector so I didn't think to look there.

OK good point, that example is incomplete, I should at least add a note there.

Would it make sense to add the validity computation as a method of the BaseNStepReturnBuffer and call it the same way as compute_returns?

Yes that sounds reasonable, using the same logic mentioned above, if multiple done=1 steps happen in a row.  I think I debated doing this, but didn't end up putting this into the replay buffer, because I was only using it in the R2D1 algorithm, in which the replay always starts at a valid sample, and returns sequences, on which valid_from_done() works.

Any news or new questions here? If not, we can close this issue?

No news at this point. Let's close it.


Can't install on Android: unable to execute 'arm-linux-androideabi-clang'

I can't install python client with pip3 install neovim on Android via Termux. What is the problem? What should I do? Is it a issue Termux itself or python-client?
Searchable transcript for Google and other places:
    unable to execute 'arm-linux-androideabi-clang': No such file or directory
    error: command 'arm-linux-androideabi-clang' failed with exit status 1

    ----------------------------------------
Command "/data/data/com.termux/files/usr/bin/python -u -c "import setuptools, tokenize;__file__='/data/data/com.termux/files/usr/tmp/pip-build-0hgh1b_7/greenlet/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /data/data/com.termux/files/usr/tmp/pip-xvrwnuql-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /data/data/com.termux/files/usr/tmp/pip-build-0hgh1b_7/greenlet/

Does termux ship with a distro package manager? Then you could try install python3-greenlet with it.

There is no such package like python3-greenlet  on central package manager (Termux uses APT). Also can't install this with pip3 (I don't know if it's what you referring to), it is ended up with same error.

Does it use Debian packages? Or some unofficial port thereof?

Yes they are unofficial ports, can't use Debian packages. They should be compiled specifically for Termux. https://github.com/termux/termux-packages

Hmm, do you have gcc and python3-dev installed?

Okay thanks, installing clang and python-dev packages via apt install solved the problem. You can close the issue as you wish.


Lstm BMI ngen bug fixes

Some fixes to BMI implementation an some unit adjustments to get integration with NGen framework working.
Changes

Change output units to m^3/s and m
Change input unit to mm/hr for assumption that the mass flux is indeed that, and this lets ngen properly convert mm/s forcing to mm/hr
Fix get_value_at_indicies to ensure it copys data into the numpy array so ngen can read it

Testing

Ran integration with NGen with reasonable output results.

Todos

Figure out why converting units of temperature to 'C' didn't trigger automatic conversion from the framework @mattw-nws @robertbartel

Checklist

[x] PR has an informative and human-readable title

[x] Changes are limited to a single goal (no scope creep)

[x] Code can be automatically merged (no conflicts)

[x] Code follows project standards (link if applicable)

[x] Passes all existing automated tests

[x] Any change in functionality is tested

[x] New functions are documented (with a description, list of inputs, and expected output)

[x] Placeholder code is flagged / future todos are captured in comments

[x] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

[x] Reviewers requested with the Reviewers tool :arrow_right:

[x] Linux

[x] MacOs

May conflict with #21 as I believe that same fix is in the PR.

Also temporarily fixes #22 by converting the temperature from an assumed unit of kelvin.  This isn't a long term fix, and needs to be addressed, hence this won't close that issue, but is related.

@hellkite500 Ah... this actually can't just be C but must be °C or degC etc...
see https://ncics.org/portfolio/other-resources/udunits2/#degree_Celsius
For whatever reason, K by itself is fine but not C (guessing disambiguation with c = the speed of light?).

@hellkite500, the comment I put in for #21 would apply here also.  Otherwise, after the suggested adjustment to the Celsius units representation, this looks fine.

@hellkite500, the comment I put in for #21 would apply here also. Otherwise, after the suggested adjustment to the Celsius units representation, this looks fine.

Ah... okay that answers the question I had about what L591-L597 did...looks like this is actually a double-edit/conflict with #21 as suggested above.

Tested unit as degC and it appeared to work correctly.


feat: wmg query endpoint impl with ontology term labels

Part 2 of 4 of story.
Branched off of https://github.com/chanzuckerberg/single-cell-data-portal/pull/1984
Reviewers
Functional:
@MDunitz
Readability:
@tihuan

Changes

Return real ontology labels in WMG API query endpoint response
Add cellxgene-schema==2.1.0 package as , as this is now used for obtaining ontology labels from the "system of record" (that repo)
update test cube generation code to support of semi-real ontology terms (i.e. actual ontology terms, but not always ones that would be admissable w.r.t. to the cellxgene dataset schema).

Definition of Done (from ticket)
QA steps (optional)

Unit tests
Manually validate query endpoint response using local Docker with generated test cube, which now uses real ontology terms:

❯ curl -X POST "http://backend.corporanet.local:5000/wmg/v1/query" -H "accept: application/json" -H "Content-Type: application/json" -d "{\"filter\":{\"organism_ontology_term_id\":\"NCBITaxon:10001\",\"tissue_ontology_term_ids\":[\"UBERON:0000002\", \"UBERON:0000003\"], \"gene_ontology_term_ids\": [\"ENSG00000000419\", \"ENSG00000000005\"]},\"response_option\":\"include_filter_dims_include_dataset_links\"}" | jq .

{
  "expression_summary": {
    "ENSG00000000005": {
      "UBERON:0000002": [
        {
          "id": "CL:0000000",
          "me": 0.01043697834432073,
          "n": 36816,
          "pc": 0,
          "tpc": 0
        },
        {
          "id": "CL:0000001",
          "me": 0.009833387294340086,
          "n": 35693,
          "pc": 0,
          "tpc": 0
        },
        {
          "id": "CL:0000002",
          "me": 0.009822686854067019,
          "n": 35165,
          "pc": 0,
          "tpc": 0
        }
      ],
      "UBERON:0000003": [
        {
          "id": "CL:0000000",
          "me": 0.010225023139072866,
          "n": 36477,
          "pc": 0,
          "tpc": 0
        },
        {
          "id": "CL:0000001",
          "me": 0.009920255662181486,
          "n": 36376,
          "pc": 0,
          "tpc": 0
        },
        {
          "id": "CL:0000002",
          "me": 0.009994227386656507,
          "n": 36607,
          "pc": 0,
          "tpc": 0
        }
      ]
    },
    "ENSG00000000419": {
      "UBERON:0000002": [
        {
          "id": "CL:0000000",
          "me": 0.010212100102863173,
          "n": 35013,
          "pc": 0,
          "tpc": 0
        },
        {
          "id": "CL:0000001",
          "me": 0.009756526782669637,
          "n": 37512,
          "pc": 0,
          "tpc": 0
        },
        {
          "id": "CL:0000002",
          "me": 0.010139412346402041,
          "n": 35635,
          "pc": 0,
          "tpc": 0
        }
      ],
      "UBERON:0000003": [
        {
          "id": "CL:0000000",
          "me": 0.010363893552216626,
          "n": 36711,
          "pc": 0,
          "tpc": 0
        },
        {
          "id": "CL:0000001",
          "me": 0.009676542883128761,
          "n": 35047,
          "pc": 0,
          "tpc": 0
        },
        {
          "id": "CL:0000002",
          "me": 0.009885592475740527,
          "n": 37175,
          "pc": 0,
          "tpc": 0
        }
      ]
    }
  },
  "snapshot_id": "168a560ab0594b7897371c65e299bfbe",
  "term_id_labels": {
    "cell_types": [
      {
        "CL:0000000": "cell"
      },
      {
        "CL:0000001": "primary cultured cell"
      },
      {
        "CL:0000002": "obsolete immortal cell line cell"
      }
    ],
    "genes": [
      {
        "ENSG00000000419": "DPM1"
      },
      {
        "ENSG00000000005": "TNMD"
      }
    ]
  }
}

Known Issues

This PR subsumed by https://github.com/chanzuckerberg/single-cell-data-portal/pull/2068.


Bad or missing version number

Your mod gives that error message, doesn't seem to want to run with 3.6.3 ... oh well

The mod hasn't been updated in a long time (last update was before the defect was added to the game). So it is not surprising that it no longer works. I might update it at some point, but probably not. Most of the benefits you get from synergy picks can be had in other ways, like with the Flyswatter from the conspire mod (you no longer see cards that you skipped in card reward screens).


New version: jendrikseipp.RedNotebook version 2.35

Pull request has been created with WinGet Updater :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:
(Automated response - build 896.)


Removed Unused Snippets from Code  [Correct PR]

Excuse the mess with the previous PR, I have closed it. The following four unused snippets were removed from the code, as they are not referenced in any Markdown documents.
translation: Snippet:MultipleInputs
translation: Snippet:OperationsHistory
translation: Snippet:PollIndividualDocuments
translation: Snippet:StartTranslationWithAzureBlobAsync
This is PR is in reference to issue #22130 .

Thank you @AhmedLeithy !!


Fix link to an effector main site

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/main/CONTRIBUTING.md.
[ ] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[x] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
[x] Content (and my changes are in index.md)

Link to the main website was changed.

Thanks again @sergeysova!


Issue 14 create custom sign in and sign up page

Change Summary

Add sign in page
Add sign up page

Change Form
Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.

[x] The pull request title has an issue number
[x] The change works by "Smoke testing" or quick testing
[ ] The change has tests
[ ] The change has documentation

Other Information
[Is there anything in particular in the review that I should be aware of?]

@NicholasChoong can you approve this too please


Add new Falcon connector

@microsoft-github-policy-service agree company="Nordantech"

@bezhan-msft

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

NOTE: this PR creates a new article if you decide to accept it, the commits must be moved to the private repo for staging and a full review. It will also need an entry in the TOC.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#reassign:ptyx507x
#assign:jonburchel


chore!: update android minSdkVersion to 23, kotlin to 1.8.22, gradle build tools to 7.4.2, and gradle to 7.5

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

v3.0.0
GitHub release

Your semantic-release bot :package::rocket:


User documentation

@abe33 I think it's best to move the user docs to the top level, as you suggested. Could you do this before/after merging into master?
Thanks!


Use with Mapbox on-premise installation

Would it be possible to use nativescript-mapbox with an on-premise installation of Mapbox? In other words, configure the mapbox server url.

I have no idea. What would that take?

By default Mapbox tiles are loaded from https://api.tiles.mapbox.com and I am looking for a way to change this to another domain which hosts Mapbox as mentioned at https://www.mapbox.com/atlas/
But I guess I should dig deeper into the Mapbox-native code.

Would be cool if you can find where in the SDK this can be configured.

This Stackoverflow page answers my question: https://stackoverflow.com/a/36837943 There is a possibility to configure a url for where sources are loaded from.


Mac version 0.10.0

Hi There
Just downloaded MIQ Booking assistance v 0.10.0 for Mac and followed instructions.
The App has fired up Chromium and I have logged in and navigated to the select date page.
App says: Found "Secure your allocation" page! Wait for beep sound, then select date and continue booking.
Browsers says it is being controlled by automated test software.  But the page is not being refreshed every 5 seconds as the App window indicates.   App window Restart Search button is also grayed out.
The MIQ site then times out and asks for re-login.

@Deroles 0.10.0 has known issues with the current miq site.


Subject Tree missing items in UUI after latest update #8 (<IP_ADDRESS>6)

Subject Tree missing items in UUI after latest update #8 (<IP_ADDRESS>6).

after the instance update (Dec 6) to update 8, the UUI subject control no longer displays all subject tree items.
the missing items seem random with no apparent reason or pattern for their absence.
the missing items don't appear in the UUI, but they exist in the tree and display correctly in the web ui.
This was working fine for months until this past Monday morning after the update.
the behavior is identical in all three orgs (dev, test, prod).
Also, the same subjects items are missing in each org.
if I apply a custom control (OOTB subject tree) to the field, then all the subject items appear correctly. (Unfortunately the OOTB control errors and won't save the results...appears not to bind the control to the data, so this is not a viable workaround.)

“Thanks for your feedback. For product-related feedback, visit https://experience.dynamics.com/ideas/.
Closing this issue.”

HI @KumarVivek ....
Not sure why this was tagged as product-feedback when it's a bug introduced in update #8.
This is not a feature request...this is a bug we're experiencing.
Thanks in advance


Dropdown positioning

What happened?
The dropdown from iX automatically detects the screen end on left or right. Means it opens automatically to the left, if it is on the right end of the screen. This does not work from bottom or top. If it is at the bottom of the screen the dropdown is out of the
"monitor".
The dropdown should automatically open to the top at this point. Same for top positioning -> it should get opened to the bottom.

What type of frontend frameware are you seeing the problem on?
Angular
Which version of iX do you use?
v1.4.0
Code to produce this issue.
No response

Already fixed with #694


Error with dropEst->velocyto workflow

I've been trying to follow your vignette for analyzing Drop-seq data with dropEst, then running velocyto. However I've encountered an error during dropest_bc_correct that I can't seem to navigate around.
Here's what I've tried:
droptag -c ~/dropEst/configs/drop_seq.xml R1.fastq.gz R2.fastq.gz -n N714_droptag

STAR --genomeDir ~/STAR_indexes/hg38-mm10/ --readFilesIn N714_droptag.fastq.gz --outSAMmultNmax 1 --runThreadN 8 --readNameSeparator space --outSAMunmapped Within --outSAMtype BAM SortedByCoordinate --outFileNamePrefix N714_droptag --readFilesCommand gunzip -c

dropest -m -V -b -o N714_dropest -g gencode_mm10-vM22_hg38-v31.gtf -L eiEIBA -c ~/dropEst/configs/drop_seq_velocyto.xml N714_droptagAligned.sortedByCoord.out.bam

velocyto tools dropest_bc_correct N714_droptagAligned.sortedByCoord.out.tagged.bam N714_dropest.rds

Everything runs to completion except the dropest_bc_correct command, which produces the following error:
2019-10-10 11:18:54,548 - INFO - Loading `merge_targets` from N714_dropest.rds using R / rpy2
Traceback (most recent call last):
 File "~/.local/bin/velocyto", line 11, in <module>
   load_entry_point('velocyto==0.17.17', 'console_scripts', 'velocyto')()
 File "~/python/3.6.6/lib/python3.6/site-packages/click/core.py", line 716, in __call__
   return self.main(*args, **kwargs)
 File "~/python/3.6.6/lib/python3.6/site-packages/click/core.py", line 696, in main
   rv = self.invoke(ctx)
 File "~/python/3.6.6/lib/python3.6/site-packages/click/core.py", line 1060, in invoke
   return _process_result(sub_ctx.command.invoke(sub_ctx))
 File "~/python/3.6.6/lib/python3.6/site-packages/click/core.py", line 1060, in invoke
   return _process_result(sub_ctx.command.invoke(sub_ctx))
 File "~/python/3.6.6/lib/python3.6/site-packages/click/core.py", line 889, in invoke
   return ctx.invoke(self.callback, **ctx.params)
 File "~/python/3.6.6/lib/python3.6/site-packages/click/core.py", line 534, in invoke
   return callback(*args, **kwargs)
 File "~/.local/lib/python3.6/site-packages/velocyto/commands/dropest_bc_correct.py", line 64, in dropest_bc_correct
   mapping = convert_r_obj(ro.r(f"rds <- readRDS('{filename}'); rds$merge_targets"))  # a dict
UnboundLocalError: local variable 'convert_r_obj' referenced before assignment

Note that specifying the -f flag for dropEst as you recommend elsewhere fails and produces empty files (no valid CBs found).
Do you have any recommendations on how to get around this issue?

The exactly same error occurred in our lab.
We'll be very appreciated if anybody could solve this problem.

The same issue happened to me.

Having the same issue. Would appreciate any sort of solution. Thanks

ditto, and bump

So I can get around this issue by installing rpy2 in my environment (I had to dig around the source to find this), but now it errors out with  AttributeError: module 'rpy2.rinterface' has no attribute 'RNULLType', which upon some searching led to this issue that suggests installing a specific version of rpy2.
Doing: pip3 install rpy2==2.9.5 appears to fix this problem
But now I am stuck on a KeyError: "tag 'CB' not present" error -- but this is another issue altogether

I encountered the exact same issues mentioned, and I found a solution.
The first problem, "UnboundLocalError: local variable 'convert_r_obj' referenced before assignment", occurred because I did not install velocyto perfectly. Reinstall it will solve the problem.
Second, "AttributeError: module 'rpy2.rinterface' has no attribute 'RNULLType'" is because that script of rpy2 after version 3.0.0 have totally rewrite, so we have to install version before rpy2 3.0.0.
The third problem. I have mentioned in  #298.
So, to solve these problems. first of all you have to create a conda evironment with python <=3.6, then conda install rpy2=2.9.4, which will install R3.6.0 simultaneously, then install the packages in velocyto tutorials, and do not forget pysam. Modify the py files and everything goes fine with the pipeline with some easy-fixed bugs.


I want a programming language for Linux that allows me to create applications using a drag-and-drop interface, similar to using Delphi to drag and drop controls in Windows. Using C++ on Linux has a high barrier to entry and a steep learning curve, making it inaccessible for many users who simply don't have the time to study it.

I want a programming language for Linux that allows me to create applications using a drag-and-drop interface, similar to using Delphi to drag and drop controls in Windows. Using C++ on Linux has a high barrier to entry and a steep learning curve, making it inaccessible for many users who simply don't have the time to study it.

Maybe your are looking for something like: Cambalache. We may provide gtk binding in the future with native backend.
The subject however is unrelated with this repository.


Support Unofficial sources of character sheets, Talents and ships

As proposed by ninjew42 on the Modiphius Forums:

Adding in various fan character sheet types like the Romulan one that’s out there and talents made by the Continuing Mission group, could include a source button for Continuing Mission resources.
I know for some species there doesn’t seem to be anything so far that suggests we’ll get talents for Jem’Hadar or Vorta for example so having the fan work added could be helpful for people wanting to make those kinds of characters so they don’t have to go back and modify the sheet after.

So it looks like some non-rules species have been added already (Borg, Gorn, Jem'Hadar, Reman, Romulan, Vorta). Where are they from? And what books are they tied to in the tool? For example, selecting core book only and TNG era, Gorn and Reman aren't available.

Those were added before I started working on the code: I think they're reflections of the NPC write-ups in various characters. So, for example, there's a Reman bodyguard written up on p. 106 of the Beta Quadrant book (with an "Reman Character" info box).

Yeah, I found a few of them later. I suppose the question is whether they should be in the tool, given that they are "unofficial"... although they do use the Modiphius species wording and attributes, so it's a weird grey area. My suggestion would be to have an "unofficial" button added to the source selection (although it's already quite crowded).

I don't think they're "unofficial". I think the point is that you can make Major NPCs that happen to be Cardassian even if you only have the "Core" book. (Obviously, the "Starfleet Academy" lifepath option is a bit weird, then, but whatevs).

I now support Continuing Mission as an unofficial source. If another source with non-trivial usage emerges, I'll look in to supporting that, as well.


package name differs from packagist name in composer.json

In the composer.json, the name is agentejo/cockpit, but on packagist, the name is aheinze/cockpit.
So this one works and lib/vendor/aheinze/cockpit was created after running:
composer install --no-dev --ignore-platform-reqs
{
    "name": "raffaelj/my-project",
    "require": {
        "aheinze/cockpit": "^0.11.0"
    }
}

But if I try to require a modified fork, the vendor name is wrong.
{
    "name": "raffaelj/my-project",
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/raffaelj/cockpit",
            "no-api": true
        }
    ],
    "require": {
        "aheinze/cockpit": "dev-wpmp-cockpit-lib"
    }
}

error: Your requirements could not be resolved to an installable set of packages.
If I change the vendor, to "agentejo", it works and the folder lib/vender/agentejo/cockpit is available with the content of my fork.
{
    "name": "raffaelj/my-project",
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/raffaelj/cockpit",
            "no-api": true
        }
    ],
    "require": {
        "agentejo/cockpit": "dev-wpmp-cockpit-lib"
    }
}

But now the folder name is different and the dependency has a wrong vendor name.

Good point. but why would you install cockpit in a vendor folder? that's why I didn't updated the package. It was strange for me to install cockpit in a vendor folder

but why would you install cockpit in a vendor folder?

I really like the concept of my cockpit-lib-skeleton with modified paths, a clean docs root and my own favicon.ico etc. So I can leave cockpit in a lib folder, I don't have to care about new files in the root, that aren't in the .gitignore file in a dev environment and I could update it with composer in a production environment.
In combination with oomphinc/composer-installers-extender, I can install composer packages of type "project" in custom folders, e. g. lib/cockpit or cockpit, which makes composer a useful tool to build a project, that depends on cockpit, but has it's own base.
I also combined the concepts above in my latest work: WPMultiplane - a fully functional Cockpit CMS with CpMultiplane and a few cockpit addons packaged as an experimental WordPress plugin. I'm still not sure, if I created something beautiful or if it was a very, very bad idea - but it works.
I needed a modified fork of cockpit and now "agentejo/cockpit" is listed in the dependency graph.


Add MAST downloader + fix broken quickstart

#254 points out the quickstart documentation was broken because of a filepath error. To make the downloading more robust + intuitive, this pull request...

introduces download_from_mast to make it easier to download basic JWST files from MAST
replaces a fragile direct link download with download_from_mast in quickstart.ipynb
updates version to 0.4.14

This closes #254 , with the updated documentation now live at https://zkbt.github.io/chromatic/quickstart/


Changes to how Mutators are used.

Summarize the changes made and why they are needed.
I created this PR to simplify how mutators work and how they are implemented.
I did this by moving the mutators all to the shared library in models, and adding a list of mutators to each package manager to specify which mutators are implemented for that package manager.
For example, NuGet and NPM are both not using the UnicodeHomoglyphMutator.cs mutator.
I did cherry-pick some of the files from #277, the oss-find-squats-lib files for SquatChecker, and FindSquatsResult, as well as the changes for FindSquatsTool in oss-find-squats.
Testing
Summarize what testing (automated or manual) has been done to verify this change.
Checklist
Please complete the following before publishing this PR. For any below items that are not applicable or otherwise can’t be completed,
strike out item (place ~~ around it on both sides) and add a note to the end explaining why this can’t be done.

[ ] Unit Testing: Changed code is covered by passing unit tests.
[ ] Acceptance Testing: Changes to API definitions and/or behavior are covered by passing acceptance / integration tests.
[X] Code Comments: Where not self-evident, changed code includes comments describing its purpose and/or why it was written a certain way.
[ ] Logging & Metrics: This change has sufficient logging & metrics for us to diagnose problems in production.

TODO:

[ ] Add BitFlip mutator back.
[ ] Maybe more mutators?

@jpinz Thanks for the contribution. I'll give this a close look tomorrow.
I do like giving the mutators each their own class, but I'm less sure about adding mutator information to each package manager - that seems to be to be an easy way to get unexpected behavior if you don't remember to add a new mutator to one to the N package managers.

@jpinz Thanks for the contribution. I'll give this a close look tomorrow.
I do like giving the mutators each their own class, but I'm less sure about adding mutator information to each package manager - that seems to be to be an easy way to get unexpected behavior if you don't remember to add a new mutator to one to the N package managers.

Thanks for checking it out!
I get the concern, I have a feeling we can probably find a decent solution to that. Also, a good argument for detailed unit tests 😄
The PR is in draft because I anticipated a lot of good conversation stemming from the initial publish of just this draft. So I am wide open to suggestions from everyone!

I'm still working on the best way to integrate changing the mutators to be classes instead of methods.  I think my main concern with bringing the mutators into the shared library is

They're not really shared
The way the pipelines work any changes to shared will trigger a rebuild of all of the tools. For now we are only publishing find-squats but eventually that might not be the case. Given that those other components do not actually use the mutators it would create a lot of churn anytime the mutators were updated.

I think I will be keeping the mutators in the FindSquatsLib, but its possible maybe to implement it as an extension method that is available for the packagemangers in the find squats lib.

I incorporated a lot of this into #277 with the extension method implementation.

More updates to #277 including refactoring to use a MutatorType enum and returning Mutation objects.

Closing as #277 included most of this work.


Fix data URI scramble (#16098)

Backport #16098

No prefix for data uris.

Added test to prevent regressions.

:rocket:


Improve repr for none and auto

Related Issue: #2747
Please carefully review those changed PNG, I'm not 100% sure they are rendered as expected.

This changes repr(auto) to type(auto), but the proposal in #2747 to change repr(type(auto)) to type(auto) while keeping repr(auto) as auto, specifically so that the two can be distinguished.

Sorry I misunderstood. I will fix this later.

Thanks!


use get_sstable_data_files method in ccm

@ptnapoleon, please review.
This should only be merged after https://github.com/pcmanus/ccm/pull/540

+1


Code should be built on pull request

We should always build when a PR is made, which isn't currently happening.
PR's must require a successful build to be approved.

Closed by #116.


Change commit message format to use colon instead of a dash

It seems that people have converged to using a colon in the commit messages, so let's make it official
Fixes #90: Fix code in http2 adaptor that was doing improper q2 related logging

(Anything else in there that should change?)

Codecov Report
Base: 27.59% // Head: 27.59% // No change to project coverage :thumbsup:

Coverage data is based on head (4aa59da) compared to base (d0fe9b4).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #766   +/-   ##
=======================================
  Coverage   27.59%   27.59%
=======================================
  Files         131      131
  Lines       31452    31452
  Branches     5036     5036
=======================================
  Hits         8679     8679
  Misses      21668    21668
  Partials     1105     1105

Flag
Coverage Δ

unittests
27.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

It seems reasonable to me; I find colons more clear.


Added detection of the TIMESTAMP datatype.

This seems like a good idea.  Can you provide a unit test which demonstrates it's utility?

I've added the unit test.
This conflicts with #115, however I will happily rebase/rewrite the pull which is pulled last.


feat(rbac): patterns on rbac policy objects

Motivation:
There is no way to capture multiple objects in a single policy today and
match against it. The outcome is that policies for basic permission,
where the objects are not catalog entities, are not fine grained. Either
you have permission on an action on all the category, or not at all.
For example the workflows that the orchestrator plugin exposes are not
a catalog entities. They only have a workflow ID the plugin exposes and
there is no way today to deny or allow some of the workflows usage.
I.e, this is the permission a workflow have today for read:
orchestrator.workflow.read, read, allow

Modification:
The only modification for the rbac plugin is to use a regexp match:
[matchers]
m = g(r.sub, p.sub) && regexMatch(r.obj, p.obj) && r.act == p.act

Result:
With this change we have the permission be matched by a regexp:
orchestrator.workflows ,       read, allow  // catches all workflow
orchestrator.workflows.banned ,read, deny   // catches banned workflow

Resolves: https://github.com/janus-idp/backstage-plugins/issues/2025

need to figure out why it does not work with the tempEnforcer

@PatAKnight
@rgolangh implemented orchestrator workflow authorization by creating a role for each workflow instead of using conditions.
This requires a small change in RBAC plugin.
WDYT?

@PatAKnight @rgolangh implemented orchestrator workflow authorization by creating a role for each workflow instead of using conditions. This requires a small change in RBAC plugin. WDYT?

just to make sure there is no confusion, it is not a role per workflow. it is a policy that can match against multiple objects. The object identifier in the orchestrator case is the a string of the form orchestartor.workflow.$WORKFLOW_ID, for the route endpoint /workflows/$WORKFLOW_ID.

@AndrienkoAleksandr @PatAKnight please review

Sorry I am just now getting to this.
I left a couple of comments in the PR but also wanted to include some thoughts as well.
I realize now that I may have misunderstood what you were trying to accomplish with using pattern matching on permission policies. I was under the impression that you were actually trying to reduce the need to define many permission policies that all have a common resource type. An example in my head is something like the following:
# Assume role:default/example is already defined

# catalog-entity resource type has three different permissions associated with
# it so defining permission policies would look something like this
p, role:default/example, catalog-entity, read, allow
p, role:default/example, catalog-entity, delete, allow
p, role:default/example, catalog-entity, update, allow

# With the policy matching that I was thinking about it would instead match based
# on the resource type with the wildcard on the action itself

wildcarding the action is useful on its own for composing roles. But yes, this is not what I was using it for.

thus granting read, update, and delete
p, role:default/example, catalog-entity, *, allow

I am not certain at the moment if this is possible though.

The main difference in the orchestrator permission usage is that we don't have a resourceType, but instead we use basic or named permission since we don't have our object in the catalog.
Correct me if I'm wrong, conditions could not be declared on the policy file itself right? you have to use the rbac ui to add those?

Correct me if I'm wrong, conditions could not be declared on the policy file itself right? you have to use the rbac ui to add those?

we introduced separated external file for conditional policies for RHDH 1.3: https://github.com/janus-idp/backstage-plugins/blob/ef9ee685be72508eb6c5b174da2565076515cb56/plugins/rbac-backend/README.md#configuring-conditional-policies-via-file

given the feedback I'm closing this PR for now. The future work might use conditions instead or just reopen and continue with this PR. I leave this to @batzionb


Laid the groundwork for websocket communication

Hey @maf0115,
I open this Pull Request to be able to provide you with feedback on your track.

I will try to summarize the desired behavior to you:

In your constructor: Subscribe to rosout and open the websocket
In your subcriber callback:

Check that your websocket is still open -> if not, open it again like in you contructor
Format the received data from rosout into a format specified with @di-math
Send the data over the websocket

In your destructor: Close the websocket

Another question: Why are you using a websocket for that? (maybe also to @di-math)
Websockets are TCP based, which is rather problematic for our appliance, where we can already tell that we will have a rather loosy connection.
Why don't you use basic UDP instead?
Would be much easier to handle, faster and all in all better suited for our appliance?
See here for a basic example: https://wiki.python.org/moin/UdpCommunication#:~:text=Here's simple code to post a note by UDP in Python 3%3A

Let's open up a new PR, since we got major changes.


Handle possible strict mode bugs

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

/check-enforcer evaluate

/check-enforcer help

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Improved subpixel text rendering for OLED (WRGB stripe, RGB triangular)

Description of the new feature / enhancement
ClearType alters anti-aliasing assuming an RGB strip subpixel configuration. More WOLED (WRGB stripe) and QD-OLED (RGB triangular) monitors are coming to market and have noticeable chromatic aberration/color fringing on edges of text. It would nice if Windows had a built-in option to alter text rendering based on subpixel configuration of the monitor.
Scenario when this would be used?
When using a WOLED or QD-OLED monitor with WRGB stripe or RGB triangular subpixel configuration to improve text clarity.
Supporting information
https://www.youtube.com/watch?v=52eiLP3Zy-s
https://youtu.be/k-6yc1dA--0?t=602

Windows has a built-in thinghy to adjust ClearType.

@Jay-o-Way please reopen.
I work with monitor manufacturers.
This is incorrectly closed.
As founder of Blur Busters / TestUFO, I work with computer monitor manufacturers; so I will explain
What ClearType Is Compatible With:
ClearType is only compatible with vertical-stripe RGB and BGR pixel structure.
 AND

What ClearType is NOT Compatible With:
ClearType tuner is 100% unable to fix this.
LG WOLED, both their televisions and monitors
It is not possible to control the white pixel, but ClearType can at least go into a special "RBG" mode where the blue subpixel is in the middle.  I did some PhotoShop tests and it actually makes text better on an LG WOLED.
Samsung QD-OLED

is this a duplicate of #25595 then? :)

No.  Superseded by #25595.

Fixed my comment

Excellent -- thanks for removing the label.
Now please choose which of the two to mark as duplicate.  Shall I crosspost everything from #25568 over here, so the other item can be closed as duplicate?

Excellent -- thanks for removing the label.
Now please choose which of the two to mark as duplicate.  Shall I crosspost everything from #25568 over here, so the other item can be closed as duplicate?    My gigantic image-filled white-paperesque commentary will be moved to the non-duplicate that you choose.
Duplicate decision tree:
Pros: #25595 already has comments & reactions and useful re-commentary added
Cons: #25595 is newer  than #25568
@mdrejhon Vote: Keep #25595, close #25568

Formal request:
Please close this item as duplicate
(The other person was so impressed that he thinks I should work for Microsoft.  That golden comment makes #25595 win the duplicate contest)
#25568 Effectively Closed

No response so far.

Yeah, this is a subject that only few people will have inside knowledge on.

...win the duplicate contest

I wouldn't look at it that way. Things can take quite some time here. Many, many issues and not enough people to handle it all. Of the people that are involved, we have many volunteers.

/dup #25595


Any CONTRIBUTING guidelines for node-esapi?

@DeadAlready & @chrisisbeef ,
The node-esapi repo does not contain any CONTRIBUTING.md guidelines. We (as a corporation) are preparing a PR bringing more code to node-esapi from owasp-esapi-js code (also adding bug fixes & a test harness), and are wondering wether we need to undergo any kind of constraint (document sign-off... etc) before submitting the code.
Thanks

I will leave it up to @DeadAlready as it's his project, we brought it
under the ESAPI umbrella a couple years ago to raise awareness of it's
existence and provide some consistency across the various
implementations and forks of the ESAPI project. @kwwall now runs the
ESAPI project proper and may have thoughts here as well.
On 2/13/2017 7:23 AM, Francois-Xavier Kowalski wrote:

@DeadAlready https://github.com/DeadAlready & @chrisisbeef
https://github.com/chrisisbeef ,
The node-esapi repo does not contain any CONTRIBUTING.md guidelines.
We (as a corporation) are preparing a PR bringing more code to
node-esapi from owasp-esapi-js code (also adding bug fixes & a test
harness), and are wondering wether we need to undergo any kind of
constraint (document sign-off... etc) before submitting the code.
Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ESAPI/node-esapi/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAl_CsAthGpbIxZ0h4UdeZPKcpvYybUxks5rcGdkgaJpZM4L_Qro.

No real constraints from my side. Only that it follows the same coding style (4 space indentation and same line { etc ) and at least passes the measly test that are there at the moment.

Ok thank you.

After looking further in the code, we realized that node-esapi is about shipping the browser ESAPI code from a Node.js/Express server to a Browser. This is not what we want to achieve (run on Node.js), so I will close this issue & raise the question on the esapi-dev mailing instead.


Error when controlling sensibo using homekit

Home Assistant release with the issue: 0.66.1

Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.): CentOS 7 (latest)

Component/platform: homekit and sensibo

https://www.home-assistant.io/components/climate.sensibo/
https://www.home-assistant.io/components/homekit/

Description of problem:
I can control my Sensibo AC via Home Assistanct
I can connect Home Assistant to Apple HomeKit. I can see both the home assistant bridge and the sensibo AC as devices in HomeKit
When trying to control the AC device via the apple home app nothing happens. In home assistance I see this error:
2018-04-15 17:05:30 ERROR (MainThread) [homeassistant.core] Error executing service <ServiceCall climate.set_operation_mode: operation_mode=off, entity_id=['climate.living_room']>
Traceback (most recent call last):
  File "/srv/homeassistant/lib64/python3.6/site-packages/homeassistant/core.py", line 1002, in _event_to_service_call
    await service_handler.func(service_call)
  File "/srv/homeassistant/lib64/python3.6/site-packages/homeassistant/components/climate/__init__.py", line 392, in async_operation_set_service
    await climate.async_set_operation_mode(operation_mode)
  File "/srv/homeassistant/lib64/python3.6/site-packages/homeassistant/components/climate/sensibo.py", line 292, in async_set_operation_mode
    self._id, 'mode', operation_mode, self._ac_states)
  File "/srv/homeassistant/lib64/python3.6/site-packages/pysensibo/__init__.py", line 76, in async_set_ac_state_property
    return (yield from resp.json())['result']
  File "/srv/homeassistant/lib64/python3.6/site-packages/aiohttp/client_reqrep.py", line 850, in json
    headers=self.headers)
aiohttp.client_exceptions.ContentTypeError: 0, message='Attempt to decode JSON with unexpected mimetype: text/html'

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
# Sensibo
climate:
  - platform: sensibo
    api_key: !secret sensibo_api

# Homekit
homekit:

Traceback (if applicable): Not sure what to add here...

Additional information:

Please try the update to 0.67.0 first.

working fine from frontend
upgraded to 0.67 but issue still exists.
2018-04-15 18:28:52 ERROR (MainThread) [homeassistant.core] Error executing service <ServiceCall climate.set_operation_mode: operation_mode=off, entity_id=['climate.living_room']>
Traceback (most recent call last):
  File "/srv/homeassistant/lib64/python3.6/site-packages/homeassistant/core.py", line 1002, in _event_to_service_call
    await service_handler.func(service_call)
  File "/srv/homeassistant/lib64/python3.6/site-packages/homeassistant/components/climate/__init__.py", line 392, in async_operation_set_service
    await climate.async_set_operation_mode(operation_mode)
  File "/srv/homeassistant/lib64/python3.6/site-packages/homeassistant/components/climate/sensibo.py", line 292, in async_set_operation_mode
    self._id, 'mode', operation_mode, self._ac_states)
  File "/srv/homeassistant/lib64/python3.6/site-packages/pysensibo/__init__.py", line 76, in async_set_ac_state_property
    return (yield from resp.json())['result']
  File "/srv/homeassistant/lib64/python3.6/site-packages/aiohttp/client_reqrep.py", line 869, in json
    headers=self.headers)
aiohttp.client_exceptions.ContentTypeError: 0, message='Attempt to decode JSON with unexpected mimetype: text/html'
2018-04-15 18:28:55 DEBUG (Thread-8) [pyhap.hap_server] Request PUT from address '('<IP_ADDRESS>', 52766)' for path '/characteristics'.
2018-04-15 18:28:55 DEBUG (Thread-8) [homeassistant.components.homekit.type_thermostats] climate.living_room: Set heat-cool to 2
2018-04-15 18:28:55 INFO (Thread-8) [pyhap.hap_server] <IP_ADDRESS> - "PUT /characteristics HTTP/1.1" 207 -
2018-04-15 18:29:00 DEBUG (Thread-8) [pyhap.hap_server] Request PUT from address '('<IP_ADDRESS>', 52766)' for path '/characteristics'.
2018-04-15 18:29:00 DEBUG (Thread-8) [homeassistant.components.homekit.type_thermostats] climate.living_room: Set target temperature to 25.00°C
2018-04-15 18:29:00 INFO (Thread-8) [pyhap.hap_server] <IP_ADDRESS> - "PUT /characteristics HTTP/1.1" 207 -
2018-04-15 18:29:02 DEBUG (Thread-8) [pyhap.hap_server] Request PUT from address '('<IP_ADDRESS>', 52766)' for path '/characteristics'.
2018-04-15 18:29:02 DEBUG (Thread-8) [homeassistant.components.homekit.type_thermostats] climate.living_room: Set target temperature to 22.00°C
2018-04-15 18:29:02 INFO (Thread-8) [pyhap.hap_server] <IP_ADDRESS> - "PUT /characteristics HTTP/1.1" 207 -
2018-04-15 18:29:02 DEBUG (Thread-8) [pyhap.hap_server] Request PUT from address '('<IP_ADDRESS>', 52766)' for path '/characteristics'.
2018-04-15 18:29:02 DEBUG (Thread-8) [homeassistant.components.homekit.type_thermostats] climate.living_room: Set target temperature to 21.00°C
2018-04-15 18:29:02 INFO (Thread-8) [pyhap.hap_server] <IP_ADDRESS> - "PUT /characteristics HTTP/1.1" 207 -

Is it only the operation mode off that is causing the issue?

no, any operation mode

One more piece of information - I can now verify it's a Sensibo Issue:

I connected my philips hue lights to homekit via homeassistant and everything works as it should.
When the AC is on, the A/C seems to work, but I can't pinpoint the exact issue

Off mode never works
When I changed temperature the state also seems to sync (state+temp)
When I change just state nothing happens
I can't switch on / off in any situation

If it's not a HomeKit issue, I unfortunately can't help you. Maybe someone else might.
In that case it could be a good idea to close this issue and open a new one (without HomeKit in it's title), Without that I'm not sure anyone will look at this, since it already has some replies and others might think it's already being handled.

Can we tag it as sensibo plugin issue and assigning to whomever wrote the plugin? I believe it should be on their issue list

@andrey-git Can you take a look at this? I belive you're the codeowner of the cimate.sensibo platform.

If it works from the UI, but not from homekit, I would assume homekit is sending wrong parameters to the service calls. Unfortunately sensibo's error message is unhelpful. I'll try to fix that - hopefully it will help with the original issue.

hi andrey. any luck getting it to work?
fyi - the homebridge plugin works without issues (i have the original version, not sky, but I believe I used the sky plugin)

I still didn't get to propagating sensibo's error.
But now looking at the log above it seems suspicious:
Error executing service <ServiceCall climate.set_operation_mode: operation_mode=off, entity_id=['climate.living_room']>

This is not how you turn Sensibo off. If Homekit does that - it's a bug.
Sensibo has a turn_on/turn_off services instead of set_operation_mode to off.


Match header font and styles to current styleguide

We have some slight differences in font and size for the nav items:

Reopening to tweak a couple header styles match the links with size and weight of the text:


Update doc homepage monthly downloads

Link to the Page
https://www.skeleton.dev/
Describe the Issue (screenshots encouraged!)
This was 45k+/month, but is now closer to 60k+/month!

https://www.npmjs.com/package/@skeletonlabs/skeleton


Incorrect UUID validation regex

Package version eg. v8, v9:
v9
Issue, Question or Enhancement:
The regular expression validation for UUIDs only allows lowercase characters.
RFC 4122 allows uppercase characters for input of functions but requires lowercase characters on output:

Each field is treated as an integer and has its value printed as a
zero-filled hexadecimal digit string with the most significant
digit first.  The hexadecimal values "a" through "f" are output as
lower case characters and are case insensitive on input.

In addition the acceptable hex digits are (according to section 3 of the RFC):
hexDigit =
    "0" / "1" / "2" / "3" / "4" / "5" / "6" / "7" / "8" / "9" /
    "a" / "b" / "c" / "d" / "e" / "f" /
    "A" / "B" / "C" / "D" / "E" / "F"

Code sample, to showcase or reproduce:
v := validator.New()
err := v.Var("479992BD-5A1E-43AA-9268-597F581380F0", "uuid4") // err will not be nil

As asked in the PR https://github.com/go-playground/validator/pull/403#issuecomment-432718271

Closing as it's fixed


Android: Add landscape layout for Main Activity

Android: Add landscape layout for Main Activity

User will be locked in portrait mode in Android.  Therefore, no use case for landscape mode.


amazon env vars missing

Describe the bug: (a clear and concise description of what the bug is)
when choosing to use awskms I don't see the needed env vars injected into drun:
AWS_DEFAULT_REGION
AWS_REGION
AWS_ACCESS_KEY_ID
AWS_SECRET_ACCESS_KEY

In the current setup these are needed for sops.
Next to that I see AWS_DEFAULT_REGION  and AWS_REGION are also missing from vault operator.

As this was a bit sloppy work, I am now wondering what else is missing from vault operator?

The region var is already there in config part of vault operator.
I will add other vars for aws to drun env.


Command missing: New-AzureAdContact, Set-AzureAdContact

Hello,
the following commands are missing:

New-AzureAdContact (AzureAd)
Set-AzureAdAccount (AzureAd)
New-Contact (ExchangeOnlineManagement)
Remove-Contact (ExchangeOnlineManagement)

PS C:\> Get-command -noun AzureADContact

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Cmdlet          Get-AzureADContact                                 <IP_ADDRESS>  AzureAdPreview
Cmdlet          Remove-AzureADContact                              <IP_ADDRESS>  AzureAdPreview

PS C:\> Get-command -noun Contact

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Function        Get-Contact                                        1.0        tmp_hcroih1x.2yk
Function        Set-Contact                                        1.0        tmp_hcroih1x.2yk

PS C:\> Get-command -noun MailContact

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Function        Get-MailContact                                    1.0        tmp_hcroih1x.2yk
Function        New-MailContact                                    1.0        tmp_hcroih1x.2yk
Function        Remove-MailContact                                 1.0        tmp_hcroih1x.2yk
Function        Set-MailContact                                    1.0        tmp_hcroih1x.2yk

If someone could tell me how I can create Contact objects (for use in Teams!) in AzureAd or in a specific Global Address List, I would appreciate it. I need to cover the full lifecycle: New/Get/Set/Remove
I am trying to solve a migration issue moving from Skype to Teams and create Contact Objects for Analog Devices we have in the organisation.
Thanks,
David

Cross-promoted in office-docs-powershell as well: https://github.com/MicrosoftDocs/office-docs-powershell/issues/7016

Cross-promoted in office-docs-powershell as well: https://github.com/MicrosoftDocs/office-docs-powershell/issues/7016

@DEberhardt
I have opened a work item for this so that engineering can take a look and possibly address this. Unfortunately, these examples are coded in the source and generated automatically, so any updates would get overwritten once we update this again. We need to get engineering to change this in the code. At this point, I am going to go ahead and close this issue and will post any update I have here. Thanks!
Bill

@DEberhardt
I have opened a work item for this so that engineering can take a look and possibly address this. Unfortunately, these examples are coded in the source and generated automatically, so any updates would get overwritten once we update this again. We need to get engineering to change this in the code. At this point, I am going to go ahead and close this issue and will post any update I have here. Thanks!
Bill

Thank you, Bill.
I've also created a UserVoice entry for it - albeit a more generic one pertaining to the PowerShell CmdLets for Teams:
https://microsoftteams.uservoice.com/forums/555103-public/suggestions/42449707-powershell-commands-for-contact-objects-common-ar


KCSD implentation

Here is kernel-CSD implementation for 1D, 2D, and 3D cases of electrode configurations. Also included is MoIKCSD (See Ness et.al, 2015).
The kCSD used here is a mirror of https://github.com/Neuroinflab/kCSD-python/tree/elephant
Further demos on how to use kCSD is illustrated here https://github.com/Neuroinflab/kCSD-python (see tests folder).

Coverage decreased (-27.1%) to 65.258% when pulling abf0986ae56d8dfa059a52b91044795a2be6b9ff on ccluri:feature/kcsd2d into 5ba20851ea4c85e76680b2b0b896909dbe275e49 on NeuralEnsemble:master.

@ccluri thanks for the PR. Can you check your code for Python 3 compatibility?

Coverage decreased (-27.1%) to 65.251% when pulling 09597006e99b3df86ad83c9a6ebf3e9763469de8 on ccluri:feature/kcsd2d into 5ba20851ea4c85e76680b2b0b896909dbe275e49 on NeuralEnsemble:master.

Do not forget to add your names into the authors list. See elephant/doc/authors.rst.

Thanks for the feedback. I made the suggested changes.

Coverage decreased (-27.2%) to 65.143% when pulling 9b7540f22e0878f9852f1030a76d9b75bee518be on ccluri:feature/kcsd2d into 5ba20851ea4c85e76680b2b0b896909dbe275e49 on NeuralEnsemble:master.

@ccluri can you check for the conflicts, that appeared after last merge (561ff0fd94282085ae1f7b752390e1d732bfa560)

@alperyeg : If we can have a consensus on the design aspect of CSD methods, I will go ahead and make necessary changes, while including iCSD.
@espenhgn : perhaps you can review / comment on my implementation (specifically csd.py) - as I feel this is more generic, and can include your iCSD and related methods as well, and is extend-able to other CSD implementations such as iCSD 2D, 3D - which are not available at the moment.
One critical difference I see, is the way quantities units are handled - I simply pad them in csd.py and abstain from using units in kCSD. But you seem to use units in your icsd.py

Hi @ccluri,
It looks good, but I'd wait for @alperyeg to merge pull #94 and try and resolve the conflict(s) that may have incurred since they merged #55.
I would fully support having one unified interface to the different CSD estimation methods that may also be extended to newer methods in the future (like the recent qCSD and rCSD methods by Kropf&Shmuel in Neural Computation, DOI:10.1162/NECO_a_00846). If you can do that it would be great.
Whether or not the codes should use quantities internally like I did is not my decision to make, in my experience certain numpy operations stripped the physical units which led to pretty ugly code in places, as well as slowing things down quite a bit because of testing for "compatible" units. I think with proper testing with different dimensionality in the input (lfp in V, mV, uV, etc.) problems can be identified.

@ccluri  I already merged pull https://github.com/NeuralEnsemble/elephant/pull/94. You can have a look at the conflicts without further delay :)

@alperyeg Currently I am writing my thesis, and will work on this in the second week of September.

Differences between the two implementations (PART 1)

Folder organization
In iCSD: elephant/elephant/icsd contains the relevant file - icsd.py, also includes a .mat file for testing(?), and a demo_icsd.py file.
in kCSD elephant/elephant/current_source_density has the relevant files for kcsd KCSD.py, basic_functions.py and a generic utility_functions.py files.

estimate_csd
In iCSD: elephant/elephant/csd.py  fetches  inputs parameters,
from .icsd.icsd import estimate_csd
def estimate_csd(lfp, coord_electrode, sigma, method='standard', diam=None,
                         h=None, sigma_top=None, tol=1E-6, num_steps=200,
                         f_type='identity', f_order=None):

In kCSD: elephant/elephant/csd.py is generic for 1D, 2D and 3D cases of kCSD, the equivalent of the estimate_csd of icsd seems to be function CSD in csd.py ; this also includes fake FWD model generation, and some out of place generate electrode positions.
Python     from elephant.current_source_density import KCSD     def CSD(analog_signals, coords=None, method=None, params={}, cv_params={}):     def FWD(csd_profile, ele_xx, ele_yy=None, ele_zz=None, xlims=[0.,1.], ylims=[0.,1.], zlims=[0.,1.], res=50):

units
In iCSD:  is unit conscious from the start; the methods implemented, this also checks for some patches related to SI unit for Siemens.
In kCSD: adds units only superficially in the csd.py file.

base CSD class
In icsd.py : The base icsd CSD class,  has functions get_csd, and filter_csd which 'Spatial filtering of the CSD estimate, using an N-point filter' (?)
In KCSD.py : the base kCSD CSD class validates the inputs of potentials, and electrode positions - and a sanity check (un-used) which quantifies the accuracy of the estimated csd with ground truth.

tests cases
In iCSD: test/test_icsd.py  has functions here to obtain the potentials in different planes , disks, cylinder which is then used by the test cases. assertion is based on assert_array_almost_equal between true csd and csd obtained - slightly different tests
In KCSD : test/test_kcsd.py uses FWD function from csd.py
  pots = CSD.FWD(self.csd_profile, self.ele_pos)

Tests are about method initialization, and the assertion is based on computation of RMSE error.

PART 2
https://github.com/NeuralEnsemble/elephant/blob/master/elephant/icsd/icsd.py#L830
this should be self._f_mat1

Coverage decreased (-1.04%) to 88.908% when pulling 70aacdf5bf8087d65e6a4f627f93fcf9b3dec0c1 on ccluri:feature/kcsd2d into 7c37aa8f971709c62e58716caae431e04eafed69 on NeuralEnsemble:master.

Coverage decreased (-1.02%) to 88.933% when pulling 971ff0c6654bced40f411240828223cb12bcfd01 on ccluri:feature/kcsd2d into 7c37aa8f971709c62e58716caae431e04eafed69 on NeuralEnsemble:master.

Coverage increased (+0.3%) to 90.223% when pulling f3a8d8982b16cae0a0c517cca4305557dfcc5c4d on ccluri:feature/kcsd2d into 7c37aa8f971709c62e58716caae431e04eafed69 on NeuralEnsemble:master.

Coverage increased (+0.3%) to 90.223% when pulling f3a8d8982b16cae0a0c517cca4305557dfcc5c4d on ccluri:feature/kcsd2d into 7c37aa8f971709c62e58716caae431e04eafed69 on NeuralEnsemble:master.

Coverage increased (+0.3%) to 90.223% when pulling f3a8d8982b16cae0a0c517cca4305557dfcc5c4d on ccluri:feature/kcsd2d into 7c37aa8f971709c62e58716caae431e04eafed69 on NeuralEnsemble:master.

@espenhgn Can you check if the changes are okay for you? Its good to go from my side.

Coverage increased (+0.7%) to 90.651% when pulling 2a7a01c7e2389a4d9b9be1cf08370822d400e958 on ccluri:feature/kcsd2d into 7c37aa8f971709c62e58716caae431e04eafed69 on NeuralEnsemble:master.

@alperyeg, I'm supporting merging pull request.

@ccluri: Sorry for the delay.
Michael (@mdenker)  and me had a short look. The code looks very good! Just some minor things we addressed.
Can you also rename the file name of csd.py to current_source_density.py and the directory to current_source_density_src (+change everywhere accordingly) to stay conform within Elephant and to be more readable? Thank you!

Hi @alperyeg @mdenker thanks for the feedback!
I commented the main function in current_source_density.py file - but I can remove it if you need it that way.
Other than that, I made the suggested changes. Let me know if I need to make any other tweaks. Cheers!

Coverage increased (+1.09%) to 91.042% when pulling ea47d45a871ad131b9ce06810c3f869b1f83abef on ccluri:feature/kcsd2d into 7c37aa8f971709c62e58716caae431e04eafed69 on NeuralEnsemble:master.

Very nice! Yes, please remove the commented lines. Thank you very much.

Coverage increased (+1.09%) to 91.042% when pulling 77e7eae4b8c9b333026d4b5e24043f7c74fadc6f on ccluri:feature/kcsd2d into 7c37aa8f971709c62e58716caae431e04eafed69 on NeuralEnsemble:master.

Thanks! Cheers!

Tx!


Creation of custom captions

Hi, Could you pls explain/share code that how could I create the captions encoding ".t7 files" for the custom captions I have for my images.

@abhinav3 We are not responsible for creating *.t7 files. We only convert existing *.t7 files to our format. If you are going to prepare your own dataset for training, you don't need to create *.t7 files. I would suggest that you directly convert your dataset into our format. Please refer to our code. In the code, datum.char is caption and datum.img is path of image file. For more details of the variables, please also refer to one of *.t7 files.

Thanks @woozzu for quick revert. I have gone through a sample "*.t7"  file.
The datum dictionary has four keys as show below : {'char': tensor([[20, 20, 20,  ..., 20, 20, 20],         [ 8,  8,  8,  ...,  8,  8,  8],         [ 5,  9,  5,  ...,  9,  9,  9],         ...,         [ 0,  0,  0,  ...,  0,  0,  0],         [ 0,  0,  0,  ...,  0,  0,  0],         [ 0,  0,  0,  ...,  0,  0,  0]], dtype=torch.uint8), 'img': 'jpg/image_06734.jpg', 'txt': tensor([[ 0.0664, -0.0793,  0.0847,  ...,  0.0018,  0.1846,  0.0951],         [ 0.0192, -0.0550, -0.1423,  ..., -0.0616,  0.0886,  0.0831],         [ 0.0400, -0.0200,  0.1444,  ...,  0.0043,  0.1317, -0.0429],         ...,         [ 0.1213,  0.0238, -0.0126,  ..., -0.1251,  0.1515, -0.0064],         [-0.0198, -0.0224, -0.0541,  ..., -0.1071,  0.0768,  0.0258],         [ 0.0940, -0.0376, -0.0518,  ..., -0.1196,  0.0490,  0.0206]]), 'word': tensor([[ 292, 5383,  292,  292, 5383, 5383,  851, 5383, 5383, 5383],         [  36, 2805, 1870, 4852, 2805, 2805, 2805, 2805, 2805, 2805],         [4630,  732, 5253,   36, 2373, 2373, 1285, 2373, 2373, 2373],         [ 292, 4608, 4630,  611, 5230,   36, 4740,  408, 5230, 4852],         [2805, 2707,  292, 2466, 4608, 1121, 2707, 4608, 4720,   36],         [4476,  381, 5363, 4630,   36, 4476, 2441,   36, 4608, 1034],         [4608,  890, 4852,  381, 2707, 4608,   36, 1285,   36, 3820],         [ 890, 2085,   36, 1285,  851, 2707, 1121,  851, 1285, 1034],         [2085, 1285, 1938,  381,  381, 2373,  732,  999, 2589,  851],         [2707,   36, 1035, 4450, 1608,  851, 1907, 5231, 1972,  381],         [ 611, 1121,  292, 2707,  890,  999, 4852,    1, 2707, 3976],         [ 851, 4476, 4164,  108,  292, 3976,    1,    1, 4852,    1],         [ 999,  793, 3976,    1, 5231,    1,    1,    1, 5026,    1],         [5231, 1444, 2707,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,  108,    1,    1,    1,    1,    1,    1,    1],         [   1,    1, 2707,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,  292,    1,    1,    1,    1,    1,    1,    1],         [   1,    1, 3748,    1,    1,    1,    1,    1,    1,    1],         [   1,    1, 5323,    1,    1,    1,    1,    1,    1,    1],         [   1,    1, 1608,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,  732,    1,    1,    1,    1,    1,    1,    1],         [   1,    1, 1598,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,    1,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,    1,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,    1,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,    1,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,    1,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,    1,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,    1,    1,    1,    1,    1,    1,    1,    1],         [   1,    1,    1,    1,    1,    1,    1,    1,    1,    1]],        dtype=torch.int16)}
As you suggest that datum.char is caption and datum.img is path of image file, which are input to the TAGAN model.
I got confused for datum.txt and datum.word. If you are aware of usage for these keys, could you pls help us understand.
Thanks, Abhinav

@abhinav3 Both datum.txt and datum.word are pre-computed embedding vectors used in the original paper where these *.t7 files come from. As far as I know, txt is character-based embedding vectors and word is word-based embedding vectors. In our paper, we do not use these vectors to deal with new sentences that are not included in the dataset.

Thanks a lot for the explanation!


feat: increase maximum number of rows in editor

Fixes #1500
12 rows might be too little for some users.
Changes

Set a maximum number of rows in the editor to 30.

I wonder if is more easy to use this field when it take more space than the window height. Isn't making UX worst? I would be ok to have it bigger within the window height.

If the user wants to make it longer, bad UX would be not to let the user do it.

But the thing is that you cant keep it smaller, ideally user should be able to set the height he want and we should keep this height for him.

Oh, I got what you mean. It resizes automatically as user types to the max size.
I think it might make sense to don't use autoResize feature at all and just let the user resize it themselves.

Right @Sekhmet


Allow additional options to GRPC server

Description of change
This adds a way to add additional GRPC Server options with FX without the need to modify platform code. This should be useful when adding custom interceptors to the main platform GRPC server.
Checklist

[x] Tested in playground or other setup
[ ] Screenshot (Grafana) from playground added to PR for 15+ minute run

This change is


Main novel list wont download

When downloading the list I get error code 502 but only for the main novels, the other categories work just fine

It appears to have fixed itself


keyed with cache directive causing Uncaught (in promise) TypeError: partValue.pop is not a function

Which package(s) are affected?
Lit Core (lit / lit-html / lit-element / reactive-element)
Description
I have a scenario where I'm trying to cache something that is rendered based on a provided key. If the key changes, I want the cached object to be re-rendered. If the key is a certain value, I want to render something different. When going from rendering the cached object to rendering something different, an uncaught exception is logged in the console, causing unexpected behavior on the rendered page.
The exception I see in the console:
Uncaught (in promise) TypeError: partValue.pop is not a function
    at CacheDirective.update (cache.js:27:41)
    at CacheDirective._$resolve (directive.js:42:21)
    at resolveDirective (lit-html.js:659:57)
    at resolveDirective (lit-html.js:659:17)
    at ChildPart._$setValue (lit-html.js:823:17)
    at TemplateInstance._update (lit-html.js:737:26)
    at ChildPart._commitTemplateResult (lit-html.js:984:35)
    at ChildPart._$setValue (lit-html.js:847:18)
    at render (lit-html.js:1459:10)
    at MyElement.update (lit-element.js:120:28)

Reproduction
The text should change to This is a bad key but it doesn't.
https://lit.dev/playground/#sample=examples%2Fdirective-cache&project=W3sibmFtZSI6Im15LWVsZW1lbnQudHMiLCJjb250ZW50IjoiaW1wb3J0IHtMaXRFbGVtZW50LCBodG1sfSBmcm9tICdsaXQnO1xuaW1wb3J0IHtjdXN0b21FbGVtZW50LCBwcm9wZXJ0eX0gZnJvbSAnbGl0L2RlY29yYXRvcnMuanMnO1xuaW1wb3J0IHtjYWNoZX0gZnJvbSAnbGl0L2RpcmVjdGl2ZXMvY2FjaGUuanMnO1xuaW1wb3J0IHtrZXllZH0gZnJvbSAnbGl0L2RpcmVjdGl2ZXMva2V5ZWQuanMnO1xuXG5AY3VzdG9tRWxlbWVudCgnbXktZWxlbWVudCcpXG5leHBvcnQgY2xhc3MgTXlFbGVtZW50IGV4dGVuZHMgTGl0RWxlbWVudCB7XG4gIFxuICBAcHJvcGVydHkoe3R5cGU6IE51bWJlcn0pXG4gIGtleSA9IDE7XG5cbiAgcmVuZGVyKCkge1xuICAgIHJldHVybiBodG1sYFxuICAgICAgPGgzPmNhY2hlIGRpcmVjdGl2ZSBleGFtcGxlPC9oMz5cblxuICAgICAgPGJ1dHRvbiBAY2xpY2s9JHt0aGlzLnRvZ2dsZVZpZXd9PkluY3JlbWVudCBrZXk8L2J1dHRvbj48aHI-XG5cbiAgICAgICAgXG4gICAgICAke2tleWVkKHRoaXMua2V5LCBjYWNoZSh0aGlzLnJlbmRlclZpZXcoKSkpfVxuICAgIGA7XG4gIH1cbiAgXG4gIHJlbmRlclZpZXcoKSB7XG4gICAgaWYgKHRoaXMua2V5ID09PSAzKSB7XG4gICAgICAvKipcbiAgICAgIENhdXNlcyBcbiAgICAgIFVuY2F1Z2h0IChpbiBwcm9taXNlKSBUeXBlRXJyb3I6IHBhcnRWYWx1ZS5wb3AgaXMgbm90IGEgZnVuY3Rpb25cbiAgICAgICAgYXQgQ2FjaGVEaXJlY3RpdmUudXBkYXRlIChjYWNoZS5qczoyNzo0MSlcbiAgICAgICAgYXQgQ2FjaGVEaXJlY3RpdmUuXyRyZXNvbHZlIChkaXJlY3RpdmUuanM6NDI6MjEpXG4gICAgICAgIGF0IHJlc29sdmVEaXJlY3RpdmUgKGxpdC1odG1sLmpzOjY1OTo1NylcbiAgICAgICAgYXQgcmVzb2x2ZURpcmVjdGl2ZSAobGl0LWh0bWwuanM6NjU5OjE3KVxuICAgICAgICBhdCBDaGlsZFBhcnQuXyRzZXRWYWx1ZSAobGl0LWh0bWwuanM6ODIzOjE3KVxuICAgICAgICBhdCBUZW1wbGF0ZUluc3RhbmNlLl91cGRhdGUgKGxpdC1odG1sLmpzOjczNzoyNilcbiAgICAgICAgYXQgQ2hpbGRQYXJ0Ll9jb21taXRUZW1wbGF0ZVJlc3VsdCAobGl0LWh0bWwuanM6OTg0OjM1KVxuICAgICAgICBhdCBDaGlsZFBhcnQuXyRzZXRWYWx1ZSAobGl0LWh0bWwuanM6ODQ3OjE4KVxuICAgICAgICBhdCByZW5kZXIgKGxpdC1odG1sLmpzOjE0NTk6MTApXG4gICAgICAgIGF0IE15RWxlbWVudC51cGRhdGUgKGxpdC1lbGVtZW50LmpzOjEyMDoyOClcbiAgICAgICovXG4gICAgICByZXR1cm4gaHRtbGBUaGlzIGlzIGEgYmFkIGtleWA7IC8vIGNhdXNlcyBVbmNhdWdodCAoaW4gcHJvbWlzZSkgVHlwZUVycm9yOiBwYXJ0VmFsdWUucG9wIGlzIG5vdCBhIGZ1bmN0aW9uXG4gICAgfVxuICAgcmV0dXJuIGh0bWxgS2V5IFZhbHVlOiAke3RoaXMua2V5fWA7IFxuICB9XG5cbiAgcHJpdmF0ZSB0b2dnbGVWaWV3KCkge1xuICAgIHRoaXMua2V5Kys7XG4gIH1cbn1cbiJ9LHsibmFtZSI6ImluZGV4Lmh0bWwiLCJjb250ZW50IjoiPHN0eWxlPlxuICBib2R5IHsgZm9udC1mYW1pbHk6IHNhbnMtc2VyaWY7IH1cbjwvc3R5bGU-XG5cbjxzY3JpcHQgdHlwZT1cIm1vZHVsZVwiIHNyYz1cIi4vbXktZWxlbWVudC5qc1wiPjwvc2NyaXB0PlxuXG48bXktZWxlbWVudD48L215LWVsZW1lbnQ-XG4ifSx7Im5hbWUiOiJwYWNrYWdlLmpzb24iLCJjb250ZW50Ijoie1xuICBcImRlcGVuZGVuY2llc1wiOiB7XG4gICAgXCJsaXRcIjogXCJeMi4wLjBcIixcbiAgICBcIkBsaXQvcmVhY3RpdmUtZWxlbWVudFwiOiBcIl4xLjAuMFwiLFxuICAgIFwibGl0LWVsZW1lbnRcIjogXCJeMy4wLjBcIixcbiAgICBcImxpdC1odG1sXCI6IFwiXjIuMC4wXCJcbiAgfVxufSIsImhpZGRlbiI6dHJ1ZX1d
Workaround
I have not found a workaround.
Is this a regression?
No or unsure. This never worked, or I haven't tried before.
Affected versions
2.2.8
Browser/OS/Node environment
Browser: Chrome: 109.0.5414.120
OS: Windows 10
Node: v16.14.0
NPM: 8.3.1

im gonna guess its because if you have a directive in a directive, the part is the overall outer part:
html`
  ${directiveA(directiveB('foo'))}
`;

would have one Part, a ChildPart which is the same one passed to both directives.
so this comment:
https://github.com/lit/lit/blob/e51ff229843aaac22805bc0a39bf5e6db6862a5c/packages/lit-html/src/directives/cache.ts#L50-L51
isn't true since the part could have already been through a parent directive at that point, resulting in it not being an array.
no clue how this'd be fixed but that seems to be why it errors for anyone who looks into this


Rendering of shortcodes with underscores in their names is broken

The Using shortcodes section in the docs says:

Lastly, a shortcode name (and thus the corresponding .html file) as well as the arguments name can only contain numbers, letters and underscores, or in Regex terms the following: [0-9A-Za-z_].

Taking the 'quote' shortcode example from that page, putting the body on a new line to work around issues #469, and then renaming the shortcode to "quote_a_human", I then get the output:
As someone said:

{% quote_a

A quote
-- Vincent

This is a bit weird because the shortcode does work...it just also outputs some of the shortcode markup itself, up to and including the last underscore in the shortcode name, into the rendering.
Furthermore, if I insert a line break into the shortcode reference as so:
{% quote_a_human(
    author="Vincent") %}
A quote
{% end %}

Then the output becomes:
As someone said:

{% quote_a_human(

A quote
-- Vincent

(I'd like to be able to put line breaks in because in the actual shortcode I'm writing there are several arguments to the shortcode with long values, and it makes the shortcode reference in the markdown file easier to read.)

Looks like the shortcode parser still has lots of bugs :(

Bah. See issues 469. Turns out the gutenberg build that I was creating when rebuilding wasn't the one my script was installing (release/debug confusion). So this particular issue seems to be an old bug that is fixed in current master.
Appologies for wasting your time.


Unable to query for value presence

Hi, if I query for a TypedName value I'm not able to use the prettyPrint() method anymore. It seems that querying changes the asn1 object state.
The following code (extracted by the ldap3 library) doesn't work as intended:

class LDAPOID(univ.OctetString):
    # LDAPOID ::= OCTET STRING -- Constrained to <numericoid>
    pass

class RequestName(LDAPOID):
    # requestName      [0] LDAPOID
    pass

class RequestValue(univ.OctetString):
    # requestValue     [1] OCTET STRING
    encoding = 'utf-8'

class ExtendedRequest(univ.Sequence):
    # ExtendedRequest ::= [APPLICATION 23] SEQUENCE {
    #     requestName      [0] LDAPOID,
    #     requestValue     [1] OCTET STRING OPTIONAL }
    componentType = namedtype.NamedTypes(namedtype.NamedType('requestName', RequestName()),
                               namedtype.OptionalNamedType('requestValue', RequestValue()))

e = ExtendedRequest()
print(e.prettyPrint())
# output is
# ExtendedRequest:
# empty object (as expected)

e['requestName'] = '<IP_ADDRESS>.4.1.4<IP_ADDRESS>'
print(e.prettyPrint())
# output is
# ExtendedRequest:
#  requestName=<IP_ADDRESS>.4.1.4<IP_ADDRESS>  (as expected)

# now somewhere in the code I want to check if requestValue is present:
if e['requestValue'].hasValue():
    pass  # do something

# now I cannot use prettyPrint() anymore
print(e.prettyPrint())

I get an exception:
 File "test.py", line 35, in <module>

    print(e.prettyPrint())
  File "C:\Python\Python37\lib\site-packages\pyasn1\type\univ.py", line 2467, in prettyPrint
    representation, componentValue.prettyPrint(scope)
  File "C:\Python\Python37\lib\site-packages\pyasn1\type\univ.py", line 926, in prettyPrint
    numbers = self.asNumbers()
  File "C:\Python\Python37\lib\site-packages\pyasn1\type\univ.py", line 896, in asNumbers
    return tuple(self._value)
  File "C:\Python\Python37\lib\site-packages\pyasn1\type\base.py", line 199, in plug
    raise error.PyAsn1Error('Attempted "%s" operation on ASN.1 schema object' % name)
pyasn1.error.PyAsn1Error: Attempted "__iter__" operation on ASN.1 schema object

This worked in previous pyasn1 packages where I just checked for e['requestName'] without the hasValue() method.
What am I doing wrong?

Hey, Giovanni!
I think you run into a bug in pyasn1! I hope it is fixed by commit 530dbaf32d2f40068e05933480d93df3bb3c5854.
Could you please try current master, if it works for you I will make a release with this fix right away.
Cheers!

Hi Ilya, I've tested the latest version in master and it works again. Thanks!

Release 0.4.4 is out on PyPI.
Thanks for the heads up! ;-)


ThomasAwesome shuffle mobbers

This is an attempt to revive #18 which is a great feature.
As a user of the app
In order to mix things up
And not having the same person that I am taking over from
And not having the same person that I am handing over to
And not having to remove and re-add mobbers to achieve this
I want to be able to shuffle the order of the mobbers

The placement of the button is kind of bugging me, and I'm trying to decide if it's worth changing.
In the current placement, the shuffle button looks like it's the first mobber on the list (especially with the gray striping). If you have mobbers already, then maybe that's where you want the button. But if there aren't any mobbers yet then does it distract from adding them?
Maybe the button should go at the bottom, under the "Add" form? If you have a bunch of mobbers it would push the button down so you have to scroll though. But overall, I'm thinking I like it under better than on top... thoughts?

I tried both and settled on above, but I am okay either way. One idea could be to keep it where it is but disable/hide the shuffle if there are no mobbers

After trying it a few ways, I feel like the top placement improves without the gray background. The striping feels like it's part of the list of mobbers. But when it's white, it blends back into the page background and the first mobber has the gray background.
Only showing it for 2+ mobbers is an interesting idea. It gets it out of the way when you can't really use it and stops it from distracting new users from adding their mobbers. But then it may not be obvious why it's appearing / disappearing. Disabling for < 2 mobbers is a nicer touch, but doesn't solve the placement problem. /shrug
I'll try to get some other people to voice their opinions on the placement.

I definitely don't like the gray striping, but I'm not a fan of the while either as there's no separation from this button's row and the mobber list. Is there another color that fits the UX theme that we can use to help distinguish this as distinct from the mobber list but also not bland?

Moved shuffle button below "add mobber" form
Renamed class mobberActions to mobberListActions to signal actions on the list as a whole
Attempted writing a test in timer-state.spec.js, mobbers dependency is not easily stubable with the current code. Not writing a test at this time but I think testing with a mocked mobbers is a good idea once that is possible.

Fair enough re: the test. I'll take a stab at it post-merge.


Chaining move directions. When leaving a node, should be possible to query if it has any instruction on where to go next

yeah that works


Swap out YAML and other config loaders with Lua 5.4 compatible ones

Currently there are some optional dependencies including on yaml that are not Lua 5.4 compatible. These should be made easier to swap out for a loader of choice including a YAML reader that works in Lua 5.4, or works with pure Lua, or whatever.

That module has gone unmaintained, the more common one is https://github.com/gvvaughan/lyaml I think. Not sure it supports Windows though.


[New game]: LONELY SOUL 👾

🎮 Game Request
Game logic and basic description

You are a lonely ghost in a dark cave.
Avoid spikes and holes by jumping while light wheakens.

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @0xabdulkhalid !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @0xabdulkhalid!
We are already having a similar game request in #1061 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.

Hey @0xabdulkhalid ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇