Im 87

is there any special setup to run this? As soon as I start this API branch I get:
events.js:85
throw er; // Unhandled 'error' event
Error: listen EADDRINUSE
at exports._errnoException (util.js:746:11)
at Server._listen2 (net.js:1146:14)
at listen (net.js:1172:10)
at Server.listen (net.js:1257:5)
at EventEmitter.app.listen (...\node_modules\express\lib\application.js:595:24)
at Object. (...\app.js:40:5)
at Module._compile (module.js:460:26)
at Object.Module._extensions..js (module.js:478:10)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
Process finished with exit code 1

This branch requires local redis set up. The remote redis server can only be accessed by dev or prod machine.

Default for redis rconfig is set to host <IP_ADDRESS> and port 6379.
Instructions for running Redis locally can be found here: http://redis.io/topics/quickstart


Implement unit test for custom_operation

It doesn't really do anything, but the custom_operation should be tested.

This issue was moved to bitshares/bitshares-core#54


Cannot authenticate in laravel

when using FMModel in user.php model in laravel it fails to authenticate. says username and passwords are incorrect

No, I've set the filemaker credentials in .env. but when I'm going to fetch record from user table it gives null.But when trying to fetch record using id or name it gives the appropriate record.
I've tried many ways, i saw that if i remove the @ from emailid into filemaker and then query it without @ in email id it gives the record but if i give @ in file maker in email id and then query with the email id i gives null

Remember that "@" is a special character in a FileMaker find - https://help.claris.com/en/pro-help/content/finding-text.html
If you're searching for an email you'll need to make sure that the email address is quoted or you'll need to escape the '@' character.

Ouuuu.... really very thanks.. I'm new to FileMaker, so I didn't know I had to escape @. thanks


upgrading to typescript 2.0

Now that typescript 2.0 is out we may want to update our dependencies and check that everything works well with TSLint. Also, we may be able to get rid of the typings folder since we will be able to do npm install @types/[insert-package-here] and typescript should be able to pick up the definitions.

Although v2.0 was published, since the main package is still 1.8.10 (when doing npm i typescript, it installs 1.8.10 yet), I'll wait for 2.0 being replacing it as the main one.

Hey @buzinas , are you sure? Because looking at npm now the current version seems to be 2.0.3

At the time I was writing, yeah, I'm sure. But I didn't test it today, so I don't know if it's no longer true. I'm completely busy today, but I'll check this later, when I got sometime.

Sure! I was just commenting because I was doing the upgrade today and I noticed it was official, so I had that fresh on my mind :)

Thanks!


Error on accessing bin_edges of BinnedSpikeTrain

When trying to access the bin_edges property of BinnedSpikeTrain object the following error occurs:
Traceback (most recent call last):
  File "test_elephant.py", line 14, in <module>
    edges = binned.bin_edges
  File "/home/bartosz/.pyenv/versions/anaconda3-2.3.0/lib/python3.4/site-packages/elephant-0.3.0-py3.4.egg/elephant/conversion.py", line 585, in bin_edges
    self.num_bins + 1, endpoint=True),
  File "/home/bartosz/.pyenv/versions/anaconda3-2.3.0/lib/python3.4/site-packages/numpy/core/function_base.py", line 125, in linspace
    return y.astype(dtype, copy=False)
TypeError: astype() got an unexpected keyword argument 'copy'

The error can be reproduced with the following script:
from elephant.conversion import BinnedSpikeTrain
import neo
from quantities import ms
import numpy as np

spiketimes = np.arange(10)*10
tmax = spiketimes.max()

spiketrain = neo.SpikeTrain(spiketimes * ms, t_stop=tmax * ms)
binned = BinnedSpikeTrain(spiketrain, t_start=0 * ms, binsize=50*ms)
edges = binned.bin_edges

It appears in python 3.44, with master branch of Elephant and neo, quantities 0.11.1 and numpy 1.11.0 (I think it does not occur with earlier version of numpy).
The problem is with the following with conversions.py:L584
        return pq.Quantity(np.linspace(self.t_start, self.t_stop,
                                       self.num_bins + 1, endpoint=True),
                           units=self.binsize.units)

It seems that numpy.linspace returns a quantity object when t_start and t_stop are also quantity objects. Contrary to what the docstring says the Quantity.astype method does not take copy parameters (numpy.astype does). The workaround is to use t_start and t_stop without their units:
np.linspace(self.t_start.magnitude, self.t_stop.magnitude,
                                       self.num_bins + 1, endpoint=True)

This bug was first spotted by @medelero

Thank you @btel I am creating a new pull request fixing this issue.


parallel: 20230922 -> 20231022

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/parallel/versions.
meta.description for parallel is: Shell tool for executing jobs in parallel
meta.homepage for parallel is: https://www.gnu.org/software/parallel/
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 20231022 with grep in /nix/store/q31rcymfq4a4bjbwd6zv6nnp8dvry24x-parallel-20231022
found 20231022 in filename of file in /nix/store/q31rcymfq4a4bjbwd6zv6nnp8dvry24x-parallel-20231022

Rebuild report (if merged into master) (click to expand)

16 total rebuild path(s)

16 package rebuild(s)

First fifty rebuilds by attrpath
adapta-gtk-theme
bash-preexec
bats
image_optim
jpeg-archive
kicad
kicad-unstable
last
locate-dominating-file
packcc
parallel
parallel-full
plata-theme
profile-cleaner
sonic-pi
tebreak

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/q31rcymfq4a4bjbwd6zv6nnp8dvry24x-parallel-20231022 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A parallel https://github.com/r-ryantm/nixpkgs/archive/c75fed2f17dfcab315beb9be110f3e3eda053dfe.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c75fed2f17dfcab315beb9be110f3e3eda053dfe#parallel

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/q31rcymfq4a4bjbwd6zv6nnp8dvry24x-parallel-20231022
ls -la /nix/store/q31rcymfq4a4bjbwd6zv6nnp8dvry24x-parallel-20231022/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

16 packages built:

adapta-gtk-theme
bash-preexec
bats
image_optim
jpeg-archive
kicad
kicad-unstable
last
locate-dominating-file
packcc
parallel
parallel-full
plata-theme
profile-cleaner
sonic-pi
tebreak

Maintainer pings
cc @pSub @vrthra @tomberek for testing.


[Enhancement] conditional expressions support array/map/struct/json types

Problem Summary:
Fixes # (issue)
conditional expressions support array/map/struct/json types, including if, ifnull, nullif, coalesc, case... when...
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[x] This pr needs user documentation (for new or modified features or behaviors)

[x] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:disappointed: fail : 6 / 10 (60.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/PolymorphicFunctionAnalyzer.java
5
9
55.56%
[194, 195, 196, 197]

:large_blue_circle:
com/starrocks/sql/common/TypeManager.java
1
1
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 6 / 10 (60.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/PolymorphicFunctionAnalyzer.java
5
9
55.56%
[194, 195, 196, 197]

:large_blue_circle:
com/starrocks/sql/common/TypeManager.java
1
1
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 6 / 10 (60.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/PolymorphicFunctionAnalyzer.java
5
9
55.56%
[194, 195, 196, 197]

:large_blue_circle:
com/starrocks/sql/common/TypeManager.java
1
1
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 6 / 10 (60.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/PolymorphicFunctionAnalyzer.java
5
9
55.56%
[194, 195, 196, 197]

:large_blue_circle:
com/starrocks/sql/common/TypeManager.java
1
1
100.00%
[]


node-scenarios: several enhancements

Add Duration to ElasticSearch: duration
Add Note time till ready: node_ready_time
Fix 2024-11-06 04:02:44,865 [ERROR] 'namespace'

Hello, thanks for opening this issue(s). I believe the code for these changes will actually need to go into the lowest level of our code base (krkn-lib). would you be able to create an issue there and link the two! Thanks for the ideas/contributions

Open issue under  (krkn-lib)


Using row-stride with matrixmultiply

Fixing bug with MatrixSlice multiplication. Adding more test coverage.

:+1: :1234:  :runner:


Config Store - Mechanism for supporting reload of settings at the runtime

We need to allow clients of the config-store to see the latest change.
We can implement it using one of these two approaches:

"pull" mode: the client will refresh the setting value at regular intervals
"push" mode: listener api where clients subscribe to the setting change modification.

If we want 1, we should be ok with a cache. Client’s are interacting with the config-store using the ConfigStore interface so that we can hide the cache implementation there. The refresh interval can be exposed using the config-store API, registering default to for example 1 second.
If we want 2, we need a thread that polls the config for changes, an event API, etc. It's more complicated and maybe not needed for a V_1

After some more thoughts, I think we may not need to implement any strategy for reloading from the YAML config store.
The ConfigStoreYaml#getConfigForModule(String) loads the underlying stargate-config.yaml file every time its called. Because of that, the caller will get the fresh version of it every time it's accessed. It's up to the caller to decide when this method should be called. If the ConfigStoreYaml client needs to have runtime reload, it should call getConfigForModule every time the settings are accessed.
On the other hand, reading the file every time may be problematic from the performance perspective:
Map<String, Map<String, Object>> result =
          mapper.readValue(configFilePath.toFile(), yamlConfigType);

and we may consider adding a cache atop of it. I've created a simple JMH micro-benchmark comparing loading the file every time this method is called versus caching the file for 30 seconds (exact duration yet to be decided):

@Fork(1)
@Warmup(iterations = 5)
@Measurement(iterations = 10)
@BenchmarkMode(Mode.AverageTime)
@OutputTimeUnit(TimeUnit.MILLISECONDS)
public class ConfigStoreYamlBenchmark {
  private static final int NUMBER_OF_ITERATIONS = 100_000;

  @Benchmark
  public void directLoad(Blackhole blackhole) {
	  // given
	  Path path =
			  Paths.get(
					  Objects.requireNonNull(getClass().getClassLoader().getResource("stargate-config.yaml"))
							  .getPath());
	  ConfigStoreYaml configStoreYaml = new ConfigStoreYaml(path);

	  for(int i = 0; i < NUMBER_OF_ITERATIONS; i++) {

		// when
		ConfigWithOverrides configForModule1 = configStoreYaml.getConfigForModule("extension-1");
		ConfigWithOverrides configForModule2 = configStoreYaml.getConfigForModule("extension-2");

		// then
		blackhole.consume(configForModule1);
		blackhole.consume(configForModule2);
	}
  }

	@Benchmark
	public void loadWithCache(Blackhole blackhole){
		// given
		Path path =
				Paths.get(
						Objects.requireNonNull(getClass().getClassLoader().getResource("stargate-config.yaml"))
								.getPath());
		ConfigStoreYaml configStoreYaml = new ConfigStoreYaml(path);

		for(int i = 0; i < NUMBER_OF_ITERATIONS; i++) {

			// when
			ConfigWithOverrides configForModule1 = configStoreYaml.getConfigForModuleWithCache("extension-1");
			ConfigWithOverrides configForModule2 = configStoreYaml.getConfigForModuleWithCache("extension-2");

			// then
			blackhole.consume(configForModule1);
			blackhole.consume(configForModule2);
		}
	}

}

The results show that there is a huge difference between both approaches:
Benchmark                               Mode  Cnt      Score     Error  Units
ConfigStoreYamlBenchmark.directLoad     avgt   10  10486.425 ± 287.736  ms/op
ConfigStoreYamlBenchmark.loadWithCache  avgt   10     32.270 ±   0.465  ms/op

If the callers expect runtime changes, they will need to call the getConfigForModule every time any setting is accessed.
Caching seems to be very important from the performance perspective.
However, we need to remember and document that It will slow down the propagation of changes from the underlying stargate-config.yaml.
Are we ok with that? wdyt (@dougwettlaufer @dimas-b)

Caching the file SGTM :+1:
It may still be useful to have a notification API for services that do not request configs periodically, but still want to reload settings when they change. I suppose something simple like a CRC32 on the whole file with periodic background checks should be fine. WDYT?

A bit tangential: What if we unregisterd and re-registered the config service for each time the file changes? This way dependent modules receive config registered/unregistered notifications from the OSGi framework and restart as appropriate.

A bit tangential: What if we unregistered and re-registered the config service for each time the file changes? This way, dependent modules receive config registered/unregistered notifications from the OSGi framework and restart as appropriate.

I think it is a bit too much to restart the whole service when the config-store file changes. It may be quite frequently, and also not every setting change means that we need to restart the whole service. If there are settings that can be changed at runtime, then by definition, they should not require restarting the whole service, isn't it?
I was also thinking about using the OSGi lifecycle - it is described in the design doc).
However, I think that it's better to decouple the config-store from the OSGi API. It may be feasible, but I think that currently, we may not need it. If it turns out that most settings require restarting the module, maybe we can consider it again, wdyt?

Caching the file SGTM 👍
It may still be useful to have a notification API for services that do not request configs periodically, but still want to reload settings when they change. I suppose something simple like a CRC32 on the whole file with periodic background checks should be fine. WDYT?

Yes, listener API would be nice but I think that we agreed that we don't need it for V_1.
If we will pick a low refresh interval (i.e. 30 seconds) and clients will access every config setting directly through getConfigForModule method it should be enough for first use cases, wdyt?

I think that it's better to decouple the config-store from the OSGi API. It may be feasible, but I think that currently, we may not need it. If it turns out that most settings require restarting the module, maybe we can consider it again, wdyt?

Dependent services still have the flexibility to reload config from the new service instance without restarting themselves. OSGi does not mandate dependent service restarts or dependency changes, AFAIK.

Yes, listener API would be nice but I think that we agreed that we don't need it for V_1.

This is beyond my knowledge :) @dougwettlaufer WDYT?

Yes, listener API would be nice but I think that we agreed that we don't need it for V_1.

This is beyond my knowledge :) @dougwettlaufer WDYT?

Please refer to the design doc and discussion about the runtime reload.

I'm getting a bit confused :) I'd be ok with not supporting config reload in V_1 completely, but if we support config reload in principle (which is what this issue is about) then why not support notifications at the same time? If we do not support change notifications in the config service, then all its clients will have to implement polling (assuming the clients do not have natural event that prompt for config reload). It seems to be that this is going to be a common use case, so it would be worth implementing in the config service itself.

For v1 we create the barebones version of "pull your config and cache it if you know it's static, read the file every time if you're expecting changes". Then for v2 we can come in and get fancy with the listeners. Right now we don't have anything that must to do runtime config reload.
I'd prefer to get a flexible config store up and running so we can get the kafka producer out the door and then we can come back in and iterate on nice to have features like config change notification and such.

SGTM... however, if we do not expect config reload in v1 in runtime, then perhaps we do not need the cache :thinking:

I agree that cache is not needed from the perspective of runtime reload. I will create a different ticket and implement it separately.
We can leave this task for V_2

I don't think this would work (or necessarily be beneficial) for Stargate V2; but if so would be designed as part of that process. Closing this issue as related to SGv1 architecture which will be in maintenance mode.


Fix gdb segfaults due to bypassed locking

Implements fix for #203 identified by @Mithy83.
I don't see any reason this should fail with the lldb backend, but I haven't checked it.

Apologies for the delay. Thank you for the PR. I've merged this into master

Apologies for the delay. Thank you for the PR. I've merged this into master


Add "status" or "active" to ProjectMembership

I like active due to the simplicity of a boolean value, but maybe there is need for an enum like status that I'm not considering


[Feature] please provide RPM package or release source code

Context
as so far i had not found any RPM package or source code package,it looks [pg_idkit] had not developed.
Proposal
if there is RPM package it is very convenient to install and experience

Hey @vkingnew thanks for this suggestion, sorry for the delay in getting to it!
I think before I can get an RPM build set up I need to start publishing proper artifacts/releases for pg_idkit -- definitely something I should get to soon, but this is probably blocked by https://github.com/VADOSWARE/pg_idkit/issues/15


Add Dockerfile for Mintmaker image

This Dockerfile has a custom Renovate image that includes rpm lockfile support as a custom manager. It does not include additional unrequested managers.

It's basically the same as the other image we have in the other repo. But I did remove some of the manager, because the request for konflux is to have only tekton, rpm lockfile, and "from" line of the container images.
I've kept the cert part because I think it might be needed for future gitlab.cee. I might have kept something unnecessary, if you notice, please tell me. Thank you

I've applied the changes.
It is weird because if I build the container locally it says /bin/sh: line 1: npm: command not found
Which is weird, because it has been installed. Any idea?

I've applied the changes. It is weird because if I build the container locally it says /bin/sh: line 1: npm: command not found Which is weird, because it has been installed. Any idea?

I thought it was in nodejs package but what if you add nodejs-npm to dnf install?

I've applied the changes. It is weird because if I build the container locally it says /bin/sh: line 1: npm: command not found Which is weird, because it has been installed. Any idea?

I thought it was in nodejs package but what if you add nodejs-npm to dnf install?

me too. Weird. I've tried also nodejs-npm and still doesn't work (weirder). I'll play a bit more.

I've addressed the changes. To be honest the npm package still isn't found. @staticf0x @qixiang could you give it a try build the container locally and see if it work for you? I tried several things, but I always get the same error.

@qixiang @staticf0x PTAL It builds successfully now. Thanks!


Code can not open a folder or file in onedrive on windows rs3 preview

Code can not open a folder or file in onedrive folder that enabled the feature of file-on-demand.
after I upgraded my windows to RS3 preview.
Code consider the path which I want to open as not exist, and try to create a new file with that path.

VSCode Version: Code 1.14.2 (cb82febafda0c8c199b9201ad274e25d9a76874e, 2017-07-19T23:34:09.706Z)
OS Version: Windows_NT ia32 10.0.16251
Extensions:

Extension
Author (truncated)
Version

html-snippets
abu
0.1.0

vscode-stylesheet-formatter
dba
0.2.0

githistory
don
0.2.3

xml
Dot
1.9.2

vscode-pandoc
Dou
0.0.8

vscode-great-icons
emm
2.1.14

live-html-previewer
hdg
0.2.0

csharp
ms-
1.12.1

PowerShell
ms-
1.4.1

azurerm-vscode-tools
msa
0.3.6

debugger-for-chrome
msj
3.1.8

vscode-json-transform
oct
0.1.2

startanyshell
rem
0.3.1

vscode-icons
rob
7.12.0

vsc-jsonsnippets
wmo
0.1.4

html-css-class-completion
Zig
1.8.0

Reproduces without extensions: Yes/No

https://github.com/Microsoft/vscode/issues/27285


Improvements to mindependency dev_requirement conflict resolution

Addresses #37450
The conflict we were running into with mindependency is best explained in context of a package.

azure-ai-language-conversations took a relative requirement on azure-identity for its tests through the dev_requirements.txt file
main azure-identity (eg what is required through dev requirement) required azure-core>1.31.0
azure-ai-language-conversations mindependency would install its minimum azure-core version of 1.28.0.
Attempting to install azure-core==1.28.0 to meet mindependency requirements would CRASH because azure-identity==1.18.0 being included REQUIRED azure-core>=1.31.0.
Because it's in the dev_requirement, mindependency didn't handle it

This PR adds:

[x] Dependency conflict resolution for the packages defined in dev_requirements

If a version is found to be incompatible, it walks backwards through time on pypi until it finds one that IS

[x] Conflict resolution for a standard version specifier. right now it only handles relative and built requirements.

We now properly leave these alone. Folks, be aware of your dev requirements not breaking mindependency!

[x] Walk the edge cases a bit further. I'm happy with the fallback of "just leave whatever is there if you don't understand it", that way folks will at least understand the errors if they see them.
[x] Add tests for is_package_incompatible and resolve_incompatible_package to cover the edge cases noted in above bullet

Member

Annoyed I missed the last additionalFilterFn when I updated it everywhere else. Thanks for the review @mccoyp I fixed the last one.


DeepSpeed Integration

🚀 Feature Request
Integration with DeepSpeed. The V0 use case is targeted only on data parallelism strategies like ZeRO.
Motivation
Necessary to train GPT models above 1.3B parameters.

MVP addressed in #109 .


aws_lab_setup needs to pull v2 branch

Currently with the aws_lab_setup, it's cloning the master branch into student's control node.  Need to change this to version: v2

Will put in a PR


Naming Conventions

i ran some tests to see whether this coding standard enforces all things mentioned in the naming conventions section of symfony's coding standard and made several modifications accordingly.
this pull request makes sure that the following additional guidelines are met:

Use camelCase, not underscores, for function and method names
Use namespaces for all classes
Prefix abstract classes with Abstract
Suffix traits with Trait
Suffix exceptions with Exception

note:
this might cause a bit of a stirr as the guidelines mentioned above haven't been enforced in the past thus people using this standard potentially end up with a bunch of phpcs errors and the need to refactor parts of their codebase (at least the main repositories i'm working on will require quite a bit of work to comply) .

@djoos any reason why this one doesn't get merged nor closed?

Hi @wickedOne,
I actually haven't been able to have a proper look at the PR yet... :-/
Kind regards,
David

hi @djoos
no problem, was just wondering & considering setting up a clone meanwhile…
just let me know,
cheers
w

Peehaps it's best to make use of your PR-fork for now until I get the chance to review the PR and get it merged into the master-branch...
Hope this helps!
Kind regards,
David

ok, i'll set up a clone as there are more sniffs on the way and it might grow into a too big a pull request…
once you've got a bit more time on your hands i'll offer them in seperate pull requests to this repository
cheers
w

Hi @wickedOne,
I think the best way forward is to put each change into a relevantly named branch - do create a PR for each of these branches so we can review and get them merged in ASAP.
Hope this helps!
Kind regards,
David

Thanks for your contribution @wickedOne - sorry for the delay in getting this PR merged...
Kind regards,
David


update(de): correct yaml localization entries

Updated the German localization in the YAML file to ensure proper accurate translations.
Note: Corrections and improvements were assisted by ChatGPT-4.

Danke!


redesign of add flow and dialog UI

prints qrc:/qml/StyledComboBox.qml:30: ReferenceError: noedit is not defined when entering add credential mode

Also looks like I get touch credentials with the checkbox unchecked


Two OneDrive accounts login in suggestions

When two onedrives log in, we can't view their folders separately at the same time. You can only access one folder.

@WallerGeque Only one account is shown on the sidebar however you can access another by typing the location on the navigation path box.

Duplicate of #1794.


SINGA-493 Update news on SINGA tutorial

The webpage look like this when I use Sphinx to build:

I forked the incubator-singa-site and put the preview of website at https://github.com/chrishkchris/incubator-singa-site/tree/SINGA-493


Add formatter and linter for python tests

[ ] Add and update tox entries.
[ ] Update CI (add checks)
[ ] Update CONTRIBUTING guide.

Hi @ericnordelo, I know some devs who'd be interested in working on this. Is there a budget for fixing this issue, and if so, how much?
I'd put this issue up on OpenQ.
Thanks!

hello @ef3n9r98! thanks, but we don't currently have budget allocated to incentivize contributions.

Closing this since it will not be relevant anymore after the ongoing Cairo 1.0 migration. If you think this is a mistake, feel free to open a new issue.


Using "search for match" sends back to the first row

Describe the bug
When reconciling, after matching an entry using the "Search for match" button, the display moves to the first rows of the first page of the project instead of staying at the point we were at.
To Reproduce
Steps to reproduce the behavior:
0. Open a project that is several pages long and reconcile a column.

Go to the second page of results
Click on the "Search for match button" on an entry that has not been automatically reconciled, which opens a modal window
Click on the match button
The rows displayed are now the first ones of the first page.

Current Results
A page that has likely already completely reconciled is displayed and we must navigate again to the page we are working on (which can become really frustrating if we have a project with a lot of pages and many entries to reconcile manually.)
Expected behavior
When the modal is closed, the same rows that were displayed before we used it are still displayed.
Screenshots

Desktop (please complete the following information):

OS: Ubuntu 18.04 & 18.10
Browser Version: Firefox 66
JRE or JDK Version: openjdk version "1.8.0_191"

OpenRefine (please complete the following information):

Version OpenRefine 3.1 Beta

Datasets
Any dataset

Hi @Ash-Crow! This is a duplicate of #33, where the following solution is proposed:

For now, what you can do is as follows:

select None in the Judgment facet to see only those rows that haven't been matched. Or if the facet isn't there, it's available in the column's menu Reconcile -> Facets ->By Judgment.
just keep on the first page and process the first row. Once you manually match it, it goes away (since its judgment is no longer None).

@wetneb Eventually, I think we should make this process easier for users.  Staying in some "Reconcile Mode" with its fancy panels is in our long-range plans, and once you are out of Reconcile Mode, then things behave normally with Facets.  I wonder if it is easy enough to introduce a Reconcile Mode now?

Yes, that discussion is still current. It's indeed a frequently requested feature.


disable csrf in dpaste

i am using dpaste docker image but i once i try to generate any link for my text it is always showing csrf protection issue . can anyone help how to disable this feature

In your settings add,
CSRF_COOKIE_SECURE = False

thanks

I added CSRF_COOKIE_SECURE = False to dpaste/apps.py like so but am still getting:
Forbidden (403)
CSRF verification failed. Request aborted.
More information is available with DEBUG=True.

You can check my instance here.

The changes should be done to local.py https://github.com/DarrenOfficial/dpaste/blob/master/dpaste/settings/local.py.example <-

I added it to my local.py, rebuilt and ran the image and am still getting the 403 error. The local.py also has DEBUG = True, and I can see this in my instance when the 403 error occurs because it shows the debug information. So, it took my custom local.py, but the CSRF disabling still doesn't work?

I've responded to your email 🙂

Edit: So, it works without https. I suppose this is intended and the way it works?

Definitely not
-- From email
Could you show me your docker configuration / docker compose file;
Additionally are you using a proxy server (i.e. NGINX, Apache, Trafeik, Caddy, ect...), if so could you send the configuration of that as well; the error might be because of a misconfigured proxy

Edit: So, it works without https. I suppose this is intended and the way it works?

What I meant was it would work without https, not that it should be run without. :sweat_smile:

Could you show me your docker configuration / docker compose file;

I didn't touch the original Docker files.

Additionally are you using a proxy server

I'm using NGINX. In /etc/nginx/sites-available/mnpd.conf:
server {
    server_name mnpd.khkm.dev www.mnpd.khkm.dev;
    server_tokens off;
    location /.well-known/acme-challenge/ {
        root /var/www/certbot;
    }
    # }
    location / {
	proxy_pass http://<IP_ADDRESS>:8001;
	proxy_buffering off;
	proxy_set_header Host $host;
	proxy_set_header X-Real-IP \$remote_addr;
    }

    listen [::]:443 ssl ipv6only=on; # managed by Certbot
    listen 443 ssl; # managed by Certbot
    ssl_certificate /etc/letsencrypt/live/mnpd.khkm.dev/fullchain.pem; # managed by Certbot
    ssl_certificate_key /etc/letsencrypt/live/mnpd.khkm.dev/privkey.pem; # managed by Certbot
    include /etc/letsencrypt/options-ssl-nginx.conf; # managed by Certbot
    ssl_dhparam /etc/letsencrypt/ssl-dhparams.pem; # managed by Certbot

}
server {
    if ($host = mnpd.khkm.dev) {
        return 301 https://$host$request_uri;
    } # managed by Certbot

    listen 80;
    listen [::]:80;
    server_name mnpd.khkm.dev www.mnpd.khkm.dev;
    return 404; # managed by Certbot
}

I see, try this reverse proxy config; this is what dpaste use in prod.
location ^~ /
{
    proxy_pass http://<IP_ADDRESS>:8001;
    proxy_set_header Host $host;
    proxy_set_header X-Real-IP $remote_addr;
    proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
    proxy_set_header REMOTE-HOST $remote_addr;

    #Persistent connection related configuration - Optional dpaste.org has it enabled.
    #add_header Access-Control-Allow-Origin *;

}


On several FIT file exports times are totally off

On several FIT file exports the times are off by days and hours. I could not find a clear indication, which activities / recording devices are affected, what I can say: I could only see the effect at activities recorded by a clock, eiteher Garmin or Polar. But a friend of mine using a Garmin 245 music has the effect, while my own activities recorded on a Garmin 245 music too are fine.

Here is an example, which should be generally available:
https://www.strava.com/activities/10252700067
By re-importing the exported FIT file you can easily see the difference

After several additional tests I see the following logic:
-the problem only occurs on runs, not on rides (sounds strange, but I could recreate this at sveral runs+rides of a friend, who records both on a Garmin forerunner 245 music)

if the problem occurs, the date shown is always the current date, the time seems to be always in the future (from the time of export)
the problem does not occur on my own runs, but seems to be there at all(?) runs of connected people
the problem is also included in TCX file exports
the problem was there with both Polar and different Garmin watches

Thank you very much for diving so deep into this problem.
For me, it would be sufficient, if the time would be in GMT, the most annoying thing currently is, that the time is offset by something variable. Even the duration seems always a bit off (a few minutes / hour)

See #214


Created gwyneth-penasiguenza.yml

Adding myself to the advocate page :)

Docs Build status updates of commit d50614a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

advocates/gwyneth-penasiguenza.yml
:x:Error

Details

advocates/gwyneth-penasiguenza.yml

Line 15, Column 1: [Error-yaml-syntax-error] While parsing a block mapping, did not find expected key.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

rats, let me fix that.


LinkFree name not clear on home page

What would you like to share?
I don't think think that on the homepage it is very clear the that the site is called Linkfree.
In the screenshot below it says: It is an open-source alternative to Linktree implemented in JavaScript. The use of it is makes it seem a bit vague and not clear that what is trying to be described it actually Linkfree.
And on the homepage, it doesn't actually state the actual name Linkfree anywhere that can clearly be seen.

Additional information
No response

Great point! It should say LinkFree is an Open Source ...

However, I can say that LinkFree looks amazing now. I remember when LinkFree was first established, and when I see it now, there is such a positive difference and change.

Thank you @CodingSpecies ! We have more features coming soon too
Don't forget to enable your profile so you can see the stats

@eddiejaoude how do I enable my profile? I have already made a profile, but I am not sure if I need to do anything else to enable the stats. Thanks.

yes you need to add another json property to true, because it is off by default, more info in the readme, but any questions lets chat in EddieHub Discord so we don't pollute this issue


Use error strings in smart contracts

Description
Since solidity 0.4.22 error strings are supported for revert and require:
pragma solidity ^0.4.22;

contract VendingMachine {
    function buy(uint amount) payable {
        if (amount > msg.value / 2 ether)
            revert("Not enough Ether provided.");
        // Alternative way to do it:
        require(
            amount <= msg.value / 2 ether,
            "Not enough Ether provided."
        );
        // Perform the purchase.
    }
}

Adding this to the smart contract will make it easier to debug where things go wrong in a function. It also makes the terrible work flow of removing requires to figure out which require is failing obsolete.
Tasklist

[ ] Add error strings to requires in the smart contracts

@loredanacirstea Do you think you can add them in this contract change (red eyes testnet 4)? I would say it's good to take advantage of the redeployment to include them.

Yes, added the milestone now.

Just some notes:

revert error messages are not currently supported by web3.py - https://github.com/ethereum/web3.py/issues/941. When they will be supported, in order to get them, one would need to eth_call the contract function and get the encoded error.
there are EIPs in discussion about standardizing status codes for errors (similar to http), that can also be handled in the EVM - https://eips.ethereum.org/EIPS/eip-1066; This is worth following in my opinion, as it could improve the automatic interaction between smart contracts.

Aside from these mentions, I am ok with just using some error strings right now and re-evaluating them after standards are clarified.

P2 because I heard error strings would have saved our time today.

If we don't hit the bytecode length after enabling cooperativeSettle(), this will be definitely implemented.  So the next task is to try enabling the cooperativeSettle() locally and see if the compilation still works.

Even if we recover cooperativeSettle(), we can still add at least a two-character message to each require.

If we recover cooperativeSettle() and add a two-character comment on each of require, the runtime codesize will be 28835 bytes, which exceeds the deployable limit of 24576 bytes.

Whether we optimize or not, any message that fits 32 bytes seems to compile into the samely sized runtime code.  So no need to go for cryptic "a" "b" "c" style messages.

I’ll just add somewhat recognizable comments under 32 bytes.

This depends on #401.

Just tried the old )PR again, we're still running into code size issues

See https://github.com/raiden-network/raiden-contracts/issues/1435 for more space saving.


The element ID for street address line 2 field changed.

generally for non-required elements, we prefer not to have them in yamls at all, to prevent errors like this. So want to just delete the block instead?

Removed that field in the last commit.


Use all Cores full?

I tadded ENV OMP_NUM_THREADS=44 to the container so faster whisper could use more cores, it works i think but not utilizing all cores full. when using 4 threats all threats are at 100%, using all 48 threats it looks like this:

i would say i have more than enaugh processing power left to use

Does the same thing occur if you try to use faster-whisper directly? (https://github.com/SYSTRAN/faster-whisper)
I'm not sure this is a problem with the ASR project.

ok its a problem from faster-whisper


Performance optimization of flight

Summary
Performance of queries via flight protocol is not optimal.
Test case
Running a query that returns 1M records. Both Dremio and the client is local.
Dataset: Samples."samples.dremio.com"."NYC-taxi-trips" (LIMIT 1000000)
Flight: 3.375s - Execution time in Dremio: <1s
ODBC: 3.703125
It looks like the time is spent entirely on the client side processing even though the data has been sent from the server in less than 1 second. Could it be this? https://github.com/narendrans/sqlalchemy_dremio/blob/master/sqlalchemy_dremio/query.py#L17
Statistics:

Call Graph:

Thanks to @rymurr. It looks like converting to pandas DF is the culprit. Processing numbers is much faster compared to strings.
SELECT passenger_count,trip_distance_mi,pickup_longitude,pickup_latitude,dropoff_longitude,fare_amount,surcharge FROM "NYC Taxi".trips limit 3000000
Flight: 2.78125
ODBC: 11.578125
TO DO: Refactor so that pandas is not used at all.

#9  WIP


Where are the instructions for Cloud Only AAD environment

In the Azure portal, Dashboard > Azure AD Domain Services > mydomain we have the following instructions with one option to follow for cloud only account, which leads to this document, but this document nor linked documents contain information on how to configure has synchronisation for cloud only customers..
This is the text on our portal, and we are trying to make sure SLDAP passwords stay in sync after the user changes the PW.
Enable Azure AD Domain Services password hash synchronization
Users cannot bind using secure LDAP or sign in to the managed domain, until you enable password hash synchronization to Azure AD Domain Services. Follow the instructions below, depending on the type of users in your Azure AD directory. Complete both sets of instructions if you have a mix of cloud-only and synced user accounts in your Azure AD directory.
Instructions for cloud-only user accounts

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

ID: 325a9114-36b2-a6b3-d0a4-19c522b7abf0
Version Independent ID: cbd84d9f-fbfd-d97d-9826-26947f64d18a
Content: Tutorial - Configure LDAPS for Azure Active Directory Domain Services
Content Source: articles/active-directory-domain-services/tutorial-configure-ldaps.md
Service: active-directory
Sub-service: domain-services
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@blouwagie Thank you for your feedback . We will investigate and update the thread further.

If you just have cloud-only accounts, you don't need to configure password hash synchronization. That process is only needed in hybrid environments where Azure AD is connected to an on-premises environment.
For cloud-only environments, each user would need to reset their password to enable their account to work in Azure AD DS - https://docs.microsoft.com/en-us/azure/active-directory-domain-services/tutorial-create-instance#enable-user-accounts-for-azure-ad-ds
Neither of these are necessarily LDAPS specific - these are steps noted in the initial deployment of an Azure AD DS managed domain in order for it to function correctly.

Hi Ian
The Azure portal leads us to believe we have to set the synchronization up
as there is one specific link to show how cloud only users have to do this.
So based on your comment the documentation itself is only not as clear and
the portal that you get after seeing up Azure AD services does say cloud
icky customers need to do this.
Thank you for replying!
Bart
bart
<EMAIL_ADDRESS>On Fri, Nov 22, 2019, 17:58 Iain Foulds<EMAIL_ADDRESS>wrote:

If you just have cloud-only accounts, you don't need to configure password
hash synchronization. That process is only needed in hybrid environments
where Azure AD is connected to an on-premises environment.
For cloud-only environments, each user would need to reset their password
to enable their account to work in Azure AD DS -
https://docs.microsoft.com/en-us/azure/active-directory-domain-services/tutorial-create-instance#enable-user-accounts-for-azure-ad-ds
Neither of these are necessarily LDAPS specific - these are steps noted in
the initial deployment of an Azure AD DS managed domain in order for it to
function correctly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/43153?email_source=notifications&email_token=AESZT5OC5GU5FETDDXRDYYLQVBPY5A5CNFSM4JPI5D72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE7DL5Y#issuecomment-557725175,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AESZT5ORR3KF2AC27D77XP3QVBPY5ANCNFSM4JPI5D7Q
.

I can relay that info to the product team for portal updates. There's really no reason to point you to the LDAPS doc, as there's no information included there, only in the initial docs on configuring a managed domain.
More information on password hashes and what's required in different scenarios you run can be found here - https://docs.microsoft.com/azure/active-directory-domain-services/administration-concepts#password-hashes

Thank you so much for helping clarify this, we've spent quite some hours
trying to connect those dots
bart
<EMAIL_ADDRESS>On Fri, Nov 22, 2019, 18:50 Iain Foulds<EMAIL_ADDRESS>wrote:

I can relay that info to the product team for portal updates. There's
really no reason to point you to the LDAPS doc, as there's no information
included there, only in the initial docs on configuring a managed domain.
More information on password hashes and what's required in different
scenarios you run can be found here -
https://docs.microsoft.com/azure/active-directory-domain-services/administration-concepts#password-hashes
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/43153?email_source=notifications&email_token=AESZT5JY53JTUV72LBIV6L3QVBV23A5CNFSM4JPI5D72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE7GCDI#issuecomment-557736205,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AESZT5ILU4KG2FNWFMWXAFDQVBV23ANCNFSM4JPI5D7Q
.

@blouwagie Hope your queries are answered by Ian . In case you have any further queries on the same , please do let us know .

@blouwagie We are closing this issue now . In case you have any further queries on this , please tag me to your reply and we will surely help you with the same.
Thank you.


Request for the Mainland China version

Traditional Chinese (HongKong/Taiwan) news already has RSS, but Simplified Chinese (Mainland China) news is not yet available.
The official website of Genshin Impact in mainland China is located at the following URL, and the structure looks the same as other countries' websites.
https://ys.mihoyo.com/main/news
If you have time, it would be helpful if you could also get RSS for mainland Chinese news on your website!

Hello @scaramouchename, thanks for your proposition!
It will require a bit of work since the news aren't in the same domain as the rest of the world but I noted in my To-Do list and will work on it when I have a bit of time! I will take you noticed when it will be available!
Have a nice day!


Feat/snapshot

@greenealexander There are some connectors on this page from CP that need to be adjusted to work with our new wallet.

@sirgawain0x is this ready to be merged or is there pending work?

Just needs a couple web3 wallet connectors and should work just like in Creative platform but using WAGMI hooks.


Refactor kernel installation and add conda recipes

This PR includes a small change to move the kernel installation part from setup.py to another script that has to be called after installation. This is a little annoying but its how most external kernels work. see: octave kernel, scilab_kernel and Jupyter extensions.
So to install correctly the SAS kernel now you would need to do:
pip install sas_kernel
python -m sas_kernel.install

This split also allows sas_kernel to be installed correctly in conda environments because the kernel.json inside an environment will need the path to that environment python binary.
There is a way to keep the current functionality of installing the kernel with pip and also add the fix for conda by refactoring setup.py and I can work on that if you guys want but for now I kept it the way most Jupyter extensions work.
The PR also includes 2 conda recipes for saspy and sas_kernel. My hope for those recipes is to make them available on conda-forge so more people in the community can get them but I wanted to see if someone wants to test them before I move them there.

When I try to run the recipe I get the following error:
What is the correct what to invoke this?
$ conda install sas_kernel --use-local
Fetching package metadata ...........

PackageNotFoundError: Packages missing in current channels:

  - sas_kernel

We have searched for the packages in the following channels:

  - https://conda.anaconda.org/anaconda-fusion/osx-64
  - https://conda.anaconda.org/anaconda-fusion/noarch
  - https://repo.continuum.io/pkgs/free/osx-64
  - https://repo.continuum.io/pkgs/free/noarch
  - https://repo.continuum.io/pkgs/r/osx-64
  - https://repo.continuum.io/pkgs/r/noarch
  - https://repo.continuum.io/pkgs/pro/osx-64
  - https://repo.continuum.io/pkgs/pro/noarch

Have you built the packages?
First you need to install conda-build go to the conda.recipe directory on this PR and build both packages. This should work:
conda install conda-build

cd sas_kernel/conda.recipe
conda build saspy sas_kernel

conda install sas_kernel --use-local

After that if you conda list you should see saspy and sas_kernel:
$ conda list | grep sas

sas_kernel                2.1.0                    py36_0    local
saspy                     1.2.2                    py36_0    local

When I tried to build this on my MBP I get the following build error:
conda_build.exceptions.DependencyNeedsBuildingError: Unsatisfiable dependencies for platform osx-64: {'metakernel’}
Not sure how to proceed. There doesn’t appear to be a conda package for metakernel.
Let me know if you need the full traceback.
Jared

On Aug 17, 2017, at 10:30 AM, Daniel Rodriguez<EMAIL_ADDRESS>wrote:
Have you built the packages?
First you need to install conda-build go to the conda.recipe directory on this PR and build both packages. This should work:
conda install conda-build
cd sas_kernel/conda.recipe
conda build saspy sas_kernel
conda install sas_kernel --use-local
After that if you conda list you should see saspy and sas_kernel:
$ conda list | grep sas
sas_kernel                2.1.0                    py36_0    local
saspy                     1.2.2                    py36_0    local
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/sassoftware/sas_kernel/pull/43#issuecomment-323090848, or mute the thread https://github.com/notifications/unsubscribe-auth/ADA5yyiqCBF_EPFrC9VKsYEkGaBudKWTks5sZE5_gaJpZM4O4Nxa.

Can you add conda-forge as a channel? and then try again.
conda config --add channels conda-forge

@danielfrg after considering the change we definitely want to have a conda package but don't want to give up the single command install. Can you restructure this PR to accomplish that as you indicated in the initial comments.
Thanks!

Definitely.
I added back the kernel installation to the setup.py so now it will work as it was working before and install kernel on pip install, so single command install. The only difference is that I added a flag for that to be optional so we can disable that when building the conda pkg.
Let me know what you think and if you can test the conda pkgs. If not is ok, I will just go ahead and move the conda recipes out of the PR to the external repo so we can merge this one.

Ok, I tried this out too, on a linux VM. I installed anaconda3, installed the conda-build, built sas_kernel/saspy; actually I only said build sas_kernel and it saw the dependancy of saspy and found it local and built it first, then sas_kernel. So that worked too! Cool.
The saspy version it installed was a way out of date version. I see there's a hard coded version in the yaml file.
So, how is this all supposed to work going forward? Where does the code go? Who builds these packages as we push code, increase version numbers and build new pip packages?
pip we do ourselves. Do we build new conda packages at that same time?
Thanks!
Tom

Yes, the recipe versions are hard coded, there is no way around that since the conda pkg are build for one version.
Going forward I am moving the recipes on this PR to conda-forge which is a community maintained repository of conda packages (almost 3000 pkgs right now). Each recipe gets its own git repo on github, called a feedstock, for example this one for tensorflow.
Each repo (feedstock) has different maintainers that update the recipe when new versions are available from upstream then the conda-forge system will automatically build new conda packages for different python versions and OSes (win, macox, linux).
I will do the submission of this recipes for conda-forge and maintain this recipes, I will be more than happy to add people who are interested on maintaining sas_py and sas_kernel.

I just removed the recipes from the PR, let me know if there is something else I need to do to finish the PR. After that all I would need is a new version of sas_kernel on pypi to build the packages on conda-forge.

Ok, so, as I understand it, the conda recipes will live in conda forge (whereever, not in saspy or sas_kernel repos). And all we need to do with those is update the versions in the coresponding yaml files when our repo versions change (when we build new pip packages).
Jaren and I would be the only ones doing that. We don't mind if we just need to do a PR from those repos to update the versions and you merge them in, or if you want us to have access the change them. Seem fine for us to just PR a new number.
Then, in sas_kernel, there a new install file and changes to the setup file. Have we verified the changes have no effect on the regular pip install? I just tried doing the conda and it worked but I haven't tried this code using pip (and pypitest to be sure the pip package works too)?
Assuming it's all fine, would there not need to be the same setup/install file changes for saspy too?
saspy is it's own package. saskernel depends on saspy being there, but not the other way around.
So does it need tweaks to be conda installed on it's own when the kernel isn't being installed?
BTW, I'm planning a building a new pip (2.1.7) for saspy some time next week when I finish a couple more issues and add doc for seom new features. We can build a new sas_kernel pip too, after merging this in. I think it's still at 2.1.0, so it'll either be 2.1.1 or we can skip it up to 2.1.7 also, just to have a common starting point for both with the new conda integration.
Thanks,
Tom

Cool, thats sounds good to me. I will add you guys as maintainers of those when i submit them to conda-forge.
I have tested that the pip installation of this patch and it works as it was working before, feel free to test it.
No, we don't need the changes on saspy, the changes here were about the kernel installation and saspy is just a regular python package that doesn't add anything extra.
The timing sounds great.

Ok, cool. That sounds good. I'll post back next week when we have new pip's built.
Thanks!
Tom

Ok, I've build new pip's for both saspy and sas_kernel. Both are now at version 2.1.7.
Now, you can work your magic and build conda installs from that.

Awesome I will do that!
Thanks so much.

Just a head up conda pkgs are now available:

https://github.com/conda-forge/sas_kernel-feedstock

Awesome! Thanks.

@danielfrg I've created a new saspy pypi package: 2.1.8. Can you create the corresponding new conda installer?
sas_kernel hasn't changed, just saspy.
Thanks,
Tom

Yes, I can handle that. Thanks for the heads up.

@danielfrg Just created version 2.2.2 of saspy. Want to make the conda version?
Thanks!
Tom

scratch that. 2.2.3 in a few minutes. broke something; gotta fixsorry,
tom

No problem, updating the conda pkg to 2.2.3 here: https://github.com/conda-forge/saspy-feedstock/pull/3
If its easier next time you can just create an issue in that repo and ping me there.

Thanks! I will do that next time, I mean after this. I built 2.2.3 out of the wrong directory. Bad day. Can you delete that one and do 2.2.4. Man, I'm sorry about this.
Thanks!
Tom

No problem at all, let me know when its ready and I will build 2.2.4

Great, thanks! It's out there
Tom

Great, I just updated the conda package to 2.2.4!

Awesome, thanks man!


Who ever changed this article; please reach out to me immediately.  Thanks

Who ever changed this article please reach out to me.  Thanks!

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

ID: 77fea1bd-47c1-ca3f-7a67-9b629a2e9d12
Version Independent ID: 7b1aacb9-b57b-b4c7-bb64-0b7a7ebb249b
Content: Manage messages in Message center
Content Source: Office365-Admin/manage/manage-messages.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

I have reached out and added a section on how to leave feedback on a message center post.


Convert Workflow.fit() to accept a Dataset instead of a ddf

This enables operator fit() methods to pass a Dataset object through to the fit() methods of underlying subgraphs.
Depends on https://github.com/NVIDIA-Merlin/core/pull/299

Looks like this PR also depends on https://github.com/NVIDIA-Merlin/core/pull/301


Fix open_command for msys in the git-for-windows SDK

Added an option for the msys value of $OSTYPE which is set by the git-for-windows SDK
https://github.com/git-for-windows/build-extra/releases/tag/git-sdk-1.0.1

Can you tell me how to test this? How do you run oh-my-zsh from Git for Windows?

Here are the steps I used:

install the 'Git for Windows SDK' from https://github.com/git-for-windows/build-extra/releases/tag/git-sdk-1.0.1 (I believe anything built on MSys that has pacman included will do)
run {installDir}\mingw64_shell.bat
install zsh using pacman: 'pacman -S zsh'
from within bash, start zsh
install oh-my-zsh according to the instructions

eventually, I made a copy of mingw64_shell.bat named mingw64_zsh.bat that replaces launching mintty.exe with /bin/sh with /bin/zsh so I wouldn't have zsh running as a child process of bash, but I don't thin that is necessary for this.

Ok, I could try this, thank you for your clear instructions. Unfortunately, this doesn't work well with arguments with spaces in them. Let me investigate further...

Ok, I found the culprit. It seems the start command will interpret the first parameter as the title if this parameter contains spaces, so if you try start "abc def" it will open a new command prompt with the title being abc def. I solved it by adding "".

Oh, that’s right. I completely forgot about that. Thanks for the quick fix.
From: Marc Cornellà<EMAIL_ADDRESS>Sent: Monday, 30 November, 2015 01:43 PM
To: robbyrussell/oh-my-zsh
Cc: Mark Ingalls
Subject: Re: [oh-my-zsh] Fix open_command for msys in the git-for-windows
SDK (#4597)
Ok, I found the culprit. It seems the start command will interpret the
first parameter as the title if this parameter contains spaces, so if you
try start "abc def" it will open a new command prompt with the title being abc
def. I solved it by adding "".
—
Reply to this email directly or view it on GitHub
https://github.com/robbyrussell/oh-my-zsh/pull/4597#issuecomment-160755614
.

No worries, thanks for your contribution!


Enable compiler warnings for Android

Description
I believe an SDK update fixed whatever issues were causing us to previously disable certain warnings on Android because as far as I can tell it's fine to use them. Accordingly I fixed a few little warnings that cropped up.

SFMLBuildMaster: Build this please.


Support for [TOC]?

I am using DOMDocument to fetch the h1 and h2 tag tree (not a good method I think) to make a tree for my app. Would be awesome if it gets implemented :smile:

I'm not sure I understand.

Well the gist is that when a users adds the [TOC] tag in the markdown it will return an ordered list of all the headings and stuff... This is the markdown one.

Since this is not part of the Markdown Extra specification, I wouldn't add it to Parsedown Extra, but it sounds like a valuable feature. It be nice to have a Parsedown Extra extension that supports it.

:+1:


Server not showing up in Friends tab

I have been running the server for a few months with no problems using Docker Desktop for Mac (Intel).  Yesterday my kids could not connect on their iPads because it was outdated.  I restarted the server to upgrade to the latest version.  Now it will not show up in the friends tab.  I also cannot connect when I enter the information manually on the servers tab.
I am using all of the default ports and settings.  Here is what I am getting from the log output:
time="2022-05-08T16:26:35Z" level=debug msg="Using /data to match uid and gid"
time="2022-05-08T16:26:35Z" level=debug msg="Resolved UID=0 from match path"
time="2022-05-08T16:26:35Z" level=debug msg="Resolved GID=0 from match path"
time="2022-05-08T16:26:35Z" level=debug msg="stdin is detached, so forwarding is disabled"
Looking up latest version...
Starting Bedrock server...
NO LOG FILE! - setting up server logging...
[2022-05-08 16:26:40:892 INFO] Starting Server
[2022-05-08 16:26:40:892 INFO] Version <IP_ADDRESS>
[2022-05-08 16:26:40:892 INFO] Session ID 11736988-edec-40d5-8ba4-09e7873211b2
[2022-05-08 16:26:40:901 INFO] Level Name: Bedrock level
[2022-05-08 16:26:40:984 ERROR] xuid or permission missing from permissions file: permissions.json
[2022-05-08 16:26:40:988 INFO] Game mode: 1 Creative
[2022-05-08 16:26:40:988 INFO] Difficulty: 1 EASY
[2022-05-08 16:26:53:753 INFO] opening worlds/Bedrock level/db
[2022-05-08 16:27:29:345 INFO] IPv4 supported, port: 19132
[2022-05-08 16:27:29:346 INFO] IPv6 not supported
[2022-05-08 16:27:32:396 INFO] Server started.
[2022-05-08 16:27:33:463 INFO] IPv4 supported, port: 46253
[2022-05-08 16:27:33:464 INFO] IPv6 not supported
I am not sure why it is reporting the permissions.json file is missing because it is there and I did make any changes other that restarting the server.  Any help would be greatly appreciated.

From that error
2022-05-08 16:26:40:984 ERROR] xuid or permission missing from permissions file: permissions.json

it sounds like something is malformed in the file.

As far as I can tell there is nothing different in the permissions file from the previous version.  After updating Docker Desktop for Mac, I made a backup of the file, deleted the original permissions.json, and restarted the server.  It is still not showing up in the game.  This is the log file output:
Looking up latest version...
Starting Bedrock server...
NO LOG FILE! - setting up server logging...
[2022-05-13 16:49:39:555 INFO] Starting Server
[2022-05-13 16:49:39:555 INFO] Version <IP_ADDRESS>
[2022-05-13 16:49:39:555 INFO] Session ID e8f6c67e-8fc4-43fc-ac87-e6c5f59cc80a
[2022-05-13 16:49:39:590 INFO] Level Name: Bedrock level
[2022-05-13 16:49:39:782 INFO] Game mode: 1 Creative
[2022-05-13 16:49:39:782 INFO] Difficulty: 1 EASY
[2022-05-13 16:49:48:204 INFO] opening worlds/Bedrock level/db
[2022-05-13 16:50:03:783 INFO] IPv4 supported, port: 19132
[2022-05-13 16:50:03:783 INFO] IPv6 not supported
[2022-05-13 16:50:04:149 INFO] Server started.
[2022-05-13 16:50:05:216 INFO] IPv4 supported, port: 34328
time="2022-05-13T16:29:17Z" level=debug msg="Forwarding signal" signal=terminated
time="2022-05-13T16:29:17Z" level=debug msg="Sending message on stdin due to SIGTERM" message=stop
time="2022-05-13T16:37:01Z" level=debug msg="Using /data to match uid and gid"
time="2022-05-13T16:37:01Z" level=debug msg="Resolved UID=0 from match path"
time="2022-05-13T16:37:01Z" level=debug msg="Resolved GID=0 from match path"
time="2022-05-13T16:37:01Z" level=debug msg="stdin is detached, so forwarding is disabled"
time="2022-05-13T16:38:50Z" level=debug msg="Forwarding signal" signal=terminated
time="2022-05-13T16:38:50Z" level=debug msg="Sending message on stdin due to SIGTERM" message=stop
time="2022-05-13T16:42:15Z" level=debug msg="Using /data to match uid and gid"
time="2022-05-13T16:42:16Z" level=debug msg="Resolved UID=0 from match path"
time="2022-05-13T16:42:16Z" level=debug msg="Resolved GID=0 from match path"
time="2022-05-13T16:42:16Z" level=debug msg="stdin is detached, so forwarding is disabled"
time="2022-05-13T16:49:28Z" level=debug msg="Forwarding signal" signal=terminated
time="2022-05-13T16:49:28Z" level=debug msg="Sending message on stdin due to SIGTERM" message=stop
time="2022-05-13T16:49:31Z" level=debug msg="Using /data to match uid and gid"
time="2022-05-13T16:49:31Z" level=debug msg="Resolved UID=0 from match path"
time="2022-05-13T16:49:31Z" level=debug msg="Resolved GID=0 from match path"
time="2022-05-13T16:49:31Z" level=debug msg="stdin is detached, so forwarding is disabled"
[2022-05-13 16:50:05:216 INFO] IPv6 not supported
Still not sure why it won't show up.  Do you have any other suggestions I can try?

I don't play on Bedrock as much Java, so my own advice is limited. Perhaps ask on the discord server.
Otherwise, maybe op'ing with commands on the servers console will get them in there.

I do have same issue after 1.19 upgrade. Has been working fine and showing up on version 1.18, from now it's not visible at all. Nothing (except upgrade) has been changed.

Sounds like an issue to report to Mojang since nothing changed with the docker image.

I do have same issue after 1.19 upgrade. Has been working fine and showing up on version 1.18, from now it's not visible at all. Nothing (except upgrade) has been changed.

Did you ever find a fix?


add PDB manager #270

Reference Issues/PRs
#270 @amorehead
What does this implement/fix? Explain your changes
Adds a utility for creating selections of experimental PDB structures
What testing did you do to verify the changes in this PR?
WIP
Draws the following metadata:
id | pdb | chain | length | molecule_type | name | sequence | ligands | source | resolution | experiment_type
-- | -- | -- | -- | -- | -- | -- | -- | -- | -- | --
100d_A | 100d | A | 10 | na | DNA/RNA (5'-R(*CP*)-D(*CP*GP*GP*CP*GP*CP*CP*GP... | CCGGCGCCGG | [SPM] |   | 1.90 | diffraction
100d_B | 100d | B | 10 | na | DNA/RNA (5'-R(*CP*)-D(*CP*GP*GP*CP*GP*CP*CP*GP... | CCGGCGCCGG | [SPM] |   | 1.90 | diffraction
101d_A | 101d | A | 12 | na | DNA (5'-D(*CP*GP*CP*GP*AP*AP*TP*TP*(CBR)P*GP*C... | CGCGAATTCGCG | [CBR, MG, NT] |   | 2.25 | diffraction
101d_B | 101d | B | 12 | na | DNA (5'-D(*CP*GP*CP*GP*AP*AP*TP*TP*(CBR)P*GP*C... | CGCGAATTCGCG | [CBR, MG, NT] |   | 2.25 | diffraction
101m_A | 101m | A | 154 | protein | MYOGLOBIN | MVLSEGEWQLVLHVWAKVEADVAGHGQDILIRLFKSHPETLEKFDR... | [HEM, NBN, SO4] | Physeter catodon | 2.07 | diffraction
... | ... | ... | ... | ... | ... | ... | ... | ... | ... | ...
9xia_A | 9xia | A | 388 | protein | XYLOSE ISOMERASE | MNYQPTPEDRFTFGLWTVGWQGRDPFGDATRRALDPVESVQRLAEL... | [DFR, MN] | Streptomyces rubiginosus | 1.90 | diffraction
9xim_A | 9xim | A | 393 | protein | D-XYLOSE ISOMERASE | SVQATREDKFSFGLWTVGWQARDAFGDATRTALDPVEAVHKLAEIG... | [MN, XLS] | Actinoplanes missouriensis | 2.40 | diffraction
9xim_B | 9xim | B | 393 | protein | D-XYLOSE ISOMERASE | SVQATREDKFSFGLWTVGWQARDAFGDATRTALDPVEAVHKLAEIG... | [MN, XLS] | Actinoplanes missouriensis | 2.40 | diffraction
9xim_C | 9xim | C | 393 | protein | D-XYLOSE ISOMERASE | SVQATREDKFSFGLWTVGWQARDAFGDATRTALDPVEAVHKLAEIG... | [MN, XLS] | Actinoplanes missouriensis | 2.40 | diffraction
9xim_D | 9xim | D | 393 | protein | D-XYLOSE ISOMERASE | SVQATREDKFSFGLWTVGWQARDAFGDATRTALDPVEAVHKLAEIG... | [MN, XLS] | Actinoplanes missouriensis | 2.40 | diffraction

Currently missing:

[x] Download method.
[ ] Clustering method with MMSeqs.

Pull Request Checklist

[ ] Added a note about the modification or contribution to the ./CHANGELOG.md file (if applicable)
[ ] Added appropriate unit test functions in the ./graphein/tests/* directories (if applicable)
[ ] Modify documentation in the corresponding Jupyter Notebook under ./notebooks/ (if applicable)
[ ] Ran python -m py.test tests/ and make sure that all unit tests pass (for small modifications, it might be sufficient to only run the specific test file, e.g., python -m py.test tests/protein/test_graphs.py)
[ ] Checked for style issues by running black . and isort .

@amorehead I've added the clustering utils - does this cover what you were hoping for?
You also mentioned some structural clustering - I also think this would be good. Do you have a preferred method?

@a-r-j, all these changes look great! I've gone ahead and created another pull request using a personal fork of Graphein's latest master branch. My hope is that we can use this pull request to finish developing the remaining functionality listed. Let me know if you have any questions, comments, or concerns.


Avoiding suffix value on ngChangeModel

When a use some suffix value attribute in the suffix parameter it is applied to the value of ngModel variable incorrectly changing my ngModel variable, could I do something to avoid this ?
This is an example of my code:
<input class="form-control default" currencyMask          [(ngModel)]="realtySimulation.entry"         (ngModelChange)="onChangeSimulation('entry',$event)"         [options]="{ prefix: 'R$ ', thousands: '.', decimal: ',', suffix: ' ('+somevalue+'%)' }"/>

Sorry, it's not an expected usage of the attribute. I can't help you with that.


[Feature Request] template-filename YAML: Support for Date Format Syntax

Hello @mo-seph !!!!!
Would be great to have support for date format syntax the same way that we have with obsidian core plugin Template
{{date}}
{{time}}
{{date:YYYY-MM-DD}}
{{time:HH:mm}}
My Workflow: Journals
I have note journals for different purposes: Dream Journal, Gratitude Journal and Anger Journal
Each one of then have is own template, filename and folder.
I have one note for day for these three journals
For example: Anger Journal

Folder Path: Journal/Anger_Journal/
Filename: AN_{{YYYY-MM-DD}}.md

If we have a new date format syntax I could create an YAML like this to my daily anger journal template
---
template-output: Anger_Journal
template-filename: AN_{{date}}
tags: [Journal/Anger]
---

{{Question1:area}}

{{Question2:area}}

{{Question3:area}}

Thanks for reading this :)
Have a great day!

See the new currentdate field type, e.g. {{now:currentdate:dd-MM-yyyy}}. Format elements are from https://moment.github.io/luxon/#/formatting. It should cover both date and time by using different formats.

Hello @mo-seph !!!!
Thank you very much !!!!!
Working perfect on version 0.1.8 Obsidian 0.12.19
I wish you a fantastic day !


Added a pokemon in index.html file

As a beginner, I have managed to try this. Thank You this was a great learning!

Thank you!
On Tue, 12 Oct 2021, 09:09 Aayush Mishra, @.***> wrote:

Merged #573 https://github.com/AM1CODES/Poke-Dex/pull/573 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AM1CODES/Poke-Dex/pull/573#event-5446734794, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASG4FWNZ6P3QQK76TCGOUWLUGOUWJANCNFSM5FVN6IPQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Normalization

What input normalization did you use?

As same as original repo, I use ImageNet mean and std normalization.


"NavigateTo" dialog successCallback Issue

The callback function is called when navigating to dialog not on clonsing it

@abdelaziz-benelfakir Can you give us more details about the issue.

When I use the function "NavigateTo" to open a webressource the callback function is called when the webressource is Opend not on close

@abdelaziz-benelfakir This issue is fixed, can you verify and let us know if this is still happening with you.

@abdelaziz-benelfakir Thanks for contacting us. As we haven’t heard back from you, I am closing this issue. If you need further help, please reopen the issue or open a new issue.


investigator comments from first test

Overall, I think the online form is easy to navigate and follow. I didn’t even need the instruction page, except when I was trying to troubleshoot the glitch. So even the PI’s will be able to figure it out. It was slower for me than using the word doc version, because there are so many fields you have to click through (rather than just dumping it all into Word). So it took a while to work through all of the fields. And some of that extra time was trying to figure out little quirks (like why it wasn’t saving the unit description). Also, uploading my CSV didn’t work at first, because it didn’t recognize the commas between fields for some reason. So the portal insisted there was just one column of data (whose header was AAAAaaaall of my column headers). I had to delete, re-save the CSV, and reupload it a few times to get it to finally notice the commas. I assume it’s an Excel issue, and eventually it sorted itself out. (But maybe being able to upload a tab-delimited txt file would be a good 2nd option…?) Now that I’m familiar with it, it’ll go faster next time. And an upside is that I don’t have to keep an eye out for updates to the Word doc. (I always send you the word doc and hope that it’s still the right one, haha.)
One other thing I noticed: every time I filled out the Word doc for you, I would start to paste in the abstract from the paper, and then read your note about how we can’t use the published abstract from a journal article. (Copyright issue, I assume.) If that’s still a concern, you might want to put that note into the form. I know I’ll forget by the next time I have a dataset to submit.

from J. Ide regarding uploading csv:
Regarding the custom unit description, I just tried it and it worked for me. Could you please send me the data package where you’re seeing this behavior? You can use the Send to Colleague feature to send it to<EMAIL_ADDRESS>Thanks!
Also, I just wanted to note that ezEML does accept tab-delimited files. You are able to specify the delimiter when you do the upload. But you probably already saw that… 😊
When saving CSV files in Excel, it’s safest to select CSV UTF-8 as the output format.

from from J. Ide regarding uploading csv:
ezEML refers to “CSV” files, for example in the button “Load Data Table from CSV File”, but it doesn’t mean that literally. I.e., when you click the button you’re taken to a page where you have the opportunity to specify the separator and can pick tab or space. They’re referred to as CSV files as a sort of shorthand for “delimited text file.” ezEML’s “help” does explain that, but perhaps I should make it clearer in the button naming.


Fix wc issue and name display

Bump reactive-dots to take into account https://github.com/tien/reactive-dot/pull/400
move chainIds to optional. WC now works well with Multix, and ppl can delegate with their Multisig using Delegit :tada:
fix an issue where the name wasn't displayed when account was selected
bump some other minor deps
update papi descriptors

@wirednkod are you ok with me force-merging, I'm the one dealing with subsequent issues in any case :)

sorry for the delay @Tbaut


sysctl doesn't support fs.file-max

Please use the following bug reporting template to help produce actionable and reproducible issues. Please try to ensure that the reproduction is minimal so that the team can go through more bugs!

A brief description
I'm trying to see if I can build coreclr on WSL, following the instructions here

The docs state
To ensure that your system can allocate enough file-handles for the corefx build run
sysctl fs.file-max. If it is less than 100000, add fs.file-max = 100000 to /etc/sysctl.conf,
and then run sudo sysctl -p.

Expected results
Prints the file-max on my box

Actual results (with terminal output if applicable)
sysctl: cannot start /proc/sys/fs/file-max: No such file or directory

Your Windows build` number
14393.351

Steps / All commands required to reproduce the error from a brand new installation
sudo sysctl fs.file-max

Strace of the failing command
N/A

Required packages and commands to install
sudo sysctl fs.file-max

See our contributing instructions for assistance.

It’ſ not exactly ſyſctl, it’ſ unimplemented ſyſfſ featureſ

Sorry, your message looks mangled on my Pc? I think you made a formatting error!

@hughbe I was using a joke android keyboard that replaces the "short s" with the historical "long ſ".

Why have one way to query something (ulimit system call) when you can have 2 or more?
This should be really quick to implement - I've filed a task to get this in.

I have exactly the same problem, while installing SAP HANA Express on WSL!
Any news on that? will it be fixed soon? (more sys/fs structure?)

I am also missing fs.aio_max_nr

commented on 8 Nov 2016
This should be really quick to implement - I've filed a task to get this in.

Good one.
Duping up to #2893 where the +1 crowd hangs out.


Error when using theme editor on VS2019 16.0-preview-3

I realize these things take time to get updated, but I didn't see an issue yet. The plugin is currently incompatible with the latest preview.

Just upgraded from 2017 and ran into this error as well. Happens when trying to load saved themes as well and have even managed to lock up and crash VS when trying to switch themes. Oddly enough it seems to still be able to switch the themes for the most part despite throwing an error each time.

Fixed the issue, and this should be a simple fix for them too.
ColorThemeEditor.dll just needs to be recompiled with reference to Microsoft.VisualStudio.ExtensionManager and .ExtensionEngine changed from <IP_ADDRESS> to <IP_ADDRESS>, as it appears these two DLLs had a major version change between preview 2 and 3, and thus <IP_ADDRESS> no longer exists.
As a temporary workaround for anyone on preview 3, you can extract the ColorThemeEditor.vsix, as it's a ZIP file, and then modify those two reference's versions in the ColorThemeEditor.dll and then ZIP the files back up, rename extension to .vsix and install that. (I'd provide the modified file but I'm not sure if that part of this plugin is open source, as the code is not in this repo).
Works perfectly fine after making that simple change.

Hey Invertex - your fix is valid. We're working on updating the extensions in the marketplace. Stay tuned for a new extension for VS 2019 that references the later version of ExtensionManager.

I'm closing this issue because it looks to me like it has been resolved. Please re-open if necessary https://marketplace.visualstudio.com/items?itemName=VisualStudioPlatformTeam.VisualStudio2019ColorThemeEditor


createCursor command throws error when used in keybinding

VSCode Version: 1.40
OS Version: Windows 10
Does this issue occur when all extensions are disabled?: Yes

Steps to Reproduce:

Add a keybinding for createCursor to ctrl+alt+enter when editorTextFocus
Use keybinding
Error thrown: Cannot read property 'lineNumber' of undefined

Expected results: The cursor is moved to the location of the mouse or the middle of the editor view. This would be helpful for when I scroll using the keyboard commands and want to drop my cursor where I'm looking without picking up my mouse to click the screen.

{
	"key": "ctrl+alt+enter",
	"command": "cursorMove",
	"when": "editorTextFocus",
	"args": {
		"to": "viewPortCenter"
	}
},

I didn't know about this.  Thank you!  Closing ticket.


Build 1.0.3

Remember to rerun the examples.

Coverage remained the same at 68.675% when pulling f7e7cfd445960d118278ae7b43df155a0f74bd5b on 1.0.3 into 81227f657c296065e258c275c895ba95e91e6de0 on master.

Coverage increased (+0.1%) to 68.817% when pulling 49bc4630cb7d9aacdd0e79b55601a1d73cb8e4d7 on 1.0.3 into 81227f657c296065e258c275c895ba95e91e6de0 on master.

Coverage increased (+0.1%) to 68.817% when pulling 7f1374cf60cec414ea5e181310fd7e10a8ac2e9c on 1.0.3 into 81227f657c296065e258c275c895ba95e91e6de0 on master.

Coverage increased (+0.1%) to 68.817% when pulling c6c219a1fe995046b52fe0d2494f5a7ccaa216d4 on 1.0.3 into 81227f657c296065e258c275c895ba95e91e6de0 on master.

Coverage increased (+0.1%) to 68.817% when pulling 800fb6179b8cd0d591962f74d3b3106bd562cbc2 on 1.0.3 into 81227f657c296065e258c275c895ba95e91e6de0 on master.

Coverage increased (+0.1%) to 68.817% when pulling bd1628b7d963f7ac471ac2eebba81926cab8d4a0 on 1.0.3 into 81227f657c296065e258c275c895ba95e91e6de0 on master.

Coverage increased (+0.1%) to 68.817% when pulling 93cc45d0e2b834677d4688edf40dd5f583d843a6 on 1.0.3 into 81227f657c296065e258c275c895ba95e91e6de0 on master.

Running the examples.

Coverage increased (+0.1%) to 68.817% when pulling d8f2c2d3a265c316929c5f0a9f9b2fd0ec7c25d6 on 1.0.3 into 81227f657c296065e258c275c895ba95e91e6de0 on master.

Coverage increased (+0.1%) to 68.817% when pulling 78145f4de5a27f4fb5f98ef2cc2621b8fa1479a1 on 1.0.3 into 81227f657c296065e258c275c895ba95e91e6de0 on master.

Coverage increased (+0.1%) to 68.817% when pulling a6d3c473eb1b107fd1a4b20f9a60bfe20d81d8e6 on 1.0.3 into 81227f657c296065e258c275c895ba95e91e6de0 on master.


docs,LRP: Add steps to restart agent and operator pods and update feature roadmap status

/test

@ysksuzuki The 2nd commit to promote the feature status to stable was incorrectly backported to older releases (< v1.16). Could you revert those? Thanks!


BUG: CategoricalIndex.get_loc returns array even if it is unique

[x] related to #11588
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

Made CategoricalIndex.get_loc to return the same result as normal Index.
data2 = pd.DataFrame([[1,2,3],[3,4,5]], index=pd.Categorical(['one', 'two']))

# in current master, it returns DataFrame with 1 row
data2.ix['one']
#      0  1  2
# one  1  2  3

# after this PR, it returns Series
data2.ix['one']
# 0    1
# 1    2
# 2    3
# Name: one, dtype: int64

@sinhrks this looks good. but is this the correct reference issue: #11588 ?

Ah, related to #11558, but can't close only by this PR. Fixed the doc and comment.

@sinhrks thanks! nice PR as usual


docs: generate sample code in the Java microgenerator

This PR was generated using Autosynth. :rainbow:
Synth log will be available here:
https://source.cloud.google.com/results/invocations/89bbfe8d-0991-4bf5-9ea3-b2a25748a524/targets

[ ] To automatically regenerate this PR, check this box.

PiperOrigin-RevId: 356341083
Source-Link: https://github.com/googleapis/googleapis/commit/8d8c008e56f1af31d57f75561e0f1848ffb29eeb

@googleapis/yoshi-java sample tests are failing.


fix: enhance Markdown text cleaning for TTS compatibility

Enhanced Markdown formatting removal
Discussion #7925
Changes

Expanded the removeFormattings function to handle more Markdown syntax
Added preservation of text within bold and italic markers

Context
Azure AI Speech Services would not process text enclosed with asterisks (i.e. bold, italic...), issue #5560
Tests
Please check the fiddle I made to test the changes (markdown formatting stripping and cleanup)

Please target dev branch next time, Thanks!


Feature: Searchbox directive

Adding following files for the search box feature

searchboxdirective.ts
searchboxdirective.test.ts

Thanks for the submission... I'm still working on docs and getting the automated tests running, but until then, can you update your submission to include two things?

[ ] rename your test file to *.spec.ts
[ ] include example usage with the folder demoBasicUsage similar to how @rolandoldengarm did here: https://github.com/rolandoldengarm/ng-officeuifabric/tree/34d4f38bd7df35e1c09bbf3ddbdbd509d98aafe1/src/components/textfield

Looks like you have some build errors... see the details of the build. It appears you have invalid path references to the type definitions in your spec file, but you don't even need those... you can remove them all together because the tsconfig.json file in the root configures TypeScript to build them.
Please ensure your build passes before submitting the PR. Getting close! :smile:

I see there is some problem with the uniqueids spec will fix it and recommit

I do not know if this commit should be counted as valid ??

I also noticed you have jQuery dependencies. It shouldn't be too hard to remove that dependency. Easiest way is to use querySelector.
Probably best / fastest is to use other techniques, like jqLite, or by not using javascript at all to bind events. In this case it's only used to attach events to HTML elements in the template, there are other, more angular ways of doing this.

Also noticed the code coverage for the directive is pretty low (~60%) which brings the overall coverage number down to ~83%... while it might seem like the bar is set unreasonably high I think with multiple people contributing to an open source project, it's important we have good test coverage.
If you need help running the local jobs to see what the test coverage is and what lines are / are not being hit, ping me here. I'm going to write a doc up about it but in the mean time, happy to help.

I agree with @rolandoldengarm regarding jQuery dependencies, I will remove it, add a controller for the operations and commit again,
@andrewconnell :I understand the purpose of high %age of code coverage, but I am not used to code coverage and might take a few more commits to get it done, Please bear with me :)

No sweat... happy to help if you request it. Might try running the code coverage reports locally though and not having to wait for them to be reported here.
They are build automatically when you run the tests. When you run $ gulp test or ./node_modules/karma/bin/karma start ./build/karma.conf.js --single-run, karma internally uses istanbul and creates a /coverage folder. Nested within that you'll see an HTML report which is the same thing that coveralls.io is reporting in your build check.
BTW, soon I plan to add TypeScript vetting to the travis build pipeline... so if you throw linting errors in your TypeScript, you'll have to address those before the PR is accepted & merged. You can run a local report using gulp vet-lib-ts to see what issues you have in your TypeScript.

What should the expected coverage of code be?

Ideally pretty high… we have it set to fail an automatic build if it isn’t above 90%
If you merge the latest changes from the dev branch you’ll get an update from issue #52 that will help as we were getting penalized for not testing the dynamically generated code for webpack. The new workflow uses the coverage for the the JS we generate, not the stuff webpack generates, which is more correct. IOW, files like core.ts & components.ts are now up to 100%

Just pulled this down to run locally... looks good! I noticed two things you do need to address (one of which we aren't accounting for in the automatic Travis builds, but it will eventually cause a failing build):

remove the console.log() statements from your tests... we should only see test results when you run gulp test --specs
you have 90+ code style issues that you can see if you run gulp vet ... these are easy fixes to knock out (*this is the one that should get added to travis builds in the future)

Your code coverage looks pretty good (90%+).
However when I try to run the demo, I'm getting an errors loading the 'demoAppmodule... one is pretty easy... you need to make sure you add the searchbox to thesrc\core\components.ts` file. However I'm seeing two other errors showing up related to uniqueId's:

Final touches to Searchbox directive commited

Removed console.log from searchboxdirective.spec.ts
Fixed gulp vet issues
Removed gulpfile.js
Resolved errors while running the demo

:+1: awesome... I'll pull it down later today and run through things again, hopefully pushing it out to dev!

I'm real sorry to do this, but I'm still having issues with the searchbox component when I try to merge it... we're close though!
First, when I went to merge stuff, there were conflicts that needed to be resolved because your branch is not in sync with the main repo's dev branch. Because you'd been on this one for a while I was going to do this for you, but then ran into blocking issues below... so next time you submit, please make sure you resync your local dev branch from upstream & rebase your branch off the updated one. Check the PULL-REQUESTS for info on resync'ing your repo from upstream to avoid merge conflicts.

[ ] there's no documentation on the scope interface in your directive... see how the ITextFieldScope is documented for reference.
[ ] one of your tests is failing: should have unique ids
[ ] suggest that you include one of your demos to have a searchbox with no text prefilled, for instance, nothing set on the uif-value attribute... in fact...
[ ] because there's already a valid value attribute, we really should be using that so we don't have a custom attribute defined?
[ ] to see changed is to have the attribute uif-search to be changed to placeholder following the same convention we are using in existing directives, specifically the textfield. This could have been caught had we had a spec in the original issue like the other directives we have... you just jumped on this one so quick which is awesome! :)

It's close... I know it seems like every time you submit something it's something different that's keeping it from getting added... really appreciate your commitment and effort here!

@andrewconnell I am sorry, I did not understand the 4th point
because there's already a valid value attribute, we really should be using that so we don't have a custom attribute defined?

Could you please elaborate on it?
Thank you

I'm referring to the uif-value attribute that the directive adds. There's already an attribute value so just use that to pass the value into the textbox to prefill it. We don't need to create our own implementation of the value attribute as the value attribute already does what you're doing with uif-value.
As for the tests... as for the tests, I don't know what to say... when I pull your PR down I am doing it on a clean branch that's been rebased off dev so it should be pretty clean. I can take another look when I merge some other PR's today and send you the details if it happens again.

I am not sure why the tsc -p ./ command fails in travis, works perfectly locally. @andrewconnell  please check

Check the casing of the name. The filename is searchBoxdirective.ts but you're referencing it with a lcase b. I'm guessing your developing on Windows where case doesn't matter... the build script runs on a Linux container where case matters.

I just saw the issue with the failing test... check out the recent find by @rolandoldengarm about unique ID's... he ran into it with the uif-toggle directive and found a solution. He talked about it in the slack channel: https://ngofficeuifabric.slack.com/archives/general/p1452564827000297

@andrewconnell I have fixed the unique id issue as per idea by @rolandoldengarm , please check and merge with the master if suitable

Unfortunately this is not ready to get merged due to multiple issues that have been raised previously.
I have checked the items off that i see you have addressed above... but you can see the ones that remain unchecked. I appreciate the work you've done, but please resolve these before resubmitting.
I have cloned your own repo and run the tests and see them failing... this says to me that you are are not verifying tests pass before submitting the PR. You can run tests yourself & they should all pass before submitting a PR.
I'm closing the PR for now and when you have these fixed, you are free to resubmit & reopen the PR.
outstanding issues

[ ] as mentioned previously, failing test on unique id's

[ ] as mentioned previously, we don't want to prefix attributes with 'uif-' that are not custom attributes... if it's an attribute that exists on the underlying HTML control that serves the same purpose. Specifically, this directive should not have uif-value rather that should simply be value
[ ] the jsdoc for the interface does not match the code... properties are listed that are not present & not all properties are documented
[ ] when I ran the demo, i'm not seeing the placeholder text get set in the search control... this should work like other placeholders - default text that's lightly colored from real text & is cleared out once the inbox control onfocus & replaced onblur


vocab.txt for longformer?

Where can we find the longformer-base-4096 vocab.txt file? I could not find one here.
Thanks!

Please check the HF LongformerTokenizer implementation. It inherits from the RoBERTa one.


do we have support for 3.7?

Sorry I've not verified it's working on 3.7. Maybe someone else has?
It's still pretty shaky from previous versions so test test before using it
in any prod environment.
Den ons 28 juli 2021 11:09Paloma Martinez @.***> skrev:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/naboo/craft-siteduplicate/issues/6, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJJ3TM43NHWVC5DO4YOXYTTZ7CMNANCNFSM5BD5UWLA
.

No worries!
I  tried it on 3.7 but didn't show up on my entries pages (where I think it should show up?)
We are planning on designing a solution, not there yet
If we have something worth to share I will post it here!

Hi @naboo I tested plugin on 3.7.22 and After duplicating action it throw a Not found Exception but it should work.
Do you plan to release a compatible version?

Sorry @LukssOne I haven't planned on doing anything with it. Feel free to check yourself.
Looking at the error message it seems the new entry isn't found in the site it's been duplicated towards. My guess is that the code isn't duplicating the entry correctly so the new entry doesn't exist in the new site?

I've just installed this and tried to use it to duplicate an entry into a new site and got the same error. Is this plugin abandoned?

Also just tried duplicating a different entry and got a different error message:

Yeah this is very abandoned at the moment. There's a lot of changes under the hood needed since all the major Craft releases since the last update. I need to pull this Plugin from the store. Will get on that!


KNAPSACK - The Knapsack Problem

The famous knapsack problem. You are packing for a vacation on the sea side and you are going to carry only one bag with capacity S (1 <= S <= 2000). You also have N (1<= N <= 2000) items that you might want to take with you to the sea side. Unfortunately you can not fit all of them in the knapsack so you will have to choose. For each item you are given its size and its value. You want to maximize the total value of all the items you are going to bring. What is this maximum total value?
Input
On the first line you are given S and N. N lines follow with two integers on each line describing one of your items.
The first number is the size of the item and the next is the value of the item.
Output
You should output a single integer on one like - the total maximum value from the best choice of items for your trip.
Example
Input:
4 5
1 8
2 4
3 0
2 5
2 3

Output:
13

make sure your code should pass all the testcase of spoj
note : please read readme.md file carefully

I want on this issue please assign to me


Excel file loading hang on

Current Behaviour (for problems)
open xls file hang on

Expected Behaviour
success when loading excel file
Steps to Reproduce

Open Data Curator using the icon
File - Open EXcel sheet
Choose sheet
Open sheet
Wait...
Abort cause too long

File used:
LIST.xlsx
Your Environment

Include details about the environment you experienced the problem - this will help us fix the bug quicker.

Data Curator version: 0.14.0
Operating System and version: windows10 64bits

Could be related to https://github.com/ODIQueensland/data-curator/issues/651

When I attempt to open LIST.xlsx, even though it has one tab "Globale", Data Curator shows,

I also get the same "Loading Data, Please Wait" screen using Data Curator v0.15.0.
I'll investigate further.

Hi @4estone @Stephen-Gates
This is related to #651
The different worksheets shown possibly have something to do with some of the random data that is in this workbook. When I copy and paste just the table data into a new workbook, just 1 worksheet shows.
The conversion from excel to csv relies on some default properties such as "double quotes" and carriage returns to try to determine where cell fields end and lines end. Unfortunately in the worksheet, there are numerous double quotes and also carriage returns within different cells. The conversion, once it strips away excel 'cells' then leaves carriage returns and tries to implement quotes for text.
@4estone, in 651 we will put in a dialog message that warns of possible problems with data - there may also be some options we can add to assist the library conversion to help it work out where to put fields and line ends (but this may or may not be possible), but at this stage quotes and line returns within excel fields interfere with conversion to csv.

Thanks @mattRedBox
@4estone I'll close this in favour of #651 - There's a reference from that issue back to this one.

Hi @4estone please see #733

First thanks for reactivity.
Copy / paste data from excel to data Curator is a good workaround

Works well in 0.18.0


Spock 1.0

I have added the cleaned up @Unroll vs. testable test from #17 to my fork. Feel free to merge it if you like. Until you fixed the current problem on branch spock_1.0 it will fail.

This change is

Thanks for providing reproducer, it really helped me to nail down what is actually going on. I adjusted the test in 334f5e8228fa1ba29b55ad6f1890cf5b25d42b4a (mainly removed println as that was causing classloading issue due to the fact that we don't bundle groovy deps when testable = false).


Error report regarding Async

I am using async in my tbi program as you do. But it always reported an errror: Atomic access violation. Do you know how I can figure this out? Once I cancelled the async and gave up outputting message in the block, it worked without reporting this problem.

I am using async in my tbi program as you do. But it always reported an errror: Atomic access violation. Do you know how I can figure this out? Once I cancelled the async and gave up outputting message in the block, it worked without reporting this problem.

I'm very curious about your problem, could you provide some details about the system you're using to compile the code and the system you're using to run the two scripts? Is this all Citrix? Is this all from the same Eclipse planning workstation? Is there a third machine involved?

Closing this issue as resolved due to no communication for 2 months