Increase target frameworks

Is your feature request related to a problem? Please describe.
I am unable to consider moving steeltoe to use Microsoft service discovery due to the very limited target frameworks supported.
Describe the solution you'd like
I would like to be able to use Microsoft service discovery library with projects older than net 8 in a similar manner to which configuration etc works.
Describe alternatives you've considered
N/a
Additional context
Is a Blocker for https://github.com/SteeltoeOSS/Steeltoe/issues/1254

@thompson-tomo Hey there! Is Steeltoe still using netstandard/net6 or are you on net8? If you've moved to net8, we can close this issue, right?

@maddymontaquila it appears that in the latest preview branches Steeltoe has moved to only dotnet 8. hence will close this issue.


Added exception handling to handle multiple entries when reading the config file

... To fix issue #55
The program now terminates with a clear message where to look for the Duplicate entries.

Nice solution, thanks for the contribution


[Bug] 联合主键表，数据量较大，单个分片容量过大，内存被撑爆

Search before asking

[X] I searched in the issues and found nothing similar.

Flink version
flink1.16
Flink CDC version
flink cdc 2.3
Database and its version
MySQL 8.0x
Minimal reproduce step
create table (col1 int, col2 int, col3 int, col4 ......, primary key(col1, col2, col3))
insert 100000000 rows;
MySqlSourceBuilder.chunkKeyColumn("col1");
col1 values(1,2,3,4,5,6,7,8,9,10)
out of memory
What did you expect to see?
data synchronized
What did you see instead?
Error: out of memory
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

Considering collaboration with developers around the world, please re-create your issue in English on Apache Jira under project Flink with component tag 'Flink CDC'. Thanks.
为了便于各个国家的开发者合作，请用英文在 Apache Jira 的 Flink项目下重新提交问题，同时将 issue 的 Component 标注为 Flink CDC。谢谢。


Bring back the ability to pass a net.Conn for the client

The new client conf is too limited for me

Hello, i tried to the last to keep the net.Conn outside gortsplib.ClientConn, but the problem is that a ClientConn owns multiple connections and thread:

the main RTSP/TCP connection
multiple RtcpReceivers
the receiver report goroutine
multiple ConnClientUdpListeners if connection type is UDP

Therefore, in my opinion it's better if ClientConn owns everything (and closes everything avoiding leaks).
You can still access the main RTSP/TCP connection by using ConnClient.NetConn().
Let me know if you have a better idea.

I understand your point. Is possible to provide way to support custom Dialer? If that possible then I do not need net.Conn pass in.

solved with #3


Dashboard - chart should be updated when the date changes. without having to refresh the page

When date changed chart should update too. right now i need to refresh the page in order to see the updated chart
Actual Behaviour:
https://user-images.githubusercontent.com/25027592/201630666-909233cc-f3f6-430e-b988-e7f8b1dd85ac.mp4

Thanks @Aslam97 We've got some big plans for the dashboard to make it extensible and more widget based.

@alecritson You guys always amaze me. can't wait to see what you guys working on. in the meantime I will just close this issue


Update tests

Coverage remained the same at 100.0% when pulling 4c3d916276f77aef9f7e5a09587d820c56554dcc on update-tests into b1b9189152c1d4bac1e6d6da20d23780547efea5 on master.


[Feature Request]: Adding warning before removing users and group.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description

Here removing of the participants can be done by mistakes and there is no warning before removal.
Adding this feature will give user better experience.
Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

so in backend/scripts/seed.js  there is a minor error regarding database connectivity.
const connectDB = require("../config/db");
await connnectDB();
the spelling is wrong and i thing it is causing fault in seeding dummy data .
please let me know if there is such problem and also i am facing the problem regarding port

Ok change it then may have been over looked


general purpose override blocks purpose in all cases

Closes PROD-1513 (again)
Description Of Changes
Ensures that purposes are excluded no matter if they are declared with an inflexible legal basis.
Code Changes

[x] check first for exclusion in TCF experience generation
[x]  check first for exclusion on the PrivacyDeclaration model method

Steps to Confirm

[ ] pushed up an alpha tag, i'll do some integration testing locally with fidesplus to make sure this works as expected e2e 👍

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

@pattisdr i think this is ready for review, previous test runs looked good and i ensured the tests i added passed locally, so i am hopeful this CI run should be green.
i'll work on getting some e2e testing locally running to be extra sure, but in the meantime, mind taking a look at the code to make sure it looks ok to you?


Inconsistent use of Trait and Instance

Adding a trait to an object can be done in several ways. Suppose we would like to add a Generator trait, we could do
a = Trait(Generator)
b = Trait(Generator(), Generator)
c = Instance(Generator)
d = Instance(Generator(), Generator)

We use all four versions. For example a and b can be found in PointSource and c in InOut. Its not really clear to me when to use one over the other. c looks like it instantiates a default value. However, I cannot find a difference between a and b. Both accept None or Generator and default to None.
We should find out if there is a difference and then settle on one solution IMO.

Traits user guide on  Instance:
https://docs.enthought.com/traits/traits_user_manual/defining.html#instance
Traits API reference on Trait:
https://docs.enthought.com/traits/traits_api_reference/traits.html#traits.traits.Trait
They explicitly state that Trait should not be used in new code and Union should be considered.
I get 67 matches for Trait in our current codebase...
@acoular/all Any suggestions?

Thank you for pointing this out. It seems like the solution is pretty obvious: replace all occurrences of Traits with either Instance, Enum, Union or Map .
I get 69 matches for Trait in Acoular, but many of them can be automatically
The only reason I can think of for these inconsistencies in Acoular is that the traits package originally did not have all of these nice predefined traits and in the user guide back then the use of the Traits factory was recommended (as far as I remember).

I'll try and prepare a PR.

I'll try and prepare a PR.

@esarradj  That's great. I've already done all Trait -> Instance replacements in #374. So it would suffice to do the others. If you'd like to do them all, I can also rebase!
If I may, I would suggest to add a test in test_classes.py that fails if Trait is used as a trait.


Buttons on the WebUI

The Buttons on the WebUI for displaying Pokemons, Eggs and Items unfortunately do not work for me. Also most of the time the map does not display Pokemon, Pokestops and Arenas.

Could you give us more info? Like browser, url you use to access the gui?
Are you up to date on the bot?
Also could you paste here console log (from the bot) and browser log (from browser console)?

Im using Google Chrome. Im connecting to <IP_ADDRESS>:8000. uBlock Origin is disabled..
Console Log when connecting to the UI:
(5344) accepted ('<IP_ADDRESS>', 53729)
[2016-09-04 13:25:49] [UI] Web client connected
(5344) accepted ('<IP_ADDRESS>', 53733)
(5344) accepted ('<IP_ADDRESS>', 53734)
(5344) accepted ('<IP_ADDRESS>', 53735)
(5344) accepted ('<IP_ADDRESS>', 53736)
Browser Log when im connecting:
Load config from storage
bundle.js:1 Connecting to http://localhost:8000
bundle.js:1 Connected to Bot
bundle.js:1 Bot Ready.
bundle.js:1 Objectcoordinates: Array[3]level: censoredplayer: Objectstorage: Objectusername: "censored"proto: Object
Browser Log when im Pressing the "ShowPokemon" Button:
.. I don't get any Browser-Console Log ..
I do get to Display the Pokemon-List when I am continuously pushing the Button on page reload (F5).

Hi. This shouldn't be a problem, but you don't have to run the UI locally. Just go to http://openpogoui.nicontoso.eu/, it will work the same.
Also, I'm trouble understanding all those "accepted" log. Do you use virtual env to run the bot?

Also, do you see a log on the bot console when you click on a UI button?

My local service on 8000 redirects me to http://openpogoui.nicontoso.eu/
The "accepted" is displayed in my console while i am connecting to my local service.
I don't know whether I am using a virtual env. I have no experince with python.
Maybe you can help me figure it out.
No there is no log change when i click on a Button in the UI.

Could you try using virtualenv? I guess there is something that conflicts with one of the prerequisites.

ok thanks i will try this out later..

I had the same problem.  Running it in virtualenv makes it work for me.
Without virtualenv, it appears that OpenPoGoUI wasn't receiving events emitted from OpenPoGoBot.  Enabling debugging for the socket plugin for OpenPoGoBot allowed me to see that OpenPoGoBot was receiving events from OpenPoGoUI, but when OpenPoGoBot responded with an event, the event never appeared in the console of the browser in OpenPoGoUI.
Looking into it, this might be the problem:
https://github.com/miguelgrinberg/Flask-SocketIO/issues/253
Basically Flask-SocketIO (the library OpenPoGoBot uses for socket-io communications) doesn't work with npm's socket.io-client because of an alleged bug in socket.io-client's code.  The Flask-SocketIO author has submitted a pull request to fix it here: https://github.com/socketio/engine.io-client/pull/483/files


Add details about weekly bug triage call in the README

Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

@Madhu-1 @ajarr PTAL .

Merging this as we need more participation in the call at the earliest.


fix(useQuery): don't inform observers about CancelledErrors

the added test case confirms the issue - it fails on master.
closes #1876

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


#461 implement json compare in tests

Implements one example of comparing the json dump from a plotly.figure against expected output. A similar approach can be applied to other plots in future.
In this PR:

move scatter plot data to test dir.
add test for scatter plot
add fixtures to sort and compare json data. This allows plotly images to be converted to json and tested against known output.

Pull Request Testing

[ ] Describe testing already performed for these changes:

[ ] Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:

[ ] Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes or No]

[ ] Do these changes include sufficient testing updates? [Yes or No]

[ ] Will this PR result in changes to the test suite? [Yes or No]
If yes, describe the new output and/or changes to the existing output:

[ ] Do these changes introduce new SonarQube findings? [Yes or No]
If yes, please describe:

[ ] Please complete this pull request review by [Fill in date].

Pull Request Checklist
See the METplus Workflow for details.

[ ] Add any new Python packages to the METplus Components Python Requirements table.
[ ] Review the source issue metadata (required labels, projects, and milestone).
[ ] Complete the PR definition above.
[ ] Ensure the PR title matches the feature or bugfix branch name.
[ ] Define the PR metadata, as permissions allow.
Select: Reviewer(s) and Development issue
Select: Milestone as the version that will include these changes
Select: Coordinated METplus-X.Y Support project for bugfix releases or METplotpy-X.Y.Z Development project for official releases
[ ] After submitting the PR, select the :gear: icon in the Development section of the right hand sidebar. Search for the issue that this PR will close and select it, if it is not already selected.
[ ] After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
[ ] Close the linked issue and delete your feature or bugfix branch from GitHub.

I'm not familiar enough with this logic to confidently review this PR, so I'll let @bikegeek review it.

closing this as changes already included in https://github.com/dtcenter/METplotpy/pull/466


Typo in cy.realMouseMove code example in README

The code example for cy.realMouseMove is using cy,realMouseUp
Currently the code example for cy.realMouseMove in README.md is:
cy.get("sector").realMouseUp(50, 50, { position: "center" }); // moves by 50px x and y from center of sector

I think the correct example should be:
cy.get("sector").realMouseMove(50, 50, { position: "center" }); // moves by 50px x and y from center of sector

Link to relevant README section https://github.com/dmtrKovalenko/cypress-real-events#cyrealmousemove
Link to the specific text I'm referring to: https://github.com/dmtrKovalenko/cypress-real-events#:~:text=cy.get("sector").-,realmouseup(50%2C 50,-%2C { position%3A "center" })%3B %2F%2F moves
@dmtrKovalenko thank you for creating and maintaining such a useful Cypress plugin!
Please let me know if there are other issues or consideration and what the next steps should be. According to the contributing guidelines we should make an issue first. I was tempted to make a PR directly but I started with an issue and will follow the next steps when I hear back from you.

:tada: This issue has been resolved in version 1.7.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Driver-Driver conflict causes unclear error message [regression from 0.4]

The following code warns that the hierarchy will be flattened:
DriverConflict: Signal '(sig signal)' is driven from multiple fragments: top, top.test; hierarchy will be flattened
proceeds with Yosys, yielding a syntax error:
ERROR: Parser error in line 126: syntax error
The line in question:
    connect \D None
from amaranth import *
from amaranth_boards import versa_ecp5_5g as FPGA
from amaranth.vendor import LatticeECP5Platform
import os
from amaranth.lib.fifo import AsyncFIFOBuffered

class TestElaboratable(Elaboratable):
	def __init__(self):
		self.signal = Signal()

	def elaborate(self, platform) -> Module:
		m = Module()

		m.d.comb += self.signal.eq(0)
		# Uncommenting this causes driver-driver conflict error
		#m.d.sync += self.signal.eq(0)

		return m

class MCVE(Elaboratable):
	def elaborate(self, platform):
		m = Module()

		m.submodules.test = test = TestElaboratable()
		m.d.sync += test.signal.eq(0)

		return m

os.environ["AMARANTH_verbose"] = "Yes"
FPGA.VersaECP55GPlatform().build(MCVE())

Presumably this is supposed to cause a driver-driver conflict error (as it does when I tried to further simplify it)

This doesn't seem to reproduce on current main:
amaranth.hdl._ir.DriverConflict: Signal (sig signal) driven from domain None at ('/home/mwk/amaranth/amaranth/hdl/_xfrm.py', 667) and domain <amaranth.hdl._cd.ClockDomain object at 0x7f1b23891f50> at ('/home/mwk/amaranth/amaranth/hdl/_xfrm.py', 667)

The error message isn't the best (and improving it is on my list), but the error is definitely thrown. What version were you testing on?

This message is improved in #1158 to: amaranth.hdl._ir.DriverConflict: Signal (sig signal) driven from domain comb at ('/home/mwk/amaranth/amaranth/hdl/_xfrm.py', 667) and domain sync at ('/home/mwk/amaranth/amaranth/hdl/_xfrm.py', 667)
An unrelated upcoming change (removing AssignmentLegalizer) will also fix up the wrong filename currently reported.

In #1173 this changes to: amaranth.hdl._ir.DriverConflict: Signal (sig signal) driven from domain comb at ('/home/mwk/amaranth/t.py', 14) and domain sync at ('/home/mwk/amaranth/t.py', 25)


[doc] Update license information in doc and README

This PR removes LGPL license from the licence doc page and the readme.

LGTM


Allow arbitrary variables to be set via CLI arguments

At the moment, we take 2 fixed parameters via CLI arguments: https://github.com/solid/community-server/blob/e1533a0869071bbeabf0edfdfd05ccf57883cdaa/src/init/CliRunner.ts#L42-L46
However, the CLI also allows different config files to be set: https://github.com/solid/community-server/blob/e1533a0869071bbeabf0edfdfd05ccf57883cdaa/src/init/CliRunner.ts#L32-L35
These config files might have different variables, so it could be good if arbitrary variables could be instantiated via CLI arguments.
Alternatively, we could have the user point to a configuration file that already has the variables instantiated. (@rubensworks, what would look like?)
Caveat here is that one argument can be calculated based on another, as seen above with http://localhost:${argv.port}/.

So if I understand correctly, we want to enable something roughly like community-solid-server ... "urn:solid-server:default:variable:port=3000" ...?
I assume this would be a power-user feature?

Alternatively, we could have the user point to a configuration file that already has the variables instantiated. (@rubensworks, what would look like?)

Not possible yet, but can easily be implemented.
(I don't see a direct value for it in this case though)

So if I understand correctly, we want to enable something roughly like community-solid-server ... "urn:solid-server:default:variable:port=3000" ...?

Or just -port 3000 really.

I assume this would be a power-user feature?

Not necessarily; it's for adding values to preset configurations. Although supplying a JSON might be easier (see below).

(I don't see a direct value for it in this case though)

Separation of concerns.
The presets (for everyone) could ship with a couple of variables. My personal config (for me) has specific values, and points to the preset.

I think I misunderstood the issue, my bad.

Or just -port 3000 really.

Yes, that would be more convenient. We'll need a mapping (JSON-LD context?) for this then.

The presets (for everyone) could ship with a couple of variables. My personal config (for me) has specific values, and points to the preset.

Got it.
So basically something like (my-setting.json):
{
  "port": 3000,
  "base": "http://mypod.org/"
}

which would avoid the need to constantly having to enter all params via the CLI.
I would suggest allowing this to be passed as a separate option next to our Components.js config,
as to allow something like:
$ community-solid-server -c config/config-default.json -s my-setting.json.

(We may want to revise the option names)
The server could interpret my-setting.json as a JSON-LD file with an implicit context, which maps things like port to urn:solid-server:default:variable:port. We can inject this into the Components.js config.

Do we need a special mechanism?
I thought we could just
$ community-solid-server -c my-setting.json

wherein my-setting.json is then actually a Components.js-compatible file that includes config/config-default.json?
Or would that be a hassle wrt lookup paths?

Do we need a special mechanism?

We'll probably need to add something to Components.js to allow variable instantiation via config files. But this should be easy to do.

wherein my-setting.json is then actually a Components.js-compatible file that includes config/config-default.json?

Also an option.
My reasoning for having it as plain JSON was that most people probably want to make use of such settings files, and will never care about making their of server config, so the overhead of seeing all the Components.js may be unneeded.
But I have no strong opinion on this.

That's a neat trick; happy with having it explicit as well (which would allow Turtle too).

Turtle is a good argument against plain JSON indeed :-)

We will also need to pass this to the WebSocketAdvertiser. Let's add this into the integration test.

How does this relate to #453 ?

Error when unknown parameters are passed to the main executable
Currently, these commands will start the server, even though their parameters are not relevant:
community-solid-server foo bar
community-solid-server --foo --bar

How does this issue relate to #453 (quoted below) ?

From reading the discussion above, this issue sees to tackle 2 things:

Setting variable values being used in Components.js to be set through a config file.
Setting these same variables through the command line.

In case the second part is still relevant this does seem to conflict with #453 (unless Components.js would have something in place to determine if unknown variables are being set), so tagging @RubenVerborgh to clarify.

For starters, foo bar are textual arguments, which we don't support. So these can definitely be rejected.
Then for --foo and --bar: these are known abbreviations; the CLI will expand them to full URIs.
So in #453 we should reject:

textual arguments
unknown abbreviations

But we can accept arbitrary variables (full URIs then).

unless Components.js would have something in place to determine if unknown variables are being set

A cool idea would be to even move the abbreviations to the Components.js configurations, and perhaps even the default values. But then we'd somehow need access to the parsed config files. Let's leave that as future work 🙂

As in #964, for many apps, it will be natural to use env-vars for configurations like secrets, and so on. Following solution seems plausible.

Store env-var-name -> componentsjs-var-nam map as Record<string, string> coponent-instantiation in a file at pre-determined path (overridable with cli-param).
Making AppRunner to first instantiate that name-mapping from var-map-config, and load variables from env, and then instantiate app, from app-config.

env vars also have advantage of many tools like dotenv topopulate env-vars from .env file, and also setting env-vars from dockerfile, etc.

hello @joachimvh ,
I tried to take a shot at this issue, as we have to pass custom variables.
Here is a branch with an implementation: feature-custom-vars.
It defines a custom class called VarResolver, that takes

yargsOptions: which will allow us to specify custom cli args and their yargs options(like default, required, type, etc..)
varComputers, a hash of handlers from componentsjs var-iri to a handler that computes it from resolved yargs arguments. These handlers can be used to compute variable , like baseUrl from baseUrl/post args, or resolving-asset-paths, etc.
strictMode, etc.

implementation need to create componentsjs config file to instantiate above class, like here.
AppRunner will first parse cli-args with base-options and resolve path to above config file and instantiates VarResolver, and then passes argv to it to resolve final vars.
Then it instantiates App with resolved vars.
We should be able to start server just like previous as default var-config is setted to match previous vars.

@damooo That is great news, thanks! Can you open a PR from this branch, so we can comment?

@damooo That is great news, thanks! Can you open a PR from this branch, so we can comment?

I am poor at jest tests, as i am not used to it. One test will fail (that of AppRunner). is it ok?

Well it's a good start, we can take it from there 🙂

Following up in #1065.


Centralize system checks to one method and rework RenvBuilder around this idea.

Instead of making system calls throughout various methods in the class, create a function (utils.get_sys_info or something) that will return system information.  Based on a brief scan, this is what RenvBuilder uses in it's logic flow to determine system information, and therefore folder names etct.:

operating system
We are currently working with linux so it would be good to throw an error if os.name returns anything other than posix.
platform
Linux (and Cygwin?) vs Windows vs Mac installs look very different.  Throw an error for darwin, and win32 (and cygwin?).
size (32 vs 64 bit platforms)
On Linux machines 32 vs 64 installs look different.  Mainly, 32 bit systems use "lib" and 64 bit systems use "lib64".  Windows filesystems are much different.  I don't know the difference between 32 and 64 bit on windows.
import os
import sys

if os.name == "posix":
    if sys.platform in ["linux", "cygwin"]:
        # do stuff
        # do stuff
        if sys.maxsize > 2**32:
            _lib = "lib64"
        else:
            _lib = "lib"
else:
    raise OSError

#43

That's perfect. I actually added a catch for that in one of my branches (sys.platform).

Also, perhaps we could consider throwing a Warning instead of an error for mac.

import os
import sys

def check_os():
    if os.name == "posix":
        if sys.platform in ["linux", "cygwin"]:
            if sys.maxsize > 2**32:
                _lib = "lib64"
            else:
                _lib = "lib"
            return _lib
    else:
        raise OSError

Even less lines:
import os
import sys

def check_os():
    if os.name == "posix":
        if sys.platform in ["linux", "cygwin"] and sys.maxsize == 2**63-1:
            _lib = "lib64"
        elif sys.platform in ["linux", "cygwin"] and sys.maxsize == 2**31-1:
            _lib = "lib"
        return _lib
    else:
        raise OSError

I like it @sdhutchins.  Whenever we get to this, the best thing we can do is think forward...
How can we set up our code so that integrating windows/Mac is easy?
We can help do this with #43 as well.

You can also do this: https://kite.com/python/examples/4187/platform-check-if-the-system-is-32-bit-or-64-bit
'
import platform
print platform.architecture()[0] == "32bit"
'
Looks better than hard coding value and have Python calculating 2**63


Failed to construct source from eventlog

I started sparklint by using commandline sbt "run -d /path/to/log/dir -r", then I got the following message and the blank view on http://localhost:23763:
"level": "WARN ", "timestamp": "2017-03-09 17:23:02,616", "classname": "com.groupon.sparklint.events.EventSourceDirectory", "body": Failed to construct source from local-148904970384
My spark version is 2.1.0, scala 2.11.8. How can I fix this problem? Thanks

Hi @hwhsu could you paste the message immediately before this message?
Basically I'm looking for the logline from FileEventSourceManager to trouble shoot further.
Thanks!

Hi, @roboxue , Thank you for your prompt reply, here is the message:
[info] Loading project definition from /home/hwhsu/sparklint/project
[info] Set current project to sparklint-spark210 (in build file:/home/hwhsu/sparklint/)
[info] Updating com.groupon.sparklint:sparklint-spark210_2.11:1.0.5-SNAPSHOT
[info] Resolved com.groupon.sparklint:sparklint-spark210_2.11:1.0.5-SNAPSHOT dependencies
[info] Fetching artifacts of com.groupon.sparklint:sparklint-spark210_2.11:1.0.5-SNAPSHOT
[info] Fetched artifacts of com.groupon.sparklint:sparklint-spark210_2.11:1.0.5-SNAPSHOT
[info] Running com.groupon.sparklint.SparklintServer -d /home/hwhsu/logs -r
{"level": "INFO ", "timestamp": "2017-03-10 09:22:10,311", "classname": "com.groupon.sparklint.common.CliSparklintConfig", "body": Parsing args: -d /home/hwhsu/logs -r}
{"level": "INFO ", "timestamp": "2017-03-10 09:22:10,368", "classname": "com.groupon.sparklint.common.CliSparklintConfig", "body": Parsed into: directory, runImmediately}
{"level": "INFO ", "timestamp": "2017-03-10 09:22:10,370", "classname": "com.groupon.sparklint.SparklintServer", "body": Loading data from directory source CliSparklintConfig(true).directorySource}
{"level": "INFO ", "timestamp": "2017-03-10 09:22:10,382", "classname": "com.groupon.sparklint.common.Scheduler", "body": Scheduled task ScheduledTask(Directory source poller [/home/hwhsu/logs],<function0>,5,0)}
{"level": "INFO ", "timestamp": "2017-03-10 09:22:10,388", "classname": "com.groupon.sparklint.SparklintServer", "body": Executing ScheduledTask Directory source poller [/home/hwhsu/logs].}
{"level": "INFO ", "timestamp": "2017-03-10 09:22:11,317", "classname": "com.groupon.sparklint.ui.UIServer", "body": Server started on 23763}
{"level": "WARN ", "timestamp": "2017-03-10 09:22:11,921", "classname": "com.groupon.sparklint.events.FileEventSourceManager", "body": Failure creating file source from local-1489049703847: org.apache.spark.sql.execution.ui.SparkListenerSQLExecutionStart}

Part of modified contents of  my built.sbt
scalaVersion := "2.11.8"
crossScalaVersions := Seq("2.10.6", "2.11.8")
unmanagedSourceDirectories in Compile += (sourceDirectory in Compile).value / BuildUtils.getSparkMajorVersion(sparkVersion.value)
unmanagedSourceDirectories in Test += (sourceDirectory in Test).value / BuildUtils.getSparkMajorVersion(sparkVersion.value)

// Dependency
// Spark
lazy val sparkVersion = SettingKey[String]("spark-version", "The version of spark library to compile against")
sparkVersion := "2.1.0"
// Non-spark
lazy val http4s = "0.13.2"
lazy val optparse = "1.1.2"
lazy val scalatest = "2.2.6"
lazy val slf4j = "1.7.16"
lazy val log4j = "1.2.17"
lazy val json4s = "3.2.11"

resolvers in ThisBuild ++= Seq(
  Resolver.sonatypeRepo("snapshot"),
  Resolver.sonatypeRepo("releases")
)
libraryDependencies ++= Seq(
  "org.apache.spark" % "spark-core_2.10" % sparkVersion.value

Thanks~

Okay this is a new type of event introduced in Spark 2.0+ and the spark's developapi is not properly updated to parse this message from string.
Easiest Fix:
replace this line with
 def apply(line: String): SparkListenerEvent = Try(shimIfNeeded(JsonProtocol.sparkEventFromJson(parse(line)))).getOrElse(new SparkListenerEvent {})
Please try it out and let me know if this works

Wow, it's work ! Thank you very much !


Data source docs for Bentley et al 2016 data

The method line references point to the blob on dev - that's OK, right?

Coverage remained the same at 72.968% when pulling 18424adc27a7546ef137496b25657335ba8bd126 on clbarnes:extrasyn_data_doc into 2832413a8bb5669b690f86faa0c18ff82e730369 on openworm:dev.

Pointing to something on dev is fine since the history of that branch isn't going to be overwritten -- past global rewrites of git history notwithstanding.
It's not written into any contributor guidelines, but could you incorporate this source into the existing list under the Neurons heading, creating a sublist for receptors, neuropeptides, neurotransmitters and innexins?


refactor: address type is strict hexstring

Refactor that I need for postage stamp work. Strictly types Address type to HexString.

next time please target that branch your PR is based on.

@nugaon sorry, that was because of some rebases that I did not propagate all the way here.


Sx1302 not working

Sx1302 / sx1303 not working with current packet forwarder. Need to investigate why

Manually running chip_id on SX1301 based device:
root@0f1e195f0f9c:/opt/sx1302# ./chip_id
CONCENTRATOR_RESET_PIN parameter not passed in, using value from the environment (val=)
./reset_lgw.sh: 33: echo: echo: I/O error
./reset_lgw.sh: 36: ./reset_lgw.sh: cannot create /sys/class/gpio/gpio/direction: Directory nonexistent
CoreCell reset through GPIO...
Opening SPI communication interface
Note: chip version is 0x00 (v0.0)
ERROR: Failed to set SX1250_0 in STANDBY_RC mode
ERROR: failed to setup radio 0
ERROR: failed to start the gateway

This correctly gives a 1 exit code
On SX1302 based:
root@b020abb3ab4e:/opt# cd sx1302
root@b020abb3ab4e:/opt/sx1302# ./chip_id -d /dev/spidev1.2
CONCENTRATOR_RESET_PIN parameter not passed in, using value from the environment (val=)
./reset_lgw.sh: 33: echo: echo: I/O error
./reset_lgw.sh: 36: ./reset_lgw.sh: cannot create /sys/class/gpio/gpio/direction: Directory nonexistent
CoreCell reset through GPIO...
Opening SPI communication interface
Note: chip version is 0x10 (v1.0)
INFO: using legacy timestamp
ARB: dual demodulation disabled for all SF

INFO: concentrator chip model ID: 0x00

INFO: concentrator EUI: 0x0016c001ff1a08f0

Closing SPI communication interface
CONCENTRATOR_RESET_PIN parameter not passed in, using value from the environment (val=)
CoreCell reset through GPIO...

This correctly gives a 0 exit code
However when running automatically on an sx1302 device, it is not recognised as being sx1302 based.
Issue, I guess, is something to do with the check:
https://github.com/NebraLtd/hm-pktfwd/blob/3f7c41eedfcbda414faec4d2cc13e149d7ca704e/pktfwd/utils.py#L62-L76
Or possibly how we use it later.
These devices on balena are running sx1302
https://dashboard.balena-cloud.com/devices/28c44310bfc8ffc6098bce2b3696336e/summary
https://dashboard.balena-cloud.com/devices/7f17117b0ff743debb90055bf21ceaa7/summary

@marvinmarnold I guess from python docs it needs to be without the stdout like this:
subprocess.run(util_chip_id_cmd, capture_output=True,
                        text=True, check=True)

as we aren't using stdout for anything currently?
maybe in future we will want to use the output to determine if sx1302 or sx1303 but not sure if that maybe is best in a different function? not sure what you think?

created a PR in #71 - unless you think there is a better way to do this?

Ok, that didn't seem to work anyway! 🤦

On second thought, I guess this is the issue:
./reset_lgw.sh: 33: echo: echo: I/O error
./reset_lgw.sh: 36: ./reset_lgw.sh: cannot create /sys/class/gpio/gpio/direction: Directory nonexistent

As it's probably throwing an exception

Weirdly, it also seems to try to reset twice....
 packet-forwarder          SX1302_LORA_PKT_FWD_FILEPATH=/opt/sx1302/lora_pkt_fwd
 packet-forwarder          SX1301_LORA_PKT_FWD_DIR=/opt/sx1301
 packet-forwarder
 packet-forwarder  2021-11-15 16:42:32,945 - [DEBUG] - pktfwd.pktfwd_app - (pktfwd_app.py).set_variant_attributes -- /opt/pktfwd/pktfwd_app.py:(91) - Variant COMP-RAKHM set with reset_pin 17 and spi_bus spidev0.0
 packet-forwarder  2021-11-15 16:42:32,946 - [DEBUG] - pktfwd.pktfwd_app - (pktfwd_app.py).start -- /opt/pktfwd/pktfwd_app.py:(39) - STARTING PKTFWD
 packet-forwarder  2021-11-15 16:42:32,948 - [DEBUG] - hm_pyhelper.miner_param - (miner_param.py).await_spi_available -- /opt/pktfwd-dependencies/hm_pyhelper/miner_param.py:(217) - SPI bus spidev0.0 Configured Correctly
 packet-forwarder  2021-11-15 16:42:32,950 - [DEBUG] - pktfwd.pktfwd_app - (pktfwd_app.py).prepare_to_start -- /opt/pktfwd/pktfwd_app.py:(73) - Region set to EU868
 packet-forwarder  2021-11-15 16:42:32,950 - [DEBUG] - pktfwd.utils - (utils.py).await_system_ready -- /opt/pktfwd/utils.py:(49) - Waiting 5 seconds for systems to be ready
 packet-forwarder  2021-11-15 16:42:37,956 - [DEBUG] - pktfwd.utils - (utils.py).await_system_ready -- /opt/pktfwd/utils.py:(51) - System now ready
 packet-forwarder  2021-11-15 16:42:37,957 - [DEBUG] - pktfwd.pktfwd_app - (pktfwd_app.py).prepare_to_start -- /opt/pktfwd/pktfwd_app.py:(76) - Finished preparing pktfwd
 packet-forwarder  2021-11-15 16:42:37,971 - [DEBUG] - pktfwd.utils - (utils.py).update_global_conf -- /opt/pktfwd/utils.py:(99) - No chip EUI. Assume SX1301
 packet-forwarder  2021-11-15 16:42:37,972 - [DEBUG] - pktfwd.utils - (utils.py).replace_sx1301_global_conf_with_regional -- /opt/pktfwd/utils.py:(116) - Copying SX1301 conf from /opt/pktfwd/config/lora_templates_sx1301/EU-global_conf.json to /opt/global_conf.json
 packet-forwarder  CONCENTRATOR_RESET_PIN parameter not passed in, using value from the environment (val=17)
 packet-forwarder  CoreCell reset through GPIO17...
 packet-forwarder  CONCENTRATOR_RESET_PIN parameter not passed in, using value from the environment (val=17)
 packet-forwarder  CoreCell reset through GPIO17...

Weirdly, it also seems to try to reset twice....

I think this is because chip_id calls reset and then immediately after we explicitly do another reset before starting the concentrator. Its a little weird that the replace_sx1301_global_conf_with_regional logline doesn't show up in between the two resets, but given there's a mix of logs coming from python vs C, not too surprising.

#72 is only a partial fix. With that change, the correct concentrator version is now detected. But it still fails to start:
2021-11-16 02:54:25,361 - [DEBUG] - pktfwd.utils - (utils.py).await_system_ready -- /opt/pktfwd/utils.py:(49) - Waiting 5 seconds for systems to be ready
2021-11-16 02:54:30,368 - [DEBUG] - pktfwd.utils - (utils.py).await_system_ready -- /opt/pktfwd/utils.py:(51) - System now ready
2021-11-16 02:54:30,370 - [DEBUG] - pktfwd.pktfwd_app - (pktfwd_app.py).prepare_to_start -- /opt/pktfwd/pktfwd_app.py:(76) - Finished preparing pktfwd
2021-11-16 02:54:32,601 - [DEBUG] - pktfwd.utils - (utils.py).is_concentrator_sx1302 -- /opt/pktfwd/utils.py:(72) - SX1302 / SX1303 Detected
2021-11-16 02:54:37,626 - [ERROR] - pktfwd.utils - (utils.py).retry_start_concentrator -- /opt/pktfwd/utils.py:(185) - lora_pkt_fwd stopped with code=1.

Hmmm strange.

@marvinmarnold just had a thought. What happens if you put the rest script both in the old location and the new location?
It used to work, so I'm wondering if the packet forwarder is expecting it somewhere else?

Actually it's showing a message saying it can't find global_conf.json so maybe that's in the wrong place:
https://github.com/NebraLtd/hm-pktfwd/blob/3f7c41eedfcbda414faec4d2cc13e149d7ca704e/pktfwd/utils.py#L128-L130
Think this should be sx1302 directory or root dir?

testing in https://github.com/NebraLtd/helium-miner-software/pull/264 and this issue will be closed when pushed to production

This is in production now. Closing.

@Benixpearlin probably an incorrect reset pin script or configuration file


A kind of filter template SEO or SEO Monitor

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
Other
🔖 Feature description
It would be great if there was a way to assign templates or group templates. Here you could set keywords such as "nofollow," "noindex," etc. So, monitoring the page for SEO parameters/metas. It's cumbersome when you have to set everything individually. Often, there are multiple options per URL.
✔ Solution
I can create a keyword template for a group. All pages assigned to the group will be automatically searched for the group's keywords. Alternatively, an SEO monitor with the option to select various SEO parameters, such as nofollow, noindex, and more.
❓ Alternatives
No response
📝 Additional Context
No response

I think this is a duplicate of either bulk actions (see https://github.com/louislam/uptime-kuma/issues/2934) or config templates (see https://github.com/louislam/uptime-kuma/issues/1323)
If you agree, could you please close this Issue, as duplicates only create immortal zombies and are really hard to issue-manage?
If not, what makes this issue unique enough to require an additional issue? (Could this be integrated into the issue linked above?) ^^

@syberx
@
We are consolidating duplicate issues a bit to make issue management easier.
I think, we should track this issue in https://github.com/louislam/uptime-kuma/issues/2934 and https://github.com/louislam/uptime-kuma/issues/1323 as there is no difference
⇒ I am going to close this as a duplicate.


How to Reverse Interview Investors

subtitles: Good Questions to ask an Investor as an early Startup Employee
tags: careers, advice, investing
category: note
slug: investor-questions
cover_image: https://user-images.githubusercontent.com/6764957/216241922-da658bb5-7aec-4da2-814f-498094196aba.png
I honestly never expected this to be a topic that was common enough to write up, however, it suddenly hit me today that it is the ultra niche topics that deserve writing up since it is the stuff that is outside the usual SERP riffraff.
Context
Most employees joining a company see a job description, apply, get the offer, and then have to make up their mind whether to join based on just the info they get through this process. However, when joining an early stage (say series B and below) startup, it's possible, even wise, to obtain more information on the company before joining, especially since it is much more uncertain what the future of the company will be (I have had friends laid off 3 months after joining a startup because they pivoted) and there's a lot less public data about the company and its founders (so you can't, say, look up their revenues, or customer reviews, or glassdoor reviews).
One way that startups have been helping prospective employees through this information gap is offering interviews with their investors. There are multiple good reasons to do this:

the investor is almost definitionally the most bullish person in the world on the company outside the company,
the investor wants to "add value" to their founders by helping to "close" you as an employee,
the employee gets special access to their investor (sometimes boardmember) that they would not normally have, to network or see things from their vantage point,
the founder gets a very small chance that some red flag is caught by the investor and the opportunity to prevent a mishire.

The more in-demand, and senior, you are, the more likely you'll want to do this anyway to collect info about the company and for them to be comfortable with you (if things go well, you'll be seeing them at board meetings soon enough).
I've been through this process twice now, and today was asked by another friend going through the same thing. I realized that this is an underserved topic in career advice since very very few people actually end up going through this process.

TLDR
This was my answer:

An expanded version follows.
Questions

The Investment Decision.

Why did you invest?
What did you find during diligence?
What did you struggle with the most before committing?
How was valuation decided?

Current state

Founders: What are their best qualities? Who do they most closely remind you of? What do you perceive as their biggest flaws?
Information: What have you learned about the company since investing, good and bad?
Risks: What do you think is the riskiest assumption of the company? How can we validate that assumption?
Market: What is attractive about this market? What do you like about the customers, how do you view the competitive landscape? (they've likely done an extreme amount of research here, take notes)

Future

Base case: What metrics are you/the company looking for to reach the next round? (in the short term this is the ballgame you are choosing to play)
Best case: What metrics would make you pre-empt the next round? (just fun to ask if the investor is particularly in love with the company)
Worst case: What metrics would make you recommend a pivot? (this is probably most likely scenario for early stage companies - watch for unusual honesty)
Exits: If the company was to be acquired, who would be the natural buyers and why? (speculation yes, but useful info on how the investor views this asset regardless)
(good ending question) What is the most ambitious potential of this company? Where does this go if everything goes right? (assess if this excites you)

Ultimately, these are just a very generic set of questions for you to start from. There's only one binary decision to be made by the time you reach this stage - join, or don't join. Have extreme focus on asking questions that will help you make that single decision; everything else is fluff.
Let me know in the comments if you can think of other questions suitable for investors.

i also keep an ungroomed list of founder questions, in case anyone wants to broaden out https://github.com/sw-yx/brain/blob/master/R - Better Questions/startup questions.md

more questions to analyze companies: https://www.thediff.co/archive/100-due-diligence-questions-checklist/


ocf-resource-agents: fix pos attribute to define source location

Description of changes
ocf-resource-agents: fix pos attribute to define source location
Without the change commands like
$ nix edit -f. ocf-resource-agents

do not point to a file.
ocf-resource-agents: expose it's inputs as attributes
With individual attributes exposed it's easier to interrogate and
upgrade constituents in nixpkgs.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3578

Fails to build currently on ofborg
checking whether gcc supports "-Wunsigned-char"... checking how to run the C preprocessor... gcc -E
configure: error: in '/build/source':
configure: error: C preprocessor "gcc -E" fails sanity check
See 'config.log' for more details
error: builder for '/nix/store/mr8p9d5f4ksd59i5290rza34x8ajp7cj-resource-agents-4.10.0.drv' failed with exit code 1;

I think it should have been fixed by https://github.com/NixOS/nixpkgs/pull/287053


Add version parameter to all of the analysis pipelines except the deprecated 10x.

I expect this PR should wait for @jishuxu's PR about outputs changes to be merged in so we can tag them together as smartseq2_v2.0.0
Please ensure the following when opening a PR:

[ ] Added or updated tests
[ ] Updated documentation
[ ] Applied style guidelines
[ ] Considered generalizability beyond our own use case

I just saw that the circleci tests are failing, could you take a look at those in case they need to be updated as well?

@samanehsan Yeah, I have no idea why the portability tests failed, consulting @mckinsel now, the worst case would be that adding a version to the pipelines is destroying the portability :(

The latest failed Mintegration test was because that the test suite itself is not updated to match the master branch of Lira.

retest this please


Extension tips

Added a couple more extensions to the tips json.

@waderyan it looks like this PR also includes changes to the Feedback widget. Is this an oversight? The changes to the extension tips looks good to me.

Yep that was an oversight. Let me fix that.

Close in favor of new PR

@waderyan no need for a new PR. I can just commit the changes to the extensiontips.


DataTrackingBug

Describe the issue
I feel like my phone had been hacked with a bug
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

[]RFC 3986: Uniform Resource Identifier (URI): Generic Syntax, Section 3.5url)


Bug Report: Combo Breaks on results screen not matching with the ones under the health bar.

If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
Windows x86_64 I believe
What version of the game are you using? Look in the bottom left corner of the main menu.
1.5.3
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
No
Please describe your issue. Provide extensive detail and images if possible.
Combo breaks on the results screen do not match combo breaks under the health bar (see image below)

shits are also counted, while on the health bar they are not. I'll be making it more clear in the next update.


Implement parameterScope policy assignment parameter metadata

See Azure/Enterprise-Scale#882
Related to #244

Trigger ADO Sync

Trigger ADO Sync

As part of improving the handling of Role Assignments for policies, I think we should implement this as part of the broader work required around issue

#439

Closing issue as duplicate, as will track against #439.


Objective C examples

Hi there,
Seems the example code in this documentation is largely in Swift, though there are scattered bits of Objective-C throughout. Would you be able to flesh out your Objective-C example code? It would be incredibly useful.
Thanks!

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

ID: 0df91a5b-58a1-d91a-0aac-8866357f8bdf
Version Independent ID: c7baedc0-0e95-80b5-d851-bb209adadd0a
Content: App Center Crashes for iOS - Visual Studio App Center
Content Source: docs/sdk/crashes/ios.md
Service: vs-appcenter
GitHub Login: @WinnieLi
Microsoft Alias: yuli1

Hi @keithburgoyne , there's a switch at the top of the page:

Apologies that it took so long to respond.


Bot token

Due to how we are doing releases + copr release we cannot use the default github account
When you use the repository's GITHUB_TOKEN to perform tasks on behalf of the GitHub Actions app, events triggered by the GITHUB_TOKEN will not create a new workflow run. This prevents you from accidentally creating recursive workflow runs.
An account was generated. It will need permissions to be a member of the project with a secret loaded.

/lgtm


Improve electron support

Currently the timings look this:
  timings: {
    start:<PHONE_NUMBER>669,
    socket: undefined,
    lookup: undefined,
    connect: undefined,
    secureConnect: undefined,
    upload:<PHONE_NUMBER>682,
    response:<PHONE_NUMBER>083,
    end: undefined,
    error: undefined,
    phases: {
      wait: undefined,
      dns: undefined,
      tcp: undefined,
      tls: undefined,
      request: NaN,
      firstByte: 401,
      download: undefined,
      total: undefined
    }
  }

Missing:

timings.end
timings.phases.request
timings.phases.download
timings.phases.total

If you add any workarounds, ensure you also open Electron issues.

They ain't welcome: https://github.com/electron/electron/issues/21517 https://github.com/electron/electron/issues/11787#issuecomment-565624923
https://github.com/electron/electron/issues/11787 was fixed by https://github.com/electron/electron/pull/21244 but there's no mention of it. The person who handled it, even didn't care to check if the issue is real though there was reproducible code provided.  I just feel this way.
For example they say "follow the Code of Conduct" but they don't follow it either.
I just posted this (instead of make I meant mark).


Fixed an issue where todo tests wouldn't be given an outcome as part of the UnitTestResults

Looks like there was one spot missed as part of #198
That change fixed running the tests, but now there's an issue with the trx file, all uses of todo don't get a an outcome tag. Tracked this down to the constants file not having a conversion of todo to "NotExecuted" so I've added that.
Before this change our pipelines were failing, counting all todo tests as failures.
I've confirmed with this change that our pipelines succeed, counting the todo tests as others.

Good catch! Thank you for this. I'm happy to merge this, just one thing: could you please squash your commits into one and set its commit message so that it adheres to conventional commit message (this is needed for the automatic release job)? Thank you

@no23reason Woops, sorry! I'm used to Azure DevOps doing the squash at merge time.
I think I got it in line with conventions but let me know if there's anything else I should change.

@MasonBergstrom Thank you so much, now it is perfect :) it should be released soon. Thanks again


⚠ Bofrost.be has degraded performance

In 1871855, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1303 ms

Resolved: Bofrost.be performance has improved in 596068b.


r

Codes such as m.casToken += 1 and item.casToken = m.casToken was being executed in almost every
command handling functions, so I decided to make a function for it.

Hi @sang-w0o.
Please rename this PR title.


Manually run jobs

Is there a way to manually run jobs in the DB? We would use it in the development and test environments, where we might need some finer control over when are the jobs running.

First, let me understand your use case a bit more. Are you talking about integration testing, where you want to test that the job has been run and that the work in the handler has been done? Or is it that you want to specify some time in the future that the job is run? Or something else?

In the development I would like to be able to run jobs without starting the worker process. For instance:

Call a domain function which queues the job
Manually trigger job from REPL

This way we can develop without starting the whole app in every scenario. This would also be convenient in some test scenarios where we might test domain functions without starting the whole app.
I think that the only change that might be needed is extracting this into a defn without marking it as a private var https://github.com/msolli/proletarian/blob/main/src/proletarian/worker.clj#L20-L41
For real integration tests, I see value in starting the worker process.

@retro Please have a look at PR #3 - would something like this work for your use case?

@msolli This is great, thank you! I think it will cover all the cases I have in mind


[README] Update suggested unit test

@samnela Thank you for the fix! PHPUnit changed to namespaced classes in version 7 and this was missed.


Fixed check for script execution events (issue #387)

Fix for issue #387.
Sebastian

Closing this pull request without merging, because I manually merged it (it had to be changed in the .htaccess file too)


Questions regarding dataset and missing file.

@rktamplayo It would be great if you could help me with a few questions regarding problems that I am facing while running the code.

I see that the train/dev/test file needs to have summaries as well. Is this a supervised summarization setting? Because I was of the notion that the paper introduced creation of synthetic datasets where the summaries where pseudo summaries selected from the input reviews. Can you clarify on how we can use some other data consisting of just raw customer reviews and produce general and aspect specific summaries for them? (No human annotated general or specific summary is available)

While running the train_sum.py I got the following error :
Traceback (most recent call last):
File "src/train_sum.py", line 19, in
from calculate_rouge import calculate
ModuleNotFoundError: No module named 'calculate_rouge'

I guess the calculate_rouge file is missing in the repo. Can you please provide it?
Awaiting your response. Thanks in advance!!

Sorry for the late response.

The summaries required by the summarization model are "pseudo-summaries". That is, they are created synthetically by the method described in the paper.
I have uploaded the missing file. Sorry again for the delay!


Is there any example about using "ddp_notebook" in jupyter notebook?

🚀 Feature

Motivation

Pitch

Alternatives

Additional context

If you enjoy Lightning, check out our other projects! ⚡

Metrics: Machine learning metrics for distributed, scalable PyTorch applications.

Lite: enables pure PyTorch users to scale their existing code on any kind of device while retaining full control over their own loops and optimization logic.

Flash: The fastest way to get a Lightning baseline! A collection of tasks for fast prototyping, baselining, fine-tuning, and solving problems with deep learning.

Bolts: Pretrained SOTA Deep Learning models, callbacks, and more for research and production with PyTorch Lightning and PyTorch.

Lightning Transformers: Flexible interface for high-performance research using SOTA Transformers leveraging PyTorch Lightning, Transformers, and Hydra.

cc @justusschock @kaushikb11 @awaelchli @akihironitta @rohitgr7

I don't think there is any example using ddp_notebook specifically. Even if we did, it'd look almost the same as existing examples in examples/.

Do you see any issue around the strategy?

Exactly as @akihironitta said, as soon as you are in a Jypyter notebook you get to use the feature without changing anything.
As soon as you use multiple devices, Trainer(devices=2) for example, ddp_notebook gets automatically activated. No code changes required.


Generate a "run_results" artifact after any Meltano run

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2805
Originally created by @tayloramurphy on 2021-06-17 14:01:03

In discussion with Stephen Bailey from Immuta he brought up the idea of having some kind of run artifact similar to what dbt does with their artifacts.
At a minimum it would be good to have some kind of summary of an elt run to be able to ship anywhere, but I could also see us having the equivalent of a manifest generated as well. This would tie in well with https://gitlab.com/meltano/meltano/-/issues/2611

View 8 previous comments from the original issue on GitLab

Still relevant

Still relevant, but I'd like to ask folks for use cases and what info would be valuable to have in it.


cluster: skip tests when no private ip address exists

Closes #1445.
As described in the issue, the other option would be to listen on the <IP_ADDRESS>/8 addresses but I fear that it might still break in some exotic environments anyway.

but that's all

Interpreting this as a lgtm. Hope you don't mind @stuartnelson3.

sorry for the delay, yeah that was a lgtm :)


Support ARM64

Why trying to add support for ARM64 to my UWP projects, I get this:

I am happy to contribute with a PR, but need to know whether you will release a new version.
Thanks!

Hi!
I think so too. But, Win2D library doesn't support for ARM64 now. I will support for ARM64 after Win2D support ARM64.
https://github.com/Microsoft/Win2D/issues/637
Thanks!

🎉 Win2D has supported AArch64 since 1.24.0.
Because I don't have Surface Pro X, I haven't tested it running on AArch64. However, I released SvgForXaml version 0.9.3, which builds in Visual Studio 2019 and depends on Win2D 1.24.0 (with added ARM64 support).
https://www.nuget.org/packages/Mntone.SvgForXaml/0.9.3


Add support for system-level config

Thanks for this PR! Seems like a good idea to allow a system config :)


Colagrossi problem description and input are inconsistent

The Colagrossi dambreak problem has slip conditions at the walls of the tank based on the input files, but the README says they are no-slip.

Chris, just fixed that.

OK, dambreak validation problems usually use slip conditions, right? We've done it both ways, but I don't recall the no-slip condition being very useful at this scale.

No-slip condition will affect some small features, like the leading edge but should not change the bulk of the flow.


feat(participant): remove jwt support in favor of access keys

This is needed for an upcoming feature. It is not breaking to consumers other than ourselves. Tested this E2E locally.

@connor4312,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

My follow-up commit will also let us automatically reconnect to the new server if the game client disconnects and reconnects somewhere else, so that's nice 😄


Create Support Page and CSS - Issue #835

Pull Request Template

Please provide all of the details below. Pull requests missing any of the information below will be closed!
Providing the issue number/url helps us identify the problem being solved and keep the issues tab clean and tidy

Create Support Page and CSS - Issue #835
Summarize what this PR fixes:-
Added Support page (support.index)
Added Support css file (support.css)
(this is regarding Issue #835)
Checklist
(Add an x between the brackets to check the items)

[x] I have summarize what this PR fixes above
[x] I have provided the issue ID/URL above
[x] I have not added Bootstrap or jQuery and used correct styling guide.
[x] This PR fixes only the issue mentioned above (create separate PRs for separate issues)
[x] I was the first one to claim the issue or I created the issue - I created the issue

Again, looks good. Would be nice to have a link in the dropdown menu and the dark mode is not working.


Add Redis store.

Add Redis support for this lib.

Is there any chance of this getting merged soon? It looks very very close to being ready.
error: this import is redundant
  --> axum-login/src/redis_store.rs:52:5
   |
52 |     use serde_json;
   |     ^^^^^^^^^^^^^^^ help: remove it entirely
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_component_path_imports
   = note: `-D clippy::single-component-path-imports` implied by `-D warnings`

    Checking axum_login_tests v0.4.1 (/home/runner/work/axum-login/axum-login/axum-login-tests)
error: useless use of `format!`
  --> axum-login/src/redis_store.rs:65:13
   |
65 |             format!("{}", self.id)
   |             ^^^^^^^^^^^^^^^^^^^^^^ help: consider using `.to_string()`: `self.id.to_string()`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_format
   = note: `-D clippy::useless-format` implied by `-D warnings`

error: could not compile `axum-login` due to 2 previous errors
warning: build failed, waiting for other jobs to finish...
Error: Process completed with exit code 101.

It looks like some checks are failing, as you pointed out.
Note that you should be able to use this directly in your own project if you implement the store as this PR does.

Fixed the clippy checks that were failing.

Fixed the clippy checks that were failing.
@maxcountryman Is it ok to merge or do you need something else from my side?

@schaschko sorry for the delay. There's a conflict on Cargo.toml now.

Closing this in favor of #73.
Also wondering if these sorts of implementations should be embedded in this crate or not: they can be entirely separate crates, for instance.

@schaschko  This should be reopened/rebased after we discuss how to proceed with the project structure.

Please let me know, I will look into it then.


Incorporate Real Data

Put all work for this issue in a branch called issue-45.
Add "real" data to website : for our page, add 15 lost items to the page with realistic information and photos.

Please disregard the last comment; I have added 15 more items to the system.


Add Invoke Command Implementation

Problem
Per interaction model, we need to invoke command client to deliver invoke command request,
InvokeCommandRequestTLV Type: Structure (Anonymous)
Element | Comments | Tag Type | Tag ID | Type | Range
CommandList |   | Context Tag | 0 | List of CommandDataElement | -
Proposed Solution
Implement Invoke command

I will add unit tests for Command and InteractionDataModelEngine


Aerc puts store references into the default config

Issue description
aerc has custom filters, and the default behaviour on first launch is to put the data directory in the ~/.config/aerc/aerc.conf file, like so:
...
[filters]
subject,~^\[PATCH=/nix/store/kvgcaxjq8z2va3i7dplwrxrvyrv5dxfi-aerc-0.1.1/share/aerc/filters/hldiff.py
text/html=/nix/store/kvgcaxjq8z2va3i7dplwrxrvyrv5dxfi-aerc-0.1.1/share/aerc/filters/html
text/*=/nix/store/kvgcaxjq8z2va3i7dplwrxrvyrv5dxfi-aerc-0.1.1/share/aerc/filters/plaintext.py

These paths will break on an update to aerc.
I'm the maintainer, but I'm not exactly sure what the best way to fix this with Nix is.

It's part of the build
https://git.sr.ht/~sircmpwn/aerc/tree/master/config/aerc.conf.in#L77
A level of function and structure would probably need to be changed to remove this.

Currently, $out/share/aerc/filters is put in PATH for aerc so in the case for text/html you can simply do text/html=html: https://github.com/NixOS/nixpkgs/blob/8d1510abfb592339e13ce8f6db6f29c1f8b72924/pkgs/applications/networking/mailreaders/aerc/default.nix#L41-L42
But as for subject,~^\[PATCH and text/* the default config now uses awk -f /nix/store/.../share/aerc/filters/[hldiff,plaintext] respecively which will need a different solution.


feat(aws-ec2-bastion-server): add permissions_boundary to role created by aws-ec2-bastion-server

what

add permissions_boundary to role created by aws-ec2-bastion-server

why

Our org requires all roles have permissions_boundary

references

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/iam_role#permissions_boundary

/terratest


Store card information does not work

I need to store card details for future use. I am doing it with locally store information transaction reference number and token.
i found from the doc to pass stored transaction reference number and token while paying with saved card. but it is showing me error "payment not processed" . Can you please let me know if i forgot something
Is there paytabs default feature to save and retrieve card details ?

It should be enabled by the our technical support team, please send a request to enable the tokenization & recurring to the support team at<EMAIL_ADDRESS>

Hello ,
What I need to share with you.  I didn't get it.
we have multiple users in future who will use this feature. so how it can be managed ?
Thanks,
Jignya

From: Mohamed Adly @.>
Sent: 11 June 2021 18:16
To: paytabscom/paytabs-ios-library-sample @.>
Cc: Jignya Panchal @.>; Author @.>
Subject: Re: [paytabscom/paytabs-ios-library-sample] Store card information does not work (#60)
It should be enabled by the our technical support team, please send a request to enable the tokenization & recurring to the support team at<EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/paytabscom/paytabs-ios-library-sample/issues/60#issuecomment-859556657, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARZUXHLQVEISZQQXTVQIHKLTSIATXANCNFSM46QSHJXQ.

First you should store the transaction reference and token after the first normal payment using the card, after that you pass the token and transaction reference saved earlier into the configuration, this will make the payment without showing card input view again.

yes
I have done the same but is showing payment process error while doing payment with stored card.

From: Mohamed Adly @.>
Sent: 11 June 2021 18:25
To: paytabscom/paytabs-ios-library-sample @.>
Cc: Jignya Panchal @.>; Author @.>
Subject: Re: [paytabscom/paytabs-ios-library-sample] Store card information does not work (#60)
First you should store the transaction reference and token after the first normal payment using the card, after that you pass the token and transaction reference saved earlier into the configuration, this will make the payment without showing card input view again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/paytabscom/paytabs-ios-library-sample/issues/60#issuecomment-859561738, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARZUXHLS4YQBDWDCSII4IPTTSIBVHANCNFSM46QSHJXQ.

It should be activated by our support team to be able to do it, send an email and will activate tokenization and recurring for you.

Okay.
I got it
But I have concern regarding in real time, when customer will use this feature then it is not possible to send tokenization.
What is the resolution for it ? In that case how to activate.

From: Mohamed Adly @.>
Sent: 11 June 2021 18:37
To: paytabscom/paytabs-ios-library-sample @.>
Cc: Jignya Panchal @.>; Author @.>
Subject: Re: [paytabscom/paytabs-ios-library-sample] Store card information does not work (#60)
It should be activated by our support team to be able to do it, send an email and will activate tokenization and recurring for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/paytabscom/paytabs-ios-library-sample/issues/60#issuecomment-859569109, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARZUXHNA3GL6CD5G6ZFWHITTSIDA7ANCNFSM46QSHJXQ.

Every customer has a separate token for every card he pay with.

Okay.
Does Paytabs provide save card/ retrieve card  feature ?

From: Mohamed Adly @.>
Sent: 13 June 2021 14:44
To: paytabscom/paytabs-ios-library-sample @.>
Cc: Jignya Panchal @.>; Author @.>
Subject: Re: [paytabscom/paytabs-ios-library-sample] Store card information does not work (#60)
You are activating your account on PayTabs to have the tokenization feature enable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/paytabscom/paytabs-ios-library-sample/issues/60#issuecomment-860179361, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARZUXHNRM4YUAE6V3BUKU33TSRZHFANCNFSM46QSHJXQ.

We don't have an API to save cards or retrieve cards? the cards are automatically saved in our servers after each payment transaction.

Okay
Thanks for the support.
Last question :
From where I can enable tokenization feature ?.
In merchant site I can't see any setting for this option.

From: Mohamed Adly @.>
Sent: 15 June 2021 15:15
To: paytabscom/paytabs-ios-library-sample @.>
Cc: Jignya Panchal @.>; Author @.>
Subject: Re: [paytabscom/paytabs-ios-library-sample] Store card information does not work (#60)
We don't have an API to save cards or retrieve cards? the cards are automatically saved in our servers after each payment transaction.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/paytabscom/paytabs-ios-library-sample/issues/60#issuecomment-861354088, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARZUXHMAECPU5PAUUO3QUD3TS4OL7ANCNFSM46QSHJXQ.

Only the activation team can enable it, Please contact them on<EMAIL_ADDRESS>

Okay
Thanks

From: Mohamed Adly @.>
Sent: 15 June 2021 15:56
To: paytabscom/paytabs-ios-library-sample @.>
Cc: Jignya Panchal @.>; Author @.>
Subject: Re: [paytabscom/paytabs-ios-library-sample] Store card information does not work (#60)
Only the activation team can enable it, Please contact them on<EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/paytabscom/paytabs-ios-library-sample/issues/60#issuecomment-861383028, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARZUXHKPECLHOA5UGJ6N6E3TS4TF7ANCNFSM46QSHJXQ.


Undefined method `last_login_from_ip_address

I am attempting to use Sorcery in a Rails4 app. I am getting the following error: undefined method `last_login_from_ip_address. Any ideas? Thanks.

Thank you @skills4ever I just had this issue - I don't think it's fixed yet.


componnet and demo not work

Hi,
I tried to install your demo but was unable to install.
I did this (and also replaced first command by manually download from github)

git clone https://github.com/jkuri/ng2-datepicker.git --depth 1
npm install
npm start

install job exit with error.
I attach the log.
Best reagrds

npm-debug.txt
I see now that ".log" file was not attached.
renamed to .txt
Best regards

fixed now.


Add AutoCloseable

Implements AutoCloseable for the subsystems so the motors shut down properly. Closes #13

This isn't ready yet, the gyroscope doesn't have it


Consider supporting non cookie scenarios for SignInManager

Can you elaborate?

Actually looks like we already have a bug for this, see https://github.com/aspnet/Identity/issues/651


Remove ASCII control characters before reading the input

libxml2 removes them during serialization which means that
an input like Link
contains the control character during cleaning but
not after it's serialized back to a string.
This basically overcomes checks in lxml_html_clean
and the output might then be malicious even the input is not.

Cc: @encukou

I think this deserves a note in CHANGES.rst. Do you prefer compiling a list of changes when doing a release, or adding them right away?

Done. I think we're gonna do a new release right after this PR so it does not make any difference here.

Rebased and ready to go out. I'll wait for a day or two for possible comments from the reporter and the maintainer of lxml.

Release 0.2.0 is out.


after v1.4, could not load typescript version

Volar v1.2.0

After updating Volar v1.4.0

Duplicate of #2637


Incorrectly decoding query parameters

Exactly the same problem as #688 (which I can confirm was fixed in Kong 0.5.3), but this time for query parameters.
If I hit /foo?param=abc%7Cdef via Kong, the %7C (pipe character) gets decoded and so the URL is passed through to the backend as /foo?param=abc|def. This causes my backend server (Spray) to reject the request.

After a quick look through the Kong source, my guess is that percent-encoding needs to be performed in the encode_args function, because it is using as input the params that nginx has (unhelpfully) decoded: https://github.com/Mashape/kong/blob/873d8280b32258fa1dc0c97c8834ea2e826c539b/kong/tools/ngx_stub.lua#L54
Technically both the param names and values should be percent-encoded. (See RFC 3986 section 3.4. The query string is made up of pchar, which means it may be percent-encoded.)

The method you pointed out is only used when tests are running (ngx is monkey-patched). The method used is ngx.encode_args which is in ngx_lua itself.
Let me quickly explain what is going on here: since Kong needs to eventually modify some query parameters (like the request-transformation plugin), they are detached from the URI, processed by plugins, and re-attached to the URI at the very last moment before proxying the request.
The problem seems to be the result of several assumptions made by ngx_lua:

It seems that ngx.get_uri_args is performing percent-decoding and does not provide an option to retrieve raw query parameters. This can be a good thing when plugins need to modify the values, but forces us to re-encode the values once they are re-attached to the URI.
Except that ngx.encode_args does not percent-encode |, nor many of the reserved characters from RFC 3986.

There is an opened pull request which addresses this on ngx_lua: openresty/lua-nginx-module#542 which seems promising, but it is hanging.
local args = {foo = ",$@|`"} -- percent-decoded by ngx.get_uri_args()
-- current behavior
ngx.encode_args(plain) -- foo=,$@|`
-- expected behavior
ngx.encode_args(plain) -- foo=%2C%24%40%7C%60

For now, I see no choice but to iterate through the query values and percent-encode each one of them with another function, such as LuaSocket's url.escape.

#752 should solve that.

Thanks! Feels good to wake up in the morning and find that both my issues have been fixed overnight :)


unable to use the command flutter --version and flutter doctor

@shagufta117
This is not Flutter related issue but you are having issues on how tools are installed on your machine. See e.g. - https://apple.stackexchange.com/questions/321858/xcrun-missing-installing-xcode-command-line-tools-not-fixing-issue
Ketan

Duplicate of #29952.


Crop fails after rotating object

I'd like to crop an object along its major axis. However, if I run:
mesh_rotated = mesh.rotate_x(20).rotate_y(10).rotate_z(5)
mesh.crop(top=20).points()

returns an empty array.

Thanks for spotting the bug.
After:
pip install -U git+https://github.com/marcomusy/vedo.git

try eg:
from vedo import *
mesh = Mesh("https://vedo.embl.es/examples/data/panther.stl")
mesh.rotate_x(20).rotate_y(10).rotate_z(5)
mesh.crop(top=0.25)
show(mesh, axes=1)

note that top=... is expressed in fractional units.

Works great, thanks!
Do you know a simple way to keep the partial faces that end up being entirely cropped? i.e. if there's a face that isn't entirely cropped, can the vertex be shifted to the boundary so the face remains? I assume crop_by_plane would have the same affect as the example you have where there'd be holes around the panther's left legs.

Yes - it's called .cut_with_plane()
https://vedo.embl.es/autodocs/content/vedo/vedo/pointcloud.html#Points.cut_with_plane


all.bash fails for test "TestAnalyzeAnnotations" & "TestAnalyzeAnnotationsGC"

Repro instructions:
cd src/
./all.bash
Failing signature:
--- FAIL: TestAnalyzeAnnotations (0.00s)
annotations_test.go:101: failed to trace the program: gWaiting in makerunnable 36:[g:8 p:0 GoSched/17 [0 0 0] [] 1eb2a ->0] [waiting GoBlockSync:11f6f,,,]
--- FAIL: TestAnalyzeAnnotationGC (0.00s)
annotations_test.go:271: failed to trace the program: p already running 24, 59:[g:66 p:2 GoStart/14 [66 0 0] [] 15984 ->0]

Thanks for reporting this issue. I was seeing this issue on mainline as well (https://github.com/golang/go/issues/28702). I will cherry-pick the fix (https://github.com/golang/go/commit/6d5caf38e37bf9aeba3291f1f0b0081f934b1187) that was pushed to mainline and will add it to our master as well.

Fixed in #8


📖 Add CONTRIBUTING guide

kind/documentation
What this PR does / why we need it:
Adds CONTRIBUTING.md to the repo and doctoc script to be able to run the tool locally and via Makefile target. Also pass docs/adr to the doctor script and add changes after run.
Checklist:

[x] squashed commits into logical changes
[x] includes documentation
[ ] adds unit tests
[ ] adds or updates e2e tests

@Danil-Grigorev @richardcase thanks for reviews, PTAL once again.


failed to import to gradle

Hello. I can import this library. Here is image:
https://i.imgsafe.org/1a3603abc0.png

Hey.
Are you sure that you are not in offline mode for gradle?

Hi. Online mode. I have to import by download and import to project.


Receive with Bolt Card and various improvements

Do not send this update to Apple yet. Only when https://github.com/DFXswiss/btc-wallet/pull/91 has also been added.


Clarifications on 19 human body measurement

Hello there,
Thanks for open-sourcing this research work.
Can I get some more clarifications for all the 19 measurements that you had given as inputs?
what does natural waist, maximum hip, natural waist rise mean and gluteal hip?
Thank you.

We manually labeled the measurements according to a human anthropometric measurements guide from the Body Lab as described in the paper.
However, they seem remove the guide from their home page.
Sorry about that.


Feature/add react

eslint-config-airbnb-base is removed since eslint-config-airbnb contains everythig as eslint-config-airbnb-base + react.

As discussed in person, index.js in root should not include React rules as well, so leave it's extends property to airbnb-base.


347 analytics

Fixes #347
This adds google analytics for screen tracking. Currently missing the enhanced ecommerce tracking event that is being sent in Holtzman. I can port that over, but that data may not be 100% consistent so I want to see how y'all are using it first, if at all.
Also added Fabric keys for Android, which expo appears to support, but has not yet tested this. @IsaacHardy figured you could take care of actually getting these keys in - https://github.com/NewSpring/Apollos/compare/347-analytics?expand=1#diff-55d8795465c20583bf4d30cee70e5ee2R39
Creator

[ ] Build relevant tests if any apply
[ ] Ensure there are no new warnings
[ ] Upload an animated GIF showcasing the solution (if it applies)
[ ] Set a relevant reviewer

Reviewer

[ ] Run test and make sure all tests pass
[ ] Review function and form on iOS
[ ] Review function and form on Android
[ ] Review function and form on Web
[ ] Review new test logic

Seems the configuration of GA in mobile apps requires Firebase. Going to block this PR until we decided on a analytic solution. CC: @FrankGrand

Closing in favor of #485


Search by tags doesn't seem to have an effect

Hello, I'm trying to understand how search works, following https://github.com/MarshalX/atproto/blob/main/packages/atproto_client/models/app/bsky/feed/search_posts.py
r = client.app.bsky.feed.search_posts(
    params=dict(
        q="py",
        # tag=["python"],  # <--- this does not do anything
        lang="en",
        sort="top",
        limit=5
    )
)

Would you expect this to work? If not, what's the way to get roughly the results you would find here? https://bsky.app/hashtag/python

Hello, the link that you provided uses app.bsky.feed.searchPosts?q=#python&limit=25&sort=top

Thanks for the dev tools trick!
I tried other documented Params such as lang, since, until, and they work as expected.  Was tags documented but not implemented?

My pleasure! Well, it is implemented in SDK for sure, but looks like not implemented on server side. I am not related to server side part. You can try to dug into server side code of search in case you need the answer :) Backend code is here https://github.com/bluesky-social/atproto


idk what is this

Describe the bug
i just logged into the discord today and i see this bug
Screenshots

Infomation (please complete the following information):
Discord channel: ((Stable))
OS: ((Windows))
Mod: ((BetterDiscord))
Discord language: ((Vietnamese))
Additional context
P/s: i editted some stuff in the .css file but today i got this bug?

Same issue here.
This still happens with the original source

It was a massive Discord update which should hopefully be fixed.


Authentication for backend and frontend

@Piparkuucinsh  add token checks for admin and upload requests

nvm

**Max commit 🔥 **


Resource not accessible by integration when creating PR with fork in Codespaces

On Codespaces:
I'm trying to create a PR from rzhao271/build-tools:rzhao271/winbat to electron/build-tools:master but I see this:

Duplicate of https://github.com/github/codespaces/issues/2243


Fix triton + torchscript guidelines to be executable

Signed-off-by: Jinwoo Park (Curt<EMAIL_ADDRESS>What this PR does / why we need it:

Fixed some misleading hyperlinks
Fixed wrong key value (image.json)
Fixed curl commands

Release note:

Fix Triton with TorchScript guidelines to be executable

@Curt-Park We actually have the exactly same tutorial on website,  what do you think just delete this and link to the website here?

@Curt-Park We actually have the exactly same tutorial on website,  what do you think just delete this and link to the website here?

Okay. I will link the website and create a PR on the website repository.

@yuzisun I just added a link you mentioned and didn't change anything else. I think the better way to fix is gathering all of the information for this tutorial in the same place (such as KServe website). However, the related codes have deleted few months ago. (https://github.com/kserve/website/commit/57c544662b9761142f171f64d4de8557675a444b)
Please let me know if you have any idea for this.

@yuzisun I just added a link you mentioned in README.md and didn't change anything else. I think the better way to fix is gathering all of the information for this tutorial in the same place (such as KServe website). However, the related codes have deleted months ago (kserve/website@57c5446) by you. I assume that there is any reason for what you've done.
Please let me know if you have any idea for this.

@Curt-Park yes the idea is to keep the doc in the website repo and code examples in the kserve repo because some of the examples are used by the e2e tests and it is also easier for contributor to create a PR for new feature with code examples.

thanks @Curt-Park !!
/lgtm
/approve


Memory leak log

when i chat some times, will occur:
(node:21201) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 end listeners added to [ReadStream]. Use emitter.setMaxListeners() to increase limit
(Use `node --trace-warnings ...` to show where the warning was created)

Hello, we tried to solve the issue.
This is what we did:
Modified the chat command to use a single stream and manage listeners properly to prevent the MaxListenersExceededWarning.
You can review changes in this commit: https://github.com/mark-addams001881/BuilderIO-ai-shell-134/commit/fdd41a9f72f9759899ee83c4e816aa5d44557d12.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

Latta AI seeks to solve problems in open source projects as part of its mission to support developers around the world. Learn more about our mission at https://latta.ai/ourmission . If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Update PR template with a link to the docs

As someone who contributes to the docs there's too many features that get added with no documentation. I understand that we can't require people to document their features, but we should at least encourage it.

See also #42431 which is somewhat more demanding too.
On the principle I agree that contributors adding features should also add documentation for it, but there are two considerations that make me somewhat reluctant to make it mandatory (but on the other hand a suggestion as done here might be a good first step):

For PRs which haven't been approved for merging yet (e.g. they don't come from an already-approved proposal where the suggested implementation was discussed), I wouldn't want to encourage contributors to write documentation for it if it's not going to be merged. On the other hand, I agree that if/when a PR is merged, the author should be asked to write documentation for it, or at least provide relevant details in an issue on godot-docs for tutorial writers to use as reference (as not all contributors are necessarily good documentation writers).

For some WIP (like all the work @reduz is doing on rendering, which changes on a weekly basis), we intentionally delay writing the relevant documentation, as the API hasn't been stabilized yet. reduz plans to write documentation for all the features he added before 4.0 goes in alpha, which would mean that the API is close to what we want it to be like in 4.0 stable. (He might forget some stuff though, so I would suggest that we open issues for each new feature that hasn't been documented yet, to make sure it gets documented before 4.0, and ideally by the feature author, or at least with the information they'd provide.)

All that to say, I think it's important that we make the need for documentation clear, but I don't know if it should be in the PR template, or if it's something that PR reviewers should point out if/when a PR is receiving positive reviews and likely to be merged.

See also #42431 which is somewhat more demanding too.
On the principle I agree that contributors adding features should also add documentation for it, but there are two considerations that make me somewhat reluctant to make it mandatory (but on the other hand a suggestion as done here might be a good first step):

For PRs which haven't been approved for merging yet (e.g. they don't come from an already-approved proposal where the suggested implementation was discussed), I wouldn't want to encourage contributors to write documentation for it if it's not going to be merged. On the other hand, I agree that if/when a PR is merged, the author should be asked to write documentation for it, or at least provide relevant details in an issue on godot-docs for tutorial writers to use as reference (as not all contributors are necessarily good documentation writers).

For some WIP (like all the work @reduz is doing on rendering, which changes on a weekly basis), we intentionally delay writing the relevant documentation, as the API hasn't been stabilized yet. reduz plans to write documentation for all the features he added before 4.0 goes in alpha, which would mean that the API is close to what we want it to be like in 4.0 stable. (He might forget some stuff though, so I would suggest that we open issues for each new feature that hasn't been documented yet, to make sure it gets documented before 4.0, and ideally by the feature author, or at least with the information they'd provide.)

All that to say, I think it's important that we make the need for documentation clear, but I don't know if it should be in the PR template, or if it's something that PR reviewers should point out if/when a PR is receiving positive reviews and likely to be merged.


Issues with encoding in windows batch

Hi,
I was trying to use ogr2osm in a windows batch but had a lot of encoding problems, because the batch always created ANSI-encoded files, but my workflow needs utf-8 encoded files. I managed to solve my issue by changing the following line:
self.f = open(self.filename, 'w', buffering = -1)
to
self.f = open(self.filename, 'w', buffering = -1, encoding="utf-8")
there is already a parameter called "encoding" but it seems it is only used for the source file, could we extend this "encoding" to be used in the destination file as well? or could we introduce another parameter for that? what are your thoughts? or do you have a tip how I can force the windows batch to output utf-8 without changing ogr2osm?
thanks for this awesome tool =)

Thanks for your bug report. This issue looks like a duplicate of pnorman/ogr2osm#15 but your solution is different and you have found a testcase where the current method has issues.
Some observations:

The encoding parameter of ogr2osm only specifies the encoding of the input file, not the encoding of the output file
The documentation of the python open() function specifies that the default encoding is used when the encoding parameter is omitted or None. This is platform dependent, I can't test it on Windows but at least for Linux it is UTF-8.
Although a clear suggestion is present, there is no strict obligation for an osm file to be encoded in UTF-8 on the OSM wiki page
According to the W3C recommendation for XML the expected encoding is UTF-8 if neither a byte order mark nor an encoding is specified, as is currently the case for ogr2osm

Given the last observation ogr2osm is supposed to output UTF-8 at the moment, eventually translating from the input file encoding if necessary. To obtain consistent behaviour across different operating systems it is as such necessary to  pass encoding='utf-8' as you suggested. I would also explicitly specify the encoding in the header then, ie <?xml version="1.0" encoding="utf-8"?>.
I can confirm the testcases still pass on Linux with your suggested modification. Can you verify if the testcases pass under Windows as well?

Thanks for the fast answer!

As far as I know both Linux and Mac use UTF-8 as their default encoding and Windows uses ANSI / Windows-1252 (at least in the german version of windows).
It seems some OSM-tools do write UTF-8 in the header, here is an example of Overpass:

<?xml version="1.0" encoding="UTF-8"?> <osm version="0.6" generator="Overpass API <IP_ADDRESS> 76e5016d"> <note>The data included in this document is from www.openstreetmap.org. The data is made available under ODbL.</note> <meta osm_base="2021-06-09T08:10:43Z"/>
After making these changes everything runs smooth in the batch.

Ok. I am not sure if the cram tests can be run as is under Windows, but can you try to convert at least test/shapefiles/japanese.shp and confirm if the formatted result matches test/japanese.xml?
In the test script the output is formatted using xmllint before comparing:
ogr2osm --encoding shift_jis --gis-order -f test/shapefiles/japanese.shp
xmllint --format japanese.osm > japanese.xml

Meanwhile I managed to test the modification in Windows, the test is conclusive. The proposed changes have been merged into master.
Thanks @Meibes for your investigation.


fix(frontend): use a new key for identity in localStoage

Use a different key in local storage so if users have an identity from devconnect it will be ignored.

Use a different key in local storage so if users have an identity from devconnect it will be ignored.

Is that for migrating users from DevConnect to v1.1?

No this was to make sure if they had an old identity in localStorage it would get ignored for the devcon version.

Won't fix due to sunsetting the project


Logged out after restart, /auth HTTP 429

Describe the bug
My pruning "targetSize" was set to 30GB (on a 40GB ssd) and I was running on 93% of device space, so I decided to set it to 25GB and restart hornet.
After restarting hornet, I got logged out from the dashboard and cannot login, since /auth gives me HTTP 429 all the time.
Pruning config:
 "pruning": {     "milestones": {       "enabled": false,       "maxMilestonesToKeep": 60480     },     "size": {       "enabled": true,       "targetSize": "25GB",       "thresholdPercentage": 10.0,       "cooldownTime": "5m"     },     "pruneReceipts": false   },
Environment information:
CPX11
2VCPU
2 GB RAM
40 GB DISK LOKAL

HORNET version 1.0.2

Additional context
Log since restarting: https://gist.github.com/ChristopherWalz/5bb661b6f569d0dfc7bfe072568d9bca

After some time the logins appears to work again, but then after refreshing the page I will get a HTTP 429 again.

Also according to the log the pruning seems to work, my hornet db is still getting bigger. After restarting the server with 25GB treshold, the database was 28.5 GB, right now its 29.13 GB.
How does the treshold work and if the treshold percentage is 10%, shouldn't it stop after 27.5 GB max?

The /auth endpoint is protected by rate limiting. This way no one can brute force the password.
Do you have multiple dashboard tabs open? Why do you refresh the page? This would always trigger the rate limiting.
    Rate:      rate.Limit(1 / 300.0), // 1 request every 5 minutes
    Burst:     10,                    // additional burst of 10 requests
    ExpiresIn: 5 * time.Minute,

Pruning starts at 25GB and tries to prune and guesses the amount of milestones to prune, to achieve 22,5GB (10%) less disc space. But this is only a target, and not hard capped. The database engine will itself do some magic, which we can't control. If the database engine decides to do a compaction, the size will increase because new files are created during compaction, data is copied over and then finally the old files are deleted and the space is freed. Which database engine do you use? We recommend rocksdb.

Do you have multiple dashboard tabs open? Why do you refresh the page? This would always trigger the rate limiting.
No, I only had 1 dashboard opened. I refreshed the page once after restarting the node and was logged out and couldn't log in again. Right now it seems to work again.

I'm using the default db engine, which seems to be rocksdb. I will keep an eye on the size used, but it already seems to decrease.
The rate limiting seems a little bit too strict for me tbh. Also it's weird, that the rate limiting is enabled when logged in as an admin. One logged in, why would there be a rate limiting? Why not just limit the login route?

I already increased the limits locally. Will push it later.
Only the /auth route is limited. But if you refresh the dashboard page, the token in your browser is lost and the /auth route is called again. But that's something in the dashboard code, not in the hornet code. Refreshing isn't the way to go ;)
Just click on the home symbol instead.


Two minor pie fixes for #2148 and #2102

Fixes: #2148

I signed it!

Woo!  Thanks for reducing entropy in crouton!
I'm a couple PRs behind in merging, but code-wise this looks perfect.

Merged in record time


Use lib-common for pvc

Looks like mariadb pvcs are not cleaned up when the mariadb instances are deleted. Let's use the lib-common methods to CreateOrPatch pvc. Whether to retain the pv when pvc are deleted would be decided by the StorageClass.

I discussed this with @dciabrin a while ago and the conclusion was that this is an expected behavior to avoid wiping out data in case the instance is deleted unintentionally. but I'd like Damien to double-check my understanding.

I discussed this with @dciabrin a while ago and the conclusion was that this is an expected behavior to avoid wiping out data in case the instance is deleted unintentionally.

Not data would be wiped out if a storageClass with reclaimPolicy: Retain is used. If we want to be different and want to ensure that we keep the data irrespective of the storageClass used, we've to make it consistent across all operators for example ovndb is deleted[1] when service is deleted.
[1] https://github.com/openstack-k8s-operators/ovn-operator/blob/main/pkg/ovndbcluster/statefulset.go#L163

Technically ovn db can be rebuilt from neutron db(though we'd lost all data inserted by octavia db), but I agree this is something we should set via reclaimPolicy in StorageClass instead of leaving pvc.

/test mariadb-operator-build-deploy-kuttl

/test mariadb-operator-build-deploy-kuttl

It sounds like there is maybe a concern here in using the shared lib-common routines in that we really want to guarantee database's aren't accidentally deleted

Is this is a concern only for mariadb? Isn't reclaimPolicy the k8s way of not accidentally deleting disk? We need to consistent across services if we want to deviate from that IMO.


Add index.html page to project

Needs to show up at http://mapzen.com/common/, can just say “hello world” for now.

Thanks @sleepylemur! I’m going to reopen this, because it needs to show up at https://mapzen.com/common/ and not https://mapzen.com/common/styleguide/.
@hanbyul-here, @louh, or @souperneon: this one’s for you.

Thanks for the clarification about /common/styleguide/, @sleepylemur.