ERROR: Service 'input-internal' failed to build:

[root@akiselev-centos7 open-nti]# make start
Use docker compose file: docker-compose.yml
IMAGE_TAG=latest docker-compose -f docker-compose.yml up -d
Building input-internal
Step 1 : FROM juniper/pyez:2.0.1
Pulling repository docker.io/juniper/pyez
ERROR: Service 'input-internal' failed to build: Error while pulling image: Get https://index.docker.io/v1/repositories/juniper/pyez/images: x509: certificate is valid for approposhq.com, *.approposhq.com, not index.docker.io
make: *** [start] Error 1
[root@akiselev-centos7 open-nti]# make update
OpenNTI - Update the files from Github
git pull
Already up-to-date.
OpenNTI - Update the containers from Docker Hub
docker pull juniper/open-nti:latest
Pulling repository docker.io/juniper/open-nti
Error while pulling image: Get https://index.docker.io/v1/repositories/juniper/open-nti/images: x509: certificate is valid for approposhq.com, *.approposhq.com, not index.docker.io
make: *** [update] Error 1
[root@akiselev-centos7 open-nti]#
Please advise on this issue

Hi @cduranm
It seems a connectivity issue from docker daemon to the internet.
In my lab (ubuntu VM) sometimes after suspend/start up my VM, the docker daemon lose it connection to internet, then after a daemon restart its solved.
Could you please try to restart docker?
Regards


Picotorrent crashes all the time

Almost every time I add torrent, PicoTorrent crashes. I press "Send crash report".
Did the author receive them?
Windows 7 x64 Ultimate

Hi! Sorry you're seeing crashes. Yes, we receive the crash dumps and are continuously using them to improve PicoTorrent :) Can you attach or send me the torrent file which makes the app crash<EMAIL_ADDRESS>

This torrent crashes pico whenever you try to select it inside the window.

Torrent file for tests: CrashTorrent.zip

Thanks! I’m on vacation right now but I’ll take a look when I get home.

Thanks! I’m on vacation right now but I’ll take a look when I get home.

@vktr Hey! crashes still here. How we can fix it?


372: Implement Eq and Hash for socket2::SockAddr

Closes: #372.
Cannot derive Eq and Hash for socket2::SockAddr because sockaddr_storage does not have both traits derived/implemented. sockaddr_storage is defined within s_no_extra_traits, my assumption it is expected to stay this way and I should not derive/implement Eq and Hash for sockaddr_storage.
TODO:

[x] add test cases
[ ] add benchmarks for socket2::SockAddr::as_socket + std::net::SocketAddr::hash vs socket2::SockAddr::hash (this implementation)
[ ] target this at master, v0.4 is only accepting backports.

Closing this PR, because it was base off of v0.4.x branch by mistake. See #374, which is based off of master.


The exe published is unclear as to how it was compiled

Thanks @eballai for pushing the code to the repository :)
However could we please also have how the exe was compiled published as a change to the readme or as a script in the repository?
I ask for this because the code you pushed up has a bat file to run the python script, and no clear way for how this 300MB executable file was compiled. I am not wanting to draw conclusions if I don't actually know but this exe has me a bit concerned considering it's size in comparison to the size of the code and the responses in issue #1 saying it's not using the tool some of us thought it was using.
I did ask in #2 but that issue was closed by the reporter and so opened my own issue to track this question I have.

Is the concern for any iterations of that exe floating around and not knowing what it does?
Because as far as I can tell the exe isn't mentioned anywhere in the repo now.

My attempts to compile this with the invisible-watermark library and OpenCV using PyInstaller produced a file that's about 100 megabytes short of the provided one. I'm still not completely clear on what libraries or options for those libraries we're actually meant to use. Could you clarify the exact libraries and versions used, in addition to how they were compiled?

Yeah, we really need everything regarding this exe to be explained.

Hi @AUSBird  Sorry for coming back to you so late. The exe was made with a software called pyinstaller. When you install it you can find many options there for building exe files.
About the size: This is a standalone executable file, which means if you want to run this application you don’t need to install anything at all on your machine. Not even python is required. If you run this application on a computer, where there is nothing installed, it works. That’s why it's around 300 MB because it contains all the packages and python interpreter to execute the code.
However, there is another method to make the watermarking process, if the exe is still causing concerns. We just updated the instructions on how to do it, it will require installing python and running the script from the terminal.

Ok, could you provide the options you used please? And document it a bit better? If you can throw things into the readme and stuff that would be amazing since being able to reproduce this would make it feel less like a suspicious piece of malware.

Also that said my apologies if I come off as attacking, aggressive or demanding or anything like that in these issues. Was put on edge with this when someone showed it to me and wanted to get to the bottom of how safe it was.

So far as I can tell: a venv wasn't used for the project, so a bunch of unrelated libraries that happened to be installed got thrown inside, even though they weren't used by the program.
Looks like the code is on the github now, though!

Hi eballai,
to me it just seems like that the publication of this was a bit rushed.
Here's a general guideline on how to make it right and as trustworthy as possible:

Make the sources public from the start
Use automatism to create a standalone executable
Make the automatism scripts available in the source so everyone knows how you create the executable

Anyway thanks for having made the sources public, so if someone doubts the legitimacy of the executable, they're able run the script normally or make their own one. That already helps a lot.
Next time please consider the three points above so nobody is questioning your intentions.
Sorry if I came over as impolite.
Regards,
Kyo

Hi @KyodaiKen, Thanks! The initial plan was to release only the python script, but I thought, a regular people would find it difficult to run it through terminal, so an app would be easier to use. This was a mistake from my end. But we hope that with the updated instructions people will be able to make the process the way they want, without feeling trapped. We are also thinking to move the conversation to Discord, where we can share ideas or suggestions, if there's interest about it.

I totally understand that. Thank you!


createGroup() of Instagram.Direct is set to private

Discussed in https://github.com/ErrorxCode/EasyInsta/discussions/40

Originally posted by ErrorxCode April  5, 2024
Here, you can talk about anything related to the project. You can ask questions, make announcements, raise queries, highlight problems and talk about the changes in the code or API. If you have any doubt, you can ask it here before creating an issue

Contribution for testing and making that method public is appreciated!


worker-src missing

There is a CSP for worker-src that seems to be missing from ContentSecurityPolicyHeaderBuilder.
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy/worker-src
Is it omitted by design or a newer feature than the class?

worker-src was added to CSP after this library was written and as it isn't maintained anymore it was never added.
If you are looking for a PHP library to create CSP headers you might want to take a look at https://github.com/paragonie/csp-builder.


[BUG] Logger doesn't work in dev

Brief description:
I believe the logger doesn't work properly, the last logged message is at 20:00. We could alao add dates to teh logger

No new logs are getting stored when you make failing requests?

No new logs are getting stored when you make failing requests?

Yes, I believe so. At least that is what the logfile in dev showed

My bad


Latest pod version does not have Swift 4.2 updates despite master branch being updated in #30

I noticed the podspec & swift syntax were updated to swift 4.2 however the latest version available via pods does not reflect these changes.

@alittle433 I just released version 0.4.1 which support Swift 5.0 and fix some critical bugs.


Stacks in Data Structures ( using C++)

Stacks in Data Structures ( using C++)
Introduction paragraph (2-3 paragraphs):
Stack is a simple data structure used for storing data (similar to a linked list). Its most important feature is the way of inserting and deleting data, which happens in O(1) time complexity, making it so efficient. It stores data like a pile of plates in a cafeteria the plates are added to the stack as they are cleaned and placed on the top. When a plate, is required it is taken from the top of the stack. It is called LIFO ( last in first out) principle that stack follows.
Key takeaways:

Will cover everything about stacks
how stacks are used
stacks ADT
Applications
Implementation
Stacks STL ( will help in coding questions)

Good afternoon and thank you for submitting your topic suggestion.
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.

Thank you for submitting your topic. @anubhavb11
After some careful consideration, it struck us that this topic may be a bit over-saturated throughout other blog sites and official documentation.
We believe this is the best way for students to build a great portfolio (for potential employers) is by building what does not exist and what can provide the most value.
Please feel free to suggest an alternate topic to explore. 🚀

I already posted 2 articles one about hashmaps and the other is about graphs
I wanna cover all types of data structures topic by topic but my issue got closed this time
I not only explain what's in the documentation  but I explain how one can code the implementation of stacks and all other data structures I learned it by doing some courses but now wanna explain others by writing articles,
Please look into the matter we wanna cover all the data structures
Linked list
Stacks
Queues
Tree
Graphs (done)
Hashmap (done)
Heap
Please look into the matter I really wanna write on all of them at section- engineering - education
@ahmadmardeni1  @hectorkambow

Replied to your message on Slack.👀 @anubhavb11


Talk to Bing AI (Copilot) in Powertoys Run

Description of the new feature / enhancement
Option to instantly ask copilot something easily.
Scenario when this would be used?
To be able to talk to copilot quickly.
Supporting information
No response

Is there an API to interact with Copilot?

I don't think so

Thank you for bringing up this issue. I would like to know if this feature, or any associated plugins, have been created recently. So, I can call windows copilot in powertoys run.

It's fine, I found Win+C shortcut which allows me to use copilot

Can we reopen as they have moved away form the win+c option!


Broker Id should be updated when streamed

Related to #96
@mcollina @nguyenthenguyen Please verify

Coverage increased (+0.004%) to 97.42% when pulling 3d495367126d2f70795ff69bf0df503211c2fddd on fixBrokerId into 3f8d6a7438545527a0efa39103a9ccb23521d376 on master.

I have tested but not fix will problems. I have 1 aedes instance:
Step 1: aedes start with id1: HJmwY81Te
Step 2: All client connect to server id1 HJmwY81Te
Step 3: Client subcribe  will topic testdevice/status to server id1 HJmwY81Te
Step 4: aedes restart with id2: XmmkY21xj for some reason, ex: server crash ...
Step 5: All client reconnect to server id2 XmmkY21xj
Step 6: Client subcribe  will topic testdevice/status to server id2 XmmkY21xj
Step 7: server id2  XmmkY21xj start clean will https://github.com/mcollina/aedes/blob/master/aedes.js#L80
Step 8: All will topic get from persistence with old id1 HJmwY81Te process by https://github.com/mcollina/aedes/blob/master/aedes.js#L92
Step 9: needsPublishing function set to true by https://github.com/mcollina/aedes/blob/master/aedes.js#L94
Step 10: server id2 XmmkY21xj publish will https://github.com/mcollina/aedes/blob/master/aedes.js#L104
Step 11: server id2 not delete old will by https://github.com/mcollina/aedes/blob/master/aedes.js#L109
because that.persistence.delWill will use id2 not id1
-> Bug: When server with id2 start _clearWillInterval  will publish all will topic of id1 but all client is online not offline

can you send a PR with a broken unit test that reproduces this error? Looking at the code, it seems correct, as the following sentence is not correct:

because that.persistence.delWill will use id2 not id1

it uses the client id, not the broker id.

@mcollina Thanks!
But now, I'm not find a solution for this
that.persistence.delWill use broker id and client id? Ex: redis persistence
https://github.com/mcollina/aedes-persistence-redis/blob/master/persistence.js#L510

@nguyenthenguyen  @mcollina  The old will is not being deleted. That's one issue.
@nguyenthenguyen What did you mean by  will publish all will topic of id1 but all client is online not offline. I didn't quite understand

@nguyenthenguyen I have reverted this change, as it was probably not correct (my bad). Can you please upload a script to reproduce the issue you are facing using MQTT.js or just mqtt-connection?

@mcollina @GavinDmello Thanks!
Step 1: Start server.js
Step 2: start client1.js
Step 3: start client2.js
Step 4: stop server.js
Step 5: start server.js
Step 6: See client2.js log

server.js
var redisPersistence = require('aedes-persistence-redis');
var aedes = require('aedes')({
    persistence: redisPersistence({
	    "port": 6379,
		"host": "<IP_ADDRESS>",
		"family": 4,
		"password": "",
		"db": 1
    }),
    heartbeatInterval: 1000 // more fast to test
});

var mqttServer = require('net').createServer(aedes.handle);

var mqttPort = 1883
mqttServer.listen(mqttPort,"<IP_ADDRESS>", function () {
  console.log('mqtt server listening on port', mqttPort)
});

client1.js
var mqtt = require('mqtt')
var client  = mqtt.connect('mqtt://localhost:1883',{
    will:{
        topic:"client1/status",
        payload: "client1 offline",
        qos: 0,
        retain: true
    }
})

client.on('connect', function () {
    console.log("Client 1 connected")
    client.publish('client1/status', 'client1 online')
})

client.on('message', function (topic, message) {
  console.log(message.toString())
})

client2.js
var mqtt = require('mqtt')
var client  = mqtt.connect('mqtt://localhost:1883',{
    will:{
        topic:"client2/status",
        payload: "client2 offline",
        qos: 0,
        retain: true
    }
})

client.on('connect', function () {
    console.log("Client 2 connected")
    client.subscribe('client1/status')
})

client.on('message', function (topic, message) {
  console.log(message.toString())
})

@nguyenthenguyen  Wills aren't getting deleted because the brokerId being old, so you're getting it on every heartbeat.
@mcollina Can we pass the brokerId to the delWill function ?

If brokerId is not the same, del the old will and put the new one and then publish ?

we can just do putWill, as that should overwrite the previous value, then publish, then delete.
However, we might have to fix some of the persistences, i.e. pass in the brokerId as you said.

@mcollina Yea. We'll have to pass those brokerIds to delWill.
@nguyenthenguyen Sorry about this. Didn't quite understand the issue at first.


stepDefinitions issue

Current behavior
I have a before() that has cy.login()
The login is run for every feature when only one feature is selected
Desired behavior
before() should only run for the selected feature
Test code to reproduce
https://github.com/aomaryus/stepDefintionsIssue
Versions

Cypress version: 10.3
Preprocessor version: 11.3.1
Node version: v12.22.7

Checklist

[ yes ] I've read the FAQ.
[ yes ] I've read Instructions for logging issues.
[ yes ] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

What is the purpose of cypress/e2e/common.feature in the example?

I removed it. Was there by mistake. e2e/common should only hold common code step definitions

@badeball  I am facing the same issue while executing a specific feature.  All Hooks (Before,beforeEach) are executed as the first step which is nowhere related to the selected feature file.

This patch fixes it.
diff --git a/cypress-cucumber-preprocessor.config.js b/cypress-cucumber-preprocessor.config.js
index 20c548d..c00dd4e 100644
--- a/cypress-cucumber-preprocessor.config.js
+++ b/cypress-cucumber-preprocessor.config.js
@@ -1,3 +1,3 @@
 module.exports = {
-  stepDefinitions: ['[filepath]/*.ts}', 'cypress/e2e/**/*.ts']
+  stepDefinitions: ['[filepart]/!(cypress.config).ts', 'cypress/e2e/common/**/*.ts']
 };

@JomyGeo, you've misconfigured stepDefinitions to include all step no matter what is selected, thus including every hook.

Hi
I tried the fix above and still see the issue
λ cat cypress-cucumber-preprocessor.config.js
module.exports = {
  stepDefinitions: ['[filepart]/!(cypress.config).ts', 'cypress/e2e/common/*.ts']
};

Also I think his path:
cypress/e2e/common/*.ts
is not sufficient - as you can see, the are definitions in cypress/e2e/**
I am not sure what to do. I have a heavily nested tree structure and it appears I have to figure out correct step Definitions path. Appreciate any help

Well, in management.feature you're referencing a step definition in public.ts, so what do you really want here?

You're also misunderstanding entirely the purpose [filepath] and [filepart] in the search pattern. They do something, IE. cypress/e2e/common/*.ts isn't the only pattern resolving to step definitions.

Well, in management.feature you're referencing a step definition in public.ts, so what do you really want here?

This means that the management.feature needs a step that is already defined in parent. This is not new stuff. It is also possible to have another directory inside management that uses steps from both management and  public.
We cannot have all steps define in common.

You're also misunderstanding entirely the purpose [filepath] and [filepart] in the search pattern. They do something, IE. cypress/e2e/common/*.ts isn't the only pattern resolving to step definitions.

Help me understand what you mean. What else would the pattern above include from my sample project.
I doubt if these are included in cypress/e2e/common/*.ts
cypress/e2e/public/**/*.ts  cypress/e2e/private/**/*.ts cypress/e2e/network/**/*.ts
I checked with colleagues and they are having the same issues and unable to upgrade.
What if you tweak my project and test it. I will share your insight with others in my org.
Lastly, I tried DEBUG=cypress:* to see if I can find an expansion for the stepDefinitions but there was none. Is there a way to see it?
Thank you.

Lastly, I tried DEBUG=cypress:* to see if I can find an expansion for the stepDefinitions but there was none. Is there a way to see it?

There's a debug statement mentioned in "Instructions for logging issues", linked in your original post.

This means that the management.feature needs a step that is already defined in parent. This is not new stuff. It is also possible to have another directory inside management that uses steps from both management and public.

That's fine, but if you in management.feature want to have available public's steps, then you also get public's before hooks.

Hi
I needed to recreate the issue. Other folks could not find it.
I have removed the reference to public step but the issue is still there.
Thank you.


Unable to debug Swift on Windows

OS: Windows 11
VSCode version: 1.61.2
CodeLLDB version: 1.6.8
Compiler: Swift 5.5
I am unable to debug any Swift applications on Windows. I keep getting the following error after my app finishes building:
Fatal Python error: initfsencoding: unable to load the file system codec
ModuleNotFoundError: No module named 'encodings'

So far, I've:

set lldb.library to the liblldb.dll that comes with Swift
set PYTHONHOME and PYTHONPATH environment variables
set the same variables in my lldb.adapterEnv

but with no success. On macOS and Linux, setting lldb.library was enough to get this working.

Verbose log
 configuration: {
  type: 'lldb',
  request: 'launch',
  name: 'Debug',
  program: '${workspaceFolder}/.build/debug/Squares.exe',
  args: [],
  cwd: '${workspaceFolder}',
  preLaunchTask: 'swift build',
  __configurationTarget: 5,
  noDebug: true,
  relativePathBase: 'c:\\Users\\steve\\Projects\\Squares'
}
liblldb: C:\Library\Developer\Toolchains\unknown-Asserts-development.xctoolchain\usr\bin\liblldb.dll
environment: {
  PYTHONHOME: 'C:\\Users\\steve\\AppData\\Local\\Programs\\Python\\Python37',
  PYTHONPATH: 'C:\\Users\\steve\\AppData\\Local\\Programs\\Python\\Python37'
}
params: { evaluateForHovers: true, commandCompletions: true }
Listening on port 52531
[2021-10-31T19:17:20.921Z DEBUG codelldb] New debug session
Fatal Python error: initfsencoding: unable to load the file system codec
ModuleNotFoundError: No module named 'encodings'
Current thread 0x000014b0 (most recent call first):
Debug adapter exit code=3221226505, signal=null.

Looks like Swft's LLDB is broken in some way.   Can you debug with CLI lldb?

I am unfamiliar with the CLI but if I try to follow the example on swift.org, I also get stuck. I can launch the executable in LLDB, but cannot set any breakpoints.  This seems to be a known issue (https://bugs.swift.org/browse/SR-14243)

I got a same problem, cannot set any breakpoints, vscode showed an hollow circle

I've noticed that when compiling in release mode, I can set breakpoints. However, without the Swift-specific LLDB, the variable names are mangled.

Another Swift lldb and Python issue. Swift lldb Python integration on Windows basically doesn't work in swift 5.6 and earlier (unlike ubuntu where it has been broken since 5.6). This is resolved in the latest 5.7 nightly.


Example for teardown_appcontext in the docs out of sync with actual codebase

The documentation shows and example of a teardown function executed at the end of the app context https://flask.palletsprojects.com/en/1.1.x/appcontext/#storing-data.
The signature of the function decorated teardown function is teardown_X() but inspecting the codebase for Flask 1.1.x the function's signature must be teardown_X(exception: Exception) https://github.com/pallets/flask/blob/1.1.1/tests/test_appctx.py#L56.
I reckon that the documentation is out of sync.
Environment

Python version: 3.7
Flask version: 1.1.1
Werkzeug version: 0.16.0

I reckon I can tackle this as my first contribution to this repo. @expobrain May I?

No need to ask to work on an issue. As long as the issue is not assigned to anyone or have a linked open PR (both can be seen in the sidebar), anyone is welcome to work on any issue.

Noted. Thanks. I want to run the changes by you before making a pull request @davidism
from flask import g

    exception = Exception('Something went wrong!')

    def get_db():
        if 'db' not in g:
            g.db = connect_to_database()

        return g.db

    @app.teardown_appcontext
    def teardown_db(exception):
        db = g.pop('db', None)

        if db is not None:
            db.close()


Models with (multiple) discrete control(_states) raise exceptions

What
Models with (multiple) discrete control raise exceptions in ribasim 2024.8.0.
For the Ribasim RWS-HWS model we are trying to build-in verdeelsleutels; discrete control controlling fractions between basins depending on upstream discharges. We have two Lobith controlling discharge over Rhine branches and Monsin controlling Meuse discharge over lower Meuse and canal systems.
Reproduction
We increased complexity starting with a model without control 1 untill crashing models 3 and 4. Except controls the rest of the models is the same:

Model without verdeelsleutels (no control), works, see: https://we.tl/t-VE2QSqci24
Model with Lobith only, works as well, see: https://we.tl/t-LDyMnJXCia
Model with Monsinonly, raises an Exception: https://we.tl/t-uW9usryf0C

Model with both Monsin and Lobith, raises another Exception: https://we.tl/t-St9vuGqJZO

Issue 3 we cannot follow; it seems to pass the validator, but than doesn't give any info. At 4, it seems the validator get's messed up with different control_states; it seems to expect control_states of Monsin at Rhine-fractions and vise-versa. As:

Node 7124 of the Exception is the Outlet at Monsin
The connected DiscreteControl node = 1000001 has only control_states with pattern Monsin_###.
FractionalFlow nodes 10653 and 7137, downstream Outlet 7124 have only control_statesw ith pattern  Monsin_###

good to keep #720 in the back of your mind

See #1569 for a fix to the validation code that triggered point 4.
Point 3 is a more difficult issue. It is thrown due to this TODO: https://github.com/Deltares/Ribasim/blob/0c19719ea764499495dad4af003f640587382376/core/src/solve.jl#L619
If you replace the Outlet with a Pump it should work. This is because the Pump will always move a 1Q over a FractionalFlow to fractions of say 0.4Q and 0.6Q.
The Outlet reduces the flow to 0 if the downstream level approaches the upstream level. But what are we supposed to do in the FractionalFlow if one path is blocked or reduced in this way? Should we give up on the fraction and just send it where it can go? Right now the assumption in the FractionalFlow is that the upstream side alone determines Q, and FractionalFlow only has to take fractions off.

Check, I'll change to a fake TabulatedRatingCurve for now as it should go to a real-one, in time, any way. Two questions:

Could it be Connectivity needs an update?FractionalFlow doesn't seem to be allowed downstream TabulatedRatingCurve either.
As we see it as a connectivity-problem, would it be an enhancement to capture this in data-validation?

Yeah good points, I added #1570.
You mean changing it to two TabulatedRatingCurves one one Basin? Right now one TabulatedRatingCurve to two FractionalFlow also works, but if we stop it from sending water to higher downstream levels that will have the same issue.


Set user nickname and nicename to user's realname

To mitigate the security issue of exposing a user's login name ( issue #72 ):

Set user's nickname to their first_name (+ last_name if present)
Set user's nicename to their first_name (+ last_name if present)
(the nicename is used in URL links and is therefore also exposed)

Examples:

"John" gets nickname "john", nicename "john"
"John Doe" gets nickname "John Doe", nicename "john_doe"

Also, don't override user's nickname if one already exists, allowing manual edit from the WP interface.

Thanks for your contribution!
Looks great at first sight - I'll have a closer look this weekend :wink:


Adds a numerical constants tooltip conversion

I open this PR so I can get some opinions on the code for this one. I have annottated few fragments that I will polish Soon™ once I get some feedback/help
Note that the tooltip api is nowhere to be found on the Monaco Editor, but decorations are able to access them, so we just create a decoration on the mouse position that opens a tooltip with the converted value. In order to do this and not lose the linking code decorations, we now have to store the decoration arrays separatedly, and emply the first one of this array for the tooltip. I think it's a good solution but I'm open to suggestions.
I'm quite sure binary numerals will get parsed as decimals by javascript, so there's that
Closes #27 #107

If you're happy after considering my comments I'm happy to merge :)

Check my answers so I know what to do/if I should do anything more at all

#30 solution probably benefits from having this merged (As this PR should introduce a "uniform" way of creating tooltips) so I'll hold an initial PR of that one until this gets merged

Great stuff; thanks -- am out for the rest of the day but will take a look
at this first thing tomorrow (I'm on Chicago time here). Thanks for your
patience!
On Sat, Mar 11, 2017 at 5:46 PM Rubén<EMAIL_ADDRESS>wrote:

#30 https://github.com/mattgodbolt/compiler-explorer/issues/30 solution
probably benefits from having this merged (As this PR should introduce a
"uniform" way of creating tooltips) so I'll hold an initial PR of that one
until this gets merged
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mattgodbolt/compiler-explorer/pull/319#issuecomment-285909651,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmsdajgg19KE4YzV6DyqwSvOuvVlQMZks5rkzJsgaJpZM4MZ78_
.

I don't see any answers (but maybe I'm looking in the wrong place!?) - but I'd be happy to merge if you are :)

On the comments. But if you're ok with having only an unified storage of decorations, then yes! Go ahead and merge it, and then I'll be able to keep working on the asm docs

Sounds great! thanks :)
On Tue, Mar 14, 2017 at 9:08 AM Rubén<EMAIL_ADDRESS>wrote:

On the comments. But if you're ok with having only an unified storage of
decorations, then yes! Go ahead and merge it, and then I'll be able to keep
working on the asm docs
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mattgodbolt/compiler-explorer/pull/319#issuecomment-286430605,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmsdYb2y87Wda7XsenntWWtzS-F7E51ks5rlp5JgaJpZM4MZ78_
.


System Initialized event

There are likely scenarios where an application may need to know that a system has been initialized / reset and respond accordingly.
Design wise, I envision a single event something like this:
void OnSystemInitialized(IMixedRealityService systemInstance)
This would enable client code to react to the change and (potentially) update existing objects to reflect the changes in the specific system (for example, the desired material or model).
This is not a breaking change as the interface would be additive and optional.
Requirements

All existing services send the system initialization event
Client code can handle system [re]initialization

Acceptance Criteria

[ ] Clients wishing to handle initialization events can respond appropriately
[ ] Clients wishing to NOT handle initialization events need not be changed

Awesome proposal, although I think this should be a task 😄

Please add some requirements and acceptance criteria.

Will look into this as part of the system interface rework that I am investigating

Closing out in favor of thinking through the entire core architecture.


Try to make the extension dynamic

So that it can show the stats for different username also
like not to hard code the username in script.js file

done 🚀


404 Client Error: Not Found("open /scratch/docker-data/tmp/docker-import-764157190/repo/boot/json: no such file or directory")

I'm squashing a large 13 GB image down to around 8.  The process seems to succeed but in the end it fails with the below exception.  It happens when squashing to a file or directly to the local repository.  In the former case, the file does get written to disk but I need to directly write to the registry.  The reason for this requirement is also strange.  If I write the tar image and then import it, it is larger than the unsquashed image.
2019-03-19 14:00:02,639 root         INFO     Squashing finished!
2019-03-19 14:00:02,829 root         DEBUG    Reading JSON metadata file '/scratch/tmp/old/5cec968d5b037cc117f6b0dad73a92483a837decfb41a87b1b476097d6862791/json'...
2019-03-19 14:00:02,830 root         DEBUG    Removing from disk already squashed layers...
2019-03-19 14:00:04,589 root         INFO     Original image size: 12725.23 MB
2019-03-19 14:00:04,589 root         INFO     Squashed image size: 8663.02 MB
2019-03-19 14:00:04,589 root         INFO     Image size decreased by 31.92 %
2019-03-19 14:00:04,589 root         INFO     New squashed image ID is c2c473cb41517738d3a343b38b87af4e3bb677fd5ebb29a85061918487cca1d1
2019-03-19 14:00:04,590 root         DEBUG    Generating tar archive for the squashed image...
2019-03-19 14:00:41,381 root         DEBUG    Archive generated
2019-03-19 14:00:41,381 root         DEBUG    Loading squashed image...
2019-03-19 14:02:09,035 urllib3.connectionpool DEBUG    http://localhost:None "POST /v1.21/images/load HTTP/1.1" 404 96
2019-03-19 14:02:09,036 root         DEBUG    Cleaning up /scratch/tmp temporary directory
2019-03-19 14:02:10,643 root         ERROR    404 Client Error: Not Found ("open /scratch/docker-data/tmp/docker-import-764157190/repo/boot/json: no such file or directory")
Traceback (most recent call last):
File "/root/.local/lib/python2.7/site-packages/docker_squash/cli.py", line 87, in run
from_layer=args.from_layer, tag=args.tag, output_path=args.output_path, tmp_dir=args.tmp_dir, development=args.development, cleanup=args.cleanup).run()
File "/root/.local/lib/python2.7/site-packages/docker_squash/squash.py", line 59, in run
return self.squash(image)
File "/root/.local/lib/python2.7/site-packages/docker_squash/squash.py", line 92, in squash
image.load_squashed_image()
File "/root/.local/lib/python2.7/site-packages/docker_squash/image.py", line 254, in load_squashed_image
self._load_image(self.new_image_dir)
File "/root/.local/lib/python2.7/site-packages/docker_squash/image.py", line 299, in _load_image
self.docker.load_image(f)
File "/root/.local/lib/python2.7/site-packages/docker/api/image.py", line 302, in load_image
self._raise_for_status(res)
File "/root/.local/lib/python2.7/site-packages/docker/api/client.py", line 258, in _raise_for_status
raise create_api_error_from_http_exception(e)
File "/root/.local/lib/python2.7/site-packages/docker/errors.py", line 31, in create_api_error_from_http_exception
raise cls(e, response=response, explanation=explanation)
NotFound: 404 Client Error: Not Found ("open /scratch/docker-data/tmp/docker-import-764157190/repo/boot/json: no such file or directory")
2019-03-19 14:02:10,678 root         ERROR    Execution failed, consult logs above. If you think this is our fault, please file an issue: https://github.com/goldmann/docker-squash/issues, thanks!

This works for me without using docker-squash:

Make a container that changes the image as in the normal docker-squash instructions
use docker export CONTAINER-ID | docker import - REPOSITORY:TAG to produce the squashed image

As far as I can tell, this produces an image similar to what docker-squash would produce.


Merge UI: Show annotations under scrollbar

Testing #161834
The old UI shows annotations under the scrollbar for unresolved conflicts, these are helpful for navigation and to get an idea of how much work there is:

The new merge editor shows this too:

I had the same and was wondering what the 'warning' was. Warnings are shown in the scrollbar vertical in a very similar way and color. Maybe conflicts could be shown more distinct from warnings.

I came across the same problem as @IanMatthewHuff with the scrollbar (ruler) decorations appearing inconsistent with the highlighted changes, and so I would like to create a PR to address it.
As I understand it, the inconsistency is caused by #156482 (which addresses issue 156037). Because the removal of ruler decorations for non-conflict changes is an intentional design, I propose adding a new setting, mergeEditor.showNonConflictingChangesInOverviewRuler, to be used here and here. Note that it doesn't address the potential confusion with warnings, but I think those appear on the right-hand side of the ruler, whereas these are in the middle.
@hediet, please let me know if you agree with this approach. Since I like to review every change when merging, conflict or not, a setting to highlight all changes in the ruler would be useful!


[NOT DONE YET] Maf88/170 move to oo plotting

NOT DONE YET! But because it's my first push I'd like to have your comments and critics.
Please go commit by commit to see the small changes I made.
Best.
A

Coverage decreased (-0.09%) to 81.388% when pulling bd152402de5c1a26042bd6d15c2e8db5ac530ff0 on maf88/170-move-to-OO-plotting into 6ebe87ceba001e7af84aeec033cd2606babeb8fb on master.

Thanks @maf88 ! Don't forget to take a look at the Jupyter Notebooks too (all in the notebooks/ directory); there's a lot of plotting going on in them, and they aren't being tested by the TravisCI tests yet. e.g. in this file, here:

Let me know if you're not familiar with Jupyter Notebooks. Also, some notebooks are stale, however for sure training_guideline.ipynb and morphometrics_extraction.ipynb are being used and have plotting.

@jcohenadad  @mathieuboudreau What is the directive with the .ipynb notebooks? It seems uncomfy to not display graphs. Or maybe we can keep pyplot but only keep a few commands (such as plt.show)? What do you think?

@maf88 started to review. First thing to note: I believe that a lot of the refactoration you've done has been to get us closer to PEP8 (e.g. line length, etc), correct?
I ran the pep8 style checker (just noticed that it's actually been renamed to pycodestyle), and it appears that the editor you're using is adding "tabs" when pressing the tab button, and not 4 "spaces". Just an FYI that true tabs are not recommended in PEP8, in particular because Python 3 does not allow mixing spaces and tabs. See here for details: https://www.python.org/dev/peps/pep-0008/#tabs-or-spaces
Output of pep8:

pep8 AxonDeepSeg/morphometrics/compute_morphometrics.py
/Users/mathieuboudreau/miniconda/envs/ads_venv/lib/python3.6/site-packages/pep8.py:2124: UserWarning:

pep8 has been renamed to pycodestyle (GitHub issue #466)
Use of the pep8 tool will be removed in a future release.
Please install and use `pycodestyle` instead.

$ pip install pycodestyle
$ pycodestyle ...

  '\n\n'
AxonDeepSeg/morphometrics/compute_morphometrics.py:23:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:23:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:23:80: E501 line too long (89 > 79 characters)
AxonDeepSeg/morphometrics/compute_morphometrics.py:24:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:25:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:26:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:52:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:52:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:53:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:54:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:55:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:57:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:96:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:96:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:97:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:98:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:99:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:100:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:114:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:114:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:115:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:116:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:117:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:133:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:133:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:134:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:134:80: W291 trailing whitespace
AxonDeepSeg/morphometrics/compute_morphometrics.py:135:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:136:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:136:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:137:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:138:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:139:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:140:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:166:80: E501 line too long (80 > 79 characters)
AxonDeepSeg/morphometrics/compute_morphometrics.py:174:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:174:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:175:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:184:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:184:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:185:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:186:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:187:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:188:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:190:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:192:80: E501 line too long (86 > 79 characters)
AxonDeepSeg/morphometrics/compute_morphometrics.py:208:80: E501 line too long (87 > 79 characters)
AxonDeepSeg/morphometrics/compute_morphometrics.py:214:9: W503 line break before binary operator
AxonDeepSeg/morphometrics/compute_morphometrics.py:214:80: E501 line too long (82 > 79 characters)
AxonDeepSeg/morphometrics/compute_morphometrics.py:215:9: W503 line break before binary operator
AxonDeepSeg/morphometrics/compute_morphometrics.py:215:80: E501 line too long (82 > 79 characters)
AxonDeepSeg/morphometrics/compute_morphometrics.py:216:9: W503 line break before binary operator
AxonDeepSeg/morphometrics/compute_morphometrics.py:236:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:236:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:237:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:238:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:239:1: W191 indentation contains tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:241:1: E101 indentation contains mixed spaces and tabs
AxonDeepSeg/morphometrics/compute_morphometrics.py:245:80: E501 line too long (83 > 79 characters)
AxonDeepSeg/morphometrics/compute_morphometrics.py:250:80: E501 line too long (87 > 79 characters)

If these aren't lines you changed and are legacy ones, then disregard. I'll change them to spaces, just wanted to be let you know in case your tab setting on your editor was the cause, and about that style checker tool.

@mathieuboudreau Nice call! I believe that my tab size settings in my current editor (VS Code) was overriden because of an automatic Detect Indentation option. Hope it works now!


Fixed HMSET argument handling on Lua redis.call.

As opposed to Redis HMSET taking field value pairs as separate arguments, MockRedis.hmset (like py-redis) takes all pairs in one dictionary. Argument list thus need to be converted when calling from Lua script using redis.call('HMSET', hashkey, field1, value1, ..., fieldN, valueN)

I see tests failed.
It didn't happen locally, I'll check and fix this, sorry.

Travis checks now fail due to a general problem with configured pip command syntax. See PR #114.


fix(client): typo in information message

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

I ran into a typo.

Congrats on your first pull request (PR)! 🎉
Thank you, for your contribution to the page! 👍
We are happy to accept these changes, and look forward to future contributions. 📝


Make RulePolicy more efficient with large trackers

Description of Problem:
Large trackers can cause a slowdown / memory spike when making a prediction with RulePolicy.
Definition of Done:

[ ] Reproduce problem
[ ] Scope possible fixes

@TyDunn I wasn't sure what squad this would belong under, lmk if it's not enable :)

@desmarchris I'd check out this thread on Slack that discusses some related things

@JEM-Mosig does your bug fix PR possibly also address this issue?

@kedz No, this will not affect RulePolicy, since I only touch the AugmentedMemoizationPolicy class and RulePolicy is derived from MemoizationPolicy.

I'd rephrase this issue to "make RulePolicy faster with large trackers"

changed it slightly bc it's not just about time but also memory/cpu consumption

@desmarchris I'm trying to reproduce this right now. To be sure: the problem was with inference only, right? Not with training. Do we have an example tracker?

correct, just at inference. I'm sending you an example for a single prediction that's 2500 lines long on Slack.

Oh, I think the linear time increase is actually the issue. Mapping policy didn't check the tracker at all (except last event), so it'd be constant time for that one.

the specific issue at hand here was literally printing 2500 lines to make a prediction caused a CPU spike that brought the pod close to its max capacity

Printing? So this has nothing to do with the policy, but only with the fact that debug was on?

I think so? but debug was needed to help find what went wrong with the bot. I don't know anyone running a bot in prod without debug turned on

I don't know anyone running a bot in prod without debug turned on
That is very concerning. It's not meant to be used this way. I suppose we can have RulePolicy just not print the state in the short term, and in the long term... find a way to store debug logs more quickly? @koaning had this idea of a streamlined debug log (--insights?), but I suppose customers would just always go for printing everything. So we'd have to add a "secret debug flag" that is really only to be used for us inside Rasa.

I see we actually have a Issue https://github.com/RasaHQ/rasa/issues/7295 for that already.
I'll close the present issue now, noting that the abovementioned issue would fix the slow debug output, which caused the present issue in the first place.
If we wanted to actually accelerate RulePolicy, we could speed up the lookup of rules. What makes this slightly complicated is that in contrast to "MappingPolicy rules", the "RulePolicy rules" can be nested. I.e. you can have a short rule inside a long rule. Even if the short rule applies, you still need to check if you can find a long rule that is more specific. Surely we can create a custom algorithm that is faster than the linear time one that we have right now. Again, given that this doesn't actually cause an issue with the customer, and the present issue is about scoping solutions, I'll close the present issue.


Bracket-only indent guides

It would be nice to have an option to render indent guides only inside bracket-defined regions instead of on every tab.

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

Indent Guides Have Breaks Where CodeLens UI is Rendered (#9604)
Feature Request: Smarter indent guides (#48772)
Indent guides are rendered for empty lines in gitlens's file blame panel (#51484)
Tree Guide Indents not showing in latest Insiders (#86675)

Right, a duplicate of #48772. That issues is more than 1 year old, so far was there any update?

/duplicate @alkaitagi please continue your contribution there.


Pace with zebra Browser Print break

It's work without pace.js but after add pace nothing, no error, no information.
I have try with "Pace.ignore(function(){} ); " no corrrection ...
the sdk can be found here
https://www.zebra.com/us/en/products/software/barcode-printers/link-os/browser-print.html

It's not printing or pace is not working?

not printing


Introducing Composed Task Runner App Starter

To build:
mvn clean install -PgenerateApps
cd apps/composedtaskrunner-task
mvn clean install

README.adoc has usage information.

Resolves spring-cloud-task-app-starters/app-starters-release#20

Resolved.


Allow later versions of coverage.

Thanks for this contribution!

Thanks for this contribution!

Thanks for this contribution!

Thanks for this contribution!


Some animating Crawdad images do not exist

Mod source
Factorio Mod Portal
Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[X] Graphical
[ ] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
Crawdad, running at 11 o'clock, has a flashing animation because some of the images are not present when it moves.

Steps to reproduce
1.Install Crawdad, a mount.
2.Putting in fuel fish.
3.Getting into Crawdad
4. Running toward 11 o'clock
Additional context
No response
Log file
factorio-current.log

Ran into the same issue with normal resolution graphics, so I popped in here and saw this. Here's the orientation I'm in when it happens, though I think I remember -- hope this helps!

Also, as just a guess from some light digging, the issue is in pyalienlifegraphics3, likely in crawdad-05.png.


Application get stuck when displayLink invalidate

In my application, when I push the RCTRootView and pop it at once,  the application will easily get stuck with no touch response.
I paused program execution and found that the program got stuck at the invalidate  method of CADisplayLink.

The react native version is 0.4.1.
Thanks.

@cxfeng1 - thanks for the report! Can you try this with the latest master branch (npm i facebook/react-native#master --save) and see if that fixes it? Ping me if it is not fixed and I'll reopen the issue!


Does not work thru iobroker.pro

Describe the bug
Starting version 0.4.0 of adapter it does not connect anymore via iobroker.pro cloud. In Chrome console I get "SocketURL not defined" error message.
Versions:

Adapter version: 0.4.1
JS-Controller version: 3.0.14
Node version: 10

Hi, gehts hier um das Frontend oder die Konfigurations-Seite?

it is about frontend

Please provide log entrys from console (f12 in the browser).

it now works with latest version after I cleared browser cache. :)
thanks!


Implement method for converting a mesh into a convex hull mesh.

Implement method used for geometry optimization, where a Mesh instance is converted into a Mesh instance containing the convex hull representation. The conversion method is implemented in the Simplify class, where the MIConvexHull Nuget package is used for the convex hull conversion. Tests for the implemented method is added to the SimplifyTests class.
AB#170740

Hm, build fails because of some missing package updates. I'll check


Customize swaggerUIPath currently broken

This is the error message which I get when following the README instructions on how to customize the endpoint paths: “Can’t read swagger JSON from http://localhost:8080/openapi” even though I’m at http://localhost:8080/swagger/ui/
Therefore I'm going to remove the following wording from the README under PR #7 (when this issue is fixed the wording below can be added back in):
If you wish, you can customize the endpoint paths:
let config = KituraOpenAPIConfig(apiPath: "/swagger", swaggerUIPath: "/swagger/ui")
KituraOpenAPI.addEndpoints(to: router, with: config)

@helenmasters I'm hoping this is now fixed by #10 - could you try on the latest tag (1.1.0)?

@helenmasters This is fixed, I have a sample project that customizes the swagger and swaggerui path: https://github.com/djones6/OpenAPIDemo/blob/master/Sources/Application/Application.swift#L36
This works locally for me.  Could you confirm, and then add the wording back in to the readme?

Tested and confirmed as fixed. Raised PR #21 to add the wording back into the README.

Closing as the issue is now fixed and the wording has been adding back in under PR #21.


Format crawl-delay to return an int

Crawl delay is intended as an integer in most specifications (https://en.wikipedia.org/wiki/Robots_exclusion_standard#Crawl-delay_directive), no need to format as float

@jazzband/members anyone to review this?

check build failures

@auvipy the build failures are due to not having a .travis.yml file and travis using default configuration (ruby).

Tests and Travis configuration are in #50, but without this change build fails

In general it's better to try having the tests and the code it fixes in the same pull request, as it's easier for reviewers to see what's impacted and try to run the test-suite on their setup (useful sometimes).
Anyway, thanks for the pull request!

@almet robots did not have any test before #50 and having a single PR with all the test infrastructure, tests and the fixes to make the test run right it would make the review of the different code changes more complicated.
Given that as of now everything has been merged without tests, I thought it would have been better to provide an easy way to review consistent code changes, instead of a gigantic PR with half a dozen of different things in

@yakky Agreed.

There's a setting in Travis to only run tests if a .travis.yml is present. I think that would be a good change and would avoid this confusion.

@yakky already defined a travis.yml we just need to rebase this branch.


Performance slowdown and high memory consumption when using dynamic imports

Bug report
When using dynamic import call over sync require I see increase in build time and in memory consumption.

What is the current behavior?
I have some private application that uses dynamic imports to implement lazy loading.
After some code simplification and enabling additional logs, If I run it I see next numbers for time and memory:
DEBUG LOG from webpack.buildChunkGraph
<t> visitModules: prepare: 2.5296 ms
<t> visitModules: visiting: 6.703999 ms
<t> visitModules: calculating available modules: 0.6491 ms
<t> visitModules: merging available modules: 0.939 ms
<t> visitModules: check modules for revisit: 0.3369 ms
<t> visitModules: prepare: 45.050496 ms
<t> visitModules: visiting: 778.289864 ms
    549028 queue items processed (265085 blocks)
    380 chunk groups connected
    380 chunk groups processed for merging (380 module sets, 0 forked, 0 + 0 modules forked, 0 + 0 modules merged into fork, 0 resulting modules)
    380 chunk group info updated (0 already connected chunk groups reconnected)
<t> visitModules: 788.805262 ms
<t> connectChunkGroups: 1.801699 ms
<t> cleanup: 0.1743 ms

client (webpack 5.87.0) compiled with 1 warning in 110049 ms
{
  clientBuildTime: 110050,
  maxMemoryRss:<PHONE_NUMBER>
}

If I replace dynamic imports with sync requires I get this:
DEBUG LOG from webpack.buildChunkGraph
<t> visitModules: prepare: 113.764402 ms
<t> visitModules: visiting: 187.628703 ms
    4353 queue items processed (2031 blocks)
    0 chunk groups connected
    0 chunk groups processed for merging (0 module sets, 0 forked, 0 + 0 modules forked, 0 + 0 modules merged into fork, 0 resulting modules)
    0 chunk group info updated (0 already connected chunk groups reconnected)
<t> visitModules: 189.600003 ms
<t> connectChunkGroups: 0.2773 ms
<t> cleanup: 0.0831 ms

client (webpack 5.87.0) compiled with 3 warnings in 100983 ms
{
  clientBuildTime: 100984,
  maxMemoryRss:<PHONE_NUMBER>
}

You can see that build with dynamic imports executing for 10 seconds longer and requires almost twice of memory.
If the current behavior is a bug, please provide the steps to reproduce.
I've created an reproduction repo that shows the issue, although the numbers are not that high as in real application.

What is the expected behavior?
I hope there could be a room to improve this to make lazy chunking less expensive for building.
For example in case of dynamic imports the more time is spend in visitModules function that has comment about possibility of removing some traversals.

Other relevant information:
webpack version:  5.87.0
Node.js version: 16
Operating System: Debian bullseye

tl;dr: This seems to be a regression in NodeJS that was introduced in NodeJS 16.11 after they upgraded the V8 engine, it's still an issue.
Details:
We've been running into memory issues in our NextJS build (latest v12) and some of our Jest test suites. The common factor between the build process and the test suites are the presence of dynamic imports in the code. The test suites that don't have dynamic imports, don't seem to have memory issues, the ones that do, do.
We eventually noticed the following Jest issue that's currently still open https://github.com/jestjs/jest/issues/11956 and the fix mentioned there (--workerIdleMemoryLimit jest argument) seemed to improve things for us on the Jest side of things. As for NextJS we're currently attempting to resolve this by tweaking the --max-old-space-size we pass to node via NODE_OPTIONS and making sure we try to calculate it based on the maximum amount of memory and cpu resources we're making available during the build process, as the NODE_OPTIONS in our case get distributed to a number of NodeJS processes that NextJS seems to spawn (could also be Webpack doing that under the hood, we didn't dig too too deeply into that side of things).
Would it be possible for Webpack to implement a similar option until the issue in NodeJS gets resolved for good?

Looks like it was fixed, just tested on<EMAIL_ADDRESS>and don't see the problem anymore, so I want to close and sorry for delay, anyway feel free to feedback


import MXNet Error: undefined symbol: gotoblas

After I install MXNet, I tried to import. Hower, it output below message:
 yuens@Spark  ~  python -c "import mxnet as mx"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/mxnet-0.9.4-py2.7.egg/mxnet/__init__.py", line 7, in <module>
    from .base import MXNetError
  File "/usr/local/lib/python2.7/dist-packages/mxnet-0.9.4-py2.7.egg/mxnet/base.py", line 11, in <module>
    import numpy as np
  File "/usr/lib/python2.7/dist-packages/numpy/__init__.py", line 180, in <module>
    from . import add_newdocs
  File "/usr/lib/python2.7/dist-packages/numpy/add_newdocs.py", line 13, in <module>
    from numpy.lib import add_newdoc
  File "/usr/lib/python2.7/dist-packages/numpy/lib/__init__.py", line 8, in <module>
    from .type_check import *
  File "/usr/lib/python2.7/dist-packages/numpy/lib/type_check.py", line 11, in <module>
    import numpy.core.numeric as _nx
  File "/usr/lib/python2.7/dist-packages/numpy/core/__init__.py", line 14, in <module>
    from . import multiarray
ImportError: /usr/lib/libblas.so.3: undefined symbol: gotoblas

@ysh329 could you please describe how you solve it in detail? Thx

I forgot detailed describe. try to re-install openblas.


Windows 10 Python 3.6: User doesn't have Visual Studio C++

User on Windows 10, Python 3.6, encounters
Error code With Python 3.6.6 (07.18.2018).docx
error. it seems to be trying to compile.
perhaps the wheels are out of date on this one?

Hi sploiber, we need to build windows wheels. And wheels for 3.7. Thanks for the bug report!


Shrink/expand broken by charmrun refactoring

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

Trying to build with '--enable-shrinkexpand' fails on compiling charmrun with the following on netlrts-darwin-x86_64:
../../bin/charmc -g -O0  -Wno-error -lm -I.. -L../libs -c -seq -DCMK_NOT_USE_CONVERSE=1 charmrun.C -o charmrun.o
charmrun.C:4300:23: error: use of undeclared identifier 'nodetab_size'
  for (int i = 0; i < nodetab_size; i++)
                      ^
charmrun.C:4301:38: error: use of undeclared identifier 'nodetab_table'; did you mean 'host_table'?
    sprintf(dest1, "%s %s", dest1, (*nodetab_table[i]).name);
                                     ^~~~~~~~~~~~~
                                     host_table
charmrun.C:1352:36: note: 'host_table' declared here
static std::vector<nodetab_host *> host_table;

Original date: 2018-03-15 18:50:47

~~https://charm.cs.illinois.edu/gerrit/3860~~ https://github.com/UIUC-PPL/charm/commit/f4a930ca908299ce7a7108a51794e0639416fefb
Are there any tests for shrink/expand?

Original date: 2018-03-15 18:53:04

examples/charm++/shrink_expand. It doesn't look like it's part of automated testing

Original date: 2018-03-15 19:12:44

The example crashes. I will have to debug this.

Original date: 2018-03-15 19:50:57

k, since it doesn't have automated testing it may have been broken for a while now. Though I believe we tested it for v6.8.0's release.

Original date: 2018-03-26 19:55:07

Fixed in: ~~https://charm.cs.illinois.edu/gerrit/3900~~ https://github.com/UIUC-PPL/charm/commit/8adeaba44208b5730883effc55bdd559fa7720d8
Tested with netlrts-darwin-x86_64 and netlrts-linux-x86_64.


Add new article to

Hi @maryjomadda!
I see that you had created another issue with some information, so I guess this was created by error. I'm going to close this one and we continue talking at https://github.com/folkswhocode/awesome-diversity/issues/51!
😄


Support dfs.client.use.datanode.hostname config property

When reading blocks, client talks to datanode by default using the datanode's
IP address returned by the name node. The problem with this approach is that
the IP address is a private IP within the cluster, so the client won't works if
it is in a different network. This commit updates the config module to support
dfs.client.use.datanode.hostname config property, and have the clients to use
this property to determine weather to use the IP address or the hostname to
talk to datanodes.

Any updates for this PR? Being able to talk to HDFS in a private network is useful to us since we are running some Spark job in the in house cluster which need to talk to our HDFS cluster in AWS, and I believe this is not a rare use case for others as well.

LGTM. @ravwojdyla what do you think?

Sorry for the long wait, but this looks good.

I merged it, but maybe one thing to add is some documentation around this for the command line client? Or add a command line flag as well? Now this can only be done through .snakebiterc or the hadoop config.


10 times more expensive??!

Am I understanding the pricing correctly, in that the v2 standard sku fixed price is about 10 times more expensive than the price for a v1 small standard sku? That's crazy!
We are currently setting up our test and stage environment, and the plan was to use an application gateway in front of each of our four VM scale sets (each scale set is a completely different service, so we can't join them). That would mean a total of 8 application gateways in non-production. That's $1400 per month in fixed costs! That is a massive increase in costs compared to the v1 sku.
Is there really no v1 version suitable for development/testing environments, with virtually no traffic (3 person team, only traffic is when we actively test something)? We are willing to reduce the "power" of each application gateway, not the number of gateways used. And we don't want a vastly different setup compared to production (meaning: we want v1 in both prod and non-prod, or v2 in both).

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

ID: faa5b8a2-11a1-6a2d-07cb-ffcf39643ed4
Version Independent ID: 298a134e-fb27-f16d-bff2-d43eb0f766fd
Content: Autoscaling and Zone-redundant Application Gateway v2
Content Source: articles/application-gateway/application-gateway-autoscaling-zone-redundant.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

@batanun App Gateway V2 has a significant change in pricing, but there are some ways that you can reduce the cost for a test / low traffic environment. See #41600 for more info

Set the instance count to 1 - you will lose any redundancy if your 1 instance fails, but on a dev envionrment that will likely be acceptable

Set the App Gateway to AutoScale with a minimum instance count of 0 - This will spin down all instances if your app has no traffic coming in. It will only work if you have large periods of inactivity.

To clarify further

Setting AutoScale to a minimum instance of 0 will keep a minimum of 2 instances on at all times to maintain high availability.

I'm sorry, but you are not very clear. We already have one instance count in our v1 application gateway. Going from 1 v1 instance to 1 v2 instance increases the fixed costs a lot. How do we reduce this fixed cost? If the instance count is 0, will the fixed cost be 0 too? We would like to only pay for when we use it (normal office hours), not night time and weekends.
Also, why do you first talk about setting it to 1, then setting it to 0? And why would setting it to 0 result in 2 instances? It get's a bit confusing :)

@batanun Your assessment is correct on the cost, App Gateway V2 will be more expensive than V1, and if you are already at a fixed 1 instance, you are at the cheapest you can get.
For Autoscaling, there will always be a minimum of 2 instances to maintain high availability, so this will always be more expensive than doing a fixed instance of 1.

Thanks for clearing that up :)
But this means that  V2 won't be an option for us, even for production. We are not interested in using V1 in testing and staging, and V2 in production. And V2 in test and staging environments just seems unreasonable with these high fixed costs. My suggestion is that you look into better V2 options for non-production use. If a v2 instance is not used at all it won't incur any significant cost at all for Microsoft, so why should we need to pay that high fixed cost?

@batanun Thanks for the feedback. Your best place to leave that feedback about the pricing is at Azure Networking's Feedback.Azure.com page.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

This is bit confusing as the comments mentioned above is not very clear.
Can you please confirm if the min autoscale instances are set to zero how many instances or capacity units are reserved or available  for high availability?


Passing WebViewBridge to Web Worker

First of all, awesome work on putting this together. Really looking forward to this being incorporated into core, it's a huge help.
I was just wondering if anyone has any experience or guidance to offer around utilising WebViewBridge within a web worker? Is it possible? How would I go about implementing this?
Thank again,
Strainy

@Strainy thanks. Need to know your scenario. also I have implemented v2 which brings RPC into the picture.
about the core, there 2 main issue outstanding which prevents me to make a PR. waiting for core team to make their decision.

I'm working on a mapping solution leveraging the Tangram library in a WebView. I want to provide data to my map over the bridge. The Tangram library uses Web Workers to request the data used to render the map in the background for performance reasons.
As an interim solution for an Offline Mapping POC I'm working on, I'd just like to reference an  external script file which will override the XMLHttpRequest prototype within the worker so that rather than requesting data over a network it'll re-route the request for the data over the RN WebView bridge.
Tangram uses importScripts() within the worker to load external dependencies. So I would need to independently initialise my own instance of WebViewBridge (i.e. outside the window object).
So it would be very helpful if I could somehow import the WebViewBridge into my scripts, rather than referencing it directly from the window object - as that won't be possible within a Worker.

I don't think it would be possible in the scope of this project. However you have to establish the communication between Web Worker and WebViewBridge yourself.
if you do fork the Tangram library then you can expose some functions which lets you send message back and forth from/to Web Worker to interact with WebViewBridge.
Also internally, WebViewBridge requires to use window.location to trigger react-native that there is a message ready to pick up, and I don't know if that possible in WebWorker.
and As I said, you have to create your own script which bridge the gap between webViewBridge and WebWorker.
I'm closing this ticket but feel free to continue the discussion here.


Add transition and animation features

I kindly suggest adding a transition and/or animation feature to the portfolio to make it more dynamic and live.

-Thank you Sylvester for your feedback. Will insert suggested features by end of the day after accessibility checks pass.


Added support for supergroups and Markdown inline link fix

Added support for supergroups with id lower than int.MinValue
Fixed Markdown inline link

Fixes #1


Dynamically increase height of container

I have an app to allow users to draw polygons on scatter plots and create a tree of plots. e.g.

The problem I have is when a user creates 4 or more plots, they get stacked on top of each other as there is no room, and so I need to allow the user to increase the size of the container by dragging. Heres the problem:

Heres the HTML:
      <div
      >
        <div
          id="workspace-container"
          style={{
            height: 350 + maxTop,
          }}
        >
        Here are the Draggables
        </div>
        <div
          id="dragger-container"
          style={{
            height: "5px",
            backgroundColor: "#333",
          }}
          onMouseDown={(e) => {

          }}
          onMouseMove={(e) => {
            console.log("in mouse move, e.clientY is ", e.clientY);

            let workspaceContainer = document.getElementById(
              "workspace-container"
            );

            // 480 is the original height of dragger-container from the top of the window
            // 355 is the original height of workspace-container
            let newHeight = e.clientY - 480 + 355;
            workspaceContainer.style.height = newHeight + "px";
          }}
          onMouseUp={(e) => {

          }}
        ></div>
    </div>

When I move the mouse over dragger-container the height of workspace-container is indeed dynamically increased. However I still cannot drag the Draggables down. It seems they dont know the height of the workspace-container has increased.

It may be something to do with the containment option here https://anseki.github.io/plain-draggable/#options-containment. But Im not sure....

Hi @Zander1983, thank you for the comment.
Sorry, my English is poor.
Do you mean that the draggable element doesn't work correctly after the size of the element was changed?
If so, you have to call position method.

[bot] No reply came, then this abandoned issue is closed.

Sorry for my late reply - you were correct, that fixed it.

:smile:


$scope isn't available, generally, to factories

Generally speaking, if you attempt to inject $scope into factories, you'll get an error like the one mentioned in the docs at https://docs.angularjs.org/error/$injector/unpr

Attempting to inject a scope object into anything that's not a controller or a directive, for example a service, will also throw an Unknown provider: $scopeProvider <- $scope error.

All this PR does is clarify the intention of the statement.

agreed. thx


LA County Dept. Public Health - Getting Started with ReportStream

@Adrian-Brewster
Do we need to do anything further for this user?
From: Emily Mabile<EMAIL_ADDRESS>Sent: Wednesday, June 15, 2022 7:22 PM
To: Report Stream (CDC<EMAIL_ADDRESS>Cc: Brewster, Adrian (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS>Subject: RE: Getting started with ReportStream
Hi Adrian,
After our call, I believe someone was planning to send the csv specs that you use for report stream.
Thanks,
Emily A. Mabile, MPH
Supervising Epidemiologist | Lab Data Operations
Acute Communicable Disease Control
Los Angeles County Department of Public Health
<EMAIL_ADDRESS>| Office:<PHONE_NUMBER>
Pronouns: she/her/hers
Why pronouns matter
Confidentiality Notice:  This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information.  Any unauthorized review, use, disclosure or distribution is prohibited.  If you are not the intended recipient (or a designated agent), please contact the sender by reply e-mail and destroy all copies of the original e-mail message.
From: Report Stream (CDC<EMAIL_ADDRESS>Sent: Wednesday, May 25, 2022 10:21 AM
To: Emily Mabile<EMAIL_ADDRESS>Cc: Brewster, Adrian (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS>Subject: Re: Getting started with ReportStream
CAUTION: External Email. Proceed Responsibly.
Hi Emily,
I can hop back in here. ReportStream uses a series of mappers and filters to ensure the data coming in can pass validation with the receiving jurisdiction's surveillance system.  In the event that you received data that were missing any key elements, you would email this address<EMAIL_ADDRESS>and we would get a ticket opened to have an engineer troubleshoot.  That said, our CSV format is pretty strict so in order for data to upload to ReportStream and pass our validation, it would need to be high quality.  We work with senders to make sure their data can conform to our requirements and the data is validated with several end users before we move it to production.
If you would like to refer users to onboard to ReportStream, I would be happy to host a Q/A or information session to ensure smooth adoption of our tools.
Let me know how I can help.
Adrian

From: Emily Mabile<EMAIL_ADDRESS>Sent: Tuesday, May 24, 2022 9:32 PM
To: Report Stream (CDC<EMAIL_ADDRESS>Cc: Brewster, Adrian (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS>Subject: RE: Getting started with ReportStream
Hi Sharon,
I am with the LA County department of Health, and I do not have any anticipated message volume.  We currently have a process to transform flat files into HL7 messages for our ELR at LA County. I am interested in transitioning a few labs from our processing stream to yours.
Let me know if you are able to meet to discuss.
Emily A. Mabile, MPH
Supervising Epidemiologist | Lab Data Operations
Acute Communicable Disease Control
Los Angeles County Department of Public Health
<EMAIL_ADDRESS>| Office:<PHONE_NUMBER>
Pronouns: she/her/hers
Why pronouns matter
Confidentiality Notice:  This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information.  Any unauthorized review, use, disclosure or distribution is prohibited.  If you are not the intended recipient (or a designated agent), please contact the sender by reply e-mail and destroy all copies of the original e-mail message.
From: Report Stream (CDC<EMAIL_ADDRESS>Sent: Tuesday, May 24, 2022 3:42 PM
To: Emily Mabile<EMAIL_ADDRESS>Cc: Brewster, Adrian (CDC/DDPHSS/OD/HITSSU) (CTR<EMAIL_ADDRESS>Subject: RE: Getting started with ReportStream
CAUTION: External Email. Proceed Responsibly.
Hi Emily,
What is your anticipated daily message volume?  Also, can you please provide the below information:
Name:
Job title:
Organization:
Email address:
Phone number:
Organization that referred me to ReportStream:
Public health department(s) I report to:
Current reporting method(s):
Best regards,
Sharon
From: Emily Mabile<EMAIL_ADDRESS>Sent: Thursday, May 19, 2022 12:38 PM
To: Report Stream (CDC<EMAIL_ADDRESS>Subject: RE: Getting started with ReportStream
Hi Adrian,
We are interested in onboarding a couple of labs, but before we do I was hoping I could learn more about your feedback process and how your team handles dirty data. I want to propose the transition to the labs with a more complete understanding of what the process will be like for them.
I am available tomorrow between 10am and 3pm pst, Monday 11am or 1pm pst, or 10am or 12:30pm pst.
Let me know if you’re available to meet at any of those times!
Emily A. Mabile, MPH
Supervising Epidemiologist | Lab Data Operations
Acute Communicable Disease Control
Los Angeles County Department of Public Health
<EMAIL_ADDRESS>| Office:<PHONE_NUMBER>
Pronouns: she/her/hers
Why pronouns matter
Confidentiality Notice:  This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information.  Any unauthorized review, use, disclosure or distribution is prohibited.  If you are not the intended recipient (or a designated agent), please contact the sender by reply e-mail and destroy all copies of the original e-mail message.
From: Report Stream (CDC<EMAIL_ADDRESS>Sent: Friday, March 25, 2022 12:22 PM
To: Emily Mabile<EMAIL_ADDRESS>Cc: Karen Swanson<EMAIL_ADDRESS>Daniel Lichtenstein-boris<EMAIL_ADDRESS>Subject: Re: Getting started with ReportStream
CAUTION: External Email. Proceed Responsibly.
Yes, absolutely.  Below is a brief snippet of information you can send out to potential CSV Uploader users and it includes next steps for signing up and getting started.
What’s ReportStream?
ReportStream is a free tool developed by the Centers for Disease Control and Prevention (CDC) and U.S. Digital Service (USDS) that makes it easy for public health data to be transferred from testing facilities to public health departments. More about this tool can be found at reportstream.cdc.gov.
Why use ReportStream?
ReportStream is 100% free and offers some other great benefits:
•        Allows you to send faster, more accurate data directly from your testing facility or lab through a single connection. You no longer have to come up with a makeshift approach to report data.
•        Test results and patient data are securely stored and protected by two-factor authentication, database encryption, and HTTPS.
•        Provides access to view your file submission history.
How do I submit to ReportStream?
Currently, ReportStream can accept either a comma-separated values (CSV) file or Health Level 7 (HL7) input data format. All you’ll need to do is register for a free ReportStream user account to get started.
Next steps:

Register for a free ReportStream user account at https://reportstream.cdc.gov/getting-started/testing-facilities/account-registration-guide
Review the ReportStream CSV Uploader Guide at https://reportstream.cdc.gov/getting-started/testing-facilities/csv-upload-guide

Questions?
If you have questions about ReportStream, you can email them directly at<EMAIL_ADDRESS>
From: Emily Mabile<EMAIL_ADDRESS>Sent: Wednesday, March 23, 2022 8:22 PM
To: Report Stream (CDC<EMAIL_ADDRESS>Cc: Karen Swanson<EMAIL_ADDRESS>Daniel Lichtenstein-boris<EMAIL_ADDRESS>Subject: RE: Getting started with ReportStream
Thanks so much for that information! Are there specific instructions that it would be helpful to include when we direct providers to you?
Emily A. Mabile, MPH
Supervising Epidemiologist | Lab Data Operations
Acute Communicable Disease Control
Los Angeles County Department of Public Health
<EMAIL_ADDRESS>| Office:<PHONE_NUMBER>
Pronouns: she/her/hers
Why pronouns matter
Confidentiality Notice:  This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information.  Any unauthorized review, use, disclosure or distribution is prohibited.  If you are not the intended recipient (or a designated agent), please contact the sender by reply e-mail and destroy all copies of the original e-mail message.
From: Report Stream (CDC<EMAIL_ADDRESS>Sent: Wednesday, March 23, 2022 12:21 PM
To: Emily Mabile<EMAIL_ADDRESS>Subject: Re: Getting started with ReportStream
CAUTION: External Email. Proceed Responsibly.
Hi Emily,
Apologies for the delay.  I had an email written up and saved in my draft folder to respond to this and must have not hit the send button!
California is one of the 4 states currently engaged in the CSV uploader pilot program. Onboarding is fairly quick assuming the data coming in can meet our CSV standard schema requirements. What sorts of questions can I answer for you?
Adrian

From: Emily Mabile<EMAIL_ADDRESS>Sent: Thursday, March 17, 2022 8:17 PM
To: Report Stream (CDC<EMAIL_ADDRESS>Subject: Getting started with ReportStream
Hi there,
I am part of the Los Angeles County Lab Data Operations unit and would like some more information on lab and provider opportunities to report through ReportStream. Specifically, is the CSV upload pilot available in California?
Thank you,
Emily A. Mabile, MPH
Supervising Epidemiologist | Lab Data Operations
Acute Communicable Disease Control
Los Angeles County Department of Public Health
<EMAIL_ADDRESS>| Office:<PHONE_NUMBER>
Pronouns: she/her/hers
Why pronouns matter

@MauriceReeves-usds will follow up with CSV spec

@MauriceReeves-usds have your sent the specs?

Hey team! Please add your planning poker estimate with ZenHub @brick-green @greene858 @jorg3lopez @oslynn @sarahnw @TomNUSDS

Email from Emily:
Hi Sharon,
Yes, you can close it. @Quratulain Ajmal if you have any questions about Report stream- please reach out directly to open a new ticket!
Thank you!
Emily A. Mabile, MPH


Prevent panic when secret is nil from vault read

Check if secret is nil to prevent panic
Adds corresponding test

fixes #28
Prevent panic during ensure dry run

/assign @simonswine

Thanks this is looking good to me
/lgtm
/approve


Inheriting an inherited scene causes error on save: Index p_idx = X out of bounds

Tested versions

Reproducible in 4.3-dev3 [d3352813e]
Reproducible in 4.3-dev2
Not reproducible in 4.2

System information
v4.3.dev.custom_build [d3352813e]
Issue description
When saving a scene that inherits from another inherited scene containing at least one child node, this error appears.

Steps to reproduce

Create a scene (scene_1)
Add a child node
Create an inherited scene from scene_1 (scene_2)
Add a child node
Create an inherited scene from scene_2 (scene_3)
Save scene_3

Minimal reproduction project (MRP)
Archive.zip

Probably duplicate of #87424 ? @Rindbee

Yes, it's a duplicate.


GEODE-7995: Creates concurrencyTest facet

Moves all concurrency style test under its own facet of testing.
Move concurrency test framework sources to geode-concurrency-test.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Can we stage this is a way that doesn't break people's PRs or lose coverage for a while? Maybe we can merge the gradle changes without moving the tests, then add the pipeline support, and then move the tests.

Can we stage this is a way that doesn't break people's PRs or lose coverage for a while? Maybe we can merge the gradle changes without moving the tests, then add the pipeline support, and then move the tests.

We could but I think the number of tests and the time between this commit and the pipeline tie in will be very short. I think the risk is very very small.

I think it would be better to split things up and move the tests after the CI changes are done - doesn't seem too hard to do. The CI changes may take a little time - we have to enhance the stress new test job as well.

With the help of @rhoughton-pivot I have pushed a commit to the CI that we are running through a test pipeline. Will wait for results of that.

Pipeline changes worked on test pipeline.
https://concourse.apachegeode-ci.info/teams/main/pipelines/pivotal-jbarrett-feature-geode-7995-c-main/jobs/ConcurrencyTestOpenJDK8/builds/1

I think this is still missing some changes to include this new test target in stress new test.
Side note - you made a lot of minor cleanup changes with this PR (maybe with some intellij inspection?). It would be nicer to split those out into a separate PR - especially since your changes are not going to get tested because they are running as part of this new gradle target that CI is not going to run on this PR.

The changes were required for them to compile. That project enforces warnings as errors. The test pipeline successfully built and ran those cleaned up tests.

The test jobs are here:
https://concourse.apachegeode-ci.info/teams/main/pipelines/pivotal-jbarrett-feature-geode-7995-c-main/jobs/ConcurrencyTestOpenJDK8/builds/1
https://concourse.apachegeode-ci.info/teams/main/pipelines/pivotal-jbarrett-feature-geode-7995-c-main/jobs/ConcurrencyTestOpenJDK11/builds/1

What do we mean by facet in this context?

I think it means (1) src set, (2) gradle target to execute tests in the src set, (3) pipeline job in CI and PR.

Very specifically we use a plugin called facet that creates all these things for us.
https://plugins.gradle.org/plugin/nebula.facet

@pivotal-jbarrett Want to pair on reviving this PR?

this PR appears to be abandoned, can it be closed?

closed due to lack of activity


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/eventing-writers
/assign knative-sandbox/eventing-writers
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/pkg f78a006...4a03844:
  > 4a03844 fix double close of channel (# 2575)
bumping knative.dev/serving dbd3fc3...d89869e:
  > d89869e Properly shutdown tracer when the work is done (# 13226)
  > b3a7557 Update net-contour nightly (# 13225)
  > 358d04e Update net-certmanager nightly (# 13224)
  > b48e0e5 Update net-kourier nightly (# 13223)

/lgtm
/approve


Deprecate dimensions / metrics APIs on brokers

Fixes https://github.com/apache/incubator-druid/issues/1810.

👍


Potential infinite recursion in value parsing

No lookahead!

Additionally, some values are also statements (i.e. functioncalls)


Remove outdated akismet module

The bundled version of akismet.py (0.2.0) is very outdated. There appears to be a more modernized version of akismet available in pypi: https://pypi.org/project/akismet/
Should remove bundled version and use the pypi package instead.

Updated!


Seems to be broken as of recently

In a private build of packer that I can't share much output of, travis-pls messages are all dumped after packer's. That means that this tool no longer works.
It did before, but as of recently it does not. I'll see if I can gather more evidence.

@smaslennikov I think that Travis CI is actually banning the script. I have also noticed that things have started failing where they shouldn't.

You can test this easily by doing something like:
$ newly-built-travispls-binary sleep 660 in the travis build. I strongly recommend you do.

I am attempting to isolate the problem via naftulikay/travis-sleaze.

I cannot verify that it is broken, so I am closing.

Is it broken again? Maybe you could request a rerun of travis on travis-sleaze to check? I've been trying to use travis-pls, but received no output.

tried this project yesterday but does not seem to work - see that build: https://travis-ci.org/github/ossia/score/jobs/767409044


Added Disk List command and tested with Hyperdeck Mini

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Adds a Disk List feature so you can take advantage of the playback functionality.

What is the current behavior? (You can also link to an open issue here)
There is no list function currently.

What is the new behavior (if this is a feature change)?
Added a Disk List command that now returns the slot id and clips on that slot available for play.

Other information:

Thanks for the PR!


docs: Correct quotation marks

This patch changes ‘ to ', to prevent errors from people that copy
and paste commands from the README.

Current coverage is 56.39% (diff: 100%)

Merging #725 into master will increase coverage by 0.02%

@@             master       #725   diff @@
==========================================
  Files            84         84
  Lines          6916       6916
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3898       3900     +2
+ Misses         2712       2710     -2
  Partials        306        306

Powered by Codecov. Last update 8bee5c9...cdce6be


Crazy fast in general but very slow to start

Linux, Arch, X11.
Alacritty is crazy fast compared to my current terminal in printing stuff but its very slow to start up, noticeably slower...
I couldn't benchmark because I couldn't get time alacritty -e 'bash -c "exit"' to work but the lag is just very clear, at least on a slightly older machine here... with termite I get 90% cpu 0.117s total time.

Duplicate of https://github.com/jwilm/alacritty/issues/782.


Multiple calls to store.ifLet

Thanks for the hard work in making this fork! I'm currently noticing an issue with calling ifLet on a Store. I have a test project that I've been messing around with to get a good feel for using this fork. I can share the test project if needed.
The issue is that I have two modals that are presented from the root view controller. Both have optional states on the root State. When presenting/dismissing the modals one after the other, I'm noticing that the ifiLet call on the store is called multiple times. This doesn't happen the first time, only after dismissing a modal once or twice, and then it happens every time after that.
Here's what my root state looks like:
struct State {
    var items: ItemsState?
    var favorites: FavoritesState?
    ...
}

I'm presenting the view controllers with the following:
let store = self.store.scope(state: \.favorites, action: Root.Action.favorites)
store.ifLet { [weak self] store in
    guard let self = self else { return }
    let vc = FavoritesViewController(store: store)
    let nc = UINavigationController(rootViewController: vc)
    nc.presentationController?.delegate = self
    self.present(nc, animated: true, completion: nil)
}

I was able to fix my issue by creating another variant of ifLet. The "fix" is to use skipRepeats() instead of .skipRepeats { ($0 != nil) == ($1 != nil) }.
@discardableResult
public func ifLet2<Wrapped: Equatable>(
    then unwrap: @escaping (Store<Wrapped, Action>) -> Void,
    else: @escaping () -> Void
) -> Disposable where State == Wrapped? {
    let elseDisposable =
        self
        .producerScope(
            state: { state -> Effect<Wrapped?, Never> in
                state
                    .skipRepeats()
            }
        )
        .startWithValues { store in
            if ViewStore(store).state == nil { `else`() }
        }

    let unwrapDisposable =
        self
        .producerScope(
            state: { state -> Effect<Wrapped, Never> in
                state
                    .skipRepeats()
                    .compactMap { $0 }
            }
        )
        .startWithValues(unwrap)

    return CompositeDisposable([elseDisposable, unwrapDisposable])
}

I don't want to break something else by doing this though, so I'm curious what I might be missing? I'll try digging in to this a bit more if I get the time.

I'm noticing that the ifLet call on the store is called multiple times.

Do you mean that the then: closure is being called multiple times, causing your view controller to be presented multiple times?
When each modal is dismissed are you making sure, in your reducer, that the state for the modal (eg items or favourites) is reset to nil?

The "fix" is to use skipRepeats() instead of .skipRepeats { ($0 != nil) == ($1 != nil) }.

The { ($0 != nil) == ($1 != nil) } comparison test (instead of using the default ==) is to test for changes from nil to non-nil and vice-versa, which is what is needed for ifLet, otherwise any change in the state whilst it's non-nil would also cause the then: closure to be called, which is not what we want.
FWIW I'm using ifLet in my work project and have not run into the problem that you describe.

Thanks for the response! I think I found my issue. I was doing store.ifLet within viewStore.produced.activeRoute.startWithValues. I didn't realize that store.ifLet was creating new subscriptions to the store each time.
viewStore.produced.activeRoute.startWithValues { [weak self] route in
  switch route {
  ...
  case .favorites:
      let store = self.store.scope(state: \.favorites, action: Root.Action.favorites)
          store.ifLet { [weak self] store in
              guard let self = self else { return }
              let vc = FavoritesViewController(store: store)
              let nc = UINavigationController(rootViewController: vc)
              nc.presentationController?.delegate = self
              self.present(nc, animated: true, completion: nil)
          }
    }
}

So every time I got a new route I was attempting to unwrap the optional store, not realizing that the previously unwrapped store was still hanging around. So the previous ifLet block was still getting called.
I think the wording on ifLet makes it seem as if you are doing an in-place unwrap once, but in reality you are creating a new subscription to an observable. I know the wording is inherited from the Point-Free repo, so I guess I can bring that concern up there. I'll close this issue. Thanks again!