Update the model name in the api doc

Hi, I am new to Ollama.
I followed the Quickstart to try Ollama with model Llama2. It is very easy to run and a very interesting project.
When I explored further in the API doc, I found that the model names are not consistent. There are llama2 and llama2:7b in the API doc. I met the issue like {"error":"model 'llama2:7b' not found, try pulling it first"}% .  From the model library section, I think llama2 is the same as llama2:7b. But from here I notice that llama2 is actually llam2:latest.
For the API doc, do you think it is better to change all the llama2:7b to llama2? It is good for the people who are following the Quickstart.

Thanks for creating an issue! This has been updated on main 😊


Missing videos in the Searching Algorithms unit plan

Missing videos and text from Unit Plan (https://docs.google.com/document/d/14Exafr-nE7s4yDuvPMn-zv81gVvK5HMS8Unu-wRW2BI/edit)

These videos are being scheduled to be worked on.
Video 1: Sequential search: Random numbered cards with pictures on them. (using numbers from 0 - 31) - SETTING: As if it’s a maths group in a class setting
Video 2: Binary search using numbered cards with pictures on them ranging from 0 - 999.
Video 3: Linking the “Treasure Hunt” activities to statistics

These are yet to be made, so please remove the text from the unit plan (if you haven't already) and we'll make these in the next round of videoing which is likely to be in September.


DS-3233: Update HandlePlugin to start/stop its own DSpaceKernel

As noted in the discussion of DS-3233, the HandlePlugin now needs its own DSpace Kernel for access to the HandleService and other services.
https://jira.duraspace.org/browse/DS-3233
This PR updates the HandlePlugin to start its own Kernel in its init() method, and destroy it in its shutdown() method.  That ensures the Kernel remains alive as long as the Handle Server is running (and using HandlePlugin).
I've tested this PR locally, and the start-handle-server script works again.  Before this PR, running start-handle-server threw errors and the Handle Server failed to startup.
UPDATE: This PR also now includes a Windows Version of the 'start-handle-script'.  See https://github.com/DSpace/DSpace/pull/1489/commits/8da4fb5bf72add113e84f47b37148a3ffd04972d

Updated this PR with a new start-handle-server.bat script for Windows.  I've tested this pretty thoroughly on my own Windows machine, and it seems to be working well.  This should help discourage anyone from attempting to start the Handle Server using the ./dspace.bat script (which won't work, see comments in DS-3233).

Looks right, and I like the script comment cleanups.
Built, installed, tested on uptodate Gentoo Linux with Oracle JDK 1.8.0_102 by running 'bin/start-handle-server'.  The process detached and is running, and has three sockets open, as expected.  Logs look good.  I'd say it works.
+1

+1 by inspection

Approved and tested (by someone other than me).  Merging


visible width of the ScaleLine on the map depends on the flag 'bar' (true/false)

Describe the bug
When using the ScaleLine component we identified that the width on the Map varies depending whether the flag bar is set to true or false.
To Reproduce
To reproduce the issue a map with a grid is helpful (i.e. SwissTopo http://wmts.geo.admin.ch/1.0.0/WMTSCapabilities.xml). We use Electron 8.2.0 (Chrome engine):

create a map with a ScaleLine and set the option bar to false
align the ScaleLine to the grid and you can see that the 1km ScaleLine almost matches the grid
switch the option bar to true in order to render a ScaleBar instead of a ScaleLine
compare the width of the ScaleBar to the previous ScaleLine
seems like that the ScaleBar also fills the area left/right of the handles of the ScaleLine

I've attached images in order to visualize the issue:
ScaleLine:

ScaleBar:

It's worth noting that the div ol-scale-bar-inner does almost exactly match the grid:

EDIT:
The CSS class .ol-scale-singlebar seems to apply a border of 1px solid black which IMO causes the issue. By using the Chrome DevTools I can disable the style and the ScaleBar gets rendered correctly:

Expected behavior
Switching to a ScaleBar should not create a visual extension of the width in comparison to the ScaleLine.

We are using v6.3.1. After upgrading to v6.3.2-dev.1590745992030 the issue is still present.
Using Chrome 83.0.4103.61 on OS X I can confirm that your example works correctly. Our app uses Electron 8.20 which uses Chromium 80.0.3987.158. I'll do more research ...

@mike-000 What makes the example work is box-sizing: border-box added to the ol-scale-singlebar class. Chrome tells me that this property is added by the bootstrap CSS:

This property is not part of the OL CSS:
.ol-scale-singlebar {
  position: relative;
  height: 10px;
  z-Index: 9;
  border: 1px solid black;
}

By adding this property I was able to make the ScaleBar work correctly.

Yes, I can reproduce the problem when opening the example in codesandbox via the "edit" link  It shouldn't be relying on the examples template to display correctly.


Getting logged out too often

My account is https://www.codewars.com/users/Laplandia
Lately I am getting logged out every minute or so. When I submit a solution, it redirect to log in screen and the solution is lost.
It might have to do with the fact that I have two codewars tabs open.

@Laplandia
Same here, frequent log out while submitting a solution & after login can't find I have to rewrite the solution of the problem.
This is so frustrating..

#1

Sorry about this. We're aware but it's been difficult to reproduce. Closing as duplicate of #1.


add transform-enforcing compartment wrapper

This is still a work in process, but I'd love to get some feedback on the approach from @erights and @kriskowal . Also, I don't really understand how new Foo() works, I'd appreciate some help making it callable as new newCompartment() instead of just newCompartment().
refs #377

Ok, ready for review. Questions:

does this belong in SES-shim? (I think  yes, by putting it in the SES-shim monorepo, it stands a better chance of keeping up-to-date with changes to the Compartment constructor)
is the package name reasonable?
is the API name (wrapInescapableCompartment) reasonable?
can you think of any other sneaky ways to escape the "inescapable" that I should explore+test?

I don't have a test that will probably fail when @kriskowal's changes land (to give each compartment a distinct Compartment constructor).. I don't think I can write such a test until after there are differences to be observed. But this will need to be rewritten when that happens, otherwise child/grandchild compartments will get the parent's constructor and will get the wrong module map.

maybe the package ought to be named @agoric/inescapable-compartment-wrapper or something..

Oh, also I must add the right package.json magic to make this importable under -r esm before it lands, I need that from the agoric-sdk side..

@agoric/inescapable-compartment-wrapper? @agoric/inescapable-compartment-transforms? @agoric/wrap-compartment-inescapably?
import { wrapInescapableCompartment } from X? import { wrapCompartment } from X? import { wrapCompartmentInescapably } from X? import { wrap } from X?

@agoric/inescapable-compartment-wrapper? @agoric/inescapable-compartment-transforms? @agoric/wrap-compartment-inescapably?
import { wrapInescapableCompartment } from X? import { wrapCompartment } from X? import { wrapCompartmentInescapably } from X? import { wrap } from X?

inescapable-compartment
jail
enforcer

Getting a little dark with inescapability metaphors. What else can’t you leave?

garden
event-horizon
border
boundary

Maybe a step back

recompartment
partition

If you’re worried about prematurely publishing, is it possible to keep this close to the code that uses it until it’s firmer? Given that we don’t know how broadly applicable this pattern is beyond metering, I’d like to keep it close to the meter.

There are also peerDependencies but they are odd. Hat tip to Domenic Denicola for that contribution to the npm ecosystem.

I guess the high-order bit is whether this lives in SES-shim or agoric-sdk, but anticipating it breaking when you introduce per-compartment Compartment constructors, I'm inclined to think it belongs in SES-shim. It's certainly not agoric-sdk -specific.
But yeah, I could move it into an un-separately-published module within @agoric/import-bundle, which is the first customer. The plus side is not needing to choose a public name yet, the downside is that it'll stop working (poorly) when you make that change. "Poorly" because I don't think this code has a direct dependency on any particular version of SES (it doesn't import SES, it merely expects to be given a Compartment argument that behaves in a particular way), and when each compartment has a distinct constructor, the wrapper will start confusing those together, silently. I could make @agoric/import-bundle depend upon a particular version of SES, but it really needs to be the same version as what @agoric/install-ses uses or things will get really weird.
It kind of comes down to how we anticipate the SES ecosystem going. We expect that there will be helper libraries that are widely used but live outside the Compartment proposal. We don't know what else will be in these libraries yet, nor what they ought to be called. So yeah, maybe waiting until we have a second use case is a good idea.
So I'll ask for feedback on the implementation here, but I won't land this PR. Instead, I'll move the module into agoric-sdk @agoric/import-bundle. I'll add a note to try and remind us that when we change @agoric/install-ses to use a newer version of SES (with the constructor-per-compartment), we need to rewrite the wrapper too.

closed in favor of putting the wrapper in agoric-sdk for now


[Rust] Fixes for wasm32 target

This PR fixes warnings and errors when targeting wasm32.

Update the BackendPackedCFunc signature which was changed in #4637.
Use derive_default() for bindgen to handle the generated padding field.
Add workaround for nom::length_data, which doesn't allow u64 on 32-bit architecture.
Terminate the thread safely instead of panic when crossbeam-channel is closed.
Remove unused import warnings.

@jroesch @nhynes @ehsanmok @tqchen Please help to review.

Thanks @kazum! LGTM.

Could you actually add some kind of test which checks that we compile correctly for WASM target?


jdk-8166197 backport to jdk8, jvm hang on g1 full gc work threads in offer_termination with high cpu usage

Question
will jdk-8166197 back port to jdk8 and have a plan?
https://github.com/openjdk/jdk9/commit/7e084fd81fc5ef3e8245dc2381d4f13c016746c5
we meet the same bug, jvm hang on g1 full gc work threads in offer_termination with high cpu usage
platform: AArch64

Context
Java version:
jdk8u382-b05 AArch64

Your operating system and platform:
ky10 on aarch64

You'll need to ask that question on the OpenJDK jdk8u-dev mailing list (go to openjdk.org and follow your nose).  That issue is no longer listed in the bug tracker.

You'll need to ask that question on the OpenJDK jdk8u-dev mailing list (go to openjdk.org and follow your nose). That issue is no longer listed in the bug tracker.

ok，i will try.
Thanks for your reply.


test(closest_pitch): 🧪 add more tests for parse constructor

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1055

Relevant Lines:
1056

💛  - Coveralls


Quick build from command line with latest installed version of Visual Studio

Hello folks,
Would you accept a small one-file contribution which let us build cv2pdb binary from command line with latest installed version of Visual Studio ?
If yes, then build process would look like:
native bitness: python -m minibuild
win32: python -m minibuild --model win32
win64: python -m minibuild --model win64
On my machine with VS-2017 installed target binaries are here:

bin\output\exe\msvs2017-win32\release\cv2pdb.exe
bin\output\exe\msvs2017-win64\release\cv2pdb.exe

minibuild itself is a pure python package,
so as a prerequisite for build Python-2.7 or Python-3.x is required.
And then minibuild package is supposed to be installed by pip: python -m pip install minibuild
If no, then just close this issue.

Sorry for the late reply, I've been on vacation. Thanks for considering to contribute to cv2pdb.
I don't think adding another way to build cv2pdb adds much, though. You can build it from the command line with msbuild cv2pdb.vcxproj with appropriate options for configuration and tool chain.


Error in scipy.linalg.expm for complex matrix with shape (1,1)

When using the function scipy.linalg.expm for complex matrices of shape (1,1), some values result in a IndexError: too many indices for array error. A minimum example reproducing the error is given below.
This is with Anaconda Python 2.7.11 :: Continuum Analytics, Inc. and SciPy version 0.17.0.
Note that only the value of the single matrix entry changes slightly.
Minimum example:
from scipy import sparse
from scipy.sparse import linalg
v_works = -2.02683397006j
v_error = -2.12817566856j
M_works = sparse.csc_matrix([[v_works]], dtype='complex')
M_error = sparse.csc_matrix([[v_error]], dtype='complex')
print linalg.expm(M_works)
print linalg.expm(M_error)

Given that the matrix exponential of a 1x1 matrix is trivial, the fix is probably to check this special case early on in expm

What I found very surprising is that the error depends on the value in the matrix. Any idea why this is? A breakdown of the Pade approximation or something?


Feature/routing

Hello there,
Kindly review my PR.
Changes Implemented:

Added 2 new pages : Homepage and Quotes
Added Routing for SPA navigation
Implemented basic styling

Thank you.

@Ismailco  Thank you for the review !!


Add formats for the BK.

I confirm that both work (bk800 and bk800_525) with 720/800Kb diskettes, both reading and writing, it can be merged. But, if Fluxengine doesn't need _525 versions anymore, it may need to be changed, right?

Excellent news! Yes, the profiles will need adjusting for the new scheme, but I'll do that.


feat: theme switch in nav

Closes #36819

@ojeytonwilliams, before I do the clean up, could you please take a quick look and see if this approach would work in your opinion?

I ran into:

when I tried to build.  It should be fixable (as the screenshot says it's discussed [here])(https://www.gatsbyjs.org/docs/debugging-html-builds/). as I don't think window is used until after hydration.
Also, since the editor overwrites window.__onThemeChange the theme button won't toggle once the editor mounts - the themes change perfectly, just not the button. One solution would be to attach and detach listeners when mounting and unmounting.
Finally,

happened when I navigated away from a challenge (to the curriculum) and tried to toggle.  Doesn't seem serious, mind.  It should go away once mounting and unmounting are handled.  (setting editor state is just console output I added!)
This all seems pretty minor, but it does stop me from building, so I can't check it properly!
So, finally, to answer your question - yep, looks good. I think it should work.

@ojeytonwilliams this should do it.
to build locally use the following
export NODE_OPTIONS=--max_old_space_size=4096

Left a comment on the issue thread. Lets not ship this for now. @ahmadabdolsaheb is making a fresh PR to remove the link from the nav and make light mode the default.

Just to note that Gatsby docs pages itself do not have 'signed in' users which is the use case which we are concerned about.

Closing older PRs as decayed, please re-open afresh if still relevant.


[enhancement] add a colcon ignore cmd argument for permanently ignoring packages

Hello,
I was thinking about this use case:
I'm building a base docker environment for project A (with 3-5 packages) which uses for example only one package from project B (like projectB-msgs).
I want to protect novice programmers using this environment and building further packages from project B that they do not need. But I also do not want to change project B to only build that one package by default.

The feature I would like to request would probably work like that:
colcon ignore --packages-other-than <package-list>
Which would add a COLCON_IGNORE file in each package that is not in that list. Additionally, it also takes a path like [--ignore-path <src/projectB>] to only ignore packages from a certain project/"namespace"/sub-folder.

Disclaimer:
The best way would probably be to have all packages from projectB be released through Bloom as individual packages. This way I would only need to install apt install ros2-<distro>-projectB-msgs and everything would be fine.
I'm asking this if there is some valid use for this before just implementing it and getting the PR rejected.
For me there certainly is a valid use case as we are building up a bigger software architecture and trying some bits out here and there before publishing anything as a bloom package.
Also, all the packages are not in the same workspace due to dependency collisions which are handled fine through docker. That stops me from just compiling everything and be happy.
Eager to hear your feedback on this one. Cheers!

Interesting idea. I would recommend starting by making a new class inheriting from VerbExtensionPoint in a new python package (called colcon-ignore?). Then I would recommend using the existing package discovery and selection arguments.
--packages-other-than sounds a lot like --packages-skip. If you have a ROS 2 workspace, try colcon list and colcon list --packages-skip zstd_vendor  to see the last package get skipped. --ignore-path may already be covered by the package discovery argument --base-paths. It controls where colcon looks for packages. If you have a ROS 2 workspace, try colcon list --base-paths src/ros2/rcl; it will show all packages (and only those packages) in that folder. That would be a good way to control which packages are chosen to be ignored.
I think a good example to follow is colcon-package-information and how it implements colcon list.
https://github.com/colcon/colcon-package-information/blob/3fedb42f70337dd62fe0db1b7afed7f8cf82306f/colcon_package_information/verb/list.py#L14
https://github.com/colcon/colcon-package-information/blob/3fedb42f70337dd62fe0db1b7afed7f8cf82306f/setup.cfg#L61

Once that's working in a new package in a new repo, you can release it on PyPI for anyone to use. No need to worry about a PR to colcon-core being rejected :)

Another way to solve this problem might be to use a colcon defaults.yaml that looks something like this:
{
  "build": {
    "packages-ignore": ["projectB-msgs"]
  },
  "test": {
    "packages-ignore": ["projectB-msgs"]
  }
}

That would cause both the build and test verbose to automatically ignore the listed packages unless the user specifies something else for the --packages-ignore argument. It isn't perfect, but it also doesn't require any new code.

Closing this since the functionality should be implemented outside of colcon-core.


SCRIPT1002: Syntax error In IE 10

SCRIPT1002: Syntax error
I belive it's line 45 in npm module index.js
const VALID_HTML_ATTRIBUTE_NAME = /^[^\0\t\n\f\r /<=>]+$/;

In 1.14.1 it's ok

Per the documentation, please use the babel-compiled version in the dist/ folder.


Batching - incorrect modulate with joined sprites

Godot version:

Godot 3.3 RC 8
OS/device including version:

All
Issue description:

This is a stub issue, I have already fixed the bug, see below.
Create a project with 2 similar sprites that will be joined together by batching. Set the modulate of the 1st to white, and the second to red.
Apply this shader to both:
shader_type canvas_item;

void fragment() {
	COLOR = vec4(0.7, 0.7, 0.7, 1) * MODULATE;
}

Incorrect (batching)

Correct (no batching)

Steps to reproduce:
Run minimum reproduction project. It has flash batching switched on, should show the regression.
Minimal reproduction project:

UnitTests.zip
Bug
The bug occurs because the first sprite has a white modulate, it bypasses the decision to use a FVF with modulate. The second joined sprite should force MODULATE_FVF, but this does not work because only the first joined item is considered.

Fixed by #47788.


Publish Microsoft Authored templates to azure samples browser

Publish ASA-Samples-Web-Application template to azure samples browser.
@rajeshkamal5050 and @hemarina for notification.

@fangjian04232 Could you help review this PR and merge it if it looks good?

@saragluna Could you help review this PR and merge it if it looks good? Thanks in advance.


Error in training images with larger size like 1440x960

Try to train with my own data, follow the step here #5
GPU ran out of memory, 12 GB GPU.
Any idea for the limitation of the input image size? Thanks!

Because SegNet will upsample feature maps to original image size thus cause heavy memory usage if your input image is large. One solution is that you only upsample to 1/8 or 1/4 or 1/2, and then perform bilinear interpolation to upsample (like deep-lab network).


How to use ssd500 to fine-tune new data?

How to change the model to ssd500?
Do I need to write a new SSD500 architecture, or just change the input size to 500?
Does prior_boxes_ssd300.pkl need to be changed?

See https://github.com/rykov8/ssd_keras/issues/36
As of right now, changing to SSD500 or SSD512 is not trivial.


Device creation naming convention

Is your feature request related to a problem? Please describe.
The hardware team currently using the following naming convention many times:

AIRQO WBI UNIT ACTIVE
AIQRO-WB6 UNIT ACTIVE

Describe the solution you'd like
The device name should reflect the current scenario that the hardware team uses. This means that these names should be very descriptive.
Describe alternatives you've considered

The device name will no longer be unique.
We shall have to use the channel ID as the unique ID for each Device. To have a generic structure, we shall have to consider using a more generic name like deviceCode as the unique field in the database
We can also just use the Mongo DB IDs as the unique identifiers when querying accordingly.

Additional context
This is in relation to feedback where the hardware team does not relate with the naming convention we have. The current one is aq_. Also the hardware team uses some characters of these names in some other monitoring systems---their implementations look out for some parts of the device names in order to compute a device status.

I am concerned about the ability to RENAME a device. The device name should be unique and permanent so we can alwasy follow a device when conducting analysis. If device names are changing this is a risk. So perhaps a chance to name it on creation but not rename during its lifetime.
Name of device should not be dependent on site / location
As mentioned in meeting channel ID should relate to one and only one device. The same channel ID cannot be used for a new device in the same location. This will make analysis extremely difficult and collocation impossible.
Is the naming issue a blocker to achieving any other goals or a preference? Interested to hear
The needs of analysts need to be considered alongside he needs of hardware team. I am nervous about changing this at this stage. There are many other higher priority tasks namely getting the data out there.

I am concerned about the ability to RENAME a device. The device name should be unique and permanent so we can alwasy follow a device when conducting analysis. If device names are changing this is a risk. So perhaps a chance to name it on creation but not rename during its lifetime.
Name of device should not be dependent on site / location
As mentioned in meeting channel ID should relate to one and only one device. The same channel ID cannot be used for a new device in the same location. This will make analysis extremely difficult and collocation impossible.
Is the naming issue a blocker to achieving any other goals or a preference? Interested to hear
The needs of analysts need to be considered alongside he needs of hardware team. I am nervous about changing this at this stage. There are many other higher priority tasks namely getting the data out there.

Thanks @easybeinggreen , good points you have raised. Thanks, much appreciated! :)  I will share some quick thoughts alongside your thoughts:

According to the hardware team (Joel), the naming convention issue negatively affects their user experience (UX). And a bad UX reduces their adoption rate/speed of the platform. If they (hardware team) experience/share any UX issues and we do not address them, they will eventually minimise their interactions with the tool which will end up frustrating all stakeholders involved. I guess approaching UI/UX development from the angle of data analysis needs can bring about a bad UX/UI. We should be able to get all the data we need without directly translating data analysis formats/needs on the UI/frontend of the system - this issue is a great example that could also be borrowed for other sections like site registry.
I also like the point of allowing them to use their own device naming conventions but prevent them from renaming it during its lifetime >> sounds good and I guess it will not negatively affect their UX in the process.
Unless I am missing something, this somehow "uncomfortable" change does not prevent/stop data from getting out there. One can easily retrieve data using other unique fields like channel IDs, mongoDB IDs or even device Codes.
Could you shed more light on this statement? >>> "The same channel ID cannot be used for a new device in the same location". The statement makes a lot of sense and it is very relatable, I am just not sure how it connects with this issue. How does this proposed change bring about the utilisation of the same channel ID in the same location/site? How does the location/site come into the picture?
As for "name of device should be unique and permanent"......the channel ID, mongoDB ID, clarity deviceCode can also be used for analysis since they are also unique and permanent. Could there be any foreseen challenge in using other already existing unique fields (channel ID, deviceCode and mongoDB IDs) for analysis? Could it be that the analysts want the unique identifiers to be more descriptive in nature?
On a side note, this naming convention issue could also be one of the "invisible" blockers for site registry. What if it also negatively affects the user experience of the frontend users? Using system generated unique IDs for data retrieval could somehow ensure that the analysts get the data they want without negatively affecting the UX of the system by the UI users (Joel and rest of hardware team) hence a win-win of sorts.

This was adddressed.


Added Swedish Language

Based on the Danish strings.

Coverage decreased (-4.7%) to 87.481% when pulling a1fa4aebc0d191a085ff25e2da9ec97086d99bff on falk-h:swedish-language into d28a7cda618d5d6b95d5a9b6f71129a0870a1962 on MarshallOfSound:master.


[Business Analytics] [API Backend Usage Summary] Alignment issues in the table

Description:
Please see the screenshot below.

Noticed this in multiple tables. So need to review all the tables for a similar issue.


web-app-launcher not working

when running web-app-launcher.py I get the following error:
web-app-launcher.py", line 64, in main
    updater = Updater(TELEGRAM_API_TOKEN)
TypeError: __init__() missing 1 required positional argument: 'update_queue'

It looks like the Updater in the most recent version of PTB requires a second argument but only one is specified in the current code.

Hi @kercos, thanks for reporting!
I think the best is to update the  code and make it work with the new function. I will check that as soon as I have time!
Anyway, in case you manage to fix it yourself I would be happy to merge your code!
What version are you using?

Hi @MBoretto,
I'm currently running PTB 20.6.
If I'm able to solve it, I'll let you know.


Add scripts to download AudioSet

Summary
This PR adds scripts to download AudioSet.
In addition, SSAST starts to support pretrained weights.

Codecov Report
Attention: Patch coverage is 86.80203% with 26 lines in your changes are missing coverage. Please review.

Project coverage is 83.94%. Comparing base (46e6fa8) to head (22a0651).
Report is 2 commits behind head on main.

Files
Patch %
Lines

audyn/models/ssast.py
86.73%
26 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
- Coverage   83.96%   83.94%   -0.02%
==========================================
  Files          97       97
  Lines       10618    10797     +179
==========================================
+ Hits         8915     9064     +149
- Misses       1703     1733      +30

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix/Consolidate Docker Configs

Description
fzaninotto/faker cannot run on PHP > 7, and the Composer docker image uses PHP 8.
So, needed to move the composer install step into the PHP container, which is running 7.4
Went ahead and consolidate the Note containers into a single one via a new DockerFile.
Motivation and context
Wanted to simplify the docker config, but also needed composer install to run successfully.
How has this been tested?
Self-tested, chyea.
Screenshots (if appropriate)
NA
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:
Go over all the following points, and put an x in all the boxes that apply.
Please, please, please, don't send your pull request until all of the boxes are ticked. Once your pull request is created, it will trigger a build on our continuous integration server to make sure your tests and code style pass.

[x] I have read the CONTRIBUTING document.
[x] My pull request addresses exactly one patch/feature.
[x] I have created a branch for this patch/feature.
[x] Each individual commit in the pull request is meaningful.
[x] If my change requires a change to the documentation, I have updated it accordingly.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

:tada: This PR is included in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Setting up a MUI X License with Vite

Describe the bug
I have a components library where a Material-UI X Data Grid Pro is defined. It sets the license using the library's available methods.
The main app, which was created using react-scripts installs this library through yarn and uses the components provided within.
After migrating from react-scripts to Vite, the license of MUI-X is no longer set and a "unlicensed product" message is shown.
I've tried to set up the license inside the main app instead of the components library using the documentation to see if it fixes the problem:
LicenseInfo.setLicenseKey(import.meta.env.VITE_REACT_APP_X_GRID_LICENSE_INFO);
It seems like the license is set but apparently it's not because, as told by MUI "There are known issues with using MUI X components in a Vite environment that could affect license validation. Specifically, the verifyLicense function relies on environment variables such as process.env.NODE_ENV. Vite, by default, does not include process.env in the same way as Webpack, which can lead to issues in determining the correct environment and thus affect the license validation process."
Is there a way to set it up using process.env.NODE_ENV?
Thank you for your help
Reproduction

Steps to reproduce
No response
System Info
System:
    OS: macOS 14.1
  Binaries:
    Node: 22.0.0 - ~/.nvm/versions/node/v22.0.0/bin/node
    npm: 10.5.1 - ~/.nvm/versions/node/v22.0.0/bin/npm
    pnpm: 9.7.0 - ~/.nvm/versions/node/v22.0.0/bin/pnpm
  Browsers:
    Chrome: 127.0.6533.122
    Edge: Not Found
    Safari: 17.1
  npmPackages:
    @vite: 5.3.4,
    @mui/x-data-grid-pro: 4.0.2
    @mui/x-license-pro:  4.0.2
    @types/react: ^17.0.5 => 17.0.11
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2
    typescript: ^4.9.4 => 4.9.4

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

you can try the configuration of define below.
import { defineConfig } from 'vite'

export default defineConfig(({mode }) => {
  return {
    define: {
      'process.env': {
        NODE_ENV: mode
      }
    }
  }
})


Driver: QMolecule

I've got a molecule with calculated all the integrals in the spin-orbital basis (one_body_integrals and two_body_integrals). I want to run the computation of ground state energy with VQE.
I think that the best idea would be to prepare an object like QMolecule and set up its attributes manually and provide it to the qiskit_chemistry.core.hamiltonian.Hamiltonian.run() directly. I think it could be faster and easier than recalculate the integrals to MO basis and write the HDF5 file.
Do you think that it is a good approach or maybe this kind of task could be done simpler?
Going further, would it be useful to implement a driver to QiskitChemistry for providing a QMolecule object directly?

This issue really would be better in qiskit-chemistry repo since it seems more directly related. But I will address it here.
The QMolecule is the output of a chemistry driver, it contains integrals (in MO basis), electron counts, orbital energies etc. The HDF5 file is a stored version of the data in the QMolecule whereby a QMolecule can be created from such a file. It can be used when running a chemistry driver say to store the result so you know next time its the exact same molecular data. You could do this to allow the MOs to be created just once and reuse them if that computation were lengthy. We have an HDF5 driver that acts a chemistry driver reading a named HDF5 file and outputting a QMolecule so you can reuse etc. In normal use of driver such as PySCF, Gaussian etc an HDF5 is not created, the MO basis integrals come from the chemistry program/library.
If you want to want to somehow plug into a QMolecule integrals you got from elsewhere, as if you were a chemistry driver, then you would need to populate it out as the chemistry drivers do. From then on it can be used to create the FermionicOperator etc as we do today.
In answer to this Going further, would it be useful to implement a driver to QiskitChemistry for providing a QMolecule object directly? since the QMolecule is the output of a driver having it as input to a driver does not seem to make sense - such a driver would seem to be a null/empty driver and do nothing.

In the absence of any further discussion I am closing this


Python version 3.12 or later incompatible! I got this error when installing telseal!

/workspaces/Private-Set-Intersection/TenSEAL/build/temp.linux-x86_64-cpython-312/_deps/com_pybind_pybind11-src/include/pybind11/cast.h:449:26: error: invalid use of incomplete type ‘PyFrameObject’ {aka ‘struct _frame’}
449 |             frame = frame->f_back;
|                          ^~
In file included from /usr/local/python/3.12.1/include/python3.12/Python.h:42,
from /workspaces/Private-Set-Intersection/TenSEAL/build/temp.linux-x86_64-cpython-312/_deps/com_pybind_pybind11-src/include/pybind11/detail/common.h:124,
from /workspaces/Private-Set-Intersection/TenSEAL/build/temp.linux-x86_64-cpython-312/_deps/com_pybind_pybind11-src/include/pybind11/pytypes.h:12,
from /workspaces/Private-Set-Intersection/TenSEAL/build/temp.linux-x86_64-cpython-312/_deps/com_pybind_pybind11-src/include/pybind11/cast.h:13,
from /workspaces/Private-Set-Intersection/TenSEAL/build/temp.linux-x86_64-cpython-312/_deps/com_pybind_pybind11-src/include/pybind11/attr.h:13,
from /workspaces/Private-Set-Intersection/TenSEAL/build/temp.linux-x86_64-cpython-312/_deps/com_pybind_pybind11-src/include/pybind11/pybind11.h:45,
from /workspaces/Private-Set-Intersection/TenSEAL/build/temp.linux-x86_64-cpython-312/_deps/com_pybind_pybind11-src/include/pybind11/complex.h:12,
from /workspaces/Private-Set-Intersection/TenSEAL/tenseal/sealapi/sealapi_encode.cpp:1:

Hi!
I am not sure how to help. You can try installing an old version of TenSEAL, as used in the implementation.
tenseal==0.3.4


Executor doesn't implement cetl::rtti

For example: https://github.com/OpenCyphal-Garage/libcyphal/blob/main/include/libcyphal/executor.hpp#L108
We have the same methods defined but they are not virtual and the cetl::rtti interface is not used.

The line you mention is not really about IExecutor but its Callback::Interface. But you are right - I need review all places where rtti is implemented. The reason why it's working currently (and causing any problems I'm aware of) is b/c cetl::rtti_cast<> implementation is not requiring currently to inherit from cetl::rtti interface - @pavel-kirienko Pavel, is it a feature or a bug? ;)
IMO on one hand it could be a feature, namely:

if you have very strict and predictable scenario from which type to what type you gonna rtti_cast
then you don't really need it be polymorphic - executor's callback stuff proves it (for better or worse)
On the other hand, it is probably not good that we have rtti interface defined but don't enforce it properly... f.e. if I add to the interface a new pure virtual method then I would expect to see compilation failures about classes being abstract...

That interface is mostly needed for the rtti_helper, so consider it a feature. We should implement that interface at least for regularity, though (making it final to avoid vtable lookup overheads where possible).


Remove ENTITY_TYPE references

See https://github.com/Senzing/community-roadmap/issues/66

PR merged


Allow for linktime conditional branching

Part of https://github.com/scala-js/scala-js/issues/4991
Motivation
The motivation behind this feature was to allow switching the implementation based on whether the target is Wasm running in the browser (WasmJS) or a standalone Wasm application. For example, with regular expressions, WasmJS could reuse the JavaScript Regex implementation, while a standalone Wasm application would use a pure-Scala Regex implementation.
While the original motivation is from Wasm use cases, the feature itself is self-contained and can be used for optimization purposes in the JavaScript backend as well.
How to
The main idea is to introducing the new IR node LInkTimeIf like:

To put into writing something we discussed offline at some point: I believe the easiest and cleanest way to do this is with a new IR node that would look like

case class LinkTimeIf(cond: LinkTimeCondition, thenp: Tree, elsep: Tree) extends Tree

Then, our 3 main link-time passes process it as follows:

the Analyzer, for the reachability analysis, deals with it during the Infos builder. Since the Infos builder walks the IR trees to build Infos, it can resolve the LinkTimeCondition and only recurse in the appropriate branch
the OptimizerCore would trivially replace the LinkTimeIf with either thenp or elsep, allowing further optimizations
the backends would do the same, if any LinkTimeIf survives until then (normally, only if the optimizer is disabled)

The precise nature of LinkTimeCondition remains to be determined, but it would fairly limited. Definitely not a full Tree.

Originally posted by @sjrd in https://github.com/scala-js/scala-js/issues/4991#issuecomment-2154680386
Where the LinkTimeCondition should be like same as scala-native's LinktimeCondition
How to: frontend

Introduce @LinkTime (or @resolvedAtLinktime like SN) annotation, only the annotated values (functions?) can be used as part of LinkTimeCondition
if expression will be translated to LinkTimeIf instead of If, when the condition is only composed of @LinkTime annotated value, Literals, and BinaryOps.

For example, the if will be translated to LinkTimeIf because the condition is composed of @LinkTime annotated booleans + BinaryOp. (if it's mixed with the runtime-value, we should fail linking).
@LinkTime
def foo: Boolean = ...
@LinkTime
def bar: Boolean = ...

if (foo && bar) { ... }

I'm not sure about magically turning an if into a LinkTimeIf depending on some non-local information. That would be quite brittle, as there would no compile-time reporting if something that should be a LinkTimeIf falls back to a regular If.
I believe the more Scala.js way to do this would be a dedicated primitive in scala.scalajs.LinkingInfo. Something that we would write
LinkingInfo.linkTimeIf(LinkingInfo.isWebAssembly) {
  ifTrue
} {
  ifFalse
}


Update workflow, add MIT License

Thought I'd send an update to this :)
unfortunately it fails with
>   {dpss requires Signal Processing Toolbox.
~~https://github.com/sappelhoff/matprep_artifacts/actions/runs/12037815775/job/33562095751#step:6:160~~
Did you encounter this problem when you first coded this up @a-hurst ?
EDIT: it passes now: https://github.com/sappelhoff/matprep_artifacts/actions/runs/12157766138

I think ideally we would move towards doing the matprep artifacts as part of pypreps CI 🤔 see also https://github.com/sappelhoff/pyprep/issues/160

@sappelhoff Sorry for the delayed response! Found out at the end of last week that my PhD supervisor is leaving our university for a Dean position halfway across the country which has thrown me and the other members of the lab for a bit of a loop 😓

I think ideally we would move towards doing the matprep artifacts as part of pypreps CI 🤔 see also sappelhoff/pyprep#160

Definitely open to this if it makes maintenance easier! I guess you'd need to set up a separate Github Action that generates the artifacts and stores them in a way that can be easily pulled by the unit test scripts?
My only thought re: maintainability is that if the idea is to regularly & automatically regenerate the artefacts using the latest EEGLAB and PREP, it might be useful to have a check of some kind to test the hashes of the artifacts and see whether they've actually changed since last time. Without this, a stray EEGLAB change that ends up breaking MatPREP could cause the artifacts to change and the PyPREP CI to fail and we'd have a hard time pinning down whether it was something we broke or them!

Found out at the end of last week that my PhD supervisor is leaving our university for a Dean position halfway across the country which has thrown me and the other members of the lab for a bit of a loop 😓

oh no, that is tough. How far along are you with your PhD? I hope you will find a solution 😬

I guess you'd need to set up a separate Github Action that generates the artifacts and stores them in a way that can be easily pulled by the unit test scripts?

I think it will be possible to have this just as a step, and make the following steps depend on the successful completion of that step

My only thought re: maintainability is that if the idea is to regularly & automatically regenerate the artefacts using the latest EEGLAB and PREP, it might be useful to have a check of some kind to test the hashes of the artifacts and see whether they've actually changed since last time.

This is a good point! We should do exactly that.

As for this PR, I pinned the EEGLAB version now, so this is ready for a merge :)
all further plans that we discussed above might have to wait a bit.

Great, merged!

oh no, that is tough. How far along are you with your PhD? I hope you will find a solution 😬

Have about a year left! Supervisor says he'll stay a professor emeritus at our current institution so we won't need to find new supervisors and the lab space will stay up and running until we're all done (according to him anyway), but there's still a lot of unknowns ahead. I think I should be fine (done a good chunk of thesis data collection already) but since he's going to be gone by March and will also very busy after (and before) the move we're essentially going to have to learn to run the lab without him.
(Also sorry, tried to respond and somehow ended up editing your comment by mistake! Fixed now)

learn to run the lab without him

having gotten to know you a little bit (online) over the past years, I have a high confidence that you'll make it ;-)
still ... these things happen, but one always hopes not to be in the position oneself. Fingers crossed for you!


Plans to support NextJS 13 Server Actions?

Any plans to support server actions?
https://nextjs.org/docs/app/building-your-application/data-fetching/server-actions
would be great to have
  async function addItem(data) {
    "use server";
    new createServerActionSupabaseClient({cookies: cookies()});
  }

We are looking into it but at the moment this is an experimental feature and likely to change so we will likely wait until it becomes stable.

Look like I spoke too soon. Since the API is the same as the RouteHandler at the moment it looks like one of my team mate (@dijonmusters) has added in provision for it in the .next release which will be made final soon. https://github.com/supabase/auth-helpers/blob/next/packages/nextjs/src/serverActionClient.ts

Thank you!


Json validation and fix

Description

This PR consolidates json string validation and loading from index, report and llm three parts into one piece.
including trim extract description from llm result, replace or convert unfit chars, using json_repair for the finally repair before json loading. wrapped all functions into try_parse_json_object() as unique one function to use.
There are two duplicated json_clean_up functions in both index and llm section, consolidate into one under llm model.
Furthermore, wrap json_clean_up into try_parse_json_object() function without Exception for friendly retry

Related Issues
Proposed Changes
add json-repair
Checklist

[x] I have tested these changes locally.
[x] I have reviewed the code changes.
[x] I have updated the documentation (if necessary).
[ ] I have added appropriate unit tests (if applicable).

Additional Notes
[Add any additional notes or context that may be helpful for the reviewer(s).]

this is fruther consolidate version for
JSON parsing: always fix all incoming json when operating _manual_json mode #551
https://github.com/microsoft/graphrag/pull/551

Hi @AlonsoGuevara, Could you please review it? Happy to work on it and taken any suggestion.

rebased from main as requested

merge as #801


Choose Highlight Languages

This PR allows content creators the ability to specify the languages to include using the langs array param in the frontmatter of the page

hi @bad-hombres
thanks for your pr. this feature is what I wanted to do earlier. 👍
And the other thing should be done:
please add corresponding params and useful comment at exampleSite/config.yml

If they're not replying, I can send in a PR implementing it 😅

If they're not replying, I can send in a PR implementing it 😅
okay~


HHH-17612 DefaultRevisionEntity: Illegal argument on static metamodel field injection

https://hibernate.atlassian.net/browse/HHH-17612

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

Superseded by https://github.com/hibernate/hibernate-orm/pull/9158


Improvements

I have just added some changes that were useful for my project and maybe they can be useful for other users too:

Allow public access to Alert properties.
Add a new property to AlertOnBoarding to set the contentMode of the UIImageView.
Add new outlets to AlertChildPageViewController to allow adjusting the aspectRatio of the imageView (increasing or reducing its size).

Thanks for the pull request!


gitignore update

I don't know why bu some times npm creates log files with name like npm-debug.log.2546534529. I added them into .gitignore

I just noticed that project is packed with .vscode folder. I added it into .npmignore


Submodule Use Case

I've utilized FLARE-EMU in a project as a submodule to allow me to make modifications in which I can later potentially submit as PRs.  An included __init__.py file would prevent me from having to check for it within my scripts & create it if needed. Is this something that you could consider adding, or would you suggest an alternate workflow?

For use with IDA Pro, it is recommended to copy the scripts to IDA Pro's python directory.


run with more parallelism (-count=3)

go help testflag
-count n
    Run each test, benchmark, and fuzz seed n times (default 1).
    If -cpu is set, run n times for each GOMAXPROCS value.

go test -race -cover -coverprofile=coverage.out -timeout 70s ./... -count=3

We won't do this.


Git Bash opens in new window on Windows

The default terminal is Powershell, but I want it to be Git Bash.
I have set Shell Override to C:\Program Files\Git\git-bash.exe but this opens in a new window.

Shell Override: C:\Program Files\Git\bin\sh.exe
Shell Arguments: --login

I was curious as to how some people got git bash running in Terminal-Plus on Windows, so thanks for this!
I'm glad that you were able to resolve your issue. Thank you for using Terminal-Plus.

It's great. Although I'm having an issue which is preventing me from using it. As I type and hit backspace and move about the command line, weird things happen. Characters move around and things get displaced. Some characters but unpassable and undeletable.
The only resolution is to hit enter - Esc or Ctrl+C do not work.
Not sure if this is your problem or a problem with Powershell and Git Bash Shell...

Just tried it in normal Powershell, no issues there.
I tried to capture a screen recording but Recordit was having trouble with my resolution I think... Will try again in the morning.

I'll have to test this myself when I have time. The recording you mentioned would be nice so I can see exactly what is going wrong.

See how all the text is being rearranged. Soon after the clip starts, a can't traverse back past a...

Thanks for the animation, I see what is happening now. Unfortunately I won't have time to test this myself until next week. Sorry for the inconvenience. I'll be sure to let you know once I have tested this and fixed it.

Hi @jeremyramin, I was wondering if you've made any progress sorting this out for Git Bash; I'd love to make use of it but it's impossible to with this backspace bug.

have same issue with git bash as @matthewcanty has

Hi @jeremyramin I'm still having the same issue with git bash as @mattcanty has in 2017, please fix it. Please


"from" keyword doesn't seem to be taken into account properly

The from sbt keyword as for instance in:
"ccl.northwestern.edu" % "netlogo" % netLogo5Version % "provided" from s"https://github.com/NetLogo/NetLogo/releases/download/$netLogo5Version/NetLogo.jar"
should be top priority over the local and remote resolutions. It was working fine with M13 but with M14 coursier try to reslove the dependency even if the location is specified using "from".

It was sometimes done this way by accident, in M13, the correct behavior seems to give the URL a low priority, per the SBT scaladoc of ModuleID.from:
  /**
   * Specifies a URL from which the main artifact for this dependency can be downloaded.
   * This value is only consulted if the module is not found in a repository.
   * It is not included in published metadata.
   */

It's weird, it doesn't work the way it is described here in SBT.When I disable coursier, the resolution append as I would expected (from is given highest priority and the compilation is successful, whereas it doesn't work using coursier M14).

Actually I think I misinterpreted the SBT error. The "from" keyword doesn't seem to be taken into account at all by coursier. In my dependencies I have:
libraryDependencies += "ccl.northwestern.edu" % "netlogo" % netLogo5Version % "provided" from s"https://github.com/NetLogo/NetLogo/releases/download/$netLogo5Version/NetLogo.jar"
and I get this error:
coursier.ResolutionException: Encountered 1 error(s) in dependency resolution:   ccl.northwestern.edu:netlogo:5.3.1:     not found:       /home/reuillon/.ivy2/local/ccl.northwestern.edu/netlogo/5.3.1/ivys/ivy.xml       https://repo1.maven.org/maven2/ccl/northwestern/edu/netlogo/5.3.1/netlogo-5.3.1.pom       https://oss.sonatype.org/content/repositories/snapshots/ccl/northwestern/edu/netlogo/5.3.1/netlogo-5.3.1.pom       https://oss.sonatype.org/content/repositories/releases/ccl/northwestern/edu/netlogo/5.3.1/netlogo-5.3.1.pom       https://github.com/NetLogo/NetLogo/releases/download/5.3.1/ [error] (ccl-northwestern-edu-netlogo5/*:coursierResolution) coursier.ResolutionException: Encountered 1 error(s) in dependency resolution: [error]   ccl.northwestern.edu:netlogo:5.3.1: [error]     not found: [error]       /home/reuillon/.ivy2/local/ccl.northwestern.edu/netlogo/5.3.1/ivys/ivy.xml [error]       https://repo1.maven.org/maven2/ccl/northwestern/edu/netlogo/5.3.1/netlogo-5.3.1.pom [error]       https://oss.sonatype.org/content/repositories/snapshots/ccl/northwestern/edu/netlogo/5.3.1/netlogo-5.3.1.pom [error]       https://oss.sonatype.org/content/repositories/releases/ccl/northwestern/edu/netlogo/5.3.1/netlogo-5.3.1.pom [error]       https://github.com/NetLogo/NetLogo/releases/download/5.3.1/ [error] Total time: 143 s, completed 20 sept. 2016 14:46:20
And it works when coursier is disabled.

Should be fixed by https://github.com/alexarchambault/coursier/issues/373, included in 1.0.0-M14-7 and 1.0.0-M14-8 in particular.


all cores at 100% when using dynmap-structures.

Hello,
I tried the plugin on bukkit 1.8.8R0.1-SNAPSHOT with a pre-generated world wit a radius of 10000 (using worldborder). After starting the server it will start running at 100% on all cores within a minute.
I am running the next plugings (33): Stables, GroupManager, WorldEdit, FrameProtect, Vault, PlotMe, Prism, WorldGuard, dynmap, DynmapPlotMe, Chairs, Recycler, PaintWar, Lockette, Sleep, MiningTNT, PlotMe-DefaultGenerator, PunchWood, Essentials, Citizens, Jobs, WorldBorder, EssentialsSpawn, Multiverse-Core, dtlTraders, Sentry, pvparena, LimitedCreative, CompatNoCheatPlus, ChestShop, NoCheatPlus, Multiverse-Portals, Multiverse-NetherPortals
and OpenJDK Runtime Environment (IcedTea 2.6.1) (7u85-2.6.1-5ubuntu<IP_ADDRESS>)
OpenJDK 64-Bit Server VM (build 24.85-b03, mixed mode) on Ubuntu 14.04 64bit with all updates
Can you help me diagnose the problem? is it the size of the world or might there be an other problem?
Best regards,
Thierry.

Could be the size of the world, specifically because that increases the file size for all the structure's NBT .dat files. Which version of the plugin are you using and which structures do you have enabled in the dynmap-structures config.yml? I'll do a few tests after you get back to me and see what I can do.

The size of the world is 5.8GB and I use version dynmap-structures-0.1.3-all.jar. It takes a few minutes to happen or maybe a player has to start playing on the server. It also looks like the map is complete, it shows structures until the world border, but maybe it starts updating the map?

Here
is my config file:
structures:
fortress: true
mineshaft: false
monument: true
stronghold: false
temple: true
witch: true
village: true
layer:
name: Structures
hidebydefault: false
layerprio: 10
nolabels: false
minzoom: 0
Thanks for having a look at it, it would be a really nice addition to our server. :)

I have removed all plugins, except dynmap and dynmap-structures. It looked better, but only after a minute or two one core stayed at 100% again, the other core was between 30 en 40%, even when nobody was online. I had hoped it was only in combination with an other plugin, that I might have could done without.
Let me know if there is something else I can test for you.

I haven't had time to look into this yet, though I should in a day or two. Just want to make sure; dynmap isn't generating tiles, right? dynmap-structures should only ever be using CPU on startup (to read the structures files once) and whenever the structures files are changed (generally from new chunks with structures in them are being generated and written to disk).

I should have a new version out with a fix soon!
I made a mistake with the file watcher - I had never used it before and I had a line out of a loop when it should have been inside.
Thank you so much for reporting this problem!

Closed by commit/77c4d0d94de3ca5f5768fd1c2132be5384055801.

The new version should be available on dev.bukkit.org shortly.

Thank you for the quick fix. I have tested it on my server and it runs great now.
With some luck I can get the kids to dos some more exploring now. :)


Chrome/Firefox Extension?

Kindof a dumb idea, but would be handy for chromebooks if its even possible
It would basically be a button you could click, it would launch code-server in a window/tab, it would probably not have the binaries availiable (git, tools, the likes) or one could store the files on a EXT USB, or a FAT32 and just ignore permissions that one would see in a ext partition. Or have it function just like sshcode where you supply user/pass/host, there is a SSH implementation for chromebooks/chrome
chrome.google.com/webstore/detail/secure-shell-app/pnhechapfaindjhompbnflcldabbghjo < APP
chrome.google.com/webstore/detail/secure-shell-extension/iodihamcpbpeioajjeobimgagajmlibd < EXTENSION
Again, dumb idea, not gonna be surprised if this is closed imediatly, but in theory, it would be interesting!

This is out of scope for code-server. We have sail for this.

Well not everyone can use Docker. Specifically those with a WINDOWS HOME
LISCENCE, or have too many issues with Docker for it to even be a viable
option at all.
On Mon, Sep 30, 2019, 3:22 PM Ayane Satomi<EMAIL_ADDRESS>wrote:

Closed #1043 https://github.com/cdr/code-server/issues/1043.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cdr/code-server/issues/1043?email_source=notifications&email_token=ACPQOXUXLARPUSMF6TSYXVDQMJ33LA5CNFSM4I37FCHKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOT5YTJ3Q#event-2674996462,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACPQOXVNZXTFOYI4S4L3PR3QMJ33LANCNFSM4I37FCHA
.

Again, out of scope of the project. We won't be discussing this further. Such a function is something reserved for PaaSes. We're not a PaaS, we're a IDE.

If you really want to, you can use this IDE as a building block, make a firefox extension that makes a button that will automatically make a workspace or clone stuff for you, ala Eclipse Che. But then, we're not a workspace server - we're a IDE. There's a difference between a PaaS and a Cloud IDE.


Use race id to compare races

At the moment we use race timestamps to identify races. With #431 we add a UUID as unique identifier of a race. We should use this also for listing and comparing races. For backwards-compatibility reasons we will leave one release between #431 and this one.

Let's stick with the timestamp as default for now as it is unique for the typical user and simpler to understand (the UUID is opaque but the user can understand the timestamp).


update AWS provider pinning to v3 for TF 0.13 version compat

what
do not restrict the aws provider version to just v2
why
https://github.com/cloudposse/terraform-aws-route53-cluster-hostname/issues/23
the aws cloud provider has v13 fixes and requirements also
references
closes #46
https://github.com/cloudposse/terraform-aws-s3-bucket/issues/46

you seem to be saying the inverse here as oppose to the dynamodb repo, should I leave the defined limits on or off here?

I have gone with the suggestions made by @aknysh here: https://github.com/cloudposse/terraform-aws-dynamodb/pull/62#discussion_r480600230

/rebuild-readme

/test all


Disable visual in android since they run in linux-chrome instead

This change is

Reviewed 1 of 1 files at r1.
Review status: :shipit: complete! all files reviewed, all discussions resolved

Comments from Reviewable


Issue new release for new Firecracker version and CNI changes

The SDK could use a new official release that includes the recent upgrade to Firecracker v0.18.0 and the support for CNI-configured network interfaces.
Both those changes are backwards incompatible, which needs to be included in the release notes.

This is done: https://github.com/firecracker-microvm/firecracker-go-sdk/releases/tag/v0.19.0


Web support

Would there be interest in adding support for Web through the File System API?
It would be pretty straightforward, as it's all natively async anyway and there would be no need to spawn any blocking threads. Probably the most interesting part is documentation and converting errors to io::Error.
All major browsers already support this, except Safari which doesn't support writing files in async, but only in sync.
Happy to make the PR of course!

I guess this falls under #24.
My main concern is how the File System API you linked would map onto the std::fs API. From a brief look at the API it looks like it doesn't map too well onto the file system of paths. I guess reading and writing would be relatively the same, but I'm concerned about how APIs like create_dir or metadata would map onto the web API.
I guess we could have a set of extension functions for opening the Files, having the Files open for reading and writing, and make all of the other APIs return "unsupported" errors. Does this make much sense?

I guess reading and writing would be relatively the same, but I'm concerned about how APIs like create_dir or metadata would map onto the web API.

I think we should use OPFS by default, see StorageManager.getDirectory(). This would allow us to use create_dir(), see FileSystemDirectoryHandle.getDirectoryHandle().
metadata is also mostly covered, but we would have to create our own type that mirrors Std's Metadata:

accessed: not available
created: not available
file_type/is_dir/is_file: FileSystemHandle.kind (accessed through FileSystemFileHandle.getFile())
len: Blob.size (inherited through File, accessed through FileSystemFileHandle.getFile())
modified: File.lastModified (accessed through FileSystemFileHandle.getFile())
permissions:

readonly: FileSystemHandle.queryPermission() (only supported on Chrome, but considering we are using OPFS it's no-op anyway)
set_readonly: FileSystemHandle.queryPermission() (only supported on Chrome, but considering we are using OPFS it's no-op anyway)

Will look into making a PR soon!


Replace names of technologies in 'Tech Stack Used' in READMe.md with their respective icons. #12

Hey there! thanks for making a Pull Request! Unfortunately, this has been assigned to someone else. Kindly have a look at #3 instead


Change travis config to terminate w/ error if unit tests fail

Describe the bug
Change travis build to terminate if unit tests fail
Expected behavior
Build should fail if there are failing tests

This is related to the build configs; opened another ticket for it.


appengine-java8/images - should use google-cloud-storage instead of gcs api

Do you mean com.google.cloud.storage instead of com.google.appengine.tools.cloudstorage?
What is the reason?
And can the blobstore part in the example also be removed?

Java8 can use standard networking and the Veneer's work there.  It lowers the support surface as well.

I'll look, I know that what I'd like to do is show some examples using the Java Image API's for J8 now that we no longer have restrictions on using them.

Notice: this issue has been closed because it has been inactive for 699 days. You may reopen this issue if it has been closed in error.


Feature/multi id vanes

Add starting up the hall effect sensor in rc.sensors
Extend the hall driver to accept different i2c addresses
Implement handling the hall measurements in the sensors module and publish the airflow angles messages
Add the airflow angles mavlink stream

Tested in lab with two different vanes with EZG3

I updated the formatting with astyle.
I would not be totally opposed to changing the airflow angles to the same ID and adding an additional field indicating if it is the aoa or slip. However, for the moment I would like to keep it as is, since the whole post processing pipeline works with those two uOrb message types and we would have to change it quite a bit to work with this new definition. We could do this change in a new PR after everything is tested in the current form, which is the state on our older firmware.
At some point we will have to associate the measurements either with the aoa or slip and I think the sensors module is a good way to do it.

Merging to address addiitonal changes in https://github.com/ethz-asl/ethzasl_fw_px4/pull/18


[Supply] Support Android custom closed tracks

Feature Request
Allow supply track parameter to be a custom name corresponding to a user-defined closed track on the google play store.
Motivation Behind Feature
At this years google IO, google changed all beta tracks to be open betas by default. This is problematic for anyone who wants to do a closed beta. To provide this functionality, google now allows multiple closed tracks that can have custom names. Fastlane doesn't seem to support custom tracks, since supply fails with track must be one of 'production', 'beta', 'alpha'... etc. This is very problematic for anyone requiring more than one closed track (e.g. a closed alpha and beta).
Previously it was possible to set beta tracks to be closed betas, so having closed beta and alpha tracks was easily achievable with Fastlane. However now that beta tracks are open by default, you need a custom closed track to recreate that functionality, and this is not currently possible with Fastlane (supply).
Feature Description
Supply allow upload apk to custom closed track.
Alternatives or Workarounds
Workaround currently is to upload apks manually to the custom closed track.

@authuser Hey, this is actually implemented in the latest master (https://github.com/fastlane/fastlane/pull/12487). This will get pushed out in the next release 😊


No autocompletion for C++ on VSCode, test failed.

Thanks in advance for sharing cquery.

Run tests failed, see the logs below:
(FYI, I have clang-6.0 installed from apt)

[doctest] doctest version is "1.2.7"
[doctest] run with "--help" for options
	Arg /usr/bin/clang != %clang
===============================================================================
/home/sywe1/Downloads/cquery/src/c_cpp_properties.cc(108)
TEST SUITE: CCppProperties
TEST CASE:  basic

/home/sywe1/Downloads/cquery/src/c_cpp_properties.cc(159) FATAL ERROR!
  REQUIRE( args_equal )
with expansion:
  REQUIRE( false )

(   0.000s) [main thread  ]clang_system_include_extractor.cc:60    | Unable to parse clang output
asdfasdf
   #include <...> search starts here:
one
Enasdfd of search list.
asdfasdf
===============================================================================
[doctest] test cases:     69 |     68 passed |      1 failed |      0 skipped
[doctest] assertions:    430 |    429 passed |      1 failed |
[doctest] Status: FAILURE!

No autocompletion on VS Code
I'm working on a cpp project and using cmake as the build tool. The compile_commands.json file is created during configuration in the build directory. But neither creating a link  to the compile_commands.json in the project root nor setting ./build/ to the cquery.misc.compilationDatabaseDirectory will work. The only working solution is to manually create a .cquery and add some includes, but it seems quite awkward.  Also, I have disabled the c/c++ extensions to avoid conflicts, but I am still not able to figure out why it does not work.

The output of cquey is
[Error - 12:00:33 AM] Request textDocument/documentLink failed.
  Message: /home/sywe1/swei-project/software/test/loop_synthesis_ops_test.cpp is being indexed.
  Code: -32002
[Error - 12:00:33 AM] Request textDocument/codeAction failed.
  Message: /home/sywe1/swei-project/software/test/loop_synthesis_ops_test.cpp is being indexed.
  Code: -32002
[Error - 12:00:33 AM] Request textDocument/codeLens failed.
  Message: /home/sywe1/swei-project/software/test/loop_synthesis_ops_test.cpp is being indexed.
  Code: -32002
[Error - 12:00:33 AM] Request textDocument/codeAction failed.
  Message: /home/sywe1/swei-project/software/test/loop_synthesis_ops_test.cpp is being indexed.
  Code: -32002

I think I have figured out. The cxx compiler used by cmake_tool plugin which generates compile_commands.json is g++, after switching to clang++, everything is perfect.


How to apply the nowcasting command?

Hi:
I want to apply nowcasting cli command. I found there are four functions:
'nowcast_accumulate, nowcast_extrapolate, nowcast_optical_flow,
nowcast_optical_flow_from_winds' . So what's the differences
between  nowcast_optical_flow and nowcast_optical_flow_from_winds,
What's the order of these four functions? Are there examples to use those functions?
Thanks very much.
Dave

Hi Dave,
I'll try and run through a few things that might help.

nowcast_optical_flow

One of the two ways to calculate advection velocities, following the method in:

Bowler, N., Pierce, C. and Seed, A. 2004: Development of a precipitation nowcasting algorithm based upon optical flow techniques. Journal of Hydrology, 288, 74-91.

This calculates optical flow velocities between the radar fields at T-1 to T+0 and between the radar fields at T-2 to T-1. An average of the two calculated velocities is returned.

nowcast_optical_flow_from_winds

The alternative method for calculating advection velocities. This works by picking a model wind level (steering flow) and advecting the T-1 radar field to T+0 using these wind velocities. The advected field is then compared to the T+0 radar field and the perturbation to the model wind velocities required to better match the two fields is calculated. The perturbed velocities are returned.

nowcast_extrapolate

Used for precipitation rates. Extrapolates the observed precipitation rate out at the desired time intervals (lead_time_interval) out to a specified maximum lead time (max_lead_time)

nowcast_accumulate

Used for precipitation accumulations. Extrapolates the observed precipitation rate out at 1 minute intervals out to the specified maximum lead time (max_lead_time) and from these calculates accumulations with the specified period (accumulation_period).
The user can specify the lead time interval at which they want accumulations to be produced (lead_time_interval). So for example a user might want 15 minute precipitation accumulations, but may wish to produce these every 5 minutes (overlapping); accumulation_period=15, lead_time_interval=5.

The order in which to call the steps is to calculate your advection velocities using one of the optical flow methods and then use the resulting velocities to calculate your outputs; use extrapolate to produce rate fields, and use accumulate to produce accumulation fields.
You'll notice that these steps also require an orographic enhancement field. There are tools to calculate this field within IMPROVER. This calculates the expected component of precipitation over hills / mountains that is triggered by orographic uplift. The radar fields are modified to remove this component before calculating the advection velocities. The reasoning here is that we would not expect to see this component move, it's related to the underlying static terrain. The same orographic enhancement field is also required as input to the extrapolation or accumulation steps as in these we reintroduce this enhancement in the original locations. If you want to simply technically demonstrate that you can string these things together you can create a suitable field of zeros, but note that the advection velocities calculated may be poor.
There's a lot more that can be done, e.g. spatially blending the nowcast fields into model fields to make up for missing areas of radar coverage. Hopefully the above is enough to help you make progress.

Hi bayliffe,
Thank you for your detailed reply. I learn a lot and make the nowcast processing clear. I want to further understanding of the nowcast_optical_flow_from_winds and orographic enhancement methods. Are there any papers or documents about these methods? Thanks again.
Dave

@NMC-DAVE I can confirm that there are no documents or papers I can point you to in relation to the nowcast_optical_flow_from_winds method. My colleague provided the following comment describing how the method came about:

It seemed like a sensible idea based on the original optical flow paper (Bowler 2004), as optical flow relies on the assumption of continuity and that the perturbations / movement is smaller than a certain radius, and gets less stable as the displacement between frames gets larger.  It was also attempting to deal with the situation where there isn’t enough rain to reliably calculate an advection velocity

Hopefully all of the above has been useful. If you have enough information for now, please close this issue.
Regards,
Ben

There has been no further correspondence. Closing this issue.


Update

👷 Deploy Preview for 11tytask processing.

Name
Link

🔨 Latest commit
20b1f7651d1eaf20df1f99b89b21a972bb19d50c

🔍 Latest deploy log
https://app.netlify.com/sites/11tytask/deploys/653a858017b6e8000802374b


sink/mq(ticdc): flush waits for all events to be acked

What problem does this PR solve?

Issue Number: ref https://github.com/pingcap/tiflow/issues/5656
What is changed and how it works?
flush waits for all events to be acked.
Check List
Tests

Unit test

Questions
Will it cause performance regression or break compatibility?
None
Do you need to update user documentation, design documentation or monitoring documentation?
None
Release note
Fix an issue where MQ sink checkpointTs may advance early

/run-all-tests

Codecov Report

Merging #5657 (2850fae) into master (fcea4d5) will increase coverage by 0.5698%.
The diff coverage is 58.1241%.

:exclamation: Current head 2850fae differs from pull request most recent head 9be83e0. Consider uploading reports for the commit 9be83e0 to get more accurate results

Flag
Coverage Δ

cdc
62.0000% <62.7978%> (+0.8519%)
:arrow_up:

dm
51.9966% <29.5774%> (-0.0461%)
:arrow_down:

engine
58.7260% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #5657        +/-   ##
================================================
+ Coverage   56.0768%   56.6466%   +0.5698%
================================================
  Files           535        675       +140
  Lines         70143      79002      +8859
================================================
+ Hits          39334      44752      +5418
- Misses        27078      30049      +2971
- Partials       3731       4201       +470

/run-all-tests

/run-all-tests

/merge

/run-integration-test
/run-kafka-integration-test


Change options passed to ls on rubies search

On macOS High Sierra the options to ls must be lower case.
Previously when running
$ source /usr/local/share/chruby/chruby.sh
would result in
Unrecognized option: 'A'.

The -a flag would stop ignoring entries that start with a dot. The desired behavior from the -A flag is to not list the implied . and .. directories.

I have a macOS High Sierra and ls -A (with a capital 'a') is working correctly
$ sw_vers -productVersion
10.13.1
$ ls -A ~/Tmp/test
t1.txt t2.txt


Build spaCy for conda (conda package out of date)

We need to host the conda build, to make sure it stays up-to-date. Currently conda installs v0.89. The tutorial code is often broken for this version, esp. for word vectors.
Example build file here: https://github.com/conda/conda-recipes/tree/master/spacy
See also the conda docs.

@henningpeters , or anyone who wants to help out? :)

Have a look at this:
https://anaconda.org/henningpeters/spacy/files
Build scripts are in my fork at:
https://github.com/henningpeters/spaCy
https://github.com/henningpeters/spaCy/commit/6f3ea516849ba260492faf1782f4c4f4998b334e
If we want to integrate this with spaCy we have to think how / if we mix it with the current pip workflow, any suggestions/preferences?
I didn't have any experience with conda and was sceptical why we need another package manager, but from a technical design I really like it over anything that's been out there in Python packaging until now.

Great, thanks.
Haven't checked the build yet, but try removing Cython as a run-time dependency. Usually we only want to build with Cython, and let people install from the .c and .cpp files.

Ok, btw: I didn't check the package thoroughly enough, getting following error on import:
ImportError: No module named 'spacy.tokenizer'
Any ideas?

I downloaded and unpacked it, and yeah the .cpp, .c or .so files aren't there --- so, it's not been built correctly.
I would expect there to be some sort of mechanism for including data files and extension modules in the package. So, I think that part of the config still needs to be written.

How do you install a local build (via setup.py build_ext --inplace) system-wide? The problem is that conda expects the build to be installed in the system and then fetches the files that got installed.

Building conda packages from spacy source is trickier than expected. Not clear how to integrate well with setuptools/pip dependencies. Hence, for now conda builds are based on pypi (which is the standard approach anyways). Here you go:
https://github.com/henningpeters/spaCy-conda
https://travis-ci.org/henningpeters/spaCy-conda
https://anaconda.org/henningpeters/spacy
This also means that we are packaging spacy 0.93 including unidecode and the old license terms. Once we are based on text-unidecode (as long as it isn't officially packaged) we will use:
https://github.com/henningpeters/text-unidecode-conda
https://anaconda.org/henningpeters/text-unidecode

I'll give a clean install using this conda on my reformatted rMBP.
rMBP: Retina, 13-inch, Late 2013
Yosemite: 10.10.5 (14F27
Processor: 2.4 GHz Intel Core i5
Memoru: 4 GB 1600 MHz DDR3
Graphics: Intel Iris 1536 MB

Is this resolved now? Specifically, when we push new spaCy to PyPi, do we also ensure that a new version is pushed to conda?

it's not ensured, but building and uploading conda packages is automated
that's ok as one need to manually adjust dependencies, etc.
hence i would say make it as closed


add: Isotonic Regression

Describe the algorithm/code
Isotonic regression is a regression technique in which the predictor variable is monotonically related to the target variable. This means that as the value of the predictor variable increases, the value of the target variable either increases or decreases in a consistent, non-oscillating manner. PLease assign me this issue under hacktoberfest tag
Do you want to work on this feature?

[X] I am willing to implement this feature.

Hi @adi271001! 👋🏻
You can start working on this issue! Do let me know if you need any help. ✨

created pull request and added codes


feat(NODE-5718): add ReadConcernMajorityNotAvailableYet to retryable errors

Description
Add ReadConcernMajorityNotAvailableYet to retryable errors
What is changing?
See above.
Is there new documentation needed for these changes?
No, spec compliance.
What is the motivation for this change?
Spec compliance.

Release Highlight

Add ReadConcernMajorityNotAvailableYet to retryable errors
ReadConcernMajorityNotAvailableYet (error code 134) is now a retryable read error.

Double check the following

[x] Ran npm run check:lint script
[x] Self-review completed using the steps outlined here
[x] PR title follows the correct format: type(NODE-xxxx)[!]: description

Example: feat(NODE-1234)!: rewriting everything in coffeescript

[x] Changes are covered by tests
[x] New TODOs have a related JIRA ticket

Passing on all relevant CI configurations: https://spruce.mongodb.com/version/6670a443863b1a000783bb6d/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC


CurveFi hack fixes

Fixed formatting for headers and added target entities
Fixed confirmed stolen funds and added platform-specific losses
Fixed Failure Cause
Removed unnecessary information from Losses and Attackers section
Added extra information about Attackers

/qualitycheck

False claims double checked and confirmed.


Fixes issue where instantiation with an integer

did not correctly set value on variable.
This fix used property decorator to get and set
the value and performs the necessary checks and
conversions in the setter

tested on QLM in QT3.


CentOS 7 package glibc 2.26

Hi, I'm having trouble using the package for CentOS 7. Installation fails with following:
[john@localhost ~]$ sudo yum install sysmonforlinux
Loaded plugins: fastestmirror
Loading mirror speeds from cached hostfile
 * base: merlin.fit.vutbr.cz
 * extras: merlin.fit.vutbr.cz
 * updates: merlin.fit.vutbr.cz
Resolving Dependencies
--> Running transaction check
---> Package sysmonforlinux.x86_64 0:1.0.0-1 will be installed
--> Processing Dependency: libstdc++.so.6(CXXABI_1.3.9)(64bit) for package: sysmonforlinux-1.0.0-1.x86_64
--> Processing Dependency: libsysinternalsEBPF.so()(64bit) for package: sysmonforlinux-1.0.0-1.x86_64
--> Running transaction check
---> Package sysinternalsebpf.x86_64 0:1.0.0-1 will be installed
--> Processing Dependency: libc.so.6(GLIBC_2.26)(64bit) for package: sysinternalsebpf-1.0.0-1.x86_64
--> Processing Dependency: libjson-glib-1.0.so.0()(64bit) for package: sysinternalsebpf-1.0.0-1.x86_64
---> Package sysmonforlinux.x86_64 0:1.0.0-1 will be installed
--> Processing Dependency: libstdc++.so.6(CXXABI_1.3.9)(64bit) for package: sysmonforlinux-1.0.0-1.x86_64
--> Running transaction check
---> Package json-glib.x86_64 0:1.4.2-2.el7 will be installed
---> Package sysinternalsebpf.x86_64 0:1.0.0-1 will be installed
--> Processing Dependency: libc.so.6(GLIBC_2.26)(64bit) for package: sysinternalsebpf-1.0.0-1.x86_64
---> Package sysmonforlinux.x86_64 0:1.0.0-1 will be installed
--> Processing Dependency: libstdc++.so.6(CXXABI_1.3.9)(64bit) for package: sysmonforlinux-1.0.0-1.x86_64
--> Finished Dependency Resolution
Error: Package: sysinternalsebpf-1.0.0-1.x86_64 (packages-microsoft-com-prod)
           Requires: libc.so.6(GLIBC_2.26)(64bit)
Error: Package: sysmonforlinux-1.0.0-1.x86_64 (packages-microsoft-com-prod)
           Requires: libstdc++.so.6(CXXABI_1.3.9)(64bit)
 You could try using --skip-broken to work around the problem
 You could try running: rpm -Va --nofiles --nodigest

This was a clean and updated CentOS 7 VM (minimal). I'm not 100% sure, but I don't think glibc 2.26 is in the official CentOS 7 repos.

Probably the same issue as #28 just didn't notice it.

We aren't supporting Centos 7 right now unfortunately. I'm going to close this issue as we've got no way to resolve it.

Sure no problem, I'll just suggest removing CentOS 7 then from INSTALL.md

Totally agree. I will tidy it up shortly.

Totally agree. I will tidy it up shortly.
Hello,
Please let me know that does SysmonForLinux support on Solaris 11 ?
Thanks you.

We aren't supporting Centos 7 right now unfortunately. I'm going to close this issue as we've got no way to resolve it.

I got the same error. Please can u explain why do we got this error ? Thanks you very much!!!

I got the same error for RHEL 7.9 please let us know what should i do ? to resolve this issue .
Thank you .


react-native v0.49 support?

Can you please support the latest React-Native version?  Thanks!
Also, I get the following error when running, not sure if it is related:
\MainApplication.java:6: error: package com.dhansiddh.rnfirebasedynamiclinks does not exist import com.dhansiddh.rnfirebasedynamiclinks.RNFirebaseDynamicLinksPackage;

I added the following in app\build.gradle (not mentioned in the instructions though):
compile project(':react-native-firebase-dynamic-links')
This made the above error disappear.
However, I now still have the next error (it was there also before):
\MainActivity.java:19: error: cannot find symbol
        RNFirebaseDynamicLinksModule.attachDynamicLinkListener(this);
        ^
  symbol:   variable RNFirebaseDynamicLinksModule
  location: class MainActivity

Any ideas what I should do next? Thanks!

Related to that, I see you are using a specific version for invites:
compile "com.google.firebase:firebase-invites:11.0.0"
This causes problems (cannot compile due to "duplicate entry" error), as other dependencies use version 11.8.0 (or above in the future).
Can this become a "+"?
compile "com.google.firebase:firebase-invites:11.0.0"


making the request failed (SSL peer certificate or SSH remote key was not OK)

Just copy your code  from example/web.php and paste own key and secret. what am i doing wrong?

@OlehKuzmyak , it looks like a local issue with self-signed certificate.
For further investigation, could you please provide following details:
how do you execute the example?
do you use some virtual host or localhost?
what certificate do you see on your end?

-using laravel, code executes at controller. I didn't change code only copy to controller.
-localhost.

don't understand about cerificate. Where i can see this?


In some cases it would be helpful to parse comments from BNGL files

bionetgen.bngmodel nicely parses models. In some cases, it may be helpful to parse the comments out of files.
For example, the comment of Vec is currently parsed to None
begin parameters
 Vec   1000*Vcell # Volume of extracellular space around each cell (1/cell density)
end parameters

model = bionetgen.bngmodel(...)
model.parameters['Vec'].comment

The comments are not parsed at all right now due to how parsing works currently (conversion to an XML file and reading off of that). XML conversion rips out all comments so it's not possible to parse the comments with this method. We are working on a new parsing method right now and once that's done, all comments will be parsed and will be added to .comment attribute. I'll update the issue as I make progress on this.


Update calculator.py

Per conversation in the issues area, I've added the sine method to the class of Calculator and fixed some spacing.
@GregHilston - how do I get both the test and the calculator file in the same pull request?

@jkoors Since one of the goals of this repository is to learn some Github basics, lets see if we can do these two pull requests in one one, as they're related.
I'd have a look here: https://help.github.com/articles/creating-a-pull-request-from-a-fork/
How are you interacting with Github? Are you using the terminal or Github for Windows?
Basically since you've demonstrated you know how to create pull requests, you'll just do both the

adding new code to the calculator.py
adding new tests to test.py

in one go.
Additionally, you may want to name the pull requests a little better, instead of
Updated X, something like Added sine functionality and unit tests.
If anything didn't make sense or you get stuck, please reach out to me here

Yeah, was doing this from the web UI and I appreciate this learning section.
FYI - I've coded HTML, CSS, ASP and PHP websites.  I've also been learning Python for the past 8 months... still totally green, but learning and being in the tech biz as a project manager and Agilist for over 10 years.  So this is my first dive into the world of Git, which I've always been eager to dive in and learn.
So yesterday, I went from the web UI, to CLI commands and now I think I have it.  So let me give that a try and bare with me while I learn!
Thanks for your help and support.  This is great and fun for me.

@jkoors Yea no worries at all man! Take all the time you need. Again if you run into any issues with the python, tests, CiCd or Github don't hesitate to reach out to me! I'm glad to gad


Add confirmed interview slot for each candidate

Hi @charmainehly ! I was thinking if we could push this to v1.3 instead because it is bit pointless to create this attribute without the scheduling function yet! 😅 What do you think?

Shifted to 1.3!!

Refer to #195


Fixing bower.json to build css

i forked the project today and it's really nice ! after some work, i found that the css file wasn't being created in the build. So i adjusted bower.json

I don't have any intention on shipping a "good" css-file along with angular-tablesort. I think every developer has to decide for themselves how they want to style their table. See for example http://ux.stackexchange.com/questions/37564/use-up-or-down-arrow-to-represent-sort-ascending-at-table-header for a reason...


Info suggestions

*Old bird getting activated by apparatus being taken away.
*Interiors (Factory,Mansion,Mineshaft)

Good suggestion

Good suggestion

Will be added in the next update


Return error and status not content is empty

Fix issue where unsuccessful responses with no content would return a "content is empty" error from various serialization libraries instead of information about the status code.
Add a regression test for this as well.

To provide more context over Andrew's comment:

changelog entry new patch version, date to today, change listed under the changes section.
the product version also needs to be aligned.
Thanks a lot for the contribution!


Add install instruction via Homebrew

We don't actually have any installation instructions other than for make. I know the script is available in a few other package managers so we should probably think about what precedent we want to set here. Maybe we should switch to using the wiki a bit more for this stuff?

I don't know how this project is and will be large..
If you assume this project will get bigger, moving some instructions from README to wiki is good idea.

@wkentaro I'll merge for now and possibly move it (with other stuff) to the wiki later :)


FIX update complex type parquet reader

Proposed changes
Issue Number: close #xxx

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

run buildall

LGTM

run buildall

run clickbench

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.6 seconds
stream load tsv:          514 seconds loaded<PHONE_NUMBER>9 Bytes, about 138 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.3 seconds inserted 10000000 Rows, about 341K ops/s
storage size:<PHONE_NUMBER>8 Bytes


Clarify expected results for any and all when operating on empty arrays

This PR

adds explicit guidance for any and all when operating on empty arrays. For the former, the test result should be False and, for the latter, the test result should be True. This matches both NumPy et al behavior (np.any and np.all), TF behavior (tf.reduce_any and tf.reduce_all), and Python behavior (any and all).
adds a note for both any and all that +-inf and NaN all evaluate to True.

Thanks, @honno!


Datadog Agent 7.52.1 use OpenSSL 3.0.8 with vulnerabilities

Hello,
We use Datadog Agent 7.52.1 for Windows and the MDE return usage of OpenSSL 3.0.8 in the following libraries as Vulnerable component :
c:\program   files\datadog\datadog   agent\embedded3\lib\site-packages\confluent_kafka.libs\libcrypto-3-x64-635e87f2c9173c8128924a94337627b3.dll
c:\program files\datadog\datadog   agent\embedded3\lib\site-packages\confluent_kafka.libs\libssl-3-x64-a0018292260ae8557aa3cd7db7d50307.dll
And Datadog Agent use also the version OpenSSL 3.0.13:
c:\program   files\datadog\datadog agent\embedded3\dlls\libcrypto-3.dll
c:\program files\datadog\datadog   agent\embedded3\dlls\libssl-3.dll
Please help to resolve this issue.
Thanks
Best
Didier

👋 @dpey2mtl thanks for bringing this to our attention.
I'll ask a quick question to clarify: is your issue with just the first OpenSSL version (3.0.8 coming from confluent kafka) or also with v3.0.13 that the agent uses?

Hello,
Thanks to answer.
The issue is simple update OpenSSL version (3.0.8 coming from confluent kafka) to resolve all vulnerabilities and provide only ONE up to date OpenSSL version.
Thanks

@dpey2mtl thanks for the quick response.
Unfortunately the solution is not as simple as the issue. We are not the maintainers of the confluent kafka package, so we would have to repackage it to update the vendored openssl dependency. For technical reasons this is too tricky on windows.
Please feel free to open an issue with the confluent kafka maintainers. I will close this issue as there's nothing we can do, apologies :(


Alazar: refactor DLL API related things

Main point of the PR
This PR extracts all C-level calls to the Alazar SDK into its own class to make it easier for that class to manage the use of background threads (and other stuff, if needed, like asyncio), and remove all these low-level details out of the driver class to make the latter manageable.
This is done by:

Make the driver use a dedicated AlazarAPIAPI class that is based on WrappedDll class and DllWrapperMeta metaclass.

DllWrapperMeta ensures that there is one instance of the class per given DLL path (kind of singleton).
WrappedDll uses DllWrapperMeta class:

creates lock and thread pool executor for the DLL calls
assigns ctypes to DLL function arguments and returns (specified in signatures attribute)
WrappedDll allows AlazarATSAPI methods (that represent Alazar ATS API functions) to accept qcodes Parameters, and makes them aware of TraceParameters of the alazar driver.

AlazarATSAPI class defines ctypes signatures for relevant DLL functions, as well as bound methods which represent them; the class also implements convenience methods which make work with the API functions more convenient from the driver class point of view.

Note that the new DLL wrapper class here is somewhat coupled to the Alazar usage, generalizing it to "any DLL" does not bring much value now.
Thanks to @cgranade for the initial bulk of the work in https://github.com/QCoDeS/Qcodes/pull/1379 PR. However, asyncio-related parts were not taken into this PR.
Other changes in this PR:

move many convenience methods from alazar ats driver class to alazar api class
add minor tests for alazar api class
add minor tests for alazar dma buffer class
create a module with alazar api constants, populate it with ones that are used in the driver
attempt to cleanup docstrings in the driver
refactor get_idn of the alazar driver
refactor everything related to querying capabilities from alazar driver class into CapabilityHelper class that the driver uses
Fix get_num_channels alazar driver method for 4 and 8 channel configurations

To-Do:

[ ] interacting rebase/squash all the commits, perhaps to a new branch, and new PR
[ ] run test acquisitions to see if the driver was not broken :)

Epilogue:
From Alazar API header files:
#define MEDIMUM_EXTERNAL_CLOCK	0x00000003UL //TYPO
#define MEDIUM_EXTERNAL_CLOCK	0x00000003UL

@QCoDeS/core I'd like you to have a look at the code to ensure this PR is going the right direction. The work btw is almost done, see the to-do list. If you like the general structure - perfect.

@QCoDeS/core Could you have another look at the PR? I finally approached the "actual testing" stage. And sorry that this PR takes so much time. Please have a look at the updated description as well.

@jenshnielsen If you have time, i'd appreciate if you test this PR on your setup. I am also going to test it with my 9360 card. In particular, it would be great if you have some tests at hand which can show that the overall performance did not degrade.

After CI is done, i'll squash-merge it (rebasing ~2000 commits was not successful at all).