[TFLite] Add int8 and int16x8 support for the EXP operator

Hello,
This PR adds quantized int8 and int16 support for the EXP operator. It uses a lookup table to do so.
Thanks,
Thibaut

Thanks for the review.
Aren't the changes in https://github.com/tensorflow/tensorflow/pull/58278/files#diff-49276b87644be533e959f64f5f3ad6dcd2bab42a2391c2b2d27c20d08ac6cdc5R29 sufficient for op_tests? It is similar to https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/testing/op_tests/relu.py tests.

@alankelly in the terms of quantized kernel wise, it looks okay to me.
Could you review this change as the TFLite kernel view point?

Hi @alankelly Can you please review this PR ? Thank you!

Hi @alankelly Can you please review this PR ? Thank you!

Can you please fix the conflict and I will merge tomorrow?

@alankelly Thank you, I fixed the conflict.

Thanks, started the merge. Reduce should also land today once the Americans wake up and approve.


fix: appVersion version 0.10.1 into 0.10.2

Signed-off-by: fengshunli<EMAIL_ADDRESS>Description
Related issues

Close #XXX

Remove this section if you don't have related PRs.
Checklist

[x] I've read the guidelines for contributing to this repository.
[x] I've added tests that prove my fix is effective or that my feature works.
[x] I've updated the documentation with the relevant information (if needed).
[ ] I've added usage information (if the PR introduces new options)
[ ] I've included a "before" and "after" example to the description (if the PR is a user interface change).

@fengshunli v0.10.2 is a helm release only therefore the appVersion is still on v0.10.1
later on this week we are planning to cut v.0.11.0 which will include all latest code changes.

@fengshunli v0.10.2 is a helm release only therefore the appVersion is still on v0.10.1
later on this week we are planning to cut v.0.11.0 which will include all latest code changes.

@chen-keinan I modified some configuration

@chen-keinan I modified some configuration

Can you please elaborate , not sure I understand which configuration

@chen-keinan I modified some configuration

Can you please elaborate , not sure I understand which configuration

v0.10.1 upgrade to v0.10.2 related configuration

Note Formatting Issues

v0.10.1 upgrade to v0.10.2 related configuration

saw it and thats why I comment that v0.10.2 is helm release (only helm chart changes) , it should not include app changes , therefore we are not changing the appVersion.
The latest app changes will be released later on this week with  v0.11.0 , so appVersion will become v0.11.0 on next release.

v0.10.1 upgrade to v0.10.2 related configuration

saw it and thats why I comment that v0.10.2 is helm release (only helm chart changes) , it should not include app changes , therefore we are not changing the appVersion
The latest app changes will be released later on this week with v0.11.0 , so appVersion will become v0.11.0 on next release.

ok

@fengshunli I assume we can close this PR , trivy-operator is in version v0.11.0. wdyt ?

@fengshunli I assume we can close this PR , trivy-operator is in version v0.11.0. wdyt ?

ok

Closing , chart update with v0.11.0


Faster saving when on a remote system

Vscode is one of the few editors that does IO on another thread, so that it is actually possible to code over ssh
HOWEVER
It is really slow, and I think this is because it is doing atomic renaming and fsync.
And you can't just point me to the ssh extension because WHERE IS THE CODE?

I assume you are using the VS Code Remote-SSH extension?
Can you provide more background on your setup?


Update manage-phone-numbers-for-your-organization.md

Modifying and adding extra information that will be useful when receiving new requests here at the Service Desk

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Add details to error message on why cuda support is disabled.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2308   +/-   ##
=======================================
  Coverage   86.63%   86.63%
=======================================
  Files         314      314
  Lines       58170    58170
  Branches     6014     6014
=======================================
  Hits        50395    50395
  Misses       6815     6815
  Partials      960      960

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 9657df6...33f5ea3. Read the comment docs.


Investigate why Chobby receives duplicate ADDUSER commands

Fireball has reported that Chobby is receiving duplicate ADDUSER commands. I tried to look into it briefly but not really sure.
https://discord.com/channels/549281623154229250/1195389146873729204/1258490543676199022
@geekingfrog Any ideas?

Closing for now as it doesn't seem to be a high priority issue, with enough information to reproduce reliably. Feel free to reopen if needed.


Deprecate project.exec and project.javaexec

And de-incubate their replacements
Fixes #30481

@bot-gradle test this without PTS

I've triggered the following builds with parameters: -DenablePredictiveTestSelection=false for you. Click here to see all build failures.

Pull Request Feedback (Trigger) (Check)

@bot-gradle test this without PTS

I've triggered the following builds with parameters: -DenablePredictiveTestSelection=false for you. Click here to see all build failures.

Pull Request Feedback (Trigger) (Check)

And then I realized there is also Script.exec which is somewhat the same as Project.exec but different 😭 I'll address it in the follow-up PR.

I've triggered the following builds with parameters: -DenablePredictiveTestSelection=false for you. Click here to see all build failures.

Pull Request Feedback (Trigger) (Check)

And then I realized there is also Script.exec which is somewhat the same as Project.exec but different 😭 I'll address it in the follow-up PR.

Fixing this straight away turned out to be easier.


Pex discards the current interpreter's PATH entry when it is a directory entry.

This means no other interpreters in the current interpreter's directory (sibling interpreters) can be found. This is an egregious problem since interpreters are commonly installed to /usr/bin; i.e.: /usr/bin/python2.7, /usr/bin/python3.6, ... etc will all be siblings and if a PEX is run with any one of them, the others won't be discoverable.
N.B.: Pex only encounters this condition when PEX_PYTHON_PATH / --python-path are used, populating the path function parameter below:
https://github.com/pantsbuild/pex/blob/93b0442c9574a0bba018249d842dc2ac29dc1e27/pex/pex_bootstrapper.py#L40-L45
And the bug is here where we remove both the interpreter binary path (which is ok) and the interpreter binary directory (which is the bug):
https://github.com/pantsbuild/pex/blob/93b0442c9574a0bba018249d842dc2ac29dc1e27/pex/pex_bootstrapper.py#L83-L89

Can this fix be cherry-picked into 2.0.0? If I have to upgrade to 2.1.0, I will have to add refactoring our python_requirement_library entries to my sub-task list.


introduced error code for better handling of the responses from…

Here is a quick draft for introducing a well-defined error code(s) that can return correct responses back to the user, filled with correct status, entity and message.
The benefits of this approach:

single exception thrown with different error codes, easy to test from service perspective (unit tests catch and check error code)
error code connected to status, rather than exception type to status

more flexibility
easier to catch wrong status codes we have now (DocumentDB.java was throwing 400 instead of 404 on not existing keyspace imo)

single place where error response is created, easy to adapt in future (for example adding extra response header, changing entity, etc)
easy way to adapt default messages in future
no need to repeat default messages all around the code
easier to integrate with metrics/tracing and "record" error code as well

Goal of the finished PR:

remove DocumentAPIRequestException completely
adapt tests to fully check the expected error code

Finalized, rebased and ready for final review.

Blocked by #937


OS#17677370 : Avoid allocating local value table before legality check

\cc @LouisLaf

} NEXT_INSTR_IN_BLOCK;

Should this be folded into the new loop bellow?

Refers to: lib/Backend/FlowGraph.cpp:4683 in 6964647. [](commit_id = 6964647ec9f086d3f5e1d0eef05d2972f6740959, deletion_comment = False)

The second loop is only going over the instructions we intend to skip, whereas the first loop is going over the block where we start our optimization, if successful a proven branch will be inserted at the end of this block.


Adding json to controllers

Does not closes https://github.com/reflectoring/coderadar/issues/86 completely, but it helps.
Is it correct?

Hi @cunhazera , yes this is correct. Would you be willing to add this to all controllers, so we can merge this PR?

@maximAtanasov of course. Sorry for the delay. Imma do that today

Fixed in #294


Revert Session.from_id 0.15.0 changes

This reverts commit 8fa0472e15ea36341acd6de67efa54a9815354e8. But keeps the backend parameter deprecation

Summary
Details and comments
Fixes #1228

@mriedem

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 45.99%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
2489

Relevant Lines:
5412

💛  - Coveralls


Enable MPI package versions 3 and 4

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Designed to allow installing alongside Rubin package.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

requirements: host: mpich==3.* must contain a space between the name and the pin, i.e. mpich ==3.*
requirements: host: mpich==4.* must contain a space between the name and the pin, i.e. mpich ==4.*

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender


No old scripts found

I wonder why it tells me: "No old scripts found."
Since I have in the decompiled_scripts folder the old files inside... I just checked and saw that with decompiling it changed them in decompiled_scripts, so the code cant detect old scripts at all...
Or am I wrong?

It's correct


shutdown behavior and carbonlink

Hi,
I'm currently trying out go-carbon to see whether it can handle 8 million metrics a minute and replace carbon-cache in our setup.
I was wondering if go-carbon writes all metrics to disk that are in the cache on a stop or restart event.
I had a few million metrics in the cache and the restart was so quick that it seemed they were dropped. Is that correct?
I also have troubles getting the carbonlink working.
In graphite-web I have carbonlink configured with: <IP_ADDRESS>:port:someID
But metrics only show up after they have been written to disk. (currently 2000 seconds :-/)
Any tips for debugging this?

In regards of first part of the question, look for [dump] config section. It explained in README.md

If you replacing multiple carbon-caches with single go-carbon? could you please elaborate your migration plan a bit?

@fsperling 130K points a second is quite doable with go-carbon, but at such rate you will have to tune the config a bit.
Try to use
high enough max-size
and
write-strategy=noop
Increase max-cpu to use all or almost all available cores
scale "workers" accordingly (we are using half of max-cpu)
throttle your writes in order to prevent your disk saturation
max-updates-per-second = (we are using 30000 on disks capable to handle up to 100000 write iops)
keep in mind that whisper update causes write/read amplification on archives with many layers.
Follow recommendation on https://github.com/lomik/go-carbon#os-tuning
@deniszh before we migrated to go-carbon we were using many carbon-cache per host with carbon-c-relay on front of them.

Hi,
Thanks for the tip with the dump section.
Currently we have haproxy -> carbon-relay -> carbon-cache
There are 16 carbon-cache processes running.
When I started a go-carbon process I added it to

the destinations of the carbon-relays
graphite-web config: CARBONLINK_HOSTS = ["<IP_ADDRESS>:12201:01", "<IP_ADDRESS>:12202:02", "<IP_ADDRESS>:1220x:0x", ... , "<IP_ADDRESS>:20005:17" ]
With 16 entries for carbon-caches and the last one for go-carbon.
So I had go-carbon and carbon-cache running at the same time. And I slowly increased the amount of metrics that are send to go-carbon.

I imagine that for each query graphite is querying all of the carbonlink hosts as the requested metrics might be split across multiple caches.
The go-carbon config looks like:
[carbonlink]
listen = ":20005"
enabled = true

@azhiltsov that is really interesting with the read/write amplification.
Is that because whenever I add a new value to a whisper file it has to be checked if older values have to be aggregated and moved around?
Or when I read values across multiple retention periods I have to do a read for each layer/retention?

@fsperling
write amplification happens if you have a multi-layer whisper file. Each write to it requires update of the lower precision archives, if I am reading this right:
https://github.com/go-graphite/go-whisper/blob/73c5cbd69142cbef9bd8ef9c6d485a8d34b4554c/whisper.go#L465
Read amplification has the same root cause. In order to perform the computation you need to read the values.
In regards of your previous comment. I can't comment further as we don't use the python version for a years now, but running go-carbon + carbonapi instead

I figured out the problem with the carbonlink.
go-carbon was configured to write statistics to "local" i.e. it put them directly into the go-carbon cache.
The carbonlink config in graphite-web however was looking for the metrcis based on some hashing algo. With tcpdump I could see that for each carbonlink host got queries for different go-carbon metrics that they didnt have.
My solution was to send the go-carbon statistics to the (local) haproxy in front of the relays.

@fsperling : Indeed, it's unfortunately not possible to use go-carbon and normal caches with carbonlink in a same time.


Only skip first char of extracted filename if it's a slash.

I was extracting from a zip file which doesn't have / at the beginning of its file entries, so the extracted files were getting the first letter chopped off. I don't know if this also happens with tar files?

Looks good, thanks! :))


completion request should flush any unsyncd overlay changes

We should add a flush() method to buffer updaters, so that a completion request operates on the very latest changes.

This is a side-effect of switching to using onDidStopChanging instead of onDidChange inside of the buffer updaters -- so, now that we're not sending realtime changes to the analysis server, we have to manually synchronize whenever autocomplete is called. dart-tools takes this approach and in retrospect it is a pretty bad one: we'll have to manually sync every editor pane before every action. Formatting, quick fixes, autocomplete, etc.
It leaves a lot of room for future errors of the data inconsistency kind, I think, and makes the system harder to reason about.

I agree that changing back to onDidChange from onDidStopChanging will make for a simpler system, with less room for programmer error. I'm worried about the UX of immediately telling users about warnings. They should have some grace period of typing where we don't tell them if there are issues with their code. Just delaying reporting the results by a fixed period will just push the problem down the road a second or two.
We could introduce some buffing logic in the ErrorRepository class. The right logic might mean that we can use the onDidChange event w/o any negative UX.

In my opinion the correct approach is having the underlying system be as real time as possible while its consumers do whatever it is they need to do to handle the incoming data. ErrorRepository would be a bad place to do the flow control in that respect. What if a non-ui component wanted real time access to the errors data?
I think that'd be better handled at the consumer level: DartLinterConsumer should determine what batch of errors is most appropriate to show, and when.
NB: If Timer.periodic worked in Atom we could just use the baked-in debounce method on the error stream DartLinterConsumer users.

There's an issue with Timer.periodic?

I thought I emailed about this, but I can't find it so I guess not:
I can't get Timer.periodic to work in Atom -- it immediately generates some kind of exception from the bowels of dart2js. debounce uses Timer.periodic under the hood. A normal timer seems to work fine.

In my opinion the correct approach is having the underlying system be as real time as possible while its consumers do whatever it is they need to do to handle the incoming data. ErrorRepository would be a bad place to do the flow control in that respect. What if a non-ui component wanted real time access to the errors data?

Makes sense to me. I'll change the code back to onDidChange, and add a todo: into DartLinterConsumer to revisit buffering.

We probably need to post-process the compiled JS a bit to get dart2js happy in the atom context. We've had to do this for a few different JS object and functions.
https://github.com/dart-atom/dartlang/blob/master/tool/grind.dart#L76
https://github.com/dart-atom/dartlang/blob/master/lib/impl/rebuild.dart#L79

Ah, that explains why some globals I expected to be in context weren't.


Update Syn

Also fixes a compile issue with the nightly feature and updates procinfo to use the same fork as TiKV.
PTAL @breeswish  @Hoverbear

ping @Hoverbear @breeswish changed procinfo to version 4 so we can still publish the crate. PTAL.

LGTM. I prefer to add testing script of the static-metric to the travis as well (e.g. building the library as well as its examples) to make sure that everything is ok.

ping @Hoverbear @breeswish PTAL

How about the https://github.com/pingcap/rust-prometheus/pull/268#issuecomment-528170310 ?

How about the #268 (comment) ?

@breeswish  Its is addressed in https://github.com/pingcap/rust-prometheus/pull/268/files#diff-354f30a63fb0907d4ad57269548329e3, I think

@kennytm PTAL (I've addressed @Hoverbear's and your comments)


implements_protocol type

Adds a type that checks whether the given datastructure implements a particular procotol.

[x] The type itself
[x] The builtin function that wraps it.
[ ] StreamData generator. Currently WIP, and a bit of a tough nut to crack since there is no general way to go from a module to a properly-initialized struct defined in that module.
[ ] Nicer formatter output.
[ ] More tests

As for naming, currently I use implements_protocol(ProtocolName) as name.
However, maybe just impl(ProtocolName) is better?

Progress has been made; I finally figured out what the problem was with the FixedMap test.
Its type was wrong, accepting arbitrary TypeCheck-types as keys, whereas they should have been arbitrary literal values instead.

A significant amount of tests has been added, which means that I am now almost happy with this PR.
Here are the things I still want to improve before merging:

Further increase code coverage. (Although I might do this in a separate PR)
Rename implements_protocol to impl.


No Response from Assistant API | Console Error: S3OnePlatform.Assistant

I am unable to get any src/examples/voice demo to respond.  I can push the button or say "Okay Google" (depending on demo) and the button light responds, but there is no audio response from the speaker.  On the console, I am seeing 100% 499 errors from google.internal.speech.S3OnePlatform.Assistant
Can anyone help me resolve this issue?
Here's my environment:
Raspberry Pi 2
Google Voice Kit (V1 HAT)
Raspbian Stretch (2018-06-27)
Exec check_audio.py results: "The audio seems to be working."
Here's my  .asoundrc:
options snd_rpi_googlevoicehat_soundcard index=0

pcm.softvol {
    type softvol
    slave.pcm dmix
    control {
        name Master
        card 0
    }
}

pcm.micboost {
    type route
    slave.pcm dsnoop
    ttable {
        0.0 30.0
        1.1 30.0
    }
}

pcm.!default {
    type asym
    playback.pcm "plug:softvol"
    capture.pcm "plug:micboost"
}

ctl.!default {
    type hw
    card 0
}

I have the very same issue

Please try to use the latest SD card image from https://github.com/google/aiyprojects-raspbian/releases.
ALSA config is not required anymore, everything is PulseAudio-based.

Still no joy.  I just installed the new SD image, and still receive no response, with 100% google.internal.speech.S3OnePlatform.Assistant 499 errors.

@ndmax Can you please confirm your environment:
Raspberry Pi 2
Google Voice Kit (V1 HAT)
Google AIY image (AIY Kits Release 2018-08-03) from  https://github.com/google/aiyprojects-raspbian/releases
And having assistant.json file in /home/pi ?

Raspberry Pi 2
Google
Voice Kit (V1 HAT)
Google AIY image (AIY Kits Release 2018-08-03) from https://github.com/google/aiyprojects raspbian/releases
And having assistant.json file in /home/pi ?

@manoj7410 All of the above information is correct.

I too have created a new SD with version 2018-08-03 and likewise everything works (including checkpoint for audio) except no audio response. Console indicates "vision" response only.
The last line of response reads:
"'command': 'action.devices.commands.VISUAL_OUTPUT'}]}]}}]}"

@ronmillsap Can share the whole response text? I'm not sure what "'command': 'action.devices.commands.VISUAL_OUTPUT'}]}]}}]}" means.

Ok, but it's long.
pi@raspberrypi:~/AIY-projects-python $ src/examples/voice/assistant_library_demo.py
ON_MUTED_CHANGED:
{'is_muted': False}
/opt/aiy/projects-python/src/aiy/drivers/led.py:51: RuntimeWarning: This channel is already in use, continuing anyway.  Use GPIO.setwarnings(False) to disable warnings.
GPIO.setup(channel, GPIO.OUT)
ON_START_FINISHED
Say "OK, Google" then speak, or press Ctrl+C to quit...
ON_DEVICE_ACTION:
{'inputs': [{'payload': {'commands': [{'execution': [{'params': {'html': '                 Meet your Google Assistant Ask it questions. Tell it to do things. It's always ready to help. Access your calendar and photos, control smart devices, and ask about anything from the weather to sports. All with just your voice.   Continue      When you speak to Google, you're speaking to AIY Projects too. Google and AIY Projects use what you say and your location to determine who can best help you. Both their terms of service and privacy policy apply.     Google Partners   Google partners are businesses that have a commercial relationship with Google.   Services and your privacy  Google sends services you talk to a unique code.  Google Terms of Service   Google Privacy Policy   I understand      Allow Google & AIY Projects to share information with each other To activate the Google Assistant, Google and AIY Projects need permission to share specific information with each other about your TV. If you allow this, you'll activate the Google Assistant and AIY Projects Voice Services. If you don’t, voice services won’t work. You can turn this off at any time. Learn more at g.co/assistant/disable    The shared information is about:   Your activity and account Your spoken requests and their transcripts, your email address, profile photo, and other info to help you sign in to or out of your account.   Your TV This TV's location (its IP address or a physical address you add), its unique ID (factory resetting the TV resets its ID), and its state, like if it’s muted.  Allow No thanks      Get the full Assistant experience On your phone, just tell your Assistant to "set up my Assistant device". \n \nDon't have the Assistant yet? Visit g.co/assistant/setup   Set up later     Google Partners Google partners are businesses that have a commercial relationship with Google. The Google Assistant can work with Google partners to help you get things done, whether it's buying movie tickets or getting your sink repaired.     Learn more    Close     Services and your privacy Google sends services you talk to a unique code. That way, services can remember things like your preferences during conversations. You can view, manage and reset the data sent to services on each service's page in the Explore section of this app.     Learn more    Close   Something went wrong. Try again.          '}, 'command': 'action.devices.commands.VISUAL_OUTPUT'}], 'devices': [{'id': '37057F0B6CE93D1E25BD457AA121FE08'}]}]}, 'intent': 'action.devices.EXECUTE'}]}

Here it is after an "OK Google"
ON_CONVERSATION_TURN_STARTED
ON_END_OF_UTTERANCE
ON_RECOGNIZING_SPEECH_FINISHED:
{'text': 'what time is it'}
ON_DEVICE_ACTION:
{'inputs': [{'payload': {'commands': [{'execution': [{'params': {'html': '                 Meet your Google Assistant Ask it questions. Tell it to do things. It's always ready to help. Access your calendar and photos, control smart devices, and ask about anything from the weather to sports. All with just your voice.   Continue      When you speak to Google, you're speaking to AIY Projects too. Google and AIY Projects use what you say and your location to determine who can best help you. Both their terms of service and privacy policy apply.     Google Partners   Google partners are businesses that have a commercial relationship with Google.   Services and your privacy  Google sends services you talk to a unique code.  Google Terms of Service   Google Privacy Policy   I understand      Allow Google & AIY Projects to share information with each other To activate the Google Assistant, Google and AIY Projects need permission to share specific information with each other about your TV. If you allow this, you'll activate the Google Assistant and AIY Projects Voice Services. If you don’t, voice services won’t work. You can turn this off at any time. Learn more at g.co/assistant/disable    The shared information is about:   Your activity and account Your spoken requests and their transcripts, your email address, profile photo, and other info to help you sign in to or out of your account.   Your TV This TV's location (its IP address or a physical address you add), its unique ID (factory resetting the TV resets its ID), and its state, like if it’s muted.  Allow No thanks      Get the full Assistant experience On your phone, just tell your Assistant to "set up my Assistant device". \n \nDon't have the Assistant yet? Visit g.co/assistant/setup   Set up later     Google Partners Google partners are businesses that have a commercial relationship with Google. The Google Assistant can work with Google partners to help you get things done, whether it's buying movie tickets or getting your sink repaired.     Learn more    Close     Services and your privacy Google sends services you talk to a unique code. That way, services can remember things like your preferences during conversations. You can view, manage and reset the data sent to services on each service's page in the Explore section of this app.     Learn more    Close   Something went wrong. Try again.          '}, 'command': 'action.devices.commands.VISUAL_OUTPUT'}], 'devices': [{'id': '37057F0B6CE93D1E25BD457AA121FE08'}]}]}, 'intent': 'action.devices.EXECUTE'}]}
ON_NO_RESPONSE

@ronmillsap Did you try to run assistant_grpc_demo too ? Does it work ?

@ronmillsap Is there any voice response when these long messages are printed? Did you follow step "Give permissions to the Google Assistant API" from https://aiyprojects.withgoogle.com/voice/?

Dimitri, sorry I missed your first question. There is no sound whatsoever with the response however the light does come on just like when there IS a response, just without any sound.
I am curious about running grpc so I will swap out the button for the led and see if grpc works or not.

Ok, set it up the button, all check points worked fine, started grpc and the light started blinking slowly (ready). Pressed the button and the following occured: however, this time the button light started blinking quick short blinks (pulse=thinking) as well as the slow blinks.
Build info: Sat Aug 4 00:20:37 UTC 2018 @ 51293f3
Dev terminal is ready! See the demos in 'src/examples' to get started.
Note: the MagPi Essentials are not up-to-date. See https://goo.gl/8hEcfo
pi@raspberrypi:~/AIY-projects-python $ src/examples/voice/assistant_grpc_demo.py/opt/aiy/projects-python/src/aiy/_drivers/_led.py:51: RuntimeWarning: This channel is already in use, continuing anyway.  Use GPIO.setwarnings(False) to disable warnings.
GPIO.setup(channel, GPIO.OUT)
[2018-08-09 10:01:20,658] INFO:recorder:started recording
Press the button and speak
Listening...
[2018-08-09 10:01:48,349] INFO:speech:transcript: what
[2018-08-09 10:01:48,355] INFO:speech:transcript: what's
[2018-08-09 10:01:48,357] INFO:speech:transcript: what time
[2018-08-09 10:01:48,461] INFO:speech:transcript: what time is
[2018-08-09 10:01:48,614] INFO:speech:transcript: what time is it
[2018-08-09 10:01:48,797] INFO:speech:transcript: what time  is it
[2018-08-09 10:01:49,159] INFO:speech:transcript: what time is  it
[2018-08-09 10:01:49,198] INFO:speech:transcript: what time is it
[2018-08-09 10:01:50,633] INFO:speech:event_type: 1
[2018-08-09 10:01:50,699] INFO:speech:transcript: what time is it
You said " what time is it "
Press the button and speak
Listening...
[2018-08-09 10:03:09,648] INFO:speech:transcript: ok
[2018-08-09 10:03:09,709] INFO:speech:transcript: Okay Goo
[2018-08-09 10:03:10,016] INFO:speech:transcript: OK Google
[2018-08-09 10:03:10,658] INFO:speech:transcript: OK Google
[2018-08-09 10:03:18,065] INFO:speech:transcript: OK Google  what
[2018-08-09 10:03:18,206] INFO:speech:transcript: OK Google  what time
[2018-08-09 10:03:18,625] INFO:speech:transcript: OK Google  what time is
[2018-08-09 10:03:18,699] INFO:speech:transcript: OK Google  what time is it
[2018-08-09 10:03:18,702] INFO:speech:transcript: OK Google what  time is it
[2018-08-09 10:03:18,753] INFO:speech:transcript: OK Google what time  is it
[2018-08-09 10:03:18,974] INFO:speech:transcript: OK Google what time is  it
[2018-08-09 10:03:18,978] INFO:speech:transcript: OK Google what time is it
[2018-08-09 10:03:19,802] INFO:speech:event_type: 1
[2018-08-09 10:03:19,860] INFO:speech:transcript: OK Google what time is it
You said " OK Google what time is it "
Press the button and speak

@ronmillsap Interesting, not sure what's happening here, output looks good. Just to double check, you said that all checkpoints passed, right? So you were able to record then listen to your voice during sound check procedure?

Yes Dimitri, today I walked through the instructions again. In troubleshooting, it suggested to repeat step 7 which is to get the oAth2.0 client. So I tried new credentials but got the very same results.
I do wonder what the consequences of the warning about "this channel is already in use" as that warning is always shown when I start up?

@ronmillsap Can you please look at the similar issue #447? Looks like everything was resolved there by just creating a new project. Regarding this channel is already in use, that shouldn't be a problem but we'll try to figure out and fix this.

Yes I will give it a go. Thanks for not giving up!

ok Dmitriy, i was able to get everything working properly by setting up a new project. not sure EXACTLY what that means or why it works but all is working. I'll take that. Thank you for all the help!

@ronmillsap Great! It's actually more related to Google Assistant Library than to our project :)

Looks like this problem doesn't happen anymore.


Angular SSR (request path contains unescaped characters)

Which @angular/* package(s) are the source of the bug?
Don't known / other
Is this a regression?
Yes
Description
I have a path with a slug in Arabic characters, like www.test.com/test-details/هذا-هو-الإختبار, when I run npm run dev:ssr and open this route, SSR fails and server logs ( request path contains unescaped characters ), I tried many solutions, one of them worked for me but gives me an unwanted result, the solution was to use encodeURIComponent to encode the slug, but I want to keep slug as it is in the URL, so is there other solution?
Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
TypeError [ERR_UNESCAPED_CHARACTERS]: Request path contains unescaped characters

Please provide the environment you discovered this bug in (run ng version)
Angular CLI: 16.2.11
Node: 18.13.0
Package Manager: npm 9.7.1
OS: win32 x64

Angular: 16.2.12
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, platform-server
... router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1602.11
@angular-devkit/build-angular   16.2.11
@angular-devkit/core            16.2.11
@angular-devkit/schematics      16.2.11
@angular/cdk                    17.1.2
@angular/cli                    16.2.11
@nguniversal/builders           16.2.0
@nguniversal/express-engine     16.2.0
@schematics/angular             16.2.11
rxjs                            7.8.1
typescript                      5.1.6
zone.js                         0.13.3

Anything else?
No response

Hi @EmadAbdou, I am unable to replicate this problem.  Can you setup a minimal repro please?
You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.
This might be related to your directory structure so its really important to get an accurate repro to diagnose this.

Closing due to lack of communication.


Can't build by Bazel as an external imported project.

Please answer these questions before submitting your issue.
Should this be an issue in the gRPC issue tracker?
Create new issues for bugs and feature requests. An issue needs to be actionable. General gRPC discussions and usage questions belong to:

grpc.io mailing list
StackOverflow, with grpc tag

Please don't double post your questions in more locations; we are monitoring both channels, and the time spent de-duplicating questions is better spent answering more user questions.
What version of gRPC and what language are you using?
1.4.3
What operating system (Linux, Windows, …) and version?
Linux
What runtime / compiler are you using (e.g. python version or version of gcc)
Bazel 0.4.5
What did you do?
If possible, provide a recipe for reproducing the error. Try being specific and include code snippets if helpful.
Import grpc as an external repo by git_repository, then can't build.
What did you expect to see?
Build successful.
What did you see instead?
❯❯❯ bazel build @com_github_grpc_grpc//:grpc
ERROR: /usr/local/google/home/ningr/.cache/bazel/_bazel_ningr/5cdbcab7751afe004080d5ce1e95b149/external/com_github_grpc_grpc/BUILD:1008:1: no such target '//external:cares': target 'cares' not declared in package 'external' (did you mean 'ceres'?) defined by /usr/local/google/home/ningr/redwood_gob/WORKSPACE and referenced by '@com_github_grpc_grpc//:grpc_resolver_dns_ares'.
ERROR: Analysis of target '@com_github_grpc_grpc//:grpc' failed; build aborted.
ERROR: /usr/local/google/home/ningr/.cache/bazel/_bazel_ningr/5cdbcab7751afe004080d5ce1e95b149/external/com_github_grpc_grpc/bazel/BUILD:41:1: missing mandatory positional argument 'well_known_protos' while calling cc_grpc_library(name, srcs, deps, proto_only, well_known_protos, generate_mock, use_external = False, **kwargs).
ERROR: package contains errors: bazel.
ERROR: error loading package '@com_github_grpc_grpc//bazel': Package 'bazel' contains errors.
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
Anything else we should know about your project / environment?

I have the same issue.

the same issue when i  build //tensorflow-serving/model_servers:tensorflow_model_server.

+1 when building TensorFlow Serving after syncing TF to head.
Is there any any way to configure grpc to use a default 'cares' package without having to provide our own binding?

The point of Bazel is to be fully hermetic. When building a subproject using Bazel, you HAVE to provide source code bindings for all of its dependencies. That's both a pro and a con of Bazel. So, I'd say, "working as intended".
Since TensorFlow uses gRPC as a subproject, they are binding all of its dependencies in their workspace script. If they have updated the gRPC submodule without providing bindings for c-ares, that's a bug in TensorFlow, not gRPC.

It would be nice for users of gRPC if you provide default implementations of everything so we don't need to provide additional bindings (or make them optional for advanced users rather than mandatory for everyone).
As I understand it, you're using this implementation and have to provide your own BUILD file. We wouldn't be able to use it directly in our WORKSPACE since it's just a directory in your repo rather than a whole git_repository, and we wouldn't be able to use the actual cares repo because it's not a Bazel project with a BUILD file. What's the recommended way to include this binding?

You can see what we are doing in our own WORKSPACE file:
new_local_repository(
    name = "submodule_cares",
    path = "third_party/cares",
    build_file = "third_party/cares/cares.BUILD",
)

Feel free to steal our cares.BUILD file and associated configuration files (the generated .h files) and use the same exact submodule sha1. Basically, you'd need to duplicate 3 or 4 files into your project, and that'd be it.

It would be a bonus if those default dependency bindings used in WORKSPACE are packed into a function and put in a bzl file. This will be super convenient for importing.
Something like:
load("//bazel/deps.bzl", "bind_grpc_deps")
bind_grpc_deps()

(Here is an example from rules_protobuf, https://github.com/pubref/rules_protobuf/blob/master/WORKSPACE)

I don't really have any problem with this, save it that right now we're actually using the submodules themselves from the disk, and not actual git submodules from within git. Once thing that I've been anal about in the past (and I still am tbh), is making sure we have everything in sync and that people don't change the submodules by mistake. Therefore we have that test script here: https://github.com/grpc/grpc/blob/master/tools/run_tests/sanity/check_submodules.sh
I feel it should be possible to test the list of submodules from the bzl file too, but I'm not 100% sure how. What do you think ?

Yes, it's possible to verify the hash code of imported repositories by bazel.
(Example: https://github.com/pubref/rules_protobuf/blob/master/protobuf/internal/require.bzl#L17)
And it will be nice to use "new_git_repository" (rather than "new_local_repository") to let bazel download/manage those 3rd-party dependencies. It'll be easier to import grpc from other bazel project. People can simply copy/paste those "new_git_repository" into their own WORKSPACE files.
WDYT?

Any movement on this topic? It'd be great to be able to use Bazel to build a project that uses gRPC without extra workarounds to get gRPC to build.

+1
I have the same issue when building tensorflow_serving 1.3.0

Hi @nicolasnoble, I can work on this.
Make Bazel build not rely on git submodules, and use the identical version of dependencies listed here. Please assign this bug to me.

I've drafted a PR (https://github.com/grpc/grpc/pull/13468) for the fix. Please take a look. @nicolasnoble

I am closing this since the PR https://github.com/grpc/grpc/pull/13468 is closed


Initialization error (angular 2 language service).

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions
Windows 10, WebStorm 2018.2
Angular CLI: 7.0.1
Node: 10.12.0
OS: win32 x64
Angular: 7.0.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.10.1
@angular-devkit/build-angular     0.10.1
@angular-devkit/build-optimizer   0.10.1
@angular-devkit/build-webpack     0.10.1
@angular-devkit/core              7.0.1
@angular-devkit/schematics        7.0.1
@angular/cli                      7.0.1
@angular/platform-server          6.1.10
@ngtools/webpack                  7.0.1
@schematics/angular               7.0.1
@schematics/update                0.10.1
rxjs                              6.3.3
typescript                        3.1.3
webpack                           4.19.1

Repro steps
Create a new project
Try to use Angular Language Service
The log given by the failure
Error:Initialization error (angular 2 language service). Cannot read property 'CommandTypes' of undefined

@alan-agius4 hi. The problem occurs on an empty project, right after I do ng new repro-app and ng server.

Lock file should be enough.
I am unable to replicate any issues when doing the following;
ng new foo
cd foo
ng serve foo

ng new repro-app
Open created project in WebStorm
In Webstorm Terminal run ng serve

new TypeScript tab opens with the error

https://github.com/maxkorz/repro-app

I'm sorry, but this issue is not caused by Angular CLI. Please contact the author(s) of the Webstorm project or file an issue on their issue tracker.

@alan-agius4 you saying that Angular Language Service 7.x is not compatible with present IDEs?

Thanks, I found an issue in WebStorm tracker https://youtrack.jetbrains.com/issue/WEB-34463


Week 5

Probably all of my changes
7/2/19

Issue is your imports frc/SmartDashboard/SmartDashboard.h, should be frc/smartdashboard/SmartDashboard.h (note capitalization)


make lemmas about isomorphisms accept more iso parameters. new lemmas ab...

...out groupoids

@javra, it seems that there is a problem in hott/algebra/groupoid.lean file. Could you check?
https://travis-ci.org/leanprover/lean/jobs/50582522

Ah thanks, I accidentally added some unfinished and unused theorems...

I squashed the commits and pushed to blessed.


Line does not render correctly with KIVY_WINDOW=x11 when opacity is less than 1.0

Software Versions

Python: 3.8.10
OS: Ubuntu 20.04
Kivy: v2.2.1
Kivy installation method: pipenv

Describe the bug
When the opacity of a line is set to < 1.0 the line is rendered incorrectly
Expected behavior
The Line should be drawn as specified (an ellipse in this following example)
To Reproduce

Test the example with python test.py
This will show you the correct rendering
Test the example with KIVY_WINDOW=x11 python test.py
This will show you the bug

Code and Logs and screenshots
# test.py

from kivy.app import App
from kivy.lang import Builder

kv = """
Widget:
    canvas:
        Color:
            rgba: 0, 1, 0, 0.5
        Line:
            width: 10
            ellipse: self.x , self.y , self.width , self.height
"""

class TestApp(App):
    def build(self):
        return Builder.load_string(kv)

if __name__ == '__main__':
    TestApp().run()

sdl2 vs x11
I've also tried the official canvas lines example and got the same results:

working with opacity 1.0 on both sdl2 and x11

when setting any opacity the sdl2 continues to work but the x11 fails
Additional context
It looks like a stencil is activated when the opacity is lower that 1 here.
I believe all  stencil operations are misbehaving after some test on a rapberry pi as well.

Fixed via #8383


Negative delay freeze Godot

Godot version:
Godot 3.2.4 rc 2
OS
Ubuntu 20.04 - Ubuntu 3.36 X11
Issue description:
Executing
OS.delay_usec(-1)

or
OS.delay_msec(-1)

freeze entire Godot

This is probably because the value overflows, resulting in a very long delay.

Yeah it can likely be considered working as intended... but I guess we can ERR_FAIL if the parameter is negative to prevent this, we don't have to support unsigned int wrapping as a feature here.

The parameter is an uint32_t, so how can we detect whether it was actually passed as a negative value?

Maybe bind the scripting method to a proxy _delay_usec(int p_delay) that does the check and calls the original method, so it's still available for C++ with proper type checks.


torchsummary has been renamed to torchinfo

All lines contained torchsummary throughout the book has to be renamed to torchinfo, i.e.:
from torchinfo import summary
Furthermore, all lines containing summary have to edited with the new syntax:
summary(model, (32,3,64,64))

Library is still backwards compatible so there's no need to change any code


Use disk cache to run wd tests

This PR does a few things:

It updates the pyodide bucket to one that includes a file called all_wheels.zip in its GitHub release.
It uses this file to create a bazel repo that contains all the wheels
It passes a path to the bazel repo as --disk-cache-dir when the test is being run.

Open questions:

Does this work on Windows
Does this break tests on edgeworker somehow

Why not just download the wheels from R2? The main reason is that the R2 bucket is not accessible from our build infrastructure.

The main reason is that the R2 bucket is not accessible from our build infrastructure.

Why is that? Can't we make the R2 bucket accessible from there?


Unable to find entry point named "objc_msgSend_stret" in shared library '/usr/lib/libobjc.A.dylib For Apple Silicon Based Macs

When running on macOS (2021 M1 Pro) an exception error is raised, with the details: "Unable to find entry point named "objc_msgSend_stret" in shared library '/usr/lib/libobjc.A.dylib"
Tested it on my old PC (Windows) with no issues so is Mac related.

what macOS version are you running?

MacOS Ventura 13.1

I checked my usr/lib/ folder and I cannot seem to find libobjc.A.dylib.
That is likely the issue but I have no idea how to manually get the file to put it there.

Any updates on how to fix this?

I believe the fix here should be the same as what was done in this commit). The short answer is that objc_msgSend_stret just shouldn't be used in this codepath on M1 processors. The longer answer is that the code already has some notion of whether the stret function variants should be used, but it's not handled very consistently (see this function and the places it is used), and it doesn't take processor arch into account (thus the TODO marker there...).
If anyone with an M1 processor is able to test the quick fix (e.g. the same thing that @frenzibyte did in the commit above), I'd be happy to merge it in.

Also, it would help to know what stack trace you see when you get this exception. Is this still a problem in the latest version of the code?

@lkj01010 Are you sure you're running latest version of Veldrid? This path is impossible to reach for M1/M2-based machines:
https://github.com/veldrid/veldrid/blob/63b6300f7da014214452dd6ea9a6653ff202bd67/src/Veldrid.MetalBindings/NSView.cs#L26-L34


When opacity is changed default colormap changes to grayscale from magma

When loading a volume if one changes the opacity (with the menu either open or in its preview state) the colormap switches from magma to grayscale.
replication: https://interactive-viewer.apps.hbp.eu/#/a:juelich:iav:atlas:v1.0.0:2/t:minds:core:referencespace:v1.0.0:265d32a0-3d84-40a5-926f-bf89f68212b9/p:minds:core:parcellationatlas:v1.0.0:05655b58-3b6f-49db-b285-64b5a0276f83/rn:f255401/@:0.0.0.-W000..2_Q5ui.2-9bFF.-C_vw._E3bw..mJN..2L7M.AIlZ.34-H..2s2/x-overlay-layer:precomputed:%2F%2Fhttps:%2F%2Fdata-proxy.ebrains.eu%2Fapi%2Fv1%2Fbuckets%2Fd1-d2-expression%2Fd1_ng

the magma colormap is set via meta.json's  override.customShader, I presume?
This is an interaction I did not account for. I will create a fix in the coming release.
I should mention that, as mentioned in https://github.com/FZJ-INM1-BDA/siibra-explorer/pull/1348#issuecomment-1658904430 , one of the limitation is that custom shader cannot be controlled by the widget (except for opacity).
we will update the component to reflect this.

I don't believe the issue is related to meta.json, I have experienced this on volumes where meta.json is not set. to illustrate this I have removed meta.json from the above example and the behaviour persists.

I stand corrected.
I believe I know the reason: inconsistent default. In siibra-explorer, we add the layer with magma as default colormap. But on interaction with ng-layer-tune, default is greyscale.
I would like to set magma to be default everywhere, if there is no objections.

will be fixed by https://github.com/FZJ-INM1-BDA/siibra-explorer/pull/1391 and release of 2.12.4

Magma is my favourite as well! Fantastic thank you for the fix :)

fixed by release of 2.12.4


Update mongodb-driver-core to 4.4.0

Updates org.mongodb:mongodb-driver-core from 4.3.4 to 4.4.0.
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 = "org.mongodb", artifactId = "mongodb-driver-core" } ]

labels: library-update, early-semver-minor, semver-spec-minor

https://jira.mongodb.org/browse/JAVA-4403?jql=project %3D JAVA AND issuetype %3D Bug AND status %3D Closed AND fixVersion %3D 4.4.0 ORDER BY component ASC


add unbind op

添加paddle unbind支持

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@jiuyuedeyu156 Hello 感谢你的贡献，我明天会审核你的PR

Unbind实现可以还参考

torch opset 9 ubind


@conda-forge-admin, please add bot automerge

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/frictionqpotspringblock-feedstock#46.


victor-mono: init at 1.2.1

Motivation for this change
Fixes #64465.
Haven't tested installation but inspecting output LGTM-- requester want
to confirm this works for you? :)
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

It seems I've merged the competing PR, seems that this one installed .otf too.
Too late though.

Yes, is there a reason to prefer one over the other?
Or to "only" include one, if upstream provides both? (space?)
Never sure which way to go with these :).

Oh, I see--I'll take a look and maybe submit a PR with the difference
:).
On Sat, 13 Jul 2019 09:07:24 -0700, worldofpeace<EMAIL_ADDRESS>wrote:

It seems I've merged the competing PR, seems that this one installed .otf too.
Too late though.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixpkgs/pull/64544#issuecomment-511133611Non-text part: text/html

I'd distribute both.

Updated, hopefully "best of both"? :)


fix: remove legacy React.render() from the editor

fix https://github.com/excalidraw/excalidraw/issues/5551

@dwelle @ad1992 This change makes the context menu not render on react projects running on React 17, that are embedding Excalidraw


let default apply if msg.delay not set in override mode.

Coverage remained the same at 86.03% when pulling 27b1adcfb081623f86b2c38ad89bbea0b3baba96 on msg.delay into a42e99c4aa19c2aa981558e78d61554f73ae39bf on master.

Coverage increased (+0.04%) to 86.072% when pulling 1f14bf72a5a6c302cf802046a12968ad50354fbe on msg.delay into a42e99c4aa19c2aa981558e78d61554f73ae39bf on master.


ImageryLayerCollection move proposal

Hi,
This is my first submission so first at all I want to thanks the devs for this great tool !
Cesium: 1.42.1
I do not want to create a pull request because I think is too short. In one of my project I needed
to reorder the Layer in the ImageryLayerCollection but I did not find any move(src, dst) method so here is the code I use :
Cesium.ImageryLayerCollection.prototype.move = function (layer, toIndex) {

  var index = this._layers.indexOf(layer);
  if (index == -1 || index == toIndex || toIndex <0 || toIndex > (this._layers.length - 1)) {
    return;
  }

  this._layers.splice(index, 1);
  this._layers.splice(toIndex, 0, layer);

  this._update();

  this.layerMoved.raiseEvent(layer, toIndex, index);
};

It would be great if this could be in some future release.
Regards

Hi @jeanpul, welcome!
Gnerally, please ask questions over at the Cesium Forum.  There are members of the community and developers from the team who answer these kinds of questions there. Additionally, Cesium's a big project, so we use GitHub for feature requests and bug tracking exclusively.
While there is indeed no move function, we have raise/lower methods in ImageryLayerCollection. If move is a function you'd like to see, please contribute and open a PR with your changes! This isn't too short at all.


Deleting roles with users

Hey,
it seems to be that you have to clear all users from a role, that you can delete it then.
public async Task<IdentityResult> DeleteRoleAsync(string roleName) {           if(await this.GetApplicationRole(roleName) is ApplicationRole role) {               return await this.RoleManager.DeleteAsync(role);           }           return ErrorConstants.RoleNotFound;       }
If the role has users assigned, the IdentityResult will still be succeeded, but the role still persists in the database.

Hi,
I've just pushed a new version to NuGet which should take care of this issue.
It's v<IP_ADDRESS>

hey,
unfortunately, that did not work. I´ve noticed that the XPRoleStoreValidator validates against this condition.
            if (role != null) {                 int userCount = (int)role.Session.Evaluate(typeof(XpoDxUser),                     CriteriaOperator.Parse("Count"),                     CriteriaOperator.Parse("Roles[Id == ?]", role.ID));                 if (userCount > 0)                     result = new DataValidationResult<TKey> {                         ResultType = DataValidationResultType.Error,                         ID = role.ID,                         Message = String.Format("Role '{0}' cannot be deleted because there are users in this Role", role.Name)                     };             }
The role object will be deleted (and within its users) when the validation is successful (which will never succeed)
I´ve forked this repo and modified it on my end.


Don't use GetCSC for non-blocking keypad scans.

You say not to use GetCSC for USB, but you still used GetCSC. I fixed that. You're welcome. :P

Wtf. GetCSC is perfectly fine. Perhaps you were thinking about the older usb driver.

Then why the “don’t use getcsc for USB” in the source? In any case, my routine is faster. Sorry I wasted your time I guess

There are many other places where getcsc is being used. I've removed the old comments so that it avoids any more confusion.


feat: Add the Action Atom

🔗 Linked issue
#161
❓ Type of change
Add a JS component with no twig template to trigger action on other components.

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Resolves #161
Component can be used like the following example:
<button
  type="button"
  data-component="Action"
  data-option-on="click"
  data-option-target="Panel Modal"
  data-option-selector="[data-can-be-closed]"
  data-option-method="close">
  Close Panel & Modal with [data-can-be-closed]
</button>

→ Show on the playground
📝 Checklist
Feature

[x] Possibility to target multiple component types
[x] Add an option to configure the type of event triggering the action
[ ] (optional) Possibility to pass arguments to the method

Pull request

[x] I have linked an issue or discussion.
[ ] I have added tests (if possible).
[x] I have updated the documentation accordingly.
[x] I have updated the changelog.

@titouanmathis I think that the PR is ready. Do you think that I need to add tests for this new feature ?


infopdata

issue for table - infopdata.
It is main table for plant information.
column header -
ipid -
pid -
locid -
biblioid -
scid -
profilecode -

REFACTOR
TABLE
profile
COLUMNS
profile_id
plant_id (FK)
location_id (FK)
biblio_id (FK)
date_id (FK) => MOVE to VALUE
legacy_profile_id (JEA*..)
percent (NUMBER) or maybe range
plantPart_id (FK) => MOVE to PROFILE
condition_id (FK) => MOVE to PROFILE
method_id (FK) => MOVE to PROFILE
method VARCHAR not normalized


created a Makefile which can be used to build a pkg

update README.md to document the make command
added a gitignore
I set the pkg version to 1.0.0, but I can change it. It might be helpful to add a 'release' or git tag to avoid any confusion about version numbers in the future.

Thanks!
I'd prefer to not automatically run makecatalogs after importing for safety reasons. Can you remove that part?
I'll make a tag. I don't plan on needing to update this anytime soon, but putting that info in the Makefile explicitly as part of the tag makes total sense to me. Then it's still easy for someone to override.


make the promoter print out a JSON dump (summary) of everything that happened

This paves the way for other tooling to integrate with the promoter, and also simplifies what to look at when we parse the logs as humans after a promotion run (in Prow).
The logs are cumbersome to parse by human eyes, so at least printing out a (prettified) JSON at the very end of a promotion run could make it easier to see what went OK and what went wrong.

/remove-lifecycle stale
I think this issue will really help increase the readability of the logs, which are currently pretty verbose.

/remove-lifecycle stale

/remove-lifecycle stale

I can take a look at this issue

/assign @yodahekinsew


Cannot require any modules

OS: Ubuntu 19.10 x86_64
Kernel: 5.3.0-53-generic
NectarJS v0.6.1
I cannot compile my project because of "require error"s. It happens for every module I require in my script.
nectar ./index.js --verbose
[*] Generating source file
[!] Require error : fs
If I remove fs, it gives me something else (the next module).
Am I missing something ?

Hi @Teln0 ,
there is no fs module for the moment, there is an fs-std module with simple functions (readFileSync, writeFileSync ...)
We will add support for the standard node ecosystem soon

what about other modules like http ?

@Teln0 it is the same, i'm finishing the base of NectarJS, then, we will be able to add any module and platforms (libuv, node modules ...)

@Teln0 I started the njs-http module, you can check the http.js example

@Teln0 it is the same, i'm finishing the base of NectarJS, then, we will be able to add any module and platforms (libuv, node modules ...)

So basically, you have to port the standard library/base modules and then after that, most/all node modules should work? What kind of timeline are you looking at? I know they're never accurate. Just curious as I'd love to be able to compile a native version of gundb.

So basically, you have to port the standard library/base modules and then after that, most/all node modules should work?
-> basically yes, i just integrated libuv for node env

What kind of timeline are you looking at? I know they're never accurate. Just curious as I'd love to be able to compile a native version of gundb.
-> i think between 3 and 6 months it will be more usable with Node, so a more things will compile

Just found this project, and I think it's fantastic. Really looking forward to using it once it's more stable with Node. A preliminary compilation of console.log('Hello world') came in at just 176KB vs over 40MB with other projects that bake in the entire nodejs run time, which is amazing!

Hi @josephschmitt ,
Thank you very much !
Arduino examples fits on Arduino nano with less than 10ko of rom, where NodeJS (or any other vm) will never fits

It would be nice actually to have a list of features supported and what is on road map, instead of keeping developers hitting the wall with all kinds of strange issues.
[*] Generating source file
[!] Warning : index file of module escape-string-regexp seems empty
/usr/local/Cellar/node/14.5.0/lib/node_modules/nectarjs/compiler/native/lib/genRequire.js:208
        var _Ext = fileSource.split(".");
                              ^

TypeError: Cannot read property 'split' of undefined

and it dies on just two lines of code:
const chalk = require('chalk');
console.log(chalk.blue('hello'));

Hi @skitsanos ,
Thank you for the feedback,
The full supported modules, right now, are the nectar_modules,
Before supportind nodejs modules, we are finishing to support ES3, after we will focus on the NodeJS env
The error you have right now is because one file of the module seems empty, or there is a bug, i will check it right now

@skitsanos Roadmap and doc are updated


Optimize Map

Map as previously implemented used a linear search to find keys, resulting in O(n) put/delete performance, which isn't what one expects from a map (or a Set, which uses Map under the hood). It's slower than we find acceptable for general use. This appears to have been intentional, to keep the implementation simple, but that makes the class more of a reference implementation than a polyfill, and we actually want a polyfill, to allow us to use the Ds interface while we figure out the source of a segfault in the extension.
To improve the situation, we implement Map using two parallel arrays: one to maintain order (required by the interface) and another to provide a lookup table by key. We store the same PairRef (a container for key and value) in both tables so that finding a pair in either array is sufficient to find it in the other one, even as the order array changes.
This approach leads to a Map/Set implementation that's roughly 8 times slower than using SplObjectStorage for new stdClass instances and 5 times slower than using native arrays directly for integers, and uses roughly 3 times more memory across the board. For example, storing around 32K integers eats up ~2M for a native array and ~6M in our Map, and takes 5–10ms for the native array versus ~25ms for our Map. It's not surprising that the PHP implementation is slower and more memory hungry than the C implementations, but the new Map/Set polyfill is significantly faster than the previous one (which had made it less than 1% through the benchmark after 10 minutes where the new implementation finishes the benchmark in about 2 minutes) for a modest increase in memory usage (because we add a lookup table to the ordered list of pairs).
In order to test on PHP 8 (and on a machine with the extension enabled), I had to make some minor changes to the test config and the test suite. Note that we use a fork of the test repo, and we use the php8 branch on that fork.

After these changes, we still need to implement custom serialization logic on the polyfill so that its serialized data is compatible with serialized data created by the extension.

I didn't actually intend to open this pull against upstream, sorry! However, if you're interested in making the Map/Set implementations a bit more complex in order to make them faster, I believe this does the trick. I'd be happy to make changes to get this code merged.

The map implementation of the polyfill is linear because I couldn't think of a better way to support object keys. I have no problem with internal complexity. I will take a closer look at this implementation, it sounds promising, thank you for opening this PR accidentally.


Cannot return the affected rows count after updating

Environment
My Environment :
DBMS: ClickHouse (ver. 20.10.54441)
Case sensitivity: plain=exact, delimited=exact
Driver: com.github.housepower.clickhouse.native.jdbc (ver. 54380, JDBC1.1)
Effective version: ClickHouse (ver. <IP_ADDRESS>)
Using this Driver ,in any updating operation ,I cannot see the message of affected rows count.
Is that the Driver's Setting or the function which not yet implemented ？

I think this is the reason of the problem

So Your SQL is not kind of insert SQL?
What's your update SQL?

That is my SQL for updata
ALTER TABLE data_page_product_info UPDATE product_id=200 WHERE data_page_id=999;

That is my SQL for updata
ALTER TABLE data_page_product_info UPDATE product_id=200 WHERE data_page_id=999;

You should know that clickhouse's alter update is async update, you never know when it is finished.
See: https://clickhouse.com/docs/en/sql-reference/statements/alter/update/


Text and other elements removed from container

Fix to https://github.com/jakezatecky/d3-funnel/issues/36

Thanks for the pull request and tests!


Update player.rb

lib/nba_stats/player.rb, line 45 [r1] (raw file):
it should be foo

Comments from the review on Reviewable.io


native Keras dependency removed

Description
Fixes one Keras dependency in tensorflow.addons.optimizers, uses tensorflow.keras instead.
Type of change

Bug fix

Checklist:

I've properly formatted my code according to the guidelines

How Has This Been Tested?
The imports running fine without keras module installed.

@hyang0129
You are owner of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.


Release azure-cli-vm 0.1.2rc1

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Release of 'azure-cli-vm' with version '0.1.2rc1' successful. View at https://pypi.python.org/pypi/azure-cli-vm/0.1.2rc1.

GitHub release created. https://github.com/Azure/azure-cli/releases.


Error when extracting date cell from Dataframe

Brief Description
When trying to extract one cell using iloc, dovpanda runs into an error. Sometimes, despite the error, this suggestion is made:
The shape of the returned series from slicing is (1,) Which suggests you are interested in the value and not in a new series. Try instead: series.iat[row, col]

System Information

OS: Ubuntu 18.04
Python: 3.7.5
Jupyter Lab: 1.14

Minimally Reproducible Code

df = pd.DataFrame([pd.Timestamp('01-01-2019')])
df.iloc[0][0]

Error Messages

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-25-dc9af601df62> in <module>
      1 df = pd.DataFrame([pd.Timestamp('01-01-2019')])
----> 2 df.iloc[0][0]

~/anaconda3/envs/x/lib/python3.7/site-packages/dovpanda/base.py in run(*args, **kwargs)
    161                 for post in posts:
    162                     if self.similar <= post.stop_nudge:
--> 163                         post.replacement(ret, arguments)
    164             return ret
    165

~/anaconda3/envs/x/lib/python3.7/site-packages/dovpanda/core.py in suggest_at_iat(res, arguments)
    186 def suggest_at_iat(res, arguments):
    187     self = arguments.get('self')
--> 188     shp = res.shape
    189     if res.ndim < 1:  # Sometimes specific slicing will return value
    190         return

AttributeError: 'Timestamp' object has no attribute 'shape'

Thanks!
A similar error as been issues in #77, will be closed by #79 and released on next version.

some behind the scenes: The dovpanda hint jumps on the df.iloc[0] part, while the error is raised by the second [0]. A workaround until next release: try df.iat[0,0]


add link to example of tile.wrap method behavior (closes #30)

I created a bl.ocks.org example demonstrating the tile.wrap method behavior and added links from the images in README.md.

Looks good to me! Thank you.


WIP: Fix #200

#200 is important to me so I started working on a fix for it (it appears to not happen in windows, so I just need to make the X11 behavior more consistent with the correct behavior on Windows), but I ran into a road block that I described here. Is there any chance that I could get some help here? Also, I know that the current code on this branch is a mess -- don't worry, I'll clean it up quite a bit once it's working.

Hey,
Thanks for the PR. What exactly do you need help with? I haven't done much in this area so I'm not sure of how much assistance I can be.

Hey,
Thanks for the PR. What exactly do you need help with? I haven't done much in this area so I'm not sure of how much assistance I can be.

I need help figuring out why XCreateIC is returning null and fixing it.

I need help figuring out why XCreateIC is returning null and fixing it. (I can probably fix it myself once I know why it's happening)

I have never used this API so I'm not sure what could be wrong here

@emoon It's all ready to merge now. With regard to the bug I asked for help on, it turned out that I just made a small error in the C FFI for XCreateIC.

Thanks!


Concurrent indexing fails when indexing both factory and instance events

In some case, it might be useful.
For example, Biconomy Factory initializes new instances with the initial module value. Over time, an instance will emit EnableModule and DisableModule events, changing the module list. In that scenario, it is crucial to get the factory event first and then the instance events.
When trying to setup the indexing logic like that in Ponder, we eventually run into a race condition.
To reproduce, run the indexing for the following Ponder graph: https://github.com/Destiner/ponder-biconomy (might require a few retries).
Eventually, you will run into a race condition:
10:27:29 PM INFO  indexing   Indexed 10 BiconomyAccountFactory 2:AccountCreation events (chainId=80001 block=41619833 logIndex=5)
10:27:30 PM INFO  historical Sync is 89.4% complete with ~48s remaining (contract=BiconomyAccount2)
10:27:30 PM INFO  historical Sync is 89.4% complete with ~48s remaining (contract=BiconomyAccountFactory 2)
10:27:31 PM INFO  indexing   Indexed 1 BiconomyAccountFactory 2:AccountCreation event (chainId=80001 block=41620003 logIndex=1)
10:27:31 PM INFO  indexing   Indexed 13 BiconomyAccountFactory 2:AccountCreation events (chainId=80001 block=41622528 logIndex=12)
10:27:32 PM INFO  indexing   Indexed 15 BiconomyAccountFactory 2:AccountCreation events (chainId=80001 block=41628643 logIndex=11)
Account not found
10:27:32 PM WARN  indexing   Indexing function failed, retrying... (event=BiconomyAccount2:EnabledModule, block=41628643, error=TypeError: Cannot read properties of null (reading 'modules'))

Here's how the issue looks when you run into it locally:
22:35:33 INFO  indexing   Indexed 10 BiconomyAccountFactory 2:AccountCreation events (chainId=80001 block=40457685 logIndex=96)
Account not found
22:35:33 WARN  indexing   Indexing function failed, retrying... (event=BiconomyAccount2:EnabledModule, block=40452969, error=TypeError: Cannot read properties of null (reading 'modules'))
Account not found
22:35:33 WARN  indexing   Indexing function failed, retrying... (event=BiconomyAccount2:EnabledModule, block=40452969, error=TypeError: Cannot read properties of null (reading 'modules'))
Account not found
22:35:33 WARN  indexing   Indexing function failed, retrying... (event=BiconomyAccount2:EnabledModule, block=40452969, error=TypeError: Cannot read properties of null (reading 'modules'))
Account not found
22:35:33 ERROR indexing   Error while processing "BiconomyAccount2:EnabledModule" event at block 40452969:
TypeError: Cannot read properties of null (reading 'modules')
    at /Users/destiner/code/scope/ponder-biconomy/src/biconomyAccount2.ts:16:11
  14 |   await context.db.Account.update({
  15 |     id: getId(event.log.address, context.network.chainId),
> 16 |     data: {
     |           ^
  17 |       modules: [...account!.modules, module],
  18 |     },
  19 |   });
Event args:
  module:  0x000002FbFfedd9B33F4E7156F2DE8D48945E7489
22:35:42 INFO  historical Sync is 88.3% complete with ~46s remaining (contract=BiconomyAccount2)

Hi, thanks for opening. I was able to reproduce this on 0.2.10, but it looks like we fixed the bug that caused this in 0.2.11 yesterday. Not able to reproduce the error on 0.2.13 just now after many attempts.
Let me know if you are still seeing the issue on 0.2.13.

Tested on 0.2.14 and couldn't reproduce as well.


dwm.exe high RAM usage

Microsoft PowerToys version
current
Installation method
GitHub
Running as admin
No
Area(s) with issue?
FancyZones
Steps to reproduce
Install PowerToys. Work some time with FancyZones on your widescreen monitor. At some point your PC is getting slow. Take a look to the ressource monitor and you see dwm.exe needs more than 4GB of RAM. After deinstalling PowerToys, everything is ok again.
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

/bugreport

Actually I have uninstalled the tool and can't provide that information. But I want to add that I now use a tool from Samsung called "Easy Settings Box". I am not sure how it will end, but I see a increase of RAM usage in dwm.exe every time I place a Window in the Grid. Actually it is using 600MB (normally 150MB) and it is still growing.

High RAM usage of dmw.exe is usually evidence of an issue with the video driver. Try going to your device manufacturer's website and downloading the latest driver and see if that helps. /needinfo

Everything is up to date at my notebook. But I have to say, that I also have installed some other software before I see my notebook is getting slow. But I see that dwm.exe is not growing without an Gridmanager.

Is "Report Bug" save, or does it leak some privat information?

You can generate the files from the Bug Report and review them before submitting the .zip here.


List fields break Table equality

Version: Parquet.Net v.3.1.4
Runtime Version: .Net Framework v4.7
OS: Windows
Expected behavior
Table.Equals() respects value semantics, and tables with equal content therefore return true.
Actual behavior
List fields seem always fail comparison. I'm guessing this is something to do with them being represented using arrays?
Code snippet reproducing the behavior
  [<Fact>]
  let ``Lists are comparable``() =
    let sch = Schema(ListField("ints", DataField("int", DataType.Int32, hasNulls=false)))

    let tbl1 = Table(sch)
    tbl1.Add(Row([| 1; 1; 1 |]))

    let tbl2 = Table(sch)
    tbl2.Add(Row([| 1; 1; 1 |]))

    Assert.True(tbl1.Equals(tbl2, true))

Exception thrown is as follows:
Message: System.ArgumentException : tables are different at row 0. this: {'ints': [1, 1, 1]}, other: {'ints': [1, 1, 1]}
---- System.ArgumentException : values are not equal at position 0 (System.Int32[] != System.Int32[])

I think this will still fail with nested arrays? The EqualTo helper should check if il and ir are arrays and if so recurse into itself if so instead of calling .Equals().


(do not merge) Test appveyor

This branch has rodjek's changes plus some appveyor config, so we should see appveyor results against his changes

Changes Unknown when pulling 382934a01418378c355aedcdec08bebecff33ed4 on james-stocks:test_windows into ** on rodjek:master**.

Closing - will open a PR against a clean branch now that @rodjek's changes are complete


stack.c: Add Stack using C

A stack is an abstract data type that serves as a collection of elements, with two principal operations:

push, which adds an element to the collection, and
pop, which removes the most recently added element that was not yet removed.

Closes: #10

Thanks for contributing @Anish-Jay


Build command (Npm install) question during deployment - where is this setting?

I should have set my deployment to run build commands. How do I manually set this so the next deployment includes the npm install command?

Is it .deployment with
[config]
SCM_DO_BUILD_DURING_DEPLOYMENT=true

Hi @diberry
That's correct.  When a user clicks "Yes" at this prompt, we generate the .deployment file for them that enables the app setting SCM_DO_BUILD_DURING_DEPLOYMENT to true, which enables the npm install command to run on the target server.
If you want the prompt to come back, you can change this setting in VS Code.

In the issue that I linked, we have discussed configuring the app setting by default without prompting the user as this is a very crucial step to ensuring that the app deploys successfully.

Will be fixed by https://github.com/microsoft/vscode-azureappservice/pull/2024

#2024 fixes this issue for newly created web apps. SCM_DO_BUILD_DURING_DEPLOY is set to 'true' by default.
For existing ones, you can now add an App Setting "SCM_DO_BUILD_DURING_DEPLOY" and set this value to 'true'.


Next.js + vanilla-extract Fast Refresh loop

Describe the bug
When I'm using vanilla-extract with Next.js, and I'm editing an x.css.ts file it happens periodically that Fast Refresh goes on a rampage and refreshes constantly until I kill it. Is this happening to anyone else?
This only happens on my vanilla-extract Next.js projects.
Happens to me on an M1 iMac and an Intel Macbook Pro.
Link to reproduction
No link to reproduction, but here's a video. This seems to happen once every 100 fast refreshes or something.
https://user-images.githubusercontent.com/8494120/162764795-a2ab25cb-f9aa-4304-b9bb-b7b81cf2f9e2.mov
System Info
Output of npx envinfo --system --npmPackages @vanilla-extract/css,@vanilla-extract/webpack-plugin,@vanilla-extract/esbuild-plugin,@vanilla-extract/vite-plugin,@vanilla-extract/sprinkles,webpack,esbuild,vite --binaries --browsers:
  System:
    OS: macOS 12.2.1
    CPU: (8) arm64 Apple M1
    Memory: 79.20 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.14.2 - ~/.nvm/versions/node/v16.14.2/bin/node
    Yarn: 1.22.18 - /opt/homebrew/bin/yarn
    npm: 8.5.0 - ~/.nvm/versions/node/v16.14.2/bin/npm
  Browsers:
    Chrome: 100.0.4896.75
    Safari: 15.3

20 days without a response means that this issue is most likely on my end. I'll close this and investigate locally.


Autocomplete: Error in nextjs

To Reproduce
Steps to reproduce the behavior:

Checkout https://github.com/pksjce/primer-nextjs
Run npm run dev
Goto http://localhost:3000/form
See this error

Expected behavior
No error should be seen
Additional context
nextjs is running with SSR enabled
cc - @mperrotti

Adding themeprovider fixed this. Though, should it give an error saying themeprovider not present instead of breaking willy-nilly?


Add MissionBoardState

Note: this contains the changes in https://github.com/PMDCollab/PMDC/pull/37.


Adding User Authentication with OpenID Connect

Issue / Steps to reproduce the problem
I have been following the steps from the documentation for adding user authentication with OpenID Connect. This link: [https://identityserver4.readthedocs.io/en/release/quickstarts/3_interactive_login.html].
So the setup is, on one hand, the standard Asp.Net Core MVC Web App acting as the host for  IdentityServer4. The app is the standard template with Authentication type set to use Individual User Accounts which sets up the Asp.Net Identity for user management.
On the other hand there is another MVC web app,acting as a client, configured to use the implicit grant types.
All the configuration is just copied and pasted from the above link.
The problem I am encountering which I can't get passed is the following scenario:
start up both client app and IdentityServer4 host app.
The HomeController Index action on the client app is marked with the Authorize attribute, so on accessing the homepage the client redirects to IdentityServer4 app to authenticate the user and setup the login cookie. Up to this point everything works great, including the login of the user on the identity server instance. The problem is that after the user is logged on he is not redirected back to the client app, instead he is redirected to the the login page on identity server instance, with the following return URL:
http://localhost:5000/account/login?returnUrl=%2Fconnect%2Fauthorize%2Fcallback%3Fclient_id%3Dmvc%26redirect_uri%3Dhttp%253A%252F%252Flocalhost%253A5002%252Fsignin-oidc%26response_type%3Did_token%26scope%3Dopenid%2520profile%26response_mode%3Dform_post%26nonce%3D636593250073027688.YTU1MGQ4N2QtYmE4Mi00ZGRhLWE2ZWMtMmZlMGI5YTFkMjVkNzU1NDJhMGQtNTI2Mi00MmQwLTg4NjMtMjA3M2E3YjkzZmEx%26state%3DCfDJ8Cji98DpbOVNgLHtSuu73e7zJapnDOV-07ayG5XosX1Btfjud2vJXCTULMCILjfStfvUG8TDOK7fJMSfN8hVQSG0XCpqtPlJPSxaKogj4tOnPQEpn4b41pSql4ZzNiu3iBfI3g6_nPa0ERgyhLQdcBuQipHsMndvPWS9Tn2-XUM7vRBtckuyhfAt2fRwrJYHwd6q1sVlGKnzT_PQuaYui_YE11W0yQTTwhVuF0GGxIk32SG8XmeptHqYwxDUG3mwxB--xfn_3m1NCV0gP2j6l9yQO4vwrcZxdBqiKtbfIsfQ7nn5cImowqogUC9gMmy4RA%26x-client-SKU%3DID_NET%26x-client-ver%3D<IP_ADDRESS>
Other changes I've made were to replace the login methods from the AccountController created by the Visual Studio template, with the login methods from the QuickstartUI AccountController, since I want to keep the Registration logic it provides.
Also marked the login methods with the AllowAnnonymous attribute, since this caused an infinite loop.
I can provide code sample if needed. If anyone can help me with this issue I would be very grateful.
Thank you

@hflorin How did you fix this?

Facing same issue....
@hflorin  Please mention the fix.


As audioDepacketizer is embedded to OpusPacket, OpusPacket implements IsPartitionHead/IsPartitionTail.
It's correct that IsPartitionHead/IsPartitionTail always return true since Opus RTP packet doesn't allow fragmentation.

@at-wat Thank you very much for your enthusiastic answer
When I used the following example, I found that the interface was not fully implemented.
https://github.com/pion/example-webrtc-applications/blob/f939bc71df2ae20b23049475fc8277deafe794aa/rtmp-to-webrtc/main.go#L92
Thank you very much
Let me know that it is correct to always return true
In addition, for this example, I tried to use mkfifo to create two pipe files to receive the output. I don’t know if there is any problem.
https://github.com/pion/example-webrtc-applications/tree/master/rtmp-to-webrtc


Configuration does not persist

The configuration does not persist because the volume is incorrectly indicated in the Dockerfile file, where /kafka-manager/configuration is indicated, but the correct path is /kafka-manager/conf , see in https://github.com/yahoo/kafka-manager/tree/master/conf

Hi @javieryanez, fixed for latest tag. See https://hub.docker.com/r/hlebalbau/kafka-manager/tags/.
Please, test and close issue if its okey.
Thanks for issue ;)

Well, the title of the problem isn't really correct. The correct title should be'the path of volume is wrong'. To persist, it is necessary to use named volumes. This does not detract from the fact that the correction is good, it avoids creating a useless volume.
Thanks for the quick fix!


Updating table for docs

Signed-off-by: Anshumaan<EMAIL_ADDRESS>Description
Fixing docs table, scheduled correct view in editor, but contorting in Docs overview.
This PR fixes #2908
Notes for Reviewers
This's definitely beginner-friendly issue, but litte complex for me as I can't able to schedule correctly in my editor. So, pl excuse me for that!
Signed commits

[x] Yes, I signed my commits.

@amino19 LGTM 🎉
Also, could you update the branch?


Upgrade failed in VIC if replication policies exist

This is a blocker for vic appliance upgrade.
After the fix is applied, user needs to update the trigger from manual to immediate, such that when an image is pushed the new image will be replicated.
A workaround is to disable all replication policies before upgrade/migration.

Fixed.


Whitelist URLs to not intercept?

I have set up fakerest with
fetchMock.mock('^http://fakeapi', restServer.getHandler())
Now when I want to test my auth api, I have a request going out fetch(request) with request being:
Request {method: "POST", url: "http://localhost:4040/v1/auth/login", ...
Unfortunately the request does not go out but gets intercepted and results in the error:
Uncaught Error: No fallback response defined for GET to [object Request]
    at FetchMock.webpackJsonp../node_modules/fetch-mock/es5/fetch-mock.js.FetchMock.fetchMock (fetch-mock.js:104)
    at eval (eval at webpackJsonp../src/components/menu/authentication/authClient.js.__webpack_exports__.a (authClient.js:13), <anonymous>:1:1)
    at webpackJsonp../src/components/menu/authentication/authClient.js.__webpack_exports__.a (authClient.js:13)
    at runCallEffect (proc.js:503)
    at runEffect (proc.js:425)
    at next (proc.js:306)
    at currCb (proc.js:378)
    at proc.js:489
    at exec (scheduler.js:19)
    at flush (scheduler.js:60)

Is there a way to let requests to a specific host through?

What if you loose the ^?

@fzaninotto if I change it to fetchMock.mock('^http://fakeapi', restServer.getHandler()) all requests (even those ment for fakeapi throw that Uncaught Error above.

And how did you initialize your restServer?

@fzaninotto
const restServer = new FakeRest.FetchServer('http://fakeapi');

Then I suspect a bug in the fetch-mock package, which is a dependency...

Adding a .spy():
fetchMock.mock('^http://fakeapi', restServer.getHandler()).spy() did the trick. This lets other URLs through.


Small percentage fix

Small UI fix.
Tried to do this on master but got some error stating that "at least 2 approving reviews are required".

Small UI fix. Tried to do this on master but got some error stating that "at least 2 approving reviews are required".

Yeah, i have applied some rules. I think is a bad practice to push to master directly. So we should both create PR's before pushing to master. This will avoid some mistakes probably.

Alright, that's fair.


Break static-pages.yaml up into multiple files

How is this something that has still not happened yet what.
(04 Feb 2022)

(Not doing this til after data-processing is merged because it'll be a very easy change then and we're meanwhile still making additions to static-pages.txt, i.e. the changelog mostly)
(04 Feb 2022)

This was done in cb3edb0b2bdea87cd29559e4a9df2e7c062a4df7!


(PR-528) working with kali-hernandez

Rebased against master
Update docs
Add test for new functionality

Released in v2.25.0


React 17 dependency issue with react-popper

Following the instructions on the README to install the latest version using npx create-react-app.
Reaching this line there's a dependency issue npm i reactstrap react react-dom:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR!   peer react@"*" from<EMAIL_ADDRESS>npm ERR!   node_modules/@testing-library/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/react-dom
npm ERR!     peer react-dom@"*" from<EMAIL_ADDRESS>npm ERR!     node_modules/@testing-library/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     1 more (reactstrap)
npm ERR!   2 more (the root project, reactstrap)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^15.0.0 || ^16.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/reactstrap/node_modules/react-popper
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/reactstrap
npm ERR!     reactstrap@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Can we just bump the version of react-popper to a version that supports React 17?

Unfortunately we won't be migrating to react-popper v2 anytime soon.  We're currently on the latest version of 1.x and I think that's been deprecated for some time.

I am facing this issue as well on<EMAIL_ADDRESS>Is there any plan to update the version of react-popper to a version that is compatible with React 17? I was looking through all the branches and even the bootstrap 5 branch does not seem to bump up the version.
I was also looking at the various comments in this thread - https://github.com/reactstrap/reactstrap/issues/1748 and its unclear whether reactstrap is going to support bootstrap 5 or is react-bootstrap the path forward?

Hi @gauravshah27, we have a Bootstrap v5 PR pending from a fork that includes Popper v2 update.
Perhaps some of this Popper update can be backported to v4?  https://github.com/gthomas-appfolio/reactstrap/pull/31/files
Note this above includes changes needed for Bootstrap 5 too - not everything is needed to update reactstrap v4

Thanks @bestguy for the reply. It would be really helpful if we can have a version 4.x with this update. Right now the move to npm 7 for our React 17 based apps is not possible because of how npm 7 treats the peer dependencies. Is there any work planned to roll out a v4 with this change?

I'm currently working with the Popper team to release a 1.x version with react 17 support.
Meanwhile you could use the --legacy-peer-deps flag in npm to ignore the new peer deps behavior.

@kyletsang thanks a lot for the prompt reply. Really appreciate the work. I have tested it out already with --legacy-peer-deps and that seems to work fine. But would love to have the 1.x version that you mentioned.

react-popper 1.3.8 has been published with React 17 support

@kyletsang thanks for letting us know and for updating react-popper.

react-popper 1.3.8 has been published with React 17 support

I am not seeing the react 17 support in this version. The peer dependencies still look as follows:
"peerDependencies": {
   "react":"0.14.x || ^15.0.0 || ^16.0.0"
}

I am still seeing the above issue with<EMAIL_ADDRESS>and react@17

react-popper 1.3.8 has been published with React 17 support

I am not seeing the react 17 support in this version. The peer dependencies still look as follows:
"peerDependencies": {
   "react":"0.14.x || ^15.0.0 || ^16.0.0"
}

I am still seeing the above issue with<EMAIL_ADDRESS>and react@17
Also, I am unable to find the tag on github - https://github.com/popperjs/react-popper and the last cut tag is 1.3.6 eventhough I am able to download 1.3.8.
Also this page is giving me 404 - https://github.com/popperjs/react-popper/tree/v1.3.8
this one works fine - https://github.com/popperjs/react-popper/tree/v1.3.6

/cc @FezVrasta
Looks like the react 17 peer deps didn't make it into the release:
<EMAIL_ADDRESS>

🤦♂ I'll take a look tomorrow morning, sorry 😞

1.3.9 should be good

Thanks @FezVrasta!


v0.10.1 - SSR breaks because of React-Popper + No /dist builds in 1.0beta5

The UMD build is broken in v0.10.1 because of:
Manager.propTypes = process.env.NODE_ENV !== "production" ? {
The production build should use plugin that optimizes and removes this code ahead of time. In Webpack it was the Define plugin. Then the remove prop-types plugin will correctly wrap it and leave it as {}.
What's the best way to resolve this?
0.10.1

Rollup. I can find the corresponding plugin to ensure it optimizes it out and submit a PR for an immediate patch fix. Or are you not releasing updates anymore?

1.x.x

What's the situation here? There's currently no /dist. This is a huge problem. Reactstrap utilizes external opt-in dependency for Bootstrap and is a huge community. https://github.com/reactstrap/reactstrap

Ref: #142

@virgofx for v0.10.1 see this issue I've raised with babel-plugin-transform-react-remove-prop-types.
Basically the default mode that is supposed to entirely remove the propTypes from production builds is not working for React stateless functional components and therefore I can't fix this issue. As you've found in this issue process.env.NODE_ENV is not available in the browser unless it is provided by something like the DefinePlugin in Webpack or similar in your own bundling process. This is a requirement that I'm not keen on either, but for now I would suggest that you do one of two things:

Find a way to provide process.env.NODE_ENV in your application (Server side by the sounds of it)
Raise a PR to entirely remove the babel-plugin-transform-react-remove-prop-types plugin's usage from rollup.config.js

I'm not sure about the stateless components; however, the issue with regards to process can be fixed quite easily. I'll submit a PR which fixes this.

@virgofx Try following the replication steps I've posted on that ticket about stateless functional components and you'll understand. The point is that babel-plugin-transform-react-remove-prop-types should work properly, but it doesn't because of a bug with removing propTypes from functional components that ReactPopper uses, which means we can just remove {mode: wrap} from the plugin config on v0.x.

So seems to fix

Continue to use mode: 'wrap' but add the Rollup replace plugin -- which will clean that all up ... or
Just switch to "removeImport": true

Regardless, the Rollup replace plugin (equivalent of Webpack's define) should be used regardless so my thought is to just leave as mode:wrap which is intended for libraries that may be consumed externally.
(Minus any areas with SFC that don't work) ... Regardless, server rendering should be fixed after that, correct?

Fixed in 0.10.2


Is there anyway to display the total table in one page with Scroll?

Hi,
When I was trying to exporting the table to PPT the table is spaning over the slides.
I just want to display the whole table in one slide.
Thanks in advance

Hi @RavalikaP ,
There is currently no option to "fit" a table onto a slide.


Training code

Hello,
Congrats, for the wonderful work. I am facing certain issues while training on the IsoGD database.
This part of the code throwing index out range error.
tf.contrib.layers.l2_regularizer(weight_decay)(networks.all_params[50])
IndexError: list index out of range
I changed the index, but getting very poor accuracy. What could be the reason, could you plz help me in resolving the issue.
Thanking You

I can generate RGB images,but cannot generate flow images.It seems that .brox is not used.Can you help me? THX! @sumeetssaurav

You can obtain the RGB/Depth/Flow images of IsoGD from the link: https://pan.baidu.com/s/12qC5sVhQAPizpsr0axVehw   Password: fymm

@sumeetssaurav This work is done with the old version of Tensorlayer. You can refer to the new version using keras in TF-1.2: https://github.com/GuangmingZhu/ContinuousGR and https://github.com/GuangmingZhu/AttentioninConvLSTM.


Crash in Carto Mobile when local reference table overflow (max=512)

I use Carto Mobile in my project. Carto version is: 4.3.0-rc.3@aar . I When adding 7 or more layers to the map, there is a problem and the program crashes:

E/dalvikvm: Failed adding to JNI local ref table (has 512 entries)

E/dalvikvm: VM aborting

A/libc: Fatal signal 11 (SIGSEGV) at 0xdeadd00d (code=1), thread 1528 (aa.ir.mobilegis) ```

thank you help me to solve this problem

Hello. I need a small test program demonstrating the issue. Just adding 7 vector tile layers or vector layers works as expected.

for Example 7 Layer . I mean Processing be Many ,after i add many Layer , Carto does not Processing Layers and android studio show : local reference table overflow (max=512) .
is there buffer or remove past processing of Carto to dont crash .

@sedghian Sorry, but that is not really helpful. Could you make a small test project that demonstrates this crash? Without this we are unable to help you.

could I call with you in skype and share my project ?
My skype id : sedghian

@sedghian Perhaps you could take our HelloMap sample as a reference and add the relevant code to the Java file (https://github.com/CartoDB/mobile-android-samples/blob/master/HelloMapAndroid/app/src/main/java/com/carto/hellomap/android/HelloMapActivity.java)? That would be helpful. Currently the given log and screenshots do not even point to the SDK as being the reason for the crash.

@mtehver  ok . thanks for your answer.

@sedghian Just pinging about this. I would like to investigate this, but can not do this without any sample. Can you help with this?

Closing this due to inactivity and due to no relevant info available. Without reproducing it locally it is not even clear if this is SDK related at all.