Update target language if the bundle exists

Fixes: https://github.com/IBM-Bluemix/gp-deliverypipeline/issues/21

Depends on an update to gp-cli including https://github.com/IBM-Bluemix/gp-java-tools/commit/efd89f925d0f25f1344422f007ace34c7c712637 https://github.com/IBM-Bluemix/gp-java-tools/issues/49
Don’t assume an error to create means the bundle exists, check the exception type

:construction: This commit will not build until the .jar is updated… but the code should be stable

as expected: https://travis-ci.org/IBM-Bluemix/gp-deliverypipeline/builds/202801922#L434

updated. should be good to merge if clean

I haven't tested locally, but the change looks good.


feat: Add New Non-Actor Effects Like the Default Sheets

When creating a new effect on an item or any other non-actor entity, the effect is automatically created with the name of the effect owner and its icon.
Currently, New Tidy is just adding a new effect the same way for any document type, which is the Actor style.
Update New Tidy to behave like Old Tidy and the Default Sheet when it comes to adding effects to non-actor documents.

Closing as completed.


synchronize schema spec

What
APM agent json schema automatic sync
Why
Changeset

https://github.com/elastic/apm-server/commit/121300686 intake: Handle transaction.dropped_spans_stats (https://github.com/elastic/apm-server/pull/6200)

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-09-23T12:43:53.392+0000

Duration: 58 min 10 sec

Commit: 65cd562fb723a5e9c068632ae98f7edd7462d28b

Test stats :test_tube:

Test
Results

Failed
0

Passed
2404

Skipped
19

Total
2423

:green_heart: Flaky test report
Tests succeeded.

Expand to view the summary

Test stats :test_tube:

Test
Results

Failed
0

Passed
2404

Skipped
19

Total
2423

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark test.

run compatibility tests : Run the JDK Compatibility test.

run integration tests : Run the APM-ITs.

Superseded by #2166


protoc-gen-go: export oneof interface types

This pull request addresses #261 , which asks to export the generated isX_Union interface types.
This allows for manipulation methods which return oneof types, which can be helpful for writing switch statements. Exporting generated oneof interfaces enables the construction and return of union types, which can be useful for writing helper methods (among other things).
make test passes.
Resubmitting after #587 oddity.

Just making sure this is not lost in the ether.

Please!

@neild PTAL

Sorry, I completely missed this.
I don't think we should do this.
In the protobuf data model, there is no such thing as a single "oneof field" that aggregates the individual components of a oneof. Instead a oneof is just a set of mutually exclusive fields which behave just like non-oneof fields, except setting one implicitly clears all the others.
Exporting this type would introduce a concept into the Go generated code which has no equivalent in most (all?) other languages' protobuf implementions. I don't think that's a good idea.
In addition, the current Go implementation oneofs leaks too many details of the implementation to the user. I think we can do a lot to improve that; in particular, adding setters for oneof fields would be a big help. This PR goes in the wrong direction, by exposing even more of those implementation details.

/cc @bcmills who created the original issue.

That's fair, and it makes sense not obscure the protobuf semantics for the sake of convenience.
The particular use case which prompted this was that, in order to write a function that produces an object which can slot into a oneof field, we currently have to return an instance of the enclosing type with only the oneof field populated, then grab the field value off that parent object. Not intolerable — only a slight inconvenience.
I'd be happy to contribute another MR to improve the oneof implementation if you have thoughts on specific steps to get there.

Closing per @neild's comment: https://github.com/golang/protobuf/pull/588#issuecomment-434882302


Pass boot config template instead of install script template to esx i…

…nstaller
Unfortunately I missed this in testing to my latest update to https://github.com/RackHD/on-http/pull/302
We are passing in the wrong variable for the boot config template (passing in the kickstart instead of the boot script which has the kernel options, and points to the kickstart). This will fix the early installation error:
kernel=<FILEPATH> must be set in http://<IP_ADDRESS>:9080/api/current/templates/esx-ks
Fatal error: 32 (Syntax)

Just ran a dry run of ESX with these options and things work now.
@RackHD/corecommitters @jlongever @WangWinson

+1

👍

:+1:

:+1:


Any way to catch Apple Music?

Hello, great tool, getting m3u8 from apple music can download and parse its ts files correctly, but no audio is found (there is a .aac file with correct file size and length in VLC).

Download is successful btw, just took the picture before it closed itself.
Here is the link for reference/testing:
https://aod-ssl.itunes.apple.com/itunes-assets/Music118/v4/2e/7f/14/2e7f1480-c95b-b02c-fa24-5e285d6ce3a8/mzaf_A1445310459.rphq.aac.wa.m3u8

不支持ISO-23001-7加密方式,

Excuse me, I want to know how to download Apple Music, and how to get the m3u8 link, how to get the DRM key, can you tell me in detail, thank you!


[CT-1140] [BUG] Integration Tests not cleaning up artifacts

Describe the bug
Integration test in branches 1.1 and 1.0 began failing after we exceeded the max number of allowed tables in our sandbox account for the alternate_db (project: dbt-test-alt). Once we dropped the old tables in that project the tests began to succeed again.
Root cause is there are a few  (~5) tests where database objects aren't getting cleaned up after a run resulting in a steady accumulation. Since we set a limit of 10k on the number of objects to return when generating docs the tests fail since the created object is not in the catalog.json file docs generates.
Options:

Document this behavior and a script to clean up tables (minimum path forward)
Add better handling for cleaning up objects in alternate_db/dbt-test-alt
Update dataset default table TTL so tables get automatically deleted

Option #1 is not satisfying but as this is not a frequent pain point (i.e. we will hit this issue maybe once a year) and is pretty easily fixed when it occurs this is not untenable.
Steps To Reproduce
Run these tests until the number of tables exceeds 10k
Expected behavior
Tests should clean up database objects.
Screenshots and log output
See:

https://github.com/dbt-labs/dbt-bigquery/runs/8059670397?check_suite_focus=true#step:7:231

System information
The output of dbt --version:
<output goes here>

The operating system you're using:
The output of python --version:
Additional context
Add any other context about the problem here.

tagging in a new conversion we are seeing also leaving artifacts post run: namely the new Alias tests
dbt-core
BigQuery
single test in bigquery leaving a model after run, was able to get a differnt model to stay after two different variations.

A model staying after changing database using {"database": os.getenv("BIGQUERY_TEST_ALT_DATABASE")}on a model which leaves a dataset in the alt database project.
Using model_b": {"schema": unique_schema + "_alt"} version of test from dbt-core by default leaving a dataset with an odd naming structure in the default bigquery project.

As we can see these hangups are around changes to schema in bigquery which means we may need to update pytest fixtures to have a step to verify against multiple databases rather than using self.database during our drop_schema calls


Configuring exit codes with the parameter --non-zero-exit-code is not working

CloudFormation Lint Version
cfn-lint 0.66.1
What operating system are you using?
Ubuntu host and running cfn-lint in docker built from repo (docker build --tag cfn-python-lint:latest .)
Describe the bug
I have a template with an unused parameter. Running cfn-lint on the template gives the expected warning W2001 and the exit code is 4. However, when I used the parameter --non-zero-exit-code none or --non-zero-exit-code error, the exist code is still 4.
Expected behavior
According to the documentation, setting none or error should make the exit code 0.
Reproduction template
Template (template.yaml):
Parameters:
  UnusedParameter:
    Type: String
Resources:
  LogGroup:
    Type: AWS::Logs::LogGroup

Commands and output:
$ cfn-lint -t template.yaml
W2001 Parameter UnusedParameter not used.
template.yaml:2:3

$ echo $?
4
$ cfn-lint --non-zero-exit-code none -t template.yaml
W2001 Parameter UnusedParameter not used.
template.yaml:2:3

$ echo $?
4
$ cfn-lint --version
cfn-lint 0.66.1

You are using v0.66.1 can you upgrade... we just released v0.86.3 today.  My assumption is this has been resolved for a while.
(cfn-python-lint-wusv) ➜  cfn-python-lint git:(main) cfn-lint --non-zero-exit-code none -t local/issue/3136.yaml
W2001 Parameter UnusedParameter not used.
local/issue/3136.yaml:2:3

(cfn-python-lint-wusv) ➜  cfn-python-lint git:(main) echo $?
0


Why is website required?

It's only used in three places: package.json, license, and readme.md.
These locations could easily be edited in by the user later, if they don't want to immediately provide a website url.

Why not? If you don't have one you can just provide your GitHub/Twitter profile.

That makes sense. I was thinking more along the lines of project webpage, which might not exist so early on.


Directory Structure

Hello, I am new to react and im starting my first project with this generator. Can you please suggest better directory structure for multiple routes, screens, actions, store and for all necessary directory.
Thanks.

I recommend you to learn about Flux and its frameworks like Redux. This way you can imagine how to organize your projects


Upgrade to Django 1.7.10

https://www.djangoproject.com/weblog/2015/aug/18/security-releases/
Getting tests started.

Aside from the foo.bar file, r+

Yeah, I don't know where that came from. It's this weird floating commit on master.


Only the exact match ch is detected by eog, misc and stim_channel of read_raw_edf

Describe the bug
eog, misc, and stim_channel of read_raw_edf are used to assign the channel type for each channel.
Currently, only the exact matched channel can be detected.
Nevertheless, some tutorials, such as Sleep stage classification from polysomnography (PSG) data - MNE 0.24.1 documentation, assume partial match and the results may differ from what is expected.
stim_channel is checked by _check_stim_channel.
eog and misc is checked directly in_get_info.
I'm not sure if this is a bug or intentional behavior.
The behavior of some programs may change if this is changed to an exact match.
(If the EDF file contains a long name channel whose part matches another stim/misc/eog channel...)
If this is intentional behavior, these tutorials should be fixed.
Otherwise, these codes should be updated to allow the partial match.
Steps to reproduce
import mne

files = mne.datasets.sleep_physionet.age.fetch_data(subjects=[0], recording=[1])
raw = mne.io.read_raw_edf(files[0][0], eog=['eog'], stim_channel='marker', misc=['rectal'])
for c, t in zip(raw.ch_names, raw.get_channel_types()):
    print(f'{c}: {t}')

Expected results
EEG Fpz-Cz: eeg
EEG Pz-Oz: eeg
EOG horizontal: eog
Resp oro-nasal: eeg
EMG submental: eeg
Temp rectal: misc
Event marker: stim

Actual results
EEG Fpz-Cz: eeg
EEG Pz-Oz: eeg
EOG horizontal: eeg
Resp oro-nasal: eeg
EMG submental: eeg
Temp rectal: eeg
Event marker: eeg

All channels show eeg (default type).
Additional information
mne.sys_info():
Platform:       Linux-5.4.144+-x86_64-with-Ubuntu-18.04-bionic
Python:         3.7.12 (default, Jan 15 2022, 18:48:18)  [GCC 7.5.0]
Executable:     /usr/bin/python3
CPU:            x86_64: 2 cores
Memory:         12.7 GB

mne:            0.24.1
numpy:          1.21.5 {blas=openblas, lapack=openblas}
scipy:          1.4.1
matplotlib:     3.2.2 {backend=module://ipykernel.pylab.backend_inline}

sklearn:        1.0.2
numba:          0.51.2
nibabel:        3.0.2
nilearn:        Not found
dipy:           Not found
cupy:           Not found
pandas:         1.3.5
mayavi:         Not found
pyvista:        Not found
pyvistaqt:      Not found
ipyvtklink:     Not found
vtk:            Not found
PyQt5:          Not found
ipympl:         Not found
mne_qt_browser: Not found
pooch:          v1.6.0

I think we should update the tutorial – these parameters are meant to match exact names only. Note that you can always re-assign channels types after reading with mne.rename_channels(), which is more flexible because it also accepts a callable or a dict.

Thanks. It's fine for me the usage is confirmed.
Thank you for the rename function, too.

Yes, I can do at least for the sleep stage classification part
and will try to find others if exist.

It seems no other documents have the problem.
(some uses read_raw_edf(stim_channel=, but they seem full names of the channels)

good catch. It's great if you can send a PR.

Message ID: @.***>


BUG: MultiIndex.get_level_values created from Categorical raises AttributeError

Closes #10460.
xref: #10464.

OK, try working on #10464.


How to run multiple skills on flask-ask

Hallo,
I m not clear on how to run multiple skills in flask-ask.
I have local raspi server and two different skills. I use ngrok to forward intends to my raspi server on port 5000.
Please help.

This is out of scope of Flask-Ask, but why not try
ngrok http 5001

to run it on a different port and get a new unique URL

sorry, ngrok allows only open one port at the same time....see below.
Any other suggestions?

Tunnel session failed: Your account 'name' is limited to 1 simultaneous ngrok client session.
Active ngrok client sessions in region 'us': (none)

EDIT:
Ok, I need to start ngrok with authoken and config file to open two tunnels, e.g. 5000 and 5002. BTW: max. is 4 tunnels with free account and 8 tunnels with basic account.
Is there any "FREE" reverse proxy that allows unlimited tunnels?
@johnwheeler:
I think you can close the issue. You pointed me to the right direction and its working.
Furthermore, Flask-ask is a really great starting point for alexa skill developer. So easy to use. I had my own very privat skill running in one day only. Many thanks for all you great work!

Edit2:
Last night I had another idea how to redesign my skills to use only one entry. I could make it like this:
Alexa, ask myskills for myapp1
Alexa, ask myskills for myapp2
Alexa, as myskills
What skill you want to use?
What you think?

I don't know much about flask but you might want to look at/try and put
them on different routes?
app = Flask(name)
ask = Ask(app, "/")
otherAsk = Ask(app, "/other")
It might be they will need to be in different files as I'm not sure if the
@ask.launch etc are linked to the ask variable.
On 14 Nov 2016 7:32 am, "drschlaumeier"<EMAIL_ADDRESS>wrote:

Edit2:
Last night I had another idea how to redesign my skills to use only one
entry. I could make it like this:
Alexa, ask myskills for myapp1
Alexa, ask myskills for myapp2
Alexa, as myskills
What skill you want to use?
What you think?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/johnwheeler/flask-ask/issues/45#issuecomment-260266567,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHZY9MYLJSTX2cmzP7keVMqMYRHGrIEpks5q-A6hgaJpZM4Kwqn2
.

app = Flask(name)
ask = Ask(app, "/")
otherAsk = Ask(app, "/other")

Tried this for multiple skills with different routes and I get the following error.
Traceback (most recent call last):
  File "test.py", line 7, in <module>
    otherAsk = Ask(app, '/other')
  File ""/usr/local/lib/python3.6/site-packages/flask_ask/core.py", line 81, in __init__
    self.init_app(app, path)
  File ""/usr/local/lib/python3.6/site-packages/flask_ask/core.py", line 121, in init_app
    app.add_url_rule(self._route, view_func=self._flask_view_func, methods=['POST'])
  File ""/usr/local/lib/python3.6/site-packages/flask/app.py", line 64, in wrapper_func
    return f(self, *args, **kwargs)
  File ""/usr/local/lib/python3.6/site-packages/flask/app.py", line 1051, in add_url_rule
    'existing endpoint function: %s' % endpoint)
AssertionError: View function mapping is overwriting an existing endpoint function: _flask_view_func

Is this possible with flask-ask?


WindowManager.ShowMessageBox is not showing custom View

I have created a custom MessageBoxViewModel that is implementing IMessageBoxViewModel along with a custom view, MsgBoxViewModel and MsgBoxView respectively. I have set my IoC container (AutoFacBootstrapper) as follows.
builder.RegisterType().As().ExternallyOwned();
However, when I call
_windowManager.ShowMessageBox("test", "test");
It shows up, but not using the view I custom styled. Seems like its using some default view.

Hmm, looks like I broke that a few years ago... https://github.com/canton7/Stylet/blob/master/Stylet/WindowManager.cs#L130

You can grab an intermediate build from the NuGet feed https://ci.appveyor.com/nuget/stylet

Was this update pushed to the newest version? I even tried to update using the original link above and I still seems to bu using the default view.

No, I haven't yet released this (I should do, thanks for the reminder) but the intermediate version <IP_ADDRESS> from the nuget feed above should work.
Someone submitted a PR based on master which triggered some builds with newer version numbers than <IP_ADDRESS>, which is probably what you're installing.

Checking in again, could this fix be pushed to the latest version?

Sorry for the delay, released!


Using Pebble to render HTML Emails in Spring Boot 2

I have a Spring Boot 2 MVC-type web application and which uses Pebble as a template engine. I am a bit of a Java noob, but so far, all works well.
However, I am now trying to also render HTML emails in this application, using Pebble also.
For instance, I am trying to autowire the PebbleEngine pebbleEngine in the user registration controller and passing it to a Template class I created which contains the following function:
public String render(PebbleEngine pebbleEngine) throws Exception {
PebbleTemplate template = pebbleEngine.getTemplate(this.view);
template.evaluate(Template.writer, this.model, this.locale);
return writer.toString();
}
I am using a newly instantiated StringWriter in the Template class for writer. Model, Locale and View are also properties of the Template class.
This almost works, BUT, for some reason, the content of my HTML registration emails (which use inheritance, i.e. a register.pebble template which extends an emailBase.pebble template) always get rendered twice.....
I also tried to autowire the PebbleEngine pebbleEngine inside the Template class, but this never works.
How do we (best) use Pebble to correctly render emails in a Spring Boot 2 web application which already uses Pebble as a template engine?

It doesn't seem to be related to pebble but related to the way you use spring mvc. Can help you much here without a working example of your controller. The value you return in a controller should be the template and you should just fill the model.
Please take a look at https://github.com/PebbleTemplates/pebble-example-spring

Apologies, I should have clarified, but I am using the following dependency:
<dependency>
<groupId>io.pebbletemplates</groupId>
<artifactId>pebble-spring-boot-2-starter</artifactId>
<version>3.0.0</version>
</dependency>
The relevant parts of the  controller simply do the following (attributes is the model HashMap):
...
@Autowired
PebbleEngine pebbleEngine;
...
Template template = new Template("email/registration", attributes, locale);
try {
content = template.render(pebbleEngine);
} catch (Exception e) {
e.printStackTrace();
}
...
The relevant parts of the Template pojo class look like this:
...
private static Writer writer = new StringWriter();
...
public String render(PebbleEngine engine) throws Exception {
PebbleTemplate template = engine.getTemplate(this.view);
template.evaluate(Template.writer, this.model, this.locale);
return Template.writer.toString();
}
...

Please provide a working sample if you want some help. I don't have enough information to help you or to determine if it's really a bug.

Wow... If you want some help, i suggest that you submit a test case or a link to a github project to demonstrate you're setup. I already answered that you'll probably need to setup a second PebbleEngine to render html. I have already used pebble in a project that was used to render html and render email and I used 2 PebbleEngine. Maybe we can add a guide on the wiki.
https://github.com/PebbleTemplates/pebble-spring-boot-starter has been merged with the pebble repository(https://github.com/PebbleTemplates/pebble/tree/master/pebble-spring). That's why it has been archived and it still activily suppoorted.
Documentation on  http://mitchellbosecke.com/pebble/home is no longer up to date. Mitchell has quit the project and didn't remove the documentation. Can't do much here. The official documentation is on the wiki(https://github.com/PebbleTemplates/pebble/wiki/documentation) like said in the README of the pebble project.

Thanks for the quick reply and the clarification re the spring-boot-starter project. As I said, I am a bit of a Java (and especially Spring) noob, so when you casually say I have to setup a second engine to render html emails, that, unfortunately, means exactly nothing to me. Obviously, that is my problem, but yes, I think an example somewhere would be very helpful, at least for me (and I am sure there are others who may appreciate it). The project you mention where you used 2 engines, is that a Spring Boot project and if so, where can one find it?
In my setup above, I have tried to instantiate a new engine in my Template class like so:
private static PebbleEngine engine = new PebbleEngine.Builder().build();
instead of passing it into my render() function in my Template class, but that did not work and it also does not find my templates anymore.

Just added an example here that reuse the same PebbleEngine:
https://github.com/PebbleTemplates/pebble-example-spring/tree/master/spring-boot-with-email

Thanks for this, but I don't think that's what I am looking for and I am not sure, whether you actually tried to send an html email with this. In principle, what you are doing in this example I don't think is too dissimilar from what I am doing and yes, as I said in my original post, I always got my webapp to render and send the html email, BUT, the html content is always being rendered TWICE.
When you google for sending html emails with Spring Boot, there are many complete and well documented articles/ examples around for Thymeleaf, Velocity etc., alas, none for Pebble.
I have now literally wasted days with this (luckily I am just working on a personal little project), but it has taken me less time to write the entire application, than it has to try and figure-out this little thing and as I said, this is not the first time, that I have wasted hours/ days when working with Pebble.
So, while I will certainly continue to use Twig when working with PHP, I think I will need to simply find an other template engine for Java.
But I wish you guys all the best of luck!

You should change your attitude if you want help from others. I'm 100% sure it's a problem with your setup. Don't know why you're using a Template object but like you said, you're a java/spring noob and you'll certainly need a lot of practice.
I also wasted my time trying to help you and wish you best of luck ...


Simplifying requires

This fixes https://github.com/github/linguist/issues/2223
Not sure why we weren't just doing this originally / cc @bkeepers

We used to require each file individually, but don't do that anymore.  :+1:


allow additional options to be specified for associations

Affects BelongsTo,  BelongsToMany, HasMany, HasOne, and ForeignKey attributes. Compatibility with existing parameter that takes string name of foreign keys has been provided by letting the foreign key be a string (existing functionality) or the appropriate association options (new functionality). Additionally, ForeignKey can now receive options. By default it simply sets the foreign key name to the property name, which should be backwards compatible.
I did not add tests for the Many-to-Many cases, because additional options outside setting the FK column names and table name are mostly superfluous.
Fixes #34

Codecov Report

Merging #42 into master will decrease coverage by 0.11%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
- Coverage    94.5%   94.38%   -0.12%
==========================================
  Files          65       65
  Lines         692      713      +21
  Branches       91       99       +8
==========================================
+ Hits          654      673      +19
- Misses         14       15       +1
- Partials       24       25       +1

Impacted Files
Coverage Δ

lib/models/BaseSequelize.ts
88.33% <100%> (+0.4%)
:arrow_up:

lib/annotations/association/HasMany.ts
100% <100%> (ø)
:arrow_up:

lib/annotations/association/BelongsToMany.ts
100% <100%> (ø)
:arrow_up:

lib/annotations/association/HasOne.ts
100% <100%> (ø)
:arrow_up:

lib/annotations/association/BelongsTo.ts
100% <100%> (ø)
:arrow_up:

lib/annotations/ForeignKey.ts
100% <100%> (ø)
:arrow_up:

lib/services/association.ts
83.05% <60%> (-1.57%)
:arrow_down:

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 76b2920...d8b92e0. Read the comment docs.

@HackedByChinese thanks for implementing this. Looks good.

codecov/patch — 92.85% of diff hit (target 94.5%)

is not an issue. I will merge it anyway

Hello! Any comment on when we might see this in a release? Thanks!

Hey, I'm planning to release a new version at the end of this weekend or not later than next week. To make this happen I need to fix #43 and #37 .

@HackedByChinese Hey, I removed the AssociationForeignKeyOptions  from the ForeignKey annotation, because these options only consists of a name and ColumnOptions. These options should rather be set via Column annotation, which is already part of sequelize-typescript.
So that the following example, which shows your implementation..
@ForeignKey(() => Model, {name: 'fooId', field: 'foo_id'})
fooId: number;

..should be achieved with
@ForeignKey(() => Model)
@Column({field: 'foo_id'})
fooId: number;

I think that this approach makes the api much cleaner. What do you think?
Additionally I added support for otherKey in the AssociationOptions of BelongsToMany.

Sure, that makes sense. Thanks!


Drop pep517

All tests pass on Linux without it.
Thank you for contributing to Pipenv!

Good find, this is no longer required now that we dropped requirementslib 🎉


Feature request: generic event hook system (a la acpid)

So, the point of this suggestion is to cover as many use cases as possible to prevent feature creep in the base code and the amount of load on the base code devs, while still getting extra features that cover minority use cases.
In my case, I'm very interested in the potential of Syncplay for study groups. Khan Academy has a gamification system that lets you keep track of your study activity and rewards you with Meaningless Internet Points (TM), which are fun to receive, despite being mostly meaningless. :P
I saw that Khan Academy seems to have decent API documentation, which should be a breeze to use from Python -- so it totally has potential. A hook system would allow for these sorts of solutions to be built for all sorts of obscure needs. I'd be willing to develop the integration hooks myself if the hook system was easy enough to figure out! And of course release it under some FOSS license because I like FOSS. :3
Off the top of my head, the hook system could also be used for lots of other things as separate projects, including:

Optional audio notifications of events (In several instances, I had accidentally pressed m in MPV previously, and wasn't aware someone had started the video)
Optional toaster or other GUI notifications (In other instances, I had my whole pulseaudio system muted because I'm a dork, or was otherwise unable to hear laptop audio due to a glitch with my bluetooth headset -- so these notifications would be great for me)
Integration with other learning platforms (e.g., Moodle)
If some hacker somewhere wants to make their raspberry pi controlled coffee maker start brewing coffee whenever a video is queued, they can totally do it without forking the base code. (And catch-all for any other obscure scripting use case I'm not imagining.)

Let me know what you think!

I like the idea of making Syncpplay more extensible in a way that avoids feature creep, but I don't see myself as having time to code such a thing in the foreseeable future.
For mpv users some things are possible at present via modification/extension of syncplay.lua as the mpv script can actually run Syncplay /commands and has access to notifications, etc. It is possible that more could be done in this regard to allow mpv scripts greater information and access.

I thought syncplay.lua was for VLC?

@alxpettit Ah yes, I should have said syncplayintf.lua: https://github.com/Syncplay/syncplay/blob/master/resources/syncplayintf.lua

We do not currently have any volunteers to work on this project, but if someone volunteers either to work on making Syncplay more extensible or specifically on adding Khan Academy support then please let me know. If nobody comes forwards then I will close the issue but add it to https://syncplay.pl/about/ideas/ as an idea that someone could help progress.

I volunteer! I don't have a lot of free time, but fortunately Python is the main language I like to code in, and I don't think I'm too rusty.

So, it's been awhile, and I'm going to finally start going through the code in my free time and see if I can figure out where the event hooks should be placed and how they should be implemented.
Another option is to use a plugin system for dynamically loading in Python code instead of calling external applications -- which is potentially a more polished way of doing it, but may make it more difficult for non-Python programmers to interact with the system. I'd like to make sure it's as useful to as many people as possible, and not just me. :P
Having a single program called whenever an event happens, and having the entire thing event-driven, seems like it ought to work for most use cases that I'm envisioning. If a user wanted multiple scripts for each, they could just use a switch case thingy. If they want to do it in Python, or Bash, or C++ -- it'll work just fine. The only requirement is that they understand how to parse the arguments.
So, the next question is how the arguments should be formatted. I'm tempted to do it with just a single JSON string, since Syncplay already handles JSON and it's a good way of unambiguously structuring nested data for when arbitrarily large amounts of contextual information are needed.
But on the other hand, if someone wants to make a handler in Bash, I suspect they're going to have a bad time trying to cope with JSON input.

Another concern is whether someone would want external code to continuously run in the background or not. Forcing an event-driven architecture on this implementation would mean that applications that might benefit from continuously running code inserted into Syncplay itself, would have a harder time. For example... if I wanted to write a Syncplay app that did a lot of complex regex for some reason -- it would have to reload the interpreter executable, any libraries needed, and recompile the regex, every time it had to process something. Which means that this application is only acceptable if this inefficiency won't affect too many use cases. (Of course, if they were desperate enough I suppose someone could write a two-part application with a "trigger script" that just got launched by the event hook system and then quickly passed the data straight to a server component over IPC and then exited...)

Um... not really getting a lot of feedback here. I guess I'll just start doing stuff and hope that it's actually compatible with what the community wants.

@alxpettit Hey. Sorry for not getting to you earlier.
I don't think you have to worry about using json in bash scripts, as there's jq that can help handling it. And I think jsons would be the most readable solution here too.
As for the way you'd handle the event sourcing, I don't really have an opinion on that. IMO it's best if you just implemented your app in parallel and you'd see what works best.
Simple solution would be Twisted's plugin system

If this is how Syncplay was coded from the start to use an event system then a lot of current Syncplay features could have been plugins, simplifying the main codebase. However, adding an event system at this stage could add complexity to Syncplay which could make it harder to maintain the codebase. I have now added your idea to https://syncplay.pl/about/ideas/ so that others can be informed by this discussion.


how to we set preference to change the download location in chrome while using pytest mozwebqa

Please paste the output of the following command here: sudo edit-chroot -all

Please describe your issue:
how to we set preference to change the download location in chrome while using pytest mozwebqa.
i tried with below command
--chromeopts='{"prefs":["download.default_directory":"c:\ler","download.directory_upgrade":true]}'
but its not working.
Please help on this
If known, describe the steps to reproduce the issue:

#262


Pasting lists show up as blocks, and don't get saved

To reproduce:

Paste this list into a text box

Foo
Bar
Char
Mar

Refresh the page

Closed in #161


Resolution does not work as it should

I am trying the png recorder and I want it to record Large frames (6048x4023). It will create the frames in that size but they will contain only upscaled images. I imagine the framebuffer that is being captured should be sized up also?

yes, of course.


Small README update

Current dependencies on/for this PR:

main

PR #59   👈

This comment was auto-generated by Graphite.


Add sky_only setting to DirectionalLight3Ds

This fixes an issue I have had with sky shaders since the very beginning. When the light drops below the "horizon" it still affects all the objects in the scene. This is terrible for day/night cycles. And worse yet, if you hide the light, it would no longer be tracked by the sky shader and your next DirectionalLight3D would become the sun. The only solution was to set the color to black, but then you are creating shadows, and doing an extra light pass on every object in the scene for no reason.
The solution is to add a setting to DirectionalLight3Ds that removes them from being used everywhere except sky shaders.

I suppose this is also useful when you use a fully-baked workflow with BakedLightmap?

@Calinou I don't think it will really be helpful for baked stuff. When set to sky only it won't be used in bake or in shading at render time.

Thanks!


[sw/dif] DRAFT: Introduce an I2C DIF

The DIF is still somewhat incomplete: I have read/write byte functions, but I really want to add functions for doing simple I2C transactions. I also still need to add tests.

Just making sure @igk8 @martin-lueker and @tunghoang290780 see this

Added unit tests. Note that I've used designated initializers in the C++, which is still pending style approval (there's some details we haven't sorted out but my uses here are not contentious).


Need for a Official function to project a pointcloud to rangeview

Hi, thank you for providing such a wonderful dataset. However, I need to project a processed pointcloud to rangeview. I use a third party function which work in KITTI. It need a fov_up and fov_down of lidar. I try the param in you papar, fov_up==2.4, fov_down==-17.6, but it doesn't work. So if you  provide a official code for project a pointcloud to rangeview will be nice.

@rezama @patzm @vatavua


Rake Task to set max_import_file_size and max_import_table_row_count

Context
Working on community edition of builder. I would like to have a rake task to set max_import_file_size and max_import_table_row_count to some chosen values
Additional info
I'm very willing to write this myself if necessary and have it reviewed. I would need some additional info, though

Should the rake task fail if we try to set up max_import_file_size greater than quota_in_bytes? I'd say yes, but just want to check
We should clarify what the rake should do in case user belongs to an org. Currently builder sets those limits to the owner for every org user created, so the task would take into account.

Finally this is something we may not want to do...

About your questions:

Maybe, but that's not rake concern but model one. Let's address it in #11758.
Assign it to an organization user will override the one previously copied from the organization owner. IMHO, that might make sense for special users, so I wouldn't treat an organization user as an special case.

@rporres single user, by username?

That should work, @juanignaciosl

This is in acceptance in Builder board. @rporres do you want to test it yourself? Also, I'd like to learn more about this community edition, I'll grab you at some random point and shoot some questions your way.

@rporres can you do the acceptance, as this was your need? No hurry, but we'd like to know :-)

hey, sorry. I had forgotten to answer.
Sure I will do the acceptance. It will be delayed maybe one or two days. Is that ok?

Yes, no problem, I just wanted to know :-) I'll assign it to you so we know the reason for the delay.
Thanks!

CR and review
PR: https://github.com/CartoDB/cartodb/issues/11772

Test1: modify existing users (Organization owner included)

cartodb_production=# select username,max_import_file_size,max_import_table_row_count from users;
 username | max_import_file_size | max_import_table_row_count
----------+----------------------+----------------------------
 user1    |            157286400 |                     500000
 carto    |            157286400 |                     500000

carto_rake "cartodb:set_import_limits["user1",1057286400,1000000]"
carto_rake "cartodb:set_import_limits["carto",1057286400,1000000]"

postgres cartodb_production -c "select username,max_import_file_size,max_import_table_row_count from users;"
 username | max_import_file_size | max_import_table_row_count
----------+----------------------+----------------------------
 user1    |<PHONE_NUMBER> |                    1000000
 carto    |<PHONE_NUMBER> |                    1000000
(2 rows)

Test2: Create a new user and check it properly inherits Org Owner limits

root@omnibus-i-builder-dataservices-rhel6-user-jvillarf:/opt/carto/builder-tools/embedded/bin# psql -U postgres cartodb_production -c "select username,max_import_file_size,max_import_table_row_count from users;"
 username | max_import_file_size | max_import_table_row_count
----------+----------------------+----------------------------
 limitok  |<PHONE_NUMBER> |                    1000000
 user1    |<PHONE_NUMBER> |                    1000000
 carto    |<PHONE_NUMBER> |                    1000000
(3 rows)

So, the rake task works fine. Thanks a lot!
@javitonino @juanignaciosl you can move this ticket to the next phase!

Thx for taking care, @jvillarf

Merged!


Dynamic batch support

Hi there, does it support dynamic batch, I can't find the related introduction.

This related to #133 ?


CMS Upgrades version 4.0.x to 4.1.2

Automated CMS upgrades will run upgrading customers from CMS version 4.0.x to 4.1.1 where they have the option to automatically upgrade to new minor versions enabled.
As each CMS is upgraded, there will be a brief period (normally a few minutes) where the CMS may be unavailable.
Upgrades are carried out at off-peak times for the region that your CMS is hosted in.


Footer design improved

Issue Number
fixes #5
ISSUE #5
Describe the changes you've made
Improved footer design
LGM-SoC'21

@Jagannath8 footer design is not good.
Follow the website theme, your footer looks too odd.
Use black-red color, and apply some creative effects.


Make all maths function work with all types of numbers

Just a matter of wrapping the functions in T : SomeNumber and convert to float when passing it through

duplicate of #87


I create place.py to add "food menu" module

I want to add "Food menu" module!
"Food menu" module works as follows:
users send "place" module to JARVIS then JARVIS pick one of many foods and send a message about the picked menu to user!
So I create "place.py"
Short description of what this resolves:
Changes proposed in this pull request:

Fixes: #

Thank you for taking the time to contribute! This is sincerely appreciated. 😇
(Powered by GitMate.io)


move expect-type to https://github.com/mmkal/expect-type

start move to separate repos with expect-type
CC @papb who has a long-standing PR open. There were a couple of changes made as part of the move to do with the latest typescript version, so the PR may need to be tweaked slightly when moved to https://github.com/mmkal/expect-type

gonna merge this in, might or might not fix the build since everything must go

Oops thanks for the heads up, I will look into it later


Universal classifier with data-reuploading

Description of the Change:
Added an example which implements a simple Neural Network with a data-uploading circuit discussed in https://arxiv.org/abs/1907.02085
I am a bit confused about how the plots are rendered?
@josh146

This is great, thanks @quantshah! Have you run make docs to compile the documentation? Sphinx gallery will run the Python script, and automatically embed the plots created under each plt.show() command

Oh I see. Running it now and updating the PR with the latest plots.

Hi,
I think this is ready for review. If you want to see how it looks then check this pdf that I generated locally:
Universal Quantum Classifier — PennyLane 0.4.0 documentation.pdf

Something cool would be to show how the decision boundary keeps updating with training as an animation/gif. But maybe I will add it later.

Thanks for the very thorough review @josh146. I took care of all the comments. I would just mention that using pl as the abbreviation for pennylane makes more sense to me than qml. Since we also have a qml toolbox. Would it be possible to change all the qml references (in docs, code, examples etc.) to pl sometime?
@mariaschuld If you could please have a check on the explanation part. I read the paper and tried to explain it to the best of my understanding.

Hey @quantshah, will most likely only get to it after Wednesday, but I spoke to one of the authors of the paper recently and it is great to have this implemented so quickly!

I would just mention that using pl as the abbreviation for pennylane makes more sense to me than qml.

Definitely something worth discussing. Using qml is a naming convention used before PennyLane acquired its existing name. Until we make a decision to change the naming convention across the board, however, best to keep everything consistent.

Also, could we turn this into a template, together with the most important options mentioned in the paper? It should be almost trivial, but requires careful test-writing and documentation...

Hi all, any updates on the status of this PR? Looks like there were some requested changes/comments that need to be addressed?

@quantshah said he will be working on this this week.
Once #291 is merged, we will need to change the target of this PR to master.

In regards to @mariaschuld's comments, I do agree that this is a more basic introduction, but I think that's ok.
We can certainly make sure that the "core" tutorials build off one another.
But it also makes sense to have under the "QML algorithms" section of the gallery different tutorials that are not necessarily sequential or dependent on one another. For instance, we could mark some tutorials as "Beginner" or "Introductory" and others as "Advanced" or "Expert". A user might end up at this tutorial as the first one they clicked on, depending on what catches their eye.

Hi everyone, thank you so much for the careful and detailed review. I addressed some of the comments and rebased against latest master. I am doing much more with this code in a personal project but this should be good enough for anyone to start playing with the idea presented in the original paper.
This is ready for a re-review. Do I need to manually remove the "skip" tutorials? I am a bit confused about that.
Also, in the original approach, the authors train using L-BFGS (which I do not mention here anymore as @co9olguy suggested it might get confusing), but I use ADAM. Now the issue is, I am using only a small number of epochs and with each run, the decision boundary is slightly different even though the accuracy is really good. This might also be due to using a very simple fidelity based cost function instead of the "weighted fidelity" that the authors use.
Sorry for the delay here since I myself am trying to understand this work better at the theory level.

This is ready for a re-review. Do I need to manually remove the "skip" tutorials? I am a bit confused about that.

Previously, we used skip in filenames to make a tutorial as not to be executed by Sphinx gallery. Now, we instead use an explicit run in the filename to mark those that should be run. Those without run are implicitly skipped.
However it looks like a previous merge caused the old files to reappear in this PR; could you delete the following:
examples/pennylane_QGAN.py
examples/pennylane_advanced_usage.py
examples/pennylane_gaussian_transformation.py
examples/pennylane_qubit_rotation.py
examples/pennylane_skip_plugins_hybrid.py
examples/pennylane_skip_pytorch_noise.py
examples/pennylane_skip_quantum_neural_net.py
examples/pennylane_skip_state_preparation.py
examples/pennylane_skip_variational_classifier.py
examples/pennylane_variational_quantum_eigensolver.py

In addition, could you revert the following files to the version in master:
git checkout origin/master doc/_static/thumbs/code.png
git checkout origin/master pennylane/decorator.py

Thanks!

Hi @co9olguy and @mariaschuld I am waiting for your approval on this. Sometimes the training goes really well and we get upto 91% in the test data accuracy (even more than what they got in the paper)
Epoch: 30 | Loss: 0.111283 | Train accuracy: 0.880000 | Test accuracy: 0.917500.

This is of course dependent on the hyper parameters / regularisation etc. So at this point, I am not so worried about reproducing the exact results of the paper. But rather just showing this new way of loading data into a Quantum Neural Network (via the unitaries).
Works decently with other types of data too:


Client Login via device authorization flow

The client will fetch the device authorization flow information from management and will use it to login using our hosted provider.

moved the OAuth package to internal as we only have one option
use management URL in the service configuration
Added SSO login to UI and CLI clients
Check if the client is logged in when starting UI
retrieve device authorization flow information from management

@mlsmaycon do we really need --sso?
When there is no setup key provided then it should be interactive login by default, right?

@mlsmaycon do we really need --sso?
When there is no setup key provided then it should be interactive login by default, right?

we do for now, for self-hosted users, we will make it default in new releases

@mlsmaycon
When doing first run:
sudo rm -rf /etc/wiretrustee/config.json && sudo systemctl restart wiretrustee.service
Log output. The PermissionDenied error is okay, right?  We first try to log in and see, that the device has not been registered.
Maybe we shouldn't out it? It is a part of normal logic, not something abnormal. Or maybe we could just state in log that the status is LoginRequired.
time="2022-05-12T09:19:37+02:00" level=info msg="starting service"
time="2022-05-12T09:19:37+02:00" level=info msg="generating new config /etc/wiretrustee/config.json"
time="2022-05-12T09:19:37+02:00" level=info msg="started daemon server: /var/run/wiretrustee.sock"
time="2022-05-12T09:19:37+02:00" level=error msg="peer registration required. Please run wiretrustee login command first"
time="2022-05-12T09:19:37+02:00" level=warning msg="rpc error: code = PermissionDenied desc = provided peer with the key wgPubKey yv1sR4rlyEON8EwZbgt7h3kul8j5IF3ED31wWSJahxM= is not registered"

@mlsmaycon
When doing the first run and then
wiretrustee up --management-url http://localhost:33073
There is an error in the log:
time="2022-05-12T09:24:22+02:00" level=info msg="starting service"
time="2022-05-12T09:24:22+02:00" level=info msg="generating new config /etc/wiretrustee/config.json"
time="2022-05-12T09:24:22+02:00" level=info msg="started daemon server: /var/run/wiretrustee.sock"
time="2022-05-12T09:24:22+02:00" level=error msg="peer registration required. Please run wiretrustee login command first"
time="2022-05-12T09:24:22+02:00" level=warning msg="rpc error: code = PermissionDenied desc = provided peer with the key wgPubKey AjjCR5QapL4tMT2qLFUk5E/t8zpcwDKl2w8yuAmhY1k= is not registered"
time="2022-05-12T09:24:27+02:00" level=info msg="new Management URL provided, updated to http://localhost:33073 (old value https://api.wiretrustee.com:33073)"
time="2022-05-12T09:24:27+02:00" level=info msg="new Admin Panel URL provided, updated to https://app.netbird.io (old value <nil>)"
time="2022-05-12T09:24:32+02:00" level=info msg="peer has been successfully registered on Management Service"
time="2022-05-12T09:24:33+02:00" level=info msg="connected to the Signal Service stream"
time="2022-05-12T09:24:33+02:00" level=info msg="Wiretrustee engine started, my IP is: <IP_ADDRESS>/24"
time="2022-05-12T09:24:33+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:33+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:34+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:34+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:35+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:35+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:36+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:36+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:36+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:36+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:37+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:37+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:37+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:37+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"

@mlsmaycon
When doing the first run and then wiretrustee up --management-url http://localhost:33073
There is an error in the log:
time="2022-05-12T09:24:22+02:00" level=info msg="starting service"
time="2022-05-12T09:24:22+02:00" level=info msg="generating new config /etc/wiretrustee/config.json"
time="2022-05-12T09:24:22+02:00" level=info msg="started daemon server: /var/run/wiretrustee.sock"
time="2022-05-12T09:24:22+02:00" level=error msg="peer registration required. Please run wiretrustee login command first"
time="2022-05-12T09:24:22+02:00" level=warning msg="rpc error: code = PermissionDenied desc = provided peer with the key wgPubKey AjjCR5QapL4tMT2qLFUk5E/t8zpcwDKl2w8yuAmhY1k= is not registered"
time="2022-05-12T09:24:27+02:00" level=info msg="new Management URL provided, updated to http://localhost:33073 (old value https://api.wiretrustee.com:33073)"
time="2022-05-12T09:24:27+02:00" level=info msg="new Admin Panel URL provided, updated to https://app.netbird.io (old value <nil>)"
time="2022-05-12T09:24:32+02:00" level=info msg="peer has been successfully registered on Management Service"
time="2022-05-12T09:24:33+02:00" level=info msg="connected to the Signal Service stream"
time="2022-05-12T09:24:33+02:00" level=info msg="Wiretrustee engine started, my IP is: <IP_ADDRESS>/24"
time="2022-05-12T09:24:33+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:33+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:34+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:34+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:35+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:35+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:36+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:36+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:36+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:36+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:37+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:37+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"
time="2022-05-12T09:24:37+02:00" level=info msg="connected to the Management Service stream"
time="2022-05-12T09:24:37+02:00" level=error msg="failed handling an update message received from Management Service: wgtypes: failed to parse base64-encoded key: illegal base64 data at input byte 44"

Was a local issue with peers in testing account


parsedatetime is not an optional requirement

Since v4.0.0 parsedatetime has been a requirement. (https://github.com/insanum/gcalcli/commit/166a58a81068cef858caece10e41db591f98ac53) parsedatetime was originally an "extra", a recommended optional package. The commit adding it as a hard dependency didn't remove it from the list of extras.
Before this change both docs/README.md and setup.py list parsedatetime twice. After this change parsedatetime is not described as optional.

thank you kindly!


Add more features on the landing page.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
There are only two images currently on the landing page, it should have one from each category
Describe the solution you'd like
A clear and concise description of what you want to happen.
I would like to add more such feature images
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@sau-mili  okay go ahead , i will mark you after seeing ur PR


Use standard find_package for ITK in SimpleITK Config

To properly import ITK's targets and it's transitive dependencies, use
the find_package(ITK). This issues was detected when enabling IODCMTK,
and the examples failing to link properly.

@thewtex @dzenanz This enables building against ITKv5 with TBB. The TBB libraries are not picked up properly.

It is good that you found the issue!


Add include? to list, uniquelist

Motivation

Support include? method for Kredis::Types::List

Modification

Allow List to proxy lpos (refer: https://redis.io/commands/lpos/)
Add some example of List#include?

Result

Users can use include? without load members of list to memory

lpos requires version >= 6.0.6. Close PR until found a nice way to support lower versions.

Docs are sufficient here since it's a new capability, e.g. list.include? requires Redis 6+ with LPOS support. People can keep using list.elements.include? otherwise.


Add generic StatusController that can be mapped to /health for liveness probe across services

This work was initially implemented and reviewed here: https://github.com/RedHatInsights/catalog-api/pull/528
This is basically the extraction of the (albeit tiny) controller that can then be added to any application and then mapped to /health as in the dummy app here. The goal is to use this across the microservices to have a liveness probe watch this URL.

cc @carbonin

@bdunne want to take a look?

@bdunne the goal is to pull the fields into kibana or something - that way we can keep track of the state over time etc.

@bdunne the goal is to pull the fields into kibana or something - that way we can keep track of the state over time etc.

To get this data into Kibana you would need to log it.
If there's a metric that you want to keep track over time, it should probably be registered in the Prometheus export server

If we split the migration issues out of this and just focus on the liveness probe, maybe we should just return HEAD 200?  After reading this blog article it sounds like our liveness probe shouldn't even check the connection to the database.  What do you think?

@bdunne  Yes, I'm ok with removing most (if not all) of that logic in the controller as long as we make the pod impossible to start if the database migrations fail.


ERROR | Exception : GPT3 error: module 'openai' has no attribute 'ChatCompletion'

How to fix this?
D:\ShortGPT-stable\ShortGPT-stable>python runShortGPT.py
Running on local URL:  http://<IP_ADDRESS>:31415
To create a public link, set share=True in launch().
Step 1 _generateScript
Error communicating with OpenAI: module 'openai' has no attribute 'ChatCompletion'
Error communicating with OpenAI: module 'openai' has no attribute 'ChatCompletion'
Error communicating with OpenAI: module 'openai' has no attribute 'ChatCompletion'
Error communicating with OpenAI: module 'openai' has no attribute 'ChatCompletion'
Error   File "D:\ShortGPT-stable\ShortGPT-stable\gui\short_automation_ui.py", line 61, in create_short
for step_num, step_info in shortEngine.makeContent():
File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\engine\abstract_content_engine.py", line 72, in makeContent
self.stepDictcurrentStep
File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\engine\reddit_short_engine.py", line 38, in _generateScript
self._db_script, _ = self.__getRealisticStory(max_tries=1)
File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\engine\reddit_short_engine.py", line 25, in __getRealisticStory
new_script = self.__generateRandomStory()
File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\engine\reddit_short_engine.py", line 16, in __generateRandomStory
question = reddit_gpt.getInterestingRedditQuestion()
File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\gpt\reddit_gpt.py", line 17, in getInterestingRedditQuestion
return gpt_utils.gpt3Turbo_completion(chat_prompt=chat, system=system, temp=1.08)
File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\gpt\gpt_utils.py", line 88, in gpt3Turbo_completion
raise Exception("GPT3 error: %s" % oops)

How to fix this?
D:\ShortGPT-stable\ShortGPT-stable>python runShortGPT.py Running on local URL: http://<IP_ADDRESS>:31415
To create a public link, set share=True in launch(). Step 1 _generateScript Error communicating with OpenAI: module 'openai' has no attribute 'ChatCompletion' Error communicating with OpenAI: module 'openai' has no attribute 'ChatCompletion' Error communicating with OpenAI: module 'openai' has no attribute 'ChatCompletion' Error communicating with OpenAI: module 'openai' has no attribute 'ChatCompletion' Error File "D:\ShortGPT-stable\ShortGPT-stable\gui\short_automation_ui.py", line 61, in create_short for step_num, step_info in shortEngine.makeContent(): File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\engine\abstract_content_engine.py", line 72, in makeContent self.stepDictcurrentStep File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\engine\reddit_short_engine.py", line 38, in _generateScript self._db_script, _ = self.__getRealisticStory(max_tries=1) File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\engine\reddit_short_engine.py", line 25, in __getRealisticStory new_script = self.__generateRandomStory() File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\engine\reddit_short_engine.py", line 16, in __generateRandomStory question = reddit_gpt.getInterestingRedditQuestion() File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\gpt\reddit_gpt.py", line 17, in getInterestingRedditQuestion return gpt_utils.gpt3Turbo_completion(chat_prompt=chat, system=system, temp=1.08) File "D:\ShortGPT-stable\ShortGPT-stable\shortGPT\gpt\gpt_utils.py", line 88, in gpt3Turbo_completion raise Exception("GPT3 error: %s" % oops)

the OpenAI pip library seems to be miss-installed , try pip install --upgrade openai


VS Code hangs and couldn't even be terminated. Needed to reboot my machine

VSCode Version: 1.5 insider
OS Version:

No steps. Here is what I could capture:

I exited VS Code but two processes stayed around
"C:\Program Files (x86)\Microsoft VS Code Insiders\Code - Insiders.exe"
"C:\Program Files (x86)\Microsoft VS Code Insiders\Code - Insiders.exe" --reporter-url=https://ticinocrashreporter.azurewebsites.net/crash --application-name=VSCode --v=1

trying to kill one of them reported on the first one:

starting code again didn't work since it tried to reuse these two process. The start simply hung.

@bpasero FYI. I will add additional information if I see it again.

And then you can reopen.


Communicate the socket used to connect to the MQTT server

This allows the handler to monitor the socket and advise a reconnect if the socket is not using the best available internet interface.

Awaiting a new release of Tortoise311 with https://github.com/smartrent/tortoise311/pull/53


Fix type resolution when package.json has a path mapping

In the process of tracking down the issue, I created a test case, and from there it was a simple fix, so here we are with a PR and no issue for the bug (sorry).
The test should hopefully illustrate the issue. If you do import "./foo", and foo is directory containing a package.json with a "types" entry, then typescript will resolve "./foo" using that entry (rather than the usual "./foo/index" behavior). However, the previous implementation of fileExists and readFile didn't load package.json (because it isn't part of the program object), so the language service ended up not resolving the file.
Without the fix, the test fails with this error:
Expected type to be: type Bar = string | number, got: type Bar = any

The change to getDefaultLibFileName and the added properties aren't necessary for this particular bug (feel free to revert them), but they seemed like they might be good for a standard service host implementation, based on the example at https://github.com/microsoft/TypeScript/wiki/Using-the-Compiler-API#incremental-build-support-using-the-language-services.

This should be available in v0.2.2. Cheers!

Cheers, thanks!


tests: change assert to require in remaining places

As suggested by @agilgur5:
Implement remaining if assert.* -> require.* changes as mentioned in #13401 (comment) etc?

Hello @Joibel, can I start working on this? It looks like a good task for newcomers.


Add text during resting state

Edited the script in the sample and in the package so hopefully the changes are preserved
Added a plus sign to the screen during the eyes open resting state
Added "Close your eyes" on the screen during eyes closed rs

I verified on a fresh project with Emily.  Merging into main.


Relationship to Caches & ScriptCache in ServiceWorkerGlobalScope

The relationship between caches & scriptcache to each serviceworkerglobalscope is not clear in the spec:

Do many serviceworker versions point to the same caches?  Yes would make sense.
It seems like there should be one script cache per service worker version (due to different scripts being loaded per version).  Is this the case?

Since it seems like there are different answers here, then this means we'll be needing slightly different cache architecture for script vs normal caches.  I know currently this is very much the case, as script cache isn't even hooked up (and it lives on it's own SimpleCache).

There is no concept in the ServiceWorker life cycle of some external script on the main thread modifying the Cache.  Its not like the SW would reload from that, etc.
I think we should remove scriptCache now and maybe add it back if we have the concept of a read only Cache.
We're not implementing it in gecko for the foreseeable future. @inexorabletash also says "no plans to implement it any time soon in blink, either".

Done! https://github.com/slightlyoff/ServiceWorker/commit/88e970d7c17e6871f48a490d14e9a693d94e89d0
I think it's a good feature, and may be brought back to help with https://github.com/slightlyoff/ServiceWorker/issues/639. We should at least have script cache as a concept to explain how and when importScripts scripts are cached.

Yes, I spec'ed the internal slot, script resource map for now which stores the importScripts scripts https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#dfn-script-resource-map.
We can use this internal slot the spec the relevant attributes when needed.
Closing the issue for now.


[ Discuss ] MQTT MqttPubAckMessage should be recyclable?

Motivation
Relate issue:  https://github.com/streamnative/mop/issues/388
Because of a lot of PUBACK response will produce many object. therefor, we want to reduce GC pressure by recycle MqttPubAckMessage.
But the current condition is we can not recycle this object, because of this object all field is immutable.
Do you have any good suggestion for this or could i submit a PR to add RecyclableMqttPubAckMessage ?
Netty version

4.1.67.Final

Thanks
Mattisonchao

@chrisvest  @normanmaurer
Could you give me more suggestion when you have time ?

The MqttPubAckMessage object itself only has 4 reference fields, and the attached headers also look pretty small.
Adding a Recycler.Handle will add some overhead, and you'll need to remember to call recycle on them.
To make it worth it, you'd also have to make the fixed and variable headers mutable, which goes against how the codec has been designed with all messages being immutable.
So this sounds like a bigger change, unless you can make simplifying assumptions about those fixed and variable headers.

@chrisvest
We will reconsider this part.
Thanks for your answer. i will close this issue.


Adding init to fix zombie process in builder and gameplay

Added init to docker-compose on another branch


Allow implicitly typing class methods and constructor

Suggestion
In TypeScript's documentation of classes, it says:

In this example, we perhaps expected that s’s type would be influenced by the name: string parameter of check. It is not - implements clauses don’t change how the class body is checked or its type inferred.

I discovered while trying to type a .js class with @implements that while I thought the issue was with the JSDoc keyword, the issue is with implements itself. There is currently no way to implicitly type the methods of a class with a single definition. This would be IMO a great feature for TypeScript, especially as some libs move to lean more on JSDoc.
🔍 Search Terms

@implements implements
✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[ ] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
Allow implicit typing of the methods of a class from one place
📃 Motivating Example
So, take this example:
export interface Print {
  prototype: {
    print(txt: string): void;
  }
}

/** @class */
export const TextBook: Print = function() {}
TextBook.prototype.print = function(txt) {}

You'll note if you type-check this code that txt does not need explicit typing. However, AFAICT, there's no way to type a class this way in TypeScript. Meaning that classes actually have less TypeScript support (in this instance) then the traditional JavaScript pattern that class syntax often replaces.
In fact, as noted in the above documentation, TypeScript specifically does not provide this functionality even while noting that developers might expect this to be supported in some way (if only because the prototype syntax is supported).
In other words, there is no way to type the print method here using an external class, which makes type-checking a .js file with JSDoc more difficult than its .prototype. equivalent, or making a single type for a class nearly impossible. As in:
export const TextBook2: Print = class {
  print(txt) {}. // the type of `txt` will not be inferred
}

Note also that using .prototype. allows us to check for exact types and excess properties, whereas the class pattern offers no equivalent:

Again, this infers that TypeScript currently has no way to accurately describe the shape of a class externally, even though this seems to available for all other primitives and built-in objects.
💻 Use Cases
I'd like to be able to describe a class using a single JSDoc @type import, or, similarly, use a declaration like const Foo: MyClass = class {} to correctly type all properties, methods, and the constructor. This would maintain parity with what a developer can do right now in TypeScript with function prototypes.
Possible workaround
If, for some reason, there's some syntactic reasoning behind not fully supporting class types, it would be great if we could somehow tell TypeScript to support a type shape by describing it's constructor / prototype without actually using classic function prototypes. In other words, do what TypeScript can already do right now with Function.prototype, but apply it to the class syntax.

Looks like a duplicate of #32082 (presumably if implements clauses provided context for inference then your problem would be resolved?)

@jcalz I guess so. My motivating examples are a little different. That is, the discussion there seems focused on what classes should or shouldn't do based on the behavior of classes. My argument is what classes should do based on the behavior of every other type of primitive in TypeScript, with classes as the far outlier.
Also, I'm not proposing that the behavior of implements changes, so much that, if assigning an explicit type, the class object abides by that type. Currently, it doesn't. That to me is clearly a bug, or at least an aberration in the syntax.

I would consider this a duplicate of the linked issue as well. Currently, method declarations cannot be subject to contextual typing, and if we did allow that, then this would just all work as anticipated.


Appsync Events API with Existing Amplify Config

Amplify CLI Version
12.13.0
Question
I am trying to use AWS APPSync Event API - however it involves doing a Amplify.config step.
I think there is an error or something. I tried this in gen1 and in gen2.
The documentation says to do something like this:
import config from './amplify_events_outputs.json';

Amplify.configure(config);

but there are already existing values that i need so i tried to do this instead:
import outputs from '../../amplify_outputs.json';
import config from './amplify_events_outputs.json';

Amplify.configure({...outputs,
...config
});

It seems like I can have one or the other not both, but one has my other api amplify settings that i set up.
Again i tried this in gen1 and gen2

Hi @akilisosa thanks for raising this issue.
Are you referring to the steps on this page?
https://docs.amplify.aws/react/build-a-backend/data/connect-event-api/

Yes. However I am trying it with the angular framework. (lol i should switch to react since there is such a demand for it in the job market).
I did see this recommendation in the gen1 to use AWSAPPsyncClient:
https://docs.amplify.aws/gen1/angular/build-a-backend/graphqlapi/upgrade-guide/
which I can try next.
when I do it the way the documentation suggests i get the error: "did you run AWS.config()?" probably because of the conflicting options.

so after fighting all the other documentation i found this one:
https://docs.aws.amazon.com/appsync/latest/eventapi/event-api-getting-started.html
and that worked for me. :P

oh okay! thank you for letting us know! I'll close this issue since you're unblocked but just curious what that documentation might've had that ours was missing when it comes to angular? i can create a separate task to improve the docs specifically for Angular to prevent this from happening to others using the same framework.

@chrisbonifacio It is not angular specific. Unless - for some reason it works in react. The issue is that I would be using two end points, and amplify configure only works for one (as far as I'm aware) to get around this issue - I had to create a separate web socket connection using the traditional method wss:// etc... doing this . The documentation showed what methods i and information i needed to use and provide to set that up. If you did add that to the docs i would say something like for more advance configurations or control see this documentation...
In react - there are lots of different ways to set up the project, you might be able to do amplify.configure() in every service and get it set up that way, however if you just did it in one place at the start of the app, or in a providers component you would probably run into the same issue.
I just started working in react / redux / next.js for that other project you should see the issue i raised earlier today.


Improve measures layout

Looks like this:

Note that 2 or 3 of the descriptions (the ones that include BNF codes - the second in that screenshot) are unpleasantly long to function as chart titles. I propose that we deploy anyway (it's an improvement overall) and create a new issue to show a list of the BNF codes used in the numerator and denominator as a mouseover or similar (so we're consistent across all measures)
Closes #1039, and closes #1040

The new key looks great, but just noticed that you can click on the items to make them disappear on the graph. This works fine apart from for the deciles, as only the first decile is associated with the key item. I imagine this is a non-trivial fix? Can the key be non-interactive?


[Core] Docs Changes Needed from dbt-core Issue #10673

At a minimum, update body to include a link to the page on docs.getdbt.com requiring updates and what part(s) of the page you would like to see updated.

Hiya @nghi-ly
It looks like this was assigned by a community member and from this Git issue / link: https://github.com/dbt-labs/dbt-core/issues/10673, it looks like this has been resolved. I don't think this was discussed with Doug last week but if there are any issues which have been raised by the community and resolved, are we ok to close these off?
Kind Regards
Natalie

thanks for flagging, @nataliefiann !
i took a look at that issue. doesn't look to be a v1.9 to me.
we could document that code example jeremy shared. maybe as an FAQ somewhere, an example code block in one of the reference pages, or both? does this help?


Add authkit provider

Hey @flaviouk, thanks for the PR! Nice that you're using Authkit 😄
One thing to note here is that authkit is only a valid provider via the user_management/authorize API.
It's not listed as a valid provider in the sso/authorize endpoint, although it might be working for you, my apologies for the confusion. We'll remove that from our side since authkit is only supposed to be used in the User Management API side.
We are actively working to get a major version of the Elixir SDK shipped with a UserManagement module: https://github.com/workos/workos-elixir/pull/52

Hey @flaviouk, thanks for the PR! Nice that you're using Authkit 😄
One thing to note here is that authkit is only a valid provider via the user_management/authorize API.
It's not listed as a valid provider in the sso/authorize endpoint, although it might be working for you, my apologies for the confusion and will remove that on our side.
We're actively working to get a major version of the Elixir SDK shipped with a UserManagement module: #52

Thanks for the feedback Laura! I’ll close this PR then, and will change things on my end to not use the sso api (had no idea this wasn’t supposed to work 😅)
On the bright side, WorkOS has been extremely simple to work with! Is there an ETA on the v1 branch?

Hey @flaviouk, thanks for the PR! Nice that you're using Authkit 😄
One thing to note here is that authkit is only a valid provider via the user_management/authorize API.
It's not listed as a valid provider in the sso/authorize endpoint, although it might be working for you, my apologies for the confusion and will remove that on our side.
We're actively working to get a major version of the Elixir SDK shipped with a UserManagement module: #52

Thanks for the feedback Laura! I’ll close this PR then, and will change things on my end to not use the sso api (had no idea this wasn’t supposed to work 😅)
On the bright side, WorkOS has been extremely simple to work with! Is there an ETA on the v1 branch?

We're targeting to release until the end of the year! 😄


Update s3, sdk-core, sqs to 2.17.99

Updates

software.amazon.awssdk:s3
software.amazon.awssdk:sdk-core
software.amazon.awssdk:sqs

from 2.17.38 to 2.17.99.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #380.


How to add source address to the existing SecurityRule？

How to add source address to the existing SecurityRule？

So, looking at the docs we see that the field for source addresses is named source, which is a list according to the docs.  So, we just append a value to that list then save the rule.
Pretending we already have a security rule "foo" on our firewall:
from panos.firewall import Firewall
from panos.policies import SecurityRule, Rulebase

fw = Firewall('<IP_ADDRESS>', 'admin', 'secret')

base = Rulebase()
fw.add(base)

rule = SecurityRule("foo")
base.add(x)

rule.refresh()

rule.source.append('<IP_ADDRESS>')

rule.apply()


ModuleNotFoundError: No module named 'qa_mdt'

I am trying to install this locally on Ubuntu 22.04 but getting error. Are you able to help me please?
Following are the commands I am following with error:
conda create -n om python=3.11 -y && conda activate om
git clone https://github.com/ivcylc/qa-mdt.git && cd qa-mdt
pip install -r gradio/requirements.txt
python3 gradio/gradio_app.py
Traceback (most recent call last):
File "/home/Ubuntu/qa-mdt/gradio/gradio_app.py", line 19, in
from qa_mdt.pipeline import MOSDiffusionPipeline
ModuleNotFoundError: No module named 'qa_mdt'
(om) Ubuntu@0041-kci-prxmx10171:~/qa-mdt$

Hi, the gradio_app is made by @jadechoghari
But i think that, you should add:
import sys sys.path.append('/home/Ubuntu')

@fahdmirza Try commenting out this line in app.py: os.system('git clone https://huggingface.co/jadechoghari/openmusic').
Instead, clone it directly inside the gradio folder: git clone https://huggingface.co/jadechoghari/openmusic. Also you after it's cloned rename openmusic to qa_mdt as mentioned in the model card documentation: https://huggingface.co/jadechoghari/openmusic., then run app.py again.
I'll update the Gradio documentation to make it clearer. (It's a naming issue).


Expose DebugOptions and LogLevel through torch_ort

This pull request:

Updates __init__.py to expose DebugOptions and LogLevel through torch_ort.
Updates experimental __init__.py to expose load_from_json function.
Adds unit tests for DebugOptions and experimental.json_config.load_from_json.

we don't need to run these tests in the packaging pipeline. If we do set up the pipeline for CI, then we should probably add these tests to that.


Blank page after TOC in PDF in single page layout (with solution)

When using a single-page layout (<xsl:variable name="mirror-page-margins" select="false()"/>), there is always a blank page created after the TOC, at least when the TOC is on a single page. I haven't tested this with a 2 page TOC.

DITA-OT
2.4.6 and earlier

Formatter
FOP

To fix this, the following template in toc.xsl need to be modified:
<xsl:template name="createToc">
  <xsl:if test="$generate-toc">
    <xsl:variable name="toc">
        <xsl:choose>
            <xsl:when test="$map//*[contains(@class,' bookmap/toc ')][@href]"/>
            <xsl:when test="$map//*[contains(@class,' bookmap/toc ')]">
                <xsl:apply-templates select="/" mode="toc"/>
            </xsl:when>
            <xsl:when test="/*[contains(@class,' map/map ')][not(contains(@class,' bookmap/bookmap '))]">
                <xsl:apply-templates select="/" mode="toc"/>
                <xsl:call-template name="toc.index"/>
            </xsl:when>
        </xsl:choose>
    </xsl:variable>
    <xsl:if test="count($toc/*) > 0">
        <fo:page-sequence master-reference="toc-sequence" xsl:use-attribute-sets="page-sequence.toc">
            <xsl:call-template name="insertTocStaticContents"/>
            <fo:flow flow-name="xsl-region-body">
                <xsl:call-template name="createTocHeader"/>
                <fo:block>
                    <fo:marker marker-class-name="current-header">
                      <xsl:call-template name="getVariable">
                        <xsl:with-param name="id" select="'Table of Contents'"/>
                      </xsl:call-template>
                    </fo:marker>
                    <xsl:copy-of select="$toc"/>
                </fo:block>
            </fo:flow>
        </fo:page-sequence>
    </xsl:if>
  </xsl:if>
</xsl:template>

The line
<fo:page-sequence master-reference="toc-sequence" xsl:use-attribute-sets="page-sequence.toc">

has to be changed to:
<fo:page-sequence master-reference="toc-sequence" xsl:use-attribute-sets="page-sequence.toc" force-page-count="no-force">

I'm unsure whether this has side effects. But at least, it removes the blank page.

Yes, it has side-effects, so we can't modify the code like that. Also, the correct place to make a modification like this is in the page-sequence.toc attribute set.


Send layers of struct over TCP, possible?

I have managed to send a simple struct over TCP.  Now, I am facing a new challenge. I need to send multiple layers of struct over TCP. Is that possible in C using your library? Multiple layers means struct A contains another struct B. Inside struct B, it contains struct C and so on.

I guess you need to learn about JSON structures.
And check the Binn usage documentation.


width and height is not matched correctly when setting with a decimal

The configureSvgSize method has a problem with the regular expression matching width and height. The width and height attributes with decimals cannot be correctly matched
in packages\utils\src\loader\utils.ts:5
such as
<svg width="250.666" >
    ...
</svg>

it will get
<svg width="1em" width="250.666" >
    ...
</svg>

Thanks for pull request. Published version 2.1.14 with fix.


Add Tools link and landing page for GCViT instances

I'd like to get the GCViT instances linked from the tools menu.
Since we have four instances (and there will likely be more coming), I am thinking that we should have a landing page for them - maybe tools/gcvit/ ... which would have some instructions and pictures, much like the gcv landing page.
https://gcvit.soybase.org
https://gcvit.phaseolus.legumeinfo.org
https://gcvit.arachis.peanutbase.org
https://gcvit.cicer.legumeinfo.org
Intro text:
"GCViT: a multiple chromosome visualisation tool for visualizing resequencing or SNP array data. GCViT permits comparison of multiple accessions to a reference accession within a single dataset, to visually identify regions of similarities and differences across the genome."
There is also a tutorial ... which should probably be redone to be made more generic (and briefer):
https://youtu.be/B2gPVUipWo0
I am willing to work on this if needed, but that may need to wait until June 6, after submission of annual report and project plan.

I'm happy to take it on and preview it on dev.

Thank you @sammyjava !

Right now we've got two tools categories: "Browse and Search" and "Sequence search". I'm wondering if we should add a category for more genetic-oriented tools.

How about these categories:
Download/Query
  LIS Datastore
  LIS InterMines
  DSCensor Data Overviews

Visualize/Browse
  ZZBrowse GWAS/QTL
  CMap Map Viewer
  LIS Germplasm GIS Viewer
  Conekt Comparative Expression
  GCViT Diversity Browser
  Genome Context Viewer
  Gene Family Search

Sequence search
  LIS SequenceServer BLAST
  Funnotate: Annotate Your Sequences

Works for me, I'll punch those in.

Take a look at https://jekyll-dev.lis.ncgr.org/

I like it! The categories make more sense to me. The GCViT page looks fine. I say "ship it."

One minor thing, fresh in my mind from a talk with Nevin where this came up frequently. Currently, none of the gcvit pages have links to take you back to their respective motherships. Therefore, per #66 I guess they ought to open in new tabs/windows? Otherwise looks good to me.

In production.

One minor thing, fresh in my mind from a talk with Nevin where this came up frequently. Currently, none of the gcvit pages have links to take you back to their respective motherships. Therefore, per #66 I guess they ought to open in new tabs/windows? Otherwise looks good to me.

Yeah actually I just yanked the across-the-board target="_blank" in favor of harassing people to put the LIS header on their tools. So I'll fire up an issue for that. Having a "problem" will hopefully make it get "fixed."

Sounds good


Log to tracing migration

Migrates all usages of 'log' to tracing
Adds custom serde derive for tracing::Level

Someone PLEASE try this out before merging!  It works fine as far as I can tell, but I don't know Shuttle well enough yet to be certain there's not a huge chunk of logs somewhere missing.

Def no merging, looks like it broke some rando tests comparing log output

Self-reminder to tag the rest of the serde alternative Layer defs based on https://www.reddit.com/r/rust/comments/w3q1oq/things_i_wish_i_had_known_about_serde_json/ before this gets merged

Thanks for starting on this, but this has been completed in other PRs. :slightly_smiling_face:


docs: add langfuse

docs: add langfuse
(oss platform featuring traces, evals, prompt management and metrics to debug and improve your LLM application, 2,3k stars)

Appreciate!


[BUG] unexpected DeprecationWarning of pkg_resources from script (dev).tmpl

setuptools version
setuptools===68.0.0
Python version
Python 3.8
OS
Linux
Additional environment information
No response
Description
Since setuptools group has offically claimed that 'pkg_resources' package was deprecated at here, the use of pkg_resources by the users are reasonable to report a DeprecationWarning, however the use by setuptools itself should not report such warning as I think.
When I install by pip install -e with -e option, the installed script is generated by setuptools/script (dev).tmpl where the following statement exists:
__import__('pkg_resources').require(%(spec)r)

Therefore, I get an unexpected warning when I run the installed script:
...: DeprecationWarning: pkg_resources is deprecated as an API. See ...

A possible solution is to remove
__import__('pkg_resources').require(%(spec)r)

in setuptools/script (dev).tmpl (and probably setuptools/script.tmpl).
Expected behavior
When running a script installed by pip install -e, I expect no DeprecationWarning triggered by __import__('pkg_resources').require from setuptools/script (dev).tmpl.
How to Reproduce

Create a project named test_pkg_resources, with two files:

test_pkg_resources
  |- setup.py
  |- testrun

where, setup.py is:
import setuptools
from os.path import abspath, dirname, join
repo = dirname(abspath(__file__))
setuptools.setup(
    name = "test-proj",
    scripts=[join(repo, "testrun")],
    python_requires=">=3.7",
)

and testrun is:
#!/usr/bin/env python3
if __name__ == "__main__":
    print("Hello!")

run command to install with option -e:

python3 -m pip install -e test_pkg_resources/ --user

run installed script:

testrun

then the unexpected warning appears.

Output
.../bin/testrun:4: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
  __import__('pkg_resources').require('test-proj==0.0.0')
Hello!

Hi @kenhkl, I believe that with a sufficiently updated version of pip will should not be seeing that message.
This is likely to be happening because pip is triggering a deprecated code path.
Since the implementation of PEP 660, editable installs are expected to trigger the editable_wheel command instead of develop.
You can also force pip to use the latest (and not deprecated) behaviour with --use-pep517 flag for pip install.

Thanks @abravalheri, really looking forward to a new version of pip.
By the way, a confussion: since my occasion is to use pip to do editable installs, so your suggestion to use --use-pep517 seems not helpful to avoid the pkg_resources DeprecationWarning mentioned above.

You can also force pip to use the latest (and not deprecated) behaviour with --use-pep517 flag for pip install.

Have you tried to use pip install --use-pep517 -e .? What is the output that pip provides if you run pip -v install --use-pep517 -e .?

@abravalheri Sorry, I did not present my point clearly.
Yes, the warning is missing when I run python3 -m pip install --use-pep517 -e test_pkg_resources/ --user. And the modification on other modules called by testrun can be performed on-the-fly. However, this way differs from using python3 -m pip install -e test_pkg_resources/ --user that the modification on testrun cannot be performed dynamically.
So, if one wants such testrun to be editable installed, then a new version of pip is needed.

Have you tried to use pip install --use-pep517 -e .? What is the output that pip provides if you run pip -v install --use-pep517 -e .?

Hi @kenhkl, unfortunately I don't know how the testrun script works, but I would like to avoid touching the template that is used by easy-install, because it is hard to fully understand how that would unfold in the ecosystem (there might be old packages out there that depends on this behaviour).
There are other ways to convince pip to use the new APIs for build isolation (which will imply that the new editable_wheel command runs instead of the deprecated develop), would you consider trying one of the following:

Add a pyproject.toml file with [build-system], e.g. https://setuptools.pypa.io/en/latest/userguide/quickstart.html#basic-use
Export a PIP_USE_PEP517=1 environment variable (that should have the same effect as --use-pep517).

Thanks @abravalheri, I now have enough knowledge to deal with the unexpected warnings so far.

I believe that with a sufficiently updated version of pip will should not be seeing that message.

I am using the latest pip
$ pip -V
pip 23.2.1 from /home/user2/py310/lib/python3.10/site-packages/pip (python 3.10)

but our script az installed by pip install -e src/azure-cli still shows this warning:
$ az -v
/home/user2/py310/bin/az:4: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
  __import__('pkg_resources').require('azure-cli==2.51.0')

Hi @jiasli , I believe that pip 23.1 was supposed to use isolated builds by default (the new APIs that setuptools already implements), but they decided to post pone a little bit.
Please have a look on my previous comment for alternatives.

Have you tried to use pip install --use-pep517 -e .? What is the output that pip provides if you run pip -v install --use-pep517 -e .?

This actually worked for me, thanks!

Have you tried to use pip install --use-pep517 -e .? What is the output that pip provides if you run pip -v install --use-pep517 -e .?

This actually worked for me, thanks!

Pls help
~/dirsearch $ pip -v install --use-pep517 -e
Usage:
pip install [options]  [package-index-options] ...
pip install [options] -r  [package-index-options] ...
pip install [options] [-e]  ...
pip install [options] [-e]  ...
pip install [options] <archive url/path> ...
-e option requires 1 argument

Have you tried to use pip install --use-pep517 -e .? What is the output that pip provides if you run pip -v install --use-pep517 -e .?

This actually worked for me, thanks!

Pls help
~/dirsearch $ pip -v install --use-pep517 -e
Usage: pip install [options] [package-index-options] ... pip install [options] -r [package-index-options] ... pip install [options] [-e] ... pip install [options] [-e] ... pip install [options] <archive url/path> ...
-e option requires 1 argument

you're missing a ., -e shouldn't require any args but pip install definitely does


feat(metrics): send bitswap metric to prometheus

Enable exporting bitswap metrics to prometheus

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 45.65%. Comparing base (2469e7a) to head (5b02c87).
Report is 183 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3701      +/-   ##
==========================================
+ Coverage   44.83%   45.65%   +0.82%
==========================================
  Files         265      281      +16
  Lines       14620    16025    +1405
==========================================
+ Hits         6555     7317     +762
- Misses       7313     7869     +556
- Partials      752      839      +87

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


Maybe a mistake

If you have many options to choose from, use a switch statement. A switch statement tests a value and can have many case statements which defines various possible values. Statements are executed from the first matched case value until a break is encountered.
Isn't it "which define"?
Challenge Selecting from many options with Switch Statements has an issue.
User Agent is: Mozilla/5.0 (iPad; CPU OS 10_0_2 like Mac OS X) AppleWebKit/602.1.50 (KHTML, like Gecko) Version/10.0 Mobile/14A456 Safari/602.1.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

function caseInSwitch(val) {
  var answer = "";
  // Only change code below this line

  // Only change code above this line
  return answer;
}

// Change this value to test
caseInSwitch(1);

@marienhof nice catch! Yes, it should be "...which define..." instead of "...which defines..."
- ... statements which defines various possible values.
+ ... statements which define various possible values.

I will take this

@chakrihacker are you working on this?

@erictleung looks like @rheupler already fixed this in the seeds https://github.com/FreeCodeCamp/FreeCodeCamp/commit/8fb284d5d1fa01a7a647eed046c8245f38e2d9e5

@ryzokuken thanks for checking that out. I neglected to check the code base. Thanks! This issue appears to be resolved so I'll be closing this issue. Happy coding!