Tracking Email based on the user's timezone!

Currently the emails are being tracked base on the timezone "Asia/Colombo" Instead we can track it using the timezone of the user!

If this seems a valid issue, I would like to work on it!

At the moment, user has to manually edit routes.py to change their timezone. And it is documented within 'Installation' section of the readme.
But, more feasible and efficient way can be implemented here.
@kailashchoudhary11 you are welcome to propose your changes anytime 😊

@Dilshan-H isntead of using
generatedOn = str(dt.now().astimezone(pytz.timezone('Asia/Colombo')))
can't we directly use
generatedOn = str(dt.now().astimezone())
Won't this will give the current timezone of the user? Just a doubt

@Dilshan-H isntead of using
generatedOn = str(dt.now().astimezone(pytz.timezone('Asia/Colombo')))
can't we directly use
generatedOn = str(dt.now().astimezone())
Won't this give the current timezone of the user? Just a doubt
I read it here - GeeksforGeeks

If you are planning to host the application in a local environment, this works just fine. But when it comes to remote servers they are may be located in different timezones. That's why we need to pass timezone parameter specifically. That way we can get the datetime according to our local time zone.
Hope this clarifies you issue 😊

@Dilshan-H isntead of using
generatedOn = str(dt.now().astimezone(pytz.timezone('Asia/Colombo')))
can't we directly use
generatedOn = str(dt.now().astimezone())
Won't this give the current timezone of the user? Just a doubt
I read it here - GeeksforGeeks

If you are planning to host the application in a local environment, this works just fine. But when it comes to remote servers they are maybe located in different timezones. That's why we need to pass timezone parameter specifically. That way we can get the datetime according to our local time zone. Hope this clarifies you issue 😊

Aah, didn't think of it that way, you're absolutely right.
Just had one more doubt, do we really need to show the time of creation of thee tracker to the user? What purpose does that information serve and can we not use utc instead?

@saroshfarhan It's better to show that info as user can quickly refer their corresponding email at a later time. When there are lot of tracking links this might be useful in a way.
Also I'm hoping to implement a search functionality inside tracking link list dashboard. So, a user can search for tracking links that were generated in a specific date for example.

Hey @Dilshan-H,
We could import tzlocal and could get the local time? You can try this on a separate python file.
we would require tzlocal
pip3 install tzlocal
from datetime import datetime as dt
import pytz
from tzlocal import get_localzone

generatedOn = str(dt.now().astimezone(pytz.timezone(str(get_localzone()))))
print(generatedOn)

I think the better way would be to send the timezone from the browser using js!

I think the better way would be to send the timezone from the browser using js!

Actually that seems like a better way to achieve this.

I think the better way would be to send the timezone from the browser using js!

Actually that seems like a better way to achieve this.

So can you assign this to me

It will produce the same issue as earlier isn't it? (When used in remote servers)

It does only solve the backend time issue. Not when it runs on the server.


Move from begin/rescue/end (BRE) to RSpec metadata when running adapters specs

OK, this looks good but you've mixed in whitespace/formatting changes with actual functional changes. Do you think we can run a separate branch first which does the whitespace change, so that we can get clean commits?
thanks!

@imajes I guess I do not touch formatting, the whitespace I create a PR to remove some maybe I miss some.
I guess it is hard to see the changes because I usualy remove a full ident of code, because I removed the BRE. I think the better way to you review is clicking the "view" button on each file under the "Files Changed" tab on top.

Yeah, i guess the BRE really did screw with the indentation. It's frustrating because that change kinda gets lost in the noise of the re-indent. I think we'll just have to accept it. i'll merge this in a bit.

Done. merged via rebase. good work!


Pairwise for 10 features when evaluation

May I know what is the intuition using pairwise distance of 10-features when doing evaluation instead of using cosine-similarity. I have tried cs, but performance is worse.

The test-time augmentation scheme is explained in Section 4.4 of https://arxiv.org/pdf/1806.05622.pdf .
See my comment in #4 for the relationship between cosine distance and Euclidean.


Create RPC to expose get_hotkey_global_dynamic_tao

get_hotkey_global_dynamic_tao is the method in staking.rs
Merged to dynamic branch now.
Matching PR for Bittensor: https://github.com/opentensor/bittensor/pull/1881

Closed at Stale


conda installation trouble -- cuda drivers?

Issue: Hello, I am very new with conda and programming generally. Apologies in advance if I'm missing something ovious. I'm trying to install both the conda-forge github-pages and jekyll packages into a new environment. I've gotten the following error:
(gh-pages) C:\Users\Sam>conda config --add channels conda-forge
Warning: 'conda-forge' already in 'channels' list, moving to the top

(gh-pages) C:\Users\Sam>conda config --set channel_priority strict

(gh-pages) C:\Users\Sam>conda install -n gh-pages -c conda-forge rb-github-pages
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: \
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with your CUDA driver:

  - feature:/win-64::__cuda==10.1=0

Your installed CUDA driver is: 10.1

Note that strict channel priority may have removed packages required for satisfiability.

Thank you!
Sam

Environment (conda list):

$ conda list
(gh-pages) C:\Users\Sam>conda list
# packages in environment at D:\miniconda3\envs\gh-pages:
#
# Name                    Version                   Build  Channel

Details about  conda and system ( conda info ):

$ conda info
active environment : gh-pages
    active env location : D:\miniconda3\envs\gh-pages
            shell level : 2
       user config file : C:\Users\Sam\.condarc
 populated config files : C:\Users\Sam\.condarc
          conda version : 4.8.3
    conda-build version : not installed
         python version : 3.8.5.final.0
       virtual packages : __cuda=10.1
       base environment : D:\miniconda3  (writable)
           channel URLs : https://conda.anaconda.org/conda-forge/win-64
                          https://conda.anaconda.org/conda-forge/noarch
                          https://repo.anaconda.com/pkgs/main/win-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/win-64
                          https://repo.anaconda.com/pkgs/r/noarch
                          https://repo.anaconda.com/pkgs/msys2/win-64
                          https://repo.anaconda.com/pkgs/msys2/noarch
          package cache : D:\miniconda3\pkgs
                          C:\Users\Sam\.conda\pkgs
                          C:\Users\Sam\AppData\Local\conda\conda\pkgs
       envs directories : D:\miniconda3\envs
                          C:\Users\Sam\.conda\envs
                          C:\Users\Sam\AppData\Local\conda\conda\envs
               platform : win-64
             user-agent : conda/4.8.3 requests/2.24.0 CPython/3.8.5 Windows/10 Windows/10.0.18362
          administrator : False
             netrc file : None
           offline mode : False

Sorry this issue was missed Sam 😞
Am guessing this is related to some bug in Conda that occurred around the same time ( https://github.com/conda/conda/issues/9897 )
We haven't seen that issue in a while, but we do track it
In any event, this isn't an issue with this particular package. So will go ahead and close
If you do see this again, please comment on the Conda issue linked above with your latest reproducer. That should help us dig into what is going on


Publish v4 to npm with 'beta' tag to keep consistent behavior with 'bootstrap' module.

Publish v4 to npm with 'beta'/'next' tag to keep consistent behavior with 'bootstrap' module.
'bootstrap' module by default installs v3 with 'npm install bootstrap' and v4 with 'npm install bootstrap@next'.

hi @ravshansbox, I've updated the dist-tags for bootstrap-css-only to mirror bootstrap proper.
Now the output from npm info bootstrap-css-only shows:
'dist-tags': { latest: '3.3.7', next: '4.0.0-beta.2' },

Thanks.


Sample unit tests on bot with Dialogs and Prompts

Please create a sample of project with all the bot stuff (Dialog, prompts, etc.) unit tested to show how to mock some object such as DialogSet, WaterfallStepContext, etc.

Hi @nimbusparis, thanks for the feedback, we are planning to add additional testing examples and tools support as part of 4.5.
This page contains some background on our general approach.
For a complete list of the testing scenarios we are considering see Bot Testing Issues.
I will close this issue for now, please add your comments on the stories above if there's anything in particular you'd like to see that is not covered there.
Thanks again.
Gabo
PS: For a sneak peak of the work being doing, you can check the CoreBotTests branch (check the https://github.com/Microsoft/botbuilder-dotnet/tree/gabog/CoreBotTests/tests/Microsoft.Bot.Builder.TestBot.Tests project, it is early days and things are subject to change but it should give you an idea).


Update dev-dependencies

Eslint 4 has a stricter set of rules for indentation requiring some reformatting.
Also completely regenerates the package-lock.json to pick up new version of indirect dependencies and eliminate a lot of duplicates.

Thanks!


fix_map = scipy.io.loadmat(path)["I"]

KeyError: 'I'  , the dataset does not have the key "I" in the .mat

Hi @bondruy,
please take a look at this previous issue and let me know if this solves your problem.


plotting.py raises ImportError for matplotlib, eventhough I'm using plotly engine

Why do we check if matplotlib is installed first?
It will raise an error even though I am using a different plotting engine, e.g. plotly
I can create a PR if you think plotting.py needs revision.
try:
    import matplotlib as mpl
    import matplotlib.pyplot as plt
    import matplotlib.colors as cm
except ImportError:
    raise ImportError(
        "matplotlib is not installed. Please install it and try again.\n"
        "You can find detailed instructions at https://matplotlib.org/stable/users/installing/index.html"
    )

Hey. It's because we always use the colormap from matplotlib


Task for update_docs can timeout apparently

Stack trace:
Jul 13 18:43:51 build02 readthedocs/celery.worker.job[11229]: ERROR Task update_docs[12609cdb-ee39-45f2-b3a6-616a1a920c3c] raised unexpected: TimeLimitExceeded(3600,) [celery.worker.job:282]
Traceback (most recent call last):
  File "/home/docs/local/lib/python2.7/site-packages/billiard/pool.py", line 641, in on_hard_timeout
    raise TimeLimitExceeded(job._timeout)
TimeLimitExceeded: TimeLimitExceeded(3600,)
Jul 13 18:43:51 build02 readthedocs/celery.worker.job[11229]: ERROR Hard time limit (3600s) exceeded for update_docs[12609cdb-ee39-45f2-b3a6-616a1a920c3c] [celery.worker.job:368]

This should instead report a failure on the build, not just die. When the celery process is killed, the lock stays around, user is not notified of failure, and the build stays in "Building" state.

This is a celery setting that we have set.

We can remove it or raise it if we don't think this is an issue now that we have the Docker time-limiting in place.

Yeah, the reason for termination is clear. We'll want to do this as an argument to calling the task, as the value of Docker time limit changes with projects.


OpenMeshFX plugin to do DMM

https://libigl.github.io/tutorial/#direct-delta-mush
https://github.com/libigl/libigl/blob/main/tutorial/408_DirectDeltaMush/main.cpp
Skinned Skeleton to DMM as mesh data per frame. The skeletal is lost.
Can recover the new skeletal using dem bones with the original skeletal.

Not doing this.


Fields Placement

With the latest version (may be since .csproj migration?), the 'part' (I don't remember its name) that appeared in the 'edit content type definition' page to specify where the fields are placed amongst the parts is not available anymore.
Can you confirm it is not there anymore?
I also remember a discussion with @jtkech (May be in one of his PRs?) and that we need to find a solution to specify the Edit placement of fields and parts (... and tabs, later) in a flexible way, without restrictions.

The name is the content type where this fake part is added, e.g Page.
Just tried it (but on my csproj-updates working branch). I added a text field in the Page content type, then i got the following, so it seems to work for me.
 Page - Contains the fields for the current type

About the discussion, i don't remember exactly but the related issue is #527. Need to leave soon but i'll let you know if i retrieve more details.
Best.

OK, thank you. My bad, the 'Content type - Contains the fields for the current type' is there.


Too many shift while reading raw pressure

Hello,
while reading the pressure through lps27hhw_pressure_raw_get an extra shift is performed at the end (line 681).
https://github.com/STMicroelectronics/lps27hhw/blob/8b974831885dd9b7af4a63d36bdb9a9fc16367ad/lps27hhw_reg.c#L672-L684
Edit for future readers: n << 8 equals n * 256.
Example:
If reg[3] = {0xCC, 0x43, 0x3F} (where reg[0] is XL byte, reg[1] LSB byte and reg[2] MSB) the expected buff value is 0x003F43CC and not 0x3F43CC00.
In fact:

0x003F43CC / 4096 = 1012.24 HPa
0x3F43CC00 / 4096 = 259132.75 HPa

I can open a pull request if needed.
Thank you,
Gab

Hi gabbia, in order to keep the same logic of other devices (32 bit raw data format left aligned), this issues impact the function lps27hhw_from_lsb_to_hpa.
Thanks very much for your contribution!


Fix typo

As a title.

test comment for CLA

I signed it

I signed it

I signed it.
/check-cla

I'm sorry, close once


Request money - Message grouping incorrectly on receiver side if they send a message right after get a request money

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Send a money request from account A to account B
On account B, after App show money request, send any chat message

Expected Result:
The new chat message of account B should displayed as it's come from account B, not both requester & receiver money request.
Actual Result:
The new chat message is grouped under money request preview. We just change money request preview recently, and it displaying both requester & receiver. If the new chat message is grouped under money request preview. It make our think this new message is come from both requester & receiver.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.3.57-5
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/115492554/15aa22fa-dd0b-4b9b-8adb-0dfc7718d723
https://github.com/Expensify/App/assets/115492554/18d0196a-f21c-4f12-acc2-b8b2fbc106ec
Expensify/Expensify Issue URL:
Issue reported by: @hoangzinh
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1692406833875899
View all open jobs on GitHub

Same root cause as below so applying this solution will fix the issue
https://github.com/Expensify/App/issues/25509#issuecomment-1684907030

Proposal
Please re-state the problem that we are trying to solve in this issue.
If account A makes a money request after sending a message the avatars show up. But if account B replies after account A sent a message then the IOU preview components will not show the two avatars. There should be two avatars (account A's and account B's) on the left side of the IOU preview component
What is the root cause of that problem?
The primary reason for this discrepancy is the logic inside the isConsecutiveActionMadeByPreviousActor function, which decides how avatars should be displayed in the component. The function determines if actions should be grouped or displayed consecutively. The issue arises because, when User A sends a request for money, the actorAccountID is assigned User B's ID. This causes the function to interpret that the consecutive actions are not made by the same actor, thus leading to current scenario.
https://github.com/Expensify/App/blob/03cf0b14f82a41db004dfee27799e3cea1608ee5/src/pages/home/report/ReportActionsList.js#L263
https://github.com/Expensify/App/blob/03cf0b14f82a41db004dfee27799e3cea1608ee5/src/libs/ReportActionsUtils.js#L255-L288
What changes do you think we should make in order to solve the problem?
To rectify this:
1. Refine the Logic: Update the isConsecutiveActionMadeByPreviousActor function to accurately determine the sequence of actions, especially for IOU-related actions.
2. Utilize Existing Structures: Using the allReports structure and the function getIOUReportIDFromReportActionPreview, access the necessary data for the current and previous reports.
3. Determine IOU Actions: Introduce a check within the function to determine if the action is related to an IOU report. This can be achieved by evaluating the type of report.
4. Compare Relevant Attributes: For IOU actions, modify the comparison logic to evaluate the correct attributes for the avatars. For instance, consider attributes like managerID and ownerAccountID to ensure the right display decision is made.

// eslint-disable-next-line no-use-before-define
const currentReport = allReports[getIOUReportIDFromReportActionPreview(currentAction)];
// eslint-disable-next-line no-use-before-define
const previousReport = allReports[getIOUReportIDFromReportActionPreview(previousAction)];

// eslint-disable-next-line es/no-optional-chaining
if (isIOUReport(currentReport) && !isIOUReport(previousReport)){
    return false;
}
// Can also do this
//  if ( !isIOUReport(currentReport) && isIOUReport(previousReport)){
//     return false;
// }

if (isIOUReport(currentReport) && isIOUReport(previousAction)) {
  return currentReport.ownerAccountID === previousReport.ownerAccountID && currentReport.managerID === previousReport.managerID;
}

return currentAction.actorAccountID === previousAction.actorAccountID;

https://github.com/Expensify/App/assets/65986357/52d918ff-d5a8-4491-b18b-42bbaa77e3bb

Looks like this would be on hold for https://github.com/Expensify/App/issues/15212
Going on sabbatical so assigning a new BZ member.

Looks like this would be on hold for #15212

The other issue is closed as a design decision. I doubt the same might be the case for here as well.
@alexpensify Could you kindly check if this is the expected behaviour or a bug indeed?

Thanks, I'll start a discussion in the Slack room and confirm next steps here.

I'm catching up from being OOO but will start the discussion tomorrow.

I've started a convo here:
https://expensify.slack.com/archives/C01GTK53T8Q/p1695063873406209

Ok, closing! There is no payment here since Vit flagged this issue before this one here.


Bump nokogiri and uglifier for security

Two security updates for our dependencies.

Uglifier: https://zyan.scripts.mit.edu/blog/backdooring-js/
Nokogiri: https://github.com/sparklemotion/nokogiri/blob/master/CHANGELOG.rdoc#167rc4--2015-11-22

@dsander

:+1: Not a huge fan of depending on a release candidate, but considering the CVEs it is probably needed.

Agreed; thanks!


Add info to CONTRIBUTING about how to setup the repo

Add a really short and basic description of how to setup the development environment. Even though it's very brief, I think it's helpful for newer OSS contributors and over-thinkers to have a dev setup guide in the contributing guide.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[-] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[-] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
[-] If this is a new cop, consider making a corresponding update to the Rails Style Guide.

Happy to update this with any feedback or suggestions about how this section could be changed!


[Feature Request]: Follow Skype for Business status while presenting or being in a call

Before you file a crash report

[X] I have checked the issue tracker and have not found an issue that matches the one I'm filing.
[X] This issue doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[X] I have read the contribution guidelines.

Mattermost Desktop Version
5.4.0
Operating System
Windows 10 x64
Mattermost Server Version
8.1.0 ESR
What type of crash did you experience?
White screen (I can see the top bar, but I cannot see my Mattermost server)
Crash report details
This ain't a crash report, sorry for misusing this category.

I'd like to propose the following features:
- when I'm in a call on Skype for Business, it would be nice, if there was a status change in Mattermost to something like "in a call"
- this status may also apply to Mattermost calls, so that other users can see if I'm already in a call or am free to call
- when I'm presenting in Skype for Business, it would be nice, if there was a status change in Mattermost to "Do not disturb" or at least the notification messages should be silenced and turned off for the duration of the presentation

Log Output
No response
Additional Information
No response

As stated in the Bug issue template:
You can request features and make product suggestions here: https://mattermost.com/suggestions/
Closing.


support  tags

Right now it only seems to work in Chrome with new Audio(). This patch will also support document.createElement('audio').

Sweet, thanks! :)


supoort serialize object to json

parse_into is amazing and convinent to parse json to object.
Would be great to support serialize object to json as inverse.

Closing as duplicate of #799. But just to be clear, this is being worked on.


Make "Objective-C compatibility" header bigger

What do you think about instead of making the header bigger to link this header from right from the top? Something like "written in Swift, [compatible with Objective-C](link to the header)"?

@mattab I just did that. Let me know if you are fine with this change.

:+1:
Maybe also rename "Advanced" to "Advanced usage" and make titles below the right level so they are bigger and there are title less level.

Good point.


fix performance issue with multiple connections

Benchmarks from https://github.com/zenithdb/zenith/issues/799#issuecomment-979989814 showed that pgbench with one connection gives similar TPS on safekeepers and postgres synchronous replication, but if pgbench is started with multiple connections, safekeepers approach gives worse results compared to synchronous replication.


Benchmark Istio Status feature

The Istio status CRD feature has been implemented in a beta mode.
We should work on making it the default behavior in net-istio. To do this we need to benchmark the feature and help Istio folks iron out any performance issues we may find.

/remove lifecycle/stale


better sliding window

Summary:
We only use aggregates from all the samples we store in sliding window.
When we are hit with 80k QPS and have sliding window of 60s, it means we store and process 4.8 million stat samples, each storing time and value itself. It's not cheap, both memory wise (golang's Time is relatively big, storing two 64bit ints for time itself plus a pointer to Location, so at least 24 bytes on 64bit system).
We then copy those slices of values many times, and eventually sort all those millions of samples to get min/max and avg.
But what if I tell you that we don't need to store the samples at all?
Instead, we store a ring buffer of N aggregates (which are very cheap thanks to Welford algorithm).
This allows us to keep 'sliding' aggregates while not bothering with removal of outdated samples.
Reviewed By: deathowl
Differential Revision: D45698916

Codecov Report
Patch coverage has no change and project coverage change: -29.83 :warning:

Comparison is base (f6a1707) 58.98% compared to head (a9aa949) 29.16%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #31       +/-   ##
===========================================
- Coverage   58.98%   29.16%   -29.83%
===========================================
  Files          69       10       -59
  Lines        8172     1406     -6766
===========================================
- Hits         4820      410     -4410
+ Misses       3034      956     -2078
+ Partials      318       40      -278

see 59 files with indirect coverage changes

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


Dockerize and create deploy button for Vercel support

Would be a great idea to make this a 2 step solution. You click the deploy button on the readme and it will spin up an instance of satdress for you given certain criteria (aka domain). Something for the future, but putting there so we don't forget. Could be Herkou as well!
https://vercel.com/docs/more/deploy-button

@andrerfneves this would be indeed greatz

We can't do it on Vercel because Vercel only runs Nodejs stuff, right?
Heroku should work though, and there are probably other Heroku competitors these days.


Pool Namspaces

Does the pool need to have a slice of pointers?

Not necessarily. I used pointers to keep memory allocation low, and also because it made the test easier.
I'm currently playing around with stdlib's container/ring for Nodes instead of a []*Namespace.

Cool deal. I'd say link the PR to this issue when you get the branch ready


#16477: fix deleted user profile not showing

Describe your changes:
Fixes #16477

I worked on fixing deleted user profile not showing in details page
add manage button in detail page to support delete and restore functionality
Added playwright test for the feature.

Failure need to work on :-

Feed's API are failing for deleted user.
Admin User deleting himself API also fail, but the user is deleted
Above issue  assign to @mohityadav766

https://github.com/open-metadata/OpenMetadata/assets/66266464/c10990e0-9f1a-44d5-9add-a3968a0e02ae

Type of change:

[x] Bug fix
[x] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@ShaileshParmar11 added playwright test for soft & hard deleted and restore user from ProfilePage

@Sachin-chaurasiya changes made as per comments!


AttrDict does not provide simple constructor

This somewhat relates to #18, but I would like to raise this issue independently.
> dict(a=23) == dict(dict(a=23))
True

> AttrDict(a=23) == AttrDict(dict(a=23))
...
TypeError: __init__() got an unexpected keyword argument 'a'

The **kwargs based constructor of dict is incredibly helpful.

Fixed as of 2.0


Add setting to restart with a different datadir

From the forums:
Will be good to add settings to choose path to folder with data.
It will not need to download everything from scratch so it could use data from the old client.

This can be invoked with the datadir conf setting on a restart.

This is a duplicate of #102, which has been closed.


Update cleanModelUrl to support non-main branches

This is a small change to cleanModelUrl to allow for more flexibility and backwards compatibility in the modelUrl. The function will now only append "resolve/main/" if the provided URL does not already contain "resolve". Tested with the following URL formats to ensure that all work.

https://huggingface.co/USER/MODEL
https://huggingface.co/USER/MODEL/
https://huggingface.co/USER/MODEL/resolve/branch
https://huggingface.co/USER/MODEL/resolve/branch/

Thank you! I think this is a good change; could you add two things if you have a chance (otherwise I can also do it):

Update the unit test here to include the four cases you listed https://github.com/mlc-ai/web-llm/blob/main/tests/util.test.ts#L30

Can be verified with npm test (not yet integrated to CI yet)

Update the docstring for ModelRecord.model to list the four formats we expect: https://github.com/mlc-ai/web-llm/blob/main/src/config.ts#L239

Thanks!

Done and done!
 PASS  tests/util.test.ts
  Check getTopLogprobs correctness
    ✓ Correctness test 1 (3 ms)
    ✓ Zero top_logprobs (1 ms)
  Test clean model URL
    ✓ Input does not have branch or trailing /
    ✓ Input does not have branch but has trailing / (1 ms)
    ✓ Input has branch but does not have trailing / (1 ms)
    ✓ Input has branch and trailing/

Test Suites: 1 passed, 1 total
Tests:       6 passed, 6 total
Snapshots:   0 total
Time:        1.731 s, estimated 4 s
Ran all test suites matching /tests\/util.test.ts/i.
Done in 2.26s.

Oops, sorry, just got notified that I missed a semicolon here:
https://github.com/mlc-ai/web-llm/blob/main/src/support.ts#L75
I hope that is a quick fix, sorry again, can fix it myself if that's preferred.

No worries! I'll fix it in the next PR


kcat.strangesoft.net site is gone

👋  just to raise the issue in here since the old artifacts are not accessible either.

IIRC alure2 has never been documented anywhere, so it's probably irrelevant to the mentioned site.

@McSinyx I am sorry, I dont quite understand what you referred to. I think the site along with the releases artifacts are all gone.

What I mean was that this repository is alure 2, which is the C++ rewrite of alure 1.x (the one on the site) with a different API.

I see, that makes sense. So it is understandable the site got removed?

I don't think so, since the openal-soft's docs were also hosted over there; just wanted to point out that there is no alure2 resource on that site.

The strangesoft.net domain URL expired. I still have everything on the data host server, and will check with the server admin to see if there's another way to publicly access it. openal-soft.org is using the same host and is unaffected.

The strangesoft.net domain URL expired. I still have everything on the data host server, and will check with the server admin to see if there's another way to publicly access it. openal-soft.org is using the same host and is unaffected.

that sounds pretty good, let me switch to openal-soft.org (at least in this way, alure1 is rebuildable). Thanks @kcat!

hmm, I was mistaken, yeah, we still need kcat.strangesoft.net or any site to allow the allure1 artifacts publicly accessible.

The page can be accessed through https://kcat.tomasu.net/alure.html now. They're asking way too much to give the strangesoft.net domain back, so it's unlikely that will be back any time soon, unfortunately.

The page can be accessed through https://kcat.tomasu.net/alure.html now. They're asking way too much to give the strangesoft.net domain back, so it's unlikely that will be back any time soon, unfortunately.

yeah, I dont care which domain pointing to. In fact, I think .dev domain might be cheap ones.

yeah, I dont care which domain pointing to. In fact, I think .dev domain might be cheap ones.

Close the issue for now as the site is back. I have filed a PR to revert my change on the homebrew side.

Close the issue for now as the site is back. I have filed a PR to revert my change on the homebrew side.


Dead link in website

http://fountainjs.io/doc/doc/usage
From here:
http://fountainjs.io/doc/

Thank you for your report.
Just fixed :+1:


Add agent persistent for Copilot agents

We're working on building a Copilot agent.
We're doing user studies and have watched 2 people so far using the agent. Both (and one was the extension dev) have expected that once they invoke the agent subsequent prompts would go to the same agent.
Perhaps this could take the form of pre-populating the chat input box with the agent name once the agent has been used until the user starts a new chat or deletes the agent name from the chat.

you have the shouldrepopulate function for that                 readonly shouldRepopulate?: boolean;

This is in progress: https://github.com/microsoft/vscode/pull/204910

We have API for this now, and chat participants can opt in


Support Node v4 LTS semver-minor releases

Fixes #6473

Replace Node v4.2 with v4 in Travis config
Replace ~4.2.0 with ^4.2.0 in engines to support semver-minor bumps in LTS line

This should mean we don't have to do anything when v4.4 comes out in a few weeks.


python310Packages.botocore: 1.29.79 -> 1.31.9

Description of changes
My motivation for making this update now is to better prepare for updating botocore for urllib3's new major version.
Things done

Built on platform(s)

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

[x] 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/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

can you retarget to python-updates?

Retargeted. Thank you for the review.


Error while loading dictionary mmcif_pdbx.dic, while in conda-forge environment

Hi! I am trying to create an nf-core module from dssp to make it more useable by the bioinformatics Nextflow/nf-core community.
I am using the conda-forge::dssp=4.4.8 environment, and when I try to execute:
mkdssp 1tim.pdb test.dssp

I get the following error:
Error while loading dictionary mmcif_pdbx.dic                                                                                │
│      >> basic_filebuf::underflow error reading the file: Is a directory

Is there a solution for this?
Thanks!

This is not an issue with dssp itself, it is a configuration error in the conda environment. Please submit this bug to the dssp feedstock in conda


Bug bounty testing, please do not approve. roelfy

Bug bounty testing, please do not approve. roelfy

This looks off topic.

I flagged this PR to github. I don't know if you're a legitimate security researcher or not, but please do not do this again.


yii 2.0.14 -> <IP_ADDRESS> update broke csrf validation

What steps will reproduce the problem?
Update from yii 2.0.14 to <IP_ADDRESS>
What is the expected result?
Working csrf validation.
What do you get instead?
Bad Request (#400)
Additional info
parent::beforeAction($action) fails inside beforeAction of my site controller. Inside Yii::$app->getRequest()->validateCsrfToken(), $trueToken differs from $this->getBodyParam($this->csrfParam) (which is the one from the page's source, meta & input is identical). Looks like trueToken gets regenerated somehow.
I'm using the yii2 advanced template, csrf meta tags are set. The form of the POST is setup as described here: https://yii2-cookbook.readthedocs.io/csrf/ (Adding CSRF protection).
Please note that I haven't changed any line of code between 2.0.14 and <IP_ADDRESS>. I'm able to reproduce the bug by downgrading to 2.0.14.

Q
A

Yii version
<IP_ADDRESS>

PHP version
PHP 7.0.19

Operating system
Debian

Is that advanced template w/o modifications? How could we reproduce it?

according to the diff it could be related to #15783. relevant changes: https://github.com/yiisoft/yii2/compare/2.0.14...<IP_ADDRESS>#diff-12395c67a35c91aa5fc9ac3f6554375fR260

Is that advanced template w/o modifications? How could we reproduce it?

Nope, we just used the advanced template as base, the site is fully customized.
I'll try to reproduce it with a standard installation.

Take a look at this article https://segmentfault.com/q/1010000004450797

Do you have it cached? Getting "503 Service Unavailable" all the time...

I'm a little bit late but currently investigating the problem. Unfortunately, I haven't found the cause yet, but there seems a little difference between my live environment and the default installation (advanced).
When visiting the default site an "advanced-frontend" cookie is set, even when removing the session name and setting 'enableSession' => false in the main config (doing this the cookie is called PHPSESSID). I'm not sure why this cookie is set even if enableSession is false, and why my live env is acting differently with the same config. Maybe someone has an idea?

OK, seems not to be bug of the framework itself but edge case. Should work with the change you've made.

Uh... maybe it not goes here, but I have the following form:
<?php $form = ActiveForm::begin(['options' => ['enctype' => 'multipart/form-data']]); ?>
        <?= Html::input('hidden', \Yii::$app->request->csrfParam, \Yii::$app->request->getCsrfToken()) ?>

                    <?= ($tema->archivoaudio === '') ? "No hay archivo de audio asociado" : $tema->archivoaudio ?>

                    <?php
                    if ($tema->archivoaudio !== ''):
                        ?>

                            <?=
                            Html::a('Eliminar Audio', ['enlace/delete-audio', 'id_tema' => $tema->id], [
                                'data' => [
                                    'confirm' => '¿Desea eliminar el archivo de audio?',
                                    'method' => 'get'
                        ]])
                            ?>
                            <?php
                        endif;
                        ?>

        <?= $form->field($tema, 'tituloaudio')->textInput(['maxlength' => true]) ?>

        <?= $form->field($audioFile, 'file')->fileInput(['required' => true]) ?>

        <?php //Html::input('file', 'audio', null, ['required' => true]) ?>

            <?= Html::submitButton('Guardar', ['class' => 'btn btn-success']) ?>

        <?php ActiveForm::end(); ?>

I have commented a Html file input to achieve a validation via custom model. As you see, I have set 2 models (an active form and a non activeform). I have set a hidden input with csrf token because the activeform seems not do it with more than 1 model. The data is obviously sent via POST.
With this, the csrf token seems not be sent but tracking via xdebug and browser, the token goes but the request component seems not catch the parameter. Maybe a bug in PHP or in the Framework?
I use PHP 7.2 in LAMP.

@Zerebokep thanks a lot! I tried to generate a model with gii and got this errors, because of this regenerateCsrfToken in login - other tab made requests to REST endpoint and changed token:)


Add an "experimental" flag for rules

New rules, particularly ones which replace deprecated rules (see #184) should be introduced with an "experimental" flag.
Experimental rules would not be added/run automatically
When many experimental rules exist at the same time, individual rules can be enabled in the .lint configuration file (mechanism to be determined).
All currently experimental rules can be enabled with a CLI flag. This would allow users to "dry run" the latest changes against their dashboards/repositories.
When an experimental rule becomes part of the standard ruleset, the .lint configuration should be checked for tidiness. I.E. Formerly experimental rules that are still "enabled" in the config can be removed, the user should receive a warning/text output on on STDOUT.

@rgeyer,
As for selectively enabling experimental rules, instead of enabling all rules by providing an "experimental" flag.
Would we not be able to "avoid" the config check, warning text etc. if we turn it around and let it work the same way as the "stable" rules does?
So if one would want to use 2 out of 3 experimental rules, then they use the "experimental" flag and then create an exclusion for the experimental rule they do not want?

@rgeyer,
As for selectively enabling experimental rules, instead of enabling all rules by providing an "experimental" flag. Would we not be able to "avoid" the config check, warning text etc. if we turn it around and let it work the same way as the "stable" rules does?
So if one would want to use 2 out of 3 experimental rules, then they use the "experimental" flag and then create an exclusion for the experimental rule they do not want?

Right, excellent point. I wrote this up in a hurry and didn't think through it. I think your solution makes the most sense.
I'll update the issue accordingly. Thanks!


Adapt model comparison to new prompt generator syntax

https://github.com/computational-cell-analytics/micro-sam/blob/master/micro_sam/evaluation/model_comparison.py#L66-L76

This is done.


support websocket schemes

The client should be able to use websocket (ws:// and wss://) as scheme in addition to http:// and https://

This ticket is labeled with raspberry pi 3 bounty. Solve the ticket and I'll send you a pi

Is it already supported?

It probably requires some work to bring it up to date again, but we did create this: https://github.com/go-openapi/swaggersocket


Incompatibility with Kotlin Gradle Plugin Version in Flutter Project

When attempting to launch the Flutter project on a Redmi 8A Dual device in debug mode, the build fails with an exception. The error message indicates an incompatibility between the Kotlin Gradle plugin version required by the Android Gradle plugin and the version found in one of the project dependencies (wakelock).
Launching lib/main.dart on Redmi 8A Dual in debug mode...

FAILURE: Build failed with an exception.

* What went wrong:
The Android Gradle plugin supports only Kotlin Gradle plugin version 1.5.20 and higher.
The following dependencies do not satisfy the required version:
project ':wakelock' -> org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.50

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 740ms
Running Gradle task 'assembleDebug'...                           1,248ms
Exception: Gradle task assembleDebug failed with exit code 1

New updated version is up: 0.8.0. This should solve all the versions issue. Feel free to open this issue if needed.


UI/add participant

Fixed an error that occurred when entering excessively high capacity values, preventing server crashes.
Improved the add participant UI layout for a cleaner and more user-friendly experience.

So clean user experience and good work on the bug handling.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

manager/views/participant_views.py
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Flag
Coverage Δ

unittests
74.75% <66.66%> (-0.01%)
:arrow_down:

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

Files with missing lines
Coverage Δ

manager/views/queue_views.py
46.35% <100.00%> (+0.28%)
:arrow_up:

manager/views/participant_views.py
40.18% <50.00%> (+0.06%)
:arrow_up:

Great fixes!


Issues installing

So I've done all the prerequisites, but when I go to install via pip I get this error:
Command "/usr/bin/python -u -c "import setuptools, tokenize;__file__='/tmp/pip-build-Orkqqw/hidapi/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-qyFCCh-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-build-Orkqqw/hidapi/
This is literally my first day on linux, so please excuse any newbie mistakes.

Hello,

Have you installed rivalcfg

from this repository (git clone ...)
or from PyPI (pip install rivalcfg)?

What is your Linux distribution?

I tried both ways. Just tried installing again and it worked for some reason, not sure what changed.


fix: poll_tick does not register waker if ready

Interval::poll_tick does not register waker if it returns ready

Polls for the next instant in the interval to be reached.
This method can return the following values:
Poll::Pending if the next instant has not yet been reached.
Poll::Ready(instant) if the next instant has been reached.
When this method returns Poll::Pending, the current task is scheduled to receive a wakeup when the instant has elapsed. Note that on multiple calls to poll_tick, only the Waker from the Context passed to the most recent call is scheduled to receive a wakeup.

Codecov Report

Merging #2276 (1304ea9) into main (8b4a73d) will decrease coverage by 26.48%.
The diff coverage is 66.66%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #2276       +/-   ##
===========================================
- Coverage   72.06%   45.58%   -26.48%
===========================================
  Files         472      472
  Lines       56742    56749        +7
===========================================
- Hits        40891    25869    -15022
- Misses      15851    30880    +15029

Flag
Coverage Δ

integration-tests
18.37% <66.66%> (-0.03%)
:arrow_down:

unit-tests
38.99% <66.66%> (-28.11%)
:arrow_down:

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

Impacted Files
Coverage Δ

crates/payload/basic/src/lib.rs
0.30% <0.00%> (-0.01%)
:arrow_down:

crates/net/discv4/src/lib.rs
42.49% <50.00%> (-22.40%)
:arrow_down:

crates/net/network/src/peers/manager.rs
51.10% <100.00%> (-32.86%)
:arrow_down:

crates/net/network/src/session/active.rs
33.92% <100.00%> (-51.29%)
:arrow_down:

... and 210 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

discovered while looking into #2273
it didn't have any impact on the components touched by this pr because there are other things that wake up the tasks, network IO etc


Support for localization

Hello, thanks for the binding.
Would be nice if you could support localization too, I see apexchart supports localization https://apexcharts.com/docs/localization/
and I see chart options supports the Locale functions but it's a chart instance options
Would be nice to have a better localization support, can it be set once and work everytime without specifying it on every chart instance?
I've never used apexchart JS but looking at the doc I see you can set locale globally
Apex.chart = {
  locales: [...localeConfigurationObject],
  defaultLocale: "en"
}

In a blazor fascion I could imagine an API like
ApexChart.Configure(options =>
   options
      .AddLocale(MyCustomLocaleDefinition)
      .SetDefaultLocale("en")
)

to use in Program.cs

Hello,
I have looked inot this before, but we felt that it's as easy just to have a singleton service that would generate the a default chart options instance. then you don't have to re-create it each time.
Not sure if this make sense to you.
Regards Joakim


treewide: convert 21 fonts to stdenvNoCC.mkDerivation

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] 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/)
23.05 Release Notes (or backporting 22.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.

@ofborg build andagii andika ankacoder annapurna-sil babelstone-han barlow borg-sans-mono cascadia-code charis-sil comic-relief courier-prime cozette d2coding doulos-sil eb-garamond emacs-all-the-icons-fonts encode-sans ezra-sil fira-code fira-code-symbols fira-mono


Implement Display of Destructive Changes

The migration engine has now implemented support for checking for destructive changes. The output of applyMigration can now return a non empty array in the warnings property. If that is the case the CLI must send the force parameter as true so the user acknowledges the destructive change.
The JSON for warnings looks like this:
[
  {"description": "You are about to drop the table `User`, which is not empty (11 rows)."},
  {"description": "You are about to drop the column `name` on the `User` table, which still contains 23 non-null values."}
]

Note: The migration engine currently ignores the force flag and applies the migration even if there are destructive changes. This is temporary until the CLI has implemented support for this and we can enable the proper behavior.

I have created a specs issue about this topic.

@mavilein do we receive enough information here to show a diff?

@matthewmueller : No. Currently it is just a plain String describing the data loss. What kind of diff would you like to show?

@matthewmueller : How are your proposals related to destructive changes? I think we have this kind of data model diffing in the CLI already. A deleted field might not lead to data loss at all hence i don't see how this moves this forward.

The spec is not describing the --force flag or a replacement that we had in P1. To my understanding we definitely need that.

I originally had a confirmation dialog with an --auto-approve dialog during lift up. I removed that flow after I learned that lift up doesn't seem to use it.

We should think about when we want to display destructive changes. Shouldn’t we maybe show already in lift save? It would be annoying to run lift save and later find out that the migration is actually destructive when running lift up.

How are these messages obtained? It feels like it would be slow to learn some of this.

@matthewmueller : What do you mean with How are these messages obtained??

@mavilein for example: "You are about to drop the column name on the User table, which still contains 23 non-null values."
I'm assuming this would do something like: select count(name) from User where name is not null?
If that's the case, could that get really slow on large datasets? And it would compound if you're making sweeping changes. I'm trying to get a gauge of how slow this would be when i type lift save for an Airbnb.

@matthewmueller : Yes this is what we do. I think this is a valid concern. We have not had problems with this approach in Prisma 1 though.


Draft FAQ - Adding more FAQs from Discord.

Line 101: it'l -> it'll
Also what do you mean by "switch everyone in the game to this" on line 93?

Adjusted 👍

Lots more could be added, but I guess this is good for now, ready to be reviewed/adjusted/released. @MrGadget1024


Add baseline feature

If there are a lot of violations in your codebase and you can't at the moment fix them,
you can use the baseline feature to instruct the tool to ignore past violations.
To create a baseline file, run the check command with the set-baseline parameter as follows:
phparkitect check --set-baseline=my-baseline.json

It will produce a json file with the current list of violations.
To instruct the tool to use the baseline file, run the check command with the use-baseline parameter as follows:
phparkitect check --use-baseline=my-baseline.json

Codecov Report

Merging #314 (8adfaa8) into main (392757f) will increase coverage by 0.28%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #314      +/-   ##
============================================
+ Coverage     93.02%   93.30%   +0.28%
- Complexity      476      480       +4
============================================
  Files            60       60
  Lines          1262     1285      +23
============================================
+ Hits           1174     1199      +25
+ Misses           88       86       -2

Impacted Files
Coverage Δ

src/CLI/Command/Check.php
99.00% <100.00%> (+0.21%)
:arrow_up:

src/Rules/Violations.php
100.00% <100.00%> (ø)

src/Analyzer/NameResolver.php
74.83% <0.00%> (+1.29%)
:arrow_up:

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

I was thinking that maybe could be a little more usable if:

we decide a standard baseline filename
if the file is present is used even without any option specified
the --use-baseline is used only when you want to use a non-standard filename (is needed? maybe we can avoid it)
we add a --skip-baseline option to avoid to use the default baseline

The idea is that if a baseline is present is much more comfortable not to need to say it explicitely every time..

Make sense, we could do it in a next step. What do you think?

@fain182 @pfazzi for the sake of consistency it could be worth checking what other tools are doing, what do you think?

@fain182 @pfazzi for the sake of consistency it could be worth checking what other tools are doing, what do you think?

I basically copied the same behavior of psalm :D

I am not sure about improving it in the future because it would be a breaking change..
Phpstan uses --generate-baseline that is slightly more explicit, and if you don't pass any filename it uses the default filename.


[FEATURE] Add a footer

Description
Hey I wanted to add a footer which navigates to different pages includes about us contact us your social media links and little about motive of this website
please assign it to me under hacktoberfest 2022
Screenshots
No response
Additional information
No response

We have (kind of) infinite scrolling in our application, so how would a footer help and be implemented @abhijain2003 ?

by changing the scrolling to grid of infinite X 3 in large screen
and making the position fixed of about us will add more value to website

@mathiasayivor  what do you suggest sir?

okay @abhijain2003

thanks

Is this still available?

asking @abhijain2003 if he's working ?

@Kritika30032002 yes, it's available but what improvement do you suggest?

Open for contribution in KWOC

Hey @narayan954
Can I get this issue assigned?

Hey @narayan954 Can I get this issue assigned?

Sure, but first please tell how'd you like to implement this?
Also please mention the program KWOC/Codepeak .
Thanks

Hey @narayan954 Can I get this issue assigned?

Sure, but first please tell how'd you like to implement this? Also please mention the program KWOC/Codepeak . Thanks

Was planning on using react, css, html to create the footer..

I am new to open source, and have little experience in webd..
I am very intrigued by this project.. would like to learn and contribute simultaneously...

I would like to work on the issue ....
program:codepeak

Sure, you can go ahead @pranavajith. It's just that I've a little doubt about how'd a footer be implemented for an app that has potentially infinite scrolling.

I would like to work on the issue .... program:codepeak

sorry @Tushar98644 but the issue is claimed already, you can some other issue for now :)

@narayan954
Could you give me some directions on what languages to use, and how to proceed?

@pranavajith My suggestion is a footer on the left bottom fixed, similar to the new Instagram desktop ui(which is on the right, but we don't have that much content for that position). What's your suggestion?
You can implement that in react.

@pranavajith My suggestion is a footer on the left bottom fixed, similar to the new Instagram desktop ui(which is on the right, but we don't have that much content for that position). What's your suggestion? You can implement that in react.

Gotcha.. working on it as we speak.. thanks! Will get back to you with the developments!

Hey @narayan954 Can I work on this issue. Also tell me further how should I proceed.

Yes, you can :), follow the suggestions mentioned in latest comments and please mention your program, kwoc or codepeak

Yes, you can :), follow the suggestions mentioned in latest comments and please mention your program, kwoc or codepeak

I want to work under Codepeak. And can I create a new footer.jsx file in the src folder. Will it be ok?

Yes, you can :), follow the suggestions mentioned in latest comments and please mention your program, kwoc or codepeak

I want to work under Codepeak. And can I create a new footer.jsx file in the src folder. Will it be ok?

Great and I'd suggest creating a footer component instead.

Hi I would like to work on it for #jwoc

sorry but coding period hasn't started for jwoc yet, please follow the guidelines.

okay

I want to work on this issue under JWOC

Cool!, best of luck!

Yeah, please add it as a component. External additions might show side effects in rendering the page.

I have newer to react so i will try to add plz give me some time if possible

Actually I am newer to react so i will try to add plz give me some time if possible

Fine, take it easy and feel free to ask for help :)

@KrunalGupta02 please update with your progress

I want to work on this issue under JWOC

great, go ahead!

@narayan954 will get back to u asap

@narayan954 can u guide me a little bit of wjat i need to do here???

@narayan954 ??

My suggestion is a footer on the left bottom fixed, similar to the new Instagram desktop ui(which is on the right, but we don't have that much content for that position). What's your suggestion?
You can implement that in react.

Yeah sure, you have to create a footer and for that you have two options as in This Solution
Please confirm anyone of them and feel free to ask for more information

can i contribute to this ...

can i contribute to this ...

yes sure you can, but first please confirm your approach on this. Take your time to go through this issue and discuss how'd you solve this one. You can find details, help and context in comments above/before you

we can keep footer as fixed at bottom but again it would not look good else we can add more option at bottom

otherwise sticky footer

we can keep footer as fixed at bottom but again it would not look good else we can add more option at bottom

Yes exactly, that's why I am in doubt as well, we may look at other similar social medias and how they dealt with this.

@abcxyz31  are you working on this?

I want to work on this issue under GSSOC'23

@narayan954 sir I want to work on this issue under GSSOC'23

Sure, go ahead, but what's your approach

sorry sir I cant do this task for some reason
sorry for inconvenience

sorry sir I cant do this task for some reason
sorry for inconvenience

it's alright


Article date and weekly date order is wonky

The article's date is showing 5/19/2019, but it was updated a few days ago.  The listing of updates by week is also showing some inconsistencies. See order of updates by week below:
Week of March 02, 2020
Week of March 09, 2020
Week of March 16, 2020
-- Then it skips to --
Week of February 17, 2020
Week of February 24, 2020

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

ID: 3081f70c-d927-ce5d-d957-ca7ba7e07a55
Version Independent ID: 17704162-7ff9-015c-30a1-1cbc078f0782
Content: Microsoft Teams content updates - Microsoft Teams
Content Source: Teams/teams-updates.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@ndambrosio  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@ndambrosio thanks for reaching out. Unfortunately, these updates are generated automatically each week and I don't have the permissions to update in the correct order. This is however on the teams radar to be fixed in the near future.

@scanum this can be closed thank you.

@ndambrosio Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Update gce.md

fix bad url

This change is

@wackxu Why did you close this PR? Looks like you were able to update the URL correctly.

@chenopis I am sorry and I have reopen it.

@wackxu No worries. Thx!


SBS Feedback: 6a and 6d - Check maps centring on mobile

This seems to be intermittent/may depend on mobile device.  Can we investigate/check for patterns.

I can help with this. I have access to a BrowserStack account which I can use to do the testing. What would you like me to look at specifically?


Clean up

Fixes Issue

Changes proposed

Screenshots

Note to reviewers

Seperate Links object and Routes from database/data.ts file

It has be resolved now ma'am.


Add build scripts support from auro-library

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #458
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
This PR adds completed SSR support for newly created elements.
Type of change:
Please delete options that are not relevant.

[x] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Double clicking an anchor point with the Path tool should always create two handles

For polyline primitive shapes like the n-gon, double clicking an anchor currently creates one handle (converting the shape from a polyline to a quadratic bezier curve). But the user will likely want two handles most of the time (making it a cubic bezier curve).

Some context in these two commits:
https://github.com/GraphiteEditor/Graphite/commit/d44e7ff925dac79eb372e5f70a77e3b7a95f8a6a
https://github.com/GraphiteEditor/Graphite/commit/74b2737f1f6fdc42a4a255363ef767c9486413cd

Hi , trying to solve this issue


fix(sidebar): Fix resize using different value on mac

What this PR does 📖

Fixes resize event using different window size value on mac which causes sidebar issues
Seems to also be an issue on windows? Please verify again with this PR

Which issue(s) this PR fixes 🔨

Resolve #971

hmm. this might be fixed in my other pr. maybe it is better if i merge them all together and close this one? since they all deal with sidebar

merged into #1136


arrow-down, arrow-left: not the same

down-arrow is not an arrow. It is a triangle. There is no down-facing arrow. There is a left-facing triangle called left-triangle-arrow.
Perhaps these naming conventions could be tidied up?
I recommend first keeping them consistent among themselves, and then possibly just calling them the one thing they are. A triangle is a triangle. An arrow is an arrow.

Indeed. This something we need to work on with the icon team.
We can also consider aliases for backwards compatibility.

Doing some issue pruning... @julio8a thoughts on deprecating the icons in CW and enforcing use of https://github.com/ArcGIS/calcite-ui-icons ? There seems to be some overlap between the two sets and it'd be nice to consolidate and use one.

@macandcheese, All the icons from CW are in the calcite-ui-icons repository. I talked with @paulcpederson yesterday and we discussed adding a deprecation message for the icons in CW.
deprecation message we used in arcgis-apps-icons:
DEPRECATED
Please migrate to: calcite-ui-icons
The arcgis-apps-icons repository along with the reference page will be deleted on 9/14/2018.

We should open a separate issue for moving to calcite-ui-icons. This issue can be closed as we won't control naming anymore.


Add fast abi test component

This PR adds a test component to test fast abi consumption. The contents of the test component are copied from cppwinrt/test_component_fast

test_component_fast (and test_component_base, test_component_derived) don't follow the TestComponent pattern, which enforces 100% coverage of producer and consumer, and validates both caller and callee values (see TestRunner.cpp).  Can we wire that up so that we have confidence?


Added option to select and unsellect all items in SelectComponent

Added option to select and unsellect all items in SelectComponent  and updated demo with a corresponding example.

hi @valorkin,is the PR ok to be merged?

Hey, I am on my way to conference, I will try to review it soon

@valorkin,is it OK yet?

Hello, @valorkin is it OK now?

hello,is the repo still being maintained ? @valorkin


The JsonPathDetector class does not handle empty objects correctly

The JsonPathDetector class handles JSON objects and arrays slightly differently, due to the need to recognize an object's property keys. To handle this, the JsonPathDetector state machine uses an additional "pending" state when processing objects. During the "pending" state, no path item has been pushed for properties of the object.
When the closing brace of an empty object is encountered, the state is still "pending" since no properties have been seen. The current implementation of the state machine currently assumes that a path item has been pushed. It therefore incorrectly pops the key of the enclosing object or array.
Consider the following JSON fragment:
{
  "outer": {
    "inner": {
    } // Should have path [ "outer", "inner" ], but has path [ "outer" ]
  }
}

When a JsonPathDetector object encounters the closing brace of the "inner" object, the path is just [ "outer", "inner" ] because no nested properties have been found. Ordinarily the key of the last nested property would be popped, but the "inner" key is popped instead. The JsonChunk is therefore incorrectly given an [ "outer" ] path. The paths of all subsequent JsonChunk objects will then be incorrect.

I have implemented a unit test and fix for this issue, but cannot push it due to insufficient access.

Unit test that illustrates the problem:
test("PathDetector handles empty object", async () => {
	const stream = serializeJsonValue({
		outer: {
			inner: {}
		}
	}).pipeThrough(new JsonPathDetector());

	expect(await streamToArray(stream)).toEqual([
		{ ...objectStart(), path: [] },
		{ ...stringStart(StringRole.KEY), path: [] },
		{ ...stringChunk("outer", StringRole.KEY), path: [] },
		{ ...stringEnd(StringRole.KEY), path: [] },
		{ ...colon(), path: [] },
		{ ...objectStart(), path: ["outer"] },
		{ ...stringStart(StringRole.KEY), path: ["outer"] },
		{ ...stringChunk("inner", StringRole.KEY), path: ["outer"] },
		{ ...stringEnd(StringRole.KEY), path: ["outer"] },
		{ ...colon(), path: ["outer"] },
		{ ...objectStart(), path: ["outer", "inner"] },
		{ ...objectEnd(), path: ["outer", "inner"] },
		{ ...objectEnd(), path: ["outer"] },
		{ ...objectEnd(), path: [] }
	]);
});

Pull request https://github.com/cdauth/json-stream-es/pull/2 has been created to address this issue.

Brilliant, thanks for the report, the detailed explanation and the fix! I've released v1.2.1 containing the fix.


clasp create -> Internal error encountered.

% clasp create --title "My Project" --type Spreadsheet --rootDir ./src
Internal error encountered.

% clasp -v
2.4.1

Same

% clasp create --title "My Project" --type standalone --rootDir ./src
Internal error encountered.
% clasp -v
2.4.1
% node -v
v14.17.0
% clasp clone

Same here:
# clasp create --type sheets --title Test
Created new Google Sheet: https://drive.google.com/open?id=1tQX8qc_2fqLAdu0bVT7uUQ376eNQR9Pv7DfBfN9E-qc
Internal error encountered.

# clasp -v
2.4.1

# node -v
v16.17.0

Both spreadsheet and script project are being created, but when opening the script in browser it returns a 500 error as @bugkarma mentions.

Encountered the same issue with clasp create --type sheets on Nov 21, 2023, between 15-18 (JST). The .clasp.json file wasn't created and ended up with an internal error encountered. Couldn't create it directly from the browser either.
When I tried again later, it worked fine. There was a GCP outage in the Japan region around that time, which might have caused the issue. Saw similar reports from others on X.
If you're facing a similar error, it might be worth checking the GCP status page for any ongoing issues.
https://status.cloud.google.com/summary


Bumped production RELEASE for 1.21-17

Bumped production RELEASE for 1.21-17
closes #1143
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve

closes #1143

/unhold
/approve


Clicking on view text doesn't change views

Inside the clubs page, when clicking on "Map view" text or on "List view" text doesn't change the view.

fixed with #221


Access the directory as a signed in user

Why the "Access the directory as a signed in user" permission to be enabled for this. Does the "sign-in and read user profile" permission is not enough.

Hi,
I have raised this query on the below article.
https://docs.microsoft.com/en-us/power-bi/developer/embed-sample-for-customershttps://apac01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fpower-bi%2Fdeveloper%2Fembed-sample-for-customers&data=02|01|Pandi.Kaliappan%40lntinfotech.com|a140eb3886e243acb14308d6221989a9|02aa9fc118bc4798a020e01c854dd434|0|0|636733890410649287&sdata=3oQGQzMdgNoQE%2BLStdMG%2FlLXwBmoq2cX7ztRMAnyFro%3D&reserved=0
Regards,
Bharathi
IT Infrastructure | Information Systems Department | L&T Construction
From: Michael Blythe<EMAIL_ADDRESS>Sent: 27 September 2018 02:51
To: MicrosoftDocs/powerbi-docs<EMAIL_ADDRESS>Cc: Bharathi.AN<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/powerbi-docs] Access the directory as a signed in user (#283)
Hi @BharathiANhttps://github.com/BharathiAN -- can you give me more context on where this information is in the docs or product? Thanks!

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/powerbi-docs/issues/283#issuecomment-424873924, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AplHBxsVwiqMA6TmD3qzdKP6amZjpB-hks5ue-_FgaJpZM4W6CkH.
CAUTION: This is an external email. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Sensitivity: LNT Construction Internal Use
[L&T-Construction]              This Message and its contents is intended solely for the addressee and is proprietary. Information in this mail is for L&T Business Usage only. Any Use to other than the addressee is misuse and infringement to Proprietorship of L&T Construction. If you are not the addressee please return the mail to the sender.
L&T Construction.


chore(wallet): Upgrade RN to 0.72.3

Description
This PR ensures we're on top of RN upgrades and changes so it doesn't pile up too much down the road. This PR should also improve readability of error messages within the app when crashes occur and help with troubleshooting.

Upgrades react-native to version 0.72.3.
Upgrades other dependencies as needed.
Applies patch to react-native-camera-kit.

Type of change

[x] Dependency Upgrade

Tests

[x] No test

Slashtags seems to be broken on this build:

Thanks! Able to reproduce. Looking into it.

The error looks like it has to do with async generator code which I believe hyperswarm doesn't support. Think we had a polyfill for that once maybe that got removed or is now needed again? Maybe @limpbrains knows more.

@pwltr, I made the mistake of re-generating the yarn.lock file. As such, it bumped some of the slashtag-sdk dependency versions to something incompatible. Updated it here. Should be working as expected now.

@pwltr, I made the mistake of regenerating the yarn.lock file. As such, it bumped some of the slashtag-sdk dependency versions to something incompatible. Updated it here. Should be working as expected now.

this is why I'm for pinning all deps in package.json. Of course it doesn't solve all the issues

Any idea why e2e tests are failing? It looks like some async tasks are taking way longer in this build in the detox environment. I can make the tests pass locally by increasing the timeouts but on CI it's still not enough it seems.

Any idea why e2e tests are failing? It looks like some async tasks are taking way longer in this build in the detox environment. I can make the tests pass locally by increasing the timeouts but on CI it's still not enough it seems.

Same. I'm at a loss as to why this is failing.

Any idea why e2e tests are failing? It looks like some async tasks are taking way longer in this build in the detox environment. I can make the tests pass locally by increasing the timeouts but on CI it's still not enough it seems.

Same. I'm at a loss as to why this is failing.

Detox currently doesn't support this version of react-native so probably we will have to wait for that. I don't think it's worth breaking e2e tests unless there is major in this upgrade that we need?

Sounds good. Lets wait. One of the added benefits of the upgrade is the potential for slightly improved error messages for troubleshooting.

Before merging I think we need to test it ourselfs, especially android

Agreed. I'll run through some tests on Android in the morning.

Android build is working as expected. No hiccups thus far. Seems we've lost the e2e-ios test again? Not sure what happened there.

I think we can ignore it for now, I will try to fix it in another PR


Add deterministic metrics

[x] Closes #114.
[x] I am familiar with the contributing guidelines.
[x] Tests added.
[x] Updates entries to docs/source/api.rst for API changes.
[x] Adds descriptions to appropriate "what's new" file in docs/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

Add (standalone) functions for computing deterministic forecast error metrics to the metrics subpackage, including adding tests and updating the reports subpackage to use the functions. There are other metrics and functionality to be added to the metrics subpackage, but those will be addressed in future PRs.
Also, the commits from this PR could be squashed prior to merging.

datamodel.Reports dataclass spec includes allowable metrics. Not sure if we should make it less descriptive or if we should import a more descriptive list from the metrics module.

I'll start by revising the tests as suggested.

@wholmgren I still have more tests to add, but does what I added in d388df4 match with what you were thinking?

I was thinking of more simple, explicit inputs like true = [0, 1, 2]; test = [1, 0, 2] (but with appropriate types) and then asserting an exact value (e.g. mbe == 0, mae == 2/3). I think it's easier for someone to verify that the code is correct if it uses explicit input values.

Gotcha. I'll revise accordingly.

@dplarson looks good, thanks! I think we merge this after you've updated the API and whats new documentation. @alorenzo175 any concerns/suggestions?

Looks OK to me.

@wholmgren I've added commits updating the API and what's new docs.

@dplarson was there a reason that you didn't add KSI, OVER, and CPI in this PR? Just expediency? If so, can you make a new issue so that we don't forget about them? I feel like I might have forgotten a conversation about this.

@wholmgren It was primarily a timing thing. (I had meant to follow-up later, but forgot. So thanks for the reminder.)
I'll open a separate issue for KSI, OVER and CPI and then a PR to add them.


Missing required fields

Description
Related to issue https://github.com/opensearch-project/opensearch-java/issues/1040
Issues Resolved
Bug
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks! This is a good start, keep iterating on the test for a basic nodes info response.

Looks like a legit spec test failure.

Indeed, I will try to do this, but it's will depend of my time/work


Dsu reboots into bootloader instead of gsi

Device:
Xiaomi mi11i
Operation mode:
Root/System

same here


startOfHour-34edc6a8.js' doesn't provide an export named: 's'.

Checklist:

Please try updating to the latest available version.
Please try clearing your browser cache.

[REQUIRED] Card diagnostic information:

No cameras support this view. Check troubleshooting.

view: diagnostics
cameras_capabilities: {}

[REQUIRED] Description of problem:
Getting an error:
Camera initialization failed: The requested module 'https://mydomain/hacsfiles/frigate-hass-card/startOfHour-34edc6a8.js' doesn't provide an export named: 's'. Check troubleshooting.

[OPTIONAL] Last working release (if known):
Don't recall last working version, but it was working.
[OPTIONAL] Javascript errors shown in the web inspector:

[OPTIONAL] Additional information:

What version of the Frigate integration are you using?

Camera initialization failed: The requested module 'https://mydomain/hacsfiles/frigate-hass-card/startOfHour-34edc6a8.js' doesn't provide an export named: 's'. Check troubleshooting.

Hmm. Wondering if your browser is still caching files it shouldn't be, or your installation is somehow corrupted?

What version of the Frigate integration are you using?

5.6.0 with 14.1

Camera initialization failed: The requested module 'https://mydomain/hacsfiles/frigate-hass-card/startOfHour-34edc6a8.js' doesn't provide an export named: 's'. Check troubleshooting.

Hmm. Wondering if your browser is still caching files it shouldn't be, or your installation is somehow corrupted?

I've tried removing the card, clearing cache, reinstalling, and still doesnt work

And what version of the card?

And what version of the card?

Latest. 6.1.2

Camera initialization failed: The requested module 'https://mydomain/hacsfiles/frigate-hass-card/startOfHour-34edc6a8.js' doesn't provide an export named: 's'. Check troubleshooting.

Manually installed v6.1.2 seems to work fine. Can you paste me your config?
[If you haven't already, recommend trying a different device just to be sure there isn't some weird caching going on]

My mobile seems to work. I tried private window, that does not work.
  - type: custom:frigate-card
    cameras:
      - camera_entity: camera.front_door_bell
        live_provider: go2rtc
        triggers:
          motion: true
      - camera_entity: camera.garage_camera
        live_provider: go2rtc
      - camera_entity: camera.work_room_camera
        live_provider: go2rtc
      - camera_entity: camera.front_door
        live_provider: go2rtc
      - camera_entity: camera.driveway
        live_provider: go2rtc
      - camera_entity: camera.street
        live_provider: go2rtc
      - camera_entity: camera.back_patio
        live_provider: go2rtc
      - camera_entity: camera.backyard
        live_provider: go2rtc
      - camera_entity: camera.backyard_right_side
        live_provider: go2rtc
      - camera_entity: camera.side
        live_provider: go2rtc
      - camera_entity: camera.backyard_left_side
        live_provider: go2rtc
    view:
      dark_mode: auto
    live:
      display:
        mode: grid

FYI: Your config also works fine for me. Try a different browser on the same device? Definitely smells like a caching issue, since it seems to work on one device but not the other.

FYI: Your config also works fine for me. Try a different browser on the same device? Definitely smells like a caching issue, since it seems to work on one device but not the other.

Yup, worked on chrome on my computer and on phone. Not sure what the deal is with firefox.


feat: Implement Bitwise and Shift Operator Support

This PR introduces support for bitwise and shift operators. With this enhancement, users can now perform advanced bitwise operations, including AND, OR, XOR, and NOT, as well as left and right shift operations.
&    bitwise AND            integers
// To manage bitwise OR operations and various callbacks (predicates, built-ins, and custom functions)
|    bitwise OR             integers
// Duplicate the Caret operator to prevent confusion with the Exponent operator.
^^    bitwise XOR           integers
&^   bit clear (AND NOT)    integers

<<   left shift             integer << integer >= 0
>>   right shift            integer >> integer >= 0

Ref: #207

Let’s not override pipe operator. This will create confusion. Also ^^ is really odd.
What about using full names as operators for bitwise ops?
I already know a good one:

xor
A xor B

We just need to find nice ones for other. I know some languages do that.

And let’s implement them as functions. So we don’t need to worry about precedence.

Or as alternative solution add a special "macro": bit which will be converting those expressions:
bit(A | B ^ ~C)

And let’s implement them as functions. So we don’t need to worry about precedence.

Following the comments, I will also be adding the following built-in functions, along with their respective precedences.
// Bitwise ops
&, bitand     -    bitwise AND            integers
bor, bitor    -    bitwise OR             integers
xor, bitxor   -    bitwise XOR            integers
&^, bitnand   -    bit clear (AND NOT)    integers
~, bitnot     -    bit NOT                integer

// Bit Shift ops
<<, bitshl    -    left shift             integer << integer >= 0
>>, bitshr    -    right shift            integer >> integer >= 0
>>>, bitushr  -    unsigned right shift

But let's drop operators


Contrast coding schemes for multiclass labels

How would you encode high cardinality categorical variables if the target variable is another high cardinality categorical variable? Can not use OneHotEncoding because of high cardinality. Can not use LabelEncoding because there is no inherent order. Can not use Target Encoding / Contrast Encoding because they all compute 'mean' of the target variable, which in case of multiclass labels won't make any sense. How to approach this problem?

This topic was discussed in issue #182.
If the label has a low count of unique values (e.g.: less than 30), it is possible to extend TargetEncoder (as discussed in #182) and it will work well (given enough training data).
If the label has truly high cardinality (e.g.: more than 1000 unique values), consider using HashingEncoder.

Thanks for the response. The target label has more than 100 possible values. I don't think OneHotEncoder on target variable/ one vs all method/ training multiple predictors is a feasible approach. By HashingEncoder, do you mean using it on the target variable and then use extended TargetEncoder or directly using HashingEncoder on the categorical features?

Directly use HashingEncoder on the categorical features. The advantage of this encoding is that the count of the generated features does not depend on the cardinality of the features nor the cardinality of the label (as it is an unsupervised method just like OneHotEncoder).


Excalidraw is a bit broken with Oculus hand tracking

When using Excalidraw in the Oculus Browser (on Quest w/ hand tracking enabled), pinch-to-click results in a click event and the context menu pops up. Desired behavior would be to just register the click event and suppress the context menu (I think)

Need some Oculus devices to test :P


Document Ctrl-A and Select All Capabilities (If it exists)

I was unable to find any information on how to perform Select All capability with the Terminal. The current key bindings description describes Copy and Paste, but not Copy All (in the buffer).
Ctrl-A works to the extent that when I paste a line and perform Ctrl-A, it does select the entire line I have pasted.
The expected behavior is as you would get in the MacOS Terminal, that it will select everything in the buffer.

I believe https://github.com/microsoft/terminal/issues/4993 covers this. Ctrl+Shift+A for selectAll is listed in the spec https://github.com/microsoft/terminal/pull/2840.
Windows Terminal currently passes Ctrl+A to the session. If you run PowerShell there, then PSReadLine implements a Select All for Ctrl+A. But this is not available if you run Command Prompt instead.


Incorporating metrics from the threadpool

This is a feature request, not a bug report. It is not currently actionable.
node-clinic currently gives great insight into the event loop. For some applications this is sufficient. For others, understanding the state of the threadpool may be crucial to tuning the threadpool size or the application behavior for best performance.
The threadpool has historically been hidden in libuv without a monitoring API (but see libuv #1528). I am trying to change this. I have proposed a libuv PR to make the libuv threadpool pluggable (libuv #1726) and a corresponding Node.js PR proposing various Node-land threadpools (Node.js #22631).
I would love to see threadpool monitoring integrated into node-clinic's offerings. To that end, I'd like to understand what kind of API and metrics you think would be beneficial. Please chime in on Node.js #22957 if you have thoughts.

See https://github.com/libuv/libuv/pull/1815 :-)
Currently it is blocked by (a) @bnoordhuis and (b) some rebasing and updating that I need to do.


[UPS] CompanyName should not be present in Shipper element

Issue
Both Name and CompanyName are being populated in Shipper element.
Based on the schema

Name should only be present for Shipper and
CompanyName should only be populated for ShipTo and ShipFrom elements.

While the API may still work it causes the Shipper element to have a duplicate Name and CompanyName if only location.company_name is populated.
See https://github.com/Shopify/active_shipping/blob/master/lib/active_shipping/carriers/ups.rb#L593
CC @trishume @kmcphillips

I'm not sure if this is a related or a different issue, but will start here.
I am seeing that for UPS if I include both a name and a company_name in the origin for create_shipment, the company_name value is printed twice on the label I get back instead of the name and then company_name

@jbeker
I'm experiencing the same problem. Did you have update to know how to fix this?

@xinlangzi We did. The issue had to do with the way the fields from the Location object are transformed into the XML that gets sent to UPS. We ended up creating a subclass of ActiveShipping::UPS that overrode build_location_node to add a bunch more logic in the creation of the XML. We then use our custom subclass.
def build_location_node(xml, name, location, options = {})
# not implemented:  * Shipment/Shipper/Name element
#                   * Shipment/(ShipTo|ShipFrom)/CompanyName element
#                   * Shipment/(Shipper|ShipTo|ShipFrom)/AttentionName element
#                   * Shipment/(Shipper|ShipTo|ShipFrom)/TaxIdentificationNumber element

xml.public_send(name) do

  # "Shipper" is what gets put in the return address field
  # The return address is made up of the fiollowing fields:
  # <AttentionName>
  # <Name>
  # <Address1>
  # <Address2>
  # etc, etc.

  # "ShipFrom" *does not get printed*

  # "ShiptTo" is the recipient's address
  # <AttentionName>
  # <CompanyName>
  # <Address1>
  # <Address2>
  # etc, etc.

  # Assumptions: We will have either a Name or a Company name

  label_name = nil
  label_attention = nil
  label_company = nil

  if name == 'Shipper'

    if !location.name.blank? && !location.company_name.blank? # we have both a name and company name
      label_attention = location.name
      label_name = location.company_name
      label_company = location.name
    elsif location.name.blank? && !location.company_name.blank? # name is blank, company is not blank
      label_name = location.company_name
      label_company = location.company_name
    elsif !location.name.blank? && location.company_name.blank? # name is blank, company is not blank
      label_name = location.name
      label_company = location.name
    else
      raise "need a name or an address"
    end

  else
    if !location.name.blank? && !location.company_name.blank? # we have both a name and company name
      label_attention = location.name
      label_name = location.name
      label_company = location.company_name
    elsif location.name.blank? && !location.company_name.blank? # name is blank, company is not blank
      label_company = location.company_name
      label_name = location.company_name
    elsif !location.name.blank? && location.company_name.blank? # name is blank, company is not blank
      label_company = location.name
      label_name = location.name
    else
      raise "need a name or an address"
    end
  end

  xml.Name(label_name) unless label_name.blank?
  xml.AttentionName(label_attention) unless label_attention.blank?
  xml.CompanyName(label_company) unless label_company.blank?

  xml.PhoneNumber(location.phone.gsub(/[^\d]/, '')) unless location.phone.blank?
  xml.FaxNumber(location.fax.gsub(/[^\d]/, '')) unless location.fax.blank?

  if name == 'Shipper' and (origin_account = options[:origin_account] || @options[:origin_account])
    xml.ShipperNumber(origin_account)
  elsif name == 'ShipTo' and (destination_account = options[:destination_account] || @options[:destination_account])
    xml.ShipperAssignedIdentificationNumber(destination_account)
  end

  if phone = location.phone
    xml.PhoneNumber(phone)
  end

  xml.Address do
    xml.AddressLine1(location.address1) unless location.address1.blank?
    xml.AddressLine2(location.address2) unless location.address2.blank?
    xml.AddressLine3(location.address3) unless location.address3.blank?
    xml.City(location.city) unless location.city.blank?
    xml.StateProvinceCode(location.province) unless location.province.blank?
    # StateProvinceCode required for negotiated rates but not otherwise, for some reason
    xml.PostalCode(location.postal_code) unless location.postal_code.blank?
    xml.CountryCode(location.country_code(:alpha2)) unless location.country_code(:alpha2).blank?
    xml.ResidentialAddressIndicator(true) unless location.commercial? # the default should be that UPS returns residential rates for destinations that it doesn't know about
    # not implemented: Shipment/(Shipper|ShipTo|ShipFrom)/Address/ResidentialAddressIndicator element
  end
end
end

@jbeker
Thank you very much! I see the problem is on this shipper node. Your code works well.


An attempt at a pytorch fast_inla implementation.

 Overall, my conclusion is pretty negative in comparison with jax. I’m going to record my explorations in the repo because it’s still worth doing another look once we start playing with GPU stuff more.
Overall:

I can’t get a float64 CPU version of pytorch to be competitive with JAX. It’s about 3x slower (21us vs 7us). This isn’t a huge deal since neither of these numbers is slow, but it is disappointing since that’s the relevant comparison.
Using the M1 GPU/Apple metal backend is REALLY messy. The backend is quite immature and seems to only really support standard neural net operations well.

Some things are a lot faster. For example, with v1.13 (the version that enables the “mps” mac gpu backend), I get about a 10x speedup compared to v1.11 for something like a simple matrix multiply. That’s awesome and really important since matmuls are so fundamental.
But, what we need is a bit more complex and several of the important operations are not supported yet by the mps backend. determinants, matrix solves and inverses. a few others.
Several operations ran but returned incorrect output. torch.einsum and torch.linalg.inv just gave completely incorrect output. I made an issue: https://github.com/pytorch/pytorch/issues/78363

The other big issue is that running with the metal backend requires running in float32 (this would be true of cuda too, at least for good performance). It turns out that this causes major problems for our current Berry fast_inla code. A few of our operations fail in 32 bit floating point. The INLA optimization to find the mode fails to converge because the hessian inverse is unstable particularly for small sigma2. If I use a linear solver instead of computing the inverse directly, I can get float32 to work correctly. But that means it’s nontrivially harder to get the variance of the multivariate gaussian approximation. Anyway, this is probably another set of issues that we’ll need to explore a bit more in the future. Being able to do correct inference in 32 bit floating point precision would be really helpful for performance because it unlocks the whole GPU world. I don’t think I should dig into it more right now because it’s not critical path, but it’s good to know what kind of problems we might run into.

Closing this immediately since I don't want to merge this code, but just record some thoughts on the branch and be able to reference this elsewhere in issues.

Interesting, I had heard pytorch doesn't have great CPU performance, but I didn't think the gap would be that large. I would make sure it's getting JIT compiled.
I guess the metal backend problems aren't too surprising at this stage of development. Something we could look at is using Choleskies for the inverses and determinants, assuming the Cholesky op works!

Thoughts on float32 stability:

I think if the hessian is poorly conditioned, we could probably fall back to a first-order optimization method? Could plug in any of the standard NN optimizers or see here for fancier ones (I'd start with Adam). Jax also has a BFGS solver, which I haven't tried.
I also wonder if there's an analytic solution as sigma -> 0.
If you want to do something really fun, it's probably possible to regularize the newton steps. See e.g. this for a new newton algorithm that claims global convergence.

using Choleskies for the inverses and determinants

Oh man, I totally forgot about Cholesky!!  In the JAX/numpy worlds, it was substantially slower, but it's the "correct" algorithm in this case in a lot of ways. I suspect the JAX problem is just that the implementation there is suboptimal in some way. Anyway, I'll try that.

I would make sure it's getting JIT compiled.

Yeah, I tried using the torchscript jit. I'm not sure if I'm just doing something wrong, but I don't get much if any speedup. On the other hand, some of our more complex vectorized operations don't work in torchscript so I have to write the code quite differently.

In the JAX/numpy worlds, it was substantially slower, but it's the "correct" algorithm in this case in a lot of ways. I suspect the JAX problem is just that the implementation there is suboptimal in some way.

jax.lax.linalg contains a batched, (hopefully) fast cholesky op.
https://jax.readthedocs.io/en/latest/_autosummary/jax.lax.linalg.cholesky.html

Yeah, I tried that and found it to be a bit slower than the current
fast_invert method. On the other hand, it'll likely be necessary in 32-bit
float JAX.
On Sat, May 28, 2022 at 6:48 PM constinit @.***> wrote:

In the JAX/numpy worlds, it was substantially slower, but it's the
"correct" algorithm in this case in a lot of ways. I suspect the JAX
problem is just that the implementation there is suboptimal in some way.
jax.lax.linalg contains a batched, (hopefully) fast cholesky op.
https://jax.readthedocs.io/en/latest/_autosummary/jax.lax.linalg.cholesky.html
—
Reply to this email directly, view it on GitHub
https://github.com/Confirm-Solutions/kevlar/pull/97#issuecomment-1140339704,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABALTE2ZCI4FB6CGT5E5K7DVMKPFVANCNFSM5XCBPY7A
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


NTUEEPLUS-55. Move old repo here

What is this PR for?

RT
What type of PR is it?
Refactoring
Todos

[ ] - none

What is the Jira issue?

https://ntueeplus.atlassian.net/browse/NTUEEPLUS-55
Screenshots (if appropriate)

@noidname01