＋＋[ＷａｔｃｈＬｉｖｅ]ＮＦＬ！]＊ Ｃｏｗｂｏｙｓ ｖｓ Ｂｒｏｗｎｓ Ｌｉｖｅ ＳＴｒｅａｍｓ ＯＮ ＴＶ Ｃｈａｎｎｅｌ ０８ Ｓｅｐ ２０２４

What time, TV channel is Dallas Cowboys vs Cleveland Browns game on today? Free live stream, odds
Published: Sep. 08, 2024, 7:07 a.m.
🔴📺 GO LIVE==►► CLICK HERE TO WATCH LIVE
🔴📺 STREAMING==►► CLICK HERE TO WATCH LIVE
🔴📺 GO LIVE==►► CLICK HERE TO WATCH LIVE
🔴📺 STREAMING==►► CLICK HERE TO WATCH LIVE
By John Holland | Special to The Oregonian/OregonLive
Dak Prescott and the Dallas Cowboys visit the Cleveland Browns today with plenty of question marks about these teams heading into Week 1 of the NFL football season. This game kicks off at 1:25 p.m. PT/4:25 p.m. ET on Sunday, September 8 with a live broadcast on FOX, and streaming live on demand.
• WATCH: Cowboys vs. Browns live for FREE with Fubo (free trial), or with Sling (cheapest streaming plans, $25 off first month) or see more streaming options below.
Rewind 10 seconds
Move forward 10 seconds
Unmute
0:00
/
0:00
Full screen
Marco Arop captures 1000-metre title at World Athletics Continental Tour in ZagrebMarco Arop captures 1000-metre title at World Athletics Continental Tour in Zagreb
What TV channel is the Cowboys vs. Browns game on?
When: Kickoff takes place at 1:25 p.m. PT/4:25 p.m. ET (3:25 p.m. CT) on Sunday, September 8
Where: Huntington Bank Stadium | Cleveland, Ohio
TV Channel: FOX
How to watch live stream online: If you don’t have cable, you can still watch this game live for FREE with Fubo (free trial) or with DirecTV Stream (free trial). If you are out of free trials, the cheapest and best way to watch this game and more football is by signing up for Sling (promotional offers, cheapest streaming plans), which costs around $31 for the first month if you add the “Sports Extra” package that includes Big Ten Network, SEC Network and a few other sports channels.


function: pivot and unpivot

Feature request
It's better to have pivot function to support more flexible scenario. By now, we can only use case when to do pivot, and it has bad performance. for unpivot, we can use unnest, but still need an aggregation.
https://learn.microsoft.com/en-us/sql/t-sql/queries/from-using-pivot-and-unpivot?view=sql-server-ver15

Can I have a try?

Hi @hongism, I've noticed that you are new to the StarRocks community.
If you have any questions, feel free to join WeChat for discussion, the community will assist you with the function task: https://853921.ma3you.cn/articles/b12e90J/

Hi, I am interested in PIVOT syntax.
I think we have two ways to implement it:

See PIVOT as a syntax sugar, we can transform pivot to agg(case ... when ...)
Add a new PIVOT operator

what do you think? @wangsimo0 @kateshaowanjou

Hi @Seaven
Why this issue was closed. It is not completed as we lack unpivot function.

Hi @Seaven Can I ask why this issue was closed? It is not completed as we lack unpivot function.

it's github auto-closed because merge pivot function PR, if you need unpivot, we should create a new issue

let me reopen this. still need unpivot.


std::unique_ptr::reset causes runtime error

https://github.com/GLVis/glvis/pull/153 changed the stream_state to use a std::unique_ptrs for, among other things, the GridFunction grid_f. In the web target the std::unique_ptr::reset in https://github.com/GLVis/glvis/blob/master/lib/aux_js.cpp#L193 causes a runtime error that prevents the loading of example 3:
[Log] Switching to order 1 discontinuous vector grid function... (glvis.js, line 2293)
[Error] Unhandled Promise Rejection: RuntimeError: call_indirect to a signature that does not match (evaluating 'invoker(fn, arg0Wired, arg1Wired, arg2Wired, arg3Wired)')
	(anonymous function)
	promiseReactionJob

Removing the reset and passing the pointer to the VisualizationSceneVector3d "fixes" the issue in that the rendering actually happens. Here's the diff:
--- a/lib/aux_js.cpp
+++ b/lib/aux_js.cpp
@@ -190,8 +190,8 @@ bool startVisualization(const std::string input, const std::string data_type,
          {
             GridFunction* proj_grid_f = ProjectVectorFEGridFunction(
                                            stream_state.grid_f.get());
-            stream_state.grid_f.reset(proj_grid_f);
-            vs = new VisualizationSceneVector3d(*stream_state.grid_f);
+            //stream_state.grid_f.reset(proj_grid_f);
+            vs = new VisualizationSceneVector3d(*proj_grid_f);
          }
          else

This seems like a problem with either Emscripten or how we're building the web target but I wanted to start a discussion here.

The bug seems to be a problem in the glvis core itself rather than a js-related issue; I pushed out a PR that should fix the issue.


[Bug]: Oracle Plugin issues found during QA

Is there an existing issue for this?

[X] I have searched the existing issues

Description
This is a bug with collection of issues found during the testing of Oracle Plugin PR.
Steps To Reproduce
Refer the How To Test Section in this PR
While testing, found the following -

[x] Change name of plugin from Oracle Plugin to Oracle.

[ ] If I use admin user with blank password, the test passes.

[x] Database name field should be shown as required field, since, if we leave it blank, it throws an error to the effect - missing database name.

[ ] This message is not seen on the Edit DS page. - Whitelist the IP <IP_ADDRESS> and <IP_ADDRESS> on your database instance to connect to it. Learn more

[x] The Oracle logo image seems a little off - with some whitebackground - this is not consistent with the other logos

While testing this PR, here are a few issues found:

[ ]  For the column with data type - timestamp with local time zone, interval year to month, interval day to second, raw(256),clob, nclob and blob - the values are garbled/different.

[ ] The data displayed in the timestamp field is slightly different from what is displayed on the actual DB

[ ] The data displayed in the binary_float column is different from what is displayed in the actual DB

[x] Oracle logo needs to be changed. (It does not look like Oracle)

[ ] Found an issue of queries giving intermittent older responses/incorrect responses. Following couple of videos show the behavior -https://ddpa7mfuvv.vmaker.com/record/2ZLmcsM93YRDueE7
https://ddpa7mfuvv.vmaker.com/record/BMOQcZPP7VOkbNVa

[ ] This particular select statement is giving errors, and am not able to select a row along with timestamp specified.
select * from test_ap_or where c_varchar2={{'7_varchar2'}}
and c_nvarchar2={{'7_nvarchar2'}}
and c_number={{6}}
and c_float={{161.09}}
and c_date={{'2002-11-04'}}
and c_binary_float={{161.09}}
and c_binary_double={{161.09}}
and c_timestamp=TO_TIMESTAMP({{'04-11-2002 09:26:50.124'}})
Note: The above select statement returns the correct row without the c_timestamp section added.
In this regard - we would need to investigate the usage of date time formats and details with regards this should be added to  the Oracle documentation so that the users know how to query for date time values.

[ ] At present, the documentation links turn up empty when we access Documentation from Queries page

Public Sample App
No response
Environment
Deploy Preview
Issue video log
No response
Version
DP

On re-testing, found the following:
C_BLOB datatype is inserted as oracle.sql.BLOB@1ebc038f, but in the Oracle DB it is stored [BLOB]

Closing this issue as most of the issues are resolved, the pending ones cannot/should not be addressed and the documentation is in progress:

The datetime display follows the format that comes on official oracle cloud SQL console:

There is a known issue with client approximating very large numbers and deep fractional sequences.

The raw and blob types display has been fixed, in order to read them properly uses need to write command like select utl_raw.cast_to_varchar2(c_raw) as c_raw, utl_raw.cast_to_varchar2(c_blob) as c_blob from TYPESTEST4

Created another issue to track improved usage for inserting datetime values.


The search for domain MAssive is wrong

When you click in the pie chart for massive it uses the query http://localhost:8000/app/browse.html#/search?q=MassIVE but if some datasets are also including the word Massive, then the search will highlight them, we need to change that to only search for the repository Massive.

done


View latest imagery takes you to 2013 imagery

When you click View latest imagery, it takes you to imagery in the Philippines dated in 2013. There is imagery within the system that is 2016. Correct action should be when you click the button, you are taken to the most recent, most high resolution imagery.
May be related but when you the meta endpoint without parameters, the default sorting does not seem to be returning the most recent, most high res imagery. The 2013 imagery is being returned at the top.
@danielfdsilva Do we need to just hit a correctly sorted request when we load? Something like http://api.openaerialmap.org/meta?order_by=acquisition_end&sort=dsc.

Incorrectly ticketed here. Moved to https://github.com/hotosm/oam-browser/issues/133.


Longest Common Subsequence

Program Description
Longest Common Subsequence
Additional context
Longest Common Subsequence Solution In Recursive + DP both
Checklist

[ ] This issue has not been mentioned in existing issues.

I want to work on this issue assign it to me

Assigned.


Ability to send multiple log entries per single SQS message

In larger environments the requests made to SQS to read\write messages can soon add up, coupled with AWS charging in 64kb chunks, sending (and LogStash retrieving) single log entries per SQS message can soon become expensive.
This change adds a config flag sqs_bulk_lines (as to have no adverse affect to current deployments) to allow users to group messages in a similar way to the write_bulk function currently work (grouping messages up to a max of 256kb), only they are sent as an array list in a single SQS message rather than being sent as separate messages in one transaction.

Mind rebasing this onto master?

Merged. Thanks for the heads up.

Any chance of getting this merged? @josegonzalez. A new release would be awesome, thanks in advanced.

Sorry was afk for a bit.


Should we be updating copyright dates in files that haven't otherwise been changed? Copyrights are valid for decades in most places, so doing a global find and replace in a repo just results in excess file churn.

          Should we be updating copyright dates in files that haven't otherwise been changed? Copyrights are valid for _decades_ in most places, so doing a global find and replace in a repo just results in excess file churn.

Originally posted by @gregmedd in https://github.com/eclipse-uprotocol/up-cpp/pull/68#discussion_r1532628951
I don't know why this was marked as resolved. Copyright date updates in files that were not otherwise modified should be rolled back.

Did not mean to open this as a separate issue


[Slider] Hide the track

It doesn't seem possible to style only selected/current mark label for emphasis. All labels left of the thumbnails are given markLabelActive css.  For example, users only want see rainfall chart for 2017.  The way it's styled would give the impression that previous years are also included.

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
all labels left of active one assumed active
Steps to Reproduce 🕹
https://codesandbox.io/embed/material-demo-tfbdo
Your Environment 🌎

Tech
Version

Material-UI
v4.2.1

React
16.8

Browser
chrome

For example, users only want see rainfall chart for 2017. The way it's styled would give the impression that previous years are also included.

@vnugent Thank you for sharing the use case. It's interesting, I hadn't considered it when working on the component. Take the native browser implementations, most don't show the track by default. What do you think of introducing a new disableTrack prop that would output the following:
<Slider disableTrack={true} />

pay attention to the details
vs
<Slider disableTrack={false} />

That could work.  However could we also give track a direction, eg left-of-thumb (default) or right-of-thumb instead of defaulting it to left-to-right? I can see many applications where you want to emphasize the upper range.

However could we also give track a direction

@vnugent I believe we cover this problem in #16352. The conclusion of the thread was: use the RTL mode in this case and that we should warn when min > max.

As it turns out, we can generialize the solution with a better API, see #17215.

I started to look at this and hopefully can combine with #17215
Will push some code tomorrow


Bug fix: d10_1038_s41467_020_15543_y Dataloader

Added TCM1 (Th17/iTreg) celltype to .tsv file

@felix0097 was that the only one missing?

all fixed now @le-ander


#2 add src code

I have taken the F77 code provided by @qiUip and converted to F90. I initially committed the original version of poisson.f for clarity.
NOTE: We still require a specifically formatted input file for this to work, which do not yet have

Could you also add a README with instructions on how to run the mesh generator and the solver?

I have added this now. Is there anything else needed or can we merge?


Change command to perform shalow checkout

git clone command is modified to fetch only the latest commit.
should fix https://github.com/odk-x/tool-suite-X/issues/249

Hi @linl33 I have changed the command. I also tested the command in my machine. The size of download is reduced to 4.1 mb from nearly 1 gb after this. I think the command in docs (sync endpoint manual setup) should also be modified as this saves a lot of time and data.

@Shubhamr837 Thanks for the pull request. If you can update the command in the docs that'd be great. The git command used for cloud-init should also be updated (yml files at https://github.com/odk-x/docs/tree/master/odkx-src/files).

Thanks @linl33 I will create a PR for docs and all the cloud-init files.


Commands not found and no syntax highlighting

I am running the following version of VS Code on Ubuntu 16.04

I installed the extension from "Extensions" in the Activity Bar, and was prompted to also install the dependency Msg Language Support. To try it out, I created a new workspace according to catkin_tools quickstart (I should say that the catkin init command does not create a link to the top-level CMakeLists.txt file like catkin_init_workspace does).
Everything seemed to install fine, but when I started to try the extension by running Start Core command in my opened workspace an error appeared saying "command 'ros.startCore' not found". Similar errors appear for all other commands, so it seems like none of the commands were actually installed.
Opening a CMakeLists.txt file also shows no syntax highlighting.

I haven't tried the extension with catkin tools - can you try using catkin_init_workspace? I'm guessing a .catkin_workspace file wasn't created, which is what the extension relies on.
You'll need a separate extension for CMake syntax highlighting.

You're right that .catkin_workspace file wasn't created, but it wasn't created by either catkin init or catkin_init_workspace, at least for me. Should it be automatically generated?
Creating the empty file myself made everything work fine it seems.

Yes, I'm using Kinetic. Some ROS packages were upgraded just this morning.

Sorry, I must have missed something these past years... I have always used catkin_init_workspace or catkin init, but it seems that those commands have not been necessary for a long time. After creating ~/my_workspace/src just execute the catkin_make command in the my_workspace directory and the .catkin_workspace file will be created automatically.

Glad you found the issue. Ideally the initial build wouldn't be required, but I've added a note to the README for now at least.


sqlite database upgrade without losing user data

Let's say I have a quiz app and there are two tables in my database. In the first table(questions_table) there are quiz questions. That database is bundled in my app and the file is located in Documents directory.
In the second table(starred_questions_table) there are questions starred by users. It is created when the app starts and then populated by the users which star the tough questions.
Now I want to make an update for my app and add more quiz questions to my database(questions_table). So I add a new db file to the bundle but I also don't want in any case that my users lost their starred questions when they update the app. What's the best way to solve this problem?
If I move new database file(from bundle) to documents folder the previous file(with starred questions) will be lost. Should I make two simultaneous connections(one to database file in Bundle and second to old database file in Documents) and copy just new records from questions_table to documents directory?

Another option could be to use two different databases, and then use ATTACH/DETACH (#30)


Enable CORS for API

Is your feature request related to a problem? Please describe.
I want the Laravel API to allow Cross Domain Requests from any origin
Describe the solution you'd like

Allowed methods: All
Allowed origins: All
Allowed VERBS: all

This is already built in and done, it's working for all the API routes.
You can check the following files to ensure or improve the CORS behavior in Laravel:
backend/config/cors.php
backend/app/Http/Kernel.php


feat: Support ANY/ALL subquery

This PR adds support for parsing ANY/ALL with a subquery expression. Related tests are included.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

-2 of 40 (95.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 89.515%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/parser.rs
5
6
83.33%

tests/sqlparser_common.rs
32
33
96.97%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
8981

Relevant Lines:
10033

💛  - Coveralls


datasette install -e option

As seen in LLM and now in sqlite-utils too:

https://github.com/simonw/sqlite-utils/issues/570

Useful for developing plugins, see tutorial at https://llm.datasette.io/en/stable/plugins/tutorial-model-plugin.html

Here's the diff for adding it to sqlite-utils: https://github.com/simonw/sqlite-utils/commit/ef31210bf06f920e0890e171c3198f0b0dc8d72d

Tested this locally with:
datasette install -e ../datasette-graphql

Running datasette plugins confirmed that the plugin had been installed.

Applied the same fix as here:

https://github.com/simonw/llm/issues/136


tools:summarize_graph fails to open TFHub models due to parsing errors

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): no
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 20.04
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): v2.4.0-rc4-71-g582c8d236cb 2.4.0
Python version: Python 3.8.6 (default, Oct  9 2020, 11:59:16)
Bazel version (if compiling from source): Bazelisk version: v1.6.1
GCC/Compiler version (if compiling from source): [GCC 9.3.0] on linux
CUDA/cuDNN version: not relevant
GPU model and memory: not relevant

Describe the current behavior
summarize_graph is not able to load downloaded and extracted models from TFHub e.g. https://tfhub.dev/deepmind/biggan-128/2 due to parsing errors.
$ bazel-bin/tensorflow/tools/graph_transforms/summarize_graph --in_graph='/media/paulgavrikov/DATA/noCNN/model_conversion/biggan/compare_gan_ssgan_128x128_1/saved_model.pb'  --print_structure=true
[libprotobuf ERROR external/com_google_protobuf/src/google/protobuf/text_format.cc:324] Error parsing text-format tensorflow.GraphDef: 1:1: Invalid control characters encountered in text.
[libprotobuf ERROR external/com_google_protobuf/src/google/protobuf/text_format.cc:324] Error parsing text-format tensorflow.GraphDef: 1:2: Interpreting non ascii codepoint 154.
[libprotobuf ERROR external/com_google_protobuf/src/google/protobuf/text_format.cc:324] Error parsing text-format tensorflow.GraphDef: 1:2: Expected identifier, got: �
2021-01-17 12:13:33.563477: E tensorflow/tools/graph_transforms/summarize_graph_main.cc:320] Loading graph '/media/paulgavrikov/DATA/noCNN/model_conversion/biggan/compare_gan_ssgan_128x128_1/saved_model.pb' failed with Can't parse /media/paulgavrikov/DATA/noCNN/model_conversion/biggan/compare_gan_ssgan_128x128_1/saved_model.pb as binary proto
	 (both text and binary parsing failed for file /media/paulgavrikov/DATA/noCNN/model_conversion/biggan/compare_gan_ssgan_128x128_1/saved_model.pb)
2021-01-17 12:13:33.563504: E tensorflow/tools/graph_transforms/summarize_graph_main.cc:322] usage: bazel-bin/tensorflow/tools/graph_transforms/summarize_graph

Describe the expected behavior
The tool should load the model

@paulgavrikov
This issue is more suitable for TensorFlow Hub repo. Please post it on hub repo from here. Thanks!

@paulgavrikov
This issue is more suitable for TensorFlow Hub repo. Please post it on hub repo from here. Thanks!


print requirement info

This PR prints tool requirement info for given tools. Closes #6.

It seems like there is some issue with deps detection on osx:
$ ./pdtm -i naabu

                ____
     ____  ____/ / /_____ ___
    / __ \/ __  / __/ __ __  \
   / /_/ / /_/ / /_/ / / / / /
  / .___/\__,_/\__/_/ /_/ /_/
 /_/

                projectdiscovery.io

[INF] Current pdtm version v0.0.7 (latest)
[INF] Run `source ~/.bashrc` to add /Users/marcornvh/.pdtm/go/bin to $PATH
[INF] installing naabu...
[INF] installed naabu 2.1.6 (latest)
[INF] naabu requirements:
required To install libpcap on macOS, run the following command in the terminal: sudo brew install libpcap
$ brew install libpcap
Warning: libpcap 1.10.4 is already installed and up-to-date.

We can try a more straightforward strategy, like executing the binary just downloaded and checking the output for errors, then suggesting how to install deps. What do you think?

It appears that the issue may be related to the naming and extension convention specific to the operating system. See https://github.com/projectdiscovery/utils/blob/main/syscallutil/syscallutil_test.go#L15 for more information. We can proceed with your suggestion as it seems more convenient.

@dogancanbakir I think we can improve the multi-platform check and make it more precise with additional steps (like running the tool and seeing if any error gets printed to stdout/stderr - as previously mentioned. I changed red to yellow for now since the missing dependency might prevent the tool from running under certain conditions but not being installed as we are downloading the binary directly.


Delay between consecutive publish?

Would you recommend adding a delay between consecutive client.publish() calls. If so, how much would you recommend? Is 100ms an overkill?
Great library btw 👍

As long as your server can handle it, I've been putting them consecutively in my loop and its been fine for me. :)

Me too, seems to be ok! Just wondered if there was a recommendation publish
rate.
On Sun, 14 Aug 2016, 03:25 Kenny Shaw<EMAIL_ADDRESS>wrote:

As long as your server can handle it, I've been putting them consecutively
in my loop and its been fine for me. :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/knolleary/pubsubclient/issues/186#issuecomment-239652264,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuUPHgeV8o7NBEoyYJP_R4EybtVU_0lks5qfnyCgaJpZM4Jgob9
.

I can send at least 10 messages/s with an esp8266 to a mosquitto broker via a wifi connection.

Hi I have been experience a similar issue. I have been using Serial communication between arduino Mega +  GSM module to connect to the MQTT client but I can't send more than 9-10 Payloads(strings) per sec. Also, to ensure reliable data transmission I explicitly had to give a delay of 100-150ms. Could you help me with this?

I use delay with 50ms and it works fine.

The size of each of my payload string is about 400 bytes.  And with a lesser delay, I see "Publish Ok" at my terminal but the after about 15-20 payloads the data is not received at the Cloudant. Where do you think is the issue?


Update live-event-error-codes.md

Updated this RTMP errors page with more accurate wording.

@ningl2 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ningl2 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

Hi @IngridAtMicrosoft - Microsoft style is to use contractions and to avoid "please". Before merging, I'll reverse the changes in the pull request that don't align with Microsoft style; is that OK with you?

Hi @ningl2 - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? Thank you!

Thanks @ktoliver !

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#please-close


Android V2 embedding & Language Change

Hello,
I just installed the plugin and works perfectly on my with 2 exceptions

First I get this below message when the application builds which currently is not an issue but looks like it could be in the future
The plugin flutter_bluetooth_basic uses a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to see if this plugin
supports the Android V2 embedding. Otherwise, consider removing it since a future release
of Flutter will remove these deprecated APIs.
If you are plugin author, take a look at the docs for migrating the plugin to the V2
embedding: https://flutter.dev/go/android-plugin-migration.

Secondly I wanted to use it for printing receipts in Thai (Different language) but it kicked back an error "Invalid argument (string): Contains invalid characters.: "กหฟห""

I would appreciate if you could provide some help!

same issue

same issue

Same here

waiting for update :(

same issue

anyone solve this problem ,yet ? 😞

Any solition guys ?

I think the author went to vacation or something lol

No solution?


Look into steps that can be potentially skipped/optimized/cached

While the main issue is the W3C HTML spec simply being extremely large, it would be worth looking into, as building the W3C HTML spec takes a ridiculous amount of time.
Any possibilities of getting this faster would be nice. (especially for CI)

Profile report for later reference:
https://gist.github.com/cynthia/04c4d2122454fb26f06b7b9931b80b68

I'm not good at reading raw cProfile, but I have better profiling front-ends that I can run at work. (bikeshed profile invokes them; bikeshed profile -h tells you what additional programs need to be installed for it to work).
HTML is clearly banging on some N^2 behavior that nobody else is exercising nearly as hard; the slowest spec I know of besides this is DOM at about a 10s processing time, but processing HTML on my laptop today took 3m20s. That's totally ridiculous and clearly something fixable; I was assuming that y'all were hitting maybe a 20s or 30s build time, which is much slower than I'm happy about but still reasonably attributable to just "you have a big spec".  What you have instead is clearly a perf bug somewhere that I definitely need to fix, and in my experience this sort of thing is usually quite easy to find and fix.

All right, analyzing it a little bit, there are three big points:

~29 seconds in the HTML parser
~37 seconds in the HTML serializer
~150 seconds in cleanupHTML()

The first one I can't really touch, except by perhaps removing more inline uses of the HTML parser. I've killed most of these, but can see if I can find more; I doubt this'll remove more than a tiny amount.
The second one is probably capable of being fixed; I wrote the serializer to be easy to write and understand, not necessarily to be very efficient, and I bet there's some nice low-hanging fruit there.
The third one, tho, is definitely fixable.  That method is just running tons of selectors and doing tiny bits of work in the loops. It is not hard to instead rewrite this into a simple tree-walk that just checks each element for all the relevant conditions and does the work; at least, most of this can be simplified that way.
So a PR to simplify that method would be very appreciated, or I'll take care of it next week.

And it turns out cleanupHTML was only doing real simple selectors, so I was able to convert the entire thing into a single tree-walk that just checked properties of each element. The build on my work machine went from 2m8s to 1m16, a 40% reduction in running time.  It's still taking up a good chunk of runtime, so there's more room for improvement, but we're on our way.

I'm assuming this is https://github.com/tabatkins/bikeshed/commit/9180ad2419eb198fef2f20c8aa575acf30d8670a ? Awesome work, thanks for looking into this. (I was meaning to, but jetlag and laziness kicked in.)

Been poking at more obvious small spots, and got the runtime down to 58s on my machine now. That's another 20% reduction! There's still a number of similar spots I can poke at for a second or two apiece.


Export tweeter data throws error

I assume this is just calling attention to an unimplemented feature, but I figure I'd capture it anyway.
From any Twitter user account page (e.g., http://<IP_ADDRESS>:3000/twitter_users/14af5ec4-ee6d-4329-8ea8-ccb990ea88da), if you click the download arrow next to "Archived Media Items", Rails throws the following error:
NameError in TwitterUsersController#export_tweeter_data

undefined local variable or method `vid' for #<T::Private::Methods::DeclBuilder:0x000000010fc76f40 @decl=#<struct T::Private::Methods::Declaration mod=TwitterUsersController, params=#<Object:0x00000001084c5820>, returns=#<Object:0x00000001084c5820>, bind=#<Object:0x00000001084c5820>, mode="standard", checked=#<Object:0x00000001084c5820>, finalized=false, on_failure=#<Object:0x00000001084c5820>, override_allow_incompatible=nil, type_parameters=#<Object:0x00000001084c5820>, raw=false>>

Did you mean?

    void

Extracted source (around line #11):

10  # Exports all media items created by the currently viewed Twitter user to a JSON file
11  sig { vid }
12  def export_tweeter_data
13    tweet_archive_items = ArchiveItem.includes(archivable_item: [:author])
14                                     .where(archivable_item_type: "Sources::Tweet")

lol indeed, I stumbled upon the single sig { void } somehow. Will patch!


Unhandled Exception

Could you provide some code that makes this error occur, and also an excel file?
This will help us find the bug easier.


Train 169

High Level Description

Fix Partial download of dcos_generate_config.sh results in unrecoverable error that requires manual intervention. #1516
bump(DCOS-UI): latest bits #1540
Update DC/OS to the latest cosmos #1591

@mesosphere-mergebot bump-ee

@mellenburg:
[20:34:39]	[Step 4/5] [2017-06-01 03:34:39,543|dcos_test_utils.aws|WARNING]: No recognized resources found; using generic stack
[20:34:39]	[Step 4/5] Failed to execute script cli
[20:34:39]	[Step 4/5] Traceback (most recent call last):
[20:34:39]	[Step 4/5] File "dcos_launch/cli.py", line 154, in <module>
[20:34:39]	[Step 4/5] File "dcos_launch/cli.py", line 146, in main
[20:34:39]	[Step 4/5] File "dcos_launch/cli.py", line 135, in do_main
[20:34:39]	[Step 4/5] File "dcos_launch/util.py", line 98, in test
[20:34:39]	[Step 4/5] File "dcos_launch/onprem.py", line 138, in describe
[20:34:39]	[Step 4/5] File "dcos_launch/onprem.py", line 44, in get_onprem_cluster
[20:34:39]	[Step 4/5] File "dcos_launch/aws.py", line 148, in get_hosts
[20:34:39]	[Step 4/5] AttributeError: 'CfStack' object has no attribute 'get_host_ips'

The previous failure were due to AWS not returning cloud stack. I have re-triggered our tests.

@mellenburg  @lingmann @spahl  - please review and merge this.

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label ship-it ( it's already merged).


ScrollEvent on HeaderPanel

Let's say you a build an application with components that need to observe the scroll position to implement sticky behaviors etc. Since using HeaderPanel will not use the window element but it's own element for scrolling, there is no way we can get the scroll event back to our custom elements isn't there?
So if I were to create my own table with a sticky behavior, I need to know about the scroll position of it's parent container.
ready:function(){
        window.addEventListener('scroll', function(){
          console.log('scrollling');
        });
}

I understand we have the on-content-scroll event on the element itself. But what if we want to create components that need to be aware of the scroll event rather than integrating the listener in the App namespace itself?
For instance, is there any way we can get a scroll event from a paper-panel not a header-paper-panel? I needed to nest these two in my current project.
this will never print scrolling to the console. Any thoughts?

This issue tracker is for the core Polymer library, not elements built with it.  We'll move this to the appropriate tracker for the element.

Ok thank you. I've found a workaround by firing a custom scroll event on the dom-bind app node which all children can then register to. However, I wish there was a more elegant way to do it.


Alpha background color

Currently the background option supports RGB colors to set a background color behind a thumbnail, but it would be very useful for us to also support alpha channel (RGBA) background colors. For example, you could set a partially-transparent background of #99999933 which would result in a partially transparent thumbnail for formats that support it (PNG, WebP, etc). Formats that don't support it (e.g. JPEG) would probably just lose the alpha channel like they already do when the source image contains transparent pixels.

Hey Stephen!
The main purpose of the background option is to remove alpha. I don't even sure if vips_flatten would work with colors with alpha channels. Could you give me the use case for the background with alpha?

One example would be to supply a single color that would be transparent if the returned format supports it and a desired color if not; for example, having a background of #AAAAAA00 would be transparent if a PNG is requested, but gray if a JPEG was requested. Another example would be embedding something on a website where a slight gray shading is desired with a partially-transparent gray background color behind any areas that were originally transparent in the image. Since these links are often generated en-masse without any specific knowledge about individual images being used, this is sometimes a useful feature.
This is an existing feature that we provide customers, and we were hoping that we could transition our image backend fully to imgproxy without removing any customer features.

Do I remember right that you use the Pro version?

That's correct, this is for @Widen.

I'll add background_alpha (bga) option to Pro then.

@sagebind I added background_alpha to the latest build of Pro, you can test it. The new version will be available soon.

@DarthSim We tried out the background_alpha option and it works great for the image itself, but it doesn't work correctly on the padding when using resizing_type:fit. The padding is opaque and either solid black or solid white depending on the source image.


Remove old/unnecessary reboot bug fix

From initial testing, this bug ("transparent button background results in a loss of the default button focus styles") doesn't seem to manifest itself anywhere in Bootstrap (since we don't just set transparent background anywhere on buttons, and when we do set explicit button styles in the more specific stylings, we already do create a custom :focus style anyway)
Incidentally, this forced definition of :focus is biting us in the rear because it essentially circumvents the browser logic that only applies focus outline based on the browser's heuristic (i.e. not applying the button outline when clicked with a mouse). this forces the focus style to always show even after a mouse click, which is causing is hassle that we then need to work around - see https://github.com/twbs/bootstrap/pull/32630

Testing with Firefox (and IE, though of course v5 isn't aimed at it, but the principle is the same for v4.x which does include it) on https://deploy-preview-32631--twbs-bootstrap.netlify.app/docs/5.0/content/reboot/#forms and https://deploy-preview-32631--twbs-bootstrap.netlify.app/docs/5.0/components/buttons/#outline-buttons and various other places, I see no issues with "transparent button background	results in a loss of the default button focus styles" per the comment. It may well be that we've been cargo-culting this fix unnecessarily.

just want to wait/check with others like @mdo who may know of the original reason that patch was added in the first place... (or i could go on an archeological expedition to find the commit where this was first added)

In the future it should be possible to use :focus-visible to only show focus styles when the browser determines they are necessary however support is currently not perfect

In the future it should be possible to use :focus-visible to only show focus styles when the browser determines they are necessary however support is currently not perfect

i'd say it's worth backporting as it shouldn't actually affect anything in v4. and other PRs that i think are worth backporting as well (e.g. https://github.com/twbs/bootstrap/pull/32630#pullrequestreview-562082948) rely on this PR to be backported if they're also to be backported

i'd say it's worth backporting as it shouldn't actually affect anything in v4. and other PRs that i think are worth backporting as well (e.g. https://github.com/twbs/bootstrap/pull/32630#pullrequestreview-562082948) rely on this PR to be backported if they're also to be backported

seems that while removing this fixes the unsightly behaviour in firefox (for when buttons are used as tab riders), something still trips up chromium making it show its default outline even on mouse press. this may need a further fix. adding
button:focus:not(:focus-visible) {
  outline: 0;
}

to reboot seems to work (and as it seems to only still be a problem in chromium, and recent chromium is one of the few browsers that supports :focus-visible, this seems like a fair enough addition...thoughts?)

@mdo @MartijnCuppens please confirm if this can safely be backported or not.

@mdo @MartijnCuppens please confirm if this can safely be backported or not.


looprun fails in while loop with OSError: [WinError 10038]

I'm running ProxyBroker in an environment with Python 3.7 with all modules updated to current, worked previously on an older version of modules and I should have kept a copy of the stable environment (which I didn't).  BTW ProxyBroker is completely failing on Python 3.8.  Any idea how I resolve or is this a bug?
When running a simple while loop I'm now getting (previously worked before upgrading modules) an OSError: [WinError 10038]
Here is the code without the functions:
while len(final_proxies) < 15:
    proxies = asyncio.Queue()
    broker = Broker(proxies)
    tasks = asyncio.gather(broker.find(types=['HTTPS'], limit=no_of_proxies, countries=proxy_locations), build(proxies))
    loop = asyncio.get_event_loop()
    loop.run_until_complete(tasks)
    use_proxies = proxy_tester(new_proxies)
    use_proxies_list.extend(use_proxies)
    final_proxies = use_proxies_list
    if len(final_proxies) >= 15:
        break

Here is the error:
loop.run_until_complete(tasks)

File "C:\Users\xxx\AppData\Local\Programs\Python\Python37-32\lib\asyncio\base_events.py", line 571, in run_until_complete
self.run_forever()
File "C:\Users\xxx\AppData\Local\Programs\Python\Python37-32\lib\asyncio\base_events.py", line 539, in run_forever
self._run_once()
File "C:\Users\xxx\AppData\Local\Programs\Python\Python37-32\lib\asyncio\base_events.py", line 1739, in _run_once
event_list = self._selector.select(timeout)
File "C:\Users\xxx\AppData\Local\Programs\Python\Python37-32\lib\selectors.py", line 323, in select
r, w, _ = self._select(self._readers, self._writers, [], timeout)
File "C:\Users\xxx\AppData\Local\Programs\Python\Python37-32\lib\selectors.py", line 314, in _select
r, w, x = select.select(r, w, w, timeout)
OSError: [WinError 10038] An operation was attempted on something that is not a socket

Traceback (most recent call last):
  File "c:\program files\python37\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\program files\python37\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Program Files\Python37\Scripts\proxybroker.exe\__main__.py", line 9, in <module>
  File "c:\program files\python37\lib\site-packages\proxybroker\cli.py", line 433, in cli
    loop.run_until_complete(asyncio.gather(*tasks, loop=loop))
  File "c:\program files\python37\lib\asyncio\base_events.py", line 574, in run_until_complete
    self.run_forever()
  File "c:\program files\python37\lib\asyncio\base_events.py", line 541, in run_forever
    self._run_once()
  File "c:\program files\python37\lib\asyncio\base_events.py", line 1750, in _run_once
    event_list = self._selector.select(timeout)
  File "c:\program files\python37\lib\selectors.py", line 323, in select
    r, w, _ = self._select(self._readers, self._writers, [], timeout)
  File "c:\program files\python37\lib\selectors.py", line 314, in _select
    r, w, x = select.select(r, w, w, timeout)
OSError: [WinError 10038] 在一个非套接字上尝试了一个操作。


feat: Introduce crd categories

Description
Changes proposed in this pull request (what was done and why):

group Pipelines into their own category "telemetry"
introduce 2nd category for querying all kyma-modules "kyma-modules"

Changes refer to particular issues, PRs or documents:

Traceability

[ ] The PR is linked to a GitHub issue.
[ ] New features have a milestone set.
[ ] New features have defined acceptance criteria in a corresponding GitHub Issue, and all criteria are satisfied with this PR.
[ ] The corresponding GitHub issue has a respective area and kind label.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] Adjusted the documentation if the change is user-facing.
[ ] The feature is unit-tested
[ ] The feature is e2e-tested

Looks good! Do we have an agreement with other teams to use the kyma-modules category for module CRs?

Looks good! Do we have an agreement with other teams to use the kyma-modules category for module CRs?

nope, not yet. I assume the correct way to introduce it would be a proposal.

Let's wait for the next release (tuesday) till we merge it

/unhold


More Classic .NET Remoting support

Hi,
I have a large legacy codebase with remoting calls via .NET Remoting so, of course, we're stuck on .NET Framework. This library looks like the perfect way to migrate to .NET Core. I had a look at the Classic Remoting API and it seems not all the classes and methods are implemented.
Examples of missing elements:

WellKnownClientTypeEntry
RemotingConfiguration.GetRegisteredWellKnownClientTypes()
RemotingConfiguration.GetRegisteredWellKnownServiceTypes()
and more

I'm willing to help out and submit PRs to fill the gap, but would like to check in with your first. I assume most of the lacking Classic Remoting API was just not implemented because nobody had a need for it so far, but I also assume that some of the APIs were deliberately omitted ( https://github.com/theRainbird/CoreRemoting/issues/47 is maybe such an example ).
Can we maybe brainstorm together in what way the Classic Remoting API can be enhanced further to reduce the effort involved in migrating from .NET Framework System.Runtime.Remoting?
Thanks for this project so far.

Hi igitur,

Can we maybe brainstorm together in what way the Classic Remoting API can be enhanced further to reduce
the effort involved in migrating from .NET Framework System.Runtime.Remoting?

Yes, of course.

had a look at the Classic Remoting API and it seems not all the classes and methods are implemented.

Client Activated Objects (CAOs) are not easily realizable. This is because Castle.DynamicProxy, which is used under the hood to create proxies for remote objects in CoreRemoting, can only create proxies based on interfaces or abstract classes, but not regular classes. .NET Core has no integrated facility to create proxy objects. Castle.DynamicProxy is - as far as I know - currently the best possible solution.
This is why GetRegisteredWellKnownClientTypes and WellKnownClientTypeEntry are not implemented in CoreRemoting.
If your codebase uses CAOs, you will have to switch to Server Activated Objects (SAOs) and create interfaces for all remote classes, in order to get your code running with CoreRemoting. The proxy creation itself is easy to change. Instead of
var proxy = new RemoteClass();
write
var proxy = client.CreateProxy<IRemoteClass>();.
The object lifetime could make more trouble. In .NET Remoting the lifetime of a CAO is controlled through the client. When switching to SAO, the lifetime is controlled by the server and can be Singleton (lives forever) or SingleCall (lives only for one method call). This may impact the runtime behavior of your software.
Since the early days of .NET Remoting, CAOs have never been recommended due to their poor scalability.
There is one other limitation in CoreRemoting that may affect you (depending on the architecture of your codebase). In CoreRemoting Server and Clients are fixed roles. The server process provides services to clients. The clients consume services from the server. Remote events, delegates and callbacks can be used to notify clients actively from server side or vice versa. But a client can not provide services for the server. In .NET Remoting this possible, because .NET Remoting just remotes objects between different AppDomains (the client can also be a server).
In CoreRemoting Callbacks are special, when it comes to lifetime. Their lifetime is bound to the session. When the session is closed, references to callbacks are freed.

RemotingConfiguration.GetRegisteredWellKnownServiceTypes

A 1:1 implementation is not possible, because CoreRemoting doesn't work with object URIs and the services (remotable objects that are provided by the server process) are registered in a DI container. So you could easily get your registered services from the DI container. Please have a look at the following snippet:
var diContainer = RemotingServer.DefaultRemotingServer.ServiceRegistry; IEnumerable<Type> serviceTypes = diContainer.GetAllRegisteredTypes();
This snippet uses the default remoting server. If you have only one server, this is fine. If you host multiple servers inside one process, every server must have a name (unique instance name) and the server instance must be addressed by its name. When using the app.config to configure CoreRemoting these unique instances must also be used. See example here: https://github.com/theRainbird/CoreRemoting/blob/master/CoreRemoting.Tests/TestConfig.xml
One last thing, that may cause trouble is the ability in .NET Remoting to pass a proxy that point to a object living on server process A to server process B and then to a client which can use it to directly commuicate to server A (.NET Remoting is creating implicit network connections based on channel and proxy metadata). This is not possible with CoreRemoting. In CoreRemoting a client connects to a server. This connection can only be used for communication between this client and this server. To communication with a second server, a new client instance needed (the RemotingClient object behaves similar to the SqlConnection object in ADO.NET).

Thanks, Hagen for a very thorough reply. Let's take this step by step. I'm on board that CAOs are not scalable. But I believe our current .NET Remoting implementation uses only SAOs. I believe that SAOs are defined by <wellKnown /> elements in the server and client configuration (if you're using XML configuration, as we are) and that CAOs are defined by <activated /> elements.
Minified example of our current server and client configurations:
<!-- server -->
<configuration>
  <system.runtime.remoting>
    <application>
      <channels>
        <channel ref="tcp" port="9009" />
      </channels>
      <service>
        <wellknown mode="Singleton" type="MyProject.RemotingClass, MyProject" objectUri="RemotingClass.rem" />
      </service>
    </application>
  </system.runtime.remoting>
</configuration>

<!-- client -->
<configuration>
  <system.runtime.remoting>
    <application>
      <client>
        <wellknown type="MyProject.IRemotingClass, MyProject" url="tcp://localhost:9009/RemotingClass.rem" />
      </client>
    </application>
  </system.runtime.remoting>
</configuration>

Unless I'm wrong, the registered entries above should be retrievable via RemotingConfiguration.GetRegisteredWellKnownClientTypes() on the client and RemotingConfiguration.GetRegisteredWellKnownServiceTypes() on the server. CAOs would use the analogous .GetRegisteredActivatedClientTypes() and .GetRegisteredActivatedServiceTypes().
Let me know if you agree with the above.


when i visit my profile i can't see the number of the SOLD NFTs till i click on it

What happened?
can't see the number of the sold nfts till i click on it
and i think its a must , because it helps give a good first impression of the artist ^^
Please reproduce in steps
just visit my profile, or any profile
Expected Behavior
to see the number of the SOLD nfts before clicking on it
What browsers are you seeing the problem on?
Chrome
Are you logged in?
Yes
At which address did you encounter bug?
nft.kodadot.xyz / kodadot.xyz
Screenshots
Paste from your clipboard or upload screenshots

Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

ref #1133 & #1120 related

ref #1133 & #1120 related

Those are when the creator has created nothing so its better to not show zeros
, but in this case i did creat and i did sell but my profile not showing it
Is this a feature not a bug scenarios? lol

ref #1133 & #1120 related

Those are when the creator has created nothing so its better to not show zeros , but in this case i did creat and i did sell but my profile not showing it Is this a feature not a bug scenarios? lol

Yeah, this is not queried until you click to save some queries as how RMRK works is quite heavy on querying and to speed up loading profiles, we don't do it right now.
I got what impress you got when we don't show anything rn now. We will look on that, I've put it into backlog w/ p5

Curious @onerawartist any got better now?

@yangwao everything has became much better, but this exact issue still the same

Closed in #3070


[Permission] [API] Documentation

Purpose
Add documentation and Open API specs for better understanding of how the API works and what it is for.
Acceptance Criteria

[x] kdocs written for new db access methods
[x] kdocs written for azure PermissionsFunctions class
[x] new endpoints added to the Open API spec

Blocked by 7301


AC configs: adopt to new layout

changed paths to INT8 and INT1 models
removed explicit device setting
removed extra empy lines and some unused parameters (mo_params when IR used)

@vladimir-dudnik

please run validation job with /md option from this branch to test changes

I am not sure that I can do it, because currently MD containsdownloads models with old layout.
Besides that INT8 models will be extended with new, so I suggest just wait new OMZ package readiness.


nothing happens when you click Connect (activateBrowserWallet) if you don't have any wallet installed

there should be some kind of error handling like popup that you should inform you to install a wallet to use DApp

Try:
const {error} = useEthers()

We will add info to docs.

That will actually not work. Will develop solution soon.

Ok i read some web3React code and it turns out activate (which activateBrowserWallet uses) handles errors in 3 ways read here.
In short error should be available in const { error } = useEthers() but because useDApp defaults to read only connector the error is suppressed and is available only for few frames.
So for now error can be handled for example in this way
  const {error} = useEthers()
  useEffect(() => {
    if(error){
      alert(error?.message)
    }
  }, [error])

In PR we will add way for error to be handled the same way as its handled in web3React so as a callback and throwing on error

Release in @0.3.20,
Docs available here:
https://usedapp.readthedocs.io/en/latest/guide.html#handling-wallet-activation-errrors


Magic methods : Call to an undefined method

Since nobody has complained before, I may be missing something obvious.
I'm getting this error while calling some magic method :
Call to an undefined method App\Repository\MyRepository<App\Entity\MyEntity>::findOneByField()
My code looks like this :
$myRepository = $em->getRepository(MyEntity::class);
$myRepository->findOneByField($field);

I have configured the objectManagerLoader as specified in the documentation for a Symfony app.

It is working in another context, I close the issue.

My guess is that MyEntity doesn't have $field persisted property.

My guess is that MyEntity doesn't have $field persisted property.

Also, how App\Repository\MyRepository looks like?

Also, how App\Repository\MyRepository looks like?

namespace App\Repository;

use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
use Doctrine\Common\Persistence\ManagerRegistry;

class MyRepository extends ServiceEntityRepository
{
    public function __construct(ManagerRegistry $registry)
    {
        parent::__construct($registry, MyEntity::class);
    }
}

namespace App\Entity;

use Doctrine\ORM\Mapping as ORM;

/**
 * @ORM\Entity(repositoryClass="App\Repository\MyRepository")
 */
class MyEntity
{
    /**
     * @var string
     *
     * @ORM\Column(name="field", type="string", length=255, unique=true)
     */
    private $field;
}

namespace App\Repository;

use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
use Doctrine\Common\Persistence\ManagerRegistry;

class MyRepository extends ServiceEntityRepository
{
    public function __construct(ManagerRegistry $registry)
    {
        parent::__construct($registry, MyEntity::class);
    }
}

namespace App\Entity;

use Doctrine\ORM\Mapping as ORM;

/**
 * @ORM\Entity(repositoryClass="App\Repository\MyRepository")
 */
class MyEntity
{
    /**
     * @var string
     *
     * @ORM\Column(name="field", type="string", length=255, unique=true)
     */
    private $field;
}

On level 6 you'd get this error:
Class App\Repository\MyRepository extends generic class Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository but does not specify its types: TEntityClass

This will get rid of the error for you:
/**
 * @extends ServiceEntityRepository<MyEntity>
 */
class MyRepository extends ServiceEntityRepository

And I guess it will help you to get rid of this error too: "Call to an undefined method..."

On level 6 you'd get this error:
Class App\Repository\MyRepository extends generic class Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository but does not specify its types: TEntityClass

This will get rid of the error for you:
/**
 * @extends ServiceEntityRepository<MyEntity>
 */
class MyRepository extends ServiceEntityRepository

And I guess it will help you to get rid of this error too: "Call to an undefined method..."

Indeed, I don't get the error anymore using the @extends annotation, thanks for your help.

Indeed, I don't get the error anymore using the @extends annotation, thanks for your help.


Combining Index #399

What changes were proposed in this pull request?
1、IndexContext: getBestIndexer -> getAvailableIndexers to choose high priority indexers and a parameter named maxChooseSize to  available indexers max length.
2、add IndexerScanners to wrap Seq[Indexer] and act their operations
3、IndexContext hold IndexerScanners instead of IndexerScanner
4、IndexContext: buildScanner-> buildScanners to build a IndexerScanners instance
5、OapDataReader： filterScanner->filterScanners
6、SQLConf： add new conf "spark.sql.oap.indexer.max.use.size"
7、add CombiningIndexSuite to test Combining Index
8、add a new test case in IndexSelectionSuite to test getAvailableIndexers
9、fix other test case semantics
How was this patch tested?
1、existing case
2、IndexSelectionSuite and CombiningIndexSuite

@adrian-wang @linhong-intel @xuanyuanking @yueawang

Thanks for working on this, I'll merge this by today.

@adrian-wang thx~


lfe_io_pretty:term/1 is not so pretty for (map k0 v0 ... kn vn)

I'm mostly leaving this here as a note to self. I'd like to take a stab at this when I get time, unless someone else wants to beat me to it.
Real world example output:
(map
 #"id"
 id
 #"name"
 name
 #"known_by"
 known_by
 #"bio"
 bio
 #"image"
 image
 #"topics"
 topics
 #"links"
 links)

#m(....) forms are handed differently, if not preferably:
`#M(#"date_modified" ,date_modified
    #"date_published" ,date_published #"id" ,id #"masters" ,masters
    #"sections" ,sections #"slug" ,slug #"summary" ,summary
    #"thumbnail" ,thumbnail #"title" ,title)

It'd be cool to align keys and/or values, but it seems like that might easily become a fool's errand. What's more, that style is probably a matter of personal (and subjective) taste.
All things considered, I'd be fine with (map ...) forms being printed as #m(...) ones are currently, potentially with the precondition that (> (length form) 2), so as not to mess up any *:map/2 forms.

No, there is no special handling of the (map ...) form, it is the same as for lists and tuples. The special handling for the #m(...) literal is quite simple: it tries to keep the key and value on the same and introduces line breaks when this is not possible. It would be easy to fix the (map ...) so it behaves in the same way as for the map literal.
I think trying to put too much work into indenting the map would not be worth it.
There would be no problems with *:map/2 forms unless they were local calls but with 2 args it would look quite normal. Only question is whether you would prefer:
(map #"id" id
     #"name" name
     #"known-by" known-by)

or
(map
  #"id" id
  #"name" name
  #"known-by" known-by)

:-)

I think trying to put too much work into indenting the map would not be worth it.

I agree wholeheartedly.

Only question is whether you would prefer...

Personally, I prefer the former.

@rvirding, any chance you could implement this:
(map #"id" id
     #"name" name
     #"known-by" known-by)

If you point me in the right direction, I can make a PR too.

Yes, I have been thinking about as this one is a little tricky. My plan is to make it behave in basically the same way as for literal maps, ans preferably to reuse most of the code. Try a PR if you wish and will where we end up.

I did a hackish first pass with minimal code reuse. It worked pretty well in the few tests I ran. If we want to go with this approach, it seems like we ought to abstract out F (for normal maps it's maps:to_list/1) and turn map/4 into map/5 with the fifth parameter being F. Does that make sense to anyone but me?


Theme color is added to colorize the app

Look at the top :smile:

Actually I am open to the ideas of this color. Since the navigation bar is grey, it look good but can be better.

And as can be seen on the screenshot, I've also added "Add to homescreen" suggestion popup.
http://updates.html5rocks.com/2015/03/increasing-engagement-with-app-install-banners-in-chrome-for-android
It is shown if the user opens the webpage in 2 separate days in 2 weeks.
I need #65 to be merged to send another PR for this.

Oh, yes. I forgot to add it to Zeppelin. Thanks for a PR.
About #65 please look at the my comment.

I really love the similiarity between our Buy Tickets buttons and Chrome's Add to Homescreen button. It looks like it is the part of the website and looks great.

Yeap, but I guess we should move theme-color to variables and set it to #03a9f4. Then everywhere we will ise the same color.

Definitely. I was thinking about that. Women Techmakers colors are slightly different for example. Since the website looks ok now, I didn't start with it. It should be variable and change most of the places.
http://wtmistanbul.com/

You can do this in simple way. Jusy edit _base.scss
// Colors
$main-color: #03a9f4;
$link-color: $main-color;
$link-lighten-amount: 20%;
$base-background-color: #fff;
$base-section-background-color: $link-color;
$primary-color: #4285f4;
$primary-color-hover: #3266d5;
$icon-fill-color: #8b8b8b;


TODO

[x] Initialize new empty Application
[x] Package for different platforms through electron-compile
[x] Lint applications

[x] Provide a choice between airbnb and standard linting styles

[ ] CLI Docs explorer [WIP]
[x] Integrate electron-rebuild functionality
[x] New electron-forge make command

[x] All platforms

[x] zip files

[ ] Windows

[x] electron-winstaller --> Squirrel
[ ] electron-windows-store --> AppX

[x] MAS

[x] Should operate identically to macOS

[x] macOS

[x] make DMG packages

[ ] Linux

[x] .deb files
[x] .rpm files
[x] flatpak files
[ ] snap files
[ ] AppImage

[ ] New electron-forge import command

[x] Should prompt (and automate) replacement of electron with electron-prebuild-compile
[x] Should prompt (and automate) the installation of electron-compile as dependency
[ ] Should prompt (and automate) the installation of linting tools if none are present
[ ] Should prompt the user to remove existing build tooling (babel)

[ ] New electron-forge getting-started command (This would probably launch an HTML file locally)

[ ] Briefly explain the purpose of electron-forge
[ ] Brief descriptions and links to each maker
[ ] Explanation of the config.forge section of your package.json
[ ] Automatically prompt to show getting-started on first run

[x] Start application

[x] Verbose mode with ELECTRON_ENABLE_LOGGING and ELECTRON_ENABLE_STACK_DUMPING set

[x] Come up with a standard for providing user-defined options to all the internal packages we use
[ ] Use the debug module so that people can see what's going on if they want 👍
[x] AppVeyor CI support
[ ] Add example CI files to the template

Why you decided to use various build tools instead of using electron-builder under the hood to build AppImage and other targets?

electron-builder was not used for a couple of reasons:

Electron Forge is fundamentally built on electron-compile, which as far as I can tell is incompatible with electron-builder.
It uses Electron Packger, so it makes sense to use other packages which are compatible with that one.

If you need API to build in distributable format already packaged directory — it can be easily implemented.

@develar That's the thing we don't need that API because there is already an existing well maintained community ecosystem of electron-installer-* modules that integrate perfectly with the existing electron-packager and other electron-* modules provided by both @malept and the Electron maintainers in the Electron repository.
That's part of the reason for the creation of this tool (unifying these modules in a simple starter-friendly API while maintaining maximum customisation).

I've migrated the remaining TODOs to their own issues, so I'm closing this.

So what's the status on AppImage, what's holding it back?

@probonopd Someone to work on it, really. I haven't had any time to do it.

@malept thanks, I think @develar has good news over in https://github.com/electron-userland/electron-forge/issues/26#issuecomment-288500233


High number of drops for April - Exploiter Addresses bot

I have been watching the performance for this bot for the month of April (Exploiter Addresses bot, 0x0e82982faa7878af3fad8ddf5042762a3b78d8949da2e301f1adfedc973f25ea) and it looks like it has a high number "tx_drops" for BSC chain, this issue might be also extending to the rest of the chains. I would appreciate if you can check it out, and consider sharding as a potential solution to mitigate this.

asked nethermind to tackle

Hi, just adding that the performance issue is still ongoing for the month of May, for ETH and BSC chain.


Fix bower.json

We couldn't use dev branch with bower because of the broken bower.json.
Could you fix it please.

https://github.com/RubaXa/Sortable/pull/611
https://github.com/RubaXa/Sortable/commit/9cf9cee98349124d03caa92f084d634cdcbef890


how to enable GPU in the build

I don't see a flag to enable GPU in the install.sh. Do you know how to enable it?
Thanks,

This repo is intended for minimal dependency, enabling GPU is somehow against this goal.
If you would like to use GPU, use python version instead please.
If mxnet release building with cuda in amalgamation in the future, I will re-consider this problem again.

how fast mxnet-ssd.cpp detection object in  image(eg.VOC2007) without GPU?  thanks for your work very much!

Depending on the model/cpu/compilation mode, the run time significantly varies.
VGG16 model + 2.5GHz single thread with OPENMP disabled = 6s / image

I just wonder have you built this with GPU enabled? The demo instruction has the flag --gpu=INT. But when I run this, I got a error
"D:\Dev\Projects\mxnet\dmlc-core\include\dmlc/logging.h:235: [19:14:50] D:\Dev\Projects\mxnet\src\storage\storage.cc:50: Please compile with CUDA enabled"
But I didn't see any place to enable compiling with CUDA enabled.
I have installed CUDA8.0 with cudnn v5.1 on Win10. NVIDIA Samples run well.

@feiyang-nottingham the  author alreally said: the repo don't support  gpu


Automated API token updates

Summary
Adds a script that allows easier rotation of tokens for our data and
search API. The script will add three new tokens, or remove all except
the top three, commit the new tokens, push them to GitHub and open a
Pull Request for review. This will integrate with a Jenkins pipeline job
that will add new tokens, re-deploy all apps, then remove the old tokens
and re-deploy all apps to hopefully make it far faster and less
error-prone to cycle tokens.
Ticket
https://trello.com/c/4crGe6F0/169

Updated the script so that it also injects the new tokens into the relevant variables defined in jenkins-vars/jenkins.yaml:jenkins_env_variables.


Differences in implementation compared to paper

Hi
I read your paper about MARTA-GAN and found some differences inside the paper with the current implementation:
In chapter II, section B you explain the loss of the generator, which is a combination of the perceptual loss and the feature matching loss. The feature matching loss in described by formula 4.
It is like:
Loss = || E(f(x)) + E(f(G(z))) ||²
First of all, I think you interchanged the + with a -, since you want the expected values to match and not to add those values.
The mean value is just an estimate of the expected value calculated over the batch, thus mathematically this is not the same. Right?
Furthermore you divide the mean-value by the squared image-size. I think you did it to make this loss value independent of the image size, which makes sense, since the perceptual loss is also independent of the image size.
But why did you chose exactly this value? Did you try different values to weight the g_loss2 differently compared to g_loss1. Maybe you would get a better representation by increasing the weight of g_loss2?
Have you also tried other metrics than the mean value of the activation? For example the standard deviation of the activation? Maybe you would get a better representation, if not only the mean value of the fake features matches, but also the standard deviation?
In general: Thank you for your paper, it was fun to read and it inspires me for my own work at university.

Sorry for the later reply and thank you careful check. Yes, we made a  huge mistake about the formula 4, it should be '-' instead of '+', we will update it in the arxiv.
And about the mean  over the batch maybe you can find answer here: https://stats.stackexchange.com/questions/201452/is-it-common-practice-to-minimize-the-mean-loss-over-the-batches-instead-of-the
And about divide the mean-value by the squared image-size and the weight of the g_loss2 is actually the same thing,  is a scalar about g_loss2 how to affect g_loss. We have tried different weight about g_loss2 compared to g_loss1, but the accuracy doesn't vary too much. Actually, as the figure 4 shown, add g_loss1 only add a little bit accuracy compare to only g_loss1.
Thank again for your careful check.

Thanks for the link and for updating the paper.


Issue #280 - Gradle: No such property outputDirectory

This changeset includes a fundamental change to the way this plugin operates. The extension now allows for lazy loading of the task properties, which are normally evaluated during the configuration phase. This allows for easier extension of the plugin at a later date.
As part of this changeset I have also included some unit tests. The test specifications describe the usage of this plugin.
Unfortunately, the original author of this plugin was also the publisher and so he will need to publish this new version.

FYI - I am planning on merging this. However, I am trying to debug a problem so it might be another week before I complete the merge.

No problem. I have connected with the original author and asked for a peer review too. Hopefully he will also publish the new version.

I've reviewed the changes, it looks good for me.

Hi @wmaintw, when this has been merged would you be kind and publish with the Gradle plugins portal?

Sure, will do that once this merge request been approved.
By the way, beside Gradle plugins portal, I will publish this plugin to Maven central as well.

Wei,
Can you email me at<EMAIL_ADDRESS>Jeremy
On Jul 23, 2015 9:42 PM, "Wei Ma"<EMAIL_ADDRESS>wrote:

Sure, will do that once this merge request been approved.
By the way, beside Gradle plugins portal, I will publish this plugin to
Maven central as well.
—
Reply to this email directly or view it on GitHub
https://github.com/jeremylong/DependencyCheck/pull/282#issuecomment-124289283
.

Hi Jeremy, sure I will let you know after I publish this gradle plugin

Hi Sion,
I might know why gradle says "No such property outputDirectory", I believe that you are using version 0.0.4 instead of 0.0.5, and customize outputDirectory feature was implemented in version 0.0.5.  This is my fault, I implemented this feature and created the pull request, published it to Gradle plugin portal and maven central, but when I update the README.md file, to explain how to customize outputDirecotory property, I forgot to modify the version information in README.md file.
So would you please try again using version 0.0.5?
In addition, previously I just reviewed you changes, and today I downloaded the changes and performed a local test, and found that there's an issue, that I can not customize plugin configurations in client project.
Here's what I did:

Merge your changes to my local dependency-check-gradle project
Build this gradle plugin and publish it to local directory
Client project apply this gradle plugin
Try to customize the report output directory to "./build/security-report" in client project like this:

allprojects {
    dependencyCheck {
        outputDirectory = "./build/security-report"
    }
}

Run gradle task dependencyCheck and check the report, and found that the security reports still be placed under folder "./reports"

Would you please have a look?
Best Regards.
MA, Wei

This PR has been merged.

@wmaintw, the unit test below shows that when we override the default '.reports', with 'outputDir' the same is seen in the task:
def 'tasks use correct values when extension is used'() {
        when:
        project.dependencyCheck {
            proxyServer = '<IP_ADDRESS>'
            proxyPort = 3128
            proxyUsername = 'proxyUsername'
            proxyPassword = 'proxyPassword'
            cveUrl12Modified = 'cveUrl12Modified'
            cveUrl20Modified = 'cveUrl20Modified'
            cveStartYear = 2002
            cveUrl12Base = 'cveUrl12Base'
            cveUrl20Base = 'cveUrl20Base'
            outputDirectory = 'outputDirectory'
        }

        then:
        Task task = project.tasks.findByName( 'dependencyCheck' )
        task.proxyServer == '<IP_ADDRESS>'
        task.proxyPort == 3128
        task.proxyUsername == 'proxyUsername'
        task.proxyPassword == 'proxyPassword'
        task.cveUrl12Modified == 'cveUrl12Modified'
        task.cveUrl20Modified == 'cveUrl20Modified'
        task.cveStartYear == 2002
        task.cveUrl12Base == 'cveUrl12Base'
        task.cveUrl20Base == 'cveUrl20Base'
        task.outputDirectory == 'outputDirectory'
    }

I will write an integration test this evening to see if I can prove the same this evening.

I have just coded an integration test and it does indeed fail with the scenario you mentioned.
failing test added
I'm a little confused as the unit test would indicate this is working. I suspect I have missed something with the convention mapping. I will work on fixing this.

@wmaintw, thanks for spotting this one.  I have fixed the code and the integration test now confirms the fix.
498835015ae5860c21789da169a8dbbf54a38fe7
@jeremylong I will create another pull request.


Feat/delegationscript

A good first draft for the script.
Unit tests don't work yet, but I will through them next.

depends on https://github.com/WebOfTrust/keria/pull/49


TypeError: Cannot read property 'compilation' of undefined

I'm trying to integrate circular-dependency-plugin to see the circular imports in storybook's webpack.config.js. Here is my webpack config
webpakc.config.js
// webpack.config.js
const CircularDependencyPlugin = require('circular-dependency-plugin')
const genDefaultConfig =<EMAIL_ADDRESS>
module.exports = (baseConfig, env) => {
  const defaultConfig = genDefaultConfig(baseConfig, env)
  defaultConfig.plugins.push(
    new CircularDependencyPlugin({
      // exclude detection of files based on a RegExp
      exclude: /node_modules/,
      // add errors to webpack instead of warnings
      failOnError: true
    })
  )
  return defaultConfig
}

After running npm run storybook, we are getting this error. TypeError: Cannot read property 'compilation' of undefined.
storybook version: 3.3.15.
Thanks!

What versions of the plugin and webpack are you using?

circular-dependency-plugin version: 5.0.2
storybook version: 3.3.15
storybook is internally using webpack version of 3.10.0


Allow WebView to be put in a regular NSView (Cocoa)

makeKeyAndOrderFront: only works on NSWindow and therefore shouldn't be executed if an external handle is provided in webview_create(). With this fix, all works as expected in an NSView.

Thanks for this PR.
According to the documentation for webview_create, passing in a pointer to NSView isn't something that the caller can expect to work:
// Depending on the platform, a GtkWindow, NSWindow or HWND pointer can be
// passed here.

I've experimented a bit with related issues in the past (see #965). To become really useful I believe more work is needed for other platforms as well.
Basically my idea was to expose a "widget" that users of the library can embed using platform-specific API.

I'm not quite sure what the original intent was but the current way of initializing the library with an existing window is indeed problematic. If we argue that the library shouldn't touch the window needlessly, I would have to agree.
If you own the window then my opinion is that you need to deal with it. That's also where the idea with "widget" came in since you would know how to embed it in your own window, can move and resize it as desired, etc.
I would remove the mention of NSView in the code comment so that the library's source code don't imply that this is a supported use case (but it should be in some way).
On macOS, it seems reasonable to me to only send the makeKeyAndOrderFront: message when the library owns the window.
On Windows, the library currently does this on the window regardless of who owns it:
ShowWindow(m_window, SW_SHOW);
UpdateWindow(m_window);
SetFocus(m_window);

On Linux it does this:
gtk_widget_show_all(m_window);

This could also be undesirable behavior when you own the window.

I agree with you, you can remove the NSView from the comment. As you said, if the window is not owned by WebView it shouldn't do focusing or even show on Windows/Linux as well (for Gtk you might want to check if there is a gtk_widget_show() for the webview itself if you don't do the gtk_widget_show_all()).

Looks good to me!


Remove explicit linking that is no longer needed after switching to t…

…he C++ WinRT language projection headers in <winrt/WinrtServer.h>.

@jogerh Would you mind taking a look at this?
This is preparation work towards simplifying the client code that further that I want to introduce in a later PR.


[RFC/discussion] tools API design

Copying most of this from my notes.
Motivation
We should provide the core functionality already duplicated across the various
CEM projects in the wild.
Proposed design
It seems we need a few areas of utilities:

Type guards (e.g. isClassDeclaration)
Traversal functions (various find-like functions mostly)
Assertion functions (e.g. exportHasCustomElementExport)
Getters (e.g. getAllExportsOfKind)

Type guards

isJavaScriptModule
isJavaScriptExport
isCustomElementExport
isDeclaration
isCustomElementDeclaration
isClassDeclaration
isFunctionDeclaration
isMixinDeclaration
isVariableDeclaration
isCustomElement
isCustomElementDeclaration
isCustomElementMixinDeclaration

Traversal functions

findParent(manifests, node, assertion)

Finds the first parent of the specified node which matches the assertion function

find(manifests, node, assertion)

Finds the first child which matches the assertion function

findAll(manifests, node, assertion)

Finds all children matching the assertion function

TBD
These traversal functions would be paired often with the type guards and assertion functions.
For example, find(package, null, (n) => isJavaScriptExport(n)).
Assertion functions
TBD
Getters
TBD
Prior art
CEM analyzer

getAllExportsOfKind

Filters the exports by a particular kind

getAllDeclarationsOfKind

Filters the declarations by a particular kind

getInheritanceTree

Gets the inheritance hierarchy of a given class name, as an array of
class names

getModuleFromManifests

Finds a particular module by path in a set of manifests

getModuleForClassLike

Finds a particular module's path which contains a class or mixin with the
specified name

getClassMemberDoc

Finds a class by name within a given module, then finds a member by name
within that class.

CE language server

findClassForTagName

Finds the class from a set of manifests which has the specified tag name
associated with it

findCustomElementDeclarationFromModule

Finds the first custom element class declared in a given module

findDeclarationForTagName

Finds the custom element declaration from a set of manifests which
has the specified tag name associated with it

findCustomElementTagLike

Finds a registered tag which contains the specified string

getCustomElementTags

Finds all the registered tags from a set of manifests

findCustomElementDefinitionModule

Finds the module from a set of manifests which registered a given tag name

findTagNameForClass

Finds the tag name associated with the specified class name from a set of
manifests

moduleHasCustomElementExport

Determines if a module exports a custom element

exportHasCustomElementExport

Determines if an export is a custom element export

moduleHasCustomElementExportByName

Determines if a module exports the specified tag name

exportHascustomElementExportByName

Determines if an export is a custom element definition with a particular
name

findModuleByPath

Finds a module from a set of manifests which has the specified path

modulePathEquals

Determines if a module's path matches a given path

isCustomElementDeclaration

Determines if a declaration is a custom element declaration

webcomponents.org

getCustomElements

Finds all custom elements exported by a given package, returning their
export and declaration amongst other things

getModule

Finds the module in a given package which has a path matching the one
specified

isClassDeclaration

Determines if a declaration is a class declaration

isFunctionDeclaration

Determines if a declaration is a function declaration

isMixinDeclaration

Determines if a declaration is a mixin declaration

isVariableDeclaration

Determines if a declaration is a variable declaration

isCustomElement

Determines if a declaration is a custom element declaration or a
custom element mixin declaration

isCustomElementDeclaration

Determines if a declaration is a custom element declaration

isCustomElementMixinDeclaration

Determines if a declaration is a custom element mixin declaration

resolveReference

Resolves a Reference to an actual Module from a set of manifests

Open questions
Type guards

do we want to do what typescript did here and basically provide an is* function for every possibly type? e.g. we'd also have isClassMethod, isAttribute, etc etc.

Traversals

do we want these functions to consume (manifest, nodeOrCallback, callback)? im not such a fan or these messy overloads. but (manifest, node|null, callback) isn't much better. as in, you could traverse a manifest for something, or you could traverse a node within a manifest for something
of course everything can be simplified to find/findAll, but do we also want to provide some higher level helpers? like findExports, findDeclarations, findElement, etc etc. we probably do otherwise this package won't reduce boilerplate/duplication much

cc @justinfagnani @matsuuu @rictic

Thanks @43081j !
I think a first and high-level question we need to answer is what kind of package organization we want.
I can think of a few broad categories:

Utilities for using raw manifest data (getElements(), etc)
Validation
Manifest loader interfaces
fetch-based loaders (ie, from npm registry + unpkg)
Local filesystem-based loaders (uses require.resolve() or similar)
Other I/O based loaders?
Sites?*

The first three categories may differer in scope/purpose, but should be environment agnostic. They could be three packages, or a single one. The lines could blur a bit: resolving references across manifests might require a loader. The validator could cleanly be a separate package, though it's not strictly necessary.
The next three would differ in dependencies and environments: a local loader might have Node-specific API requirements.
What I put up in #1 and #3 are separate environment-agnostic utilities package and a validator package, taken from the webcomponents.org work, but these could be re-arranged as we like...

I could imagine something like a manifest validator site, but that might not need to be in this monorepo.

i'd be tempted to do it the same way babel did
babel has @babel/traverse, @babel/types, @babel/parser, etc etc.
so in our case we'd have:

traverse (utilities for find/get/etc of a manifest, incl traversing across multiple manifests)
type guards/assertions (utilities for asserting what type an arbitrary node is, and if it has particular properties, etc)
validation
manifest loaders (one package each)

i guess my original post here is really the first two of those

i'd be tempted to do it the same way babel did
babel has @babel/traverse, @babel/types, @babel/parser, etc etc.
so in our case we'd have:

traverse (utilities for find/get/etc of a manifest, incl traversing across multiple manifests)
type guards/assertions (utilities for asserting what type an arbitrary node is, and if it has particular properties, etc)
validation
manifest loaders (one package each)

I agree that this approach could work the best. A clear split between the packages for different use cases.
Wether the loader should be 2 packages or one, I don't know yet.
Having an API like typescript where you can create your own IO layer could work but at the same time, we could just write those 2 implementations ourselves since they will differ quite a lot

I'm not sure what we really gain from separating traverse* from type guards. I don't see a downside to keeping them in on e package, but there is a downside to making too many packages - and of need future utilities that don't fit cleanly into traverse or types, requiring another one.

I'd call it something different because getting all elements, or a module by name isn't really traversing, and this package wouldn't be an AST visitor.

With the goal of separate loader packages being to divide the be environment requirements and dependencies, I think we need a spot for just the loader interface. That could go in its own package, or the general tools package.

Having an API like typescript where you can create your own IO layer could work but at the same time, we could just write those 2 implementations ourselves since they will differ quite a lot

This is the idea with the loader interface. Something like the WC catalog needs to load manifests from an online registry, a VS Code plugin needs to load it from disk.

I just merged #1  and #3 that make separate validator and tools packages (though the validator package is empty), but I wonder if I should merge those and add the loader interface there, then I/O-specific packages can come next.

i feel like the validator package should stay separate, just to keep it focused.
even if it ends up a fairly small package, i think it makes sense its a standalone thing

I'd like an object oriented API that I could deliver to 11ty etempaltes:
<template webc:nokeep webc:for="declaration of cem.get(package).getDeclarationsForModule(module)">
  <cem-class webc:if="declaration.kind === 'class'" :@declaration="declaration"></cem-class>
  <cem-mixin webc:if="declaration.kind === 'mixin'" :@declaration="declaration"></cem-mixin>
  <cem-function webc:if="declaration.kind === 'function'" :@declaration="declaration"></cem-function>
  <cem-variable webc:if="declaration.kind === 'variable'" :@declaration="declaration"></cem-variable>
</template>


New deployments fail with "couldn't import transformers package" issue on the first run.

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangGraph/LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangGraph/LangChain rather than my code.
[X] I am sure this is better as an issue rather than a GitHub discussion, since this is a LangGraph bug and not a design question.

Example Code
## trimming messages with a tokenizer:

    trimmed_messages = trim_messages(
        messages_with_prompt,
        max_tokens=max_tokens,
        strategy="last",
        token_counter=token_counter,
        allow_partial=True,
    )

Error Message and Stack Trace (if applicable)
ImportError('Could not import transformers python package. This is needed in order to calculate get_token_ids. Please install it with `pip install transformers`.')Traceback (most recent call last):

  File "/usr/local/lib/python3.11/site-packages/langchain_core/language_models/base.py", line 61, in get_tokenizer
    from transformers import GPT2TokenizerFast  # type: ignore[import]

Description
I'm running a langgraph with trim_messages() that uses a tokenizer. I added transformers package to my pyproject file.
Every time I deploy a new version, the very first run fails with an error described above. When I kick off a new run right after, everything works OK. I'm assuming it has something to do with the way transformers package gets installed - maybe it takes more time or something.
System Info
langchain==0.3.0
langchain-anthropic==0.2.0
langchain-cli==0.0.21
langchain-community==0.2.0rc1
langchain-core==0.3.2
langchain-experimental==0.0.8
langchain-fireworks==0.1.3
langchain-google-genai==1.0.4
langchain-groq==0.1.3
langchain-mistralai==0.1.7
langchain-openai==0.1.7
langchain-text-splitters==0.3.0
langchainhub==0.1.14
mac
Python 3.9.11
Same behavior on Langgraph Cloud

could you please provide the full code snippet (with the definition of token_counter, etc)? does this issue happen outside of the deployments? i.e. when you're running locally / in a notebook?

Yup, it happens locally too. It's weird because it's definitely doesn't happen all the time, just some of the time.
Here's the snippet:
model = ChatAnthropic(model="claude-3-5-sonnet-20240620", max_tokens_to_sample=8000, temperature=0.2)
def prepare_messages(prompt: str, messages: List[HumanMessage], max_tokens: int) -> List[HumanMessage]:
messages_with_prompt = messages + [HumanMessage(content=prompt)]
trimmed_messages = trim_messages(
messages_with_prompt,
max_tokens=max_tokens,
strategy="last",
token_counter=model,
allow_partial=True,
)

could you please provide the full code snippet (with the definition of token_counter, etc)? does this issue happen outside of the deployments? i.e. when you're running locally / in a notebook?

Yup, it happens locally too. It's weird because it's definitely doesn't happen all the time, just some of the time.
Here's the snippet:
model = ChatAnthropic(model="claude-3-5-sonnet-20240620", max_tokens_to_sample=8000, temperature=0.2)
def prepare_messages(prompt: str, messages: List[HumanMessage], max_tokens: int) -> List[HumanMessage]:
messages_with_prompt = messages + [HumanMessage(content=prompt)]
trimmed_messages = trim_messages(
messages_with_prompt,
max_tokens=max_tokens,
strategy="last",
token_counter=model,
allow_partial=True,
)

@antoremin I could not reproduce this issue locally with either notebook or LangGraph API (after installing transformers / adding it to pyproject)

@vbarda I tried reimplementing this trimming method on vanilla data enrichment agent and couldn't reproduce it either. I'll debug this some more on my side then.

closing this issue, but feel free to re-open if you run into it again!