undo deletion of fields formerly unsupported but now supported

Certain fields were removed because they were not yet supported in cql_qdm_patientapi.  Those fields have since been supported, so we can remove the deletion.
Pull requests into cqm-converter require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[x] This pull request describes why these changes were made.
[x] Internal ticket for this PR: https://jira.mitre.org/browse/BONNIE-1746
[x] Internal ticket links to this PR
[x] Code diff has been done and been reviewed
[x] Tests are included and test edge cases N/A
[x] Tests have been run locally and pass
[x] If applicable, the library version number in cqm-converter.gemspec has been updated N/A

Bonnie Reviewer:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Cypress Reviewer:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Rubocop failures are not related and are new.  Restarting master branch build revealed same failures.  They should probably be taken care of in a separate PR.


Org users: Support multi org users

Users which belong to multiple organisations will have an organisations array in their app_metadata. When gathering the users associated with a given organisation, this array should also be considered.

 Organisation Switching


Update scalafmt-core to 3.7.6

About this PR
📦 Updates org.scalameta:scalafmt-core from 3.7.4 to 3.7.6
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (3.7.4).
You might want to review and update them manually.
website/package-lock.json

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:n:3

Superseded by #348.


Studio : filter sentences by configuration

Currently, all sentences, no mater from which connector they come, are gathered in the inbox and search tab with no mention on where they come from.
Would it be possible, if we have 2 different connectors (or more) to get the information about the connexion source and add a filter to select them in both tabs ?

Enedis is currently working on that issue


(fix) Prevent github workflows to run twice

https://app.shortcut.com/coinalpha/story/24389/fix-double-run-of-some-github-workflows-in-birdfeeder


how to

i have run the example/basic-setup，and the example have been seen on sketch, but where could I edit the html/jsx code，where is the source files?

react-sketchapp/examples/basic-setup/src/my-command.js

Thanks @houhr :)
@sopaco — in each the examples folders, the source can be found in src/. Hope that helps!


Axios doesn't send cookies with POST and data

Summary
I was beating my head off of the issue reported in #191 for awhile but then I noticed something. Post requests WITHOUT data work fine. Post requests WITH data seem to overwrite some config settings.
#### Context
Specifically for me, all my AJAX requests (which are CORS-protected) need a cookie to get past my authentication provider. Given that, I had set withCredentials to true in my axios instance. Everything was fine for GET and data-less POST requests. But as soon as I tried to post some data, I got hit with CORS errors again. Upon inspection, for some reason Axios stops sending the cookie header (and thus the auth token) when I specify data. The lack of the cookie means the server rejects the request (returning a 302 in my case) which the browser reports as a CORS failure, oddly enough.
I saw the message from @mike-robertson on #191 about using json.stringify. However, my data must be urlencoded. I used the query-string package to encode the parameters directly. It worked after that!
In summary, given:
import axios from 'axios';
import queryString from 'query-string';

const api = axios.create({
	withCredentials: true
});

params = {
   foo: 'bar',
   baz: 3
};

then the following will not send the cookie header.
api.post('bluh', params);

but these will:
api.post('bluh');
api.post('bluh', queryString.stringify(params));

This seems like a pretty major problem and should at least be mentioned in the docs.

axios version: v0.16.1
Environment: node v7.6.0 on KDE Neon
Chrome info:

Google Chrome	58.0.3029.81 (Official Build) (64-bit)
Revision	ac0bae59f0aa5b391517e132bf172144b1939333-refs/branch-heads/3029@{#746}
OS	Linux
JavaScript	V8 5.8.283.32

I had a similar issue, doing a a post-request  same origin axios with default configuration sent the auth-cookie, same setup as cors from other domain axios did not send the auth-cookie until I set the configuration to axios.defaults.withCredentials = true.
Would be better to have same behaviour on both types of request.

I had a very similar issues, what I understood is that the OPTIONS request that is issued before the POST request (I was doing cross-site requests) does not carry the cookie header.
If your server does not authorize OPTIONS request to be unauthenticated then he rejects it and your browser thinks that Cross-origin is denied.
I changed my server configuration to authorize unauthenticated OPTIONS requests and it solved my problem.

I had the same problem with post not sending cookies!
I solved my problem by doing
axios("http://mysite.com/api/things/", {
  method: "post",
  data: someData,
  withCredentials: true
})

For those finding this and still having trouble - my comment here may help

@rubennorte - why is this closed?

Do we have a solution for this?  As @Xcelled stated, axios is not sending cookie unless I do JSON.stringify? I do not  want to do that.

Axios & cookie

`withCredentials: true

https://developer.mozilla.org/en-US/docs/Web/API/XMLHttpRequest/withCredentials`
https://stackoverflow.com/questions/43002444/make-axios-send-cookies-in-its-requests-automatically/43178070#43178070
https://stackoverflow.com/questions/40941118/axios-wont-send-cookie-ajax-xhrfields-does-just-fine/43676021#43676021

{
    withCredentials: true
}

https://www.axios.com/tracking-cookies-are-dead-9c316a2c-33c2-40b8-9801-069df07593a8.html

Just in case anyone else stumbles across this thread "googling" for a solution.
If you are wanting to pass a struct as a set of form fields like you might do in jQuery you would likely be trying something like this when using axios (this will not work and exhibits the behavior titling this issue):
config = {url:'http://somedomain', data:{myfield:"myvalue"} } axios.request(config);
Instead of a struct pass a FormData object as follows:
formData = new FormData();  formData.append('myfield','myvalue');    config = {url:'http://somedomain',                  data:formData                 }    axios.request(config);
Doing the above allowed me to post my form data cross origin while also ensuring cookies were being passed along for the ride.  (sorry about the formatting of code github refused to do line breaks)
Hope this helps someone.

Having the same issue. And can't  have the pre-flight request be unauthenticated. The comment by @quicken is not a fix. Is there a status for this issue?

@jorgehaddadsonos are you making a cross domain request? If so make you set your CORS correctly. @quicken's fix worked for me.

Sending a post request with the data field set to the stringified version of the body I want to send works! However, if I change the method to PUT for the same request (I created the PUT endpoint as well) I run into the same pre-flight request issue again. Any idea why a PUT request would be different that a POST?

I had the same problem. this is what I did:
axios.get('url', {withCredentials: true});

And it solved my problem.
you have to set withCredentials option to true, then the axios will accept the sessions, cookies, etc.

solved withCredentials:true
axios({
  url:"http://example.com",
  mehtod:"POST",
  data: qs.stringify(form),
  withCredentials: true
});

It doesn't work with PUT. I believe we're just making the POST call simple.
On Fri, Apr 17, 2020 at 6:54 AM Fabiel Leon<EMAIL_ADDRESS>wrote:

solved withCredentials:true
axios({
url:"http://example.com",
mehtod:"POST",
data: qs.stringify(form),
withCredentials: true
});
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/axios/axios/issues/876#issuecomment-615257203, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AIA3UITPMBP6HDVCTRE7X43RNBNQDANCNFSM4DJRV2FA
.


Breadcrumbs: Misaligned in 4.5.0

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Go to breadcrumbs
Notice the misalignment

Expected behavior
No misalignments (see v4.4.0 and below)
This may be related to changes introduced in https://github.com/cengage/react-magma/pull/1304 (display: inline-flex;)

Confirmed that Breadcrumbs and Hyperlinks with icons are aligned and look as expected


EfficientFace trained FER models?

Hi! I see that you recently shared several LDG models trained for FER. Do you have a plan to also share the trained EfficientFace models trained on any FER datasets as per paper?
Thanks!

Hi! I see that you recently shared several LDG models trained for FER. Do you have a plan to also share the trained EfficientFace models trained on any FER datasets as per paper?
Thanks!

The trained EfficientFace models have been released, please check in the README.
Best,
Zeke


Line points are changing style (or changing series data reference) when itemStyle is dynamically added into series data

Version
4.2.1
Reproduction link
https://jsfiddle.net/3zmvuqyh/1/
Steps to reproduce
I tried to use two different type charts with the same data object and add itemStyle to only one of them (column type), but itemStyle was added to both charts, thus changing the line point colors in the line chart.
What is expected?
that the chart does not change the reference value and two charts have different data
What is actually happening?
Apparently the data reference is changing when added to a chart. The line chart is with the itemStyle property. It doesn't ignore the if statment, it just seems that the graphicData reference is changing.

Sorry I don't quite understand what you mean. Besides, in echarts there is no series named column, so the if statement maybe not work as you expected.


[Ethernet] Replaced all assert statements for Expects...

Hi, this is nice, just remember that ./api is allready on the include path, so the correct include statement for common is simply #include <common>. Also the reason why we take a while to respond to your PR's is simply a lack of automated testing. That's likely to be right around the corner though- @mnordsletten set up a brand new jenkins yesterday.


Gogs doesn't return memory after big commit pushed.

Gogs version (or commit ref): 0.9.97
Git version: 1.7.1
Operating system: Centos6.5
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist (usually found in log/gogs.log):

Description
Our server has only 2GB of ram.
During 450MB size commit pushed, gogs uses large memory and it doesn't return the memory after the push finished.
So we encounter 500 error(when exceeded memory), or service becomes slow down.

Hi, version too old, please upgrade.


Remove use of ember-get-config to get the current config

We have an app with a different configuration between test and development/production. We autoclear the notification on production, but not on tests to test the content of the notification.
Once we switched to embroider, we had some issues when running tests on the /tests route in the browser. The notifications started to disappear.
As the addon use ember-get-config, it seems of always get the development/production config instead of the test one. So I switched to ember way to do the same.

I'm not too sure about making this change because it would have other effects 🤔 the issue you're having in embroider is due to be fixed in https://github.com/embroider-build/embroider/pull/1412 tho

Curious to know about the side effects.
Good to know that it's an embroider bug.


Document how to do self-templating

Self-templating templates suffer from a :chicken: and :egg: problem that disallows them to use a valid copier-answers file. A workaround is documented in https://github.com/copier-org/autopretty/pull/2, but it would be better to improve this.
There seems to be some bug regarding how files/dirs ended with .tmpl override the ones without that extension.
Examples:

If my answers file is in .copier-answers.autopretty.upstream.yml and *.upstream.* is in _exclude, then it shouldn't be copied. But since it's my answers file, I need it updated. What to do?

Finally managed to do it. It was as simple as using _subdirectory 🤷🏼♂
See https://github.com/copier-org/autopretty/tree/65aeb34041cbe2281c57be0c04635ed2c61c4c80#notes-for-template-maintainers
Gotta document this.

Finally managed to do it. It was as simple as using _subdirectory 🤷🏼♂
See https://github.com/copier-org/autopretty/tree/65aeb34041cbe2281c57be0c04635ed2c61c4c80#notes-for-template-maintainers
Gotta document this.

Fixed in df2a3016b2778094d5cc32de305329f083e97fff.


Face Mask Detection

Proposal
A face mask detection model using computer vision.

I'm a GSSoC'23 contributor and I would like to work on it. Please assign this to me.

@ranodeepbanerjee - a similar issue has already been assigned (#31). Hence, we cannot assign you this issue.

I am a GSSoc'23 contributor .I would like to work on this project . Kindly assign this issue to me .


Cannot set a object's fields to a null value

i have created a object using $Instance with non final fields but when i try to set the property to null value i am getting this error.
(the nullable value is returned by function using try catch)
dart_eval runtime exception: type 'Null' is not a subtype of type '$Value' in type cast
#0      SetObjectProperty.run (package:dart_eval/src/eval/runtime/ops/objects.dart:225:57)
#1      Runtime.bridgeCall (package:dart_eval/src/eval/runtime/runtime.dart:838:12)
#2      EvalStaticFunctionPtr.call (package:dart_eval/src/eval/runtime/function.dart:86:13)

My bad I was returning null instead of $null()


Investigate substrate v3 framework

Docs: https://docs.substrate.io/v3/getting-started/overview/
Developer api docs: https://docs.rs/sc-service/0.9.0/sc_service/

Closing


Add Portal support

I’m happy to add tests, but i don’t really understand the test suite nor where the appropriate place to add them is for the v16 only feature.
@aweary from what I can tell the Portal fiber is similar to a host component in that it’s essentially a transparent wrapper over it’s rendered children. It does hold some host container info which may be required for actual DOM traversal but in terms of the RST tree this should give an accurate representation as React sees it.

Hey folks, any news here? I really want to enable tests for my React portal components 👍

Are any efforts being made to provide portal support? This seems to be the final bit in migrating our code base (and its Enzyme tests) to React 16.

I've merged master into this PR and tests seem to work.
Could you rebase to master and see if it passes?

@jquense please rebase PRs from the command line when updating them, so as to avoid merge commits :-)

@ljharb Sorry, it's not that i'm unsure how/where to write a test, but how to write ones that sufficiently test the change to ya'lls satisfaction. A single test that does a find through a portal will exercise the code path, but not particularly aggressively. If ya'll ok with none of the other traversal methods hitting this that seems fine, if not i'm not sure you want me to rewrite every test to include a portal either.
thanks for the review!

I think that the simple test you describe would be the minimum of what we'd want; ideally, we'd write tests for all the traversal methods, to ensure that none of them throw.

@jquense Are you planning on writing the test? If not, I could take a crack at it.

I'm planning on it but a bit busy, happy to get some help :)


[Template/DR] Minio is not scheduled on infra nodes

Minio's statefulset is missing the node selectors / affinity / tolerations to get scheduled on the infra nodes.
Here's a kustomize patch that can be used for the fix:
apiVersion: apps/v1
kind: StatefulSet
metadata:
  name: minio
  namespace: kube-system
spec:
  template:
    spec:
      affinity:
        nodeAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            nodeSelectorTerms:
            - matchExpressions:
              - key: node.kubernetes.io/role
                operator: In
                values:
                - infra
      tolerations:
      - effect: NoSchedule
        key: node.kubernetes.io/role
        operator: Equal
        value: infra

fixed in #233


Check meson version before build dpdk.

@abhimarathe use this issue to address comment in #1134

@LiliDeng  is this fixed in your PR?


Still having this issue, https://github.com/Microsoft/vscode-python/issues/177

Environment data

VS Code version: XXX
Extension version (available under the Extensions sidebar): XXX
OS and version: XXX
Python version (& distribution if applicable, e.g. Anaconda): XXX
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Relevant/affected Python packages and their versions: XXX

Expected behaviour
XXX
Actual behaviour
XXX
Steps to reproduce:

XXX

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
XXX

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
XXX

Please could you fill out the issue template.

it would be great, if you change static link to issue id like this ➜ #177

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


Error message: 'collections.OrderedDict object' has no attribute 'use_pypi_deployment_with_travis'

Describe the bug
Trying to create a new package and the template crashes in the interactive creation.
To Reproduce
Run the command on my local.
Expected behavior
The command finishes without crashing.
Screenshots
The command crashes after Pycharm question
Unable to create file '.travis.yml'
Error message: 'collections.OrderedDict object' has no attribute 'use_pypi_deployment_with_travis'
Context: {
    "cookiecutter": {
        "_output_dir": "/Users/moo/code/ts",
        "_template": "https://github.com/johanvergeer/cookiecutter-poetry.git",
        "command_line_interface": "none",
        "create_author_file": "n",
        "create_docs": "y",
        "email"<EMAIL_ADDRESS>        "full_name": "Mikko Ohtamaa",
        "github_username": "miohtama",
        "module_name": "top",
        "open_source_license": "MIT license",
        "project_name": "Top Framework",
        "project_short_description": "Framwork for building UNIX top-like TUI applications easily",
        "project_slug": "top-framework",
        "pypi_username": "miohtama",
        "use_pycharm": "y",
        "version": "0.1.0"
    }
}

Desktop:

OS: macOS
Python 3.9

I have the same problem (with https://cookiecutter-poetry.readthedocs.io/en/latest/readme.html).


Add NUnit Adapter Runsettings file

Add to the default template and add an item template

The file I added to the demo project works for VS 2013 and 2015. There are additional RunConfiguration settings in 2015. If you add them then an exception is thrown in 2013. I should go back and test 2012 to see what works there.
The NUnit section isn't validated at all and of course any test run parameters are currently ignored.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
22 seconds
clone-repository

✅ Succeeded
2 minutes
build-container

✅ Succeeded
16 seconds
deprecated-base-image-check

✅ Succeeded
17 seconds
inspect-image

✅ Succeeded
14 seconds
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
15 seconds
label-check

✅ Succeeded
16 seconds
optional-label-check

✅ Succeeded
10 seconds
show-summary


Capped collection is not created

I call:
var log = new LoggerConfiguration()
    .WriteTo.MongoDBCapped("mongodb://mymongodb/logs", collectionName: "rollingapplog")
    .CreateLogger();

Expected: MongoDBCapped method creates Capped collection.
Actual: It doesn't create a capped collection. A common collection is created at the moment when some log writes to the collection.
Root cause description:
I have debugged your code and found that MongoDBCapped method verifies if the collection already exists. If not, then it creates a capped collection.
It seems that the following method is broken. database.GetCollection always return some object. And it always returns true even for nonexistent collection.
internal static bool CollectionExists(this IMongoDatabase database, string collectionName)
        {
            var collection = database.GetCollection<BsonDocument>(collectionName);
            return collection != null;
        }

I use:
MongoDB v 4.0.5
MongoDB driver 2.7.3
Serilog.AspNetCore 2.1.1
Serilog.Settings.Configuration 3.0.1
Serilog.Sinks.Console 3.1.1
Serilog.Sinks.ElasticSearch 7.1.0
Serilog.Sinks.MongoDB 4.0.0
Serilog.Sinks.SyslogServer 1.0.10

Yep, I verified this as well. If you create the collection ahead of time with a cap in place, it works fine.
It's a workaround, but it'd be nice if it works out of the box.

I created a pull request to fix this issue. It would be quite great if it could be reviewed (https://github.com/serilog/serilog-sinks-mongodb/pull/54).

Fix is in latest dev.


Add types to API responses

This is not quite ready to be merged. I'm looking for an example of the response when there is a poll. Once I have that out I'll update this PR.

Thanks a million for this! Here's an example of a poll. The item response has a poll object that looks like this:

For some reason, the API only returns the poll results, so we trigger more requests to get the actual poll content. For example, if the item id === 1 and has 2 poll choices, then the contents of each poll choice can be retrieved from /2 and /3. (All this is done here)

I've added typing for the polls. As the commit mentions I moved the poll result fetching into the HN API Service. In general controllers should be unconcerned about how the API works. By moving fetching poll details into the service we maintain this separation of concerns.

Agreed, poll fetching shouldn't have been in the controller in the first place.
Genuinely can't thank you enough for this! This is amazing :)


bump up protobuf major version

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.hanabi1224 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add data source, dot syntax for server and fix crash when port is missing (fallback on the default port)

Multiple fixes:

Parse of data source= in AdoConfig which is the same as server=
Support for dot syntax: . which is the same as localhost
Fix crash when port is missing inside the server address. Fallback on the default port which are:

1433 for std tcp connection
1434 for resolving instances

I have add tests to reflect the changes.

Great. Btw, I have this PR for the ports I'd like to merge, so maybe rebase yours when I'm done and we'll merge this afterwards.
https://github.com/prisma/tiberius/pull/147

@pimeys Sure!

Can we merge this pull request?

Can you rebase and fix conflicts? See also the default port handling that has changed in the master. You might want to change your pull request to adapt to those changes.


[KPI] Litigation rate

Litigation rate = number of order lines in dispute / total number of orders


[AutoDiff] Add Array.zeroTangentVectorInitializer.

Array.zeroTangentVectorInitializer produces a TangentVector of zeros with
the same count as self.

@swift-ci Please test tensorflow


Simplify, clarify, sync style examples/mnist_net2net.py

This PR contains no semantic changes.

Build teacher network only once. It's faster, clearer, produces less output.
Sync style and naming conventions with the rest of examples/
Remove unused and duplicated code.
Simplify and clarify logic, docs, output.
Remove usage of literals for epochs.

@dolaameng, would you like to review these changes?

@ozabluda @fchollet sorry for my late reply. Yes most of changes make the code easier to read. I have one comment on removing the train_data and test_data parameters from the model maker methods. Previously my idea was to refer to global variables in the experiment methods but use the parameters in the model make method, so that people can write their own experiments on new data with the code. What do you think?

In abstract, passing arguments, instead of using globals is a better style in large projects (but this is just one non-coupled example, and is not true of singletons), and it would indeed make it easier for people to write their own experiments which do need those parameters.
But in this case, my thinking is that the overriding concern is the ease of understanding of just one isolated example. Before, one had to chase through multiple levels of indirection to see that those are just one copy of constant globals being passed around.

emm. I have a different view on this. Giving the full context of a method in one place, including all its dependencies in the parameter list actually helps me better track it. Otherwise I may need to jump around to find where its dependencies are created. Besides, putting them in the parameter list gives a chance to document their relations to the method.

I agree in principle and in abstract, but, for example, you normally don't pass global constants (or singletons) as parameters. In this case, they are indeed constants.

By constants do you mean the dataset and teacher models? I don't think they should be singleton or constants. For example, people may create multiple teacher models to compare their performances right? And you may also want to experiment with different datasets too sometime, e.g. to see how different data processing or augmentation works on the net2net?

By constants do you mean the dataset and teacher models?

yes,

I don't think they should be singleton or constants.

But they already are constants, since they are never modified after initialization. To me, in this example, they are no different than the constant epochs=3. Of course, in prevalent Python style, they are constants in spirit only (but it helps that in Python, one can't modify a global in a method, without an explicit 'global' declaration, or it'll shadow). To make them actual constants, we'd have to do something like:
https://www.safaribooksonline.com/library/view/python-cookbook/0596001673/ch05s16.html

For example, people may create multiple teacher models to compare their performances right?

Absolutely, it's easier to modify the code in the way you describe, after understanding it. I just though the example is easier to understand with globals, which is the overriding concern IMHO. I was also thinking that most people will read this example for network surgery purposes, not to study transfer learning, but who knows.

And you may also want to experiment with different datasets too sometime, e.g. to see how different data processing or augmentation works on the net2net?

I think these would be identical, either way. For example, dataset is changed in exactly the same way in exactly the same place (where the globals are initialized).

@ozabluda I understand where you were from. What you put above is more related to a personal choice of style, which may never be made to be satisfying to everyone in a.community.  I think I will stick with my opinions unless you have other reasons beyond this. But I am open to others options on this case. If more people think the code should be modified this way, I will welcome it.

Just committed, what I think is a way to pass globals as arguments into functions, while making it more obvious that it's just globals being passed around, by using standard names throughout. I think it's the best of both worlds.
BTW, you didn't pass train and validation sets to net2wider_experiment() and net2deeper_experiment() either :-). With this approach, we could do that without much downside.

@ozabluda thanks for making the changes. They look better to me!

yeah I agree a good naming convention saves a lot of troubles
My understanding of the code is that there are two parts in the example file. The make methods are reusable part to implement net2net, and the experiment parts are kinda like "scripts" for bench-marking. Unfortunately we have to mix them in one single file for simplicity. That's why experiment methods don't have parameters because they should be standalone, e.g., by using global or env vars. Again different people may have different styles.
One last comment for the change: since we have removed the 'epochs' (and should it be better named n_epochs?) out of the parameter list, it might imply to readers that all those methods should be using the same # of epochs, which is not true. The original paper didn't propose a step-wise comparison, instead, the benefit of certain methods can only be observed in a longer window. Do you think we can make it explicit by adding a doc comment?

since we have removed the 'epochs' (and should it be better named n_epochs?)

it's call "epochs" throughout Keras.

out of the parameter list, it might imply to readers that all those methods should be using the same # of epochs, which is not true. The original paper didn't propose a step-wise comparison, instead, the benefit of certain methods can only be observed in a longer window. Do you think we can make it explicit by adding a doc comment?

I do think I should put epochs parameter back. Now epochs does look like a weird exception, as in "we pass everything, except (for unclear reason) epochs, which is hard-coded'. Also the most logical place to make the decision about the epochs is in *_experiment() methods, as it could have different values there.

Agreed on both.

Added epochs parameter back, but removed its default value in make_* functions. I think it best reflects the logic discussed above - one is forced to make this decision in the  experiment() methods. Passing epochs to make_teacher_model() may or may not be logical, but making it identical to make_student_model() make it easier to understand IMHO.

@ozabluda @fchollet The changes look all good to me. Thanks!


Llamaguard returns is_safe logprob in annotation

@bkorycki do you have a plan to close these done tasks?
so we can check and help those todo tasks more clearly

Hi, @HuaizhengZhang! We are managing our work through a Github project, not through the issues. I would like it if GitHub automatically closed issues when they were marked Done in the project, but I couldn't find a way to do that. We're also working energetically on this and related code as a full-time team that talks a lot via video call and discord, so GitHub issues is very much secondary to that.
If you're looking to get involved, maybe tell us a bit about what you're interested in and what motivates you? We could find some things related to that. And if you'd like, get you into the AI Safety Working Group Discord.

Hi @wpietri , I am interested in the project and want to contribute something.
I am a founding engineer from an AI eval and safety company. This project is tremendous and super related to my current work. I hope I can join the development to exchange some ideas/designs / etc to better serve customers/community / etc

That sounds great, @HuaizhengZhang. How about you join our discord here: https://mlcommons.org/working-groups/ai-safety/ai-safety/
Then we can book a call with the team so we can all discuss the best ways for you to get involved.

@wpietri

Joined but could only see this


bundler could not find compatible versions for gem 'activerecord'

rails version is 5.1.1
activerecord version is 5.1.1
How can I use this gem in rails 5.1.1 project?

It is not possible at the moment.

I thought this wasn't resolved, but yes, there is a release 4.1.2 that covers rails 5.1.2

Yeah. We didn't do a very good job cleaning up all the random issues after the last release


background_jobs segment : add number of jobs if >1

I think it should be the default behavior, but I've added POWERLEVEL9K_BACKGROUND_JOBS_VERBOSE var to disable it :)

Cool idea! :)

@V1rgul - I love this! Excellent feature.
Can you add a commit that updates the README.md to document the new verbose variable?

@bhilburn updated :)

Great job, @V1rgul!! Thanks for the excellent contribution!


Handle timestamps correctly

This fixes #190.
However, after this there is another issue related to Laravel.
Read more.

The other issue that we have come by is because Laravel uses Doctrine/DBAL.
However they do not want to support MySQL-specific column types like timestamp.
Therefor I had to build a Laravel Package for it, but this solves the other issue.


Adopt https://www.contributor-covenant.org/version/1/4/code-of-conduct.html

I encourage the OpenSSL project to adopt the widespread https://www.contributor-covenant.org/version/1/4/code-of-conduct.html

also for your consideration: http://lists.gnu.org/archive/html/info-gnu/2018-10/msg00001.html

Oh, I do like RMS' take on it!

+1 to the way RMS did it for GNU project.

Closing this. If the project wants to change it will have to do so because of internal feelings, not opinions from outsdiers.


Check if a drag or resize is valid or not.

Created a callback function isValidMove(event, $element, dimensions) which is called on dragstop and resizestop to check if the new position/size is valid or not.

Closing as this now polluted with other changes as well.


Upgrade to Storm API 1.0.2

Support modern Storm API.
Can be merged after chill, scalding and tormenta's releases and bumping their versions.

what are the plans on this branch @ttim ?

Updated this PR: now using Storm's completeTopology to avoid Thread.sleep.
@johnynek I'm planning to merge it after tormenta's release.

@johnynek Travis is green after I disabled mima check.
I'm unsure should I merge this change or not, because even it's green it contains two versions of kryo at the same time (I don't think it breaks anything in most cases, but still). From other point it would be a little bit more convenient for me to have this branch merged. What do you think?

Yeah, maybe it is good to merge this when we scalding with kryo 0.8.3
On Tue, Jan 17, 2017 at 16:53 Timur<EMAIL_ADDRESS>wrote:

@johnynek https://github.com/johnynek Travis is green after I disabled
mima check.
I'm unsure should I merge this change or not, because even it's green it
contains two versions of kryo at the same time (I don't think it breaks
anything in most cases, but still). From other point it would be a little
bit more convenient for me to have this branch merged. What do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/twitter/summingbird/pull/699#issuecomment-273368687,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJdqDmIg1kHa58onXmKn2zt2A-ma-fks5rTX68gaJpZM4KvWsn
.

Codecov Report

Merging #699 into develop will increase coverage by 0.11%.
The diff coverage is 87.5%.

@@             Coverage Diff             @@
##           develop     #699      +/-   ##
===========================================
+ Coverage    70.85%   70.96%   +0.11%
===========================================
  Files          141      141
  Lines         3479     3475       -4
  Branches       202      201       -1
===========================================
+ Hits          2465     2466       +1
+ Misses        1014     1009       -5

Impacted Files
Coverage Δ

...witter/summingbird/storm/spout/KeyValueSpout.scala
77.77% <ø> (ø)
:arrow_up:

.../com/twitter/summingbird/storm/StormPlatform.scala
82.35% <ø> (ø)
:arrow_up:

...witter/summingbird/storm/FlatMapBoltProvider.scala
100% <ø> (ø)
:arrow_up:

...la/com/twitter/summingbird/storm/StormMetric.scala
0% <ø> (ø)
:arrow_up:

...rd/storm/collector/AggregatorOutputCollector.scala
90.9% <ø> (ø)
:arrow_up:

...tter/summingbird/storm/option/FlatMapOptions.scala
50% <ø> (ø)
:arrow_up:

.../twitter/summingbird/storm/StormStatProvider.scala
92.85% <ø> (ø)
:arrow_up:

...ter/summingbird/storm/spout/TraversableSpout.scala
100% <ø> (ø)
:arrow_up:

.../twitter/summingbird/storm/StoreStatReporter.scala
0% <ø> (ø)
:arrow_up:

...scala/com/twitter/summingbird/storm/BaseBolt.scala
77.1% <ø> (ø)
:arrow_up:

... and 6 more

Continue to review full report at Codecov.

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

@johnynek We released scalding so I think it's time to merge this PR (as soon as tests will pass). How does it sound for you?
After that I'm going to make one more PR (https://github.com/ttim/summingbird/tree/tabishev/bump_scalding_algebird_versions) which contains @piyushnarang changes for algebird (https://github.com/twitter/summingbird/pull/715) as well as scalding version bump.
After that as soon as @pankajroark will release new storehaus version we'll be able to do SB release, I guess with scala 2.12 support as well.

Yeah, I think that all sounds good.
On Wed, May 10, 2017 at 14:00 Timur<EMAIL_ADDRESS>wrote:

@johnynek https://github.com/johnynek We released scalding so I think
it's time to merge this PR (as soon as tests will pass). How does it sound
for you?
After that I'm going to make one more PR (
https://github.com/ttim/summingbird/tree/tabishev/bump_scalding_algebird_versions)
which contains @piyushnarang https://github.com/piyushnarang changes
for algebird (#715 https://github.com/twitter/summingbird/pull/715) as
well as scalding version bump.
After that as soon as @pankajroark https://github.com/pankajroark will
release new storehaus version we'll be able to do SB release, I guess with
scala 2.12 support as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/twitter/summingbird/pull/699#issuecomment-300645243,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJdlqSdRX2WVLkhkJ-QorzpRp2QYBSks5r4k-pgaJpZM4KvWsn
.


Document how to install dependencies for tests (Fixes #919)

Hi, I ran into the same issue described here: https://github.com/spf13/hugo/issues/919
And @tatsushid's solution worked for me too. This commit adds the command to the documentation.

+1

Hello @egonSchiele,
thank you for submitting this addition. I've merged your contribution as 51ee3fc73595a86ecf0b69ab6a4f27f56159224f (master) and 2fd9d71f4484ed21970efeb4fab5e2106d646b1a (v0.15.docs).
Cheers,
Digitalcraftsman

Cool, thanks @digitalcraftsman !


Adding collections handler

Implemented the following HTTP endpoints for collection operations.
PART-A - Managing collections:

GET /collections: List all collections (handled by HandlerGetCollections).
GET /collections/{collectionName}: Get collection by name (handled by HandlerGetCollection).
POST /collections/{collectionName}: Create a new collection (handled by HandlerCreateCollection).
DELETE /collections/{collectionName}: Delete a collection (handled by HandlerDeleteCollection).

PART-B - Add/remove data collections:

GET /collections/{collectionName}/get/{key}: Retrieve a value by key from a specified collection (handled by HandlerCollectionGetById).
GET /collections/{collectionName}/items: Retrieve all collection items (paginated - pageSize and page parameters)
POST /collections/{collectionName}/set: Set key-value pairs in a specified collection (handled by HandlerCollectionSet).
DELETE /collections/{collectionName}/delete/{key}: Delete a key from a specified collection (handled by HandlerCollectionDelete).
GET /collections/{collectionName}/get/{key}: Retrieve a specific key-value from a collection (handled by HandlerGetCollection).

Hi @vdmitriyev, I know that you're busy but I'm going ahead with some new developments.
Is it okay for you if I start to merge this PR and then (when you have time) you can review it and add some comments?
Thanks

Don't worry, I really appreciate your contribution.
I'll wait, no problem.
Thanks

@dmarro89 Hi
I started to review your updates from the perspective of an external user. Initially I wanted to use newly a create API by calling them from 3rd party tool. In this case it is a Python script I have added here: examples/python/daredb_tester.py.
The tests have been divided into two parts, I have edited our initial comment to establish this parts:

"PART-A - Managing collections"
"PART-B - Add/remove data collections"

I could successfully test the part "PART-A", which includes management of collections.
However, I was not that successfully with the part "PART-B". Thus, I could not add/remove data from a custom collection. Here is a error I see in the Python script, which calls API:
DEBUG - Populate DB with sample data via URL (collection: sample): https://<IP_ADDRESS>:2605/collections/sample
ERROR - HTTP Code: 400; content: b'Collection "sample" already exists\n'
ERROR - HTTP Code: 404; content: b'404 page not found\n', url: https://<IP_ADDRESS>:2605/collections/sample/getAllItems

Do you have an idea, why the database says that the collection exists, but still cannot access it.
P.S. I have also added a Taskfile to the current PR. It is just there to simplify the development workflow. See README.md for further details.

Non I'm realizing that I've not updated the PR description, sorry.
Let me update the description.
Thanks
@vdmitriyev

If you allow, I can update the python script to test the database.
Anyway I'm developing a UI interface for the database -> Here's the repo (but I haven't commited anything yet).
@vdmitriyev

@dmarro89 thanks for fast response.
Now I see that endpoints have a different names 👍
If you would like to change the Python script, just do it 😃 Now, when it is clear, that the wrong APIs have been used, I could also adapt the script.
In parallel to UI client, I would suggest to create and maintain a OpenAPI yaml file as part of the main DareDB repo, so it would be easier to evolve APIs. I will create an issue and prepare a initial draft

@dmarro89 I have also noticed, that you have adapted a subset of the suggestions listed here - https://github.com/dmarro89/dare-db/discussions/34#discussioncomment-9829371

#34 (comment)
Just to explain @vdmitriyev
Actually I've "refactored" the list of commands that you've proposed but they are all implemented.

[ ] List all collections: GET request: http://localhost:2605/collections/list/ -> I have removed the "list" from the path because it is redundant, the http method already explains the action we're doing -> GET /collections
[ ]  Create a new collection: GET request: http://localhost:2605/collections/create/:collection -> I have removed the "create" from the path because it is redundant, the http method already explains the action we're doing -> POST /collections/{collectionName}
[ ]  Get size of a collection: GET request: http://localhost:2605/collections/size/:collection -> I have merged this with the endpoints that returns all the items. you can get the size simply getting the length of the array returned.
[ ]  Iterate of a collection: GET request: http://localhost:2605/collections/iterate/:collection?page=1&pagesize=1000
page: current page with keys -> -> I have merged this with the endpoints that returns all the items
[ ]  Delete collection: DELETE request: http://localhost:2605/collections/delete/:collection -> I have removed the "delete" from the path because it is redundant, the http method already explains the action we're doing -> DELETE /collections/{collectionName}

@dmarro89 thanks for fixing the Python script. Have also answered your commend on API design here - https://github.com/dmarro89/dare-db/discussions/34#discussioncomment-10995856

As long as an API design will be finalized, the PR can be accepted and merged


Flutter web: image not showing(Unable to load asset) on web hosting

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Issue reproducible on latest stable channel. (i use flutter 3.10.2)
(Also try old flutter version like 3.0.5 still have same issues)

Create a new Flutter project
Create "assets" directory in the root of project
The assets directory is connected to the app via pubspec.yaml file like so:

assets:
    - assets/images/flutter_logo.png

flutter build web and deploy to firebase (or other web Hosting)

Running the app in localhost these assets (images) works correctly.
Running the app on real website hosting that use these assets fail.
I try put additional "assest" in url path the image is appear, So i think it point to worng path
if not use "assets" at root of image folder it works fine

Expected results
should works same as localhost
Actual results
Unable to load asset, Image failed to load
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});
  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Image.asset("images/flutter_logo.png"),
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headlineMedium,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}

pubspec.yaml
name: web_assets
description: A new Flutter project.
# The following line prevents the package from being accidentally published to
# pub.dev using `flutter pub publish`. This is preferred for private packages.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev

# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# In Android, build-name is used as versionName while build-number used as versionCode.
# Read more about Android versioning at https://developer.android.com/studio/publish/versioning
# In iOS, build-name is used as CFBundleShortVersionString while build-number is used as CFBundleVersion.
# Read more about iOS versioning at
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
# In Windows, build-name is used as the major, minor, and patch parts
# of the product and file versions while build-number is used as the build suffix.
version: 1.0.0+1

environment:
  sdk: '>=2.17.0 <4.0.0'

# Dependencies specify other packages that your package needs in order to work.
# To automatically upgrade your package dependencies to the latest versions
# consider running `flutter pub upgrade --major-versions`. Alternatively,
# dependencies can be manually updated by changing the version numbers below to
# the latest version available on pub.dev. To see which dependencies have newer
# versions available, run `flutter pub outdated`.
dependencies:
  flutter:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^1.0.2

dev_dependencies:
  flutter_test:
    sdk: flutter

  # The "flutter_lints" package below contains a set of recommended lints to
  # encourage good coding practices. The lint set provided by the package is
  # activated in the `analysis_options.yaml` file located at the root of your
  # package. See that file for information about deactivating specific lint
  # rules and activating additional ones.
  flutter_lints: ^2.0.0

# For information on the generic Dart part of this file, see the
# following page: https://dart.dev/tools/pub/pubspec

# The following section is specific to Flutter packages.
flutter:

  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

  # To add assets to your application, add an assets section, like this:
  assets:
    - assets/images/flutter_logo.png

  # An image asset can refer to one or more resolution-specific "variants", see
  # https://flutter.dev/assets-and-images/#resolution-aware

  # For details regarding adding assets from package dependencies, see
  # https://flutter.dev/assets-and-images/#from-packages

  # To add custom fonts to your application, add a fonts section here,
  # in this "flutter" section. Each entry in this list should have a
  # "family" key with the font family name, and a "fonts" key with a
  # list giving the asset and other descriptors for the font. For
  # example:
  # fonts:
  #   - family: Schyler
  #     fonts:
  #       - asset: fonts/Schyler-Regular.ttf
  #       - asset: fonts/Schyler-Italic.ttf
  #         style: italic
  #   - family: Trajan Pro
  #     fonts:
  #       - asset: fonts/TrajanPro.ttf
  #       - asset: fonts/TrajanPro_Bold.ttf
  #         weight: 700
  #
  # For details regarding fonts from package dependencies,
  # see https://flutter.dev/custom-fonts/#from-packages

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs
Failed to load resource: the server responded with a status of 404 ()

Flutter Doctor output
Doctor output
[!] Flutter (Channel stable, 3.10.2, on macOS 13.1 22C65 darwin-arm64, locale en-TH)
    • Flutter version 3.10.2 on channel stable at /Users/xxx/Documents/development/flutter
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/2.16.1/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/xxx/Documents/development/flutter. Consider adding /Users/xxx/Documents/development/flutter/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9cd3d0d9ff (2 days ago), 2023-05-23 20:57:28 -0700
    • Engine revision 90fa3ae28f
    • Dart version 3.0.2
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/xxx/Library/Android/sdk
    • Platform android-33, build-tools 30.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.78.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.64.0

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.
Process finished with exit code 0

Hi, are you setting abase-hrefin your build line? If so, can you share the parameter you're passing into it?
flutter build web --base-href "/path/"
Also, can you share the high level directory structure for you website, starting at the root?
Example:
htdocs/
     myapp/
          index.html
          assets/

Thanks!

Hi, are you setting abase-hrefin your build line? If so, can you share the parameter you're passing into it?
flutter build web --base-href "/path/"
Also, can you share the high level directory structure for you website, starting at the root? Example:
htdocs/
     myapp/
          index.html
          assets/

Thanks!

Affter create project i leave everything to default and didn't setting base-href
web > index.html
<base href="$FLUTTER_BASE_HREF">
than copy all file in build folder to deploy on firebase web hosting

Sorry i don't know how to see high level directory structure of the website that deploy on firebase hosting

Hi, is there a specific reason you've specified"build/web"as your root folder instead of the default"public"?

Can you provide the flutter code you're using to access the flutter logo image inside your app?

I already provide it on first comment in Code sample section.

Ah yes, I see it. I think you need to change it to Image.asset("assets/images/flutter_logo.png"),
since the images folder is in the assets folder as specified in your pubspec.yaml file.
Let me know if that works, thanks!

Thanks! It works now.
But still confuse why it not error when runing on localhost

I believe Firebase Hosting sets a default folder called "assets" then moves your assets folder underneath it. If you didn't want the repetitive folders on the hosting site, your pubspec.yaml could look like this:
  assets:
    - images/flutter_logo.png

and you would have an images folder instead of an assets folder within your project.
Glad to be of help!


Need Debian package script

We want such a script to build debian packages automatically (or better yet: include a debian folder so that the normal debian build tools work). Travis supports Launchpad as a deployment provider, which means if we had such a deian folder, we could automatically build debian packages and have them hosted for free on Launchpad (under https://launchpad.net/opensmalltalk, so any Ubuntu user could add the repository with ppa:opensmalltalk/vm and get debian packages for each build).
Does anyone have experience with building debian packages?

This exists for pharo. Have a look at https://github.com/pharo-project/pharo-ubuntu . This is the build script for generating source packages that can be uploaded to launchpad. See https://launchpad.net/~pharo . A simple add-apt-repository ppa:pharo/stable does the job. I started to change scripts in order to generate spur packages. (https://github.com/noha/pharo-ubuntu). I don't find much time right now but I hope in two or three weeks I have something ready. The hardest part will be to build 32bit and 64bit from the same sources. I'm not sure how difficult this will be.


Should descriptor_pool_info include FREE_DESCRIPTOR_SET flag?

From https://github.com/zed-industries/zed/issues/10018#issuecomment-2107354282

After several hours of debugging, I can confirm that this issue only occurs with Intel's drivers.
filename: igvk64.dll
fileversion: <IP_ADDRESS>02
filedescription: Vulkan(R) Driver for Intel(R) Graphics Accelerator

__int64 __fastcall vkDescriptorAllocatePoolSet(__int64 a1, __int64 a2, __int64 a3)
{
	...
	// Check descriptor_pool->dword424 is bigger than set.
	// the throw VK_ERROR_OUT_OF_POOL_MEMORY error.
	//
	// When this error occurs.
	// 0:000> dd 0x00000212bdec4878+424 l1
	// 00000212`bdec4c9c  0000ea60
	// 0:000> dd 0x00000212bdec4878+420 l1
	// 00000212`bdec4c98  0000ea60
	//
	if ( v11 && (unsigned int)(descriptor_pool->dword424 + 1) > descriptor_pool->dword420 )
        {
            sub_180470550(&v36, 20LL);
            v37[0] = v36;
            CxxThrowException(v37, (_ThrowInfo *)&_TI1_AUException_VK__);
        }
	...

	// here inc dword424
        *(_DWORD *)(v14 + 68) = descriptor_pool->dword424++;
        v30 = v12;
        v20 = v11;
        v21 = playout;
}

__int64 __fastcall vkResetDescriptorPool(__int64 hDevice, struct_descriptor_pool *descriptor_pool)
{
	//
	// VK_DESCRIPTOR_POOL_CREATE_FREE_DESCRIPTOR_SET_BIT = 1
	// defalut descriptor_pool->flags == 0
	//
	if ( descriptor_pool->flags ){
		...

		//
		//  void __fastcall sub_18039C3B0(struct_descriptor_pool *descriptor_pool, __int64 a2)
		//  {
		//	 ...
		//	 // ONLY this way to dec descriptor_pool->dword424
		//   descriptor_pool->dword424 -= *(_DWORD *)(a2 + 68) != -1;
  		//   v16 = *(unsigned int *)(a2 + 68);
		//   ...
		// 	}
		//

		sub_18039C3B0(descriptor_pool, v11);
		...
	}else{
		// NERVER DEC descriptor_pool->dword424
	}
}

nice work, @progmboy ! Would you be able to file this upstream to Intel folks? Seems rather serious.

@kvark Does adding FREE_DESCRIPTOR_SET flag affect other platforms? From Vulkan specification:

VK_DESCRIPTOR_POOL_CREATE_FREE_DESCRIPTOR_SET_BIT specifies that descriptor sets can return their individual allocations to the pool, i.e. all of vkAllocateDescriptorSets, vkFreeDescriptorSets, and vkResetDescriptorPool are allowed. Otherwise, descriptor sets allocated from the pool must not be individually freed back to the pool, i.e. only vkAllocateDescriptorSets and vkResetDescriptorPool are allowed.

Even though blade only releases entire pools, it seems like this flag only extends you to allow to release each set individually. If this fixes the Intel driver bug, and doesn't meddle with other graphics card, I think it'd be a good solution.

I suspect it may change the allocation strategy for the descriptors, thus making it slower for us to allocate new ones. And this would be potentially significant since Blade relies on allocating descriptors many times per frame, potentially thousands of times.
For this reason, and the fact that the current code agrees with Vulkan spec, I wouldn't want to put this workaround unconditionally. I am, however, open to put it explicitly as a workaround for a specific platform bug.
Looks to be affecting Windows Intel 11th gen? That's a pretty popular config...

Yeah sounds good.

Interestingly, someone else is having this issue but on an AMD GPU: https://github.com/zed-industries/zed/issues/12561#issuecomment-2156365422
I'm not sure if these are related.

more about the GPU
it's an RX 580 8G
with the latest driver of 24.3.1

Uhh, that's rather unfortunate. Maybe it's just a Windows thing in general?

Uhh, that's rather unfortunate. Maybe it's just a Windows thing in general?

but there is some good news, on zed's last commit, it looks fine
e829a8c3b0564bc902cc0d0a530099be7f0f036e

ah, ok, phew, so it was addressed by https://github.com/kvark/blade/pull/118

Closed by #129


Provide documents related to creating pulsar-instance and pulsar-cluster

https://github.com/streamnative/pulsar-examples/pull/30/files#r466933861

@Huanli-Meng will add this link when cloud doc released


Removing Manual delete check

After creating your pull request, tick these boxes if they are applicable to you.

[X] I have tested my changes against the review branch (the latest developmental version), and this pull request is targeting that branch as a base
[X] I have tested my changes on Python 3.8 or higher
[X] I have ensured my code is formatted using Black

Description
Removes usage of the _manual_delete_check function, since it only tried to delete messages that were already deleted.


Typechecker issue for SeriesMissingValueLens in ReadingsDemo branch

The error occurs when we try to access from the view created by the Missing Value Lens. There is a type-mis match between the VG-Term in THEN part and Column in the ELSE part
.
For example: if my create lens statement is as below:
CREATE LENS TESTABC1 AS SELECT * FROM ABC WITH MISSING_VALUE ('COL2');
where table ABC is CREATE TABLE ABC1(Col1 VARCHAR, Col2 INT, Col3 INT);
Here {{ TESTABC1:INTERPOLATE;0[ROWID][ROWID] }} is returning the type of first column in SELECT * FROM ABC which is used as the input query to the lens

Duplicate of #269
For now, set types in adaptive schemas before creating lenses.


Throws info 2

Merged apple/swift/master in. This fails compiling the standard library with
/Users/dfs/swift-source-2/swift/stdlib/public/core/ExistentialCollection.swift:263:26: error: method does not override any method from its superclass
  internal override func _drop(
                         ^
/Users/dfs/swift-source-2/swift/stdlib/public/core/ExistentialCollection.swift:220:17: note: potential overridden instance method '_drop(while:)' here
  internal func _drop(
                ^

This seems limited to ExistentialCollection.swift. The methods are throws and rethrows. The override mechanism may be confused now since we added part of the typed throws implementation but not all of it.

Wrong branch, sorry I will redo this.


Pass the component definition as second argument to asyncData (and fetch)

What problem does this feature solve?
First, a bit of background.
I'm building a Nuxt application but reorganized the whole directory structure to use self contained modules. I find it easier to maintain in the long run, and modules can be reused/shared with other applications easily.
These modules may include routes, components, configurations… but also Vuex modules. And to keep the application performances as high as possible, I'm splitting those modules into different chunks.
Basically, I got :

A main chunk, the application core (router, store, layout…)
A chunk for each route (layout, components, configuration, store module…)

 That way, the Vuex store is initialized with only the required modules. I'm achieving this by doing :
// Layout.vue

import module from './store.js'

export default {
  asyncData ({ store }) {
    if (!store.state.namespace) {
      store.registerModule('namespace', module)
    }
  }
}

The second step has been to move this logic into a custom mixin (while creating a custom Vue options merge strategy) :
// mixins/index.js

export const withStore = (namespace, module) => ({
  asyncData ({ store }) {
    if (!store.state[namespace]) {
      store.registerModule(namespace, module)
    }
  }
})

// Layout.vue

import module from './store.js'
import { withStore } from '@/mixins'

export default {
  mixins: [
    withStore('namespace', module)
  ]
}

This is better, but I'd like to go even further by adding a custom component options, and applying the mixin globally :
// mixins/index.js

export const withStore = {
  asyncData ({ store }) {
    const { namespace, module } = ... // <---- This is where I need more informations

    if (!store.state[namespace]) {
      store.registerModule(namespace, module)
    }
  }
}

// plugins/index.js

import Vue from 'vue'
import { withStore } from '@/mixins'

Vue.mixin(withStore)

// Layout.vue

import module from './store.js'

export default {
  store: {
    namespace: 'namespace',
    module: module
  }
}

That's what causing me issues. I'd like to access to the component options from the asyncData method.
What does the proposed changes look like?
As far as I understand, we can't access the component options from the asyncData method (Note: I'm not trying to access the component instance but the component options, which are statically defined).
My proposal is to pass the component definition as second argument to asyncData (and fetch to keep the same signature).
So, my previous mixin would be written as :
// mixins/index.js

export const withStore = {
  asyncData ({ store }, Component) {
    const { namespace, module } = Component.options.store

    if (!store.state[namespace]) {
      store.registerModule(namespace, module)
    }
  }
}

On one hand, this would solve my issue, and allow people to use component methods (this would still be undefined though). But on the other hand, this would still be undefined and most methods would crash. So, we may pass a subset of the component definition instead of passing the whole object.
What do you think about it ? Is there any other solution I didn't think of ?
I'd be happy to contribute and make a PR if this proposal is accepted. :smiley_cat:

This feature request is available on Nuxt community (#c7531)

@Atinux do you think this is worth pursuing?

Actually I like this idea and can bring more possibilities for plugins actually. I am also thinking of the  nuxt-i18n module.
One thing that bother me is the fact that we can use asyncData or fetch with a callback as 2nd argument. But the more I think, the less I think callback is still relevant for asyncData and fetch.
Would need @pi0 and @clarkdo thoughts on this subject.

@antoinerey I'm interested in this, but I'm not able to create a mixin that can accept asyncData. Do you mind sharing your custom merge strategy? I can access both functions but I have no way to pass the context as arguments, so the mixin is mostly useless.

I was using this :
import Vue from 'vue'

Vue.config.optionMergeStrategies.asyncData = (to, from) => (...args) => {
  return Promise.all([
    to && to(...args),
    from && from(...args)
  ])
}

But, it was an experiment and may have some side effects.

Did you get time to think about this proposal ? It is something you would accept a PR for ?
I'd be pleased to work on it, but I need some validation about the expected interface/usage.
@Atinux @pi0 @clarkdo

Did you get time to think about this proposal ? It is something you would accept a PR for ?
I'd be pleased to work on it, but I need some validation about the expected interface/usage.
@Atinux @pi0 @clarkdo

Side note: The proposed method signature can't be implemented before Nuxt 3 as the 2nd arg is a callback variable (to avoid breaking changes).
asnycData(ctx, cb, Component) would be theoretically possible.


"double free or corruption" error

In case of [TIMES] Statistic not equal NONE (see page 174 of Epanet2 users manual) ENclose() function tries to close TmpOutFile again giving a "double free or corruption" error.
You can reproduce error modifing example-networks/Net1.inp at line 124
from Statistic None
to Statistic AVERAGED and running epanet.
Solved adding
TmpOutFile=NULL; at line 506 in src/output.c
(included in pull request #39 )

It is a bug related to gcc/Linux.
gcc/mingw32 does not have any problem with fclose() on already closed file.

epanet2.dll (original from EPA) no problem
epanet2.dll (from OpenWaterAnalytics compiled with gcc/mingw32) no problem
epanet2.so (from OpenWaterAnalytics compiled with gcc/Linux) double free or corruption

output from  (from OpenWaterAnalytics compiled with gcc/Linux)  where Net1.inp is example-networks/Net1.inp modified at line 124 from Statistic None to Statistic AVERAGED
maurizio@VirtualBox:~/EPANET/build/Linux$ /usr/local/bin/epanet2 Net1.inp temp.txt temp.bin

EPANET Version 2.1.0

Retrieving network data...
Computing hydraulics at hour 0:00:00
Computing hydraulics at hour 1:00:00
  [omissis]
Computing hydraulics at hour 24:00:00
Computing water quality at hour 0:00:00
  [omissis]
Computing water quality at hour 24:00:00
*** Error in `/usr/local/bin/epanet2': double free or corruption (!prev): 0x092e4218 ***
Aborted (core dumped)

not sure i totally get this. this change https://github.com/OpenWaterAnalytics/EPANET/pull/39/files#diff-1e3dd7877c487d2b3b9f07d9e14e6637R506 sets the file handle to NULL after it is fclose()'d, right?

Yes, I'll explain it better....
While using statistics option output.c uses file TmpOutFile fclose () it.
The ENclose () if TmpOutFile!=NULL fclose () it again the sets TmpOutFile
to NULL.
This is not a problem with Windows but with Linux risse error "double
free....".
I see two possibile solutions:

output.c dose not fclose () TmpOutFile
output.c sets TmpOutFile to NULL after fclose () as flag for not close
again (rest of our code does in that way)

The second one sound better
Il 26/Nov/2015 01:40, "Sam Hatchett"<EMAIL_ADDRESS>ha scritto:

not sure i totally get this. this change
https://github.com/OpenWaterAnalytics/EPANET/pull/39/files#diff-1e3dd7877c487d2b3b9f07d9e14e6637R506
sets the file handle to NULL after it is fclose()'d, right?
—
Reply to this email directly or view it on GitHub
https://github.com/OpenWaterAnalytics/EPANET/issues/42#issuecomment-159767528
.

Got it. Thanks for the explanation. I was assuming the crash was at that fclose when in fact it was later down the line. Good fix.

also can report that this was not a crashing issue on Apple LLVM version 6.1.0 (clang-602.0.53) (based on LLVM 3.6.0svn)

sounds like this is fixed and queued for 2.1, so i'm closing.


Debug trait should print the SEXP pointer

We had a question on the discord about getting the SEXP pointer in Rust to determine when new SEXPs are being allocated etc. This would work automatically if we derived the default Debug implementation. Unfortunately, we have a "pretty print" debug impl which is not actually very helpful for this purpose.
Some options we have:

Add the SEXP to the debug output
Implement Pointer for Robj, so that users can print it that way. This is not incorrect in my opinion, because a Robj is just a wrapper around SEXP anyway.

I think option 1 is the best. Option 2 could lead me to think that the Robj struct is the same even when it isn't. I think it's also good to add to Robj's doc that cloning that struct would keep the same SEXP, only creating a new instance of the Robj struct.

So in Rust Pointer is implemented for all references because they are pointers, but not for all structs themselves. So currently print!("{:p}, some_robj") will fail (I think) whereas print!("{:p}, &some_robj") will succeed, and show the address of some_robj in Rust memory.
If we implemented Pointer for Robj, that would stay the same, and only  print!("{:p}, some_robj") would change.


Question: getting maximal type safety when using Option

Hi there,
I recently encountered this very nice library mainly looking for a solution to avoid working with null/undefined as much as possible, but also for its other fp capabilities. So I decided to start by using Option as much as possible in my project.
I was a little disappointed to find out that even when using Option, I don't get full "protection" from the compiler. For example, when taking this code:
interface IData {
    amount: number;
}

function f(data: IData): void {
    if (data) {

    }
}

and converting it to this code:
function fWithOption(data: Option<IData>): void {
    if (data) { // I want the compiler to shout at me here

    }
}

I didn't get any compilation warning/error for forgetting to change the if (data) to if (data.isSome()).
Is there a way to achieve that in TypeScript? I did turn on the "strict" and "strictNullChecks".
Thanks,
Boaz

Well, it's JavaScript with its implicit casts. Here it casts your data object to boolean and it's a valid condition from the perspective of the language.
General recommendation though is to stop using such ifs and other imperative instructions and replace them with operations on data instead.

@boaz-chen you can use this tslint rule https://palantir.github.io/tslint/rules/strict-boolean-expressions/

Thanks guys.


[Minimap Integration] Use same map for *.emc.gs domains

Summary
If you connect with utopia.emc.gs, for example, it should use the same set of maps as when you connect with play.emc.gs, emc.gs, smp4.emc.gs etc.
How to
Work your magic
Motivation
Reduces duplicate files on system & gives more flexibility

What do you mean "use the same set of maps"?
I am currently changing the map based on which server-world you're in/on

Connecting to utopia.emc.gs, you create a different set of maps than you do when you connect to play.emc.gs. You can reuse the same map for each of them

I get it now. It just clicked. I always use the play url


[Enum Request] Table 90 BOM Component Field 3 "Type"

We need to be able to extend the options defined on Field "Type" in Table 90 BOM Comopnent. Can you implement this field as a ENUM?
Thanks.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[Event Request] Codeunit 1535 "Approvals Mgmt." - OnBeforeShowPurchApprovalStatus

Event request with IsHandled pattern in function ShowPurchApprovalStatus
local procedure ShowPurchApprovalStatus(PurchaseHeader: Record "Purchase Header")
**var**
    **IsHandled: Boolean;**
begin
    PurchaseHeader.Find;

      **IsHandled := false;
     OnBeforeShowPurchApprovalStatus(PurchaseHeader)
        if IsHandled then
            exit;**

    case PurchaseHeader.Status of
        PurchaseHeader.Status::Released:
            Message(DocStatusChangedMsg, PurchaseHeader."Document Type", PurchaseHeader."No.", PurchaseHeader.Status);
        PurchaseHeader.Status::"Pending Approval":
            if HasOpenOrPendingApprovalEntries(PurchaseHeader.RecordId) then
                Message(PendingApprovalMsg);
        PurchaseHeader.Status::"Pending Prepayment":
            Message(DocStatusChangedMsg, PurchaseHeader."Document Type", PurchaseHeader."No.", PurchaseHeader.Status);
    end;
end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeShowPurchApprovalStatus(var PurchaseHeader, var IsHandled)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Timeout errors (with GPT2 and simple 'return' model)

Hi, thanks for this very interesting library.  I'm trying to use it to handle multiple requests to a gpt-2 chatbot on a server, currently running on a single gpu, in flask app, on apache server, on win10, using tensorflow 1.x for gpu.  I am using python 3.7 in a virtual environment.  Using ThreadedStreamer or Streamer I only get time-out response.  I'm debugging using the smallest gpt2 model, which takes ~5 seconds from launch to reponse, so I'm very confused about where your code is getting hung?  To debug, I created a short function that simply returns any input text: this works with ThreadedStreamer, but gives timeout with Streamer.  I do not know what else I can try to debug further.  (I know I wont get much performance enhancement with gpt2 using this service-streamer on single gpu right now, but I would like it to handle request queues for now, and perhaps when I use multi-gpu in the future).
Thanks in advance for any advice!
sample calls in flask app:
result_5 = interact_model_debug #the gpt2 model predictor function
streamer_5 = ThreadedStreamer(result_5, batch_size=1, max_latency=10) #None
streamer_52 = Streamer(result_5, batch_size=1, max_latency=5, worker_num=1) #None

@app.route("/gentest5", methods=['POST'])
@cross_origin()
def get_gentest5():
    print("PRINT --> starting gentest5 function")
    data = request.get_json()

    if 'text' not in data or len(data['text']) == 0 or 'key' not in data or data['key'] != apikey: #or 'model' not in data:
        abort(400)
    else:
        text = data['text']
        inputs = [text] #request.form.getlists("s")
        print("PRINT --> just before running streamer_5.predict")
        outputs = streamer_52.predict(inputs)
        return jsonify({'result': outputs})

Here is the error log:
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192] [2020-08-01 17:26:09,025] ERROR in app: Exception on /gentest5 [POST]\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192] Traceback (most recent call last):\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\flask\app.py", line 2447, in wsgi_app\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     response = self.full_dispatch_request()\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\flask\app.py", line 1952, in full_dispatch_request\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     rv = self.handle_user_exception(e)\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\flask_cors\extension.py", line 161, in wrapped_function\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     return cors_after_request(app.make_response(f(*args, **kwargs)))\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\flask\app.py", line 1821, in handle_user_exception\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     reraise(exc_type, exc_value, tb)\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\flask\_compat.py", line 39, in reraise\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     raise value\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\flask\app.py", line 1950, in full_dispatch_request\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     rv = self.dispatch_request()\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\flask\app.py", line 1936, in dispatch_request\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     return self.view_functionsrule.endpoint\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\flask_cors\decorator.py", line 128, in wrapped_function\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     resp = make_response(f(*args, **kwargs))\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:/Users/irfan/Python_coding_folder/ChatBots/GPT2Local/gpt-2\WebAPI.py", line 218, in get_gentest5\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     outputs = streamer_52.predict(inputs)\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\service_streamer\service_streamer.py", line 132, in predict\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     ret = self._output(task_id)\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\service_streamer\service_streamer.py", line 122, in _output\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     batch_result = future.result(WORKER_TIMEOUT)\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]   File "c:\apienv3\lib\site-packages\service_streamer\service_streamer.py", line 41, in result\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192]     raise TimeoutError("Task: %d Timeout" % self._id)\r
[Sat Aug 01 17:26:09.027854 2020] [wsgi:error] [pid 20964:tid 1432] [client <IP_ADDRESS>:14192] TimeoutError: Task: 0 Timeout\r

Solved for ThreadedStreamer, but still cannot get Streamer to work, with or without managedmodel.
For ThreadedStreamer, in case it's of use to anyone else, it was an input/output data mismatch.  the data that the worker passes to the function is a list [x], therefore you must called the required element input[0] to obtain the input string for your model, otherwise your model will error out.  likewise for the output, package the string as a list for the queue to handle.  On doing this, the package works beautifully in this mode.
However, I am still having timeout issues with Streamer.  Since I cleared up the data handling within flask, I'm very confused as to why threadedstreamer works but streamer does not.  I have only a single GPU, but it seems that it should still work (I want to set this up now, so that if I do slide in another GPU it's already ready.
my call for streamer, predict_X is a simple return function of whatever text is passed in:
streamer_3_managed = Streamer(totaltest.predict_X, batch_size=1, max_latency=20, worker_num=1)
@app.route("/gentest3", methods=['POST'])
@cross_origin()
def get_gentest3():
    text = data['text']
    inputs = [text]
    outputs = streamer_3_managed.predict(inputs)[0]
    return jsonify({'result': outputs})

I got the small GPT-2 model working smoothly with 4 workers on Streamer on single GPU, however the x-large model with 1 worker gives an OOM error (even though it works well with ThreadedStreamer?)
To work with Windows, I followed the same outlay as :  https://github.com/ShannonAI/service-streamer/blob/ebcc3724efc9789feb8980641533909798d4d6dc/example/flask_multigpu_example.py#L4
This made the 'freeze_support' errors disappear in win10.  My GPU is NVidea GeForce GTX 1080, with 6+GB of RAM, so technically it should support x-large GPT-2 model with 1 worker on Streamer.  Any ideas what might be still missing from my setup?  Thx in advance for any tips and tricks.  :)  I am very pleased with the progress so far, and this excellent code you've provided.

Solved for single worker, single GPU: I had added the following below 'import tensorflow as tf' in order to get the multi-worker to work for the small model.  After commenting out, the x-large model works with single worker.  So, hopefully if I add more GPUs it should require a simple tweak in my code now.
I commented these out:
#config = tf.ConfigProto()
#config.gpu_options.allow_growth = True
#session = tf.Session(config=config)
And now it works again for single worker.
I will leave this here in case it is of use to anyone else using your code for sequential generative models.  Thx for the excellent code.

In case this is of use to anyone else: multiprocessing in python through apache-mod_wsgi on windows seems impossible to achieve.   Therefore I created a dual boot in Debian, to explore API capabilities using nginx-uwsgi with service streamer.  As others noted, it doesn't seem to work with 'spawn', however changing 'spawn' to 'root' in the two call instances of service-streamer allows you to run multi-processing on a production server in linux.  This could theoretically also work in WSL2, if one wishes to stay in windows, however at present the WSL2 GPU cuda support is highly inefficient, creating a x5-10 slowdown in running (I also personally verified this on my code).  I have not yet tried redis-streamers, but I'm confident that should work without issue (as and when required), and it's nice to know that i could have multi-gpu multiprocessing support if i do go that route.

@m4gr4th34 thanks for your interest. The main difference between Streamer and ThreadedStreamer is that Streamer uses multi-process. So it will occupy double gpu memory when there are 2 workers.
Another tips: apache-mod_wsgi is outdated for python server. If you must use Windows(which is not recommended) to deploy your server, just use gevent.wsgi

Thanks, that explains why apache-mod_wsgi is so poorly documented.  I am working in linux now, I like the new debian features.  well, it seems most things in CS are poorly documented, tbh.  A question about multiprocessing: my GPT2 model saturates my single GPU, if I install a second GPU is it possible to use     streamer to use gpu1 for worker1 and gpu2 for worker2?

yes, take a look: https://github.com/ShannonAI/service-streamer#distributed-gpu-worker


feat(glasskube-doc): prepare the placeholder

Description
Describe the proposed changes made in this PR.
How was this PR tested?
Describe how you tested this PR.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Issue on going on glasskube to fix before merge: https://github.com/glasskube/glasskube/issues/818
A PR has been opened here: https://github.com/glasskube/packages/pull/150

Issue on going on glasskube to fix before merge: glasskube/glasskube#818
A PR has been opened here: glasskube/packages#150

Done


Feature/ssh bare images

Checkout the README in diff. It explains what's in this PR well

@ygrishajev Let's rename ubuntu-ssh into base-ssh ?

@andy108369 don't get it, what exactly? you mean the image? it's currently named ubuntu-2404-ssh which describes with it is. Any reason why would base-ssh be a better name?

@ygrishajev Let's rename ubuntu-ssh into base-ssh ?

@andy108369 don't get it, what exactly? you mean the image? it's currently named ubuntu-2404-ssh which describes what it is. Any reason why would base-ssh be a better name?

I didn't mean the image name, sorry for not making it clear.
I meant just the directory name since you are adding Dockerfiles for different distros (centos, debian) under the "ubuntu-ssh" directory. "base-ssh" directory name would probably be more distro-agnostic, unless you have better ideas.
Hope that clarifies the intention.

@andy108369 ah, yea, that makes more sense indeed. Just forgot to rename )


Getting started question

im probably being stupid but how do I create the note path for the default note?  I keep getting a message it doesn't like my filepath. Thanks!

Hi!  Im your case this would be 2. Area/Tasks.md

Thank you!On 23 Jul 2024, at 06:20, Simon Knittel @.***> wrote:
Hi!  Im your case this would be 2. Area/Tasks.md
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Productionize integration with MissionPlanner

It's clear we need to start solving some of this for teams.
Smart client. Client side dedupe and validation, rate reporting, status reporting, etc.
Display integration. Show obstacles for SDA.

Also look into where telemetry bottleneck and for upstream fix.

I'm not sure that showing SDA is an important feature for us to be working on. Otherwise, I agree.

I'm going to look into it. If it's reasonable I'll give it a go.


Is there a preferred citation for this package?

I am using this package in my research, and just want to check in to see how you would others to cite this package. Thank you in advance!

Hi Will.
Thanks for asking how to cite the package. I never ended up getting this package submitted to JOSS or something similar, so feel free to cite however you would like.

Great, thanks!


P-harmonics cannot be picked.

In Campbell plot in the frequency part the p-harmonics cannot be picked with a marker. The values however are interesting to see, to compare against frequencies.

feature branch merged


cargo build fails

Hi guys, I am using a custom dev baord running ubuntu 20.04 and tried to build this repo. But I receive the above error.
I tried to change the env_logger version but still receive the same error.
Please guide me.
Thanks

You need at least rust 1.77 as of now. I would advise against the ubuntu built-in toolchain which is surely out-of-date
Use the rustup toolchain manager instead - https://rustup.rs

Use Rust 1.77 and do cargo update before the build. The build does succeed, as we run it nightly in our CI pipeline.


Adding additional future looking criteria

Coverage remained the same at 94.088% when pulling 099983aa2568d7ca396d923839ea2e579adf2b56 on reshma-criteria-update into 16a11c81d8f4b2fed7b705a7f535cd2c78cbf683 on main.


Support for passing in your own versions of the http server modules

This is useful if you want to use a module like findhit-proxywrap to enable support for the PROXY protocol:
http://www.haproxy.org/download/1.8/doc/proxy-protocol.txt
PROXY protocol is used in tools like HA-Proxy, and can be optionally enabled in Amazon ELB load balancers to pass the original client IP when proxying TCP connections (similar to an X-Forwarded-For header, but for raw TCP).  This is useful if you want to run redbird on AWS behind an ELB load balancer, but have redbird terminate any HTTPS connections so you can have SNI/Let's Encrypt/HTTP2support.  Redbird will see the client's IP address rather than the load-balancer's, and pass this through in an X-Forwarded-For header.
An example of this is
//Options for proxywrap. This means the proxy will also respond to regular HTTP requests without PROXY information as well.
proxy_opts = {strict: false};
proxyWrap = require('findhit-proxywrap');
var opts = {
    port: process.env.HTTP_PORT,
    httpServerModule = proxyWrap.proxy( require('http'), proxy_opts),
    ssl: {
        http2: true,
        httpsServerModule = proxyWrap.proxy( require('http'), proxy_opts),
        spdyServerModule = proxyWrap.proxy(require('spdy').server, proxy_opts),
        port: process.env.HTTPS_PORT,
    }
}

// Create the proxy
var proxy = require('redbird')(opts);

Then configure PROXY support in the ELB http://docs.aws.amazon.com/elasticloadbalancing/latest/classic/enable-proxy-protocol.html, and configure a TCP (rather than HTTPS) listener on port 443 of the ELB

Thanks for the nice addition. A few comments:

It would be great to have and update to the README where it explains how to use the modules options, it could include the example above.
Do we need both a spdyServerModule and httpsServerModule option?  (I am asking since both cannot be used at the same time, even if the example above specifies both of them in the same proxy, maybe the option could just be called serverModule).

Yes good idea.  I'll put together a commit with these changes later today.
On Thu, 10 Aug 2017 at 02:10 Manuel Astudillo<EMAIL_ADDRESS>wrote:

Thanks for the nice addition. A few comments:

It would be great to have and update to the README where it
explains how to use the modules options, it could include the example above.
Do we need both a spdyServerModule and httpsServerModule option? (I
am asking since both cannot be used at the same time, even if the example
above specifies both of them in the same proxy, maybe the option could just
be called serverModule).

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OptimalBits/redbird/pull/117#issuecomment-321266623,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA15gezDjVmSvE4d627UJlcAWEjCnWZJks5sWb2qgaJpZM4OxiGQ
.

--
Voted BestWeb Developer at the 2011-2014 Eset Netguide Web Awards. Read more
http://www.zeald.com/blog/x_post/best-web-developer-four-years-running-00479.html
Disclaimer: Privileged information may be contained in this message. If you
are not the addressee indicated in this message (or responsible for
delivery of the message) you may not deliver or copy this message to
anyone. In such case you should destroy this message and kindly notify
Zeald. Please note, opinions, conclusions and other information in this
message that do not relate to the official business of 'Zeald New Zealand
Ltd' shall be understood as neither given nor endorsed by it.

I've made the changes suggested.  While editing the README.md I also fixed the formatting of the function reference at the bottom, which wasn't rendering github.  I also noticed the ssl.http2 option wasn't documented

Looks great. Thanks!.


Incompatibility with<EMAIL_ADDRESS>

After commit https://github.com/CaffeineMC/sodium-fabric/commit/19ee2f0b70ae7c5f3c5f76c06e8bcf3adb8d69b8, the transparent loading screen mixin can't be applied.
As a workaround, you can add a custom property in your fabric.mod.json as shown here: https://github.com/CaffeineMC/sodium-fabric/blob/9915466a81e9d5e32f03fe16b1902c1e4880d824/src/main/java/me/jellysquid/mods/sodium/common/config/SodiumConfig.java#L97-L114
The property would look like this:
"custom": {
    "sodium:options": {
        "mixin.features.gui.fast_loading_screen": false
    }
}

Thank you for the workaround!

Thanks for the report and a fix! I'll push an update tonight with it


List does not get updated

After editing the title of a post, the title does not get updated in the list. Is this expected behaviour?

Which list? Are you using the editorial workflow?

Hi @liamkeily, can you please follow the issue template? Not doing so makes it very hard to resolve issues.

Closing. @liamkeily please follow the issue template


Fix crashes when inserting large values that exceed memory set in config file

Our redis instance was crashing when inserting values that exceeded the maxmemory size or when they were just large (around 30% of the maxmemorysize. We are using 10mb of maxmemorysize and would see redis crash when inserting around 3Mb values). It's pretty easy to repro the crashes.
Found one of the issues is in Win32_QFork->AllocHeapBlock when it's called with the allocateHigh set to false (This is when you try to insert a value that exceeds the maxmemorysize).
Another issue was in networking.c->readQueryFromClient and processMultibulkBuffer (these where when you try to insert a largish value that was smaller than maxmemorysize).
These commits seem to fix the problem

Hi @krolth, I'm your friendly neighborhood Microsoft Open Technologies, Inc. Pull Request Bot (You can call me MSOTBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.msopentech.com.

TTYL, MSOTBOT;

@krolth, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSOTBOT;

Fixed in https://github.com/MSOpenTech/redis/commit/9fd03a982dd55655b8393b74de4e49ba345f48f4. Thanks!


test stdci: Set CI environment variable

Set CI environment variable as travis did for STDCI. The run-test.sh
script uses it.
Signed-off-by: Gris Ge<EMAIL_ADDRESS>Pulled out from #543

This change is

Coverage decreased (-1.8%) to 66.49% when pulling 7a04c7f5213118eb059a25342e2ed6bfefa039dc on tyll:stdci_fix into 208eda340d539cec77b15fdb64843e7aad0425f4 on nmstate:master.

Coverage decreased (-1.8%) to 66.49% when pulling 7a04c7f5213118eb059a25342e2ed6bfefa039dc on tyll:stdci_fix into 208eda340d539cec77b15fdb64843e7aad0425f4 on nmstate:master.

I do not really understand what this change does, could you please elaborate? Hopefully in the commit message so we will record this for future us.

I do not really understand what this change does, could you please elaborate? Hopefully in the commit message so we will record this for future us.

I rephrased this, hope it is now more clear.