Generate jars.txt for the api module

Rationale
Issue 45124: Update remaining modules to generate credits jars.txt file
Changes

Update build.gradle to use ExternalDependency build utility

Remove reference to labkey-client-api
alphabetize dependencies

Remove/ignore jars.txt from repo (now generated by build)

👍

@labkey-susanh Looks like the version column isn't included in the generated jars.txt file? do we need an update to the gradle plugin to include that column?

The version number is in the name of the jar (almost always), so it was deliberately removed from the generated file since it's redundant (and has been a pain to update twice).  @labkey-ians Are you seeing a place where we need to show this?

No it was just one of the diffs between the Generated Jars.txt and the previously checked in one.


chore: vrr for 39 only, ptyxis for both

do vrr and ptyxis in 39
ptyxis only in 40

Looks perfect, thanks for fixing this!


Tailwind content, purgecss problems and questions

Hey there, thanks for this great starter kit. See my repo here: https://github.com/artzte/juniper-swordplay
Bunch of n00b questions here; pardon me. I was using an older version of your theme and am trying to update.
The util classes aren't being detected in my setup, which I created following your directions here:
 hugo server --disableFastRender --verbose
Start building sites …
hugo v0.94.2-48FB9E4D linux/amd64 BuildDate=2022-03-12T10:28:42Z VendorInfo=gohugoio
INFO 2022/03/13 16:09:54 syncing static files to /
INFO 2022/03/13 16:09:54 postcss: use config file /home/eric/src/juniper-swordplay/themes/juniper-swordplay/assets/css/postcss.config.js
INFO 2022/03/13 16:09:55 postcss:
INFO 2022/03/13 16:09:55 postcss: warn - No utility classes were detected in your source files. If this is unexpected, double-check the `content` option in your Tailwind CSS configuration.
INFO 2022/03/13 16:09:55 postcss: warn - https://tailwindcss.com/docs/content-configuration

I'm confused by the tailwind config. The content entry is referencing relative to the local folder of the tailwind.config.js, which is not where the layouts are.
Also, why is purgeCSS needed? Tailwind already purges, which is why the content entries in the tailwind config are so critical.

Hey, I'd like to take a look into you problem. Your example repo just returns a 404. Could you check if it is live?

Same questions from me:
Why PurgeCSS? All my other Tailwind based projects dropped it already.
How can this possible work with the tailwind config located deep down there in the css directory? Maybe there is a setting to control its "root"?

I finally removed the PurgeCSS, as the TailwindCSS JIT compiler from v3 on only produces the necessary CSS classes. The dependency was a left-over from using TailwindCSS v2.


Report MTU size in 'weave status connections'

With current implementation, the value would be the same for all fastdp or awsvpc connections, but potentially variable between sleeve connections and between invocations.

It's pretty easy to  get it into the output by including it in OverlayConnection.DisplayName, but do we want to be more general than that?  Do we want it accessible as a separate field in weave report ?


Making the synthetic modeling subdirectory (includes GPT datasets)

Synthetic Datasets and Modeling

Added the respective datasets for level0 (construct) and level1 (ecological) stages respectively

Included both the smaller and larger datasets for each stage – train on the larger dataset and predict on the smaller

Included the modeling files for both stages.

Closing this PR because this is modeling related to the conflict project and not to the team communication toolkit.


Agent doesn't log to console correct

Agent Version and Platform
2.147.1

OS of the machine running the agent? OSX/Windows/Linux/...
Windows Server Core 2019
Azure DevOps Type and Version
dev.azure.com/sos-team
What's not working?
Script doesn't log to console. In this case I'm running terraform 0.11.11 on Windows, but I've seen the same behaviour on Linux.
Yaml part:
steps:
- powershell: |
    terraform init -input=false
    terraform plan -out=tfplan -input=false
  workingDirectory: release/peripherals/aws/terraform
  failOnStderr: false
  continueOnError: false
  displayName: Setup Peripherals

I've added a two printscreens, one with the local output when running TF manually on the agent, and one with the output of the console in Azure Pipelines. (Same agent)
The worker log when running this job is also attached. Seems to me that redirection of STDOUT/STDERR isn't correct.
Agent and Worker's Diagnostic Logs
See Worker_20190313-123824.utc.log
Worker_20190313-123824.utc.log

I am experiencing a similar issue with a newly deployed self-hosted agent running Windows 2019 Data Center:

Any output to the error stream is causing a build to fail.  We are executing a PowerShell script that has elements that write log information to the error stream (we are not in control of this output) and previously this would not cause the build to fail even though it would be reported as errors.  With the newly deployed agent these are all resulting in build failures.

We are currently attempting to rollback to a Windows Server 2016 image to verify previous behaviour

@Philo there is a setting you can use to let powershell not failed on STDERR

@TingluoHuang,
This setting will cause a failure on standard error, the default value of False should mean that it does not fail on standard error.  This behaviour is what we see on all other agents we have that are not Windows Server 2019.
We have just completed a test using the same software set on a server 2016 vs server 2019 agent, the 2019 agent fails for the same build that succeeds on 2016, using the same agent version.
Either the "Fail on Standard Error" option is inversed or not applying and causing all standard error output to fail a build.  This is preventing us from moving our build images to 2019

@Philo  i am wonder there is something to do with PowerShell version, do you mind send me a build log with with system.debug=true set as variable.

Apologies @TingluoHuang - due operational issues we needed to quickly cycle our 2019 server agents out and replace them with 2016 images.  If it is of any help the only change we made to our image build scripts (using packer and chocolately) was to change the OS base image from 2019 to 2016.

Can indeed confirm that the issue lies with Windows 2019.

This is an issue with a certain version of Windows 2019, I was able to repro it a while back.
Looks like they already fixed the problem, since I tried a new Azure VM with latest Windows Server 2019 image and the problem goes away.
I am going to close this issue for now since it does not look like an agent issue


file_stats.sh chokes on directories with spaces in the name

Sample error:
fatal: ambiguous argument 'Sources/LFSignupDataSource.h': unknown revision or path not in the working tree

That path is actually Data Sources/LFSignupDataSource.h. Probably just needs to escape spaces in the name when passing files off to git.

thanks, resolved. my bad re the long delay.


Convert server load reporting to promises

@nicolasnoble

Builds on #29008

TODO: need to set peer string


Device info screen for infrastructure providers does not wrap and is not adjustable

When I select a VM under Infrastructure Providers, select a VM, and then select "Devices", the UI does not appear to autowrap very long strings, nor is it adjustable as far as I can tell. The result is that the info on the right gets squished up like so:

Can this be modified so that it wraps, or at least me me "drag" the separator window so that I can size the columns as I like?

@djberg96 Column adjustment is not currently available in Patternfly tables, but is under development.


nltk.classify.decisiontree.demo() fails on 2.0.1rc1

What version of NLTK are you using?  (See nltk.version).  Please only
submit bug reports for the current version.
>>> nltk.version
'2.0b9'
on Mac with Python 2.7.1
What steps will reproduce the problem?  (e.g. include Python source code)
Running the Decision Tree demo (nltk.classify.decisiontree.demo()) fails on 2.0.1rc1 but runs fine on 2.0b9.
What is the expected output? What do you see instead?
Below I show the correct output from nltk 2.0b9 and the error with 2.0.1rc1:
################################
Python 2.7.1 (r271:86882M, Nov 30 2010, 10:35:34)
[GCC 4.2.1 (Apple Inc. build 5664)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import nltk
>>> nltk.version
'2.0b9'
>>> nltk.classify.decisiontree.demo()
Training classifier...
best stump for   5000 toks uses endswith(vowel)=False err=0.2556
best stump for   1802 toks uses has(y)=False         err=0.3152
best stump for   1612 toks uses startswith(k)=False  err=0.3033
...
if has(k) != False:
if count(m) == 2: return 'male'
if count(m) != 2: return 'female'
################################
################################
Python 2.7.1 (r271:86882M, Nov 30 2010, 10:35:34)
[GCC 4.2.1 (Apple Inc. build 5664)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import nltk
>>> nltk.version
'2.0.1rc1'
>>> nltk.classify.decisiontree.demo()
Training classifier...
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/nltk/classify/decisiontree.py", line 281, in demo
binary_names_demo_features)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/nltk/classify/util.py", line 182, in names_demo
classifier = trainer( [(features(n), g) for (n,g) in train] )
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/nltk/classify/decisiontree.py", line 276, in f
return DecisionTreeClassifier.train(x, binary=True, verbose=True)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/nltk/classify/decisiontree.py", line 156, in train
feature_names, labeled_featuresets, feature_values, verbose)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/nltk/classify/decisiontree.py", line 256, in best_binary_stump
fname, fval, labeled_featuresets)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/nltk/classify/decisiontree.py", line 245, in binary_stump
default = DecisionTreeClassifier(neg_fdist.max())
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/nltk/probability.py", line 283, in max
self._max_cache = max([(a,b) for (b,a) in self.items()])[1]
ValueError: max() arg is an empty sequence
################################
Please use labels and text to provide additional information.
I haven't debugged this issue, I upgraded from 2.0b9 to 2.0.1rc1 this morning. I have already filed http://code.google.com/p/nltk/issues/detail?id=667 but I don't think these are related.Migrated from http://code.google.com/p/nltk/issues/detail?id=668

Is there a fix to this? I confirm this issue too.


[Feature request] Options as callbacks

In the same way you can pass properties as callbacks (eg :) $('.els).velocity({ left: function(index, length) {} });, it would be nice to have the possibilitie to do it with options as well (delay, duration...)
A concrete case : I need to move several elements to the same position, but leaving a "trail" doing it.  For that, you add a bit more duration on each element. To achieve that, you have, for now, to loop into your elements and apply velocity on each, which is a complete waste of performances.
It would be great to do $('.els').velocity({ left: 600, top: 200 }, { duration: function(index,length) { return 600 + i * 40 } })

Note that in modern JavaScript you can do that by writing getter functions for the properties.

@Pointy the problem with that is it won't necessarily give the options people expect - some have to be per-animation, some are per-element, and at least one is global only - so specific code and documentation will say what is and isn't supported.
Saying that - really good point ;-)

I've started adding this with the stagger option - more will come over time (just need to ensure they're working properly for each one).


Please invite me to the community

Name
mintysolutions
Additional Context
I want to join vibrant community and connect fellow professionals

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is a remarkable open-source solution tailored for serverless platforms, aimed at empowering creators to craft their infrastructure solutions. 💻 We're thrilled to welcome you into our vibrant community! ✨ We eagerly anticipate your valuable contributions! 🤓 Don't forget to explore our community on Github and connect with us on LinkedIn 💜 Feel free to spread the word about our community to like-minded individuals interested in development, DevOps, technology, and collaborative contributions. 👉💜👈


bug/export with failed container

added protected method to verify bill_of_lading against
defined transport_service for any failures and return False if found

reference #27


Example refactoring extractors to minimize # scripts

Now that we have version 1.0 of extractors, we should clean them up as part of unit testing writing.
Just talked with @weiqin61 and she is going to create pull requests for ply2las, eventually flir and bin2tif to show how we'd like to structure.
Goals:

review deprecated python scripts e.g. @ZongyangLi Get_FLIR.py and move any functions/constants we use into the main terra_flir2tif.py python file
those functions can go outside the Extractor class so that any developers wanting to use parts of the code, e.g. the temperature conversion for FLIR data, can import and use just that function without needing the Clowder pipeline pieces. this also makes unit testing easier.
remove old functions - we have a lot of code that is now standardized in terrautils but we did not remove old versions from extractor scripts like Get_FLIR which could cause confusion for people reading code.

We do not want to lose any functionality or code that is usable, but instead remove redundant code and minimize number of files in each repository / make a clean and consistent structure.

@craig-willis @robkooper fyi. after further discussions with @jterstriep and @dlebauer the current thinking is...

move all non-Clowder components of extractor into a python package named after the sensor.

for example, in the stereo-rgb repo we would have a stereo-rgb python package that is generic from Clowder - it just takes input files or data and does the actual science on the data. we didn't really discuss this but I think it would make sense to include all extractor functions in the package - for example stereo-rgb package would have functions for bin2tif, canopy_cover, height_recovery, texture_analysis, etc. all the sub-extractor folders in the repo.
idea being we have a package that people can use e.g. in workbench to manipulate the files completely cleanly and separately from extractor stuff.
then we modify the extractor script (e.g. terra_bin2tif.py) to import that package and use it. we modify the Dockerfile to install that package. then we have a clear script for all our collaborators to focus on and separate out the "communicating with Clowder as an extractor" piece.

later, we can consider a new "extractors" repo that collects all of the extractor scripts in one place as they will be fairly similar, and the existing extractors-multispectral, stereo, etc. repositories just become the "science module" for that sensor.

we can also add things like R packages to work with the sensors to the "science" repo and keep the Clowder/RabbitMQ components separate.
will go over this on the call today.

I am starting splitting the extractor functions out of the extractor repository and make them into modules now. The first step is splitting 3d scanner functions out of the extractor-3dscanner repo and set up a new repo scanner_3d. The repo is a python package that continues modules and tests.
https://github.com/terraref/scanner_3d
If this refactor works, I will refactor other repos and add more tests and readme.


Write a R program to convert miles to inches

Description
Write a R program to convert miles to inches
1 mile = 63360 inches
Input  : 5
Output : 316800

How to contribute

Save the solution in program/convert_miles_to_inches.r file
Add convert_miles_to_inches.r file in convert-miles-to-inches folder

!assign


Generate usings based on generation options

Currently, we generate no usings at all.
We should look at all the repos and add using for all the namespaces.

See https://github.com/mono/gir-sharp/blob/69814a791775eb70c76469364813f783c6ccab7d/src/Gir/Generation/IndentWriter.cs#L29


Sink error result to metricsServer not working

https://github.com/alibaba/kubeskoop/blob/2e79815afe27463313ca73f44e789ea97414cc19/pkg/exporter/cmd/server.go#L299C1-L314C1
In our case, we use serveral sinks config, like:
...
      sinks:
      - name: kafka
        args:
          addr: ...
          topic: ...
      - name: elasticsearch
        args:
          addr: ...
          index: ...

if something wrong with kafka, when we start exporter, the whole metricServer will stop.

I think it's better to record the err log instead of return. Because the second sink we used is valid, we hope events push to elasticsearch.
		//sink
		sinks, err := createSink(cfg.EventConfig.EventSinks)
		if err != nil {
			log.Errorf("failed create sinks, err: %v", err)
			return
		}

https://github.com/alibaba/kubeskoop/pull/264


Multiplayer - Sub Task "Edit Button" Unclickable.

Singleplayer has no issues for me.
However when I join my friends game (which has the mod and works fine for him). For some reason my subtasks buttons are not aligned with Task buttons (See screenshot).
Hovering the Subtask Details button doesn't highlight it, and clicking has no effect.

Hello,
Thanks for the bug report.
This is really weird as the ui rendering is identical in single and multiplayer. Also as far as I know there is no difference in handling tasks depending on if it is single player or not.
Can you please try out if the bug disappears depending on if the task is assigned or not and if it is a long or short player name?
Thanks.

Hi Jonas,
Prior to opening the issue, I did try Unassigned, Assigned to me, Assigned
to my friend.
My friend's name was NoZaKu while mine was Kaustah (Mine does have a letter
longer but I don't think it's longer than most). Not sure where to change
my name for in-game, i'll try through steam before our next session.
Let me know if there's any other information you might need or if you need
me to join a game for testing.
Thanks,
On Sat, 18 Mar 2023 at 03:14, Jonas @.***> wrote:

Hello,
Thanks for the bug report.
This is really weird as the ui rendering is identical in single and
multiplayer. Also as far as I know there is no difference in handling tasks
depending on if it is single player or not.
Can you please try out if the bug disappears depending on if the task is
assigned or not and if it is a long or short player name?
Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/JonasJurczok/factorio-todo-list/issues/192#issuecomment-1474756704,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKXQT4PUYFQW56XMF3ZMWCTW4VOFLANCNFSM6AAAAAAV7GOPBI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ah nice, you already tried all that.
If you play through Steam changing your username is not that easy, afaik, because it gets pulled from the steam client.
To be honest I have no idea what could produce this effect.
Is the bug there for all subtasks or just for this specific subtask? Or just for subtasks of a specific task?
Best,
Jonas


support BYO resource groups and don't delete them on destroy

This PR addresses #474 ... It adds the ability for using your own resource group in deployments.  Additionally, resource group creation has been moved specifically to the environments rather than within modules themselves.  Modules reference the resource_group as a data item rather than a resource.  This way, the creation or reference of an existing resource group is isolated to the environments.
Documentation updates are required for this (in progress).
The reason for not using the TF prevent_destroy flag is discussed in https://github.com/microsoft/bedrock/issues/474 -- specifically, the RG could still be destroyed based on the following:  https://github.com/hashicorp/terraform/issues/17599
I'll post a comment when the documentation is updated.

Closes https://github.com/microsoft/bedrock/issues/369.

Closing - PR #549 supersedes.


No GUI interface, so in which file do I add the YAML?

RPi 4, Home Assistant 2022.11.2, Supervisor 2022.10.2, Operating System 9.3, Frontend 20221108.0 - latest
I am not seeing a GUI when I click on the integration to create a new ssh command.
In which file do I drop the YAML?
I would like to create a button to execute the command?

configuration.yaml
You can read more in Hass docs.

Works like a charm, thank you.


Add lager and mnesia as dependencies

It is easier then for tools generating releases to find all the dependencies. So if there aren't any reasons against starting mnesia/lager the OTP-way merge this PR.

I think dobby_core_lib should include mnesia as a dependency but not start it.  If dobby_core_lib starts mnesia then another application in the same node as dobby_core_lib cannot start.  We need to find a way to "share" mnesia across applications.  Maybe we can do something similar to erl_sshd?  Have our own application that is a thin wrapper for mnesia to start it and install the tables needed by the other applications?  We'll probably want to do something similar for cowboy.


Incorrect Images in Menu Sections

The images in the menu are not according to the section names.
For instance, the juice section contains images of thali, dessert, and snacks.
I would like to edit the images of each section to ensure they match the section names.
@sunny0625 Sir,  Please assign this issue to me under GSSoC'24.

This issue is already solved.
Try to contribute to different issues.


[designspaceLib]: explicitly require POSIX file paths?

On my Windows machine, I stumbled into this error after I set the instance file paths to WindowsPaths:
[...]
    for instance in designspace.instances:
        instance.filename = Path(
            "..", "build", "instance_ufos", Path(instance.filename).name  # line 336
        )
        instance.path = None
    designspace.updatePaths()
[...]

Traceback (most recent call last):
  File "C:\UsersLocal\nikolaus.waxweiler\Development\...\venv\lib\site-packages\doit\action.py", line 424, in execute
    returned_value = self.py_callable(*self.args, **kwargs)
  File "C:\UsersLocal\nikolaus.waxweiler\Development\...\dodohelper.py", line 336, in glyphs2ufo
    "..", "build", "instance_ufos", Path(instance.filename).name
  File "C:\UsersLocal\nikolaus.waxweiler\Development\...\venv\lib\site-packages\fontTools\designspaceLib\__init__.py", line 71, in setter
    value = posix(value)
  File "C:\UsersLocal\nikolaus.waxweiler\Development\...\venv\lib\site-packages\fontTools\designspaceLib\__init__.py", line 56, in posix
    new_path = posixpath.join(*path.split(os.path.sep))
AttributeError: 'WindowsPath' object has no attribute 'split'

Using .as_posix() on the Path in the above line 336 makes it work on Windows. Should the documentation be updated to mandate POSIX paths when they work cross-platform?

https://github.com/fonttools/fonttools/tree/master/Doc/source/designspaceLib#6-notes

Both filename and path must use forward slashes (/) as path separators, even on Windows.

Whoops, didn't find that one. Maybe a more explicit error message would have helped. Can write something soon.

never mind

Maybe it pays to make it a Path object.


HBASE-26430 Increase log level in DefaultHeapMemoryTuner from DEBUG to INFO

The DefaultHeapMemoryTuner chore will explain what it is doing when deciding to make changes, once per chore interval. This is not frequent logging and deserves to be at INFO level as it is making changes of some import. While making the change, also log a line when ignoring the initial periods, so operators are not surprised when nothing happens for a while.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  7s
Docker mode activated.


By default, should children elements within Locked elements be allowed touch move?

Referring to a comment by @liamcmitchell-sc ,

the API of allowTouchMove requires implementations of allowTouchMove to implement the same available scroll checks that this library is doing. This is a lot to expect of consumers to enable correct scrolling. I think the nicest behaviour for consumers of this library is to provide the most complete solution by default.

wanted to understand what the ratio of yes votes vs no votes for this one?

Letting child elements scroll reintroduces this iOS scroll bug, that body-scroll-lock addresses.
In this screenshot, I am scrolling inside the textarea, and you can see it is causing the bottom of the viewport to be cut off because the body is scrolling:

However, I can't imagine a situation where a developer would want a textarea to be unscrollable.  I say, let's provide a sensible default to allowTouchMove which exempts elements like iframes, textareas, input[type=range] from disableScrollLock.

also it must address any element with overflow: auto or scroll

Should we provide a default allowTouchMove function, or actually hardwire in these exemptions? I can't think of a situation where you wouldn't want e.g a textarea to scroll but perhaps there is. By providing a default allowTouchMove fn, anyone overriding needs to reimplement its contents (or can we expose the default allowTouchMove so it can be reused?)


Implement a flutter roller for flutter/plugins

Currently flutter/plugins uses whatever the latest stable and master are when running tests. This means that:

We are subject to out-of-band breakages due to Flutter changes (engine, framework, tool, and Dart have all broken us in the past).
Re-running tests (e.g., due to flake) can actually change what is being tested, because it's not hermetic. We've had cases where we tried re-running a flaky test, only to have a flutter change break us in the meantime, causing it to be impossible to ever make that test pass for that commit (because fixing such breakage often requires a commit in flutter/plugins). This is especially problematic for our auto-publish, which can only run when all tests are green.

To prevent this, we should use pinned versions of master and stable, with auto-rollers that update them to the latest versions on a regular basis. The master version would be expected to commit more or less all the time, as with the flutter/engine->flutter/flutter roller, while the stable version would usually not need to do anything.
/cc @godofredoc Are there pointers to what's involved in setting up an auto-roller? (The read-from-a-pinned-version for CI is something I could implement pretty easily.)

This is what skia auto-rollers do with the engine->framework and plugins->framework. Sending an email to the skia team to get more details.

This is now implemented for flutter/plugins for master, for Cirrus tests. Still to do:

LUCI
The same thing for stable if we decide it's needed. We have had an out-of-band breakage from a stable release before, but it's much less common. (I'm not sure if this would require extra work in the autoroller itself, or if it can handle arbitrary source branches already.)
If all goes well, duplicating it over to flutter/packages


docs: minor rewording for easier reading.

Motivation for this change
Minor changes to the wording to make the docs easier to read.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thanks!

also backported:
[detached HEAD 3f18caed9ec] docs: minor rewording for easier reading.
Author: Anders Riutta<EMAIL_ADDRESS>Date: Mon Jun 18 16:02:17 2018 -0700
1 file changed, 26 insertions(+), 22 deletions(-)


tender_items_summary.item_classification is null if classification.scheme is null

Similar issue to #105 but for the Classification object, used in tender.items, award.items and contract.items
cc @mrshll1001 @odscrachel

 kf-views bug: not providing an (optional) identifier scheme causes identifier to be blank

Closed by #150


Add separator for PinPut fields and optimize project

Hello! I have implemented adding a field separator to any position via the separatorPositions and separator properties.
The alignment of each field was also sorely lacking, so I added the eachFieldAlignment property. As a bonus, I optimized the project, bringing it closer to Effective Dart.

Feel free to correct. Let me know if my changes are helpful. :)

Thanks for PR I'll take a look at it ASAP.


Suggestion: Angular 2

Angular 2's rebranding is very similar to AngularJS and shouldn't take much time at all.

Updated deeb31add0e18920884128c70ef63a90db026096


linja + space in filename

I was fixing #513, and testing it. Then I found that linja has a problem if a filename/directory-name contains spaces. One of the problem is that its dependency file (.d) uses a different escaping rule than build.ninja file.

In build.ninja, ' ' goes to '$ '
In .d file, ' ' should become '\ '.

The fix breaks on OS X with Python 3.4. The argument to escape_dep is a byte instance, but the arguments to s.replace in that function are strings (str), raising the error
TypeError: 'str' does not support the buffer interface

(Also, the indentation in escape_dep should be 4 spaces, not 2.)


Fast Server reload for a ML app containing deep learning model.

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
None

Description
When we are building a ML app containing some big size models we often face issue with server starting time, as it takes a long time to load the model.
Wanted Solution
I would like to have a cache system which will keep the model in memory in case of stat reload and will server will reference back to it on restart to save loading time of model.
Wanted Code
from fastapi import keep_cache

@keep_cache.on_restart
model = classfier.load("location/")

Alternatives
None
Operating System
Linux
Operating System Details
Currently using Ubuntu18.04
FastAPI Version
0.70.0
Python Version
python3.6
Additional Context
None

As far as I know, the reload mechanism is in the Uvicorn webserver, not FastAPI. FastAPI/Starlette does not come with any integrated web-server, and it is dependent on an ASGI server (example: Uvicorn). So if you want that feature to cache something for restarting, I highly doubt that it's something related to FastAPI; I think it's more related to a specific ASGI web server that you are using.

Thanks @ahnaf-zamil  I am currently exploring ways to solve this problem in FastAPI or (in uvicorn ASGI server). Let me know if you have any suggestion how we can solve it in FastAPI (or Uvicorn ASGI) or is impossible to implement.


[browser-ext] Toolbar icon should change color when the page is already bookmarked

Congrats! :tada:  Very handy application, I love that it is vendor-agnostic so I can access and store bookmarks independent of the browser I'm running. The search and previews are also great!

When the current page is already bookmarked, the toolbar button/icon

should indicate that fact (the same way native web browser bookmark star button does), by displaying another color. So the user have instant feedback about the page.
This feature is 100% equivalent to the following one also requested in Pocket years ago, and not implemented at the time of writing:
https://bugzilla.mozilla.org/show_bug.cgi?id=1402155
Right now when you click on the button, you can see any of these, depending whether the page is already saved or not:

Again, thanks for this very cool project! ^_^ Cheers!

Actually I see this icon in the assets directory: https://github.com/raindropio/app/blob/5424677c308d004220fc078de71bff65d2118251/src/assets/icons/cloud_active.svg
But in my case I never see the cloud filled.
I'm on Brave v1.44.112 (Oct 12th, 2022) Chromium to 106.0.5249.119 (64-bit).
Thanks

Hello!
You need to give additional permission to enable saved page indicator:
https://help.raindrop.io/browser-extension#saved-indicator

Thanks for your prompt reply @exentrich. Now I realize I was missing to grant the permission to the extension, and that the perm is also required to store all tabs, enable highlight (and maybe other additional features?).

But this just brought me a new question:

Should I grant such strong "Read and change all your data on the websites you visit" permission to show the indicator status? I mean, is that perm really required for that feature? I appreciate the Principle of Least Permission, so I questioned myself if that's strictly necessary. For example, to store all tabs, other extensions can do it with reduced permission like "Read your browsing history" e.g. OneTab extension.

I'm not confortable giving extensions access to all browser data, and would prefer to give the minimal necessary to make it work.
Would that be possible? What do you think?
Cheers!


add react 19 to peer deps

This PR:

Adds React 19 to peer reps for the icons package

Please merge

please merge

Please merge it omg

Uhm, is there any issue merging this? 🤔

bro is just AFKOn 24 Oct 2024, at 12:56, codenomnom @.***> wrote:
Uhm, is there any issue merging this? 🤔
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

great work.

Please merge it ?

you can use overrides for a temporary fix.
"overrides": {
	"@radix-ui/react-icons": {
		"react": "$react"
	}
}

This should have been merged already. No issues here.

the owner is AFK he is not working at radix anymore

Merge you tyrant.

is there someone we can contact?

is there someone we can contact?

the other stupid way we can do is by making nonsense PR on all radix repo by saying "merge the radix icons" for them to notice us

please merge! why it take too long

Merge please, so the actual issues can be fixed...

https://github.com/radix-ui/primitives/issues/3195
I went to another library and raised an issue

Please merge

Is this repo still maintained? Shadcn UI compatibility is blocked due to the pending Radix icons merge.

No it’s not

On 26 Oct 2024, at 20:08, Darius Lukasukas @.***> wrote:
Is this repo still maintained? Shadcn UI compatibility is blocked due to the pending Radix icons merge.
—
Reply to this email directly, view it on GitHub https://github.com/radix-ui/icons/pull/184#issuecomment-2439678947, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATANFWLBNTIKPV2MYR4PUOLZ5PLBNAVCNFSM6AAAAABNDCWC2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZZGY3TQOJUG4.
You are receiving this because you commented.

cmon please merge this already

merge this plsss...shadcn not working cause of this

try to put issues on other radix repo so that they will notice us.

pl merge this as it is causing version mismatch in frameworks react 19^

Looks good to me, can we merge?

merge

Please merge it, shadcn is broken because of this.

@chaance  @hadihallak please merge sir

@chaance @hadihallak can you please merge this PR. It's blocking a lot of projects. Thanks,

@chaance @hadihallak  Please merge its blocking

merge it for the love of shadcn

:(

@vladyslavmoroz @colmtuite @peduarte @loklaan  can you please merge this PR. It's blocking a lot of projects. Thanks,

Please merge it man...., i can't init the shadcn ui

I don't understand. Couldn't you just npm install --force while waiting for this?


Add schema version to JSON output trajectory file

Turned the top level entity from an array to an object.  The object has a member named points which is the array of points.  The top level also has a string value _version with a leading underscore which gives the version.  The format is string.


Bit.Butil Console

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Expected Behavior
No response
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
No response
Anything else?

@Hamid-kadkhoda Thanks for contacting us.
but I don't understand what is the problem.

I'm closing this issue since there was no activity in a long time.


Enable strictNullChecks: true in tsconfig

Without setting strictNullChecks: true in your tsconfig.json, null and undefined are allowed values in every type.
For example
let p1: Promise<Response> = fetch('http://example.com')
p1.then(() => { /* will behave as expected */ })

let p2: Promise<Response> = null
p2.then(() => { /* will cause a runtime error because `then` is not a function on `null` */ })

This PR enables this important compiler flag, and attempts to make all necessary adjustments to get the codebase type-checking. A thorough review would be wise here, because while I've tried to avoid perturbing things unnecessarily, there are some places where I've had to guess whether a null or undefined is an intentionally possible value or not.

Dear @mike-north ,
I would like to incorporate this into the v4.0.0 release if it is okay for you. I will take a look this weekend more thoroughly to go through with it. But I do not know if I can finish it this weekend since there are more tasks for me to do before it which can be summarized here. I again prematurely before merging it, humbly thank you for your work on this multiple and huge pull requests that ensure the code base is more solid.

@cenk1cenk2 no worries, take your time :)
I do think that you'll enjoy the ability to trace down erroneous null and undefined things once we get this flag turned on


Different Language Support

It would be a nice idea to materialize supports different languages, i'm trying to use de date picker, but, it only comes in english, so i have to modify de package before installing it... just an idea, thanks...

See pickadate.js documentation for different languages


Front end of a blog website. can i contribute here?

I have already completed the front end part of a blog website. Can i contribute here?

I have already completed the front end part of a blog website. Can i contribute here?

Yesss sure, go ahead. Atb.

Ok. At first i have to clone ,edit and then pull request right? @sreelakshmig009

Ok. At first i have to clone ,edit and then pull request right? @sreelakshmig009

yes abd mention this issue in your pr


openBluetoothSettings appears to be Android only, not iOS only?

The openBluetoothSettings function does nothing for me on iOS, but it opens the Bluetooth settings on Android. Though in the readme, it says iOS only.
I just checked again and I even get the error on iOS: openBluetoothSettings is not available on iOS.

Yes, just a mixup in the readme. It's been updated.


Remove iOS browsers from happo

Describe the task
iOS browsers typically generate the most spurious diffs
You can find the targets in the .happo.js

Thank you @JoshLarouche ! I can see the happo diffs deleted in dev


replace rocprof with rocprofv2 for the tune gemm script

rocprofv2 is much faster than rocprof
rocprofv2 naturally support python
for 8192x8192x8192, it reduce tuning time from more than an hour to 5.22 mins with full tuning space.

Have you checked performance on a few shapes to confirm before and after is the same?

yes, I did @vgokhale
Here is with rocprof
xiaohugu@banff-cyxtera-s79-2:~/work/persistent-kernels$ cat tuning_results_main@d7a28b6_07-15-2024-09\:32\:03.yaml
- {'M': 8192, 'N': 8192, 'K': 8192, 'rowMajorA': 'T', 'rowMajorB': 'N', 'BLOCK_SIZE_M': 256, 'BLOCK_SIZE_N': 128, 'BLOCK_SIZE_K': 64, 'GROUP_SIZE_M': 32, 'num_warps': 8, 'num_stages': 0, 'waves_per_eu': 0, 'matrix_instr_nonkdim': 16, 'kpack': 2} # TFLOPS: 453.75 time(us): 2423.1

and rocprofv2
xiaohugu@banff-cyxtera-s79-2:~/work/persistent-kernels$ cat tuning_results_main@4bb2b95_07-16-2024-22\:03\:33.yaml
- {'M': 8192, 'N': 8192, 'K': 8192, 'rowMajorA': 'T', 'rowMajorB': 'N', 'BLOCK_SIZE_M': 256, 'BLOCK_SIZE_N': 128, 'BLOCK_SIZE_K': 64, 'GROUP_SIZE_M': 32, 'num_warps': 8, 'num_stages': 0, 'waves_per_eu': 0, 'matrix_instr_nonkdim': 16, 'kpack': 2} # TFLOPS: 452.34 time(us): 2430.7


Update index.md

The password is not hashes in the config file. the password is base64 encoded.

Deploy preview ready!
Built with commit 824f06e35cfeddef0f4efea56040b6d1ff780b42
https://deploy-preview-862--docker-docs.netlify.com

Thanks for the correction @burik666. I've gotten clarification from the security team and I'll update this in a moment.


Legacy image fails to start

Command: docker run progrium/consul -server -bootstrap
Keep getting the error Error response from daemon: Cannot start container: [8] System error: no such file or directory
Image ID: bde95fb7344a

I âme running into the same error.

+1

+1

Confirmed as an issue for both consul-0.4 and consul-0.3 as well.

So there's a number of problems here. First, Docker Hub rebuilds all tags when anything happens. So images on Docker Hub are not exactly immutable, otherwise this would Just Work.
The problem seems to be with opkg packages, which as of recently install just fine but then don't work. The entrypoint for this container is a script run with bash, installed from opkg.
Now I have no idea what's going on with opkg or the openwrt package repository. It's never been terribly reliable. However, I should be able to change the legecy branch to use alpine and everything should work as normal. So I'm going to try that unless anybody has updates on how to fix the opkg package issue.

This might be related: https://dev.openwrt.org/ticket/19893

Okay, legacy was patched to use alpine and installs packages that are known to work. This doesn't look good for progrium/busybox, but since alpine is what the new docker-consul images are based on we should be fine going forward. There are some interesting quirks with DNS resolution in the libc Alpine uses, so report any of those issues separately.

Can anybody confirm that it works?

Yes, it seems to be working ok, thank you for the fix!

So far so good for me as well, thanks for the quick resolve!

Looks good - thanks!

Excellent- thanks for the quick response !

Okay, if you run into issues related to musl quirks (keep an eye out for DNS related issues in particular) submit a separate issue.

Thanks for the quick response!


gizmo: accumulate results in ForEach

Other finals seem to work fine.  For example this works:
g.Emit({
"films" : g.V().Has("/film/film/directed_by", director_id).TagArray()
});
But this gets: {"error" : "(anonymous): Line 6:1 Unexpected token } (and 3 more errors)"}
g.Emit({
"films" : g.V().Has("/film/film/directed_by", director_id).ForEach(function(d){
return d;
});
});
... or any other variation of function body (like a nested g.Emit());
You might wonder what I'm trying to do... it's format an entire tree.  g.Emit() seems to be the only way to do optional fields in an object, so I was trying to figure out how to nest that.

There is no error on HEAD for first example.
Will implement accumulation a bit later.

Correction: .forEach() shouldn't accumulate results, .map() should.

Closing this as #878 will not include steps that accept a function.


Disable third nest dropdown when second nest dropdown is set to none.

Test Notes

Change the second nesting dropdown to "None" and verify that the third dropdown is disabled and you are unable to change it.

Works. not tested in IE


Remove AppTP waitlist notification channel

Task/Issue URL: https://app.asana.com/0/488551667048375/1203701902138004/f
Description
Remove the unused AppTP waitlist notification channel
Steps to test this PR
Before

[x] install application from develop
[x] long press on app icon and go to App info -> notifications
[x] "app Tracking Protection waitlist" notification channel should appear

After

[x] install application from develop
[x] long press on app icon and go to App info -> notifications
[x] "app Tracking Protection waitlist" notification channel should NOT appear

Is it expected that when updating from develop -> this branch that the appTP waitlist doesn't disappear? It works as expected when doing a fresh install of this branch.

I thought they stay for ever but it seems they can be remove by calling https://developer.android.com/reference/android/app/NotificationManager#deleteNotificationChannel(java.lang.String)
Will update PR

@karlenDimla, https://github.com/duckduckgo/Android/pull/2708/commits/bb7bbd782d81047f96ea3455c943e75354c41f04 is the best I could do :/


docs(concepts): add notice about webpack-sources

closes https://github.com/webpack/webpack.js.org/issues/3921

Thanks!


FakeData on interface that is a slice

Is possible with faker do something like this:
func BulkCreate(db *gorm.DB, records interface{}) {
	switch reflect.TypeOf(records).Kind() {
	case reflect.Slice:
		slice := reflect.ValueOf(records)
		for i := 0; i < slice.Len(); i++ {
			faker.FakeData(slice.Index(i).Interface())
			fmt.Println(slice.Index(i).Interface())
		}
		db.Create(slice.Interface())
	}
}

I've tried, but this function does not modify the slice.Index(i).Interface() with fake data.
Here a playground.
Any idea to achieve this behavior?

Hi, maybe you can try this.
https://play.golang.org/p/3MW4hrlE7vM

I've changed the faker version, you're using the old one.
I use reflect.Set to set the value back

I'm closing this issues, feel free to re-open if you have questions


update urllib3 to resolve CVE-2019-11324

What Are We Doing Here
Updating urllib3 to a newer version that patches the latest CVE.
How to Verify

All tests still pass. No functionality changes.

Codecov Report

Merging #79 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   83.39%   83.39%
=======================================
  Files          14       14
  Lines         542      542
=======================================
  Hits          452      452
  Misses         73       73
  Partials       17       17

Continue to review full report at Codecov.

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


Use OCIRepository in HelmRelease

Hello, I tried the new OCIRepository resource and it works great with private GCP Artifact Registry (Workload Identity auth) storing OCI Helm Charts.
However, I'm not able to use it with HelmRelease resources:
Unsupported value: "OCIRepository": supported values: "HelmRepository", "GitRepository", "Bucket"
Is this coming, or did I get it completely wrong and OCIRepository is not meant to be used with HelmRelease?
Thanks!

This is somewhat related to this issue https://github.com/fluxcd/source-controller/issues/867
HelmRepository with OCI config doesn't support IAM Auth, so i'm trying to use OCIRepository instead which does, but cannot figure out how to use it with HelmRelease

As per https://github.com/fluxcd/flux2/discussions/3002#discussioncomment-3422192, it seems like OCIRepository can't be used with HelmRelease. However, I was also hoping the auto-login functionality would work with HelmRepository as well.

Good spot @yafanasiev! In this case it's important that HelmRepository with OCI config supports IAM auth, as not all Helm charts are/can be public.

The auto-login does not work for Helm at present because the Helm project makes use of ORAS, which unlike the library we use for OCIRepository objects lacks a component like authn to deal with automagic authentication towards registries.
@souleb is evaluating an adapter component this week to allow ORAS' credential callback to make use of authn. I expect more to be shared about this once he's done with this assessment.


Fix blf build after compiler and manif update and de-vendoring catch2

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #78.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[x] Bump the build number if needed

Fixes #78

I propagate in here the useful changes from https://github.com/conda-forge/bipedal-locomotion-framework-feedstock/pull/76, but without coupling the compiler update with the boost update (once everything works, we can update boost).

The python bindings are going in timeout, the same error is happening in blf's upstream CI: https://github.com/ami-iit/bipedal-locomotion-framework/issues/882 .

The test failure is due to https://github.com/ami-iit/bipedal-locomotion-framework/pull/884, that can be worked around by setting avoiding to use mumps-seq 5.7.3=*_3 or setting OMP_NUM_THREADS=1, so I think we can proceed here.


Create separate 'blinking' and 'scanning' files to verify that each work

to verify that each work independently
to start understanding how the asynchronous functions run on ESP
to test & change blinking function (4)

created files, time to test/refine them


proposal: net/http: add option to limit header count in Server

Proposal
Right now net/http has Server.MaxHeaderBytes to limit request header size. Proposing an additional Server.MaxHeaderCount to limit total number of headers.
Rationale
Granted that Server already has quite a number of tunable fields, none address the issue of DOS attacks that send a boatload of headers. Even when limiting total header size to 64k, an attacker can still send ~13k headers in a single request. This puts visible pressure on the GC as the number of malicious requests scale up.
Implementing this is trivial with existing code.
Any hint of a workaround would be appreciated too.

cc @neild

Any progress?

I have made a draft of what a PR would entail. Care to share your thoughts on this? @seankhliao @neild

Not sure if anyone is going to read this. Since #66298 has been merged is there reason to look at this as well?    @neild

Sorry for the late reply.
What the HTTP receiver really cares about is the amount of bytes of headers, it's also what really matters. The number of headers tells us almost nothing because the amount of bytes is still ambiguous. If you're trying to limit the number of headers, I believe lowering the MaxHeaderBytes can also achieve the same effect. I'm concerned that adding a new configuration like MaxHeaderCount will bring more complexity to the net/http and confuse users, given that we already have an adequate solution: MaxHeaderBytes.

Tks for the response. As I've first described, the issue here is that a malicious request can have a lot of small header key-values and still fall within a reasonable MaxHeaderBytes limit. Since an incoming request header is always parsed into a map, the server will be making a lot of allocs for that request. The current code also puts a cap on the estimated map capacity, making the situation worse.
Amplifying the number of these malicious requests can put significant pressure on the gc, causing performance issues.
Reducing MaxHeaderBytes can help, but may not be feasible without affecting legitimate requests.
Even if we don't give the option to specify number of headers, a hardcoded a limit or limit configurable via env can work too.


tf.summary.create_file_writer aborts

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):2.1.0
Python version:3.7.6
Bazel version (if compiling from source):N/A
GCC/Compiler version (if compiling from source):N/A
CUDA/cuDNN version:N/A
GPU model and memory:N/A

Describe the current behavior
tf.summary.create_file_writer crash (abort)
Describe the expected behavior
expect an exception message if the input unexpected instead of crash.
Standalone code to reproduce the issue
import tensorflow as tf
import numpy as np
tf.summary.create_file_writer(logdir='', flush_millis=np.ones((1,2)))

Output:
2021-02-04 03:59:32.339427: F tensorflow/core/framework/tensor.cc:669] Check failed: 1 == NumElements() (1 vs. 2)Must have a one element tensor
Aborted (core dumped)

I have tried in colab with TF versions 2.1,2.4,nightly versions(2.5.0-dev20210203) and was able to reproduce the issue.Please, find the gist here. Thanks!

Colab is still crashing in TF 2.6 when I executed the code. Please find the gist here.Thanks!


EasyAuthServer with mysql backend setup failing

Summary:
EasyAuthServer with mysql backend setup failing
Steps to reproduce:
1 - Create MYSQL database:
# env="-e MYSQL_ROOT_PASSWORD=abcd1234 -e MYSQL_USER=josh -e MYSQL_PASSWORD=abcd1234 -e MYSQL_DATABASE=joshdb"

#  start mysql database
# mkdir -p mysql-vol
# docker run --name mysql -v $(pwd)/mysql-vol:/var/lib/mysql $env -p 3306:3306 -d mysql

# configure EasyAuthServer to use mysql
# cat > auth_server_config.json << EOF
{
    "DB_TYPE": "mysql",
    "DB_NAME": "auth",
    "DB_HOST": "<IP_ADDRESS>",
    "DB_PORT": "3306",
    "DB_USER": "codemation",
    "DB_PASSWORD": "abcd1234",
    "ISSUER": "EasyAuth",
    "SUBJECT": "EasyAuth",
    "AUDIENCE": "EasyAuthApis",
    "KEY_PATH": "/home/codemation/python/project/tests",
    "KEY_NAME": "test_key"
}
EOF

Start EasyAuthServer

from fastapi import FastAPI

from easyauth.server import EasyAuthServer

server = FastAPI()

@server.on_event('startup')
async def startup():
    server.auth = await EasyAuthServer.create(
        server,
        '/auth/token',
        env_from_file='auth_server_config.json'
    )

Errors /  Traceback
04-20 12:10 aiopyql-db-auth DEBUG    auth - execute: CREATE TABLE users (username VARCHAR(36) PRIMARY KEY UNIQUE NOT NULL, full_name TEXT , email TEXT , account_type TEXT , password TEXT , groups TEXT )
04-20 12:10 aiopyql-db-auth DEBUG    completed query: aabaa7a9-a1c0-11eb-ac04-ed4c20b9a885
04-20 12:10 aiopyql-db-auth ERROR    error during create_table - ProgrammingError(1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'groups TEXT )' at line 1")

Created tables contain columns with name reserved by mysql -
user table column - 'groups'
table named - 'groups'
https://dev.mysql.com/doc/refman/8.0/en/keywords.html
column names & table name need to be adjusted to avoid reserved keyword usage:
column: groups -> user_groups
table: groups ->  user_groups
Need to rename table & column names in affected files, monitor migration of sqlite tables, check and test postgres deployments.

Fix for issue added in https://pypi.org/project/easy-auth/0.128/ - @tisboyo - let me know what you think.

Fix for issue added in https://pypi.org/project/easy-auth/0.128/ - @tisboyo - let me know what you think.

Updated both libraries, and it worked perfectly with zero errors. Working great.


Issue - 74 Clashes autoimport

@dimacodota , could you please take a look?

@dimacodota , I think i found a better solution. Hold on for the next PR.


Mini Player Standalone

Could you also provide the option to show a standalone "mini player" like in this screenshot? =>

Looks good and minimal, especially on a "clean"  desktop 😆 .
I think only the cover and hover over for controls is very nice, but should be a option (cover only and controls when hovered over/ cover and controls showed)

This is what I have so far

Looks awesome 👏!

how about this?

This looks great!! 🎉🎉

I like the option to move the MiniPlayer from your earlier post. Do you consider to add this ?
The new design is very nice btw

yep, I will add it but it’s not ready yet. I haven’t published an update in a while so I figured I should release what I had currently that works.
Glad you like the new design

Totally understand that :) And I am very happy with the new look. Only wish that the cover art of the notification center widget had also rounded edges just like the menu bar widget. :)

Yeah, do you have a early working version yet ? I’m excited 😊

Any news on this? I would love to be able to "tear off" the artwork and have it float over my other apps the whole time.


the debugging session could not be started: FAILURE HR=0x080070081: Failed to CreateProcessAndAttachWide "NTOSKRNL.exe"

Hello,got this error on 2 PCs - Laptop and computer :
Laptop - W11 23H2 SECURE BOOT ON
Computer - W11 24H2 LTSC SECURE BOOT OFF + bcdedit debug on
got this error on NTOSKRNL.exe of W10 22H2 ( trying to debug on w11),23h2 and 24h2 ltsc.
Only happens on kernel,debug notepad.exe work fine
Run as admin / UAC off , tried running x64 didnt fix,what can I do ?

I get the same one each time I try to debug, except my error code is 0x800700C1. I've been looking on the Internet trying to find a solution for a few days now, but on every forum, no one has the same issue as me, and on every "helpful" website, they're all about Windows updates.

I found the reason for it not working: its path had a space in it. It's even in the screenshot I put! It is a little disappointing that the app isn't able to deal with these kinds of cases itself, but oh well, what're you gonna do


Design ZITADEL Customer Portal

The customer portal is a separat application from ZITADEL. Our customers are able to create a login for their organization and manage all their instances in there, aswell as their invoices.
Acceptance criteria
Instances

[ ] List all instances of a tenant
[ ] Create a new Instance
Required Attributes:
- System/Org Name
-  Admin User (E-Mail, First-, Lastname, Username)
- Default Subscription: Free
Optional:
- Subscription: Pay as you go
- Option: Custom Domain (custom domain needs to be verified after creation (acme challenge))
- Option: Data locality region
- Option: Extended SLA
- Payment Method: Billing Data including Billing contact (important to know the country (mwst))
- Payment: Discount coud
[ ] Instance Detail
- Usage
- Metrics
- Info
[ ] Change Instance:
- System name
- Subscription: Free - Pay as you go
- Option: custom domain
- Option: Data locality region
- Payment Method: Billing Data including Billing contact (important to know the country (mwst))
- Payment: Discount coud
[ ] Verify domain
- a domain has to be verified before you can use it for your instance
- e.g acme challenge (analog org domain verification in console)

Users and Role Management

[ ] add a new user
[ ] change role of a user

Invoices

[ ] change payment method
[ ] change payment contact details
[ ] list invoices
[ ] download invoice

Organization Detail

[ ] show detail (creation date, name, etc)
[ ] delete organisation (includes all instances, maybe with grace period)

Profile Page

[ ] show profile data
[ ] redirect to zitadel for edit?
[ ] authentication methods should somehow be able to configure (console, or directly in profile page)
[ ] Change newsletter / Mailinglist subscriptions

Support Requests

[ ] Create support request --> create issue in private github repo
[ ] If possible show created requests

Next Ideas:

[ ] Referral code possibility
- The customers should be able to generate a referral code, when the code is used by another customer they will get

estimated: ?? @juergrinaldi

I tried to combine the inputs and mockups from https://whimsical.com/customer-portal-CEBjdiSms4PhvYEEiGhkuo
The screendesigns below are additional to the wire frame mockups from above (whimsical).
Instance Overview:

New Instance:

Edit Instance:

I replaced the 'new instance' and 'edit instance' screenshots above regarding some comments of @fgerschwiler about 'general information' and domain verification.

Onboarding process:

Here are some inputs regarding design elements. for further reference and things like hover states etc. see the style sheets from our console UI or ask @peintnermax. he will gladly help you out.  for access to the console files, please contact @fgerschwiler (you might be asked to give us your IP for whitelisting).
The branch for the new console design is 'cnsl-rhl-nav' or on https://console-alpha.netlify.app/
basic design is in https://github.com/caos/zitadel/blob/cnsl-rhl-nav/console/src/styles/input.scss
Here are some of the colors and typography visualized and tagged:

Colors:
#BBBAFA
#FF2069
#41CABA
#FFFFFF
#8795A1
#CF4C3D
#3BC37D

#0F1522
#111827
#1A253C
#283E67
#293243
#403E3E

Hey @juergrinaldi
Is there a similar picture for the "light theme"?

generated from dark primary (#bbbafa) and light primary (#5469d4)
--theme-dark-primary-50: #ffffff;
--theme-dark-primary-contrast-50: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-100: #ffffff;
--theme-dark-primary-contrast-100: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-200: #ffffff;
--theme-dark-primary-contrast-200: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-300: #f3f3fe;
--theme-dark-primary-contrast-300: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-400: #d7d7fc;
--theme-dark-primary-contrast-400: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-500: #bbbafa;
--theme-dark-primary-contrast-500: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-600: #9f9df8;
--theme-dark-primary-contrast-600: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-700: #8381f6;
--theme-dark-primary-contrast-700: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-800: #6764f4;
--theme-dark-primary-contrast-800: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-900: #4b48f2;
--theme-dark-primary-contrast-900: #ffffff;
--theme-dark-primary-A100: #ffffff;
--theme-dark-primary-contrast-A100: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-A200: #ffffff;
--theme-dark-primary-contrast-A200: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-A400: #e8e8ff;
--theme-dark-primary-contrast-A400: hsla(0, 0%, 0%, 0.87);
--theme-dark-primary-A700: #d1d1fd;
--theme-dark-primary-contrast-A700: hsla(0, 0%, 0%, 0.87);
--theme-light-primary-50: #ffffff;
--theme-light-primary-contrast-50: hsla(0, 0%, 0%, 0.87);
--theme-light-primary-100: #ebedfa;
--theme-light-primary-contrast-100: hsla(0, 0%, 0%, 0.87);
--theme-light-primary-200: #bec6ef;
--theme-light-primary-contrast-200: hsla(0, 0%, 0%, 0.87);
--theme-light-primary-300: #8594e0;
--theme-light-primary-contrast-300: hsla(0, 0%, 0%, 0.87);
--theme-light-primary-400: #6c7eda;
--theme-light-primary-contrast-400: hsla(0, 0%, 0%, 0.87);
--theme-light-primary-500: #5469d4;
--theme-light-primary-contrast-500: #ffffff;
--theme-light-primary-600: #3c54ce;
--theme-light-primary-contrast-600: #ffffff;
--theme-light-primary-700: #2f46bc;
--theme-light-primary-contrast-700: #ffffff;
--theme-light-primary-800: #293da3;
--theme-light-primary-contrast-800: #ffffff;
--theme-light-primary-900: #23348b;
--theme-light-primary-contrast-900: #ffffff;
--theme-light-primary-A100: #ffffff;
--theme-light-primary-contrast-A100: hsla(0, 0%, 0%, 0.87);


Developer docs explanation on nanoApp field is incorrrect/misleading

Recently I encountered an issue with getting a Live App running in your production enviroment due to my misuse of the nanoApp field in my manifest file. The docs state the following

nanoApp is the plugin needed to clear sign your DApp; dappName should be the same as nanoApp"

But from my understanding this isnt always true. In my case I needed the values to be the following:
params: {
  nanoApp: "Ethereum",
  dappName: "{MY_APP_NAME}",
  ...
}

Would be great to add better explations on the params field(s) here. Also the defintions for params being smashed into a single box is easy to miss and hard to digest.

To pile on another semi-related question here. Is it possible/needed to pass multiple chains to nanoApp?  My Live App in question will require users to sign messages on multiple chains.
For example, I have a Polygon account selected and sometimes when I request a signature from user, it prompts them to open the Ethereum app to do the signing. When this occurs, the signing message fails. I understand both of these are EVM based chains but just trying to gather more information on what nanoApp really is used for.
Another maybe more clear example/question, I need signing on BTC and ETH. what should the nanoApp field value be?

Spoke with Adrian and he explained the nanoApp field to me and what we need to do to create a C plugin for our dapp.
Still think the doc site around all this could be made more clear though.
Marking this as closed.


Regenerate docs

/lgtm

@skrthomas last commit should address your comments; except for what is in the upstream k8s doc

/lgtm thank you, @jotak


[New Core]:

Core Author username
rolandking
Core .yml snippet
cores:
      display_name: Athena SNK 1986
      repository: openFPGA-Athena

I had a bad name for the platform image file so the automated script failed - that's fixed now so can the checks be re-run please.

@rolandking Thank you for your submission.
After re-running the pipeline and reviewing the logs, it still errored out. It would appear that the Platform folder was created with the incorrect casing (platform instead of Platform).
Once the folder is renamed, it should be fine.

Thanks - I just fixed this and re-uploaded as v1.1.1. I think the directory is “Platforms” (plural). I found my original template had this wrong, not sure why

On 27 Apr 2024, at 18:40, Joshua Campbell @.***> wrote:
@rolandking https://github.com/rolandking Thank you for your submission.
After re-running the pipeline and reviewing the logs, it still errored out. It would appear that the Platform folder was created with the incorrect casing (platform instead of Platform).
Once the folder is renamed, it should be fine.
—
Reply to this email directly, view it on GitHub https://github.com/openfpga-cores-inventory/analogue-pocket/issues/660#issuecomment-2080447464, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJKQPYSW2U3HA3OPYHVE6TY7N6ADAVCNFSM6AAAAABG3ZVNQOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBQGQ2DONBWGQ.
You are receiving this because you were mentioned.


How do you measure how long the person has been on the page?

First of all, fantastic work on the library. I am looking to measure 2 things

How long the person has been on the page regardless of whether they interacted or not?
How long the person has interacted on the page?
Going through your issues I found that mentioned if you idleTimeout to -1 it will stop tracking immediately the moment person goes idle
How do I not have an idle timeout at all meaning even if the person is just reading the page and not doing anything indefinitely, the timer should keep running

Suggestions are super appreciated

UPDATE I tried something but this has a drawback
TimeMe.initialize({
            currentPageName: "my-home-page", // current page
            // idleTimeoutInSeconds: 5, // stop recording time due to inactivity
            websocketOptions: { // optional
                websocketHost: "ws://your_host:your_port",
                appId: "insert-your-made-up-app-id"
            }
        });

I commented out idleTimeoutInSeconds but this has a drawback
If I try to type something on the address bar, the timer stops running but unless I click on the page again it does not start running

Sorry my bad, modified your code slightly and does the trick

        var start = Date.now();

        window.onload = function () {
            TimeMe.trackTimeOnElement('area-of-interest-1');
            TimeMe.trackTimeOnElement('area-of-interest-2');
            setInterval(function () {
                var timeSpentOnPage = TimeMe.getTimeOnCurrentPageInSeconds();
                document.getElementById('timeInSeconds').textContent = timeSpentOnPage.toFixed(2);

                var timeSpentOnElement = TimeMe.getTimeOnElementInSeconds('area-of-interest-1');
                document.getElementById('area-of-interest-time-1').textContent = timeSpentOnElement.toFixed(2);

                var timeSpentOnElement = TimeMe.getTimeOnElementInSeconds('area-of-interest-2');
                document.getElementById('area-of-interest-time-2').textContent = timeSpentOnElement.toFixed(2);

                var totalTime = (Date.now() - start) / 1000;
                document.getElementById("totalTime").textContent = totalTime.toFixed(2)
            }, 25);


Fail when unexpected fields are in runtime config

Unexpected fields are simple ignored and the problem could go unnoticed.

This behaviour was explicitly added to support config files that included daemon configs/ possibly have comments in the config file.
We should separate user configs for the daemon from genesis config and have genesis config file be strictly checked for unexpected fields


Method to clear localstorage

We need a simpel way of cleaning the localstorage.
At the moment is the localstorage saved, even after the App has been shotdown.
It will be nice to have a quick and easy way of clearing the localstorage, that the developing app has saved.

+1

+1

+1

I'm wondering, since phonegap opens a browser window in the background, If i erase my android chrome browsing data, is my phonegap data gone?


Update the ghost cars to use unique vehicles

I kind of want to randomize each ghost car model, but I want some level of consistancy.
There is a way to create a seemingly "random" sequence of events that is always the same based on the input.
A common way to do this is to create a seeded pseudorandom number generator.
Here is a code example of how this could be implemented

class SeededRNG {
      constructor(seed) {
          this.seed = seed;
      }
      nextFloat() {
          // LCG using parameters from Numerical Recipes
          this.seed = (this.seed * 1664525 +<PHONE_NUMBER>) %<PHONE_NUMBER>;
          return this.seed /<PHONE_NUMBER>;
      }
  }

  function generateSeededArray(seed, length) {
      const rng = new SeededRNG(seed);
      const result = [];
      for (let i = 0; i < length; i++) {
          result.push(Math.floor(rng.nextFloat() * 10)); // generate numbers from 0 to 9
      }
      return result;
  }

What I would want to do Is use the map index as the seed, then generate a consistent "random" list of 5 numbers for each map which will be the vehicleID of the car model to use for each ghost.

Resolved in 3c5b3f908fab5f7b2dc3aa710a018ace66876715


Support for SWIFT finetuned models

Thanks for your work!
Anyone know If LMDeploy supports models fine-tuned via SWIFT? For example, can I use LMDeploy to run inference after following this script  that fine tunes internlm-xcomposer2?

It's unclear whether SWIFT has made modifications to the model. If the structure of the model and the directory structure of the ckpt are the same as before training, inference can be carried out using lmdeploy.

@babla9 could you provide a fine tuned internlm-xcomposer2 model?

I used swift's lora fine-tuning method to fine-adjust internlm-xcomposer2 and merged it with the merge parameter method provided by swift. However, reasoning with lmdeploy failed. The error message was that the model name could not be found. I don't know why this is, but it can be solved by replacing the weight files in the model files downloaded on hugging face with the merged model weight files.

Could you provide the reproducing code?


Show cvss_version as enum in API schema for CVSS

This PR:

changes cvss_version for FlawCVSS and AffectCVSS on API schema from string to enum

Schema changes should be OK as again this is just a description change, cvss_version is in fact an enum and we just shown it as a string until now.


Latin locales

Latin countries (usually) capitalizes month names, and weekday names, much like in english

I checked CLDR data, and it uses lowercase for all of those for format purposes. Standalone month names might be capitalized, but we don't support standalone vs format versions in moment, so this is not going in.

Could you send me the link where you got that information?


Log in - Fast typing of login magic code omits digits

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to login page
Type an email and continue
Type the magic code quickly

Expected Result:
All typed digits should be captured
Actual Result:
Some digits are omitted (skipped).
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.3.57-5
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/115492554/1e1b2ead-d1bd-4188-9fbb-6edcbb535d67
https://github.com/Expensify/App/assets/115492554/079896c5-677d-409a-b229-2cc90b1a71be
Expensify/Expensify Issue URL:
Issue reported by: @tymatein
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1692185514653689
View all open jobs on GitHub

Reproducible on Android 13, Pixel 6.

What's the status on this bug? This seems kinda important given that every new user that signs up on Android will have an awful first experience trying to enter their magic code. I tried it just now and it is almost unusable.

@Swor71 Thanks can you get started on this right away?

Contributor assigned, I'll check in on Monday to bump if needed.

hey, yes, I'm currently looking into this issue

Thanks @Swor71!

Geeeeze Melvin, cool your jets!
Thanks for the update @Swor71, we'll keep an eye out for more from you Monday.

hey, I've had some issues reproducing this behaviour on an emulator but when testing on a real device (Galaxy S8, Android 9) it is rather obvious how it can be a problem, even though most people probably wouldn't type in the code that fast.
I wanted to ask, if this has been in such a state for a while or has it started recently, maybe after some refactor or pr merge in general? The reason I am asking is that, from what I could gather, nothing in particular stands out as an obvious problem, other than rather large amount of processing of the input that takes place when typing.

Just based on my own experience, I feel like this has been an issue for some time (aka not recent), but either it's been hard to repro or it's felt like a problem that most people wouldn't run into because entering the code that fast is uncommon.
Ideally, we'd want to improve the experience here so that no one is running into it, even if it would be rare to do so.
As someone not as familiar with the coding side of things, this doesn't really sound like an easy fix or at least one that would require improvements in other areas of the app to allow for faster processing, is that correct? Is that feasible via this GH or would that be more like a feature request to make a wider improvement to something?
What would you recommend @Swor71 @aimane-chnaif?

Thank you for your insight @strepanier03 and yes, you got that absolutely right - it does not seem like an easy bug fix and most likely would require a further, more holistic, look as you've said.
To my mind the amount of processing is extensive but on the other hand that probably is the requirement. At the same time I cannot guarantee that any slight refactor, simplifying the processing would solve the problem entirely but that might be a good place to start.
I might be missing some context on this issue and the requirements for that part of the app in general so suggesting any particular course of action at the moment wouldn't make much sense without the input and/or other opinions, which I'd love to hear.

Thank you @Swor71 - I'll raise this in Slack for wider discussion and link back here.

Raised here.

I debugged it lightly when reporting the error. I did not find the handlers computationally intensive. But on Android, the entire page is re-rendered many times (~4 times) with each keystroke. Probably when focusing next input the soft-key is hidden and redisplayed, which causes the screen to resize and thus the whole page is redrawn.

I apologize for the inconvenience, but I declined the offer https://github.com/Expensify/App/issues/26090#issuecomment-1703154424 because it included a bug report reward of $50 instead of $250 under the old terms. If I'm not mistaken, I'd be glad if you'd look into it. Many thanks.

@tymatein - I apply the additional $200 as a bonus when paying out. The automation creates a job at $50 now but I know to look at the bug report date to determine the final payment.

Commented here. Where it was raised again.

In the thread, https://github.com/Expensify/App/pull/24012 was linked as a possible resolution but this is still happening to me if I type really fast.

Aaah, that was reverted so we're holding until it is deployed again so we can test again.

Still held on https://github.com/Expensify/App/pull/28711. Will check again.

Thank you @strepanier03 for the https://github.com/Expensify/App/issues/26090#issuecomment-1728618560 . I actually had turned down an offer on Upwork. If that's not a problem, I would need a new offer for bug reporting that I could accept. Sorry for the inconvenience.

Sure thing @tymatein - I'll send that over when the PR is merged. This has already been on hold a long time and I don't want to make another job that is going to expire if we don't finish this soon.
I'm following along and once this is off hold I'll make the job and hire you for it.


Url encoding

Description
Added url-encoding for file-upload feature

this PR is no longer needed since I have added the changes for this in #159


tokenize_shuffle.py, re-adding accidentally removed line

also make small change to ensure AWS_SESSION_TOKEN is optional

Do we still need this @Vaishaal @jeffreywpli ?

no
On Fri, Dec 1, 2023 at 7:57 AM Achal Dave @.***> wrote:

Do we still need this @Vaishaal https://github.com/Vaishaal @jeffreywpli
https://github.com/jeffreywpli?
—
Reply to this email directly, view it on GitHub
https://github.com/mlfoundations/open_lm/pull/84#issuecomment-1836352736,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKMJEDFNYFACXAJFPQKS7LYHH46PAVCNFSM6AAAAAA7HMMF42VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZWGM2TENZTGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add basic support for ui_test-style compiletest directives

Fixes #2.
Fixes #1.

Let's just merge it, it's still not fully tested or complete but no need to block on that. Incremental improvements should come in form of follow-up commits & PRs.


Release Version 2.1.7

Update README.md (#3094)
Fixed apps page when user does not have access to ClientApplication (#3095)
Fixes #3081 - always show 'Add' button in filter editor (#3093)
Fixes #3088 - undefined Reference.reference in graphql (#3091)
Fixes #3083 - use ETag weak validator prefix (#3092)
Agent push update (#3085)
Add documentation for advanced search parameters (#3030)
Added docs on update-bucket-policies tool (#3089)
New migration generator proof-of-concept (#3015)
Fixes #3071 - useSearch hooks (#3073)
Clarify authentication Docs (#3074)

coverage: 93.91% (-0.005%) from 93.915% when pulling 152f5e2ee517e220b604e5d2e848228af3fd77be on version-2.1.7 into 5a8380e85d76852a355a7c3057473cd25619ba8d on main.


Standardization

Added CONTRIBUTING.md, standardized README.md, improved "Standard JS" conformance, bumped package versions, removed spammy notifications and removed unimplemented tests. Fixes #5 .

Added the mocha tests back. However, a new error shows up complaining there are no tests to perform...

Sorry, I didn't see that tests were still missing. I'll merge your changes anyway and someone can add tests later.


[Commands] Remove unused link against swift libraries

It doesn't look like lldbCommands actually uses swiftAST or swiftIDE.
cc @compnerd @dcci @JDevlieghere

@swift-ci please test

Great, thanks Alex!


Added getDinnerWith50PercentOff() Method

Description
This pull request adds a new method called getDinnerWith50PercentOff() to the EazyDiner class. This method allows users to retrieve a list of dinner restaurants in the given location that offer a 50% discount and their details.
Resolves: [#501]
Changes Made
-Added the getDinnerWith50PercentOff() method to the EazyDiner class.
-Implemented the logic to fetch the dinner restaurants with a 50% discount from the EazyDiner website.
-Extracted the relevant details of each restaurant, including the name, location, rating, cuisine, and price for two people.
-Formatted the data into a JSON string and returned it as the response.
Usage Example:
delhi_ncr = EazyDiner("Delhi NCR")
result = delhi_ncr.getDinnerWith50PercentOff()
print(result)

Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

[x] Have you adhered to the repository's defined coding convention rules?
[ ] Have you updated the 'documentation.md' file with the method/function documentation?
[x] Have you sent a message along with the result or response?
[x] Have you used the try-catch technique?
[ ] Has the method/class been added to the documentation (md file)?

Screenshots

Additional Notes/Comments

Any additional remarks or suggestions concerning this pull request are welcome.

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

Bro @nikhil25803 ,i have checked for mumbai,bengaluru,jaipur its working check and you can check this<EMAIL_ADDRESS>


Optional

Hi everyone,
I was wondering if there is a vertx equivalent for java.util.Optional which could be used over service proxy, json communication.
For example a service could provide that sort of mapping:

fetchOne(String sql, JsonArray params, Handler<AsyncResult<Optional>> resultHandler)
fetchOne(String sql, JsonArray params, Handler<AsyncResult<Optional>> resultHandler)

so we can have clean and maybe avoid easierly null pointer exception in the handler
fetchOne(..., ..., hr -> {
if(hr.succeeded()){
if(hr.result().isPresent())
// succeed and found
else
// succeed but not found
} else
// failed to query maybe retry ?
}
})
as a side note I was wondering why @DataObject was an annotation and not an interface.
It would then be easy create the vertx Optional.

I was thinking about something quite similar too: some way to decompose the handler into one handler for success and one for error (a bit similar to the runCatching of kotlin). This is quite a common pattern. So that when the success handler is called we're guaranteed to have a result.
But I don't know, if it adds yet another way of doing the things, that might be counter-productive for a clean API? I'm not sure.

that exist if you use the RxJava2 version of the API that will return Maybe instead of Single . We are not planning adding a similar type in Vert.x

e.g https://vertx.io/docs/apidocs/io/vertx/reactivex/ext/mongo/MongoClient.html#rxFindOne-java.lang.String-io.vertx.core.json.JsonObject-io.vertx.core.json.JsonObject-

Then do you think it is possible to have an interface equivalent to @DataObject so it's easy to implement independantly an Optional class ?

I don't understand what you mean @bailly-idmgroup-com , can you give an example ?

Sure
Currently I should be able to do something like it
`package com.idmgroup.ao.web.utils;
import io.vertx.codegen.annotations.DataObject;
import io.vertx.core.json.JsonObject;
@DataObject(generateConverter = true)
public class Optional {
private JsonObject value = null;

public Optional() {
    super();
}

public Optional(JsonObject json) {
    OptionalConverter.fromJson(json, this);
}

public JsonObject toJson() {
    JsonObject json = new JsonObject();
    OptionalConverter.toJson(this, json);
    return json;
}

public JsonObject getValue() {
    return value;
}

public void setValue(JsonObject value) {
    this.value = value;
}

public boolean isPresent() {
    return this.value != null;
}

public static Optional of(JsonObject value) {
    Optional optional = new Optional();
    optional.value = value;
    return optional;
}

public static Optional empty() {
    return new Optional();
}

}
`
but with a DataObject interface, I could do something like that
`public class Optional implements DataObject {
private D value = null;

public Optional() {
    super();
}

public Optional(JsonObject json) {
    OptionalConverter.fromJson(json, this);
}

public JsonObject toJson() {
    JsonObject json = new JsonObject();
    OptionalConverter.toJson(this, json);
    return json;
}

public D getValue() {
    return value;
}

public void setValue(D value) {
    this.value = value;
}

public boolean isPresent() {
    return this.value != null;
}

public static <D extends DataObject> Optional<D> of(D value) {
    Optional<D> optional = new Optional<>();
    optional.value = value;
    return optional;
}

public static <D extends DataObject> Optional<D> empty() {
    return new Optional<D>();
}

}`
I hope it's clearer

what would the DataObject interface do ?

hum there is something wrong with the code tag the second class header should be :
public class Optional implements DataObject


FastQA Throws InvalidArgumentError on ConcatOp

Hi
Happy to try and provide more details/data as required. The minimal details of the problem are that while training a FastQA model using Tensorflow with latest code as of 24th August. With some mixes of data we are getting the following error (stack trace below). We are using TensorFlow 1.10.0 due to the machine build, so it maybe some incompatibility.
We can run on other similar datasets without any problem. I've looked for empty inputs but nothing obvious has jumped out. We are inputting quite short questions with supporting content of about 2000 characters long. We have seen the error with much longer content. I wasn't sure if it was some mix of the words in the question or answer and the vocabulary (using Glove 6B word vecs) causing an empty tensor. It looks like it not so straightforward and its in the internal graph computation but i'd like to get some assistance or hear if anyone else has experienced anything similar.
I was able to reduce one example to two input records which i could put through a newly initialised model and using the prediction call not training which would cause the same problem. Yet if you pass the records in individually both would return a prediction without any error.
Thanks
Tony
Traceback (most recent call last):
File ".../lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1278, in _do_call
return fn(*args)
File ".../lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1263, in _run_fn
options, feed_dict, fetch_list, target_list, run_metadata)
File ".../lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1350, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.InvalidArgumentError: ConcatOp : Expected concatenating dimensions in the range [0, 0), but got 0
[[Node: jtreader/fast_qa/cond_1/segment_top_k/concat = ConcatV2[N=2, T=DT_INT64, Tidx=DT_INT32, _device="/job:localhost/replica:0/task:0/device:CPU:0"](jtreader/fast_qa/cond_1/segment_top_k/Squeeze, jtreader/fast_qa/cond_1/segment_top_k/sub_1, jtreader/fast_qa/cond_2/GatherV2/axis)]]

Hi,
I think I know what the bug is but cannot fix it right now. If my suspicions are correct, for a quick fix you should make sure that the number of examples in the dataset should not leave rest 2 when dividing by the batch-size. So either you change the batchsize or change the size of the dataset slightly (e.g., add another example or remove one). Please let me know if this works.

Hi
Thanks for that hint. I had a whole series of runs over various data splits and i can confirm that all the ones which failed had a modulo of 2 on the test set portion. It seems its ok to have a mod 2 in the training portion.
Thanks
Tony


JSHint => ESLint

With babel-eslint, developers can use the full spectrum of Babel features. The maintainer of Babel contributes to that repo, so it's safe to rely on it to stay up-to-date, too. At the moment, it's an open issue on JSHint to add support for experimental features.
Although I don't necessarily think that it's a good idea that someone use experimental features in a production app, I also don't think that one's tools should decide what one can and can't use.
If you want to stick with JSHint, then it might be worthwhile to add a note somewhere that says that this boilerplate only supports the subset of Babel features supported by JSHint.
I <3<3<3 JSHint, but for the Babel boilerplate I switched to ESLint 'cuz people were wanting to use unsupported things.

I agree, if we stick with the Babel-ES6 way of doing things, ESLint would be a good idea. Although I'm debating the inclusion of Babel/ES6. I'll open an issue once I have time to think through my thoughts more.

Not going to implement this myself. Closing this issue.


Generate correct path for different slash variations

@dstepanov you've mentioned that this PR is not yet ready due to some issues with the root entry? Should we close it for now?

I have come up with a different solution to sanitize dependency path, prefix and root in the initialization, unfortunately it changes some default behavior, please check:
https://github.com/spring-cloud/spring-cloud-zookeeper/compare/master...dstepanov:fix-slash-v2?expand=1

WDYT @marcingrzejszczak @spencergibb?

Will provide a new PR with presented solution


Add proto file for the boot inventry and change inventory manager to load conig from a config file

This pull addresses https://github.com/openconfig/bootz/issues/16. I have not modified the server significantly, and leave the change to the next pull after agreeing on the config format. Also (1) gnsi config (2) service  trust field , and (3) vendor based config will be addressed in separates prs.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

125 of 161 (77.64%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-11.2%) to 81.048%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

server/entitymanager/entitymanager.go
125
161
77.64%

Files with Coverage Reduction
New Missed Lines
%

server/entitymanager/entitymanager.go
1
81.05%

Totals

Change from base Build<PHONE_NUMBER>:
-11.2%

Covered Lines:
201

Relevant Lines:
248

💛  - Coveralls


Drinking chocolate doesn't give you saturation as you are supposed to receive 1 sat

https://github.com/eerussianguy/firmalife/blob/8320059ca174372c50810d39579838af0be182fd/src/main/resources/data/firmalife/tfc/drinkables/chocolate.json#L1-L10

This should apply when u eat your next meal, like milk. Can you check?

I tried the following:
/kill @p
/tfc player @p set hunger 4
/tfc player @p query saturation

Saturation is 5.0 / 20

Eats a potato (10% sat, 2 points)
Saturation is 7.0 / 20
Drinks Chocolate
Saturation is 7.0 / 20
Eats a potato
Saturation is 9.0 / 20
Drinks Chocolate
Saturation is 9.0 / 20
Eats a potato
Saturation is 11.0 / 20
Drinks Chocolate
Saturation is 11.0 / 20
Eats a potato
Saturation is 13.0 / 20
Drinks Chocolate
Saturation is 13.0 / 20

OK, so this is a byproduct of how TFC handles zero hunger foods, so it's a TFC issue, not a FIrmalife issue. I'm not gonna open an issue right away so I can consult with alcatraz.


fixed issues in VirtualAudioMicArray (removed arm64 build, fixed issues with x86 and x64 build)

Hi @alecont, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Ale Contenti). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


fix: include status in issue-cred ack

Signed-off-by: Filip Burlacu<EMAIL_ADDRESS>

This issue just cropped up in testing today, seems to have been a small change in aca-py to expect this ack to have a status as well.


Found with a script.

@ulfalizer Port 92 is part of the eSPI peripheral channel.  The protocol supports at least
Peripheral channel, VW Channel, Flash Channel, OOB Channel.  We are working  on the peripheral channel right now.


Export ConmonPidFile in 'podman inspect' for containers

This can help scripts provide a more meaningful message when coming
across issues [1] which require the container to be re-created.
[1] eg., https://github.com/containers/libpod/issues/2673
Signed-off-by: Debarshi Ray<EMAIL_ADDRESS>

/approve

Looks like inspect tests are failing?

Looks like inspect tests are failing?

I was waiting for failing tests to confirm that inspection of containers doesn't work with podman-remote.

LGTM

/lgtm

Thanks for all the reviews!


Dev Guide

Add developer guide (coding style, how to contribute, PR style, etc) on the repository Wiki

i guess modifying #10 to include this would make more sense lol


HTTP request for JavaScript file missing security headers

The security headers "X-Content-Type-Options" and "Strict-Transport-Security" are missing.

After thinking about it, support for security headers should be removed and instead documented to reduce conflict/unexpected behavior.