Possible to run .snap() for all HTML elements with same class name? - yes!

Hello!
As the title goes, I'd like to ask if is there any way to screenshot multiple HTML elements that have the same class name?

Nevermind, solved. Found out that XPath selectors allow for specifying the n-th occurrence of a searched element.
Reference: https://stackoverflow.com/questions/4007413/xpath-query-to-get-nth-instance-of-an-element

Yes! What you found is correct. With Python loop control and XPath, you can do something like the following block of code. And it will output all the result URLs on first page of google search.
import rpa as r

r.init()
r.url('https://google.com')
r.type('//*[@name="q"]','rpa[enter]')

# wait for result to appear before using present() to check
# present() does not wait until timeout like exist() does
r.hover('//*[@class="r"]')

for n in range(1, 1000):
    if r.present('(//*[@class="r"])[' + str(n) + ']'):
        print(r.read('(//*[@class="r"])[' + str(n) + ']/a/@href'))
    else:
        break

r.close()

Or something like below if you already know the number of elements in advance -
import rpa as r

r.init()
r.url('https://google.com')
r.type('//*[@name="q"]','rpa[enter]')

# for eg, if there is always 10 elements
for n in range(1, 11):
    print(r.read('(//*[@class="r"])[' + str(n) + ']/a/@href'))

r.close()


Standalone usage :: Remove runtime dependency on active support

related to #16

Looks good to me.  I'll merge this in today unless @victorluft or @benlangfeld has objections and avoid any trys or present?s in the future.

:+1:


Replace media-typer with content-type

assigned to @dougwilson ;)

I guess it can be... my first koa contribution

:scream_cat: :scream_cat: :scream_cat:

Actually, I should say first direct contribution :) Working on dependencies is basically indirectly contributing.


[UPDATE BONUS GUIDE] bump version to v22.11

What
What is the reason of this change?

signing key change (cdecker)
version bump with git reset --hard

Why
keeping updated to latest release versions
How
How was this change accomplished?
Scope

[ ] significant change to core configuration
[ ] independent bonus guide
[x] simple update

Draft
Draft until problematic issue regarding database upgrade warning. Requires setting flag database-upgrade for non-final versions although v22.11 should be identified as final.

https://github.com/ElementsProject/lightning/issues/5769

In case you need it, I found a way to make the check pass https://gitlab.com/1ma/dockertronics/blob/master/core-lightning/ubuntu/Dockerfile#L36
Though ideally cdecker should fix his key.

In case you need it, I found a way to make the check pass https://gitlab.com/1ma/dockertronics/blob/master/core-lightning/ubuntu/Dockerfile#L36
Though ideally cdecker should fix his key.

Thanks for pointing out!
I missed that line because gpg still responds with "good signature" which is basically ok but I agree, it should be fixed.

Added a note (upgrade section) about the possible need for setting database-upgrade=true in config because some users reported that it's required on upgrade from v0.12.1 or earlier, although there is no database migration happening and users can go back to earlier versions without problems. Pretty paradox but it is how it is.


Pl3xmap Addon

Does the Pl3xMap addon still work? If so, how do I install it? The official addons are listed as separate plugins and are installed that way. The one by Vane doesn't seem to be a standalone plugin. Thank you for your help <3

The integration was a contribution. I would assume it still works, but I can't vouch for it. If I recall correctly, it's not a standalone plugin but something you have to put inside the plexmap folder, and which is loaded by plexmap itself.

I can also recall that there used to be a separate "addons" folder. However, I believe that is no longer the case.
Could you please reach out to the contributor or something similar to see if they can fix the issue?
By the way, when using the latest version of Pl3xMap, I also receive the following error message from Vane Bedtime & Portals:
java.lang.NoClassDefFoundError: net/pl3x/map/event/Event
        at org.oddlama.vane.bedtime.BedtimePlexMapLayer.delayed_on_enable(BedtimePlexMapLayer.java:27) ~[vane-bedtime-1.12.1.jar:?]
        at org.bukkit.craftbukkit.v1_20_R1.scheduler.CraftTask.run(CraftTask.java:101) ~[paper-1.20.1.jar:git-Paper-45]
        at org.bukkit.craftbukkit.v1_20_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:480) ~[paper-1.20.1.jar:git-Paper-45]
        at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1112) ~[paper-1.20.1.jar:git-Paper-45]
        at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:318) ~[paper-1.20.1.jar:git-Paper-45]
        at java.lang.Thread.run(Thread.java:833) ~[?:?]
Caused by: java.lang.ClassNotFoundException: net.pl3x.map.event.Event
        at org.bukkit.plugin.java.PluginClassLoader.loadClass0(PluginClassLoader.java:183) ~[paper-api-1.20.1-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.plugin.java.PluginClassLoader.loadClass(PluginClassLoader.java:150) ~[paper-api-1.20.1-R0.1-SNAPSHOT.jar:?]
        at java.lang.ClassLoader.loadClass(ClassLoader.java:520) ~[?:?]
        ... 6 more
[23:47:29 WARN]: [vane-portals] Task #16 for vane-portals v1.12.1 generated an exception
java.lang.NoClassDefFoundError: net/pl3x/map/event/Event
        at org.oddlama.vane.portals.PortalPlexMapLayer.delayed_on_enable(PortalPlexMapLayer.java:27) ~[vane-portals-1.12.1.jar:?]
        at org.bukkit.craftbukkit.v1_20_R1.scheduler.CraftTask.run(CraftTask.java:101) ~[paper-1.20.1.jar:git-Paper-45]
        at org.bukkit.craftbukkit.v1_20_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:480) ~[paper-1.20.1.jar:git-Paper-45]
        at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1112) ~[paper-1.20.1.jar:git-Paper-45]
        at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:318) ~[paper-1.20.1.jar:git-Paper-45]
        at java.lang.Thread.run(Thread.java:833) ~[?:?]
Caused by: java.lang.ClassNotFoundException: net.pl3x.map.event.Event
        at org.bukkit.plugin.java.PluginClassLoader.loadClass0(PluginClassLoader.java:183) ~[paper-api-1.20.1-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.plugin.java.PluginClassLoader.loadClass(PluginClassLoader.java:150) ~[paper-api-1.20.1-R0.1-SNAPSHOT.jar:?]
        at java.lang.ClassLoader.loadClass(ClassLoader.java:520) ~[?:?]
        ... 6 more

@Serial-ATA Are you still using Plexmap with vane and maybe have some insight on this?

Yeah so Pl3xMap completely removed the addon system in v3 back in April. It now expects standalone plugins.
I was working on converting the addon over to the new system a few weeks ago and never got around to finishing it. I'll look into finishing it in the next couple weeks. Hopefully they won't change its system again.

I guess this could be beneficial for us in the long run, since now we can remove the plexmap addon completely and have the integration kick in if needed, like with dynmap or bluemap


Probable Pitchers

For the past few days I've been able to pull the probable pitchers for the day's games up until the first game starts since the game that started will no longer have the attribute "probable pitchers". Makes sense. However, today (4/10/21) prior to the first game starting, my code threw the 'GameScoreboard object has no attribute 'p_pitcher_away' error. Why would it be throwing this error before games have started?

Wasn't there a game that day that had a late non-named starter - in a bullpen session by chance? and the name wasn't matched. Going to check the tape on this one after finishing this message

Not sure this would cause it, but the first game of the day but was postponed, Pitchers showed up as TBD :
https://www.mlb.com/scores/2021-04-10

If it was listed as to TBD that shouldn't be an issue when pulling the probables since it will just show up as "-". Regardless, I was able to find a work around.


use new bloom filter package instead of containing the cpp directly

Please review the release process for BrowserServicesKit here.
Required:
Task/Issue URL: https://app.asana.com/0/903182762228930/1202443550575711/f
iOS PR:  n/a
macOS PR: n/a
What kind of version bump will this require?: Patch
Optional:
Tech Design URL:
CC:
Description:
References original cpp project which is now a swift package.
Steps to test this PR:

Drop into iOS and macOS and ensure HTTPS upgrades still work

OS Testing:

[ ] iOS 14
[ ] iOS 15
[ ] iOS 16
[ ] macOS 10.15
[ ] macOS 11
[ ] macOS 12

Internal references:
Software Engineering Expectations
Technical Design Template

Reverted bloom filter change after some feedback from the Android devs.  Will come back to this another time.


Design: Admin onboarding experience

User Story
As an administrator for Company Y, I want to setup/configure the cost management service so that I can start managing/monitoring my AWS estate (inventory/accounts).
Impacts

API: yes
UI: yes
Docs: yes

Role

Administrator is the person who has the very first access to add the company to the cost management service and the companies relevant AWS accounts.

Assumptions

We have instructions documented somewhere that is the process to link the AWS accounts to koku
The user has already installed the necessary koku project code onto their systems
The user has permission to the AWS accounts

UI Details

This would be a guide that starts when the user logins into Cost Management or even before it if there is some configuration setup done before login is even allowed (registration??)

Acceptance Criteria

[ ] Review with Mgmt UXD stakeholders
[ ] Review with HCCM stakeholders
[ ] Post designs to invision with a link

We will be using Cloud Metering's Designs for this feature.
Look at first 7 slides of the following prototype: https://redhat.invisionapp.com/share/9AI06PU8TEH


[UI Refresh] - Futures View

Description
This PR begins the changes to the new design on the futures screen. It uses the new components to implement most of the designs for the new futures screen. These changes include:

New trade sidebar
New market info box

What this PR does not include:

Changes to the translation files (these will be made in a separate PR).
Implementation of the new PositionCard
New navigation bar (including theme toggle).
Complete application logic and data.

It is important to note the following:

This screen currently defaults to the 'dark' theme.
Many of the constituent parts of this screen are either partly or non-functional. i.e. This PR is intended for entirely visual purposes.

Related issue
N/A
Motivation and Context
This PR is part of the Kwenta UI refresh.
How Has This Been Tested?
N/A
Screenshots (if appropriate):

@koredefashokun Is this the dark theme? It looks like the elite theme. The background on the dark theme seems more black and the buttons are more gray.

Also can you make a ticket that tracks these if they're not already tracked:

Changes to the translation files (these will be made in a separate PR).
Implementation of the new PositionCard
New navigation bar (including theme toggle).
Complete application logic and data.


feat: Add confirmation modal when deleting a user

Brief description. What is this change?
Add confirmation modal when deleting user
Implementation description. How did you make this change?

Display general modal to ask admin to confirm that they want to delete a user

Steps to test

Log in as admin
Delete a user in User Management table
Confirmation should appear
Cancelling should not have any effect
Confirming the delete should delete user as required

Checklist

[x] My PR name is descriptive and in imperative tense
[x] My commit messages follow conventional commits and are descriptive. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
[x] I have run the appropriate linter(s)
[x] I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR
[ ] The appropriate tests if necessary have been written

Having some errors with deleting a user on my local, if someone could test this while I try to fix that would be great :')


Missing functionality(?) : Sharing of snippets (and logbooks) via web interface

Testing logbooks on : https://scilog.development.psi.ch/overview
I cannot change the list of authors (owners) of my logbooks (only add or cancel options are available in edit logbook on first page).
Cannot send a link to a snippet to someone or add him as owner or reader.

closed by #311


fortan lib support for https://github.com/fommil/netlib-java

Added a new recipe to bcpc-hadoop cookbook that installs the two necessary packages.  This will allow worker nodes to have libblas and liblapack installed on them.  This is for support of https://github.com/fommil/netlib-java.
Testing:
ubuntu@bcpc-vm3:~$ ls /etc/alternatives/libbla*
/etc/alternatives/libblas.so.3gf
ubuntu@bcpc-vm3:~$ ls /etc/alternatives/liblapack*
/etc/alternatives/liblapack.so.3gf

Since this is to support Spark MLlib I would prefer to see the Spark recipe pull this in rather than adding another step to roles. (Roles are a pain to merge in existing clusters and I do not see a reason to not use this.)

Please try to follow rubocop advice as well.

I was thinking that if this is a generic library, we may not want it added under spark, as it will be available for other components as well.  It isn't actually directly related to spark at all.

Ok, changes made as per comments by @cbaenziger .  Also made some cosmetic changes due to complains from rubocop.


Refactor ctx.QueryX functions to not depend on forms

close #16562

No. This will not work. You still needs FormX for those Post requests. #16562 only the first step to do rename, next PRs we can add Query back but use ctx.URL for those GET requests.

I think we can do it in one ... and most func use Query in GET requests this few who doesnt can be migrated easyly within this pull
see discort maintainer channel :) @lunny

Codecov Report

Merging #16567 (3b3f8aa) into main (3705168) will increase coverage by 0.02%.
The diff coverage is 45.45%.

@@            Coverage Diff             @@
##             main   #16567      +/-   ##
==========================================
+ Coverage   45.37%   45.40%   +0.02%
==========================================
  Files         750      749       -1
  Lines       85084    84996      -88
==========================================
- Hits        38608    38591      -17
+ Misses      40234    40168      -66
+ Partials     6242     6237       -5

Impacted Files
Coverage Δ

modules/context/repo.go
52.99% <0.00%> (ø)

routers/api/v1/admin/adopt.go
0.00% <0.00%> (ø)

routers/api/v1/notify/user.go
11.62% <0.00%> (ø)

routers/api/v1/repo/key.go
17.77% <0.00%> (ø)

routers/api/v1/repo/release_attachment.go
0.00% <0.00%> (ø)

routers/api/v1/repo/topic.go
47.47% <0.00%> (ø)

routers/web/admin/admin.go
11.41% <0.00%> (ø)

routers/web/admin/emails.go
0.00% <0.00%> (ø)

routers/web/admin/hooks.go
0.00% <0.00%> (ø)

routers/web/admin/notice.go
0.00% <0.00%> (ø)

... and 66 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3705168...3b3f8aa. Read the comment docs.

@lunny now it's ready :)

CI is failing related

I should have tested it after rebase ... cleaned up relicts & untouch tests + fix goGet speedup


Prevent dangling GetAttribute calls

It appears possible that there could be a hang due to unread data from the
repo-attribute command pipes. This PR simply closes these during the defer.
Fix #18734
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

CI failed is related.

DATA RACE -- hmm do we need some kind of rw lock ?

:rocket:

Backport #18755


Calls to raw malloc/free

For use in embedded environments, grpc-c should instead call grpc's gpr_malloc, gpr_realloc, gpr_free.

This got slipped somehow. Fixed it with 3ded4a1


fix typo in log output

@moelsayed @galal-hussein can you please take a look at this trivial typo fix?
Thank you!

@moelsayed @galal-hussein can you please take a look at this trivial typo fix?
Thank you!


WIP: Explore cross-platform local store

This PR replaces the Windows platform only CredMan local store with a .NET based cross-platform local store using optional password.

A thought, and I may have missed this, but couldn't you still keep the encryption key in the Windows Credential store and continue a "promptless" method of access on Windows?

@iSazonov, @JustinGrote  Thanks for your input.  I forgot to include the WIP tag in the title.  This PR explores the possibility of a cross-platform, .NET based local secure store.  There are a number of significant problems with CredMan.  The main problem is its Windows platform only.  But in addition it only works for interactive log-in accounts, so it won't work with Windows built-in accounts or over PS remoting, and this greatly reduces the usefulness of SecretManagement module.
The main good thing about this solution is that it provides a uniform experience over all platforms.  It is also arguably more secure with password protection (with sudo like timeout) than CredMan SSO.
We have not decided if this is the right direction to take, and would like to get community opinions.  Also I'll work on publishing the design spec for this, which talks more about motivation.

@JustinGrote The only way to use this "promptless" is to configure the local store without password, or with "DoNotPrompt" but pass in a SecureString password in script using SecretManagement.  The no password option is obviously less secure, and relies on file system protections.  But this is arguably roughly as secure as CredMan with SSO.

We have not decided if this is the right direction to take, and would like to get community opinions.

Looks good.
For experience reference https://www.nuget.org/packages/SecureStore/

@PaulHigin I was thinking more that whatever the xplat store becomes, I assume it's going to be effectively like KeePass and you'll have a passphrase to unlock the encryption. What I'm saying is that there could be an option to save the passphrase in credential manager, and then when unlocking the vault, have it first check if there's an available passphrase in credential manager to match.
Basically, rather than having credential manager be the metavault as it is now, instead it can be a key repository to unlock the xplat metavault. If you're not on Windows, then it's passphrase as usual (and perhaps automatic integration with various keyring tools to store the passphrase in the future), but it's no longer used to store the actual metainfo. Make sense what I'm saying?
Of course I can do this outside of the module, but it would reduce user friction to have it baked in, especially for automation scenarios like scheduled tasks.

@JustinGrote  I see what you are saying, but at this point I feel it is something that should be performed outside SecretManagement, i.e., something a user does manually.  One concern is inadvertently opening up a security hole.  But it is something that can be considered later.
Note that even with the store configured without password, the secret data is still encrypted with a key, but the key is protected only by the file system.  Arguably, this is as secure as a SSO solution like CredMan.  Both are tied to current user account, and vulnerable to admin/root attacks.

@PaulHigin there is one massive difference. With the file system permissions method the key is still not encrypted-at-rest, which a lot of compliance (HIPAA ,etc.) will not accept. Keeping the key in windows credential manager, the key is at least encrypted at rest such that if someone stole the computer and dumped the hard drive, they can't just get the cleartext key to the secrets. They would need a valid windows login credential to still decrypt it. This is the whole reason the cryptoAPI was developed and so it's still valid.
Still, if you're set on not having that handled natively, it wouldn't be a big deal to make a helper module that stored the key this way and then called secretsmanagement or used parameter completion to supply the key.

Agreed, in this case you need to use a password for local store to protect at rest data.  I was thinking more in terms of protecting against admin/root attacks.  SSO is vulnerable during user account lifetime, and integrating it with the local store password weakens it in my opinion.  JIT password with timeout (e.g. sudo) seems stronger for dynamic scenarios.
Again, I see this as a possible nice to have, opt-in feature, and something we may want to add later.


Added accounts microservice

Closes #32 Closes #23
Description
An implementation of the account management and logging in microservice.
Fully documented using Swagger.
Tests included.
API endpoints:

GET /accounts/ - retrieves all existing accounts information
POST /accounts/ - creates an account
GET /accounts/{user_id} - retrieves information about an user
PUT /accounts/{user_id} - updates information about an user
DELETE /accounts/{user_id} - deletes an user from the database
GET /login/ - try to log in user (check if password hash is correct and get user information if so)
GET /login/{username} - get the password salt associated with the user (for hashing the password before logging in)

The login procedure:

User types in their username and password
Frontend asks the microservice for the password salt (GET /login/{username})
Frontend hashes the password using the salt
Frontend sends username and hashed password for authorization ( GET /login/)

Ready to merge


python3Packages.sanic: 21.3.2 -> 21.3.4; fix tests

Sanic 21.3.0 split up parts of itself into new packages that are consumed as dependencies, while those packages themselves need Sanic for testing, so we need a few doCheck = false here and there to break loops.
pytest-sanic is in the odd situation that it doesn't work with Sanic 21.3 (https://github.com/yunstanford/pytest-sanic/issues/50) yet it is needed to run Sanic's tests. Right now I'm just disabling checks for this combination of versions, but it might be better to mark pytest-sanic as broken and override that in sanic instead.
cc @costrouc

Motivation for this change
@dotlambda discovered that python3Packages.sanic was broken in https://github.com/NixOS/nixpkgs/pull/120447#issuecomment-827382960.
Things done

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

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

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

Result of nixpkgs-review pr 120881 at 5923f77a run on aarch64-linux 1

2 packages failed to build:

python38Packages.sanic-auth
python39Packages.sanic-auth

12 packages built successfully:

python38Packages.entrance
python38Packages.entrance-with-router-features
python38Packages.pytest-sanic
python38Packages.sanic
python38Packages.sanic-routing
python38Packages.sanic-testing
python39Packages.entrance
python39Packages.entrance-with-router-features
python39Packages.pytest-sanic
python39Packages.sanic
python39Packages.sanic-routing
python39Packages.sanic-testing

2 suggestions:

warning: duplicate-check-inputs
Dependencies listed in propagatedBuildInputs should not be repeated in checkInputs.
Detected duplicates:

python3.8-multidict-5.1.0
python3.8-uvloop-0.15.2
python3.8-ujson-4.0.2

Near pkgs/development/python-modules/sanic/default.nix:32:3:
   |
32 |   checkInputs = [
   |   ^

Near pkgs/development/python-modules/sanic/default.nix:28:3:
   |
28 |   propagatedBuildInputs = [
   |   ^

warning: duplicate-check-inputs
Dependencies listed in propagatedBuildInputs should not be repeated in checkInputs.
Detected duplicates:

python3.9-multidict-5.1.0
python3.9-uvloop-0.15.2
python3.9-ujson-4.0.2

Near pkgs/development/python-modules/sanic/default.nix:32:3:
   |
32 |   checkInputs = [
   |   ^

Near pkgs/development/python-modules/sanic/default.nix:28:3:
   |
28 |   propagatedBuildInputs = [
   |   ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

sanic-testing needed to be moved to propagatedBuildInputs. On the bright side, we can disable tests only when the new packages are used as dependencies of sanic.

I went ahead and marked pytest-sanic as broken for the current version of sanic, while unbreaking it solely for sanic's tests.

If want to make another PR fixing, sanic-auth, go ahead ;-)


Error: Could not compile template

Hello. I`m using  "nuxt": "^1.4.0", and after install the package I have this error in my project
ERROR
Error: Could not compile template .
<EMAIL_ADDRESS>error is not defined

builder.js:471 Promise.all.templatesFiles.map
[....]/[nuxt]/lib/builder/builder.js:471:17

@VarghaSabee Hello! This module is for Nuxt 2.4^

@mvrlin better mention that in the readme.

Thanks for the answer

@VarghaSabee sry, it's supports, it's just had a bug. In a few minutes I will release the new version 😊

I will wait 😊 Thx

@VarghaSabee just released. Try it out!

One more time thx for the fast support it`s works 😊 Solved


[ENHANCEMENT] [GSSOC' 21]  Making new good looking button for dark mode

@himanshu007-creator

@himanshu007-creator I have added the feature, it can be deployed as soon I am assigned
https://user-images.githubusercontent.com/32588231/114318574-37fa9480-9b2b-11eb-82f6-75d3cd5ee335.mp4

@himanshu007-creator I have added the feature, it can be deployed as soon I am assigned
comp.mp4

I have already created issue

I have already created issue

But I have already created the feature, well it's up to @himanshu007-creator now

I have already created issue

But I have already created the feature, well it's up to @himanshu007-creator now

Was you assigned or told by him to do that task ?

I have already created issue

But I have already created the feature, well it's up to @himanshu007-creator now

Was you assigned or told by him to do that task ?

nope but I just made a new issue

Please keep in mind don't work on issue until u are assigned this is not
way to work 👍🏻
On Mon, Apr 12, 2021, 10:07 AM CrispyWrecker @.***>
wrote:

I have already created issue
But I have already created the feature, well it's up to
@himanshu007-creator https://github.com/himanshu007-creator now
Was you assigned or told by him to do that task ?
nope but I just made a new issue
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/himanshu007-creator/SFC-foundations/issues/150#issuecomment-817471932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASIJKMYTVEOOW5FA4QAGWQDTIJ2J3ANCNFSM42X55YTQ
.

@rohank2502 thank you for teaching me what to do and what not to, I just added this feature in my local system and its so simple feature that it took just few lines of code to add so I added this to my local code(its not even pushed to origin) as I wanted to see how it would look so you can work according to your ethics and let me do whatever I want 💀

@Aditya201551 and @rohank2502  chillax guys, you both are correct in your own explanations.  Rohan had created the issue earlier . Going by the good opensource practice, it is advised to follow a sequence of steps to cater to everyone's needs. We need to be inclusive

before opening an issue , try to see if its already there or not
after creating an issue, wait for admin to assign it to you ( in case you wanna work)
if you are commenting on other issues, see the whole convo. as issues are assigned on first-come-first-serve basis.
SO i hope it is clear now what needs to be done. #149 will remain the valid issue. @aditi1403 you can colab with rohan there for the designing and you both can implement it together as well.
And don't mind if I sound rude or authoritative, I want you both to have a good time here✨

3. nd you both can implement it together as well.
And don't mind if I sound rude or authoritative, I want you both to have a good time here✨

Thanks !!
and sorry @Aditya201551

@himanshu007-creator sure, I understood


Error in parser.capture

Hi Junior
I run the attached file and gives me the following error what has gone wrong and how do I fix it.
error using parser.capture_parameterization/capture_parameterization_engine/process_normal (line 237)
Error: This statement is incomplete.
Error in parser.capture_parameterization/capture_parameterization_engine (line 153)
process_normal()
modelcodepapertwo1.zip
Best,
Sam

Hi Sam,
At line 76 and line 78 in file fwz_est.rs, you have added extra commas.
Cheers,
Junior

Lovely! thank you for the swift response.
greetings Junior
Best,
On Sat, Jan 28, 2017 at 2:34 PM, Junior Maih<EMAIL_ADDRESS>wrote:

Hi Sam,
At line 76 and line 78 in file fwz_est.rs, you have added extra commas.
Cheers,
Junior
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jmaih/RISE_toolbox/issues/36#issuecomment-275845865,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVNwKHc_ySdG8yMjMkaC4M2ca1NYIeFFks5rWzXhgaJpZM4LwfYM
.


Refactor media-ui navigation

See https://github.com/google/horologist/pull/300#discussion_r911223563
But also we should have a nicer API for params extraction and also building the params to navigate to.

https://github.com/google/horologist/pull/348


arch: riscv: fix hangup of multicore boot

This patch fixes hangup of RISC-V multicore boot.
Currently boot sequence uses a riscv_cpu_wake_flag to notify wakeup request for secondary core(s).
But initial value of riscv_cpu_wake_flag is undefined, so current mechanism is going to hangup if riscv_cpu_wake_flag and mhartid of secondary core have the same value.
This is an example situation of this problem:

hart1: check riscv_cpu_wake_flag (value is 1) and end the loop
hart1: set riscv_cpu_wake_flag to 0
hart0: set riscv_cpu_wake_flag to 1 hart0 expects it will be changed to 0 by hart1 but it has never happened

Note:

hart0's mhartid is 0, hart1's mhartid is 1
hart0 is main, hart1 is secondary in this example

Hi @katsuster ,
I'm sorry for such a late comment on your change.
I was away from Zephyr for some time but currently I'm back and inprogress with migration of our RISC-V codebase to recent Zephyr 3.6 release and stumbled across this fix.
Could you please clarify:

under "riscv_cpu_wake_flag is undefined" do you mean the race condition between the Secondary core access to riscv_cpu_wake_flag and the Primary core zeroing-out of the .bss section where riscv_cpu_wake_flag is placed?
If (1) is the case, then what do you think about minor optimization with explicit riscv_cpu_wake_flag initialization to -1 on declaration, this way we wouldn't introduce the extra sync variable and will eliminate the race condition described above?

Hello @amr-sc,
Thanks your comment. I think there is same problem even if riscv_cpu_wake_flag has initial value -1. This race condition happens before initializing BSS or copying DATA from ROM area to RAM.
If someone reset the CPUs after overwriting riscv_cpu_wake_flag area (and unfortunately the value of it is same as one of mhardid), the hart cannot wait notification via wake_flag from main CPU and going to hangup.

Hello @katsuster , thank you very much for your fast reply!
The issue you are highlighting here is very interesting. I was thinking that DATA section is always initialized before we start to execute the Zephyr code, since it is getting copied together with the TEXT to DDR. And the only race condition (on my system it is exactly the case) is possible when we initialize BSS from Primary core. So I'm a bit confused.
Could you please clarify the target scenario where we can have a DATA section race condition?
If I understand you correctly it is some kind of SW reset, where we should re-initialize the DATA section?
P.S.: to me it seems like current Zephyr RISC-V implementation is not ready for such scenario, e.g. we don't even have a code to copy DATA section for a single core..

Hello @arm-sc,
DATA section is copied and initialized by other boot loader before Zephyr running. If in this mode, using DATA section to set initial value to boot_flag is simple and nice solution.
But Zephyr also supports execution in place (XIP) mode by CONFIG_XIP = y. In this mode DATA section is stored in ROM and it is readonly. The kernel copy data from ROM to RAM in boot time.
RISC-V case:
__reset -> __initialize -> z_prep_c() -> z_data_copy()
If you have interest, please check CONFIG_XIP and z_data_copy() function at kernel/xip.c.

Hi @katsuster,
I didn't know about that functionality. Thanks a lot for flashing a light on it!


[#130660899] Add monitors for diego single-instance service locks and for etcd

#130660899 Monitor that single instance services are single instance (forever alone)
What?
We want to monitor all the single-instance services from Diego as we do with route-emitter in #662.
We will add monitors only for the services: bbs, nsync, tps and auctioneer, for the lock flag in consul.
We also add monitor the etcd flag cf.etc.IsLeader.
We finally improve the query a little bit by using max/min depending if we are checkign the upper or lower value. See commits.
How to review?
Code review
You can deploy the changes only in CI by doing:
cd paas-cf/terraform/datadog
mkdir tmp
cd tmp

ln -s ../auctioneer.tf ../bbs.tf ../datadog.tf ../etcd.tf ../nsync.tf ../tps.tf ../variables.tf .
terraform apply -var aws_account=ci  -var datadog_api_key=$(paas-pass datadog/dev/datadog_api_key) -var datadog_app_key=$(paas-pass datadog/dev/datadog_app_key) -var env=master

Remember to destroy later:
terraform destroy -var aws_account=ci  -var datadog_api_key=$(paas-pass datadog/dev/datadog_api_key) -var datadog_app_key=$(paas-pass datadog/dev/datadog_app_key) -var env=master

Who?
Anyone but @keymon

maybe you should also use min for upper and max for lower?

maybe you should also use min for upper and max for lower?

+1, I will do so

Out of interest, in order to make sure the values are the same in the query and threshold, could you store them in an intermediate variable? e.g.
variable lock_threshold {
  default = "1"
}

…
  query = "${
    format(
      "min(last_5m):sum:cf.auctioneer.LockHeld.v1_locks_auctioneer_lock{deployment:%s}.fill(last, 60) %s",
      var.env,
      element(list("> ${var.lock_threshold}", "< ${var.lock_threshold}"), count.index)
    )}"

  thresholds {
    critical = "${var.lock_threshold}"
  }
…

LG now. I'll check the tests once again (with min, max being created this time) and merge then. Hopefully we can fix travis build by that time...

There seems to be some problem with DataDog. The intention with min/max was to create robust alerts:

take max for the lower check to ensure that less than 1 locks held is not a random glitch situation, but something that is persistent
take min for the upper check to ensure that more than 1 locks held is a persistent situation

Nevertheless, this doesn't work so well, as data stops being sent when you have 0 locks held (both instances of component not active). The fill function in DD doesn't seem to work correctly:

We should raise this with DD. Until then, we can swap min/max criteria for the upper and lower checks. This will alert on every glitch, e.g. during deploy, but I guess that's better than no alerts...

@mtekel I changed it as you suggested.

This works now. I will re-base on master to get travis fix and then merge.

Thx!! 👍


Can't overwrite language of Markdown (.md) file type

I set the linguist-language of *.md files to JavaScript in my .gitattributes file but my project is not showing up as JavaScript. Is there anything I am doing wrong or is this a bug specifically with .md files.
https://github.com/ryanmcdermott/clean-code-javascript

By default, READMEs are considered documentation and not included in statistics. You'll need to add the following to your .gitattributes to change that behavior:
*.md linguist-documentation=false

Thank you that worked!


[FLINK-36460] Refactoring the CI matrix

What is the purpose of the change
As observed in FLINK-36332 the CI workflow has become difficult to mange. This PR is attempting to split tests along sensible lines to make the matrix more manageable.
To do so this PR splits the edge to edge workflow in two. A common workflow definition: e2e.yaml  which is then reused from multiple different test matrices.
I think this split allows better control over the variables without having to vary each dimension every time. By making the simplifying assumption that variations in the Java version are adequately tested by the maven builds and the smoke tests it effectively reduces the number of test jobs from ~225 to  about 100 (TBC)
Brief change log
Simplify CI workflows.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changes to the CustomResourceDescriptors: no
Core observer or reconciler logic that is regularly executed: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@gyfora alternative CI matrix as requested. Even if the matrix proposed in this PR doesn't feel "right" it sets things up so that the matrix is considerably easier to change.

This shows it in action on my fork:

I think I've finally got the excludes to align with main. sigh That was a lot more fuss than I wanted and doesn't end up as clean as I was hoping. However I think this represents a much cleaner way to setup test matrices.
The tests are still very slow to run as each test has to build its own copies of the containers. I think thats resolvable without publishing temporary artefacts to a container registry. Some combination of docker advice and GitHub sharing data docs along with this discussion.
That feels like a separate PR.
@gyfora could you trigger another run of the PR please (its gone green on my fork)


chore: update<EMAIL_ADDRESS>

This eliminates an unnecessary use of minimist and may finally silence the GitHub warnings. (Or not.)


[Kinda discussion] run yarn format

Summary
While making another PR I run yarn format and about a billion files changed, so obviously it hasn't been run for a while.
This PR will serve two purposes.

To apply the format in the existing codebase.
To discuss how to maybe automate this for the future, and possible changes to the .clang-format.

So as a starting point, I have run yarn format with the current settings, and here is the result. After running it, I do disagree with the MaxEmptyLinesToKeep: 1, as two lines are great to separate things like an @interface and it's #includes above etc.
I'll make a comment underneath to vote there, and any change decided, I'll cancel and rerun yarn format and push again in this PR.
Secondly, there was the idea of adding git hooks to run yarn format. Another idea could be to have it running on the CI, or maybe run it every month or something else. Feel free to discuss and offer your opinion, ideally with some comment about previous experience with any solution suggestions. :)
Changelog

[Internal] [Changed] - Code formatting with clang
Test plan
We test it?

MaxEmptyLinesToKeep
👎: 1
👍: 2
😕: remove it

I guess the conflicts are not super important right now, while we still decide and discuss, as I'll rebase and rerun with each decision.

cc @hramos

We need to revert the changes to Yoga and make sure that the formatting command excludes making changes to Yoga.

ok, so exclude ReactCommon/yoga, right?

Please exclude Yoga like Christoph mentioned, as that formatting should be fixed upstream. Once that is done, we can import this and see what our internal linter thinks about this. I am not too concerned, as the .eslintrc used internally is a superset of the one in the root of this repo.

I didn't find a way to tell clang-format to exclude a directory.. :(

I'll close this for now but will put it in the backlog to reformat all the code internally and sync it out.


Add metadata/core file/field validators

See https://github.com/AllenNeuralDynamics/aind-data-schema/issues/934
Corrupt presumably means the JSON doesn't parse?

[ ] Metadata validator
[ ] Metadata validator tests
[ ] Core validator
[ ] Core validator tests
[ ] Field validator
[ ] Field validator tests


Download Issue

can anyone please tell me how did you download this repo? using pip in environment or did you use docker ?

git clone


Safe logging

This PR changes how commands are logged and its completely opt-in, defaulting to the normal behaviour promiscuous logging.  It also adds the actual exception message to the log for easier debugging.  I'm completely open to suggestions.

Thank you, @jeffdyke, for this proposal.
I have the feeling, however, that this is not yet the best approach to solve the problem of "having a way to prevent sensitive data from being logged".
How about giving commands the ability to control their own logging?
The default could be a simple .toString (what we currently have) and sensitive commands could construct their own logging output in any way they seem fit, which would usually be a "redacted" .toString or sth like that?

@sirthias thanks for your input.  I'll give this some more time on your comments and clean it up.  Looking at your comments i did make it a bit more verbose than i needed, so your view is appreciated.

While the comments are meaningfull, they are worthless...you'd likely rather re-review the full diff, i made it much simpler, looking forward to your thoughts, no breaking changes still.  Thanks @sirthias!


Running spinningup without using Linux Subsystem on Windows (Success)

This is a step-by-step guide for running spinningup on Windows:
Download & Installation:
Step 1: Download  and Install  Anaconda  or  Miniconda  for Windows.
Step 2:

Download Microsoft c++ build tools from  here
While Installing, select  Desktop development with C++ :

Step 2:

Open Anaconda prompt
Create "spinningup" environment by executing the command conda create -n spinningup python=3.6.
Now activate "spinningup" environment by executing the command conda activate spinningup

Step 4:

Ensure that you have the "git" installed. If not, you can download it from here
Clone the "spinningup" repository using the command: git clone https://github.com/openai/spinningup.git
Once the cloning process is complete, navigate to the directory "Spinningup" and locate the "setup.py" file. Proceed to modify the Torch version to 1.7.1 within that file as shown below:

Step 5:

Install Swig by running command pip install swig
Install openCV-Python by running command pip install opencv-python==<IP_ADDRESS>
Install mpi4py by running command Conda install -c conda-forge mpi4py

Step 6:

Navigate to the "Spinningup" directory using the command: cd spinningup.
Now run command pip install -e .

Check Your Install:

Follow the spinningup tutorial to check your installation from here

you may come across error while plotting the results using the given command

Error:
Plotting from...
==================================================

==================================================
Traceback (most recent call last):
  File "C:\Users\project\spinningup\spinup\utils\plot.py", line 233, in <module>
    main()
  File "C:\Users\project\spinningup\spinup\utils\plot.py", line 230, in main
    estimator=args.est)
  File "C:\Users\project\spinningup\spinup\utils\plot.py", line 162, in make_plots
    plot_data(data, xaxis=xaxis, value=value, condition=condition, smooth=smooth, estimator=estimator)
  File "C:\Users\project\spinningup\spinup\utils\plot.py", line 31, in plot_data
    data = pd.concat(data, ignore_index=True)
  File "C:\Users\project\AppData\Local\anaconda3\envs\spinningup\lib\site-packages\pandas\core\reshape\concat.py", line 284, in concat
    sort=sort,
  File "C:\Users\project\AppData\Local\anaconda3\envs\spinningup\lib\site-packages\pandas\core\reshape\concat.py", line 331, in _init_
    raise ValueError("No objects to concatenate")
ValueError: No objects to concatenate
Traceback (most recent call last):
  File "C:\Users\project\AppData\Local\anaconda3\envs\spinningup\lib\runpy.py", line 193, in _run_module_as_main
    "_main_", mod_spec)
  File "C:\Users\project\AppData\Local\anaconda3\envs\spinningup\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\project\spinningup\spinup\run.py", line 243, in <module>
    subprocess.check_call(args, env=os.environ)
  File "C:\Users\project\AppData\Local\anaconda3\envs\spinningup\lib\subprocess.py", line 311, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['C:\\Users\\project\\AppData\\Local\\anaconda3\\envs\\spinningup\\python.exe', 'C:\\Users\\project\\spinningup\\spinup\\utils\\plot.py', 'data/installtest/installtest_s0']' returned non-zero exit status 1.

Solution:
This can be due to the path error your system would be facing. You can update the command by editing the path of installest_s0. you can just go to the folder and can copy the path:
python -m spinup.run plot C:\Users\project\spinningup\data\installtest\installtest_s0

Note: The installation has been done and run over Windows 11.

Worked perfectly thanks


Brian/update migration script

Pull Request Test Coverage Report for Build 4043

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 4038:
0.0%

Covered Lines:
888

Relevant Lines:
888

💛  - Coveralls


Complete Emission Research

Choose an API to collect the carbon emission data

Option 1:
Climatiq
https://www.climatiq.io/

Option 2:
Carbon Interface

Vehicle CO2 Estimates (easily calculate co2 emissions estimates from driving)

https://www.carboninterface.com/vehicles


Implement JWT Authorization Grants

Implement JWT Authorization Grant as defined in RFC 7523.

For those inquiring about the future of this feature, @jgrandja has confirmed its scheduled for release 1.1.0

@fkhantsi Correction. This feature is not scheduled as of yet. The link states:

We are also planning on documenting "Adding an extension authorization grant" in the reference manual via https://github.com/spring-projects/spring-authorization-server/issues/686. I scheduled it for the 1.1.0 release.

To clarify, gh-686 is scheduled for 1.1.0.

is there any sample for using JWT Bearer? I noticed 1.1.0 had been released.


Sometimes spheres bug out

Whenever there is alot of spheres in a tight space (literly inside each other) some spheres seem to bug out. They apear to have invalid coordinates (infinite, null or other invalid states)

found the reason:
When two spheres are at the same position, the solver tries to calculate in which direction they bounce away. Since they are at the same position there is no direction.


merge Wocsor's Camry-FF into my Devel

merge Wocsor's Camry-FF into my Devel
includes Gernby's FF- resonant

git merge wocsor-camry-ff


Profile shubhamghb

resolves #issueNumber
Checklist for profile addition

[x] Name the PR fix: add <your-username>.
[x] You've added file named your-username.json in the profiles directory.
[x] You've linked the PR to the correct issue.

Other additions/changes

[x] Changes made in this PR are mentioned below
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.
[x] I've read the contributing guidelines

This PR adds/fixes

Notes

This is Github Readme Profile of Shubham Maheshwari (username: shubhamghb)


ols4 is missing in several links

At the moment at least the links of the actions, the breadcrumbs and the (class) relations are not correct. In all of them "ols4" is missing in the path of the hyperlink.
Actions

source: https://www.ebi.ac.uk/ols4/ontologies
Breadcrumb

class relations

source: https://www.ebi.ac.uk/ols4/ontologies/efo/classes/http%3A%2F%2Fwww.ebi.ac.uk%2Fefo%2FEFO_0000400

Thanks, fixed!


Analysers from referenced projects does not affect referencee project

Issue Description
Project A references Project B.
Project B has one (or more) analyser nuget package(s).
I expected the analyser to look for files in both projects. but I also have to add the nuget package to project A.
Steps to Reproduce
Add an analyser to a project (Let's call it B)
On another project, reference project B
Expected Behavior
Analyse both projects
Actual Behavior
Only Project B files are analysed.
 If so, then this is likely a bug or missing feature in Omnisharp.

@cartermp yes, I am using the 3.1.2 .NET SDK

@cartermp should I post this issue somwhere else?

Thanks @Lobosque - no need to repost. I've tagged the issue differently. In the relatively near future we'll likely have more investment in the VSCode experience, but right now we're just triaging things to get a better sense for where the most issues lie.


[DNM] hack to allow undefined stero when mapping

Wanted to have this bounce off our CI to make sure this doesn't cause problems elsewhere.

See https://github.com/choderalab/perses/issues/1161 for more details

@ijpulidos has a PR coming in that will do this in a cleaner way.


Feature/nginx proxy uberjar

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: oshiryaeva:x: rapekasYou have signed the CLA already but the status is still pending? Let us recheck it.


event_service.py request_event not working for only one event

Hi,
I was setting up a VTN with the 1-minute-Server example from the docs and ran into a problem. As I am running my client in typescript I got a TypeError saying, that my oadrEvent that I got back from polling was not iterable.
The response message to the poll request from the vtn looks like this:
Responding to oadrPoll with a oadrDistributeEvent message: {'events': [], 'vtn_id': 'myvtn', 'ven_id': 'ven_id_123', 'response': {'request_id': None, 'response_code': 200, 'response_description': 'OK'}, 'request_id': '8a23b069-bf08-4769-b9fd-e9b3cfbb94d9'}.
The array which would normally hold the events is empty IF we only have one event in queue.
I cannot really explain why python does this, but when I debugged the request_event function I saw, that
        self.events[ven_id].pop(self.events[ven_id].index(event)) (line 55 in event_service.py)

pops the event not only from self.events but also from the referenced variable events, that is later used to fill the response array. And since the function later only checks if events is None, this results in giving back an empty array as a response.
I fixed this by adjusting line 46 where the reference is created and used the copy function to create a copy of self.events rather than a reference to it.
          events = utils.order_events(self.events[ven_id].copy())

Is this a real bug or am I missing something? I'm not too familiar with python. I'd love to hear back from you with some advice.

Hi Jiklas,
Thank you for debugging and fixing this, it does look like a bug.  Were you
able to submit an issue or a  pull request?
Let me know how I can support you.
Thanks,
Arila
On Fri, Nov 22, 2024 at 11:03 AM jiklas-nust @.***>
wrote:

Hi,
I was setting up a VTN with the 1-minute-Server example from the docs and
ran into a problem. As I am running my client in typescript I got a
TypeError saying, that my oadrEvent that I got back from polling was not
iterable.
The response message to the poll request from the vtn looks like this:
Responding to oadrPoll with a oadrDistributeEvent message: {'events': [],
'vtn_id': 'myvtn', 'ven_id': 'ven_id_123', 'response': {'request_id': None,
'response_code': 200, 'response_description': 'OK'}, 'request_id':
'8a23b069-bf08-4769-b9fd-e9b3cfbb94d9'}.
The array which would normally hold the events is empty IF we only have
one event in queue.
I cannot really explain why python does this, but when I debugged the
request_event function I saw, that
    self.events[ven_id].pop(self.events[ven_id].index(event)) (line 55 in event_service.py)

pops the event not only from self.events but also from the referenced
variable events, that is later used to fill the response array. And since
the function later only checks if events is None, this results in giving
back an empty array as a response.
I fixed this by adjusting line 46 where the reference is created and used
the copy function to create a copy of self.events rather than a reference
to it.
      events = utils.order_events(self.events[ven_id].copy())

Is this a real bug or am I missing something? I'm not too familiar with
python. I'd love to hear back from you with some advice.
—
Reply to this email directly, view it on GitHub
https://github.com/OpenLEADR/openleadr-python/issues/175, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIR5OPF3KM47FVDTPET6VPT2B3XNRAVCNFSM6AAAAABSI345QSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGY4DENBYGQ2DANY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Arila,
thanks for replying. Could you get me through the steps of submitting a pull request? Do I need to fork the repo in order to?
I tried creating a new branch but it won't let me push this branch to the remote.
It's my first contribution to a public repo so I'm a little bit lost here.
Thanks, Jiklas


WIP: Rework canonicalization

Work in progress. More to do.
The idea here is to remove the call to canonicalize(), and to avoid using absolute paths where possible.
This is a step along the path to supporting symlinks.
Updates #12.

Let's revisit this later.


Remove duplicate documentation for TRY002

Looks like this was accidentally added in 9cd1bf9c034bed50747ea0ff26d13f25f6d7a13f.

D'oh, thanks.


message_flags: Short-circuit if no messages changed.

Omit sending an event, and updating the database, if there are no
matching messages.
Screenshots and screen captures:
Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Nice, merged, thanks @alexmv!


Stub & record forward proxy https traffic

This allows WireMock to act as a forward (browser) proxy for HTTPS as well
as HTTP origins whilst stubbing & recording.
Step towards implementing #401.

Opened https://github.com/tomakehurst/wiremock/pull/1317 instead.


Strange formating with LC_NUMERIC and PDO driver

Hi,
I got a really strange behaviour when defining locale with LC_NUMERIC (or LC_ALL...), with fr_FR locale: pdo extract float/double as string (if I'm right, it does it for all databases), but they are formatted incorrectly (I don't know if PDO has to format number)
Bellow a test script and results obtained on this system:
PHP 7.0.15-0ubuntu<IP_ADDRESS> (cli) ( NTS )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.0.0, Copyright (c) 1998-2017 Zend Technologies
    with Zend OPcache v7.0.15-0ubuntu<IP_ADDRESS>, Copyright (c) 1999-2017, by Zend Technologies

$ pecl info sqlsrv
About pecl.php.net/sqlsrv-4.0.8
===============================
Release Type          PECL-style PHP extension (source code)
Name                  sqlsrv
Channel               pecl.php.net
Summary               Microsoft Drivers for PHP for SQL Server
                      (SQLSRV)
Description           The Microsoft Drivers for PHP for SQL Server are
                      PHP extensions that allow for the reading and
                      writing of SQL Server data from within PHP
                      scripts. The SQLSRV extension provides a
                      procedural interface while the PDO_SQLSRV
                      extension implements PDO for accessing data in
                      all editions of SQL Server 2005 and later
                      (including Azure SQL DB). These drivers rely on
                      the Microsoft ODBC Driver for SQL Server to
                      handle the low-level communication with SQL
                      Server.
                      *This package contains only the SQLSRV driver.*
Maintainers           Meet Bhagdev<EMAIL_ADDRESS>(lead)
                      Jay Kint<EMAIL_ADDRESS>(lead,
                      inactive)
                      Marie Barwin<EMAIL_ADDRESS>(lead)
                      Hadis Kakanejadi Fard<EMAIL_ADDRESS>                      (lead)
Release Date          2016-12-19 13:12:21
Release Version       4.0.8 (stable)
API Version           4.0.8 (stable)
License               The MIT License (MIT)
                      (https://opensource.org/licenses/mit)
Release Notes         [Added]
                      - Unicode Column name support.
                      [Fixed]
                      - Fixed issue with buffered cursor in PDO_SQLSRV
                      driver when CharacterSet is UTF-8.
                      - Fixed issue with empty output parameters on
                      stored procedure.
                      - Fixed memory leaks in buffered queries.
Required Dependencies PHP version 7.0.0
                      PEAR installer version 1.4.0b1 or newer
                       Operating System
                      OS/Arch matching pattern '//'
package.xml version   2.0
Last Modified         2017-03-16 13:58
Previous Installed    - None -
Version

$ pecl info pdo_sqlsrv

About pecl.php.net/pdo_sqlsrv-4.0.8
===================================
Release Type          PECL-style PHP extension (source code)
Name                  pdo_sqlsrv
Channel               pecl.php.net
Summary               Microsoft Drivers for PHP for SQL Server
                      (PDO_SQLSRV)
Description           The Microsoft Drivers for PHP for SQL Server are
                      PHP extensions that allow for the reading and
                      writing of SQL Server data from within PHP
                      scripts. The SQLSRV extension provides a
                      procedural interface while the PDO_SQLSRV
                      extension implements PDO for accessing data in
                      all editions of SQL Server 2005 and later
                      (including Azure SQL DB). These drivers rely on
                      the Microsoft ODBC Driver for SQL Server to
                      handle the low-level communication with SQL
                      Server.
                      *This package contains only the PDO_SQLSRV
                      driver.*
Maintainers           Meet Bhagdev<EMAIL_ADDRESS>(lead)
                      Jay Kint<EMAIL_ADDRESS>(lead,
                      inactive)
                      Marie Barwin<EMAIL_ADDRESS>(lead)
                      Hadis Kakanejadi Fard<EMAIL_ADDRESS>                      (lead)
Release Date          2016-12-19 13:12:19
Release Version       4.0.8 (stable)
API Version           4.0.8 (stable)
License               The MIT License (MIT)
                      (https://opensource.org/licenses/mit)
Release Notes         [Added]
                      - SQLSRV_ATTR_FETCHES_NUMERIC_TYPE attribute
                      support.
                      - Unicode Column name support.
                      [Fixed]
                      - Fixed precision issues when double data type
                      returned as strings using buffered queries.
                      - Fixed issue with buffered cursor in PDO_SQLSRV
                      driver when CharacterSet is UTF-8.
                      - Fixed segmentation fault in error cases when
                      error message is returned with emulate prepare
                      attribute is set to true.
                      - Fixed issue with empty output parameters on
                      stored procedure.
                      - Fixed memory leaks in buffered queries.
Required Dependencies PHP version 7.0.0
                      PEAR installer version 1.4.0b1 or newer
                       Operating System
                      OS/Arch matching pattern '//'
package.xml version   2.0
Last Modified         2017-03-16 13:58
Previous Installed    - None -
Version

<?php
setlocale(LC_ALL, 'fr_FR.UTF-8');
//setlocale(LC_ALL, 'en_US.UTF-8');
//setlocale(LC_ALL, 'de_DE.UTF-8');

define("DB_HOST", "<IP_ADDRESS>");
define("DB_PORT", "12345");
define("DB_NAME", "dbName");
define("DB_USER", "dbUser");
define("DB_PASS", "dbPass");

//CREATE TABLE foo_table
//(
//    ID BIGINT PRIMARY KEY NOT NULL IDENTITY,
//    FloatingField FLOAT NOT NULL
//);
//INSERT INTO foo_table (FloatingField) VALUES(3.14);

$sql = "SELECT TOP(1) * FROM foo_table";
$fieldName = 'FloatingField';

$dsn = 'sqlsrv:Server=' . DB_HOST . ',' . DB_PORT . ';Database=' . DB_NAME;

$options = array();

$dbh = new PDO($dsn, DB_USER, DB_PASS, $options);

echo 'PDO:sqlsrv' . "\n";

if ($StatementObj = $dbh->query($sql)) {
    while ($row = $StatementObj->fetch(PDO::FETCH_ASSOC)) {
        var_dump($row[$fieldName]);
        var_dump(is_float($row[$fieldName]));
        var_dump(floatval($row[$fieldName]));
        var_dump(var_export($row[$fieldName], true));
    }
}

$dsn = 'dblib:host=' . DB_HOST . ':' . DB_PORT . ';dbname=' . DB_NAME;

$options = array();

$dbh = new PDO($dsn, DB_USER, DB_PASS, $options);

echo 'PDO:dblib' . "\n";

if ($StatementObj = $dbh->query($sql)) {
    while ($row = $StatementObj->fetch(PDO::FETCH_ASSOC)) {
        var_dump($row[$fieldName]);
        var_dump(is_float($row[$fieldName]));
        var_dump(floatval($row[$fieldName]));
        var_dump(var_export($row[$fieldName], true));
    }
}

$serverName = DB_HOST . ',' . DB_PORT;
$connectionInfo = array("Database" => DB_NAME, "UID" => DB_USER, "PWD" => DB_PASS);
$conn = sqlsrv_connect($serverName, $connectionInfo);

$params = array();

$stmt = sqlsrv_query($conn, $sql, $params);

echo 'DIRECT:sqlsrv' . "\n";

while ($row = sqlsrv_fetch_array($stmt, SQLSRV_FETCH_ASSOC)) {
    var_dump($row[$fieldName]);
    var_dump(is_float($row[$fieldName]));
    var_dump(floatval($row[$fieldName]));
    var_dump(var_export($row[$fieldName], true));
}

PDO:sqlsrv
string(5) "3.,14"
bool(false)
float(1)
string(7) "'3.,14'"
PDO:dblib
float(3,14)
bool(true)
float(3,14)
string(4) "3.14"
DIRECT:sqlsrv
float(3,14)
bool(true)
float(3,14)
string(4) "3.14"

If you need more informations / test ...
Thanks

Thanks @Gounlaf we are able to reproduce the issue. We will get back to you on this.

One thing to note is our latest source, the output is:
PDO:sqlsrv
string(5) "3.,14"
bool(false)
float(3)
string(7) "'3.,14'"

So the floatval is actually 3.
If you turn PDO::SQLSRV_ATTR_FETCHES_NUMERIC_TYPE on in the connection option. the output is:
PDO:sqlsrv
float(3,14)
bool(true)
float(3,14)
string(18) "3,1400000000000001"

Hi @v-kaywon,
I can confirm that PDO::SQLSRV_ATTR_FETCHES_NUMERIC_TYPE fit my current requirements.
About that:

One thing to note is for our latest source, the output is:
PDO:sqlsrv
string(5) "3.,14"
bool(false)
float(3)
string(7) "'3.,14'"

So the floatval is actually 3.

Got the same result. I made a mistake in my copy/paste when creating the issue.

Hello @Gounlaf, I'm glad PDO::SQLSRV_ATTR_FETCHES_NUMERIC_TYPE fixes your problem now.
Our PHP driver is dependent on the Microsoft ODBC Driver for SQL Server. For now, the ODBC driver doesn't support system local yet. When the ODBC API SQLGetData is called, "3.,14" is returned instead of the "3,14".

Hi @v-kaywon,

Our PHP driver is dependent on the Microsoft ODBC Driver for SQL Server. For now, the ODBC driver doesn't support system local yet. When the ODBC API SQLGetData is called, "3.,14" is returned instead of the "3,14".

If locale is no supported, why the result is "3.,14" ? It should be "3.14". No ?

Hi @Gounlaf
Thanks for your patience. We have confirmed it's a defect in the ODBC driver, for your interest, it's filed here

hi @Gounlaf
Just so you know, msodbc 13.1.9 is out, and I've briefly tested this and believe this issue is now resolved. Could you please confirm? Thanks.

Closing this issue, @Gounlaf feel free to reopen the issue if the issue persists


Question regarding the Wall

So far I have seen does the so called Wall in the dashboard and start page displays at the moment comments made by other users, but this cannot be the only purpose.
In the app\system\modules\user\assets\ajax\WallAjax.php I have seen beside the showCommentProfile case also a show case, which displays content of the MEMBERS_WALL table.
There is also a WallForm and an EditWallForm in the user module, but I cannot find a reference to these forms. Does anybody have an idea what this Wall is good for and how to use it?

I think it's used for the background that the user can set

Maybe, but where the user can define the content?
I played a lot with the software, but I did not find an option to define the wall content.

I didn't find it either

Ajax is all over the software. Like In
 _protected/app/system/core/assets/ajax/

You will find like, rating ,smile, stat and validate core Ajax assets/libraries. Its used in the user module wall, setAjaxActivity and Api Ajax, then its used in messenger, messeges, comments, friends, note, newsletter, login, registration....
The list goes on. Guess you just can't escape the future... Skynet LOL


The result of smart_open.open (FileLikeProxy) lacks a next-method in 7.0.0 whereas in 6.4.0 (_io.TextIOWrapper) it did

Problem description
In smart_open 6.4.0 smart_open.open returns an iterable (_io.TextIOWrapper) which has a next-method for s3files; in 7.* this does not seem to be the case anymore.
e.g. smart_open.open(uri="s3://test/output.csv") returns a FileLikeProxy which is an iterable without next.

What is the expected result?
That when next() is called on the object returned by smart_open.open, an error is not thrown but the next item in the iterator is returned.

What are you seeing instead?
TypeError: 'FileLikeProxy' object is not an iterator

Steps/code to reproduce the problem
import moto
import boto3
import smart_open
print(f"{boto3.version=}")  # boto3.version='1.33.13'
print(f"{smart_open.version=}")  # smart_open.version='7.0.1'
with moto.mock_aws():
s3 = boto3.client("s3")
s3.create_bucket(Bucket="test")
s3.put_object(Bucket="test", Key="output.csv",  Body="abc\ndef\nghi")
f = smart_open.open(uri="s3://test/output.csv")
print(next(f))  # "'abc\n'" in smart_open.version = '6.4.0'
Versions
macOS-10.16-x86_64-i386-64bit
Python 3.11.7 (main, Dec 15 2023, 12:09:04) [Clang 14.0.6 ]
smart_open 7.0.1
Checklist
Before you create the issue, please make sure you have:

[X] Described the problem clearly
[X] Provided a minimal reproducible example, including any required data
[X] Provided the version numbers of the relevant software

Sounds like it may be relevant to https://github.com/piskvorky/smart_open/pull/786
@ddelange Can you please have a look?

tnx for the ping, will have a look tomorrow

PR is up ^ no idea why wrapt is not proxying __next__ by default

Thanks, excellent response time. Great job!

I'll do a bugfix release today


external_class is clobbered by name in [model] section of config.toml

The default config file defines a [model] { name: <blah> }, so when a user provides their config file in which an external_class is defined, the default name will override the external_class.
For instance, with a default config like this:
[model]
name = "ExampleAutoencoder"
# external_class = "blah"

and a user specific config like this:
[model]
# name = "ExampleAutoencoder"
external_class = "package.module.submodule.class"

The name from the default will be used as the model class used by Fibad.

This was address here: https://github.com/lincc-frameworks/fibad/pull/71


No matching manifest for linux/arm in the manifest list entries

I'm setting a node on a PC with ARM processor and ran into an issue when peforming make up command:
ERROR: no matching manifest for linux/arm in the manifest list entries. I understand that this is likely because, I'm using Armbian Linux distro which has not been added yet. Would it be possible to add settings for Armbian (even if it just copies the settings for Ubuntu 16.04)?

This is an unsupported configuration at this time.


Add tracking of function calls

Adds tracking function calls as part of tracing.
Closes #85

Codecov Report

Merging #92 (c5ecac6) into main (87c4def) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
+ Coverage   95.06%   95.10%   +0.04%
==========================================
  Files          26       26
  Lines        1499     1512      +13
==========================================
+ Hits         1425     1438      +13
  Misses         74       74

Impacted Files
Coverage Δ

pybryt/execution/__init__.py
100.00% <100.00%> (ø)

pybryt/execution/tracing.py
100.00% <100.00%> (ø)

pybryt/student.py
96.75% <100.00%> (+0.12%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 87c4def...c5ecac6. Read the comment docs.


I added auto focus to the scanner control

I added auto focus to the scanner control and i use it every time the scan for barcode failed
This helps in reading the barcode

Thanks :)


Fixed #28232 -- Made raster metadata readable and writable.

Metadata is now accessible and editable on GDALRaster and GDALBand objects.

buildbot, test on xenial.


Remove unnecessary System.ValueTuple package dependency from netstandard2.0 target

@migueldeicaza I'd like to remove System.ValueTuple package dependency for netstandard2.0 and net47 to make it cleaner, plz let me know your concerns, thanks!

@migueldeicaza  Kindly ping on this

I do not undersatnd the need for this change, please explain

@migueldeicaza  Generally it's good practise to only add dependencies when necessary, in this case System.ValueTupe package is only required for net46 target, so I removed it from netstandard and net47, does that make sense

Ok that part makes sense (the third file), but I do not understand the project format change in the first file, that looks unrelated.

@migueldeicaza  That is to fix the issue that those example projects do not run out of box,

Can you describe where it does not run?    They run for me, and they run on CI, and would like to get to the bottom of this.   Is this on Windows, or with .NET Core on Unix?

@migueldeicaza The project does not have any build script to copy native libtensorflow, how can it run out of box?

@migueldeicaza  Also none of unittests run out of box,  you are using a hacky copy in your CI script below but people who run it locally will need to copy them manually, which is super inconvenient.

cp -R $TRAVIS_BUILD_DIR/native/*.dylib $TRAVIS_BUILD_DIR/tests/TensorFlowSharp.Tests/bin/Release/

cp -R $TRAVIS_BUILD_DIR/native/*.dylib $TRAVIS_BUILD_DIR/tests/TensorFlowSharp.Tests.CSharp/bin/Release/

On a related note... Have you considered implementing a build automation system @migueldeicaza? I have had good experiences with Cake in the past (www.cakebuild.net, cross-platform, C# based, .NET Foundations). Would provide consistent repateble builds/unit-testing/packaging across local and CI. Maybe a bit heavyweight for now, but is simple enough to do if you wanted.

I defintiely agree with @hanabi1224 on this. The new csproj format is definitely more modern approach. And adding the dependency instructions in the csproj as post-build instruction would make the dotnet cli and only build tool required to make something that works out of the box.
@migueldeicaza

It is correct that currently this expects the payload to be installed, the .travis.yml file shows how to do this.
If someone wants to address how to install that, it can be done elsewhere, the changes in this pull request are not solving the root cause.


Fix some typo and bug

Despite for some typos, it seems tensor does not support the operation like float * tensor. But tensor * float works.

could you pls create a jira ticket or update the title to make it more clear, e.g. 'bug from using float * tensor in loss.py'?
thanks.

Pls also update the commit title using the title of https://issues.apache.org/jira/browse/SINGA-245 to make it similar as previous commits.


Add package-manageroption

Summary
This PR introduces a new parameter called package-manager, which can be either npm (default) or yarn. This value will be taken into consideration when installing the project dependencies.
What this PR does

adds package-manager option

Closes #65.

@ArtiomTr no pressure, but could you release v2 after this gets merged? :grin:

@ArtiomTr no pressure, but could you release v2 after this gets merged? 😁

Yes, of course, but I would like to release the v2.0-rc.1 version, since all the necessary functionality for version 2.0 is ready, but I want to tweak the documentation website a little more. Will be it okay for you? Or you want to see version 2.0 now?

@ArtiomTr I believe I can use the v2.0-rc.1 for the time being. :)

@ArtiomTr I believe I can use the v2.0-rc.1 for the time being. :)

Cool! The v2.0-rc.1 version is available now!

@all-contributors please add @taschetto for code, doc


[Doc]: README file --> Enhancement in  Feature Descriptions

Is there an existing issue for this?

[X] I have searched the existing issues

Issue Description
We can enhance Feature Descriptions from the README file.
Suggested Change
Example
Current
Price Comparisons: Find the best deals by comparing prices across multiple stores.
Improvement
Use our smart price comparison tool to automatically find the lowest prices on your favorite products from various stores. You can set up alerts for price drops, view historical price trends, and compare product ratings across stores for a smarter shopping experience.
Rationale
By making these changes, we can help the README become more user-centric and informative, clearly showing how each feature benefits the users and providing a better understanding of how to use the platform.
Urgency
High
Acknowledgements

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24-Extd contributor
[X] I'm a Hacktoberfest contributor
[X] I have starred the repository
[X] I am willing to work on this issue (optional)

@Poushmita Thankyou for raising the issue but no such changes required . you can see some other issue i would be happy to assign you and make sure to star our Repo.


Older versions can run.There are bugs in the old version of mod. There are four air pumps in the four corners of the aircraft. When it rises, there will be an angle tilt. When I destroy the air pump in that corner, the aircraft will be normal

only use forge server

仅使用伪造服务器
Forge server cannot install plug-ins


Running brigade on a production cluster side-by-side with production applications

Is there any precedence or official guidance (pros/cons) for running brigade in the same cluster that is running one's production applications? Taints/tolerations for node isolation perhaps or a resounding "don't do it"?
Has there been success or failure stories that can be shared with brigade specifically?
e.g. running builds/deployments through brigade using the same k8s nodes that are backing your production applications?
The brigade-k8s-gateway appears to require brigade to be running in the same cluster in order to gain it's benefits. Are there any plans to perhaps have that project turn into a "push" model that sends restful requests to another brigade instance just like how the webhooks work for github and bitbucket? e.g. cluster-A sends k8s events to cluster-B that has brigade installed.

You should definitely run Brigade in its own namespace. And RBAC is strongly encouraged. But when it comes to running on your prod cluster, it should be okay. We run ours on our production cluster, but our cluster is also relatively lightly used. Accounts on our prod cluster are also restricted to only highly trusted accounts.
On the k8s gw, I will have to look into that. I mean, it should be possible to have something collect k8s events and publish somewhere like a Kafka queue, and then write a Brigade GW that consumes the queue.


Dcn/checks

Added 1st stage of checks.

Do we need to display checks in Juxta, OBS, and the Audio module if the service is unavailable, or do we have any plans to implement checks in those modules, too?
In juxta, it throws an error
In OBS and Audio, it shows no content available message and the loder spins infinitely.

Since there are too much commits and "noise" on this PR due to old changes, I'm closing this PR and making a clean new one with the changes made 4 days ago on the dev branch.


i have a question~~~~help

const policies = this.options.render.csp.policies ? {...this.options.render.csp.policies} : null
^^^
SyntaxError: Unexpected token ...
at createScript (vm.js:74:10)
at Object.runInThisContext (vm.js:116:10)
at Module._compile (module.js:533:28)
at Object.Module._extensions..js (module.js:580:10)
at Module.load (module.js:503:32)
at tryModuleLoad (module.js:466:12)
at Function.Module._load (module.js:458:3)
at Module.require (module.js:513:17)
at require (internal/module.js:11:18)
at Object. (C:\GitDemo\nuxtDemo\node_modules\nuxt\lib\core\renderer.js:19:24)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>dev: nuxt
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above

Are you trying to clone this.options.render.csp.policies if it is not null?

it is likely to do with your version of node. Which node version do you use? Try upgrading to 8+

upgrade nodejs to 8.6+.

(The problematic part of the code should have been removed with nuxt/nuxt.js#2945 as well...)

It's the spread operator that's the problem, show me your nuxt config file and your babelrc if you have one. Show all your config that touches babel.
You should be compiling your javascript down to a older version using babel. What browser are you using also?

thanks, upgrade nodejs~~~


symbolic links in build tmp directories use absolute pathnames

Original issue: https://charm.cs.illinois.edu/redmine/issues/888

jimlisboa>ls -l /Projects/namd2/charm-6.6.1/multicore-linux64-iccstatic/tmp/charmc lrwxrwxrwx  1 jim tbres 24 Mar 15  2015 /Projects/namd2/charm-6.6.1/multicore-linux64-iccstatic/tmp/charmc -> ../../src/scripts/charmc* jimlisboa>ls -l /Projects/namd2/charm-6.7.0/multicore-linux64-iccstatic/tmp/charmc
lrwxrwxrwx  1 jim tbres 102 Nov  8 11:31 /Projects/namd2/charm-6.7.0/multicore-linux64-iccstatic/tmp/charmc -> /Projects/namd2/Programmers/jim/charm-6.7.0-build-2015-Nov-08-17541/charm-6.7.0-pre/src/scripts/charmc*
This is a regression of the following:
commit 8e507808a3e124f08b560122327fb70931fce35a
Author: Orion Lawlor <olawlor`acm.org>
Date:   Fri Sep 8 22:03:36 2000 +0000
Made soft links use relative pathnames instead of absolute.
This lets you move a charm++ installation without having
to recompile anything.

It also stops the confusing oddness that happens when you
move, e.g., charm to old_charm, and install a new charm.
Then the old_charm (absolute) soft links point into the
new charm directory!

Original date: 2015-11-11 16:21:08

This change was most likely a side effect of enabling out of tree build, hence assigning to Phil.

Original date: 2015-11-23 19:22:02

The change that caused this breakage was to allow out-of-tree builds, such as having the source tree on a shared FS and the build on a faster FS.
Unless this poses critical problems with practical deployment, we're not going to take the deeper changes necessary to maintain out-of-tree builds and also un-break this (copying all headers into include/ and scripts into bin/ instead of symlinking them through tmp/), because of the expectation that larger things will break when we do that.

Original date: 2015-11-24 03:23:39

The symlinks utility provides a way to turn absolute links into symbolic links, so this is fixable after the build.
We absolutely don't want to copy headers and scripts.  In-tree builds don't need to be able to move out of the tree.  The only change is to use relative links in tmp for in-tree builds, which should be possible.

Original date: 2015-11-30 21:59:27

Commits 921/922 fix this.

Original date: 2015-12-03 17:51:58

Can someone take a look at the unified fix in ~~https://charm.cs.illinois.edu/gerrit/#/c/921/~~ https://github.com/UIUC-PPL/charm/commit/6f8f9d422dff83a4e71639be4c982037877b6741 please?


Create a GraphiQL PR deploy for testing?

Using something like Netlify there could be a working build of every PR to this repo with a static deploy of graphiql which could hook up to the Star Wars API?

great idea! as in https://swapi.graph.cool/?

Every PR should get a build, for example Artsy deploys an instance of storybook js for every PR so you can preview the changes live. e.g. https://github.com/artsy/reaction/pull/2392#issuecomment-491077963

E.g. this PR https://github.com/graphql/graphiql/pull/781#issuecomment-491115471 would have been trivial to test because it would have a working copy of GraphiQL linked in the PR

agreed! I wonder... there's already the /example there, so if we just instructed that to build each time in travis, or just dropped an index.html in the root, even, we could use github-pages. Netlify would give us more options without potentially cluttering the repository

@orta the example project would be a good starting point for this possibly?

@orta you wanna take this on? Netlify would be a perfect candidate for this

Alright, yeah, I've done this on my own fork in https://github.com/orta/graphiql/pull/1 (sorry, #815 )
I'll request netlify to be added to this org (/cc admins @leebyron ) then once that's approved I can move it over.

looking good!
i would normally just use gh-pages, but netlify allows us to seamlessly host the build artifacts without committing them to source or a gh-pages branch

You can only have one gh-pages deploy, many (e.g. per PR) - so, it wouldn't be feasible to do it on a per-PR basis 👍

ohh! i see what you mean. i thought you meant a build of every PR merge. so you mean one for each PR! even better.

OK, this now has a PR to the monorepo branch - https://github.com/acao/graphiql/pull/6

thanks @orta ! this is a great feature. has made the refactoring effort a lot easier


Scrollbar can be customized to enhance overall design of the website.

@vipul-dixit-2004  !!
You should start addressing this  feature.

@Jagroop2001 assign me this JWOC

Hello @vipul-dixit-2004
I have a good hold in web development
Could you please assign me this issue after the coding periods starts?
I am from JWoC'24

Hi @Sruti-sahini  !!
This issue is assiged to @vipul-dixit-2004 ,
You can either create or select unassigned issue.

https://github.com/Jagroop2001/AcmeRockets/pull/9

https://github.com/Jagroop2001/AcmeRockets/pull/9


Python Wrapper for multicolumn dataframe

I finished the implementation of multicolumn Kdataframe. Please make python interface for these functions in the kDataframe class:
template
void addColumn(string columnName);
template
T getKmerColumnValue(string columnName,string kmer);
template
void setKmerColumnValue(string columnName,string kmer, T value);
As far as I remember, SWIG allows you to wrap template functions by defining new functions for each datatype. I suggest you make functions for these datatypes: int, float, and bool.
Please work on kDataframeMultiColumn branch.

SWIG requires to wrap all possible types in the template function.
So, if the function accepts all types of numbers like floats, doubles, integers.
and the template function is
template<typename T>
void foo(T number);

SWIG will require to wrap
void foo(int number)
void foo(double number)
void foo(float number)

So, kindly provide me all the possible data types to wrap each one of them.
Thanks.

might look into:
http://www.camillescott.org/2019/04/11/cmake-cppyy/
instead of SWIG.

@ctb Camille's solution seems to be very handy.
however, we are using SWIG for now cause it's very easy to write and time-saving to create the bindings just to make sure everything is working as expected. It will also allow us to create bindings to other languages like R.
After the beta release of the software, we will certainly consider using the very promising cppyy.

@mr-eyes start with int, float, and  bool.


refactor remove old vectorized scan node

Proposed changes
Issue Number: close #xxx
Problem summary
Remove following vectorized scan node
- volap_scan_node
- vjdbc_scan_node
- vodbc_scan_node
- ves_scan_node
Use new scan node in vec/exec/scan instead.
And remove BE config enable_new_scan_node
Checklist(Required)

Does it affect the original behavior:

[x] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.08 seconds
load time: 456 seconds
storage size:<PHONE_NUMBER>6 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221107152359_clickbench_pr_41427.html


Add author thumbnails as it is done in casper

Sorry, this is a typo in the branch name, it should mean "author-thumb" of course.

Sorry but no.
This is a major change in the design of the theme and I see no point in doing it since this is hardly a multi-author theme.
I'm not yet sure we're ever gonna need an author page, let alone an author thumbnail per post.


Update install-existing.md

Checklist

[ ] The title of the PR is clear and informative
[ ] If applicable, the changes made in the PR have proper test coverage
[ ] Issues addressed by the PR are mentioned in the description followed by Fixes.

Description

Fixes

duplicate


Cannot create svgpoint from a keyboard event in Firefox

In point.js it assumes that the event has defined clientX and clientY fields. But, if the event is a keyboard event it will not have those fields. Chrome is OK with assigning undefined to the field, but Firefox throws an exception:
TypeError: Value being assigned to SVGPoint.x is not a finite floating-point value.

d3.clientPoint isn’t supposed to be passed a KeyboardEvent. It expects a MouseEvent or a Touch.


Create reset_parameters method for ResNet blocks and models + move existing parameters init code there

This would ease re-initialization some blocks that are not meant to be pretrained/frozen:
https://github.com/pytorch/vision/blob/master/torchvision/models/resnet.py#L188-L203

Hi, I'd like to work on this feature. I have a good understanding of the ResNet code but this will be my first attempt at open source contribution. To clarify, the reset_parameter method should be able to reset/initialise parameters for any layer or block of layers passed right? Since that would serve the purpose of someone who has to customize the random init or freezing/pretraining of specific layers in the architecture? Can you please guide me on whether the reset parameters method needs to be written for a single layer or a set of layers? I can think of these changes after creating a reset_parameters method and moving the mentioned code into it:
`class ResNet(nn.Module):
def __init__(
    self,
    block: Type[Union[BasicBlock, Bottleneck]],
    layers: List[int],
    num_classes: int = 1000,
    zero_init_residual: bool = False,
    groups: int = 1,
    width_per_group: int = 64,
    replace_stride_with_dilation: Optional[List[bool]] = None,
    norm_layer: Optional[Callable[..., nn.Module]] = None
) -> None:
    super(ResNet, self).__init__()
    if norm_layer is None:
        norm_layer = nn.BatchNorm2d
    self._norm_layer = norm_layer

    self.inplanes = 64
    self.dilation = 1
    self.zero_init_residual = zero_init_residual

    if replace_stride_with_dilation is None:
        # each element in the tuple indicates if we should replace
        # the 2x2 stride with a dilated convolution instead
        replace_stride_with_dilation = [False, False, False]
    if len(replace_stride_with_dilation) != 3:
        raise ValueError("replace_stride_with_dilation should be None "
                         "or a 3-element tuple, got {}".format(replace_stride_with_dilation))
    self.groups = groups
    self.base_width = width_per_group
    self.conv1 = nn.Conv2d(3, self.inplanes, kernel_size=7, stride=2, padding=3,
                           bias=False)
    self.bn1 = norm_layer(self.inplanes)
    self.relu = nn.ReLU(inplace=True)
    self.maxpool = nn.MaxPool2d(kernel_size=3, stride=2, padding=1)
    self.layer1 = self._make_layer(block, 64, layers[0])
    self.layer2 = self._make_layer(block, 128, layers[1], stride=2,
                                   dilate=replace_stride_with_dilation[0])
    self.layer3 = self._make_layer(block, 256, layers[2], stride=2,
                                   dilate=replace_stride_with_dilation[1])
    self.layer4 = self._make_layer(block, 512, layers[3], stride=2,
                                   dilate=replace_stride_with_dilation[2])
    self.avgpool = nn.AdaptiveAvgPool2d((1, 1))
    self.fc = nn.Linear(512 * block.expansion, num_classes)

    self.reset_parameters(self.modules())

def reset_parameters(self, m) -> None:
    print('\n\n\nchecking entered RESET_PARAMETERS\n\n\n')
    for layer in m:
        if isinstance(m, nn.Conv2d):
            nn.init.kaiming_normal_(m.weight, mode='fan_out', nonlinearity='relu')
        elif isinstance(m, (nn.BatchNorm2d, nn.GroupNorm)):
            nn.init.constant_(m.weight, 1)
            nn.init.constant_(m.bias, 0)

        # Zero-initialize the last BN in each residual branch,
        # so that the residual branch starts with zeros, and each residual block behaves like an identity.
        # This improves the model by 0.2~0.3% according to https://arxiv.org/abs/1706.02677
        if self.zero_init_residual:
            if isinstance(m, Bottleneck):
                nn.init.constant_(m.bn3.weight, 0)  # type: ignore[arg-type]
            elif isinstance(m, BasicBlock):
                nn.init.constant_(m.bn2.weight, 0)  # type: ignore[arg-type]

`
What do you think about this?