Check ability to map SNOMED codes to positive and negative values in the UP

@victor-chaparro TO ADD DETAILS

We have a lookup table for this, but it seems to be set up more to map a result to a snomed code than the other way around as there are multiple results listed for a snomed (e.g. "Neg", "Negative", "Not Detected" all map to the same snomed "260385009")

I believe those lookup tables are not in the database and only stored in the repo. The UP doesn't have a way to read those value sets, but it looks like this might actually work with the code to condition table. So all we would need to is to add the values in the lookup table there. This would only be needed once we add CSV support if we decide to do, because it looks like this was only used for CSV senders?

They're in the database, that's where I looked for them....

When I was looking at this I was specifically looking at the Obx8Mapper used in the covid pipeline.
- name: abnormal_flag
    type: CODE
    valueSet: hl70078
    natFlatFileField: Abnormal_flag
    hhsGuidanceField:
    hl7Field: OBX-8
    mapper: obx8(test_result)

 when (values[0].value) {
                    "260373001" -> "A" // Detected
                    "260415000" -> "N" // Not Detected
                    "720735008" -> "A" // Presumptive positive
                    "42425007" -> "N" // Equivocal
                    "260385009" -> "N" // Negative
                    "10828004" -> "A" // Positive
                    "895231008" -> "N" // Not detected in pooled specimen
                    "462371000124108" -> "A" // Detected in pooled specimen
                    "419984006" -> "N" // Inconclusive
                    "125154007" -> "N" // Specimen unsatisfactory for evaluation
                    "455371000124106" -> "N" // Invalid result
                    "840539006" -> "A" // Disease caused by sever acute respiratory syndrome coronavirus 2 (disorder)
                    "840544004" -> "A" // Disease caused by severe acute respiratory coronavirus 2 (situation)
                    "840546002" -> "A" // Exposure to severe acute respiratory syndrome coronavirus 2 (event)
                    "840533007" -> "A" // Severe acute respiratory syndrome coronavirus 2 (organism)
                    "840536004" -> "A" // Antigen of severe acute respiratory syndrome coronavirus 2 (substance)
                    "840535000" -> "A" // Antibody to severe acute respiratory syndrome coronavirus 2 (substance)
                    "840534001" -> "A" // Severe acute respiratory syndrome coronavirus 2 vaccination (procedure)
                    "373121007" -> "N" // Test not done
                    "82334004" -> "N" // Indeterminate
                    else -> null
                }

The SNOMED codes are hardcode there, but it looks like we can actually use a lookup table instead, not sure why the covid pipeline is not doing that.

Probably because, like I said, the lookup table is set up to map one SNOMED code to multiple "display" values, not the other way around.... Although it seems like it shouldn't be hard to add the normal/abnormal flag to the test result lookup. Not sure if there's something I'm missing there.

I'm also not sure if we're missing anything. That's the purpose of the ticket, but we can revisit this when we have time.

I'm like 99% sure that if we added a "flag" value for A/N to the test_result lookup table this would work fine in UP. I'm just not sure if adding that would break the lookups that use that table in the Covid pipeline.


Fix spawn signal

Spawn is emitted only when update_health packet is received (https://github.com/andrewrk/mineflayer/blob/master/lib/plugins/health.js#L22) but it seems this packet is not received when the bot connects anymore.
How should we fix this ?

fixing the position sending apparently fixed this too (!)

cdd99349bdc6e286537bf5358818ebbd3f4066be solved this


Crimbo toy lab

Added Crimbo Toy Lab
Configured Choice Spoilers and material consumption to choice options
Added new advent calendar item
Ran "checkpowers" and fixed a bunch of accessory powers
ChoiceAdventure configuration contains a "zone", not an "adventure location" as its first argument. I fixed a bunch

I'm working on supporting multiple optional prizes from a choice adventure. Hang on.

OK, this should be good to go.


MRG: FIX: Specify exact array indicies for 3D landmark positions

Reference issue
Fixes https://github.com/mne-tools/mne-nirs/issues/393
What does this implement/fix?
The SNIRF specification has an optional 4th column in the 3D positions for landmarks. Previous vendor software did not use this column, and so we just took the whole array. But now the NIRx vendor appears (yet to be confirmed) that they are using this column. As such we need to address the first 3 array values not :.
I have a file from a user that now works with this fix. But it’s not appropriate to use as test data.
Additional information
I have emailed NIRx to ask for a test file as I dont have access to one of these machines to provide test data. But if the change doesn't break old test data, and the change makes sense relative to the specification, then I suggest we can merge without test data, and I will add it when I get my hands on a reasonable test file.
Specification
Here is the relevant part of the specification taken from https://github.com/fNIRS/snirf/blob/master/snirf_specification.md#nirsiprobelandmarkpos3d

This is a 2-D array storing the neurological landmark positions measurement from 3-D digitization and tracking systems to facilitate the registration and mapping of optical data to brain anatomy. This array should contain a minimum of 3 columns, representing the x, y and z coordinates (in LengthUnit units) of the digitized landmark positions. If a 4th column presents, it stores the index to the labels of the given landmark.

@larsoner @agramfort this is ready for review

thx @rob-luke

Thanks @agramfort


namelines in in ListOfParties, version 2.0.0

minLength should be changed from 2 to 1.
nameLines:
type: array
minItems: 1
items:
type: string
minLength: 2

Yes, this is clearly a typo in 1 of the occurrence of nameLines out of 4, I will fix it.

Done, see commit e1a22e3.


python3Packages.jurigged: init at 0.4.1

Motivation for this change
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 156598 run on x86_64-linux 1

2 packages failed to build:

python310Packages.jurigged
python39Packages.jurigged

10 packages built:

python310Packages.codefind
python310Packages.giving
python310Packages.hrepr
python310Packages.ovld
python310Packages.varname
python39Packages.codefind
python39Packages.giving
python39Packages.hrepr
python39Packages.ovld
python39Packages.varname


GDXDSD-5197 Updated googlesearch.py to  remove adding  new line character in report()

Some active code was commented out, new call  to report added. Will need to fix before full pull  request.

PR on hold until appropriate changes are made to google_mybusiness.py for similar issue.

Please see comment on https://apps.itsm.gov.bc.ca/jira/browse/GDXDSD-5197 if you have any questions on how to test these changes.


Per Token Expiration Doesn't Work

My setup:
Using Iron Router. On my route controller I have an onBeforeAction:
`if (!Meteor.user()) {
var token = Router.current().params.token;
  if (token) {
    LoginLinks.connectionLogin(token, function(e, r) {
      if (!e) {
        return console.log(r);
      }
      return console.log(e);
    });
  }`

On my route I use:
Router.route('/fillerRoute/fillerRoute/:token', {
  name: 'fillerTemplate',
  controller: UserController,
});

Meteor method to generate token:
var token = LoginLinks.generateAccessToken(user, {expiresInSeconds: 2 * 60});
switch (type) {
  case case1:
    return 'https://fillerAddress.com/fillerRoute/fillerRoute/' + token;

My issue here is that the login links never expire. When I console log out the result (in this case "r"), I get back an object with the properties of
expiresInSeconds: 120,
hashedToken: "unused"
userId: "QqTi4rWPkipHPmZtb",
when: Tue Jul 26 2016 13:58:11 GMT-0700 (PDT)
When I utilize the set global expiration method or set types method the expiration dates work.
Also, I noticed that the login links work past one use, they only do not work when a token is expired. Is this the intended behavior? The description "The main use case is sending an email or sms to your user with a link to your app that contains an OTP (one-time password)" makes it seem like one user can use the link one time prior to the expiration point.
Thanks for putting together the package!

Hi! Thanks for reporting. Thought there was a test case that covered this,
but I'll look into it next week.
It is meant to be multi-use up until expiration, and I'll note this in the
readme
On Tuesday, July 26, 2016, Nicholas Pappagalo<EMAIL_ADDRESS>wrote:

My setup:
Using Iron Router. On my route controller I have an onBeforeAction:
if (!Meteor.user()) {
var token = Router.current().params.token;
if (token) {
LoginLinks.connectionLogin(token, function(e, r) {
if (!e) {
return console.log(r);
}
return console.log(e);
});
}
On my route I use:
Router.route('/fillerRoute/fillerRoute/:token', {
name: 'fillerTemplate',
controller: UserController,
});
Meteor method to generate token:
var token = LoginLinks.generateAccessToken(user, {expiresInSeconds: 2 *
60});
switch (type) {
case case1:
return 'https://fillerAddress.com/fillerRoute/fillerRoute/' + token;
My issue here is that the login links never expire. When I console log out
the result (in this case "r"), I get back an object with the properties of
expiresInSeconds: 120,
hashedToken: "unused"
userId: "QqTi4rWPkipHPmZtb",
when: Tue Jul 26 2016 13:58:11 GMT-0700 (PDT)
When I utilize the set global expiration method or set types method the
expiration dates work.
Also, I noticed that the login links work past one use, they only do not
work when a token is expired. Is this the intended behavior? The
description "The main use case is sending an email or sms to your user with
a link to your app that contains an OTP (one-time password)" makes it seem
like one user can use the link one time prior to the expiration point.
Thanks for putting together the package!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lorensr/login-links/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPVmBrf9sN4we0aNjzCVEuc9KBG-h03ks5qZoHkgaJpZM4JVocy
.

Sorry for the delay. I've updated the readme.
I've also added a new test for per-token expiration, which is passing.
https://github.com/lorensr/login-links/blob/master/tests/client/loginHandler.js#L42-L56
Let me know if you're able to modify the test to get it to fail, or can reproduce in a blank meteor repo.


Feature: Custom Command Shortcuts

Im still missing the possibility to create shortcuts / aliases for commands in generel.
In v1, this was possible, since it was 'global aliases'  eg.  ["brb", "switch brb"], ["mobile", "switch rtmp"]
so typing !brb would be an alias to: !switch brb
(I only think its necessary to make aliases for scene-switching, so there might be a more simple solution for that to keep it simple)

I second this, looking for this feature for a long time now


Question about the requirement of GPU

Hi! I wonder how many GPUs and what kind of GPUs are required in general weakly-supervised semantic segmentation task including your work? Plus as introduced in your paper, 1 or 2 TITAN 3090 GPU is enough, so how long does it take for a single group of experiment? Thanks!

One 3090 for S3DIS
Two 3090s for ScanNetV2
Four 3090 for SemanticKITTI

How long does it take for a single group of experiment for each dataset?

Thanks!


updated readme and unit conversion application

done , please check

The readme in still conflicting, please compare it with the latest repository.

changed


Implement routing for Linode detail page

Note that the components for each tab need to be added in
src/linodes/index.js now instead of directly in the detail page.
Closes #86.

This involves some hacks to get routing to make sense in the context of react-tabs.

Coverage seems to have been reduced because I added (and did not test) the placeholder code in index.js. Don't care.

Coverage decreased (-0.4%) to 75.487% when pulling 6799b2065a94bfe9dc87412b39aac618e4198a7c on feature/detail-page-routing into 02022c040600dc4d7a8ffb54678899b2120621d9 on master.

Coverage decreased (-0.4%) to 75.487% when pulling 6799b2065a94bfe9dc87412b39aac618e4198a7c on feature/detail-page-routing into 02022c040600dc4d7a8ffb54678899b2120621d9 on master.

Coverage decreased (-0.3%) to 75.63% when pulling 01e08dc7d1ef7b9e18ec4149e65e07d34d5af7c2 on feature/detail-page-routing into 02022c040600dc4d7a8ffb54678899b2120621d9 on master.

Coverage decreased (-0.04%) to 75.843% when pulling 01e08dc7d1ef7b9e18ec4149e65e07d34d5af7c2 on feature/detail-page-routing into 02022c040600dc4d7a8ffb54678899b2120621d9 on master.

Checks are green. Merge?

Looks good.

Thanks!

My explanation was wrong about this. So cleaning that up for my own sake.
this is bound to be the object who parents the function. Calling bind overwrites that. It is used in React when you are passing a callback (where this within the callback function is intended to be the parent) from a parent to a child. Calling the callback requires this literal this.callback() that would assign this within the callback to be the child if this were not bound in the parent level.
Since render is never called from any other parent except the class it exists in, this will never be bound to anything other than what you'd expect. That's why it is not necessary in this case (or ever, probably).


Edit README and Dockerfile for Improved Reproducibility

After running Kai locally, I suggest the following adjustments to these files to facilitate code reproduction for future users:

Remove blank spaces from bash commands to make them easier to copy and paste.
Change "ProjectID" in the Dockerfile to "Enter your project ID here" to clearly indicate that the user needs to modify this line.
Include examples of how to pass environment variables when running the container.

Added to STAGING branch


Migrated repository incompatible with --all flag

I need to create a repo like this scheme:
on BitBucket created a private repo gogs_test1
and cloned it to my server like
git clone --bare git@bitbucket.org:***/gogs_test1.git
and now, developers can push/pull to my local server (to this bare repo), and i can sync that local server with BitBucket without any problem like:
to push to bitbucket
git push --all
and if needed to sync from bitbucket
git fetch origin +*:* --prune
(pull/push could be done with hooks, its not a problem too)
But, if I create New Migration via web interface

and try to push it shows
git@mgit:/repos/miroff/testgo.git$ git push --all
error: --all and --mirror are incompatible

but:
git@mgit:/repos/miroff/testgo.git$ git remote -v
origin<EMAIL_ADDRESS>(fetch)
origin<EMAIL_ADDRESS>(push)

Thanks your feedback!
I understand the content of your issue, but what does this mean?

as real bare repo

this means that seem it works line --mirror, but alike, can push on it, but cant push to remote, strange.
It perfectly works if I create normal repo in gogs, than manually add remote (as I need, but not in interface).

git@mgit:/repos/miroff/testgo.git$ git push --all
error: --all and --mirror are incompatible

Based on the error message, the way Gogs migrate a repo has conflict with --all flag. I think this is the problem.

Hi there, I've pushed fix to develop branch, you can also test on https://try.gogs.io

Close as fixed.


CRM Backend

Implement a CRM Backend, where a User can Login and edit the content of the following pages:

News (LandingPage)
Services
Firma (Team Members)
Links

Login working, but with inheritance props, probably change this in the future to context


make it useable in xhtml documents

XHTML does not allow empty attributes
Produces errors like
Failed to set the 'innerHTML' property on 'Element': The provided markup is invalid XML, and therefore cannot be inserted into an XML document.

Note: attribute minimization in XHTML is described here : http://www.w3.org/TR/2000/REC-xhtml1-20000126/#h-4.5

Are you able to add a test for this please?


feature request: two-step (localized) hint mode

In my endless quest to optimize things, I spent some time thinking about how to improve hint mode to eliminate most of the additional normal-mode navigation needed after selecting a hint. One can of course counter this by increasing the number of hint keys, but this will likely never be satisfactorial, especially when decreasing the standard hint size (in addition, most non-letter keys are harder to reach). Of course, hint mode was not made for pixel precise movement, but it would be nice to have the ability of navigating on a sufficiently fine grid using hints only. At least for me, pressing one or two additional keys is much faster than doing the fine movement using normal mode (or a trackpoint when available).
To solve this problem, one could introduce a two-step hint mode, where the first step is the regular hint mode, and after selecting a hint, another local hint mode fires immediately in a rectangle around the previously selected hint. This rectangle should of course be reasonably sized, just covering the points closest to the center of the original hint - a mathematician would call this the Voronoi cell -  plus some overlap, as for points around the middle of a line connecting two hints, we often can't decide with our eyes to which hint they have a closer distance to. Also, it should have its own set of hint keys and all hints should be reachable with just one key instead of two (at least optionally), so that in total three keypresses suffice to move the cursor on a fine grid. Basically, this second step hint mode is a one-step local grid mode with configurable keys and grid size.
For example, for my setup with a (nowadays rather moderate) 2560x1440 resolution, the standard 26 letter keys as hint keys and a hint size of 16, another 5x5 grid in a second step would be perfect.
Another thing to ponder about would be the generation of the grid. Currently, the screen is partitioned into rectangles with the same aspect ratio as the screen. At least for the two-step mode above, it would maybe improve navigation to partition into squares (in both steps), for example by discarding some of the given hint characters when building the rows.
What do you think about this proposal?
If you like it, one could also recycle the idea and offer an option to enter local hint mode in a square previously selected by grid mode, although I don't know if this would be very useful.
PS: Another solution would of course be to just increase the number of keys used for a hint from two to three. While this would yield a grid as fine as the two-step solution, the hints would quickly take up too much horizontal screen space (i.e. could not be placed properly side by side). Also, while I've developed some sort of memory of the keys needed to reach a two-letter hint, I think that this effect would not be that prominent for the 26^3 three-letter hints. The two-step procedure makes this more managable, as only 26^2+25 combinations need to be memorized.

Thanks for the comprehensive feedback.

To solve this problem, one could introduce a two-step hint mode, where the first step is the regular hint mode, and after selecting a hint, another local hint mode fires immediately in a rectangle around the previously selected hint.

I have been considering experimenting with something like this, though I suspect in practice using hjkl for adjustment will still be preferable, since you can roughly plan in advance the keystrokes you need to select a target.
Gien the influx of interest, I plan on making some non-trivial internal changes to better accommodate the various platforms which will likely also involve experimenting with new modes.

a mathematician would call this the Voronoi cell - plus some overlap

Intriguing, I suppose there is a mathematical term for everything.

Also, it should have its own set of hint keys and all hints should be reachable with just one key instead of two (at least optionally). Basically, this second step hint mode is a one-step local grid mode with configurable keys and grid size.

This is how I also envisioned it. Something akin to the current screen selection mode, with a dedicated key set.

Currently, the screen is partitioned into rectangles with the same aspect ratio as the screen.
to partition into squares (in both steps), for example by discarding some of
the given hint characters when building the rows.

This was mainly done for convenience. Earlier versions of warpd allowed the user to explicitly specify the number of rows. I will probably reintroduce something like this at some point.

If you like it, one could also recycle the idea and offer an option to enter
local hint mode in a square previously selected by grid mode, although I don't
know if this would be very useful.

This seems a little too a la carte for my tastes :P. If you are using grid mode at all, you are probably doing something wrong (or using unorthodox hardware).  It mainly exists for legacy reasons.

Another solution would of course be to just increase the number of keys used for a hint from two to three.

The gaps are intentional, since they provide a degree of contextual awareness. One can also use transparency, but I find that the gap based approach is superior. It is possible to nearly saturate the screen using hints of size 2 if you make use of all available characters, but I explicitly advise against this in the man page.

What do you think about this proposal?

I agree with much of what you propose, and will likely end up making similar additions.

To solve this problem, one could introduce a two-step hint mode, where the first step is the regular hint mode, and after selecting a hint, another local hint mode fires immediately in a rectangle around the previously selected hint.

I have been considering experimenting with something like this, though I suspect in practice using hjkl for adjustment will still be preferable, since it allows the user to plan in advance the keystrokes needed to select a target.

As the placement of hint characters is deterministic, your remark is still true for the procedure you described. For example, if the local grid is created row wise from top left to bottom right, the local hint characters yuihjkbnm would create a 3x3 local grid where h, u, k move up, left, right and j would select the center, with additional characters available to move down and on the diagonals. Of course j could be placed so that it still goes down.

Having said that, I plan on making some non-trivial internal changes given the recent influx of interest, to better accommodate the various platforms and will likely experiment with new modes in the process.

Currently, the screen is partitioned into rectangles with the same aspect ratio as the screen. to partition into squares (in both steps), for example by discarding some of the given hint characters when building the rows.

This was mainly done for convenience. Earlier versions of warpd allowed the user to explicitly specify the number of rows. I will probably reintroduce something like this at some point.

Perfect!

If you like it, one could also recycle the idea and offer an option to enter
local hint mode in a square previously selected by grid mode, although I don't
know if this would be very useful.

This seems a little too a la carte for my tastes :P. If you are using grid mode at all, you are probably doing something wrong (or using unorthodox hardware). It mainly exists for legacy reasons.

Completely agree with your comment!

Another solution would of course be to just increase the number of keys used for a hint from two to three.

The gaps are intentional, since they provide a degree of contextual awareness. One can also use transparency, but I find that the gap based approach is superior. It is possible to nearly saturate the screen using hints of size 2 if you make use of all available characters, but I explicitly advise against this in the man page.

I agree that gaps should be present for awareness, just wanted to name this alternative to promote the two-step solution.

One more thought: If the two-step hint mode gets implemented, it would be nice to have shortcuts for activating it with an automatic left- or right-click after selection. This would further improve efficiency.

As the placement of hint characters is deterministic, your remark is still true for the procedure you described. For example, if the local grid is created row wise from top left to bottom right, the local hint characters yuihjkbnm would create a 3x3 local grid where h, u, k move up, left, right and j would select the center, with additional characters available to move down and on the diagonals. Of course j could be placed so that it still goes down.

Yes, but the grid would be local, which means the user would have to wait for the hints to appear, which introduces additional latency. It is true that they could memorize the relative positions, but I suspect this introduces more cognitive overhead than simply using the arrow keys for adjustment.
Having said that, I will probably implement it anyway :P.

One more thought: If the two-step hint mode gets implemented, it would be nice to have shortcuts for activating it with an automatic left- or right-click after selection. This would further improve efficiency.

This makes sense. I had a similar idea.

Yes, but the grid would be local, which means the user would have to wait for the hints to appear before being able to make a selection, which introduces additional latency. It is true that they could memorize the relative positions, but I suspect this introduces more cognitive overhead than simply using the arrow keys for adjustment.

Whether it creates more cognitive overhead is a very interesting question. What we're discussing here is essentially how to navigate a small rectangle in the most efficient way, and it might even not be possible to give a general answer as it could depend a lot on the individual user.
For me personally,  movement by typing is much faster than adjusting the cursor in normal mode using the arrow keys. I just did some experiments using my two favorite hated gui applications I'm forced to use at work and during the time need ed for cursor adjustments by a trackpoint or in normal mode after having selected a hint (on a 26x26 grid and 2560x1440 resolution) I could easily type two to four further memorized characters, with the average probably being closer to four. So if the local grid is fine enough to keep me from having to do further adjustments, that will definitely be an improvement, and a very substantial one.
Also, I don't think that one would have to wait for the local grid to appear before deciding where one would want to navigate. The size of this grid is constant, no matter where it appears, and after some time one would surely be able to know which point is desired while typing the letters for the first grid. I will probably start using the uiojklvm-keys for a local 3x3 grid (in the hope that it is fine enough), which are very suggestively placed under the right hand. For a 5x5 grid, one could again place j in the center and pan out in all directions.
The experiment also hints at another thing to try, which would not that hard to add once local hints are implemented: One could show a 3x3 grid in the second step and keep doing this until a given threshold grid resolution or given maximum step size has been reached, where probably just a third step would be useful in practice. One could also implement this in a way where shift-typing would end this process immediately and click.

Having said that, I will probably implement it anyway :P.

Perfect, can't wait to try it out.  There will always be use cases where a physical mouse/trackball/trackpoint can never be replaced, such as CAD applications, gaming or photo editing, but for the rest, warpd already surpassed those physical devices in the vast majority of cases and is on the way to increase its advantage even further!

as it could depend a lot on the individual user.

I suppose it depends on the size of the grid. A 3x3 grid might be manageable (and is effectively akin to adding diagonal directional keys), but my (completely unscientific :P) suspicion is that anything larger than that would add latency by increasing the amount of spatial reasoning necessitated by the user. This is ignoring the visual component altogether.

For me personally, movement by typing is much faster than adjusting the cursor in normal mode using the arrow keys.  I just did some experiments using my two favorite hated gui applications I'm forced to use at work and during the time need ed for cursor adjustments by a trackpoint or in normal mode after having selected a hint (on a 26x26 grid and 2560x1440 resolution) I could easily type two to four further memorized characters, with the average probably being closer to four. So if the local grid is fine enough to keep me from having to do further adjustments, that will definitely be an improvement, and a very
substantial one.

If you have to tap the directional keys more than once or twice, you are probably doing something wrong (not enough/poorly sized hints). Once you become accustomed to it, you should be able to pinpoint any region of the screen in 2-4 strokes.  I should probably make a video demonstrating my usage at some point.

Also, I don't think that one would have to wait for the local grid to appear before deciding where one would want to navigate.

As mentioned above, this might be true for small grid, but I think even those have an additional cognitive load associated with them. I was actually thinking about using all hint characters to maximize the number of targets, though it would probably make sense to make this configurable.

I will probably start using the uiojklvm-keys for a local 3x3 grid (in the hope that it is fine enough)

Mapping the keys to their corresponding keyboard positions in space makes sense, but I don't think it completely eliminates the overhead.

One could show a 3x3 grid in the second step and keep doing this until a given threshold grid resolution or given maximum step size has been reached

Adding multiple iterations seems counterproductive. The goal is for the user to be able to select any region of the screen with as few keystrokes as possible. I think this is already possible with the current implementation in 2-4 strokes, provided the user has the right hint size.
If the mathematician in you cries out for a recursive solution, there is always grid mode ;).

Here is yet another thought that just crossed my mind: How about the ability to directly start a localized hint mode in a predefined quadrant (or, more generally, an arbitrary rectangle)?

This is close to what I had in mind in connection with your first suggestion. I think a complete grid would be overkill, but a uni-character grid consisting of len(hint_chars) characters seems like a good trade off.

In order to keep warpd close to its original spirit, it is possibly best to offer this by allowing to start hint mode from grid mode (in the selected grid), as I already hinted at in my original feature request.

I am still against mixing modes. The number of questions I receive about grid mode suggests more people are using it than perhaps should be. The only people who should really be using it are those with unorthodox hardware (warpd originally began as a project for one such person) or accessibility needs. In almost all cases, hint mode is more efficient.

This way, a localized hint mode could be started by a keyd macro

I may yet end up fusing the projects after  ;).

Perfect, can't wait to try it out. There will always be use cases where a physical mouse/trackball/trackpoint can never be replaced, such as CAD applications, gaming or photo editing, but for the rest, warpd already surpassed those physical devices in the vast majority of cases and is on the way to increase its advantage even further!

I hope you don't mind waiting a bit. Implementing this should be fairly easy, but I will need to think about the precise form I want it to take and how it will integrate with the rest of the program. I am probably going to be busy for the next two or three weeks, after which I intend to revisit warpd and make some long needed internal changes. I will prioritize this feature upon my return.

I indeed think that I'm doing something wrong, not with the amount of hints
(the size shouldn't make a difference as it doesn't change the number of
points) but the movement afterwards. Maybe I should experiment with my speed
and acceleration settings.

+1

I'm not sure if I understand you correctly and I'm also not sure if you
understand me correctly. Probably we both mean the same thing. What I wanted to
say is that there should be the possibility to start hint mode directly
localized to one of the four quadrants of the screen, still with a
two-character grid and all given hint characters. This would then offer a grid
which is much finer, hopefully eliminating the need for a second step. It's as
if one starts grid mode, selects a quadrant and then starts a full hint mode in
the selected quadrant.

Sorry, I was conflating this with your earlier suggestion (a localized set of hints based on the initial selection).
To achieve the level of precision you describe in your second idea, you would likely have to saturate the area of interest with hints, starving it of contextual cues in the process. You would additionally need some way to specify the quadrant of interest (presumably with a dedicated hotkey) which would add some additional cognitive overhead. I remain unconvinced this is more efficient than the current approach.
Here is a demo of what can presently be achieved with the right acceleration and hint_size settings:

Now that's a surprise! I was completely convinced after our last discussion
about this that the two projects should be kept separate. Maybe a good way to
go is to split the warpd functionality into platform-specific libraries and a
frontend responsible for running the daemon. Then such a library could
optionally be used by keyd as well to offer pointer manipulation.

Indeed, I have something similar in mind. There are some implementation advantages to doing this, though it will likely involve porting keyd to macos.

I absolutely don't mind and am already very happy with warpd (and keyd) as they
are. The efficiency gains they provide to me are truly substantial. Thanks
again for making these programs available and promptly implementing or very
reasonably discussing all feature requests thrown at you, this is rare and I
really appreciate it!

Thanks for the thoughtful feedback. I haven't had a chance to look at your other proposal, but intend to study it upon my return.

I finally found some time to implement this. Feedback is welcome.


Reverse iterators start off-by-one.

If I instantiate a reverse iterator like so:
            leveldb
                .iter(ReadOptions::new())
                .reverse()
                .from(start_key)
                .to(stop_key)

The first key it yields is the a key that's not in the [stop_key, start_key] range, but is actually the first one after start_key.
Example: if my db has keys ["a", "c", "e", "g"] and start_key is "d", stop_key is "a".
Then the first key yielded is "e", which is outside of the "d"->"a" range I am asking.

This is probably a bug that slipped review. The leveldb iterator protocol is annoying in the way that the first step of iteration is a little different from the following.

This is probably a bug that slipped review. The leveldb iterator protocol is annoying in the way that the first step of iteration is a little different from the following.

Ok, so actually this happens only if the from() key is not present in the DB: then the first key given will be out of range. Otherwise it's fine.
This tests both this bug and that it ends before the to() passed.
When fixing it would be nice to test with .from(&9) as well, in my workaround this needed to be handled differently.
#[test]
fn test_key_iterator_iterate_reverse_from_to() {
  let tmp = tmpdir("key_iter");
  let database = &mut open_database(tmp.path(), true);
  db_put_simple(database, 2, &[2]);
  db_put_simple(database, 4, &[4]);
  db_put_simple(database, 6, &[6]);
  db_put_simple(database, 8, &[8]);

  let iterable: &mut Iterable<i32> = database;

  let read_opts = ReadOptions::new();
  let mut iter = iterable.keys_iter(read_opts).reverse().from(&7).to(&3);
  let vec: Vec<i32> = iter.collect();
  assert_eq!(vec, vec![6, 4]);
}

Ok, so actually this happens only if the from() key is not present in the DB: then the first key given will be out of range. Otherwise it's fine.
This tests both this bug and that it ends before the to() passed.
When fixing it would be nice to test with .from(&9) as well, in my workaround this needed to be handled differently.
#[test]
fn test_key_iterator_iterate_reverse_from_to() {
  let tmp = tmpdir("key_iter");
  let database = &mut open_database(tmp.path(), true);
  db_put_simple(database, 2, &[2]);
  db_put_simple(database, 4, &[4]);
  db_put_simple(database, 6, &[6]);
  db_put_simple(database, 8, &[8]);

  let iterable: &mut Iterable<i32> = database;

  let read_opts = ReadOptions::new();
  let mut iter = iterable.keys_iter(read_opts).reverse().from(&7).to(&3);
  let vec: Vec<i32> = iter.collect();
  assert_eq!(vec, vec![6, 4]);
}


Extend translation capability

Extended translation capability to the Node-Headings in the Tree pane

Will close this one and reopen one with just the context._
(Still trying to understand the finer details of how I can make changes to a pull request already submitted...)


Test installation at univ

To target more users, we should check the installation procedure on school computers
Facts :

we are not root on it
python3 may not be installed

The real problem is to install the required modules like 'requests'... Do you think we can ask admins to install it? Or, we have to code our HTTP Python module... (:D)
I think (but i'm not sure) Python3 is installed on school computers - some students work PAC with those computers for example.

I packaged prof to be installable using pip, which can install packages inside user home dir. So it could be a matter of getting pip on Debian.
Or we could make a virtualenv, providing all dependancies.

Personnaly, I prefer to make a virtualenv... But if you have allready make the PIP packaging, i'm ok :-)

Did it just now
Installation
HTTP_PROXY=cache-etu.univ-lille1.fr:3128 pip-3.2 install --user prof
Execution
~/.local/bin/prof


Add nickname into signup & users

[ ] Add the nickname to the JWT

[ ] Add the nickname to the JWT

Also can you write few sentences on our documentation that now we have ability to use nickname

Also can you write few sentences on our documentation that now we have ability to use nickname

I fixed the bug of nickname not validate uniqueness of case-sensitive nickname value. Please check. Thank you.

I fixed the bug of nickname not validate uniqueness of case-sensitive nickname value. Please check. Thank you.


Changed the background colors in the dark and light mode

Related Issue
Closes #724
Description
I have added different colors other than simple black and white in the dark and light modes respectively. This will make the website look more beautify. Hope you like this little contribution from my side.
Screenshots

ig previous one is good

are you talking about both the themes or a specific one?

are you talking about both the themes or a specific one?

black

ig we don't change the bg color so I'm closing this PR


Server.start static method was removed in a patch release

Expected behaviour
Patch releases should not break things.
Actual behaviour
v2.0.3 removed the Server.start static method, breaking functionality that worked on v2.0.2.
Environment Details

Karma version (output of karma --version): 2.0.3
Relevant part of your karma.config.js file N/A

Steps to reproduce the behaviour

Attempt to start a server with require('karma').Server.start({...}).

It looks like the static method has been deprecated for a while, but no warning was given.

Source said
Deprecated since version 0.13, to be removed in 0.14

I hope it’s not the expectation that consumers read the source before upgrading.


How to log to both Rolling File and stdout

I'm using this from your README . But this only log to file, how to log to stdout at the same time ?

// create a rolling file logger based on date/time that fires process events
const opts = {
errorEventName:'error',
logDirectory:'/mylogfiles', // NOTE: folder must exist and be writable...
fileNamePattern:'roll-.log',
dateFormat:'YYYY.MM.DD'
};
const log = require('simple-node-logger').createRollingFileLogger( opts );

You don't need a manager.createConsoleAppender(); by default .createLogManager() checks if there no logging interface it creates a console logger by default.
Otherwise it creates duplicate logs in console. Similar issue: https://github.com/darrylwest/simple-node-logger/issues/17


Elemens repositioning error in Window demo

See video
https://www.youtube.com/watch?v=xCY7v9DBc7I
Regards, Alexey Knyshev

This was not really a bug but part of the way items are laid out by default, which is centered in Y direction. Fixed in 7ad73ca

This was not really a bug but part of the way items are laid out by default, which is centered in Y direction. Fixed in 7ad73ca


@Hurumap ChartTooltip

Description
Moves ChartTooltip component to Hurumap core.
NB: couldn't get sx or any other Material UI specific feature to work for Tooltip inside hurumap/core except style  + useTheme() properties.
Fixes #776
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Screenshots
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

NB: couldn't get sx or any other Material UI specific feature to work for Tooltip inside hurumap/core except style + useTheme() properties.

Didn't see the above comment when I was review @koechkevin. What issues did you run into?

NB: couldn't get sx or any other Material UI specific feature to work for Tooltip inside hurumap/core except style + useTheme() properties.

Didn't see the above comment when I was review @koechkevin. What issues did you run into?

@kilemensi sx styles just don't get applied when you use inside hurumap-core package.

Not sure I get you @koechkevin ... but let me pull and run locally.

I think style work because style have higher priority that classes (sx). For sx to work, we'll need to figure out how to use custom theme.

I think we'll go with inline style here because if we were to use custom theme as specified here,  we'd have to write custom CSS. Since we may need to use theme properties, it would mean having to write Internal CSS in a JS file which is not ideal.  Style prop is the closest thing in terms of usage to sx in my view.

I think we'll go with inline ...

You're the closest to the code @koechkevin but If I understood that doc, no, we wouldn't need to create custom CSS file. sx should be all we need.
    <ChartOrChartParentComponent
      ...
      sx={{
        "& .vg-tooltip.custom-theme": {
           color: "red",
        }
      }}
      >

& .vg-tooltip.custom-theme can be & #vg-tooltip-element.vg-tooltip.custom-theme if we know/can set id of Tooltip element.
But, at the moment, job one is to know, does it work?

I think we'll go with inline ...

You're the closest to the code @koechkevin but If I understood that doc, no, we wouldn't need to create custom CSS file. sx should be all we need.
    <ChartOrChartParentComponent
      ...
      sx={{
        "& .vg-tooltip.custom-theme": {
           color: "red",
        }
      }}
      >

& .vg-tooltip.custom-theme can be & #vg-tooltip-element.vg-tooltip.custom-theme if we know/can set id of Tooltip element.
But, at the moment, job one is to know, does it work?

This wouldn't work on the tooltip @kilemensi the parent component of Tooltip is a div which doesn't support sx. Also, its children are  React components which would have to accept sx props independently

... it doesn't need to be a direct parent @koechkevin . & .class would work on any parent; &>.classonly works on direct parent (if I still remember my CSS correctly)

@kelvinkipruto @kilemensi I was able to get sx to work with vega tooltip. Please have a re look at this PR

@kelvinkipruto @kilemensi I was able to get sx to work with vega tooltip. Please have a re look at this PR

That's the thinking/tenacity I am about!!

👍🏽
Another approach that we could have taken using the recommended custom theme is via MUI's Global CSS Override. It can style even elements in a portal. We wouldn't even need a standalone Tooltip component since we'll just be styling the Vega-Tooltip itself.

This could work as well @kilemensi but I'd love to have this approach tried to see if there are challenges that would necessitate it. it'd also mean a major change that would best be implemented in a new iteration/version.


Initialize governance-policy-addon-controller

/cc @mprahl

I'd like us to set up the "correct" GitHub fork relationship for this sooner rather than later. Thoughts?

Just that I hadn't thought about how to do that yet. 😉 "Replacing it with a new fork" sounds risky, though any method probably involves risk and an abundance of caution. Unless you have a plan in mind, I'd have to look into it...

Considering the forking, I think it makes sense to force-push commits from stolostron to the main branch and then use this PR to show/review the differences. Just a min...or an hour. Or tomorrow. Who knows. 🕐 😆

Looks like GitHub was smart enough to close this to prevent catastrophe! Nice. main has been updated and updates continue in:

#2


Added cypher for businvest2200 (GFCF by industry and asset)

What
Busi invest 2 of 2 change.
How to review
Try it.
Who can review
anyone

outdated.


Feature: option formatDocumentOnManualSaveOnly

When config.editor.autoSave is enabled (in any form) and config.editor.formatOnSave is disabled, I would like to format the document when explicitly saving (Ctrl+S).
I currently emulate this behavior with a custom set of keybinds (the following is in the Nix language):
  programs.vscode.keybindings = let
    formatDocumentOnManualSaveOnlyCondition = lib.concatStringsSep " " [
      # manually saving should only format when auto-saving is enabled
      # in some form, and when the file doesn't already
      # get formatted on every save
      "config.editor.autoSave != off"
      "&& !config.editor.formatOnSave"
      # any other clauses match the default
      # ctrl+k ctrl+f manual format command
      "&& editorHasDocumentFormattingProvider"
      "&& editorTextFocus"
      "&& !editorReadonly"
      "&& !inCompositeEditor"
    ];
  in [
    ### FORMAT DOCUMENT ON MANUAL SAVE ONLY ###
    {  # remove the default action for saving document
        "key" = "ctrl+s";
        "command" = "-workbench.action.files.save";
        "when" = formatDocumentOnManualSaveOnlyCondition;
    }
    {  # formatting behavior identical to the default ctrl+k ctrl+f
        "key" = "ctrl+s";
        "command" = "editor.action.formatDocument";
        "when" = formatDocumentOnManualSaveOnlyCondition;
    }
    {  # re-introduce default save action, but in new order after format
        "key" = "ctrl+s";
        "command" = "workbench.action.files.save";
        "when" = formatDocumentOnManualSaveOnlyCondition;
    }
    ### END ###
  ];

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Mirror control plane status onto HostedCluster version history

Keep the HostedCluster's  version history's semantic version field
in sync with the HostedControlPlane.

works for me
/lgtm


Sandbox Microsoft learn appeal issue

Hi, I activated the sandbox, first try ever (Exercise - Create a website hosted in Azure) but there was an error and I summited an appeal. I must add after few tries (few days) as also the appeal was not working correctly.
After few days after finally successful summited appeal I see the following message:

I never activated or used the sandbox before.
How is that possible that I violated any TOS if I never used it before?
Probably is some kind of bug as I see here a lot of issues open regarding the sandbox.
Let me know where I can send my email address so you can check what's going on with my account.
Thanks
Danylo

Same thing.

Is there any resolution on this issue?

Hi @mlottner, can you take a look on my issue?
I can't complete the Ignite 2020 Cloud Skills Challenge which is ending in 10 days.
Can you give me an email or similar where I can send you my details?
Thank you!
Dany


Math rending issue after converting the doc

Have a look here -
https://mlguidebook.com/en/latest/SupportVectorMachine/Explore.html#Algorithm
It was working 4 days ago. Now with the new readthedocs build it is not working.

This repo's github - https://github.com/NishantBaheti/MachineLearningExploration

The problem is the indentation within equations, see #753.
I don't think this has anything to do with readthedocs.org, AFAICT the breaking change was introduced in your commit https://github.com/NishantBaheti/MachineLearningExploration/commit/807dc767b257b5984c341befeda0bb6ab0088347.
If you have suggestions for a proper fix, please comment on #753.
In the meantime, you can remove all the indentation from your equations and it should work.
You should also have a look at your Sphinx warnings (e.g. in https://readthedocs.org/projects/machinelearningexploration/builds/22098131/). They also complain about indentation.


Fix for report setting on analytics

Description of changes:

Changes to include questions with the display as null on the dashboard
Refactor code to do an bulk update or delete operation instead of in a loop

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #1836 (b434010) into main (b34ac34) will increase coverage by 0.03%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main    #1836      +/-   ##
==========================================
+ Coverage   71.47%   71.50%   +0.03%
==========================================
  Files         396      396
  Lines       12051    12050       -1
  Branches      910      910
==========================================
+ Hits         8613     8616       +3
+ Misses       3291     3287       -4
  Partials      147      147

Flag
Coverage Δ

metapi
79.23% <40.00%> (+0.06%)
:arrow_up:

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

Impacted Files
Coverage Δ

met-api/src/met_api/models/report_setting.py
79.06% <37.50%> (-5.15%)
:arrow_down:

...api/src/met_api/services/report_setting_service.py
67.44% <42.85%> (+5.48%)
:arrow_up:


Legendary User rank issues and a possible suggestion

The Issue
I've recently noticed that vrcx can show some people who haven't even reached "system_trust_legend" or a "Veteran User" rank for short as a "Legendary User" since I assume you only check for a string "system_legend" in tags[] array to determine if a user is "legendary".

This finding came to me when I've used in game boolean to determine if user has a legendary rank (Veteran User) + "system_legend" tag and noticed that one of my friends doesn't meet this criteria,

but they're being shown as a legendary user on VRCX.

The Suggestion
Making the requirements for "Legendary User" more stricter like in this case above, checking for other tags for example like this:
if (user.tags.Contains("system_legend") && user.tags.Contains("system_trust_legend") && user.tags.Contains("system_trust_trusted"))
{
    return TrustRanks.Legendary;
}

I know this issue is little bit over too top since the game itself doesn't use "system_legend" at all and it's not even broadcasted through the APIUser class in game (meaning unless you manually fetch the user in game, you can't even see the tag). But since they still haven't removed these tags from people's accounts, probably because they were auto assigned to people when trust system was in development by the API itself. I think this would be quite the nice addition to it in overall.

https://github.com/pypy-vrc/VRCX/commit/7f2acf69ff1b8d392ea912faa47a5f8a938ef043


Neri assignment5 submission

Great work @04neri
Score - 100/100
Taking 5 off for late submission
Final Score -95


Add a link to the Telegram bot API reference to README.md

That will make it easier to check and debug this implementation.

After a bit of careful reading, I found that it's here: https://core.telegram.org/bots/api

I linked https://core.telegram.org/bots in the README, which itself has a link to the API-doc in the first paragraph. I thought that would be enough...
Furthermore I will add the API link to the rustdoc when writing a proper documentation. But I guess adding the API-Link to the README too is not bad ;)


Runtime error: vss_search() only support vss_search_params() as a 2nd parameter for SQLite versions below 3.41.0

Hey! Thanks for doing this project, it is super cool!
Runtime error: vss_search() only support vss_search_params() as a 2nd parameter for SQLite versions below 3.41.0
I hit this issue doing a simple example like the documentation.

Any guidance on getting around it. Other than having to downgrade sqlite, or is that what is recommended? I saw that the issue on the sqlite site had seemed to be addressed. Maybe I should pull the sqlite-vss code and try to build locally? Thanks!
https://github.com/asg017/sqlite-vss/blob/589c2cada6f720cce85c8cc80cfd3e0e43600f85/src/vss-extension.cpp#L835
https://sqlite.org/forum/info/6b32f818ba1d97ef

Hey thanks for filing! That error message isn't great, but I think there's an internal bug with it (it's showing the wrong error message in this statement)
Can you add a LIMIT clause to your query, to something like this?
select rowid, distance
from vss_section_text
where vss_search(
  search_embeddings,
  (select section_embeddings from section_text where rowid = 9)
)
limit 10;

Some background info: the LIMIT clause is enforced on vss_search queries because without it, the entire vector database will be search + sorted which is expensive. Though now that I think about it, I think it causes more confusion that anything, so I'll possibly drop that requirement in a future release
Let me know if the LIMIT doesn't fix your problem!

That was absolutely it! (And, I had another problem with my dimension size being wrong for what it should have been. But, fixing those two things worked like a champ!  Thanks!


need Help with training indic dataset

Hi,
I m new to deep learning, I need to understand 3 things from this project. Please excuse my clumsy question but I need to know the answers.

can I train on my own dataset which is hindi language and text is in latin.( hindi written in english)
python train.py --output_directory=outdir --log_directory=logdir ( what path for the dataset? where do i define the path for my dataset?)
after completing the training which I m assuming it will give me checkpoint file. how do use it or get pretrainned.pt  flile?

I m new to this so I need to understand, last question. I've 2 rtx3090 with nvlink and i m using windows 10 and anaconda how do i use both the gpu to train.
Please answer these question.
Regards,
Sid

really no one from contributors can answer a simple question? these are still rocket science for me but really no one to answer?

I also don' know how to set the dataset path after I download the dataset!


Fixes profile page

Related Issue

Closes: #624
Description of Changes
The mentor/dashboard/profile page was enhanced and used tailwind css.
Earlier button was disappearing in mobile view, now its fixed.
Checklist:

[x] My code adheres to the established style guidelines of this project.
[ ] I have conducted a self-review of my code.
[ ] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots
https://github.com/anmode/grabtern-frontend/assets/91741581/36e4982b-4341-4b13-b680-3c430b35dac8

Please provide any necessary screenshots to illustrate the changes made.

Payment page is in progress right now. Improved Edit profile page.
This is payment page. Still working on it.

Payment Page design


Nadeko flowers on an Unknown#????

We were messing with the gambling section of the commands and then now we have this
http://prntscr.com/jywh6z
We gave everyone flowers and then we took them away 1 by one. Except for Unknown#????
Is there a way I can reset them or can I manually change this Unknowns flowers?

There's a command for resetting currency. You could also delete the offending person from the database.

Thanks for the reply, I would like to use a reset command but I do not see one on the Nadeko commands website. Could you perhaps tell me the command to do so?

It's the very last command in the XP section on https://nadekobot.me/commands .xpreset

Thanks for the reply, I would like to use a reset command but I do not see one on the Nadeko commands website. Could you perhaps tell me the command to do so?


[QUESTION] Will ObjectId.NewObjectId() creates duplicate id in the same collection?

I know if I insert an object into the collection the object will get _id automatically, but if I manually call ObjectId.NewObjectId() function to create an _id for the object, then insert into collection, is it possible that the _id is duplicate?

Hello, we have issue in our database (litedb 4). With _id type of Guid and Guid.NewGuid() sometimes inserts two identical rows. So I am interested in answer too.


Redirect does not work after authentication

Environment

Elixir & Erlang/OTP versions (elixir --version): 1.13
Operating system: macOS 12.2.1
How have you started Livebook (mix phx.server, livebook CLI, Docker, etc): MIX_ENV=prod iex -S mix phx.server
Livebook version (use git rev-parse HEAD if running with mix): 0.5.2

Current behavior
(my local livebook server host : http://localhost:8080)
when import livemd file to livebook (like this, http://localhost:8080/import?url=https://github.com/josevalim/aoc/blob/main/2021/index.livemd)
if use password or token, after authentication redirect to main page (http://localhost:8080/)
Discussion
I'd like to solve it. So I thought two cases.

put session referer before authentication
(But, It doesn't seem to work in token mode. maybe session is lost when raise LivebookWeb.InvalidTokenError)
append redirect url to query params

What is your opinion? What should I do?

But, It doesn't seem to work in token mode. maybe session is lost when raise LivebookWeb.InvalidTokenError

For token auth we would currently render the token auth form, then submitting the form would GET the same URL with ?token from the form. The issue however is that query params are lost, which is relevant for /import. I think we may need to handle token auth under the same /authenticate route as password, that we would redirect to instead of raising InvalidTokenError.

Closed via #1112.


added ignore to mitigate duplicates

If the meta_values is exactly the same it should skip the insert
tested on gb6 ftricomi_gca032361325v1_core_113_1

Just confirm for me - the IGNORE works to block to INSERT if the values match keys with UNIQUE constraints? But if there's no constraint, what happens?

Running the pipeline several time I noticed that during the first insert, the query inserts the values because there's no conflict. In the subsequent inserts: If the same INSERT IGNORE query runs again without changes, it will skip inserting duplicates because MySQL sees that the row already exists and that IGNORE is in place. If no change occurs in the values of species_id, meta_key, or meta_value, the database consider them as redundant rows and skip them.
the best option would be adding UNIQUE in the schema but it is not feasible at this stage so the IGNORE should prevent the manual deletion of duplicates in case the pipeline will run multiple times.


please package for fedora

Would be nice of course. I am not familiar at all with the Fedora process, but I would prefer eovim to be a bit more polished before starting packing it.


add option icon

Preview

For mac users this icon will make sense

Rather option-key? With an alt tag?


rust icon added

Here is my screenshot for rust icon

Cool. Thx. Looks awesome 👍
I think we should set the Image for awesome-patched mode to an empty string as well. Being able to use the original code point from devicons is the advantage of fontconfig, and is not possible on the other modes..

Set empty icon fore awesome-patched mode.

@orhanbalci - Thanks so much for the great PR! We are really excited to get this upstream.
Two requests:

Can you add a commit that adds the Rust icon to the README so that users will know about it?
It looks like you may have inadvertently introduced whitespace on one line. I'll mark it, below:

@orhanbalci - Looks awesome!! Merged! =) Thanks for the great contribution.

@bhilburn I added a link to rust icon in README and fixed the whitespace

@bhilburn thanks for feedback =)


[Bug] incorrect order of instances if multiframe

Describe the Bug
Order of instances appears to be random when each instance is multiframe
Steps to Reproduce
Open series with multiframe instances, e.g. typical US
The current behavior
Order of instances appears to be random and is not sorted by InstanceNumber.
The expected behavior
Order of instances is sorted by InstanceNumber
OS
Windows 10
Node version
18.12
Browser
Chrome 120

Proposed solution
update extensions/default/src/getSopClassHandlerModule.js
add
InstanceNumber: instance.InstanceNumber
To
(24) imageSet.setAttributes
and
after
displaySets.push(displaySet);
Inside (117) if (isMultiFrame(instance))
add
displaySets.sort((a, b) => (a.InstanceNumber > b.InstanceNumber ? 1 : -1));

Agreed there is a bug for stack viewport, instances should be ordered by instance number


Fix prediction modal

Implementing new template broke, at some point, the ability to open the modal to predict results.
Enable the modal again. Check the possibility to add modals as suggested by the Vue template.

Looks great! We should also check if the modal looks good for penalties kicks


Support for Textual Inversion Models?

Greetings,
are the .pt format supported? If not, is there plans to do so?

Thank you,

#814 number 4 shows how to use a custom embedding. There might also be something in the docs (which I should probably get around to reading at some point 🙄)

I can confirm that calling dream.py with the Usada Pekora.pt file from the showcase will work correctly for me on Windows. However, doing the training myself locally, I had no luck so far with the resulting .pt files. Although dream.py seems to start correctly, my style file seems to wreck it entirely so that it does not generate images at all.
I can use the .bin files from the Diffusers concept library or trained myself via their colab using the keyword (not the placeholder, though).

The .pt format is supported; closing. @happylittlecloud - if you have a problem with .pt files or .bin files, please create an issue for that specific issue.


Separate the command line commands in the getting started guide last part

1. Issue, if available:
2. Description of changes:
Similar to https://github.com/aws/karpenter/pull/1412
Includes all the rest of the scripts in the getting started guide.
3. How was this change tested?
4. Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: link to issue
[ ] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Can we just keep it to 1 file that has all steps that results in a karpenter install w/ Grafana?

@bwagner5 The steps that I have separated are those that are either separate concerns, or complex to do in one shot because we need to wait in between, and check the status and etc... Adding all of them to install.sh would break install.sh at this point and make it less useful.
Furthermore this doesn't stop us from having a script that does add the monitoring as well, I'll probably have to add some steps to the scripts that aren't in the documentation, so if you don't mind I'll have that as a separate PR when it's ready.

@spring1843  Looks like you changed a lot of existing files, did you rebase before merging this? Can we make sure we didn't override any previous changes?


Is here a better way to parameterize markdown URLs?

I know, it's not markdown-etiquette, but is there a way to parameterize markdown URLs based on an environment variable, for example to helpfully point to resources, dependent on the execution environment chosen:
[link to azure vm metrics in my prod environment](https://portal.azure.com/org/some/resource-group-${{ ENV }}/resource/kv/djjkshd)
Currently I'm using this beauty, which eats up space and fragments the document.
xdg-open https://portal.azure.com/org/some/resource-group-$ENV/resource/kv/djjkshd

Hi, @chile12! Thanks for filing this feature request.
If I understand correctly, the idea is to do interpolation/substitution in the markdown pre-rendering. That's an interesting idea. The values would come out of the ENV at the time of rendering?

You got it, thanks for taking it serious.
Currently, I'm using an env-var provided via command prompt, to set the environment in question. Which works fine for code cells.
Already took a look at Jekyll and Liquid to tinker with markdown and variables, but this might clash with Runme?


build: add support for armv7 binary release

Modify targeted goarch and ignore irelevant configs (e.g windows / arm  and darwi n/ arm)

@lmussier Could you fix the DCO check?

@lmussier Thanks for your contribution. You may refer to this doc to fix the DCO issue and submit signed git commit: https://github.com/notaryproject/.github/blob/main/CONTRIBUTING.md#commit

@lmussier Could you fix the DCO check?

I'll do.

Should be good know.

Hi @notaryproject/notaryproject-notation-maintainers anything I have to do to retrigger a DCO check - I lmight have missed something also...

@lmussier Could you force push again to fix the DCO issue? It hangs because of the GitHub bugs at that time.

@lmussier Could you try this comment to generate a new commit id and push it again. Thanks!
git commit --amend --no-edit

@lmussier Could you try this comment to generate a new commit id and push it again. Thanks!
git commit --amend --no-edit

Done but might have messedup the signature, this new commit is not 'verified' for github.
5and sorry for the delay, notifications are no well handeled on my side...

@lmussier Please try:
git commit --amend --no-edit -S

@lmussier The PR is approved but merging is blocked since ec15b082abb547cb11a100caad372577a34b8d8b is not signed. Could you sign that commit so that we can merge this PR?


I can't scroll under IOS11.4 and I can't click on it anymore

I can't scroll under IOS11.4 and I can't click on it anymore. I could have done it on ios 11.3

Iam china my name MingGe
Repair IOS11.4 that can't be clicked and unable to scroll BUG
https://github.com/mingge8888/fastclick


Handle KlustaKwik2 parameters

How to specify them? In the prm file? (for 0.2.0 we have phy spikesort that acts as a drop-in replacement for klusta)
Should we maintain strict compatibility with the old PRM files?
Have KK2 parameter names and values changed compared to KK1?
What should be the default parameter of num_starting_clusters?
Should we rename num_starting_clusters to n_starting_clusters (at least in phy) for  consistency with all other parameters in phy?

cc @nippoo @thesamovar

Let's pass them in a dict in the PRM file.
No we don't need to maintain compatibility...
Yes we should rename n_starting_clusters,
we should have:
traces = {
'data_files': ['blah.dat', 'blib.bin'],
'n_channels': 43,
'sample_rate': 30000,
'voltage_gain_multiplier': 1.0
}

spikedetekt = {
'blob': 1,
'blib' : 'foo',
'blab' = True
}

klustakwik = {
'n_starting_clusters': 42,
'blib' : 'foo',
'blab' = True
}

Everyone will need to update their PRM files if we don't maintain compatibility. For example, if an old PRM file contained
...
sample_rate = 25000
nchannels = 129
...

the new one will contain:
spikedetekt = {
    'sample_rate': 25000,
    'n_channels': 129,
}

Same for KlustaKwik parameters.
Otherwise, using an old PRM file with phy will result in an error.
@nsteinme @kdharris101 what do you think? (question is: should we maintain compatibility with the PRM files between the old klustasuite and phy)

The new KK2 doesn't use all the same parameters... if we're going to stick with .prm for a while (which looks likely) then we might as well have nice support for different clustering algorithms etc, with the dicts having the same name as the module...

I think breaking it is fine for me, though if it wouldn't be hard to write
a converter, why not? Maybe others would be annoyed.
On Mon, Jun 29, 2015 at 7:28 PM, Cyrille Rossant<EMAIL_ADDRESS>wrote:

Everyone will need to update their PRM files if we don't maintain
compatibility. For example, if an old PRM file contained
...
sample_rate = 25000
nchannels = 129
...
the new one will contain:
spikedetekt = {
'sample_rate': 25000,
'n_channels': 129,
}
Same for KlustaKwik parameters.
Otherwise, using an old PRM file with phy will result in an error.
@nsteinme https://github.com/nsteinme @kdharris101
https://github.com/kdharris101 what do you think? (question is: should
we maintain compatibility with the PRM files between the old klustasuite
and phy)
—
Reply to this email directly or view it on GitHub
https://github.com/kwikteam/phy/issues/445#issuecomment-116786125.

I think breaking it is fine for me, though if it wouldn't be hard to write
a converter, why not? Maybe others would be annoyed.

that sounds like a good idea

Converter is ok but some parameters don't have equivalents so the ensemble of parameters a user specifies might not make sense in KK2. Perhaps it's better to force users to rethink parameters for the new version? I don't have strong feelings on the matter.
Default values of parameters should come from KK2, they are all specified in the parameters.py file. It should use the latest values and not be hard coded in phy I think.
We shouldn't rename num_starting_clusters in KK2 because the KK2 code consistently uses num_ and not n_ for quite a very variables (and generally there is a preference for slightly longer, more descriptive variable names). Happy to have a translation in phy but my personal opinion is that it feels less clean to me.

Default values of parameters should come from KK2, they are all specified in the parameters.py file. It should use the latest values and not be hard coded in phy I think.

agreed, maybe we can have klustakwik = {} in phy's user settings, meaning that the defaults come from KK2 and not phy

We shouldn't rename num_starting_clusters in KK2 because the KK2 code consistently uses num_ and not n_ for quite a very variables (and generally there is a preference for slightly longer, more descriptive variable names). Happy to have a translation in phy but my personal opinion is that it feels less clean to me.

I didn't mean to rename the variable in KK2, but in phy. Contrary to all other KK2 variables, this one shows up because specifying it explicitly seems to be necessary when launching the algorithm. If there is a way to make this parameter optional (i.e. have a sensible default coded directly in KK2?), then we don't have to rename anything. The user who wants to use a specific parameter will use num_starting_clusters because this is a KK2-specific parameter, like all other KK2 parameters.
(I don't feel strongly about n_ or num_ but I chose the former because it's the standard convention in scikit-learn and other popular SciPy-based libraries)

Yeah, perhaps I should change the way that works so that it's the same as all the other parameters. I'll do that: it's probably the easiest option. I'll make an issue for that.

Yeah, let's not write a converter. Lots of people are still using random old .prm files; let's make a really nice documented example one (or a web page) and let people upgrade. It really won't take more than about 2 minutes if people want to blindly copy values on...
n_foo is the defined standard in phy, rather than num_foo. Any other clustering/spike detection algorithms that end up within the phy codebase will use n_foo throughout so I get Cyrille's point - though I think we have bigger issues on our plate - and I think the people who notice will probably be in the single-digits! ;-)

If all the parameters have different names between KK1 and KK2 - and as a
start toward building/testing the flexibility to include new algorithms -
maybe "klustakwik1 = {...}" and "klustakwik2 = {...}" should be different
parameter sets altogether, with each of those algorithms callable?
And you're all wrong - it should be "theNumberOf_foo" ;)
On Tue, Jun 30, 2015 at 12:14 AM, nippoo<EMAIL_ADDRESS>wrote:

Yeah, let's not write a converter. Lots of people are still using random
old .prm files; let's make a really nice documented example one (or a web
page) and let people upgrade. It really won't take more than about 2
minutes if people want to blindly copy values on...
n_foo is the defined standard in phy, rather than num_foo. Any other
clustering/spike detection algorithms that end up within the phy codebase
will use n_foo throughout so I get Cyrille's point - though I think we
have bigger issues on our plate - and I think the people who notice will
probably be in the single-digits! ;-)
—
Reply to this email directly or view it on GitHub
https://github.com/kwikteam/phy/issues/445#issuecomment-116876754.

klustakwik1 will not work within phy (it always relied on a nasty hack with writing / reading .fet/clu files etc... but yes, if it were, it would have a different parameter set!

Hm but then I am confused - I thought the point was that an arbitrary
clustering algorithm could be used so long as it could accept the data and
return the results. If it needs to write those data to some location on the
hard drive, as a hack, why not let it? I'm sure there will be someone who
wants to include an algorithm that wants to do something even stranger,
don't you think?
---------- Forwarded message ----------
From: nippoo<EMAIL_ADDRESS>Date: Tue, Jun 30, 2015 at 12:16 PM
Subject: Re: [phy] Handle KlustaKwik2 parameters (#445)
To: kwikteam/phy<EMAIL_ADDRESS>Cc: nsteinme<EMAIL_ADDRESS>klustakwik1 will not work within phy (it always relied on a nasty hack with
writing / reading .fet/clu files etc... but yes, if it were, it would have
a different parameter set!
—
Reply to this email directly or view it on GitHub
https://github.com/kwikteam/phy/issues/445#issuecomment-117128386.

Yes, any clustering algorithm can be used so long as it:

accepts the data as a set of NumPy arrays
returns the data as a set of NumPy arrays
takes a Python dict of parameters.

KK1 does none of the above. If someone wants to write a Python bridge program which will do the above (basically write fet/fmask, convert dict to a key=value set of arguments, execute the program, capture the logging output, then open and return the .clu), then phy makes it easy to do so, but it's not something we're planning to do!

@nsteinme is there any reason to want to use KK1 over KK2? I wouldn't have thought it was worth putting any great effort into this?

No, it's not worth putting in great effort. But I would say that our
comparison of the results of KK2 versus the results of KK1 has been
superficial at best so far ("yes, they both look good!" quoth I, several
times), and if we're building a system that will make it possible to do
fast/easy/automatic comparisons between algorithms or parameter sets, why
not check this? I mean, I have definitely bought all-in to KK2 already, and
am using it for everything I run. But KK1 has how many years of history of
people using it and being happy with the outcome? KK2 just doesn't have
this kind of track record yet, does it? If I were a new user of phy/kk2 and
I saw that there was a way to compare clustering algorithms - I would for
sure want to test the old thing I had been using against the new. But
again, personally I believe KK2 works great and so I fundamentally agree
that this not worth putting any great effort into it.
On Tue, Jun 30, 2015 at 3:36 PM, Dan Goodman<EMAIL_ADDRESS>wrote:

@nsteinme https://github.com/nsteinme is there any reason to want to
use KK1 over KK2? I wouldn't have thought it was worth putting any great
effort into this?
—
Reply to this email directly or view it on GitHub
https://github.com/kwikteam/phy/issues/445#issuecomment-117210026.

@thesamovar where can I find the list of default parameter names in KK2?

https://github.com/kwikteam/klustakwik2/blob/master/klustakwik2/default_parameters.py


[Character|People] Query issues for single results

Just like #111
MAL redirects to the result if there's only 1 single result. Work around is to parse the page it redirects to as a result.

Fixed. Funny thing, character pages (as of now), don't redirect if there's a single item.
Here's an example: https://myanimelist.net/character.php?q=Yuzuriha%2C+Inori


NavList gives an exception when using rails view variants.

Problem
On a rails 7 application that is using rails view variant I'm getting the following exception when rendering the NavList component (using the default example)

This only happens when a variant is set, and so far I've only experienced an issue with the Navlist component.

Hey @xocs, thanks for letting us know you ran into this. It's happening because of a pretty gnarly bug in the view_component framework but I think I have a handle on it. Once that fix is released you'll need to upgrade your version of view_component.

@camertron Excellent, thank you for picking this up so quickly. I'm really enjoying working with primer, great work!

Awesome, thanks for that positive feedback!


Backward data GEMM solvers and invokers

Backward GEMM is now split between the following solvers:

GemmBwd1x1_0_2 - applicable when 2d && kernel is 1x1 && conv pads are 0x0 && conv strides are 2x2
GemmBwd1x1_0_1 - applicable when kernel is 1x1 && conv pads are 0x0 && conv strides are 1x1
GemmBwdRest - applicable when none of the solvers above is.

"The rest" kind of the solver logic is flawed, but at the moment, it is implemented as it worked before. Changes to that may be applied in separate PR later.
I have updated solver ids in the find-db.

To be honest I am not sure who to add as reviewers.

Please update from develop. Each PR needs to be checked with the most recent clang-tidy checks.

@DrizztDoUrden HIP build failed. Ditto Hip Clang Tidy check.

"Bf16 Hip Release gfx908" failed twice in a row. Please look at CI logs. In the past, it failed also at run 7, so the issue is not from the recent changes.
I will run my tests tonight (but this is gfx906).

Sure, I'll try to fix it today, not much to do this weekend xD

@DrizztDoUrden gfx906, ROCm 4.0, memory access fault with BF16 when -i > 1:
$ ./bin/MIOpenDriver convbfp16 -x 1 -y 1 -W 14 -H 14 -c 1024 -n 8 -k 512 -p 0 -q 0 -u 2 -v 2 -l 1 -j 1 -g 1 -F 2 -V 0 -i 2
...
MIOpen(HIP): Info [FindConvBwdDataAlgorithm] miopenConvolutionBwdDataAlgoGEMM   0.194634        1204224
MIOpen(HIP): Info [FindConvBwdDataAlgorithm] miopenConvolutionBwdDataAlgoDirect 0.26096 0
MIOpen(HIP): Info [FindConvBwdDataAlgorithm] BWD Chosen Algorithm: GemmBwd1x1_stride2 , 1204224, 0.194634
MIOpen(HIP): Info [ConvolutionBackwardData] algo = 0, workspace = 1204224
MIOpen(HIP): Info [ConvolutionBackwardData] algo = 0, workspace = 1204224
Memory access fault by GPU node-2 (Agent handle: 0x172c720) on address 0x7f627bd61000. Reason: Page not present or supervisor privilege.
Aborted (core dumped)

With -i 1 it does not fail with GPU memory access fault, but validation fails:
$ ./bin/MIOpenDriver convbfp16 -x 1 -y 1 -W 14 -H 14 -c 1024 -n 8 -k 512 -p 0 -q 0 -u 2 -v 2 -l 1 -j 1 -g 1 -F 2 -V 0 -i 1 -V 1
...
MIOpen(HIP): Info [FindConvBwdDataAlgorithm] miopenConvolutionBwdDataAlgoGEMM   0.195769        1204224
MIOpen(HIP): Info [FindConvBwdDataAlgorithm] miopenConvolutionBwdDataAlgoDirect 0.2624  0
MIOpen(HIP): Info [FindConvBwdDataAlgorithm] BWD Chosen Algorithm: GemmBwd1x1_stride2 , 1204224, 0.195769
MIOpen(HIP): Info [ConvolutionBackwardData] algo = 0, workspace = 1204224
Read data from input file /spb-hlc-fs2/1/users/atamazov/miopen-verify2/conv_bwd_dat_0_2_0_1_8x1024x14x14_512x1024x1x1_0x0_2x2_1x1_0x0_0_0_GPUbfloat16_REFfloat
Backward Convolution Data Failed: 0.114971 > 0.0656

GEMM in baseline 7bd59992a works correctly.

Fixed

@DrizztDoUrden FP32, FP16, BF16, both OpenCL and HIP backends. Incorrect time measurements during Backward Convolution phase (after Find, MODE=NORMAL) for the GemmBwdRest solver.
Whenever GemmBwdRest solver is selected (in the log: BWD Chosen Algorithm: GemmBwdRest), the measured time is ~2X bigger than it should be (and actually measured during Find). It is even bigger than Wall-clock time:
Wall-clock Time Backward Data Conv. Elapsed: 7.74295 ms...
...
GPU Kernel Time Backward Data Conv. Elapsed: 13.993815 ms (average)

Time measurements in Immediate Mode are also affected.
This is minor issue, but we need to fix it because it affects performance testing.
8e7a0fb8e.ddu_bwd-gemm-solvers.logs.zip

No idea why it seems stuck on GH. It looks finished on Jenkins...

Btw, fixed the last mentioned issue.


TypeError: object is not a function

I used node-phantom simple before and loved it.  It seems to have changed a little over the past few months.  I am getting TypeError object is not a function.  However, it seems to be in my node_modules
I installed node phantom js as well.  Not sure what the problem is.
TypeError: object is not a function
at /Users/stevenkauyedauty/projects/finalProject/moneyScrape/node_modules/node-phantom-simple/node-phantom-simple.js:203:20
at ChildProcess. (/Users/stevenkauyedauty/projects/finalProject/moneyScrape/node_modules/node-phantom-simple/node-phantom-simple.js:93:10)
at ChildProcess.g (events.js:180:16)
at ChildProcess.emit (events.js:95:17)
at Process.ChildProcess._handle.onexit (child_process.js:808:12)

No ideas. Need code example. All changes are recorded in changelog.
https://github.com/baudehlo/node-phantom-simple/blob/2.0.1/node-phantom-simple.js#L203 line from your error does not contain code. You do not seems to use latest version.

Nevermind, it was my generator.  I switched to a different one and had no problems.  Thanks

On Jul 29, 2015, at 10:33 PM, Vitaly Puzrin<EMAIL_ADDRESS>wrote:
No ideas. Need code example. All changes are recorded in changelog.
https://github.com/baudehlo/node-phantom-simple/blob/2.0.1/node-phantom-simple.js#L203 https://github.com/baudehlo/node-phantom-simple/blob/2.0.1/node-phantom-simple.js#L203 line from your error does not contain code. You do not seems to use latest version.
—
Reply to this email directly or view it on GitHub https://github.com/baudehlo/node-phantom-simple/issues/66#issuecomment-126179363.


Preset: node-style-guide

Fixes #1071
Initial preset config.
Not sure if the preset should only include rules based on the explicit statements or also include rules based on the code in the examples that is implied.
An example is function spacing.
function BankAccount() {
}
setTimeout(function() {
  client.connect(afterConnect);
}, 1000);

the rules could be inferred to be
"disallowSpacesInFunction": {
    "beforeOpeningRoundBrace": true
},
"requireSpacesInFunction": {
    "beforeOpeningCurlyBrace": true
},

I can move that huge table of possible rules here as well?

Coverage increased (+0.0%) to 98.86% when pulling d5c5ebc42b98fdc730c9aa0f5c921d4105498c5d on hzoo:node-style-guide-preset into 28ba35b23ca70a767f7e03fbb9e56617f74203a4 on jscs-dev:master.

Not sure if the preset should only include rules based on the explicit statements or also include rules based on the code in the examples that is implied.

Also add rules implied from sample.
LGTM

Ok I added some the implied ones like function spacing but there's probably a few rules I didn't see to add based on the style.
We can also think about adding some of the new rules at a later point.

@felixge would you mind checking this out?

Safe to land after rebasing?

rebased - I didn't add anything

Ok I realized I didn't add to the overview or add a line to run the test for the preset. We should update CONTRIBUTING.md https://github.com/jscs-dev/node-jscs/blob/master/CONTRIBUTING.md#proposing-additional-presets on that stuff. Can do another commit/PR later if no one else will take i

Good work @hzoo!


Gaia names

Use gaia.ssete9 base names in Epsilon0.*

Still in progress ...

Rename hydra-battles constants using gaia's conventions (in progress)


DuerOS development suite is a team of enthusiasts can group - 费元星

DuerOS development suite is can set a team of enthusiasts like millet enthusiasts clan, the inside has been big guy before product launch, can you advice, first to do statement analysis data, the final improvement advice.Told the official team will be very helpful, thank you!
Such as:

In Beijing, in haidian, more in west two flag, the most in the science and technology park, can at any time about it.
To understand a language, has a strong development ability.
The Angle, and dare to try.

费元星
beijing xierqi
给我发邮件

12


fix: replace root index.html for nested mode (#104)

Does this pr conform the merger requirements? It seems that no one has handled this PR/issue for a long time.


doen't work for facebook urls

Thanks for the very useful package, however, it doesn't work for Facebook urls (the public groups), such as
https://www.facebook.com/groups/1166154663436469/permalink/4206825836035988
I get this error : check_hostname requires server_hostname

Seems like an issue with urllib: https://stackoverflow.com/questions/66642705/why-requests-raise-this-exception-check-hostname-requires-server-hostname
Can you try urllib3==1.25.8 ?

@pgaref I fixed it the other day, and forgot to remove the issue. But thanks, and yes, that's the solution

Thats great @enaserianhanzaei thanks for confirming!
Happy to accept your FIX if you want to contribute to the project :)


[FT] Fail faster when passing unsupported metrics to InferenceEndpointModel

Issue encountered
When passing an unsupported metric (e.g. single-token metric) to InferenceEndpointModel, an error is raised from Pipeline.evaluate:
ValueError: Endpoint models can't use single token metrics. Change the metric to the standard version

This happens once the endpoint has been created and the model has been created and loaded.
I wonder if the error could be raised earlier for optimization reasons: the process fails faster and resources are not wasted.
Solution/Feature
Raise the error before the endpoint has been created.
Possible alternatives
Leave it as it is.

Hm, good idea for the checks, especially for costly metrics. We would also want the LLM as judge to stop if the open ai key is not provided for example.

Also linked to https://github.com/huggingface/lighteval/issues/355


Implement metric to show peak days

Implement metric to shows activity per day name for:

[x] Posts
[ ] Comments
[ ] Likes
[ ] Shares
[ ] All activity

https://github.com/hellowin/kanca/pull/80

https://github.com/hellowin/kanca/pull/81 comments

https://github.com/hellowin/kanca/pull/82 activities


twitch chat fails

I have problems with twitch chat, it does not load ...I'm using -git version, for now none of the chats I've tried have worked (like twitch.tv/angryjoeshow, twitch.tv/maliveth)...it seems this problem is back, or is it just me?

I can confirm, just tried to watch a stream on Twitch, and noticed the chat did not function. (I am also running the git version.)

I can confirm, just tried to watch a stream on Twitch, and noticed the chat did not function. (I am also running the git version.)

I've just realized that I checked twitch chat to only show messages, but never tried to post any message myself. Are you talking about sending messages? Or chat doesn't load at all?

@i-rinat : it does not load at all, I get : "Sorry, we were unable to connect to chat. Reconnecting..

Hi,
I can confirm this too.
Twitch chat failed to load with the error message "Sorry, we were unable to connect to chat. Reconnecting.."
Can we help you to debug it ?

[fresh] [warning] handle_tcp_connect_stage2, evdns returned code 3, count = 0
[fresh] [warning] ppb_tcp_socket_get_local_address, not connected
[fresh] [warning] ppb_tcp_socket_get_remote_address, not connected

I believe that this console output is relevant to the twitch chat not connecting as it repeats 4 times every at reconnect attempt the chat does.

evdns returned code 3

3 means DNS_ERR_NOTEXIST. It's highly unlikely actual DNS lookup fails. More likely ppb_tcp_socket_connect() is supplied with a wrong address, which can be there due to another error.
To verify, reconfigure with cmake -DTRACE_ALL=1 .. and build freshwrapper again. Done so, output from browser will be enormously large, so it's better to collect it to some file (firefox > log, for example). What is interesting, is what parameters ppb_tcp_socket_connect is called with. host= and port= are deserving special attention.
Reconfiguring with cmake -DTRACE_ALL=0 .. and building again will disable verbose output.

@i-rinat I got this:
ppb_tcp_socket_connect tcp_socket=321, host=telegraph.twitch.tv, port=843, callback={.func=0x7fa7789f65f0, .user_data=0x7fa777cdb860, flags=0}
ppb_tcp_socket_connect tcp_socket=476, host=telegraph.twitch.tv, port=443, callback={.func=0x7fa7789f65f0, .user_data=0x7fa777cdb940, flags=0}
ppb_tcp_socket_connect_with_net_address tcp_socket=514, addr={0x7fa773009c9c}, callback={.func=0x7fa7789f65f0, .user_data=0x7fa7752a6790, .flags=0}
ppb_tcp_socket_connect tcp_socket=767, host=<IP_ADDRESS>, port=843, callback={.func=0x7fa7789f65f0, .user_data=0x7fa7752a6b90, flags=0}
ppb_tcp_socket_connect tcp_socket=782, host=<IP_ADDRESS>, port=443, callback={.func=0x7fa7789f65f0, .user_data=0x7fa7752a6b80, flags=0}
ppb_tcp_socket_connect tcp_socket=1525, host=<IP_ADDRESS>, port=843, callback={.func=0x7fa7789f65f0, .user_data=0x7fa7752a6de0, flags=0}
ppb_tcp_socket_connect tcp_socket=1541, host=<IP_ADDRESS>, port=443, callback={.func=0x7fa7789f65f0, .user_data=0x7fa7752a6de0, flags=0}
...there is way more
the whole log file is here: https://app.younited.com/?shareObject=a33dfe4a-b790-a6df-1e57-01e1883183f3

@i-rinat : why no fix yet?

@ahjolinna, there is no fix for the issue yet because:
(a) I can't reproduce it, chat loads fine on my machine, so I can't even guess what goes wrong on your setup. Log file doesn't help too much — there are records about established connections, but there are also records about failed for unknown reason ones (<IP_ADDRESS>:80 is available here, for example);
(b) I'm doing this project on my free time. I'm trying to make it work well, but I'm not a wizard.
If you have an idea what's wrong with the particular piece of code and/or have an idea how to fix it, please let me know. If you have a patch, which solves some issue and doesn't break anything else, please share it, I'll merge it. (After style cleanup, perhaps.)

@i-rinat : I just installed freshplayerplugin-git on my ultrabook and it seems that twitch chat works on it

I'm not sure that I can help, but I uploaded my log file here : http://dl.free.fr/kmSHOSALc
Thanks for your help and your support.

@ahjolinna, do you have any packet filtering enabled on your desktop machine?

@Bevito, @ahjolinna, please try 317eda1a0f2b31be43624cf0fecfc35fdcdc99eb. In the logs from both of you all failed DNS resolutions was already IP addresses. So I've added another code branch to detect whenever passed host name is in appropriate format.

@i-rinat : now it works ...thanks

Great, now it works.
Thanks a lot !

So, that was the reason...
But do /etc/resolv.conf really differ on your laptop and desktop, @ahjolinna ?

Desktop:
search elisa-laajakaista.fi
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>
Laptop:
nameserver <IP_ADDRESS>

I see. Laptops talks to a local caching DNS server (dnsmasq, perhaps), which handles IP addresses as well. I've tried now to switch to Google's DNS (<IP_ADDRESS>), and have reproduced bug too.


⚠ Bofrost.fr has degraded performance

In 1c7904a, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2587 ms

Resolved: Bofrost.fr performance has improved in a29519e.


Fastnlight: Watch +44<PHONE_NUMBER>

Fastnlight requests the watch of the watch_number +44<PHONE_NUMBER>. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


fix: update path in documentation

awesome ty!

oops, sorry i closed as i wanted to put more stuff in the pull. well, keeping the bullets for next shot  :)

Ah ok, will leave it alone next time until I know that's all. Keep in mind that (as soon as I have time for Gekko) I am working on rewriting a huge part on core in another branch.
See here for details.


Start QEMU emulation in EL3

QEMU support for EL3 is mature enough that we can start directly from EL3 and drop to EL1 in the reset routine. Remove also the legacy SWITCH_TO_EL1 because there is no need to have Zephyr running at EL3.
This makes the emulation also closer to what happens on real hardware where the CPU starts usually from EL3.

So are you also going to remove rest of the el3 code as they become stale after this.

Yeah, good point. I think we can remove all the EL2 code as well since we are dropping in EL1 anyway at start.

gentle ping about this PR